)]}'
{"ironic/common/image_service.py":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"a9eda0d04b0137d324a741dbe1a27228f257f6d4","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"# Import this opt early so that it is available when registering"},{"line_number":39,"context_line":"# glance_opts below."},{"line_number":40,"context_line":"CONF.import_opt(\u0027my_ip\u0027, \u0027ironic.netconf\u0027)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"def import_versioned_module(version, submodule\u003dNone):"}],"source_content_type":"text/x-python","patch_set":10,"id":"1aa78d24_ece6ecb2","line":40,"updated":"2016-07-05 21:29:36.000000000","message":"L38-40 is no longer needed.","commit_id":"b6644abf5da3bef0e02445c6b51c60bd23ace669"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"5d2ecd9e05580ee952be253589c197dedfbc8a3c","unresolved":false,"context_lines":[{"line_number":32,"context_line":"from ironic.common import keystone"},{"line_number":33,"context_line":"from ironic.common import utils"},{"line_number":34,"context_line":"from ironic.conf import CONF"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"IMAGE_CHUNK_SIZE \u003d 1024 * 1024  # 1mb"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"1aa78d24_16c7f096","line":35,"updated":"2016-07-06 08:58:44.000000000","message":"Apparently the my_ip option still needs to be imported somewhere, see the previous patch. \n\nLogs: http://logs.openstack.org/29/304829/11/check/gate-tempest-dsvm-ironic-agent_ssh/ef90db2/logs/screen-ir-api.txt.gz#_2016-07-06_02_28_43_699","commit_id":"ac28f60cd3f3e3dbb69e507e739940912f72038c"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"c914f4c8a93bc4b89a15671cdc43ad6becfa2603","unresolved":false,"context_lines":[{"line_number":32,"context_line":"from ironic.common import keystone"},{"line_number":33,"context_line":"from ironic.common import utils"},{"line_number":34,"context_line":"from ironic.conf import CONF"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"IMAGE_CHUNK_SIZE \u003d 1024 * 1024  # 1mb"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"1aa78d24_d6972838","line":35,"in_reply_to":"1aa78d24_16c7f096","updated":"2016-07-06 09:02:23.000000000","message":"starting from centralize default options all works fine: https://review.openstack.org/#/c/309070/13\nSo might be move 309070 to the bottom of the chain?","commit_id":"ac28f60cd3f3e3dbb69e507e739940912f72038c"}]}