)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"667385e12b8ff0174b268af3a75695c09c1d3ef1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f3cd912a_07fcb887","updated":"2026-01-15 20:02:50.000000000","message":"Maybe we can make this a flag of prune-images instead of a brand new command ?","commit_id":"713d0b5c939deb4b73ee080a42bfda4bd84fcd52"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"755b2138aa838e66ed3170bd3efa80a29ccce4ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a4e75fb2_6666bdea","in_reply_to":"1fbf6015_c37b920e","updated":"2026-01-15 20:12:55.000000000","message":"Acknowledged","commit_id":"713d0b5c939deb4b73ee080a42bfda4bd84fcd52"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"6acccbf971915b6372b393f6929bc5b8da64e092","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1412322e_e3f0654b","in_reply_to":"6976e5c8_4de3df47","updated":"2026-01-17 10:46:36.000000000","message":"I don\u0027t think the rmi approach is necessarily more unsafe, since without --force it\u0027ll just.. not work. rmi on used images will error out, otherwise it\u0027ll behave like image prune, so I would argue that both commands are pretty \"safe\" as long as no force flag is passed.\n\nFrom a maintainability perspective I prefer a unified command but as you said, someone from the core team needs to decide which way to go.","commit_id":"713d0b5c939deb4b73ee080a42bfda4bd84fcd52"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"6ecd271d7c37200db407ecac8852e346232d76ae","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6976e5c8_4de3df47","in_reply_to":"9189642d_057fcf6b","updated":"2026-01-17 09:59:26.000000000","message":"I agree that conceptually both operations are about image cleanup.\nHowever, I intentionally introduced a separate command to avoid changing the long-standing semantics of prune-images, which is currently documented and widely understood as dangling-only.\n\nRemoving tagged images via docker rmi is significantly more destructive than a standard prune, so I wanted this to be:\nexplicit in naming,\nopt-in,\nand impossible to trigger accidentally via existing automation.\n\nI’m open to discussing a future unification under a single command if that’s the preferred direction. Ultimately, I think this is a matter of maintainer/reviewer preference — if there is interest in a flag-based approach instead, I’m happy to adapt the implementation to match what the project considers safer and more consistent.","commit_id":"713d0b5c939deb4b73ee080a42bfda4bd84fcd52"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"1625def2e6d404cd53499050295cbce6897d7678","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9189642d_057fcf6b","in_reply_to":"93fb077e_3f5d0cf3","updated":"2026-01-17 09:37:51.000000000","message":"please don\u0027t mark ongoing discussions as resolved - I think this was done by accident?","commit_id":"713d0b5c939deb4b73ee080a42bfda4bd84fcd52"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"c0c66d9babd064feb37a946a46f7ad2d515845d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"93fb077e_3f5d0cf3","in_reply_to":"a4e75fb2_6666bdea","updated":"2026-01-15 21:18:04.000000000","message":"Yes, pruning images is safer, but it essentially does the same thing you\u0027re trying to achieve, hence my suggestion about making it a flag. \n\nkolla image-prune would prune everything, while the --tag(s) flag would only delete a specific tag. it can\u0027t be prune because of the filter limitation of `image prune`, but it\u0027s the same process/intent, ie removing older images.","commit_id":"713d0b5c939deb4b73ee080a42bfda4bd84fcd52"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"057a17b922729407ea6de93902ada86c1c1c2cd1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1fbf6015_c37b920e","in_reply_to":"f3cd912a_07fcb887","updated":"2026-01-15 20:12:19.000000000","message":"Actually, the prune-images command already exists and serves a different purpose. According to the documentation, it is strictly used to remove orphaned (dangling) Docker images on hosts\n\nkolla-ansible prune-images -i INVENTORY is used to prune orphaned Docker images on hosts.\n\nYou can find this definition here: https://github.com/openstack/kolla-ansible/blob/master/doc/source/user/operating-kolla.rst?plain\u003d1#L265","commit_id":"713d0b5c939deb4b73ee080a42bfda4bd84fcd52"}]}