)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33330,"name":"Karl Kloppenborg","email":"k@rl.ag","username":"karl_kloppenborg"},"change_message_id":"aed77cf98bd43821347a9ac1aa1fa5d9b590432a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c6dc3015_051a9df6","updated":"2023-04-28 04:38:35.000000000","message":"LGMT","commit_id":"484a1c68b47e41bb4fee04259fd89a684201fe74"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"7d76cca0e7a76ac3dcb7c92920e02830145d5ad6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ea4212fb_c4ea7674","in_reply_to":"23973e7d_bba55ef4","updated":"2023-05-01 14:31:43.000000000","message":"Done","commit_id":"484a1c68b47e41bb4fee04259fd89a684201fe74"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"2640e3a6081010b80e2db72cbdc4c7e2b405611e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"23973e7d_bba55ef4","in_reply_to":"c6dc3015_051a9df6","updated":"2023-04-28 12:54:02.000000000","message":"Yesterday I discovered some Python dependency issues with the ceph-mgr modules in the ceph-daemon image, so I marked this WIP until I get that resolved. It looks like https://review.opendev.org/c/openstack/openstack-helm-images/+/881758 resolves it though, so my plan is to get that merged and then update this PS to include a new ceph-daemon image.","commit_id":"484a1c68b47e41bb4fee04259fd89a684201fe74"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"d6198c7f17be0d12473e08b85ea9098a22edc564","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f08fe17e_fdee5684","updated":"2023-05-02 08:27:03.000000000","message":"I am absoulutely for the ceph update but there are to point I would like to highlight\n1) It would be nice to have ceph version as a part of the image tag\n2) It would be better to use the same image tag for all ceph images","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":33330,"name":"Karl Kloppenborg","email":"k@rl.ag","username":"karl_kloppenborg"},"change_message_id":"8df4d8fdb232efc6c7458cb42b9eb0e7731e2ce4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e11e2a02_f65a8c34","updated":"2023-05-03 01:53:06.000000000","message":"Putting this up for discussion on image tagging before we merge this in.\nSee comments from @vladimirK","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":33330,"name":"Karl Kloppenborg","email":"k@rl.ag","username":"karl_kloppenborg"},"change_message_id":"066b9b132b14b604eaeb4842c1a7e3d460d6ed91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"302c448f_aa2b15f6","updated":"2023-05-04 04:28:27.000000000","message":"Yes that\u0027s a fair point @Stephen, I am happy to +1 now.","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"65a1061f3af7a268a5f5b6912d873d7d86218c0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d8d17a81_6b4d6fe3","updated":"2023-05-01 14:31:15.000000000","message":"recheck","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"124858a1ad801c43792d0dc3994c52a458d3cf3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"41d58bf4_033b596c","in_reply_to":"302c448f_aa2b15f6","updated":"2023-05-05 13:54:49.000000000","message":"Vladimir put together https://review.opendev.org/c/openstack/openstack-helm-images/+/882283 to build all of the Ceph images together and include the Ceph version in the image tags.","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":33330,"name":"Karl Kloppenborg","email":"k@rl.ag","username":"karl_kloppenborg"},"change_message_id":"066b9b132b14b604eaeb4842c1a7e3d460d6ed91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3b501fca_1b61a49f","in_reply_to":"322fd396_a8fb8f25","updated":"2023-05-04 04:28:27.000000000","message":"Ack","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":33330,"name":"Karl Kloppenborg","email":"k@rl.ag","username":"karl_kloppenborg"},"change_message_id":"8df4d8fdb232efc6c7458cb42b9eb0e7731e2ce4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4017b849_30ae1c54","in_reply_to":"7c7edde2_11e47ed7","updated":"2023-05-03 01:53:06.000000000","message":"I agree with this, it makes intent more verbose and ceph is one of those things you don\u0027t want to accidentally get confused between versions...","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"5aa8092fb08691d8c4cfd35c9e3e35c6d9bec04d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"322fd396_a8fb8f25","in_reply_to":"e11e2a02_f65a8c34","updated":"2023-05-03 14:06:52.000000000","message":"I agree with the points, but I don\u0027t think they need to prevent this change from merging. The points represent a change in how Ceph images are built and tagged, which hasn\u0027t changed in years and has been considered acceptable for countless Ceph updates in the past. They are great ideas and should be implemented, but as Vladimir and I discussed them, the conclusion was that they need not prevent this update from merging.\n\nThat being said, this update isn\u0027t necessarily critical either. 17.2.6 released fairly soon after the 17.2.5 update, so the thought was simply that we could make a quick update to the latest before 17.2.5 got too settled.","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"a0c7fdeeecf70b069fd9ff9bb3b3658badd843ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7c7edde2_11e47ed7","in_reply_to":"f08fe17e_fdee5684","updated":"2023-05-02 08:48:45.000000000","message":"*...two points...","commit_id":"f5d1d24513a8d067a66c64d4e5723c22c4c3e6aa"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"0e15987519d7fc499c7e03f1ad228c7de1b089a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"22574869_205bef55","updated":"2023-05-09 12:24:51.000000000","message":"recheck","commit_id":"5885d60e7264876e50ebb0e67df457a4ae7b3ac6"},{"author":{"_account_id":33330,"name":"Karl Kloppenborg","email":"k@rl.ag","username":"karl_kloppenborg"},"change_message_id":"199d3dd87ae603f1faa3a68f58e34f76c561bfb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3196b932_d654fe61","updated":"2023-05-10 00:47:49.000000000","message":"recheck","commit_id":"45b492bcf742f872f4926231c9e0b6ac2c4bffc5"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"4f8495e188adefb66c95077a4fae4c62b506bfc4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f0aa2c54_624723e5","updated":"2023-05-09 18:52:14.000000000","message":"recheck","commit_id":"45b492bcf742f872f4926231c9e0b6ac2c4bffc5"}]}