)]}'
{"nova/api/ec2/cloud.py":[{"author":{"_account_id":1812,"name":"p-draigbrady","email":"P@draigBrady.com","username":"p-draigbrady"},"change_message_id":"6e4511ffc62ed94fb3bcb85a06cb68c655ab1193","unresolved":false,"context_lines":[{"line_number":773,"context_line":"                    err\u003derr % (group_name, \"GroupDescription\"))"},{"line_number":774,"context_line":"        else:"},{"line_number":775,"context_line":"            # Amazon accepts more symbols."},{"line_number":776,"context_line":"            # So, allow POSIX [:print:] characters."},{"line_number":777,"context_line":"            if not re.match(r\u0027^[\\x20-\\x7E]+$\u0027, group_name):"},{"line_number":778,"context_line":"                err \u003d _(\"Value (%s) for parameter %s is invalid.\""},{"line_number":779,"context_line":"                        \" Content is limited to characters\""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAHX%2F%2F4xQ%3D","line":776,"updated":"2012-05-23 10:44:28.000000000","message":"A nitpick. There is no mention of ASCII here,\nand many may not be aware that the RE matches the ASCII subset of UTF8.\n\nI would word the comment as:\nSo, allow ASCII [:print:] characters","commit_id":"2f54ce994c7dda3b8b027e448eb9a17d967d1bd7"},{"author":{"_account_id":688,"name":"Sandy Walsh","email":"sandy.walsh@rackspace.com","username":"sandy-walsh"},"change_message_id":"17248081a845b24d8509752af5c6dfea93e1e1c6","unresolved":false,"context_lines":[{"line_number":775,"context_line":"                               \"param\": \"GroupName\"})"},{"line_number":776,"context_line":"            if not re.match(\u0027^[a-zA-Z0-9_\\- ]+$\u0027, group_description):"},{"line_number":777,"context_line":"                raise exception.InvalidParameterValue("},{"line_number":778,"context_line":"                    err\u003derr % {\"value\": group_name,"},{"line_number":779,"context_line":"                               \"param\": \"GroupDescription\"})"},{"line_number":780,"context_line":"        else:"},{"line_number":781,"context_line":"            # Amazon accepts more symbols."}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAHX%2F%2F2%2Fk%3D","line":778,"updated":"2012-05-28 19:18:45.000000000","message":"group_description?","commit_id":"841b7d373ef355251f3b9b918478ed7bce0e2a07"},{"author":{"_account_id":688,"name":"Sandy Walsh","email":"sandy.walsh@rackspace.com","username":"sandy-walsh"},"change_message_id":"17248081a845b24d8509752af5c6dfea93e1e1c6","unresolved":false,"context_lines":[{"line_number":786,"context_line":"                        \" from the [:print:] class.\")"},{"line_number":787,"context_line":"                raise exception.InvalidParameterValue("},{"line_number":788,"context_line":"                    err\u003derr % {\"value\": group_name,"},{"line_number":789,"context_line":"                               \"param\": \"GroupName\"})"},{"line_number":790,"context_line":""},{"line_number":791,"context_line":"        if len(group_name) \u003e 255:"},{"line_number":792,"context_line":"            err \u003d _(\"Value (%s) for parameter GroupName is invalid.\""}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAHX%2F%2F2%2Fg%3D","line":789,"updated":"2012-05-28 19:18:45.000000000","message":"Do we no longer require checking for group description?","commit_id":"841b7d373ef355251f3b9b918478ed7bce0e2a07"}],"nova/tests/test_api.py":[{"author":{"_account_id":688,"name":"Sandy Walsh","email":"sandy.walsh@rackspace.com","username":"sandy-walsh"},"change_message_id":"17248081a845b24d8509752af5c6dfea93e1e1c6","unresolved":false,"context_lines":[{"line_number":358,"context_line":"        test_raise \u003d ["},{"line_number":359,"context_line":"            (True, bad_amazon_ec2, \"test desc\"),"},{"line_number":360,"context_line":"            (True, \"test name\", bad_amazon_ec2),"},{"line_number":361,"context_line":"            (False, bad_strict_ec2, \"test desc\"),"},{"line_number":362,"context_line":"        ]"},{"line_number":363,"context_line":"        for test in test_raise:"},{"line_number":364,"context_line":"            self.expect_http()"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAHX%2F%2F2%2Fc%3D","line":361,"updated":"2012-05-28 19:18:45.000000000","message":"Should there be a test for bad description too?","commit_id":"841b7d373ef355251f3b9b918478ed7bce0e2a07"}]}