)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"338bd0219266484dd91cfca414e1c3aea8b99193","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This follows a recent discovery about calling this without a context"},{"line_number":10,"context_line":"and potentially exposing private flavors to unauthorized users. This"},{"line_number":11,"context_line":"patch changes the signature to require context (and take it first,"},{"line_number":12,"context_line":"which is more covnentional)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Generated-by: Claude Opus 4.6"},{"line_number":15,"context_line":"Change-Id: I036e7623c20ff1959d7db469c9094d47da681973"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"af143369_ca5c07fb","line":12,"range":{"start_line":12,"start_character":14,"end_line":12,"end_character":26},"updated":"2026-05-07 16:53:54.000000000","message":"nit:conventional","commit_id":"265b8afa5dfda599c70977c09e8b4ecfb4ac9b5c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e763575c717d74ca0ad5c35d8346f4e42446b8de","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This follows a recent discovery about calling this without a context"},{"line_number":10,"context_line":"and potentially exposing private flavors to unauthorized users. This"},{"line_number":11,"context_line":"patch changes the signature to require context (and take it first,"},{"line_number":12,"context_line":"which is more covnentional)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Generated-by: Claude Opus 4.6"},{"line_number":15,"context_line":"Change-Id: I036e7623c20ff1959d7db469c9094d47da681973"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"014e32b2_de921559","line":12,"range":{"start_line":12,"start_character":14,"end_line":12,"end_character":26},"in_reply_to":"af143369_ca5c07fb","updated":"2026-05-07 17:04:42.000000000","message":"Done","commit_id":"265b8afa5dfda599c70977c09e8b4ecfb4ac9b5c"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a626a10df3f672688937b644763562c92b2de331","unresolved":true,"context_lines":[{"line_number":10,"context_line":"and potentially exposing private flavors to unauthorized users. This"},{"line_number":11,"context_line":"patch changes the signature to require context (and take it first,"},{"line_number":12,"context_line":"which is more conventional)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Generated-by: Claude Opus 4.6"},{"line_number":15,"context_line":"Change-Id: I036e7623c20ff1959d7db469c9094d47da681973"},{"line_number":16,"context_line":"Signed-off-by: Dan Smith \u003cdansmith@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"82493bef_d307a031","line":13,"updated":"2026-05-07 17:52:14.000000000","message":"nit: you coudl have related bug but ya you refered to it above so just a nit","commit_id":"b146935c2187c085bb034f0e7f84f51d4a1b1dc4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"338bd0219266484dd91cfca414e1c3aea8b99193","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"31a784d8_3cbc1b28","updated":"2026-05-07 16:53:54.000000000","message":"Two nits neither serious enough to trigger a respin.","commit_id":"265b8afa5dfda599c70977c09e8b4ecfb4ac9b5c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e763575c717d74ca0ad5c35d8346f4e42446b8de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ee6b715e_27445254","updated":"2026-05-07 17:04:42.000000000","message":"Thanks","commit_id":"b146935c2187c085bb034f0e7f84f51d4a1b1dc4"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"1ed5be4a97bfa18d9b8dda9a065492e9447ad8d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"98da26af_beb360b7","updated":"2026-05-07 17:59:15.000000000","message":"fwiw","commit_id":"b146935c2187c085bb034f0e7f84f51d4a1b1dc4"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a626a10df3f672688937b644763562c92b2de331","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c615cceb_d93c16f2","updated":"2026-05-07 17:52:14.000000000","message":"gibi is happy and i agreee that htis resolves the comment they left on the orginal fix so looks good to me","commit_id":"b146935c2187c085bb034f0e7f84f51d4a1b1dc4"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"8a5942018970024909db9ddc292c4ce27d969bf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3923840f_76652683","updated":"2026-05-07 20:21:21.000000000","message":"recheck failed to install ceph","commit_id":"b146935c2187c085bb034f0e7f84f51d4a1b1dc4"}],"nova/api/openstack/compute/flavors.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a626a10df3f672688937b644763562c92b2de331","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        \"\"\"Return data about the given flavor id.\"\"\""},{"line_number":183,"context_line":"        context \u003d req.environ[\u0027nova.context\u0027]"},{"line_number":184,"context_line":"        try:"},{"line_number":185,"context_line":"            flavor \u003d flavors.get_flavor_by_flavor_id(context, id)"},{"line_number":186,"context_line":"        except exception.FlavorNotFound as e:"},{"line_number":187,"context_line":"            raise webob.exc.HTTPNotFound(explanation\u003de.format_message())"},{"line_number":188,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"24a3a8b8_76e289c9","line":185,"updated":"2026-05-07 17:52:14.000000000","message":"+1 for puting context first to alight to our normal convetnion","commit_id":"b146935c2187c085bb034f0e7f84f51d4a1b1dc4"}],"nova/tests/unit/api/openstack/fakes.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"338bd0219266484dd91cfca414e1c3aea8b99193","unresolved":true,"context_lines":[{"line_number":451,"context_line":""},{"line_number":452,"context_line":"    sys_meta \u003d flavors.save_flavor_info("},{"line_number":453,"context_line":"        {}, flavors.get_flavor_by_flavor_id("},{"line_number":454,"context_line":"            context.get_admin_context(), int(flavor_id)))"},{"line_number":455,"context_line":"    sys_meta.update(system_metadata or {})"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    if host is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"60b0b1fb_683ac043","line":454,"updated":"2026-05-07 16:53:54.000000000","message":"nit: we can get a single admin context and use it twice here and at L480","commit_id":"265b8afa5dfda599c70977c09e8b4ecfb4ac9b5c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e763575c717d74ca0ad5c35d8346f4e42446b8de","unresolved":false,"context_lines":[{"line_number":451,"context_line":""},{"line_number":452,"context_line":"    sys_meta \u003d flavors.save_flavor_info("},{"line_number":453,"context_line":"        {}, flavors.get_flavor_by_flavor_id("},{"line_number":454,"context_line":"            context.get_admin_context(), int(flavor_id)))"},{"line_number":455,"context_line":"    sys_meta.update(system_metadata or {})"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":"    if host is not None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"d0ccc0ee_2e67503f","line":454,"in_reply_to":"60b0b1fb_683ac043","updated":"2026-05-07 17:04:42.000000000","message":"Done","commit_id":"265b8afa5dfda599c70977c09e8b4ecfb4ac9b5c"}]}
