)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"e91ddd8932ce1c97446e49ef93baa2da5f90b7e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"5d2b3896_3e32b2f8","updated":"2021-11-22 12:28:31.000000000","message":"We need https://review.opendev.org/c/openstack/openstack-ansible/+/818733 for upgrade jobs","commit_id":"eb9a0c6ceaae98fe992bab1f9b5be797ba8a99b4"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"4ae927d520d3bd7ff2e444c1d4c90707886280af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"be1c9bd4_22c3926f","updated":"2021-11-23 18:38:48.000000000","message":"recheck","commit_id":"eb9a0c6ceaae98fe992bab1f9b5be797ba8a99b4"}],"tasks/keystone_install.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"88072930095866606da037ece3d573ba78114774","unresolved":true,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# If the web server being removed was never installed, this task will fail when trying"},{"line_number":51,"context_line":"# to stop/disable it. The task is therefore set to never fail."},{"line_number":52,"context_line":"- name: Remove nginx web server"},{"line_number":53,"context_line":"  when: \"(\u0027repo_all\u0027 in groups and inventory_hostname not in groups[\u0027repo_all\u0027]) or (\u0027repo_all\u0027 not in groups)\""},{"line_number":54,"context_line":"  block:"},{"line_number":55,"context_line":"    - name: Ensure nginx web server is not running/enabled"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7fedbf06_a6c7dfc0","line":52,"range":{"start_line":52,"start_character":8,"end_line":52,"end_character":31},"updated":"2021-11-12 18:15:28.000000000","message":"does this break the repo server on a metal deployment (which i think has nginx serving the python wheels?)","commit_id":"7897677d6a44546800cba57a72f1ed8661dcfade"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"c0592dc60018d7e35b3d56578536a1d375a8be99","unresolved":true,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# If the web server being removed was never installed, this task will fail when trying"},{"line_number":51,"context_line":"# to stop/disable it. The task is therefore set to never fail."},{"line_number":52,"context_line":"- name: Remove nginx web server"},{"line_number":53,"context_line":"  when: \"(\u0027repo_all\u0027 in groups and inventory_hostname not in groups[\u0027repo_all\u0027]) or (\u0027repo_all\u0027 not in groups)\""},{"line_number":54,"context_line":"  block:"},{"line_number":55,"context_line":"    - name: Ensure nginx web server is not running/enabled"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1c8e9cc8_537214cf","line":52,"range":{"start_line":52,"start_character":8,"end_line":52,"end_character":31},"in_reply_to":"7fedbf06_a6c7dfc0","updated":"2021-11-16 15:12:25.000000000","message":"well, I actually hoped that condition should handle that. Probably it\u0027s worth just to disable site instead of dropping nginx...","commit_id":"7897677d6a44546800cba57a72f1ed8661dcfade"}]}
