)]}'
{"id":"openstack%2Fopenstack-ansible~851764","triplet_id":"openstack%2Fopenstack-ansible~master~Ic76b2c211484fb107d8d23f4ef6e6cc9a4ddec4f","project":"openstack/openstack-ansible","branch":"master","topic":"osa-ironic-tidy","attention_set":{},"removed_from_attention_set":{"28619":{"account":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"last_update":"2022-08-08 14:19:46.000000000","reason":"Change was submitted"},"25023":{"account":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"last_update":"2022-08-08 14:19:46.000000000","reason":"Change was submitted"}},"hashtags":[],"change_id":"Ic76b2c211484fb107d8d23f4ef6e6cc9a4ddec4f","subject":"Do not create {hostname}-host_containers group as child of other groups","status":"MERGED","created":"2022-08-01 17:41:42.000000000","updated":"2022-08-08 14:22:30.000000000","submitted":"2022-08-08 14:19:46.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":4,"unresolved_comment_count":1,"has_review_started":true,"submission_id":"851764-osa-ironic-tidy","meta_rev_id":"bf6dface069230f5dc6601085ed54a6248d1e05f","_number":851764,"virtual_id_number":851764,"owner":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},{"value":0,"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2022-08-08 14:19:46.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"all":[{"value":2,"date":"2022-08-08 10:04:19.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},{"value":2,"date":"2022-08-03 07:57:45.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"all":[{"value":1,"date":"2022-08-08 10:04:19.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},{"value":0,"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},{"value":0,"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2022-08-01 19:48:24.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2022-08-02 10:27:17.000000000","updated_by":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"reviewer":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"state":"CC"},{"updated":"2022-08-03 07:57:45.000000000","updated_by":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"reviewer":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"state":"REVIEWER"},{"updated":"2022-08-08 10:04:19.000000000","updated_by":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"reviewer":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"state":"REVIEWER"}],"messages":[{"id":"bddeac9fd723140917a310ef747aeaa48481a4e7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-01 17:41:42.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"3381f95d23300b25fcd239fb3adf2314863985c5","tag":"autogenerated:gerrit:setWorkInProgress","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-01 17:41:55.000000000","message":"Set Work In Progress","accounts_in_message":[],"_revision_number":1},{"id":"d6af1626c3633321540f41234ada338cb14fb488","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-01 17:43:31.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"d9df1165084e77f5039b0e38759510bce7cc2c28","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-01 19:48:24.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/26dc33f3c1c24894b8642de98e7d033f\n\n- openstack-ansible-linters-ubuntu-focal https://zuul.opendev.org/t/openstack/build/865949cdfc04490bbccd6dce23435058 : FAILURE in 19m 40s\n- openstack-ansible-varstest-aio https://zuul.opendev.org/t/openstack/build/83fda13137504f0eb8e8b02c97f716b0 : SUCCESS in 21m 03s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/60a77dc0349a4b4d850b8d1522adb668 : SUCCESS in 11m 07s\n- openstack-ansible-deploy-aio_lxc-centos-9-stream https://zuul.opendev.org/t/openstack/build/4eb4a29b7b524dc2906bafb4652c7443 : FAILURE in 39m 47s\n- openstack-ansible-deploy-aio_lxc-debian-bullseye https://zuul.opendev.org/t/openstack/build/0dae2d9bb5d9429599ad92b3c4d9a977 : FAILURE in 34m 46s\n- openstack-ansible-deploy-aio_lxc-ubuntu-focal https://zuul.opendev.org/t/openstack/build/053da6fd3614472c96a87086bc125651 : FAILURE in 35m 02s\n- openstack-ansible-deploy-aio_lxc-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/ab541a11df3745809997de5aa2347e22 : FAILURE in 36m 17s (non-voting)\n- openstack-ansible-deploy-aio_metal-debian-bullseye https://zuul.opendev.org/t/openstack/build/0d689f19d9f24807aa4289b155701474 : SUCCESS in 1h 19m 27s\n- openstack-ansible-deploy-aio_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/01c63ea2b7c84284823aa19d4f0784aa : SUCCESS in 1h 26m 52s\n- openstack-ansible-deploy-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/061b0155a09e442395daccbad828ad94 : SUCCESS in 1h 34m 47s\n- openstack-ansible-deploy-aio_metal-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/4c15e98e1efb4cbd84b637f580beec17 : SUCCESS in 1h 28m 04s (non-voting)\n- openstack-ansible-upgrade-aio_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/1181ca1d1eb8491d8c86dbb6947dddee : FAILURE in 1h 33m 13s (non-voting)\n- openstack-ansible-upgrade-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/c4cd799436664cb79393d50b2174de7d : SUCCESS in 1h 58m 39s\n- openstack-ansible-deploy-aio_distro_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/984c31e1eee947dab7eec2dc215f2990 : FAILURE in 31m 45s (non-voting)\n- openstack-ansible-deploy-aio_distro_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/05bcac5aea16445981e5577973fd0c27 : FAILURE in 29m 57s (non-voting)\n- openstack-ansible-deploy-aio_ceph-ubuntu-focal https://zuul.opendev.org/t/openstack/build/5b126005fe004c26a63c93229b955780 : FAILURE in 33m 30s\n- openstack-ansible-deploy-aio_ceph-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/50271ae827da4ee1945c09dc0bded7e1 : FAILURE in 36m 11s (non-voting)\n- openstack-ansible-deploy-aio_proxy-ubuntu-focal https://zuul.opendev.org/t/openstack/build/bd4e2ec56ba04a4eb2f4f4b2645e680b : FAILURE in 33m 53s","accounts_in_message":[],"_revision_number":2},{"id":"3c986f8a897f4c5b7411938793f2561e8dcc55ca","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-08-02 10:27:17.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"45f31e63b8162879c9c7b2992397616f13f524d8","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-08-02 15:55:04.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"002fff91710b922781db0fa98b4a57f95618d8e1","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-02 16:52:08.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"4c5e5877461fef78ad2433e65b7f1e13402bcfd8","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-02 19:32:18.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/443b65d61e7d474ea58ab198a1350a1d\n\n- openstack-ansible-linters-ubuntu-focal https://zuul.opendev.org/t/openstack/build/23219e4ff40f45b8b0480a4ac3bdfb01 : SUCCESS in 20m 45s\n- openstack-ansible-varstest-aio https://zuul.opendev.org/t/openstack/build/8f824099bfce4d17a035bfaaaffb7b7d : SUCCESS in 22m 27s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/7adc858046354f7b901b64abb7ce3d5a : SUCCESS in 11m 11s\n- openstack-ansible-deploy-aio_lxc-centos-9-stream https://zuul.opendev.org/t/openstack/build/a10af8ef0f10466aa7b2f90faa255fdd : SUCCESS in 2h 00m 56s\n- openstack-ansible-deploy-aio_lxc-debian-bullseye https://zuul.opendev.org/t/openstack/build/e8215abf0152460d9ce4aed4fb205fc3 : SUCCESS in 1h 56m 51s\n- openstack-ansible-deploy-aio_lxc-ubuntu-focal https://zuul.opendev.org/t/openstack/build/d8a5ff96a5df457ab3414d3591e52451 : SUCCESS in 2h 32m 44s\n- openstack-ansible-deploy-aio_lxc-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/db9b6eb3725243a4bf8e7227911c749b : SUCCESS in 2h 00m 06s (non-voting)\n- openstack-ansible-deploy-aio_metal-debian-bullseye https://zuul.opendev.org/t/openstack/build/e62b0036f6d347c3b2f0ded019228f9f : SUCCESS in 1h 17m 46s\n- openstack-ansible-deploy-aio_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/fb5d0f5388ac412392d0ec047501c0da : SUCCESS in 1h 33m 36s\n- openstack-ansible-deploy-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/ea001e883dbb46ba94764f4245ed8584 : SUCCESS in 1h 25m 28s\n- openstack-ansible-deploy-aio_metal-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/7ee5b43a75c649869ce15338a3c09f14 : SUCCESS in 1h 22m 12s (non-voting)\n- openstack-ansible-upgrade-aio_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/2ce51570f8e54a9384dcfe34a19d8531 : SUCCESS in 1h 51m 06s (non-voting)\n- openstack-ansible-upgrade-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/2decbbd0c298448fb23ea690e86ed9eb : SUCCESS in 2h 10m 37s\n- openstack-ansible-deploy-aio_distro_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/9a96f309d58d48d29cbca5b190056b8f : FAILURE in 35m 47s (non-voting)\n- openstack-ansible-deploy-aio_distro_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/2391d229f5344ed1b9a62522e22f93a5 : FAILURE in 32m 27s (non-voting)\n- openstack-ansible-deploy-aio_ceph-ubuntu-focal https://zuul.opendev.org/t/openstack/build/ba503157ee324a32974526c1feb6e08c : SUCCESS in 2h 28m 49s\n- openstack-ansible-deploy-aio_ceph-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/abdab32dedaf4d81b4d1eeaa6ab28a57 : FAILURE in 48m 54s (non-voting)\n- openstack-ansible-deploy-aio_proxy-ubuntu-focal https://zuul.opendev.org/t/openstack/build/d4de49664a1b4747ac15f355625351ac : SUCCESS in 2h 11m 24s","accounts_in_message":[],"_revision_number":4},{"id":"c427345aee12a025332aa9a3796f75f7f22caa59","tag":"autogenerated:gerrit:newWipPatchSet","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-02 21:06:20.000000000","message":"Patch Set 5: Commit message was updated.","accounts_in_message":[],"_revision_number":5},{"id":"ea0cb164a70d0886cb01b23dd58ac6464516759e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-02 23:38:50.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/dab82d72bb6844e0bfc8a9fd00af6e0b\n\n- openstack-ansible-linters-ubuntu-focal https://zuul.opendev.org/t/openstack/build/326b8c4de56946d8a062a44143685623 : SUCCESS in 21m 22s\n- openstack-ansible-varstest-aio https://zuul.opendev.org/t/openstack/build/2f5bc5fbf71a4265b96b38f15386466c : SUCCESS in 23m 14s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5cdac77021484416bc7d5185c4f1d9df : SUCCESS in 9m 19s\n- openstack-ansible-deploy-aio_lxc-centos-9-stream https://zuul.opendev.org/t/openstack/build/d98856bdef5244499c13371a45962e5f : SUCCESS in 2h 06m 51s\n- openstack-ansible-deploy-aio_lxc-debian-bullseye https://zuul.opendev.org/t/openstack/build/bbc1aaff4159426c83aaab9435b8a3da : SUCCESS in 2h 17m 25s\n- openstack-ansible-deploy-aio_lxc-ubuntu-focal https://zuul.opendev.org/t/openstack/build/9fd3e74f75c148238eeac6ef999cec3d : SUCCESS in 2h 15m 04s\n- openstack-ansible-deploy-aio_lxc-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/d922d643f09649ec97e8f30ce6cee7fb : SUCCESS in 2h 00m 21s (non-voting)\n- openstack-ansible-deploy-aio_metal-debian-bullseye https://zuul.opendev.org/t/openstack/build/a846da0efc8141cf92e3da29d7cfff46 : SUCCESS in 1h 27m 59s\n- openstack-ansible-deploy-aio_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/29e7f608d4cb4a378e6ed9bfc622aca5 : SUCCESS in 1h 30m 08s\n- openstack-ansible-deploy-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/26fbbf84dd0e437bae63c33568ac7b89 : SUCCESS in 1h 42m 55s\n- openstack-ansible-deploy-aio_metal-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/31b7dba5d80f4c6baff1a96abe71a810 : SUCCESS in 1h 27m 16s (non-voting)\n- openstack-ansible-upgrade-aio_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/a5bfdf42c31446e7bb329582585d3005 : SUCCESS in 1h 53m 57s (non-voting)\n- openstack-ansible-upgrade-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/281129761aa84110b1907746c5709226 : SUCCESS in 1h 59m 58s\n- openstack-ansible-deploy-aio_distro_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/67b71879059d469288447a2603865fca : FAILURE in 37m 26s (non-voting)\n- openstack-ansible-deploy-aio_distro_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/e15397fbca0444799d3ec02cd349d69b : FAILURE in 35m 25s (non-voting)\n- openstack-ansible-deploy-aio_ceph-ubuntu-focal https://zuul.opendev.org/t/openstack/build/574bcd14f0274f2ca77d766fe0aa44b0 : SUCCESS in 2h 29m 09s\n- openstack-ansible-deploy-aio_ceph-ubuntu-jammy https://zuul.opendev.org/t/openstack/build/b65ca644d9b04a0aa62939cbe4afe952 : FAILURE in 53m 37s (non-voting)\n- openstack-ansible-deploy-aio_proxy-ubuntu-focal https://zuul.opendev.org/t/openstack/build/81a9c5f0735042688b89c3eecc66f3a5 : SUCCESS in 2h 12m 26s","accounts_in_message":[],"_revision_number":5},{"id":"f0c7ad9b73bc3226d83af21547df9016973b5d83","author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"date":"2022-08-03 07:57:45.000000000","message":"Patch Set 5: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"b6c0124cdbac85916593d89f0284fe1906507c9b","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-03 08:19:18.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"16b40e6d400b223da112540c77ec70274c8ba35b","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-03 08:36:54.000000000","message":"Patch Set 5:\n\n(1 comment)\n\nThis change is ready for review.","accounts_in_message":[],"_revision_number":5},{"id":"ba0394e80c5df3e8a9ddb7dbe4022f9d4913cf78","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-04 17:09:12.000000000","message":"Topic 851764 removed","accounts_in_message":[],"_revision_number":5},{"id":"6a13a29d151cf06335a75748b136a99572d19689","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"date":"2022-08-04 17:09:14.000000000","message":"Topic set to osa-ironic-tidy","accounts_in_message":[],"_revision_number":5},{"id":"77c469e21f8302a7e8bbe1097fb5afd4e07f8da7","author":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"},"date":"2022-08-08 10:04:19.000000000","message":"Patch Set 5: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":5},{"id":"785eadcc28443a13edc3f9d58b3b3b253359017a","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-08 10:06:48.000000000","message":"Patch Set 5: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":5},{"id":"9782c44d9c9d1bd7fb7fd0b9d9ec7fda882d4a8f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-08 14:19:46.000000000","message":"Patch Set 5: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/16c25ee1c79f4173b0e9c3bbddefa42d\n\n- openstack-ansible-linters-ubuntu-focal https://zuul.opendev.org/t/openstack/build/fbecdb9546784dc78ff9db1ab0b2ec5b : SUCCESS in 14m 32s\n- openstack-ansible-varstest-aio https://zuul.opendev.org/t/openstack/build/609b9a235f7e45aa843800ff2a3cb6c8 : SUCCESS in 14m 54s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a3ca3df850da465e864aa5e175ad6ae7 : SUCCESS in 11m 24s\n- openstack-ansible-deploy-aio_lxc-centos-9-stream https://zuul.opendev.org/t/openstack/build/67400cdd30cd47db8b4160c929abcde8 : SUCCESS in 2h 00m 31s\n- openstack-ansible-deploy-aio_lxc-debian-bullseye https://zuul.opendev.org/t/openstack/build/981084791b4446b582978f7db87feca3 : SUCCESS in 1h 55m 21s\n- openstack-ansible-deploy-aio_lxc-ubuntu-focal https://zuul.opendev.org/t/openstack/build/86c9126a74794e35a5ce94e1d76c69b0 : SUCCESS in 2h 06m 22s\n- openstack-ansible-deploy-aio_metal-debian-bullseye https://zuul.opendev.org/t/openstack/build/aa68f66766dd4e7eb7ad01fef6f24dcc : SUCCESS in 1h 16m 20s\n- openstack-ansible-deploy-aio_metal-centos-9-stream https://zuul.opendev.org/t/openstack/build/eef52558553a4fb1891cb1626142c4b2 : SUCCESS in 1h 41m 06s\n- openstack-ansible-deploy-aio_metal-ubuntu-focal https://zuul.opendev.org/t/openstack/build/87b1d757a095419bad4bab955e5a3485 : SUCCESS in 1h 31m 50s\n- openstack-ansible-deploy-aio_ceph-ubuntu-focal https://zuul.opendev.org/t/openstack/build/bbe30141a6e24b0fa7ed529c6300173b : SUCCESS in 2h 17m 13s\n- openstack-ansible-deploy-aio_proxy-ubuntu-focal https://zuul.opendev.org/t/openstack/build/53e28a4206c048f6b7228e45bba7c2b3 : SUCCESS in 1h 55m 40s","accounts_in_message":[],"_revision_number":5},{"id":"ebf593cde26e047ba385380f08f4e1b95f3ad07b","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-08 14:19:46.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":5},{"id":"bf6dface069230f5dc6601085ed54a6248d1e05f","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2022-08-08 14:22:30.000000000","message":"Patch Set 5:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/cb33e8cf2c864f11ad4bdd021907aa14\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5b16a8e640814cd7b269e43cfff6576d : SUCCESS in 1m 15s","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"c76a4d5dfe6c22b17064a11d7c60aa5b629ab439","revisions":{"c607bc546aa6cb97c1e9330e00c7b5f50e9f903a":{"kind":"REWORK","_number":1,"created":"2022-08-01 17:41:42.000000000","uploader":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"ref":"refs/changes/64/851764/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/64/851764/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/1"}}},"commit":{"parents":[{"commit":"3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5","subject":"Merge \"rabbitmq: default to using TLS for management user interface\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5"}]}],"author":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-01 17:31:34.000000000","tz":60},"committer":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-01 17:40:48.000000000","tz":60},"subject":"Do not create {hostname}-host_containers group as child of other containers","message":"Do not create {hostname}-host_containers group as child of other containers\n\nThis code adds entries like aio1-host_containers as a child group\nof many containers in the inventory.\n\nThe side effect is that the ansible magic variable group_names for\na particular LXC container lists many group names associated with\nall of the containers on its physical host.\n\nRoles such as os_ironic crate several containers and use conditional\nlogic to deploy the correct components into each container using the\ncontents of group_names. This does not work correctly when group_names\ncontains all of the possible ironic container groups.\n\nThis patch removes the code which adds host container groups as\nchildren of other container groups.\n\nChange-Id: Ic76b2c211484fb107d8d23f4ef6e6cc9a4ddec4f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/c607bc546aa6cb97c1e9330e00c7b5f50e9f903a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/c607bc546aa6cb97c1e9330e00c7b5f50e9f903a"}]},"branch":"refs/heads/master"},"5bdb70602dc35042d72edb5e220f74f29123dbd5":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2022-08-01 17:43:31.000000000","uploader":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"ref":"refs/changes/64/851764/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/64/851764/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/2"}}},"commit":{"parents":[{"commit":"3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5","subject":"Merge \"rabbitmq: default to using TLS for management user interface\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5"}]}],"author":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-01 17:31:34.000000000","tz":60},"committer":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-01 17:42:42.000000000","tz":60},"subject":"Do not create {hostname}-host_containers group as child of other groups","message":"Do not create {hostname}-host_containers group as child of other groups\n\nThis code adds entries like aio1-host_containers as a child group\nof many other container type groups  in the inventory.\n\nThe side effect is that the ansible magic variable group_names for\na particular LXC container lists many group names associated with\nall of the containers on its physical host.\n\nRoles such as os_ironic crate several containers and use conditional\nlogic to deploy the correct components into each container using the\ncontents of group_names. This does not work correctly when group_names\ncontains all of the possible ironic container groups.\n\nThis patch removes the code which adds host container groups as\nchildren of other container type groups.\n\nChange-Id: Ic76b2c211484fb107d8d23f4ef6e6cc9a4ddec4f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/5bdb70602dc35042d72edb5e220f74f29123dbd5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/5bdb70602dc35042d72edb5e220f74f29123dbd5"}]},"branch":"refs/heads/master"},"9b65d08a1e52e4423785a833e052f46cecb36090":{"kind":"REWORK","_number":3,"created":"2022-08-02 15:55:04.000000000","uploader":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"ref":"refs/changes/64/851764/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/64/851764/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/3"}}},"commit":{"parents":[{"commit":"3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5","subject":"Merge \"rabbitmq: default to using TLS for management user interface\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5"}]}],"author":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-01 17:31:34.000000000","tz":60},"committer":{"name":"Dmitriy Rabotyagov","email":"dmitriy.rabotyagov@citynetwork.eu","date":"2022-08-02 15:54:51.000000000","tz":120},"subject":"Do not create {hostname}-host_containers group as child of other groups","message":"Do not create {hostname}-host_containers group as child of other groups\n\nThis code adds entries like aio1-host_containers as a child group\nof many other container type groups  in the inventory.\n\nThe side effect is that the ansible magic variable group_names for\na particular LXC container lists many group names associated with\nall of the containers on its physical host.\n\nRoles such as os_ironic crate several containers and use conditional\nlogic to deploy the correct components into each container using the\ncontents of group_names. This does not work correctly when group_names\ncontains all of the possible ironic container groups.\n\nThis patch removes the code which adds host container groups as\nchildren of other container type groups.\n\nChange-Id: Ic76b2c211484fb107d8d23f4ef6e6cc9a4ddec4f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/9b65d08a1e52e4423785a833e052f46cecb36090"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/9b65d08a1e52e4423785a833e052f46cecb36090"}]},"branch":"refs/heads/master"},"c0b8465c0c7b778bef937e699565921aeccfc7c2":{"kind":"REWORK","_number":4,"created":"2022-08-02 16:52:08.000000000","uploader":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"ref":"refs/changes/64/851764/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/64/851764/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/4"}}},"commit":{"parents":[{"commit":"3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5","subject":"Merge \"rabbitmq: default to using TLS for management user interface\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5"}]}],"author":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-01 17:31:34.000000000","tz":60},"committer":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-02 16:50:57.000000000","tz":60},"subject":"Do not create {hostname}-host_containers group as child of other groups","message":"Do not create {hostname}-host_containers group as child of other groups\n\nThe existing code adds entries like aio1-host_containers as a child group\nof many other container type groups  in the inventory.\n\nThe side effect is that the ansible magic variable group_names for\na particular LXC container lists many group names associated with\nall of the containers on its physical host.\n\nRoles such as os_ironic crate several containers and use conditional\nlogic to deploy the correct components into each container using the\ncontents of group_names. This does not work correctly when group_names\ncontains all of the possible ironic container groups.\n\nThis patch removes the code which adds host container groups as\nchildren of other container type groups.\n\nRemoval of the {hostname}-host_containers group from each container\nreveals a further bug, where any belongs_to directives in env.d\nfile container_skel sections were not processed. There is different\nfunctionality in skel_load() and container_skel_load() which is the\ncause of this.\n\nThis patch adds a call to _parse_belongs_to() into\ncontainer_skel_load() so that any groups defined with \u0027belongs_to\u0027\nin container_skel are correctly added as children to the\ncorresponding parent group.\n\nChange-Id: Ic76b2c211484fb107d8d23f4ef6e6cc9a4ddec4f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/c0b8465c0c7b778bef937e699565921aeccfc7c2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/c0b8465c0c7b778bef937e699565921aeccfc7c2"}]},"branch":"refs/heads/master"},"c76a4d5dfe6c22b17064a11d7c60aa5b629ab439":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2022-08-02 21:06:20.000000000","uploader":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"ref":"refs/changes/64/851764/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-ansible","ref":"refs/changes/64/851764/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-ansible refs/changes/64/851764/5"}}},"commit":{"parents":[{"commit":"3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5","subject":"Merge \"rabbitmq: default to using TLS for management user interface\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/3e4ee68a7363da35fd6207dd773dfa90d1fbf6b5"}]}],"author":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-01 17:31:34.000000000","tz":60},"committer":{"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","date":"2022-08-02 21:06:20.000000000","tz":0},"subject":"Do not create {hostname}-host_containers group as child of other groups","message":"Do not create {hostname}-host_containers group as child of other groups\n\nThe existing code adds entries like aio1-host_containers as a child group\nof many other container type groups in the inventory.\n\nThe side effect is that the ansible magic variable group_names for\na particular LXC container lists many group names associated with\nall of the containers on its physical host.\n\nRoles such as os_ironic create several containers and use conditional\nlogic to deploy the correct components into each container using the\ncontents of group_names. This does not work correctly when group_names\ncontains all of the possible ironic container groups.\n\nThis patch removes the code which adds host container groups as\nchildren of other container type groups.\n\nRemoval of the {hostname}-host_containers group from each container\nreveals a further bug, where any belongs_to directives in env.d\nfile container_skel sections were not processed. There is different\nfunctionality in skel_load() and container_skel_load() which is the\ncause of this.\n\nThis patch adds a call to _parse_belongs_to() into\ncontainer_skel_load() so that any groups defined with \u0027belongs_to\u0027\nin container_skel are correctly added as children to the\ncorresponding parent group.\n\nChange-Id: Ic76b2c211484fb107d8d23f4ef6e6cc9a4ddec4f\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/c76a4d5dfe6c22b17064a11d7c60aa5b629ab439"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-ansible/commit/c76a4d5dfe6c22b17064a11d7c60aa5b629ab439"}]},"branch":"refs/heads/master","description":"Edit commit message"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":31542,"name":"Andrew Bonney","email":"andrew.bonney@bbc.co.uk","username":"andrewbonney"}},{"label":"Backport-Candidate","status":"MAY"}]}],"submit_requirements":[]}