)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"566af5c72691707248df7ee472b009db9648e49e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b4051c1e_002c03f8","updated":"2026-08-26 10:12:41.000000000","message":"I left this comment in [the SDK change](https://review.opendev.org/c/openstack/openstacksdk/+/1002400/comment/10c1c201_3883246c/), but as part of this work I think it would make sense to add an `id` / `uuid` (depending on conventions here) field to the `/v1/secret-stores` API responses. Accepting a `secret_store_id` field in the request for the new `PUT /v1/secrets/{id}/secret-store` doesn\u0027t make sense otherwise, since secret stores are currently \"identified\" by the `secret_store_ref` href field and consumers need to process this field to extract a usable UUID.\n\n-1 just to catch your eye / based on the fact this appears to need CI fixes anyway.","commit_id":"5882370444637924f07ddd4ca913f46bc8cd1eeb"}],"barbican/model/repositories.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3167563ec64959c725c3cb8abdbdcde413ded92e","unresolved":true,"context_lines":[{"line_number":814,"context_line":"        now \u003d timeutils.utcnow()"},{"line_number":815,"context_line":""},{"line_number":816,"context_line":"        for k, v in metadata.items():"},{"line_number":817,"context_line":"            if v is None:"},{"line_number":818,"context_line":"                continue"},{"line_number":819,"context_line":"            meta_model \u003d models.SecretStoreMetadatum(k, v)"},{"line_number":820,"context_line":"            meta_model.updated_at \u003d now"}],"source_content_type":"text/x-python","patch_set":1,"id":"d8bdf48c_bf129d69","line":817,"updated":"2026-08-26 10:14:47.000000000","message":"nit: this feels a like a fix that should be dragged out to a precursor patch (and potentially backported?)","commit_id":"5882370444637924f07ddd4ca913f46bc8cd1eeb"}]}