)]}'
{"devstack/lib/ironic":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5aed9baeeef79ff4fdf06eca0a7b06890992aa32","unresolved":false,"context_lines":[{"line_number":1623,"context_line":"    iniset $TEMPEST_CONFIG compute image_ref_alt $image_uuid"},{"line_number":1624,"context_line":""},{"line_number":1625,"context_line":"    iniset $TEMPEST_CONFIG auth create_isolated_networks False"},{"line_number":1626,"context_line":"    # NOTE(jroll) this disables multitenant network tests from tempest\u0027s"},{"line_number":1627,"context_line":"    # tree, but not from our tree. This is a bit inconsistent, we should"},{"line_number":1628,"context_line":"    # fix it."},{"line_number":1629,"context_line":"    iniset $TEMPEST_CONFIG network shared_physical_network True"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3a71b18c_2a7b8b8d","line":1626,"range":{"start_line":1626,"start_character":23,"end_line":1626,"end_character":31},"updated":"2016-12-05 01:21:32.000000000","message":"you mean enable right ?","commit_id":"0da98dee34f2b79fe092661ffc311717db739304"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"aaac05b26b395ae8eef27828cdb6cb0716d7affe","unresolved":false,"context_lines":[{"line_number":1623,"context_line":"    iniset $TEMPEST_CONFIG compute image_ref_alt $image_uuid"},{"line_number":1624,"context_line":""},{"line_number":1625,"context_line":"    iniset $TEMPEST_CONFIG auth create_isolated_networks False"},{"line_number":1626,"context_line":"    # NOTE(jroll) this disables multitenant network tests from tempest\u0027s"},{"line_number":1627,"context_line":"    # tree, but not from our tree. This is a bit inconsistent, we should"},{"line_number":1628,"context_line":"    # fix it."},{"line_number":1629,"context_line":"    iniset $TEMPEST_CONFIG network shared_physical_network True"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3a71b18c_2a962b3f","line":1626,"range":{"start_line":1626,"start_character":23,"end_line":1626,"end_character":31},"in_reply_to":"3a71b18c_2a7b8b8d","updated":"2016-12-05 01:48:17.000000000","message":"got it. disable is right.\n\nGot confused with this which should be changed? - https://review.openstack.org/#/c/355586/22/tempest/scenario/test_network_v6.py","commit_id":"0da98dee34f2b79fe092661ffc311717db739304"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"9b17a670131c0c40afec662da8773e375883f5c8","unresolved":false,"context_lines":[{"line_number":1626,"context_line":"    # NOTE(jroll) this disables multitenant network tests from tempest\u0027s"},{"line_number":1627,"context_line":"    # tree, but not from our tree. This is a bit inconsistent, we should"},{"line_number":1628,"context_line":"    # fix it."},{"line_number":1629,"context_line":"    iniset $TEMPEST_CONFIG network shared_physical_network True"},{"line_number":1630,"context_line":"    iniset $TEMPEST_CONFIG network-feature-enabled port_admin_state_change False"},{"line_number":1631,"context_line":"}"},{"line_number":1632,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"5a74a57a_2312fb09","line":1629,"range":{"start_line":1629,"start_character":35,"end_line":1629,"end_character":58},"updated":"2016-12-01 07:36:16.000000000","message":"I can\u0027t find this option in tempest tree, where it is used?","commit_id":"0da98dee34f2b79fe092661ffc311717db739304"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"8780ef0a0c8f30a4bbd23ae8cf78cd59eef74d70","unresolved":false,"context_lines":[{"line_number":1626,"context_line":"    # NOTE(jroll) this disables multitenant network tests from tempest\u0027s"},{"line_number":1627,"context_line":"    # tree, but not from our tree. This is a bit inconsistent, we should"},{"line_number":1628,"context_line":"    # fix it."},{"line_number":1629,"context_line":"    iniset $TEMPEST_CONFIG network shared_physical_network True"},{"line_number":1630,"context_line":"    iniset $TEMPEST_CONFIG network-feature-enabled port_admin_state_change False"},{"line_number":1631,"context_line":"}"},{"line_number":1632,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3a71b18c_08efec87","line":1629,"range":{"start_line":1629,"start_character":35,"end_line":1629,"end_character":58},"in_reply_to":"3a71b18c_c8dc1415","updated":"2016-12-02 12:13:19.000000000","message":"I see, thanks for clarification. Ironic patch should go first seems.","commit_id":"0da98dee34f2b79fe092661ffc311717db739304"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"a7cc74945b4f2a604b90f4d4ac8605f6cb790418","unresolved":false,"context_lines":[{"line_number":1626,"context_line":"    # NOTE(jroll) this disables multitenant network tests from tempest\u0027s"},{"line_number":1627,"context_line":"    # tree, but not from our tree. This is a bit inconsistent, we should"},{"line_number":1628,"context_line":"    # fix it."},{"line_number":1629,"context_line":"    iniset $TEMPEST_CONFIG network shared_physical_network True"},{"line_number":1630,"context_line":"    iniset $TEMPEST_CONFIG network-feature-enabled port_admin_state_change False"},{"line_number":1631,"context_line":"}"},{"line_number":1632,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3a71b18c_c8dc1415","line":1629,"range":{"start_line":1629,"start_character":35,"end_line":1629,"end_character":58},"in_reply_to":"5a74a57a_2312fb09","updated":"2016-12-02 12:09:02.000000000","message":"It\u0027s added in this patch to replace some of the \"if ironic is enabled\" stuff for network tests: https://review.openstack.org/#/c/355586/","commit_id":"0da98dee34f2b79fe092661ffc311717db739304"}]}