)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"648436eaee77d3394e0dfb7cad0b64e641ca7e48","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"57fb05f7_da62d9b2","updated":"2023-09-06 11:23:42.000000000","message":"Can the tempest patch be resurrect and pushed as a proper patch? I didn\u0027t notice this comment (sorry) and ended up writing a simpler version, which I\u0027m going to abandon: https://review.opendev.org/c/openstack/tempest/+/893863","commit_id":"d9eb005024a8c1b402b3c4a2eb51342df64325ba"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"71b5a96ea13254f1e0ed1bd24089a8fdd583b443","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"703692ff_da19a75e","in_reply_to":"57fb05f7_da62d9b2","updated":"2023-09-15 01:54:14.000000000","message":"Ah yeah.. mine is more complicated because I added testing of an attached data volume (previously it only tested an attached boot volume). I used this patch to demonstrate the behavior of the regression and how it was introduced specifically in Wallaby whereas in Victoria retype worked fine.\n\nThe tricky thing here is that updating the test alone is not enough to cover testing of the regression because repro of the conditions requires the presence of two Ceph backends with the retype migrating from one backend to the other. Without two Ceph backends, the tests will pass even with the bug unfixed.\n\nI\u0027ll look into this more and see if there\u0027s a low cost way we could incorporate multibackend into our testing and if so, remove the DNM. Thank you for noticing and for commenting 🙂","commit_id":"d9eb005024a8c1b402b3c4a2eb51342df64325ba"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"d8da81c25d486a8a6d627ba895be7e2fdad34d45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"df9cbdaa_f357d9e8","in_reply_to":"703692ff_da19a75e","updated":"2023-09-15 14:50:26.000000000","message":"Maybe the best place is the devstack plugin inside devstack-plugin-ceph.git. We could always create the second pool, but enable it only if requested (with a new devstack variable, which would also set the proper variable with the cinder backends). That way all the jobs could benefit from this. Maybe we can discuss about this a bit more.","commit_id":"d9eb005024a8c1b402b3c4a2eb51342df64325ba"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"4830210c82c2e3629eced979ecc53fe928a9550e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"d75c85ce_48c98332","in_reply_to":"bccddf89_bb009af2","updated":"2023-09-16 21:48:03.000000000","message":"I made a try at a devstack-plugin-ceph change as a starting point:\n\nhttps://review.opendev.org/c/openstack/devstack-plugin-ceph/+/895533\n\nIt seems to be working based on the latest CI run on PS19 and checking the cinder conf there. That run is using the default CEPHADM_DEPLOY \u003d True.\n\nThe CI run on PS18 was with CEPHADM_DEPLOY \u003d False and also seemed to work. Note that on PS18 the Wallaby and Victoria CI runs are not correct because that was before I realized I would need to propose the devstack-plugin-ceph change to stable/wallaby and stable/victoria for it to work right. The master CI run is correct though.","commit_id":"d9eb005024a8c1b402b3c4a2eb51342df64325ba"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"69587149f1a405dc07f7742e591b0c881a4bfacd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"bccddf89_bb009af2","in_reply_to":"df9cbdaa_f357d9e8","updated":"2023-09-15 19:59:25.000000000","message":"I was thinking the same thing, to split up the setup pieces to the appropriate repos and make it reusable and configurable. I looked around in devstack-plugin-ceph briefly and didn\u0027t see an exceedingly obvious way to add that part ... so I think a bit more thought is required. Definitely happy to discuss it further.","commit_id":"d9eb005024a8c1b402b3c4a2eb51342df64325ba"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a684136bf8033d8319801992e1637a08acf43910","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":20,"id":"1db9cd59_cafebbd7","updated":"2023-09-22 02:22:18.000000000","message":"Note to reviewers: you can see this patch working in the cinder-tempest-ceph-multibackend job run on PS19:\n\nhttps://review.opendev.org/c/openstack/tempest/+/890360/20#message-362f103345618b4430fab824d3fcd08030d38f04\n\nPS19 contains temporary changes that run a job with two Ceph backends, each with their own pool so that the migration happens across the pools.","commit_id":"771872fd736b19193119de2952def670b0aac4cf"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"e7eec5efaa567fc5f474958f0614aad332ae9b14","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":20,"id":"d147b2f6_6710bafc","in_reply_to":"1db9cd59_cafebbd7","updated":"2023-09-22 20:24:59.000000000","message":"It can also be seen working on this proposed addition of a cinder-tempest-ceph-multibackend job:\n\nhttps://review.opendev.org/c/openstack/cinder/+/896172","commit_id":"771872fd736b19193119de2952def670b0aac4cf"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3be28c0a58f6c2f0a0fb964456edca41e4651b10","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"665f6993_82144ced","updated":"2023-12-14 22:32:18.000000000","message":"lgtm","commit_id":"bd76355ef536d22c65cf83e694b9953579b9f740"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"e647e339f322191fb7d71f3f8cf4aaa7fb985bce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"86c82e2a_d78e25ae","updated":"2023-12-20 19:52:33.000000000","message":"lgtm","commit_id":"bd76355ef536d22c65cf83e694b9953579b9f740"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"abaa2f4e258ed89b7f4aa48f1a7e34145efd6515","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"e0bece6f_2ac83812","updated":"2023-12-20 19:52:53.000000000","message":"recheck to trigger CI run to see the latest results","commit_id":"bd76355ef536d22c65cf83e694b9953579b9f740"}]}