)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"7222e17b831046460ab72d9deab60ee3a0042fc6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Somesh Yadav \u003cyadavsms@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-01-17 13:53:46 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Tempest Neutron test cases for create_port, list_ports and update_port are falied"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The test cases are failed for multi node environment because while create a port with \"binding:host_id\""},{"line_number":10,"context_line":"for single node host_id can be get with"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f96bb07_aa7541fc","line":7,"updated":"2018-01-23 06:10:24.000000000","message":"This first line could be updated. Could you take a look at GitCommitMessages[0]? I feel this is good for the bug subject but for the patch.\nAnd \n\n The first line should be limited to 50 characters\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"98233ab21067970dca7ed43fdb0bf8f9d65f071d"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"7222e17b831046460ab72d9deab60ee3a0042fc6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Tempest Neutron test cases for create_port, list_ports and update_port are falied"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The test cases are failed for multi node environment because while create a port with \"binding:host_id\""},{"line_number":10,"context_line":"for single node host_id can be get with"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"        cls.host_id \u003d socket.gethostname()"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f96bb07_ca70cd0b","line":9,"updated":"2018-01-23 06:10:24.000000000","message":"I feel \"multi node environment\" is not accurate. I think you can write more details as you said in PS1 here. And,\n\n these subsequent lines should be wrapped at 72 characters.","commit_id":"98233ab21067970dca7ed43fdb0bf8f9d65f071d"}],"tempest/api/network/admin/test_ports.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"95347305a4a80c4726486d57b8618098e306d219","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    @decorators.idempotent_id(\u00278e8569c1-9ac7-44db-8bc1-f5fb2814f29b\u0027)"},{"line_number":40,"context_line":"    def test_create_port_binding_ext_attr(self):"},{"line_number":41,"context_line":"        post_body \u003d {\"network_id\": self.network[\u0027id\u0027],"},{"line_number":42,"context_line":"                     \"binding:host_id\": self.host_id}"},{"line_number":43,"context_line":"        body \u003d self.admin_ports_client.create_port(**post_body)"},{"line_number":44,"context_line":"        port \u003d body[\u0027port\u0027]"},{"line_number":45,"context_line":"        self.addCleanup(self.admin_ports_client.delete_port, port[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_e4d8ebef","line":42,"range":{"start_line":42,"start_character":22,"end_line":42,"end_character":37},"updated":"2018-01-10 08:24:39.000000000","message":"I wonder if binding:host_id needs to be a valid compute node?","commit_id":"151a744feb5cdd0875272c219309aa7e478d76ab"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"901a3e51da41278e1e4ccf2185be4c18dbccd160","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    @decorators.idempotent_id(\u00278e8569c1-9ac7-44db-8bc1-f5fb2814f29b\u0027)"},{"line_number":40,"context_line":"    def test_create_port_binding_ext_attr(self):"},{"line_number":41,"context_line":"        post_body \u003d {\"network_id\": self.network[\u0027id\u0027],"},{"line_number":42,"context_line":"                     \"binding:host_id\": self.host_id}"},{"line_number":43,"context_line":"        body \u003d self.admin_ports_client.create_port(**post_body)"},{"line_number":44,"context_line":"        port \u003d body[\u0027port\u0027]"},{"line_number":45,"context_line":"        self.addCleanup(self.admin_ports_client.delete_port, port[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f96bb07_4a027d71","line":42,"updated":"2018-01-23 06:06:22.000000000","message":"It sounds reasonable to me. Can you put the details to the commit message? I think it should be useful to understand this patch much more.","commit_id":"151a744feb5cdd0875272c219309aa7e478d76ab"},{"author":{"_account_id":27605,"name":"Somesh Yadav","email":"yadavsms@gmail.com","username":"yadavsms"},"change_message_id":"98b251196447c8815efe55fb281710e8b827e1b5","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    @decorators.idempotent_id(\u00278e8569c1-9ac7-44db-8bc1-f5fb2814f29b\u0027)"},{"line_number":40,"context_line":"    def test_create_port_binding_ext_attr(self):"},{"line_number":41,"context_line":"        post_body \u003d {\"network_id\": self.network[\u0027id\u0027],"},{"line_number":42,"context_line":"                     \"binding:host_id\": self.host_id}"},{"line_number":43,"context_line":"        body \u003d self.admin_ports_client.create_port(**post_body)"},{"line_number":44,"context_line":"        port \u003d body[\u0027port\u0027]"},{"line_number":45,"context_line":"        self.addCleanup(self.admin_ports_client.delete_port, port[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_82954a89","line":42,"range":{"start_line":42,"start_character":22,"end_line":42,"end_character":37},"in_reply_to":"9f91af0f_e4d8ebef","updated":"2018-01-10 13:40:45.000000000","message":"i am using open contrail as neutron plugin. And the compute profile is SRIOV and DPDK.When binding:host_id of a port is not specified it does not bind the port to any given host. here we are specifying \"cls.host_id \u003d socket.hostname()\" hostname of tempest machine. Vrouter will look for config with tempest hostname but not found anything and raise the exception.","commit_id":"151a744feb5cdd0875272c219309aa7e478d76ab"}],"tempest/clients.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"95347305a4a80c4726486d57b8618098e306d219","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.network_versions_client \u003d self.network.NetworkVersionsClient()"},{"line_number":69,"context_line":"        self.service_providers_client \u003d self.network.ServiceProvidersClient()"},{"line_number":70,"context_line":"        self.tags_client \u003d self.network.TagsClient()"},{"line_number":71,"context_line":"        self.network_hypervisor_client \u003d self.compute.HypervisorClient()"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"    def _set_image_clients(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_64313b5a","line":71,"range":{"start_line":71,"start_character":8,"end_line":71,"end_character":72},"updated":"2018-01-10 08:24:39.000000000","message":"this is not needed","commit_id":"151a744feb5cdd0875272c219309aa7e478d76ab"},{"author":{"_account_id":27605,"name":"Somesh Yadav","email":"yadavsms@gmail.com","username":"yadavsms"},"change_message_id":"98b251196447c8815efe55fb281710e8b827e1b5","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.network_versions_client \u003d self.network.NetworkVersionsClient()"},{"line_number":69,"context_line":"        self.service_providers_client \u003d self.network.ServiceProvidersClient()"},{"line_number":70,"context_line":"        self.tags_client \u003d self.network.TagsClient()"},{"line_number":71,"context_line":"        self.network_hypervisor_client \u003d self.compute.HypervisorClient()"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"    def _set_image_clients(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f91af0f_22e55e38","line":71,"range":{"start_line":71,"start_character":8,"end_line":71,"end_character":72},"in_reply_to":"9f91af0f_64313b5a","updated":"2018-01-10 13:40:45.000000000","message":"i will  remove it.","commit_id":"151a744feb5cdd0875272c219309aa7e478d76ab"}]}