)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c09d15c3eb4f2c0fff9849921854b013b941753a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Johnston \u003cnate.johnston@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-09-03 21:57:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for bulk port functioning with security groups"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to ensure that bulk port creation results in the proper"},{"line_number":10,"context_line":"assignment of security groups, add some testing."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7faddb67_1468c7b7","line":7,"updated":"2019-09-04 01:38:57.000000000","message":"This is adding more than tests, and is there a related bug to link?","commit_id":"65e28450c9b7b4ebd0fc872e76cd3c270794db87"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"1ee5bf89e64402bad5adc24bd9f7a1181d23ff28","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Johnston \u003cnate.johnston@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-09-03 21:57:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for bulk port functioning with security groups"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In order to ensure that bulk port creation results in the proper"},{"line_number":10,"context_line":"assignment of security groups, add some testing."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7faddb67_3b05a8c2","line":7,"in_reply_to":"7faddb67_1468c7b7","updated":"2019-09-04 13:41:51.000000000","message":"Done","commit_id":"65e28450c9b7b4ebd0fc872e76cd3c270794db87"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f70cd127fe99dff8bfcd9e3f991a460e39d76bd5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Johnston \u003cnate.johnston@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-09-04 09:39:37 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for bulk port functioning with security groups"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When bulk ports are created with a security group supplied, the"},{"line_number":10,"context_line":"resulting port(s) should only have that security group assigned. But the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7faddb67_feccde34","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":13},"updated":"2019-09-06 18:40:54.000000000","message":"s/Fix","commit_id":"9b0377351f615642e5b631221ea6f74d93749df2"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"6a64b553f43ad3cb41df92fb7365dfeffef5d2ad","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Johnston \u003cnate.johnston@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-09-06 13:27:11 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for bulk port functioning with security groups"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When bulk ports are created with a security group supplied, the"},{"line_number":10,"context_line":"resulting port(s) should only have that security group assigned. But the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"5faad753_59b145e1","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":56},"updated":"2019-09-06 15:25:53.000000000","message":"Wording/commit message nit, but the actual fix sounds more important for first line than adding tests?","commit_id":"f469f84449fadb3141fd9f4151f396828012e7ee"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"e76c24784eb8ed4c56a177bb79c93f00771b77eb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Johnston \u003cnate.johnston@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2019-09-06 13:27:11 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add tests for bulk port functioning with security groups"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When bulk ports are created with a security group supplied, the"},{"line_number":10,"context_line":"resulting port(s) should only have that security group assigned. But the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"5faad753_90bc676a","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":56},"in_reply_to":"5faad753_59b145e1","updated":"2019-09-07 05:19:44.000000000","message":"Done","commit_id":"f469f84449fadb3141fd9f4151f396828012e7ee"}],"neutron/tests/functional/pecan_wsgi/test_controllers.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"f3b52223066a09f46c92971b48d475c634d2a06a","unresolved":false,"context_lines":[{"line_number":507,"context_line":"                    \"name\": \"functest\","},{"line_number":508,"context_line":"                    \"description\": \"Functional test\"}},"},{"line_number":509,"context_line":"                headers\u003d{\u0027X-Project-Id\u0027: \u0027tenid\u0027})"},{"line_number":510,"context_line":"        self.assertEqual(sg_response.status_int, 201)"},{"line_number":511,"context_line":"        sg_json_body \u003d jsonutils.loads(sg_response.body)"},{"line_number":512,"context_line":"        self.assertIn(\u0027security_group\u0027, sg_json_body)"},{"line_number":513,"context_line":"        sg_id \u003d sg_json_body[\u0027security_group\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":9,"id":"5faad753_933b1f9b","line":510,"range":{"start_line":510,"start_character":25,"end_line":510,"end_character":52},"updated":"2019-09-06 13:44:02.000000000","message":"nit: we are using everywhere pattern (expected, actual). But I see that it\u0027s in opposite direction in more places in this file. So maybe we can \"fix\" it in follow-up patch.","commit_id":"f469f84449fadb3141fd9f4151f396828012e7ee"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"d70bc46177d18ad54d465c4b1c7d054cbd262e8b","unresolved":false,"context_lines":[{"line_number":507,"context_line":"                    \"name\": \"functest\","},{"line_number":508,"context_line":"                    \"description\": \"Functional test\"}},"},{"line_number":509,"context_line":"                headers\u003d{\u0027X-Project-Id\u0027: \u0027tenid\u0027})"},{"line_number":510,"context_line":"        self.assertEqual(sg_response.status_int, 201)"},{"line_number":511,"context_line":"        sg_json_body \u003d jsonutils.loads(sg_response.body)"},{"line_number":512,"context_line":"        self.assertIn(\u0027security_group\u0027, sg_json_body)"},{"line_number":513,"context_line":"        sg_id \u003d sg_json_body[\u0027security_group\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":9,"id":"5faad753_904a277e","line":510,"range":{"start_line":510,"start_character":25,"end_line":510,"end_character":52},"in_reply_to":"5faad753_933b1f9b","updated":"2019-09-07 05:22:10.000000000","message":"I at least fixed these.","commit_id":"f469f84449fadb3141fd9f4151f396828012e7ee"}]}