)]}'
{"nova/tests/integrated/test_api_samples.py":[{"author":{"_account_id":4601,"name":"Tiago Mello","email":"tmello@linux.vnet.ibm.com","username":"tmello"},"change_message_id":"02aafdbe1c5099c3176dbd237681d60cd5f91347","unresolved":false,"context_lines":[{"line_number":505,"context_line":""},{"line_number":506,"context_line":"class SecurityGroupsSampleJsonTest(ServersSampleBase):"},{"line_number":507,"context_line":"    extension_name \u003d \"nova.api.openstack.compute.contrib\" + \\"},{"line_number":508,"context_line":"                     \".security_groups.Security_groups\""},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def test_security_group_create(self):"},{"line_number":511,"context_line":"        subs \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2Faos%3D","line":508,"updated":"2012-09-12 21:43:33.000000000","message":"Nit: Someone told me that we should avoid adding + \\ and we can do the same with:\n\nextension_name \u003d (\"nova.api.openstack.compute.contrib\"\n                  \".security_groups.Security_groups\")","commit_id":"258008bbdc237682afdc3e53196e02827d3a387c"},{"author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"change_message_id":"8855f95a9f84357393ab3ddda9e6288552c5a27b","unresolved":false,"context_lines":[{"line_number":505,"context_line":""},{"line_number":506,"context_line":"class SecurityGroupsSampleJsonTest(ServersSampleBase):"},{"line_number":507,"context_line":"    extension_name \u003d \"nova.api.openstack.compute.contrib\" + \\"},{"line_number":508,"context_line":"                     \".security_groups.Security_groups\""},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"    def test_security_group_create(self):"},{"line_number":511,"context_line":"        subs \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2FaN0%3D","line":508,"updated":"2012-09-13 05:59:07.000000000","message":"good point although it looks like the code above forgot to do it too.","commit_id":"258008bbdc237682afdc3e53196e02827d3a387c"},{"author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"change_message_id":"8855f95a9f84357393ab3ddda9e6288552c5a27b","unresolved":false,"context_lines":[{"line_number":527,"context_line":""},{"line_number":528,"context_line":"    def test_security_groups_get(self):"},{"line_number":529,"context_line":"        \"\"\"Get api sample of security groups get request\"\"\""},{"line_number":530,"context_line":"        security_group_id \u003d \u00271\u0027"},{"line_number":531,"context_line":"        response \u003d self._do_get(\u0027os-security-groups/%s\u0027 % security_group_id)"},{"line_number":532,"context_line":"        subs \u003d self._get_regexes()"},{"line_number":533,"context_line":"        return self._verify_response(\u0027security-groups-get-resp\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2FaN4%3D","line":530,"updated":"2012-09-13 05:59:07.000000000","message":"yeah this could be an int, but whatever.","commit_id":"258008bbdc237682afdc3e53196e02827d3a387c"},{"author":{"_account_id":4601,"name":"Tiago Mello","email":"tmello@linux.vnet.ibm.com","username":"tmello"},"change_message_id":"02aafdbe1c5099c3176dbd237681d60cd5f91347","unresolved":false,"context_lines":[{"line_number":528,"context_line":"    def test_security_groups_get(self):"},{"line_number":529,"context_line":"        \"\"\"Get api sample of security groups get request\"\"\""},{"line_number":530,"context_line":"        security_group_id \u003d \u00271\u0027"},{"line_number":531,"context_line":"        response \u003d self._do_get(\u0027os-security-groups/%s\u0027 % security_group_id)"},{"line_number":532,"context_line":"        subs \u003d self._get_regexes()"},{"line_number":533,"context_line":"        return self._verify_response(\u0027security-groups-get-resp\u0027,"},{"line_number":534,"context_line":"                                      subs, response)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2Famg%3D","line":531,"updated":"2012-09-12 21:43:33.000000000","message":"Nit: Couldn\u0027t it be a number instead of a string and use %d?","commit_id":"258008bbdc237682afdc3e53196e02827d3a387c"}]}