)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"dfc7faa58a225c888a8a75125d2d3d1f9f74bed9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add Swift storage policy to extend testing"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Swift supports storage policies for a long time, making it possible to"},{"line_number":10,"context_line":"store specific objects on different devices[1]."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch adds a second storage policy to support testing API features"},{"line_number":13,"context_line":"that rely on storage policies, eg. per-policy quotas[2]. There are no"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"0c7e044f_b98023f4","line":10,"updated":"2025-12-17 16:50:12.000000000","message":"oh so anti affinity or and erasure encoding policies like ceph cool.","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"9bda84380ee95fca300214815d0c4444d12b04b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8e549da7_5c5af1db","updated":"2025-10-28 15:54:16.000000000","message":"Change limited to swift, so it shouldn\u0027t break anything.","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9bb7b6b14a809460efafd54b15eaf0575261ac91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f5fd6cff_a7cb86f8","updated":"2026-02-13 16:17:35.000000000","message":"let\u0027s merge it and If needed (sean comment), we can fix it in followup.","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"543d3355df413018301fad4344cfa56749bcf4c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f718ad4b_28a7e1c1","updated":"2026-02-05 01:46:23.000000000","message":"lgtm, there is a new tempest test for this enw policy https://review.opendev.org/c/openstack/tempest/+/960813","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"24920847f66581e19d94e6adaf1a565cd7f53883","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eed8b182_7c5f993a","updated":"2026-02-13 16:45:22.000000000","message":"no im fine with this i just wanted to make sure this actully was safe and should work","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"f39f9eccd48fd2186082e423745b56839c2cabe0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9adee74e_66234cf4","updated":"2025-08-12 17:25:46.000000000","message":"recheck\n\nA single (possibly unrelated) error devstack-platform-debian-bookworm (tempest.api.volume.test_volumes_actions.VolumesActionsTest.test_volume_upload). That job has been mostly stable lately.","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"5ae118f256696311d118275fce7cd548c6907a01","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7995dffe_4c24d9d6","updated":"2025-07-23 08:18:40.000000000","message":"recheck Timeout during volume deletion on teardown","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"}],"lib/swift":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"dfc7faa58a225c888a8a75125d2d3d1f9f74bed9","unresolved":true,"context_lines":[{"line_number":719,"context_line":""},{"line_number":720,"context_line":"        # An additional storage policy requires an object ring as well."},{"line_number":721,"context_line":"        # Re-using the previously created one to use the same devices."},{"line_number":722,"context_line":"        cp object.ring.gz object-1.ring.gz"},{"line_number":723,"context_line":"    } \u0026\u0026 popd \u003e/dev/null"},{"line_number":724,"context_line":"}"},{"line_number":725,"context_line":""}],"source_content_type":"text/x-swift","patch_set":2,"id":"a11c88f5_17a120e6","line":722,"updated":"2025-12-17 16:50:12.000000000","message":"this feel like a hack.\nbut if actully valid then ok.\n\nyou effectivly just duplicating the db and avoidign building the ring again\n\nbefore merging this it would be nice to see this used in a tempest job\nis there one that demonstrates that this works ?","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"ff953b4cdefd72edb1e8b5949e341f5086d6ec29","unresolved":true,"context_lines":[{"line_number":719,"context_line":""},{"line_number":720,"context_line":"        # An additional storage policy requires an object ring as well."},{"line_number":721,"context_line":"        # Re-using the previously created one to use the same devices."},{"line_number":722,"context_line":"        cp object.ring.gz object-1.ring.gz"},{"line_number":723,"context_line":"    } \u0026\u0026 popd \u003e/dev/null"},{"line_number":724,"context_line":"}"},{"line_number":725,"context_line":""}],"source_content_type":"text/x-swift","patch_set":2,"id":"dec93206_d8bf00ca","line":722,"in_reply_to":"5650d336_ea9fa289","updated":"2026-02-13 16:09:15.000000000","message":"Yes, this is basically a hack - but given that the ring content would be exactly the same (except for the embedded name) using the same input device and random seed , this is fine to use.","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"543d3355df413018301fad4344cfa56749bcf4c4","unresolved":true,"context_lines":[{"line_number":719,"context_line":""},{"line_number":720,"context_line":"        # An additional storage policy requires an object ring as well."},{"line_number":721,"context_line":"        # Re-using the previously created one to use the same devices."},{"line_number":722,"context_line":"        cp object.ring.gz object-1.ring.gz"},{"line_number":723,"context_line":"    } \u0026\u0026 popd \u003e/dev/null"},{"line_number":724,"context_line":"}"},{"line_number":725,"context_line":""}],"source_content_type":"text/x-swift","patch_set":2,"id":"5650d336_ea9fa289","line":722,"in_reply_to":"a11c88f5_17a120e6","updated":"2026-02-05 01:46:23.000000000","message":"there is new test for this new policy (index 1) where this  object-1 ring is used in swift.\n- https://review.opendev.org/c/openstack/tempest/+/960813\n\n- https://zuul.opendev.org/t/openstack/build/e6d52e4ac999474c9ed9e279bfcb2f0e/log/controller/logs/etc/swift/swift_conf.txt#226","commit_id":"bab34bbade0da8f6b7f3fd23628522ec24688807"}]}