)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"86aec7b021fd014670bbc31e3d1fb45fbe4010b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5f349fbd_e6565e8e","updated":"2026-10-06 00:27:11.000000000","message":"This has been tested and is working.","commit_id":"4118c7e81fc67fc99bbc9e0e30011f687b699786"},{"author":{"_account_id":35125,"name":"Mauricio Harley","email":"mharley@redhat.com","username":"mharley-rh"},"change_message_id":"bd9c894ccedd9143546294ec5816124cf1b66265","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"614fe2dc_1a6d223b","updated":"2026-10-07 14:28:30.000000000","message":"Small observation: This change and 1002399 both modify `rewrap_secret()` in `resources.py`, and neither is based on the other. Whichever merges second will need a rebase to reconcile the two.","commit_id":"2e7905baa7b1b879c20bcc2aae7a5fee51fa563e"},{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"689fbc16eac2970d1736d1a1fee0be14ef4fc446","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b18a7363_47407560","updated":"2026-10-07 22:28:42.000000000","message":"@mharley@redhat.com just a rebase - please re-review, thanks!","commit_id":"d0a90d6f00d4a5b1796b4e42d3b9f8702b2f7814"},{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"75c0100a4eba66dec430f94b6ffb7c6a1de57751","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"914e5a0a_fdc44f33","updated":"2026-10-08 15:59:06.000000000","message":"recheck","commit_id":"d0a90d6f00d4a5b1796b4e42d3b9f8702b2f7814"}],"barbican/plugin/resources.py":[{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"28bc03adc95d59c24b35907c671ff3a687ca18d3","unresolved":true,"context_lines":[{"line_number":563,"context_line":"    task.plugin_name \u003d plugin_name"},{"line_number":564,"context_line":"    task.plugin_meta \u003d meta"},{"line_number":565,"context_line":"    task.reason \u003d reason"},{"line_number":566,"context_line":"    task.retry_at \u003d timeutils.utcnow()"},{"line_number":567,"context_line":"    task.retry_count \u003d 0"},{"line_number":568,"context_line":""},{"line_number":569,"context_line":"    cleanup_repo \u003d repos.get_secret_store_cleanup_tasks_repository()"}],"source_content_type":"text/x-python","patch_set":4,"id":"f84202b1_b3790efc","line":566,"updated":"2026-10-06 11:00:23.000000000","message":"Looks like there are 2 casts this and later by the scheduler.\n\nAI proposes setting initial retry_at to now + secret_store_cleanup_retry_msec so the scheduler is fallback if the first cast is lost, and/or claim the row (FOR UPDATE / status ACTIVE) before calling the plugin.\n\nBut maybe we should drop this and let it be cast one time below?","commit_id":"4118c7e81fc67fc99bbc9e0e30011f687b699786"},{"author":{"_account_id":9914,"name":"Ade Lee","email":"alee@redhat.com","username":"alee"},"change_message_id":"c546971ad631c789f2e56bb5df68fefce8f179e9","unresolved":false,"context_lines":[{"line_number":563,"context_line":"    task.plugin_name \u003d plugin_name"},{"line_number":564,"context_line":"    task.plugin_meta \u003d meta"},{"line_number":565,"context_line":"    task.reason \u003d reason"},{"line_number":566,"context_line":"    task.retry_at \u003d timeutils.utcnow()"},{"line_number":567,"context_line":"    task.retry_count \u003d 0"},{"line_number":568,"context_line":""},{"line_number":569,"context_line":"    cleanup_repo \u003d repos.get_secret_store_cleanup_tasks_repository()"}],"source_content_type":"text/x-python","patch_set":4,"id":"9183c1a1_fc73fa33","line":566,"in_reply_to":"f84202b1_b3790efc","updated":"2026-10-07 03:43:54.000000000","message":"Going to make this even easier and just let the retry worker schedule it.  So we add the entry and the retry-worker sees it and enqueues accordingly.  This will result in a short delay (up to one worker cycle), but thats not a problem here.  This makes the code simpler.","commit_id":"4118c7e81fc67fc99bbc9e0e30011f687b699786"}]}