)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d64d13db6ef09b886fd2e7cc08a8477d3b8b669e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5c00159a_f6038231","updated":"2026-08-07 05:22:59.000000000","message":"IIUC the problem with this solution is that the labels will only be present for deployments created with 2025.1 or later, they won\u0027t be there for older deployments, see the discussion when the labels were added https://review.opendev.org/c/openstack/kolla-ansible/+/943394\n\nnot sure how to best handle it, maybe fallback to the old code if the filtered volume list returns empty?","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"934ccc003358ccd0feb3229010259d7f563f210b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"715ed828_dd1dc00c","updated":"2026-08-07 05:41:15.000000000","message":"Please reenable container engine migration tests in https://opendev.org/openstack/kolla-ansible/src/commit/f76bd31e6fefc3b830acaca6975c3bc50cd18a64/zuul.d/project.yaml#L15","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":39354,"name":"Dato Simó","display_name":"Dato Simó","email":"asimo@whitestack.com","username":"dato-ws","status":"Software Engineer at Whitestack"},"change_message_id":"d997ebc923dd3af72166adfba047daad59d3154c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e48125f7_d42f8286","in_reply_to":"5c00159a_f6038231","updated":"2026-08-07 18:32:40.000000000","message":"Ah, I see, good point.\n\n\u003e maybe fallback to the old code if the filtered volume list returns empty\n\nI\u0027m fine with that, but we could also do a merge of the volumes obtained both ways. Any advice/preference?\n\n(I\u0027m not sure if it\u0027ll be common to have mixed labelled/non-labelled volumes, that\u0027s why I\u0027m unsure...)\n\nThanks for your comment.","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":39354,"name":"Dato Simó","display_name":"Dato Simó","email":"asimo@whitestack.com","username":"dato-ws","status":"Software Engineer at Whitestack"},"change_message_id":"d997ebc923dd3af72166adfba047daad59d3154c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bdc5ff77_6b6989dd","in_reply_to":"715ed828_dd1dc00c","updated":"2026-08-07 18:32:40.000000000","message":"I\u0027ve reenabled the scenario (and added a note in the commit description). I guess Zuul will reprocess the changeset and report here?","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":39354,"name":"Dato Simó","display_name":"Dato Simó","email":"asimo@whitestack.com","username":"dato-ws","status":"Software Engineer at Whitestack"},"change_message_id":"d96084ca794b2a1a9108d4d328ab86008660b644","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"344652f8_4e771675","in_reply_to":"bdb8a1bf_35ed2e97","updated":"2026-08-27 20:00:41.000000000","message":"(marking as resolved, it was pointed out to me this is what corresponds if my part of the changes is done)","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"c86a72b6c582dec4593e0844ab4e4ec6681fa6db","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f402b305_0990f144","in_reply_to":"bdc5ff77_6b6989dd","updated":"2026-08-11 14:24:27.000000000","message":"it did and the result shows that the issue which causes the jobs to fail is still present. on one hand we cannot expect you to fix this issue now, which is unrelated to your change, on the other hand the result is that these jobs cannot actually verify your change. not sure how to best proceed here","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":39354,"name":"Dato Simó","display_name":"Dato Simó","email":"asimo@whitestack.com","username":"dato-ws","status":"Software Engineer at Whitestack"},"change_message_id":"cd235f737330438b7193dd0e318a9750258abaea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e62696e3_10ef64b4","in_reply_to":"db3af327_88f4431a","updated":"2026-08-27 20:01:30.000000000","message":"Acknowledged","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"c86a72b6c582dec4593e0844ab4e4ec6681fa6db","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f88259b8_6f1c5c40","in_reply_to":"e48125f7_d42f8286","updated":"2026-08-11 14:24:27.000000000","message":"merging sound even better to me. a deployment that has both older and newer  containers running (like because some service was added at a later point in time), might indeed see a mix of both types","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":39354,"name":"Dato Simó","display_name":"Dato Simó","email":"asimo@whitestack.com","username":"dato-ws","status":"Software Engineer at Whitestack"},"change_message_id":"cf91fcc5e8272b16279319d738c89a6273ac12fe","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"db3af327_88f4431a","in_reply_to":"f402b305_0990f144","updated":"2026-08-27 19:52:20.000000000","message":"Right :-/ \n\nI might find time to take a look at the CI failure, not sure if I\u0027ll manage to reproduce it.\n\nCheers.","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"},{"author":{"_account_id":39354,"name":"Dato Simó","display_name":"Dato Simó","email":"asimo@whitestack.com","username":"dato-ws","status":"Software Engineer at Whitestack"},"change_message_id":"cf91fcc5e8272b16279319d738c89a6273ac12fe","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bdb8a1bf_35ed2e97","in_reply_to":"f88259b8_6f1c5c40","updated":"2026-08-27 19:52:20.000000000","message":"\u003e merging sound even better to me\n\nDone in v4, thank you. (The original container inspect logic is preserved, merged with `docker volume ls` output.)","commit_id":"9de20202b2b8a9078579009e6c5a6e6c17aa78fe"}]}