)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"ef806b7b3dab151acba8b211fbf9838020a4a989","unresolved":false,"context_lines":[{"line_number":11,"context_line":"fixed it."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: If7b8bf9c9dca72678d89c804d116bd7c0c58ffff"},{"line_number":14,"context_line":"Closes-Bug: #1707319"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"df3967d1_0abec664","line":14,"updated":"2017-07-31 14:41:20.000000000","message":"Like I said in the bug report, I don\u0027t feel it\u0027s really a bug, just a nice feature if you want all your ports to have the same security groups at once (when booting). Nothing really critical here, just an UX improvement.","commit_id":"23d7bbf68ae4166116b53a71f95983b32fc94c44"}],"nova/network/neutronv2/api.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"ef806b7b3dab151acba8b211fbf9838020a4a989","unresolved":false,"context_lines":[{"line_number":876,"context_line":"        security_groups \u003d self._clean_security_groups_for_update("},{"line_number":877,"context_line":"                                                        security_groups)"},{"line_number":878,"context_line":"        security_group_ids \u003d self._process_security_groups("},{"line_number":879,"context_line":"                                    instance, neutron, security_groups)"},{"line_number":880,"context_line":""},{"line_number":881,"context_line":"        # We always need admin_client to build nw_info,"},{"line_number":882,"context_line":"        # we sometimes need it when updating ports"}],"source_content_type":"text/x-python","patch_set":4,"id":"df3967d1_8a03d642","line":879,"updated":"2017-07-31 14:41:20.000000000","message":"I wouldn\u0027t make a second HTTP call to Neutron for getting something I already know IMHO.\n\nSince we already update ports (even for the newly created ports), I would just make sure that if I create a new port, its security groups would be just \u0027default\u0027, so that L889 would apply for both.","commit_id":"23d7bbf68ae4166116b53a71f95983b32fc94c44"}]}