)]}'
{"nova/api/openstack/compute/contrib/security_groups.py":[{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"644fbfe26a69c0bab5a0611e0d0c0b0b18e5a890","unresolved":false,"context_lines":[{"line_number":486,"context_line":"                for server in servers:"},{"line_number":487,"context_line":"                    groups \u003d sg_instance_bindings.get(server[\u0027id\u0027])"},{"line_number":488,"context_line":"                    if groups:"},{"line_number":489,"context_line":"                        server[key] \u003d groups"},{"line_number":490,"context_line":"            # In this section of code len(servers) \u003d\u003d 1 as you can only POST"},{"line_number":491,"context_line":"            # one server in an API request."},{"line_number":492,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2F01Q%3D","side":"PARENT","line":489,"updated":"2013-05-22 06:55:54.000000000","message":"We have here only last value of groups after loop has finished.","commit_id":"5d40fb635b1abac3828245124e392a4c9af52f60"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"40fd463d371ad2dba2ab8546062050c6e6e50d46","unresolved":false,"context_lines":[{"line_number":486,"context_line":"                for server in servers:"},{"line_number":487,"context_line":"                    groups \u003d sg_instance_bindings.get(server[\u0027id\u0027])"},{"line_number":488,"context_line":"                    if groups:"},{"line_number":489,"context_line":"                        server[key] \u003d groups"},{"line_number":490,"context_line":"            # In this section of code len(servers) \u003d\u003d 1 as you can only POST"},{"line_number":491,"context_line":"            # one server in an API request."},{"line_number":492,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2F00k%3D","side":"PARENT","line":489,"in_reply_to":"AAAAPH%2F%2F01Q%3D","updated":"2013-05-22 06:58:49.000000000","message":"groups is updated on line 487...","commit_id":"5d40fb635b1abac3828245124e392a4c9af52f60"},{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"644fbfe26a69c0bab5a0611e0d0c0b0b18e5a890","unresolved":false,"context_lines":[{"line_number":495,"context_line":"                    for server in servers:"},{"line_number":496,"context_line":"                        groups \u003d sg_instance_bindings.get(server[\u0027id\u0027])"},{"line_number":497,"context_line":"                        if groups:"},{"line_number":498,"context_line":"                            server[key] \u003d groups"},{"line_number":499,"context_line":"            # In this section of code len(servers) \u003d\u003d 1 as you can only POST"},{"line_number":500,"context_line":"            # one server in an API request."},{"line_number":501,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2F01Y%3D","line":498,"updated":"2013-05-22 06:55:54.000000000","message":"We have here only last value of groups after loop has finished. Is there tests for this code?","commit_id":"0604e77654d0c7eb5fda9600e9ca604da656d856"},{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"43e3da6e3d0374d8d641e30707f0790a90ddd9f9","unresolved":false,"context_lines":[{"line_number":495,"context_line":"                    for server in servers:"},{"line_number":496,"context_line":"                        groups \u003d sg_instance_bindings.get(server[\u0027id\u0027])"},{"line_number":497,"context_line":"                        if groups:"},{"line_number":498,"context_line":"                            server[key] \u003d groups"},{"line_number":499,"context_line":"            # In this section of code len(servers) \u003d\u003d 1 as you can only POST"},{"line_number":500,"context_line":"            # one server in an API request."},{"line_number":501,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2F00A%3D","line":498,"in_reply_to":"AAAAPH%2F%2F00o%3D","updated":"2013-05-22 07:05:23.000000000","message":"Oh sorry","commit_id":"0604e77654d0c7eb5fda9600e9ca604da656d856"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"40fd463d371ad2dba2ab8546062050c6e6e50d46","unresolved":false,"context_lines":[{"line_number":495,"context_line":"                    for server in servers:"},{"line_number":496,"context_line":"                        groups \u003d sg_instance_bindings.get(server[\u0027id\u0027])"},{"line_number":497,"context_line":"                        if groups:"},{"line_number":498,"context_line":"                            server[key] \u003d groups"},{"line_number":499,"context_line":"            # In this section of code len(servers) \u003d\u003d 1 as you can only POST"},{"line_number":500,"context_line":"            # one server in an API request."},{"line_number":501,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2F00o%3D","line":498,"in_reply_to":"AAAAPH%2F%2F01Y%3D","updated":"2013-05-22 06:58:49.000000000","message":"groups is updated on line 496 I don\u0027t see the issue.","commit_id":"0604e77654d0c7eb5fda9600e9ca604da656d856"}]}