)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"1c2cbb90a5b73fe5506b2c12cc39009c05c87681","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b735897e_db9017f9","updated":"2022-10-19 00:21:31.000000000","message":"As pointed out by John in the related BZ - need to handle case where the same role is used in both inventories","commit_id":"98bcce4cc24ddef52fa2e813b1e3a1deffa10b08"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"c299755982b2fa8993c01b326c0db3b9c28df924","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3e350773_4c039283","updated":"2022-10-19 13:05:29.000000000","message":"Can this document be updated so it\u0027s clear how the user will use the result of this change?\n\nI assume it will be something like this:\n\n  ansible-playbook -i directory\n  \nWhere directory contains overcloud.yaml, site1.yaml, site2.yaml. \n\nI suppose we\u0027ll also need an update to tripleo-validations which removes tripleo-ansible-inventory.\n\nhttps://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/post_deployment/pre_cache_images.html#multi-stacks-inventory","commit_id":"354351d7d5f0b7be52ee8c90e337827cf2ff6914"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"7b5330b4b7ebf8b27147935e9d05010763fdf670","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ec506cc7_82c51a25","updated":"2022-10-19 10:08:57.000000000","message":"recheck","commit_id":"354351d7d5f0b7be52ee8c90e337827cf2ff6914"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"8a6f544fc9fdcc2409d9903a5357889d7106263f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d297c07d_b4ea1436","in_reply_to":"3e350773_4c039283","updated":"2022-10-19 14:05:54.000000000","message":"can also use:\n\n    ansible-playbook -i overcloud.yaml -i site1.yaml -i site2.yaml","commit_id":"354351d7d5f0b7be52ee8c90e337827cf2ff6914"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"94e6609c1854c2b3bd57e3a8e2cf12c26e16ff85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"693bc306_161e73eb","in_reply_to":"d297c07d_b4ea1436","updated":"2022-10-20 13:24:10.000000000","message":"done here https://review.opendev.org/c/openstack/tripleo-docs/+/862142","commit_id":"354351d7d5f0b7be52ee8c90e337827cf2ff6914"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"c42486b2ca4e66339a6bb38ebb2e81b9fb21411b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"771806ef_67b0adb1","updated":"2022-10-19 14:42:29.000000000","message":"(undercloud) [stack@undercloud-0 ~]$ ansible -i inventories -m debug -a \"msg\u003d\\\"{{groups[\u0027CellController\u0027] | map(\u0027extract\u0027, hostvars)}}\\\"\" Controller | grep vip\n            \"ctlplane_vip\": \"192.168.24.52\",\n            \"external_vip\": \"10.0.0.129\",\n            \"internal_api_vip\": \"172.17.1.73\",\n            \"storage_mgmt_vip\": \"172.17.4.97\",\n            \"storage_vip\": \"172.17.3.121\",\n(undercloud) [stack@undercloud-0 ~]$ ansible -i inventories -m debug -a \"msg\u003d\\\"{{groups[\u0027Controller\u0027] | map(\u0027extract\u0027, hostvars)}}\\\"\" Controller | grep vip\n            \"ctlplane_vip\": \"192.168.24.25\",\n            \"external_vip\": \"10.0.0.141\",\n            \"internal_api_vip\": \"172.17.1.63\",\n            \"storage_mgmt_vip\": \"172.17.4.20\",\n            \"storage_vip\": \"172.17.3.77\",\n","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"14a858fa506be5f328f523d33c01361dc7957496","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0b6650fb_34649207","updated":"2022-10-19 15:13:52.000000000","message":"...meant to say Mathieu, sorry!","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"b40e75f3676fcb72741e4d94f35a33cad2967187","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"653c56d3_0c23dffe","updated":"2022-10-19 15:11:43.000000000","message":"@Daniel, could you please or someone from your team to review and test this change to check if there is no impact for your ovs-\u003eovn migration script?","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"dee6b7deedd8066a633c480502b8da3634c01ffe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f20cebfa_98aa39c6","updated":"2022-10-19 15:12:39.000000000","message":"@Mattew please take a look. What is expected for tripleo-validation here, if anything?","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"04016821ba8b22e6d5570b5951f01a745cae79af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e5b81c71_03e6627f","updated":"2022-10-20 08:33:46.000000000","message":"Thank you for working on this, looks overall good to me, I will take time to test it properly on my dev env.","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"1c5f44425bc1460ad17446cb49bb9be18e82860a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7ced1556_07abd909","updated":"2022-10-19 16:47:36.000000000","message":"Thanks Ollie, Seems like a good idea. I can +2 when it\u0027s ready (e.g. green CI, no -1 on workflow)","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"63de3c84e75a5e1248bf8995e4b05c935b6f1125","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0ad34e95_afc1888b","updated":"2022-10-19 15:26:18.000000000","message":"^^","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"9356453a0585314e72fc306d23926ebdf2896db0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"aabdfb28_c0b00164","updated":"2022-10-19 14:42:05.000000000","message":"it works for my testing","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"96834e0df0eb8c00816c14c677249fc0f8dcd66f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"835360fb_eae090d5","updated":"2022-10-19 14:33:10.000000000","message":"the produced inventory looks promising now, I\u0027ll test adhoc and come back\nMany thanks!","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"94e6609c1854c2b3bd57e3a8e2cf12c26e16ff85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d689d436_0a3a6cfe","in_reply_to":"192c13b2_1bc5a1d2","updated":"2022-10-20 13:24:10.000000000","message":"Done","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"301cbfd6ec6c18fa6a948f01a18bd50bc7289293","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"192c13b2_1bc5a1d2","in_reply_to":"653c56d3_0c23dffe","updated":"2022-10-20 12:13:58.000000000","message":"I\u0027ve tested generate_ansible_inventory_file() from ovn_migration.sh and get the same result","commit_id":"acd622bc4b523350ff0384766fa7bac62aaf7a38"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"09dcdeaba1a2d17ef7ff7aaa2b4ca3d6563cdcaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e6e8d439_013019af","updated":"2022-10-24 14:16:30.000000000","message":"PTAL","commit_id":"2c485efa86824cbd9d171b00961a87e948cb6854"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"e4480c2531b6288427ae9f02c8cd7a349b7b392e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c81ede9a_585b262e","updated":"2022-10-24 13:23:44.000000000","message":"check-rdo","commit_id":"2c485efa86824cbd9d171b00961a87e948cb6854"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"10a24cc8d08a2497c22107a0af21f26cce76e533","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d43959ed_c6234fa6","updated":"2022-10-20 14:41:07.000000000","message":"check-rdo","commit_id":"2c485efa86824cbd9d171b00961a87e948cb6854"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"6a26a9791d68aa40ff79f940fe0764d29ea19d05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"342be919_7152578b","updated":"2022-11-16 09:23:51.000000000","message":"recheck","commit_id":"2c485efa86824cbd9d171b00961a87e948cb6854"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"d42ee8c9727cd8a8d0c933f425a1714e98ea81e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"42f6d321_27e2d98c","updated":"2022-10-20 16:52:14.000000000","message":"recheck","commit_id":"2c485efa86824cbd9d171b00961a87e948cb6854"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"270e80308dd3456688f09d4d02f8f657acc48177","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"abb64376_b1a5253a","updated":"2022-11-21 14:04:35.000000000","message":"recheck","commit_id":"2c485efa86824cbd9d171b00961a87e948cb6854"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"1aa04cde707cde2046ebe1a54bca7b476cce89f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d2d30d93_8b0f9399","updated":"2022-11-17 14:10:45.000000000","message":"recheck","commit_id":"2c485efa86824cbd9d171b00961a87e948cb6854"}]}