)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"4c91da79dcb7c623ec2a8f40ca311410527848b7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch breaks out the security group logic that converts security"},{"line_number":10,"context_line":"group names to ids in allocate_for_instance to it\u0027s own method in"},{"line_number":11,"context_line":"an attempt to shortten allocate_for_instance and make it more managable."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-bug: #1279941"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAUX%2F%2FIt4%3D","line":11,"updated":"2014-02-16 13:31:17.000000000","message":"shortten \u003d\u003e shorten","commit_id":"25a41b59d56218eea8740680f93aa705931373f2"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"03751e501759c520ce787be76d2c37fbe9a761c6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch breaks out the security group logic that converts security"},{"line_number":10,"context_line":"group names to ids in allocate_for_instance to it\u0027s own method in"},{"line_number":11,"context_line":"an attempt to shortten allocate_for_instance and make it more managable."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-bug: #1279941"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAUX%2F%2BzFg%3D","line":11,"in_reply_to":"AAAAUX%2F%2FIt4%3D","updated":"2014-02-19 00:14:39.000000000","message":"Done","commit_id":"25a41b59d56218eea8740680f93aa705931373f2"}],"nova/network/neutronv2/api.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"7ae2ebbd8e70e404d26a74b734170a11cd997028","unresolved":false,"context_lines":[{"line_number":299,"context_line":"            return network_model.NetworkInfo([])"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        security_groups \u003d kwargs.get(\u0027security_groups\u0027, [])"},{"line_number":302,"context_line":"        security_group_info \u003d []"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"        # TODO(arosen) Should optimize more to do direct query for security"},{"line_number":305,"context_line":"        # group if len(security_groups) \u003d\u003d 1"}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_4e1b531d","line":302,"updated":"2014-08-06 05:15:16.000000000","message":"don\u0027t need this line anymore I think?","commit_id":"ac144921fba9e59adb40d128bce91e597de36459"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"8750cbaab88a298df86f65cf1fbabec7dfd09d0d","unresolved":false,"context_lines":[{"line_number":299,"context_line":"            return network_model.NetworkInfo([])"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        security_groups \u003d kwargs.get(\u0027security_groups\u0027, [])"},{"line_number":302,"context_line":"        security_group_info \u003d []"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"        # TODO(arosen) Should optimize more to do direct query for security"},{"line_number":305,"context_line":"        # group if len(security_groups) \u003d\u003d 1"}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_9b5e8850","line":302,"in_reply_to":"baada198_4e1b531d","updated":"2014-08-06 06:38:45.000000000","message":"Yup!","commit_id":"ac144921fba9e59adb40d128bce91e597de36459"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"c4d9bb81dfda00ac6f452a658692628fe4d354bc","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        # group if len(security_groups) \u003d\u003d 1"},{"line_number":305,"context_line":"        user_security_groups \u003d []"},{"line_number":306,"context_line":"        if len(security_groups):"},{"line_number":307,"context_line":"            search_opts \u003d {\u0027tenant_id\u0027: instance[\u0027project_id\u0027]}"},{"line_number":308,"context_line":"            user_security_groups \u003d neutron.list_security_groups("},{"line_number":309,"context_line":"                **search_opts).get(\u0027security_groups\u0027)"},{"line_number":310,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"baada198_6f672eb9","line":307,"updated":"2014-08-06 08:22:36.000000000","message":"Why not put these 2 lines inside of \u0027gather\u0027 and move the gather call to here (defaulting security_group_info to [] on line 305 above instead of what you have there now)?","commit_id":"d3f16df534bddfb079f1589aaf8d30ff3f3e618d"}]}