)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"9f22561f93bee9d7881b01deb39138d5c555f7a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"46cc55e9_a695c2ce","updated":"2023-12-19 16:21:32.000000000","message":"A much more elegant solution than my previous precheck. I\u0027ve tested this myself, works well. It is good that this brings attention to the specific queues which are incorrect.","commit_id":"afa241788db2b1a6a2043eb49b1b69441efd1110"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"907b1f32cdb996c03e48fa56479baf0cb77633e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"263749c3_253fa2c4","updated":"2023-12-19 16:59:14.000000000","message":"Merging counting in 2*+1 as +2 in christmas period ;-)","commit_id":"afa241788db2b1a6a2043eb49b1b69441efd1110"}],"ansible/roles/rabbitmq/tasks/precheck.yml":[{"author":{"_account_id":34579,"name":"Juan Pablo Suazo","email":"jsuazo@whitestack.com","username":"jsuazo"},"change_message_id":"e6e2cf96e6b0027acfc7e75453c19111805448b6","unresolved":true,"context_lines":[{"line_number":236,"context_line":"    - name: Check if RabbitMQ quorum queues need to be configured"},{"line_number":237,"context_line":"      assert:"},{"line_number":238,"context_line":"        # TODO(mattcress): remove skipping reply and fanout once https://review.opendev.org/c/openstack/oslo.messaging/+/888479 is merged."},{"line_number":239,"context_line":"        that: \"{{ not (item.name.startswith(\u0027reply_\u0027) or item.name is match(\u0027*_fanout_*\u0027)) and item.name\u0027(item.type !\u003d \u0027quorum\u0027) }}\""},{"line_number":240,"context_line":"        fail_msg: \u003e"},{"line_number":241,"context_line":"          om_enable_rabbitmq_quorum_queues is True but {{ item.name }} is a non-quorum queue."},{"line_number":242,"context_line":"          Currently the procedure to migrate to quorum queues is manual."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"661003d8_db4e1ce6","line":239,"updated":"2023-12-18 14:59:51.000000000","message":"Im not sure this is correct, Zuul job previously failed with: `expected token \u0027end of print statement\u0027, got \u0027string\u0027`, meaning, here junga is not seing the final `}}`. I suspect this part: `name\u0027(item.type !\u003d \u0027quorum\u0027)` has an extra `\u0027`.","commit_id":"ae9a3dac3a13391307f41a97da609b0c94f43680"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a7ce105a0083dcda9068bb34bfa85634f5d17768","unresolved":false,"context_lines":[{"line_number":236,"context_line":"    - name: Check if RabbitMQ quorum queues need to be configured"},{"line_number":237,"context_line":"      assert:"},{"line_number":238,"context_line":"        # TODO(mattcress): remove skipping reply and fanout once https://review.opendev.org/c/openstack/oslo.messaging/+/888479 is merged."},{"line_number":239,"context_line":"        that: \"{{ not (item.name.startswith(\u0027reply_\u0027) or item.name is match(\u0027*_fanout_*\u0027)) and item.name\u0027(item.type !\u003d \u0027quorum\u0027) }}\""},{"line_number":240,"context_line":"        fail_msg: \u003e"},{"line_number":241,"context_line":"          om_enable_rabbitmq_quorum_queues is True but {{ item.name }} is a non-quorum queue."},{"line_number":242,"context_line":"          Currently the procedure to migrate to quorum queues is manual."}],"source_content_type":"text/x-yaml","patch_set":9,"id":"d9cf6c29_6e007656","line":239,"in_reply_to":"661003d8_db4e1ce6","updated":"2023-12-18 16:00:25.000000000","message":"Thanks, I can read ansible-lint - this patch is work in progress, so that\u0027s not targeted for reviewing right now.","commit_id":"ae9a3dac3a13391307f41a97da609b0c94f43680"}]}