)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"b0231fc21e968da6464faaeab8cf8f973d618b4b","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Create image model object with input data"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The image create operation creates an empty model object at the"},{"line_number":10,"context_line":"beggining of the method. If the image schema has a required property"},{"line_number":11,"context_line":"the empty model creation always raises ValueError because some"},{"line_number":12,"context_line":"properties are required but not specified even though the value"},{"line_number":13,"context_line":"in the argument."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"72d8434d_f76b7914","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":9},"updated":"2026-08-03 14:39:25.000000000","message":"beginning*","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"},{"author":{"_account_id":8878,"name":"Masahito Muroi","email":"masahito.muroi@g.softbank.co.jp","username":"masa"},"change_message_id":"b454d0bd99f6172a091b41710f8b96611b6c942a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Create image model object with input data"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The image create operation creates an empty model object at the"},{"line_number":10,"context_line":"beggining of the method. If the image schema has a required property"},{"line_number":11,"context_line":"the empty model creation always raises ValueError because some"},{"line_number":12,"context_line":"properties are required but not specified even though the value"},{"line_number":13,"context_line":"in the argument."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9dd50c6f_97bb9e6c","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":9},"in_reply_to":"72d8434d_f76b7914","updated":"2026-08-05 04:47:56.000000000","message":"Done","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"b0231fc21e968da6464faaeab8cf8f973d618b4b","unresolved":true,"context_lines":[{"line_number":10,"context_line":"beggining of the method. If the image schema has a required property"},{"line_number":11,"context_line":"the empty model creation always raises ValueError because some"},{"line_number":12,"context_line":"properties are required but not specified even though the value"},{"line_number":13,"context_line":"in the argument."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This commit updates the first model object creation to use the"},{"line_number":16,"context_line":"kwarg values to avoid the ValueError in case of the required is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1cac4abe_9b4a93c7","line":13,"range":{"start_line":13,"start_character":3,"end_line":13,"end_character":6},"updated":"2026-08-03 14:39:25.000000000","message":"The end of the sentence does not make much sense.","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"},{"author":{"_account_id":8878,"name":"Masahito Muroi","email":"masahito.muroi@g.softbank.co.jp","username":"masa"},"change_message_id":"b454d0bd99f6172a091b41710f8b96611b6c942a","unresolved":false,"context_lines":[{"line_number":10,"context_line":"beggining of the method. If the image schema has a required property"},{"line_number":11,"context_line":"the empty model creation always raises ValueError because some"},{"line_number":12,"context_line":"properties are required but not specified even though the value"},{"line_number":13,"context_line":"in the argument."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This commit updates the first model object creation to use the"},{"line_number":16,"context_line":"kwarg values to avoid the ValueError in case of the required is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"545643b2_6d087c5f","line":13,"range":{"start_line":13,"start_character":3,"end_line":13,"end_character":6},"in_reply_to":"1cac4abe_9b4a93c7","updated":"2026-08-05 04:47:56.000000000","message":"Thanks. revised the message.","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"b0231fc21e968da6464faaeab8cf8f973d618b4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"971bd830_7fe480a5","updated":"2026-08-03 14:39:25.000000000","message":"Good catch, I got a couple of comments inline, otherwise this looks good to me!","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"},{"author":{"_account_id":8878,"name":"Masahito Muroi","email":"masahito.muroi@g.softbank.co.jp","username":"masa"},"change_message_id":"b454d0bd99f6172a091b41710f8b96611b6c942a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c730ed25_7d970851","updated":"2026-08-05 04:47:56.000000000","message":"Thanks for the quick review. Updated.","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"}],"glanceclient/tests/unit/v2/test_images.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"b0231fc21e968da6464faaeab8cf8f973d618b4b","unresolved":true,"context_lines":[{"line_number":1059,"context_line":"            \"id\": \"id1\","},{"line_number":1060,"context_line":"        }"},{"line_number":1061,"context_line":"        with testtools.ExpectedException(TypeError):"},{"line_number":1062,"context_line":"            controller.create(**properties)"},{"line_number":1063,"context_line":""},{"line_number":1064,"context_line":"    def test_delete_image(self):"},{"line_number":1065,"context_line":"        self.controller.delete(\u002787b634c1-f893-33c9-28a9-e5673c99239a\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ee9adce_e18ff970","line":1062,"range":{"start_line":1062,"start_character":12,"end_line":1062,"end_character":29},"updated":"2026-08-03 14:39:25.000000000","message":"Honestly I would squash these tests and just do:\n\n    # With the required property\n    ...\n    # Without the required property\n    ...","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"},{"author":{"_account_id":8878,"name":"Masahito Muroi","email":"masahito.muroi@g.softbank.co.jp","username":"masa"},"change_message_id":"b454d0bd99f6172a091b41710f8b96611b6c942a","unresolved":false,"context_lines":[{"line_number":1059,"context_line":"            \"id\": \"id1\","},{"line_number":1060,"context_line":"        }"},{"line_number":1061,"context_line":"        with testtools.ExpectedException(TypeError):"},{"line_number":1062,"context_line":"            controller.create(**properties)"},{"line_number":1063,"context_line":""},{"line_number":1064,"context_line":"    def test_delete_image(self):"},{"line_number":1065,"context_line":"        self.controller.delete(\u002787b634c1-f893-33c9-28a9-e5673c99239a\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"192f62be_e15496fd","line":1062,"range":{"start_line":1062,"start_character":12,"end_line":1062,"end_character":29},"in_reply_to":"9ee9adce_e18ff970","updated":"2026-08-05 04:47:56.000000000","message":"Done","commit_id":"df114b6b4de19477e14942b23a2e3ec3bd80dafe"}]}