)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"337f12235643245e1f6eafa053ead8a2c85a623d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0355a1a4_3872a270","updated":"2026-04-06 20:47:07.000000000","message":"Here are a few known differences between this Python patch vs. original AngularJS\n\n1. You will notice Python including filter tabs for Non-Public vs. Public \n   Projects. These tabs are driven by the OwnerFIlter class, part of the Python \n   ImagesTable. I propose we defer this parity work for a follow-up patch (if we \n   decide we need 100% parity in this regard)\n2. This review is not in parity with AngularJS support of its Glance memtadata \n   namespace display. We should defer this to a follow-up patch.\n3. Python shows more fields than Angular - do we need 100% parity in this regard?","commit_id":"1d09db96951ec987e2a99046eb55b88c1d4f8f1d"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"a5ef13039d8a8e2a21497bf120e4d81f81f96de7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"15bf7de3_ecea3a59","in_reply_to":"009bd842_e64a4c54","updated":"2026-04-07 23:52:36.000000000","message":"Sounds good - keeping the extra fields. I agree - leaving this patch as is with chevron support - we can plan other patches for further parity with AngularJS version.","commit_id":"1d09db96951ec987e2a99046eb55b88c1d4f8f1d"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"f25f2b91537f479c71e3d9adb3a75617b33559eb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"009bd842_e64a4c54","in_reply_to":"0355a1a4_3872a270","updated":"2026-04-07 09:41:43.000000000","message":"I think this should be only about the chevrons, any other differences can have their own tasks with separate patches.\n\nMore fields is fine, we just don\u0027t want a regression.","commit_id":"1d09db96951ec987e2a99046eb55b88c1d4f8f1d"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"7dae45b6e736856c1c7611bcee0de4e36523e3c2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"808b862a_4e2f63b2","updated":"2026-04-06 20:58:23.000000000","message":"I just realized this Python version is missing the drop down options for Edit Image, update Metadata, and Deactivate Image. I am investigating getting these added asap.","commit_id":"f60c36e515ba9f74bd732e1dba19a14fb4a1af60"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"a5ef13039d8a8e2a21497bf120e4d81f81f96de7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"70a0dbae_39dac196","in_reply_to":"808b862a_4e2f63b2","updated":"2026-04-07 23:52:36.000000000","message":"current version of Python implementation has the same drop down - \"Launch\" and \"Create Volume\". Since this patch is for chevron changes only, extending this drop down for parity with the AngularJS version can be done with a different patch.","commit_id":"f60c36e515ba9f74bd732e1dba19a14fb4a1af60"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"c7d397204ff7d0a1e7038228e51dbe192fdfe4db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"65bea54a_17f0b871","updated":"2026-04-08 13:19:31.000000000","message":"Looks good, thank you!","commit_id":"143c37fa92852d868ce9836462b35c2d9e9b4b37"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"116accff33017525a700136410d3198d52e7928f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"547bbc88_386948a9","updated":"2026-04-07 15:50:55.000000000","message":"recheck","commit_id":"143c37fa92852d868ce9836462b35c2d9e9b4b37"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"ab6af1ba14a7704123b8403a47a036cbe3d73770","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"79180d84_5b0330e8","updated":"2026-04-07 19:42:23.000000000","message":"recheck","commit_id":"143c37fa92852d868ce9836462b35c2d9e9b4b37"},{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"799c0b992f468c4c5d67fe36f80287fed1247565","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"a9739833_47490617","updated":"2026-04-16 15:18:21.000000000","message":"Tested in on my env, the chevrons work well. I found some discrepancies with the AngularJS version, but I\u0027m ok if we fix them in a follow up patch. I\u0027ll just write them down here for better tracking and to start the discussion:\n- 2 columns for the properties instead of 4\n- grey color for properties names, black is more visible\n- for some images there\u0027s more to just properties. Say, for Cirros we display \"Libvirt Driver Options for Images\" plus \"Random Number Generator Device\" and \"Detail Information\", and we don\u0027t have it in Django version","commit_id":"e7c9e39182dcb74579f227b1e61b30b8fa5b4ceb"}],"openstack_dashboard/dashboards/project/images/views.py":[{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"f25f2b91537f479c71e3d9adb3a75617b33559eb","unresolved":true,"context_lines":[{"line_number":88,"context_line":"                for image in images:"},{"line_number":89,"context_line":"                    image.tenant_name \u003d tenant_dict.get(image.owner)"},{"line_number":90,"context_line":"            except Exception:"},{"line_number":91,"context_line":"                LOG.warning(\u0027Unable to retrieve project names for images.\u0027)"},{"line_number":92,"context_line":"        return images"}],"source_content_type":"text/x-python","patch_set":8,"id":"232865f3_09a52b5b","line":91,"updated":"2026-04-07 09:41:43.000000000","message":"Looks like this api call is not mocked in the following tests:\nFAILED openstack_dashboard/dashboards/project/images/tests.py::ImagesAndSnapshotsTests::test_index\nFAILED openstack_dashboard/dashboards/project/images/tests.py::ImagesAndSnapshotsTests::test_snapshot_actions\nFAILED openstack_dashboard/dashboards/project/instances/tests.py::InstanceTests::test_create_instance_snapshot","commit_id":"f60c36e515ba9f74bd732e1dba19a14fb4a1af60"},{"author":{"_account_id":32432,"name":"Owen McGonagle","email":"omcgonag@redhat.com","username":"omcgonag"},"change_message_id":"a5ef13039d8a8e2a21497bf120e4d81f81f96de7","unresolved":false,"context_lines":[{"line_number":88,"context_line":"                for image in images:"},{"line_number":89,"context_line":"                    image.tenant_name \u003d tenant_dict.get(image.owner)"},{"line_number":90,"context_line":"            except Exception:"},{"line_number":91,"context_line":"                LOG.warning(\u0027Unable to retrieve project names for images.\u0027)"},{"line_number":92,"context_line":"        return images"}],"source_content_type":"text/x-python","patch_set":8,"id":"ad24437d_0ef8e6d4","line":91,"in_reply_to":"232865f3_09a52b5b","updated":"2026-04-07 23:52:36.000000000","message":"missing mocked tests added.","commit_id":"f60c36e515ba9f74bd732e1dba19a14fb4a1af60"}]}