)]}'
{"tempest/scenario/manager.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"72874238c751a7af5503386f29603ce7200ab0b8","unresolved":false,"context_lines":[{"line_number":819,"context_line":"        # with 2 subnets"},{"line_number":820,"context_line":"        p_status \u003d [\u0027ACTIVE\u0027]"},{"line_number":821,"context_line":"        if CONF.service_available.ironic:"},{"line_number":822,"context_line":"            p_status.append(\u0027DOWN\u0027)"},{"line_number":823,"context_line":"        port_map \u003d [(p[\"id\"], fxip[\"ip_address\"])"},{"line_number":824,"context_line":"                    for p in ports"},{"line_number":825,"context_line":"                    for fxip in p[\"fixed_ips\"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"9abb7d3a_d8b13de3","line":822,"range":{"start_line":822,"start_character":12,"end_line":822,"end_character":35},"updated":"2016-05-30 18:06:39.000000000","message":"Better to write the reason why we need to append \"DOWN\" as expected as the NOTE here.","commit_id":"c2e46bda17fa39980eb391cd21078b27ed09eaad"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"d27dd890a80f02043eb2d1de79e367403f957712","unresolved":false,"context_lines":[{"line_number":819,"context_line":"        # with 2 subnets"},{"line_number":820,"context_line":"        p_status \u003d [\u0027ACTIVE\u0027]"},{"line_number":821,"context_line":"        if CONF.service_available.ironic:"},{"line_number":822,"context_line":"            p_status.append(\u0027DOWN\u0027)"},{"line_number":823,"context_line":"        port_map \u003d [(p[\"id\"], fxip[\"ip_address\"])"},{"line_number":824,"context_line":"                    for p in ports"},{"line_number":825,"context_line":"                    for fxip in p[\"fixed_ips\"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"9abb7d3a_01197179","line":822,"range":{"start_line":822,"start_character":12,"end_line":822,"end_character":35},"in_reply_to":"9abb7d3a_d8b13de3","updated":"2016-05-31 11:55:23.000000000","message":"Done","commit_id":"c2e46bda17fa39980eb391cd21078b27ed09eaad"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aef06ffa8f2f81dfd0eb437eb3e5d633dd667b7d","unresolved":false,"context_lines":[{"line_number":820,"context_line":"        p_status \u003d [\u0027ACTIVE\u0027]"},{"line_number":821,"context_line":"        # NOTE(vsaienko) with Ironic instances leave on separate hardware servers"},{"line_number":822,"context_line":"        # Neutron do not bind ports for Ironic instances, as result port remain in"},{"line_number":823,"context_line":"        # DOWN state."},{"line_number":824,"context_line":"        if CONF.service_available.ironic:"},{"line_number":825,"context_line":"            p_status.append(\u0027DOWN\u0027)"},{"line_number":826,"context_line":"        port_map \u003d [(p[\"id\"], fxip[\"ip_address\"])"}],"source_content_type":"text/x-python","patch_set":3,"id":"9abb7d3a_0a0faf88","line":823,"updated":"2016-05-31 18:35:17.000000000","message":"Please check pep8 failure reason:\n\n | pep8 runtests: commands[0] | flake8\n |   /home/jenkins/workspace/gate-tempest-pep8$ /home/jenkins/workspace/gate-tempest-pep8/.tox/pep8/bin/flake8 \n | ./tempest/scenario/manager.py:821:80: E501 line too long (81 \u003e 79 characters)\n |         # NOTE(vsaienko) with Ironic instances leave on separate hardware servers\n |                                                                                ^\n | ./tempest/scenario/manager.py:822:80: E501 line too long (82 \u003e 79 characters)\n |         # Neutron do not bind ports for Ironic instances, as result port remain in\n |                                                                                ^\n | ERROR: InvocationError: \u0027/home/jenkins/workspace/gate-tempest-pep8/.tox/pep8/bin/flake8\u0027\n\nWe need to fix the above lines to make them shorter than 79 chars as the message.","commit_id":"031f1c25b329e67098927188c7084cd0511f00a7"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"f81115e789b5c91e2bc706ba683c68b1cc783a81","unresolved":false,"context_lines":[{"line_number":820,"context_line":"        p_status \u003d [\u0027ACTIVE\u0027]"},{"line_number":821,"context_line":"        # NOTE(vsaienko) with Ironic instances leave on separate hardware servers"},{"line_number":822,"context_line":"        # Neutron do not bind ports for Ironic instances, as result port remain in"},{"line_number":823,"context_line":"        # DOWN state."},{"line_number":824,"context_line":"        if CONF.service_available.ironic:"},{"line_number":825,"context_line":"            p_status.append(\u0027DOWN\u0027)"},{"line_number":826,"context_line":"        port_map \u003d [(p[\"id\"], fxip[\"ip_address\"])"}],"source_content_type":"text/x-python","patch_set":3,"id":"9abb7d3a_ccc2d033","line":823,"in_reply_to":"9abb7d3a_0a0faf88","updated":"2016-06-01 06:59:07.000000000","message":"I did editing through web, and missed line length.","commit_id":"031f1c25b329e67098927188c7084cd0511f00a7"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"83ca60942392f990b1a9a2cbd3a02e3522ff05f2","unresolved":false,"context_lines":[{"line_number":818,"context_line":"        # If the network is dual-stack (IPv4 + IPv6), this port is associated"},{"line_number":819,"context_line":"        # with 2 subnets"},{"line_number":820,"context_line":"        p_status \u003d [\u0027ACTIVE\u0027]"},{"line_number":821,"context_line":"        # NOTE(vsaienko) with Ironic instances leaves on separate hardware"},{"line_number":822,"context_line":"        # servers. Neutron do not bind ports for Ironic instances, as result"},{"line_number":823,"context_line":"        # port remain in DOWN state."},{"line_number":824,"context_line":"        if CONF.service_available.ironic:"},{"line_number":825,"context_line":"            p_status.append(\u0027DOWN\u0027)"},{"line_number":826,"context_line":"        port_map \u003d [(p[\"id\"], fxip[\"ip_address\"])"}],"source_content_type":"text/x-python","patch_set":4,"id":"9abb7d3a_f419f27a","line":823,"range":{"start_line":821,"start_character":8,"end_line":823,"end_character":36},"updated":"2016-06-02 16:53:55.000000000","message":"# NOTE(vsaienko) with Ironic, instances live on separate hardware\n # servers. Neutron does not bind ports for Ironic instances, as a result\n # the port remains in the DOWN state.","commit_id":"095c7ea2b9c83e14d18e1cfbb718eb264b4ac9d1"}]}