)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"5082f1c72b4dff23cc8ff5ebafcad10a0075d5e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"8a61ade3_f9b88623","updated":"2022-09-19 06:46:14.000000000","message":"recheck","commit_id":"87abf3baf69f318ac3244e6dca806fae8435ecee"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"cbffa6a4c7d9efcba0f9fd8673f84e0bed3b32c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"9b11a8ee_ef117bf1","updated":"2022-09-19 23:10:52.000000000","message":"recheck","commit_id":"87abf3baf69f318ac3244e6dca806fae8435ecee"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"775bf072d9ab559c397e394c2b074ecdce413a1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"bd2acbe0_81a21394","updated":"2022-10-11 20:44:03.000000000","message":"recheck","commit_id":"7e9229c86d9f46105df20c280a8221ea41f3b4b7"}],"playbooks/bootstrap-bridge.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"230a5be3c4a17b0c8de7ee9f9d7b044f89349fdc","unresolved":true,"context_lines":[{"line_number":1,"context_line":"- hosts: bridge.openstack.org:!disabled"},{"line_number":2,"context_line":"  name: \"Bridge: bootstrap the bastion host\""},{"line_number":3,"context_line":"  become: true"},{"line_number":4,"context_line":"  tasks:"},{"line_number":5,"context_line":"    # Note for production use we expect to take the defaults; unit"},{"line_number":6,"context_line":"    # test jobs override this to test with latest upstream ansible."},{"line_number":7,"context_line":"    # For example, if there is a fix on the ansible stable branch we"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"04db5a99_69f58de5","line":4,"updated":"2022-10-03 22:12:26.000000000","message":"Is there a reason for converting this to a list of include_role tasks rather than keeping this as a roles list?","commit_id":"71eb73f53af20f89e49dcac2b864a70fe0fe48ab"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"df9e237ef87adc224a7e0e44cbbdc0dcb4983030","unresolved":true,"context_lines":[{"line_number":1,"context_line":"- hosts: bridge.openstack.org:!disabled"},{"line_number":2,"context_line":"  name: \"Bridge: bootstrap the bastion host\""},{"line_number":3,"context_line":"  become: true"},{"line_number":4,"context_line":"  tasks:"},{"line_number":5,"context_line":"    # Note for production use we expect to take the defaults; unit"},{"line_number":6,"context_line":"    # test jobs override this to test with latest upstream ansible."},{"line_number":7,"context_line":"    # For example, if there is a fix on the ansible stable branch we"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"f4fecc09_629f2fb7","line":4,"in_reply_to":"04db5a99_69f58de5","updated":"2022-10-10 00:23:28.000000000","message":"I\u0027m not 100% sure but I probably at some point in the stack was inserting debug messages or something inbetween.  I think the task list just keeps it a bit more flexible","commit_id":"71eb73f53af20f89e49dcac2b864a70fe0fe48ab"}],"playbooks/roles/install-ansible/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"230a5be3c4a17b0c8de7ee9f9d7b044f89349fdc","unresolved":true,"context_lines":[{"line_number":134,"context_line":"    name: \u0027{{ install_ansible_openstacksdk_name | default(\"openstacksdk\") }}\u0027"},{"line_number":135,"context_line":"    version: \u0027{{ _install_ansible_openstacksdk_version | default(omit) }}\u0027"},{"line_number":136,"context_line":"    state: \u0027{{ _install_openstacksdk_state | default(omit) }}\u0027"},{"line_number":137,"context_line":"    virtualenv: \u0027/usr/ansible-venv\u0027"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"- name: Ensure /etc/ansible and /etc/ansible/hosts"},{"line_number":140,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"2f9d3d1a_95de01e9","line":137,"updated":"2022-10-03 22:12:26.000000000","message":"Note, corvus discovered when booting the new tracing.opendev.org server that modern openstacksdk does not work against rax. We may end up needing to curate a few of these venvs in order to talk to different clouds. I think this change (and the others in this stack) are a good start towards being able to manage those with our config management.","commit_id":"71eb73f53af20f89e49dcac2b864a70fe0fe48ab"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"df9e237ef87adc224a7e0e44cbbdc0dcb4983030","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    name: \u0027{{ install_ansible_openstacksdk_name | default(\"openstacksdk\") }}\u0027"},{"line_number":135,"context_line":"    version: \u0027{{ _install_ansible_openstacksdk_version | default(omit) }}\u0027"},{"line_number":136,"context_line":"    state: \u0027{{ _install_openstacksdk_state | default(omit) }}\u0027"},{"line_number":137,"context_line":"    virtualenv: \u0027/usr/ansible-venv\u0027"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"- name: Ensure /etc/ansible and /etc/ansible/hosts"},{"line_number":140,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"2be62eec_c2e0158d","line":137,"in_reply_to":"2f9d3d1a_95de01e9","updated":"2022-10-10 00:23:28.000000000","message":"OK, this is good to note.  As you say, I think this is best handled with discrete enivronments -- it will make more sense if we just keep parallel versions (not withstanding that was exactly what openstacksdk was supposed to avoid ... but I do realise the world moves on :)","commit_id":"71eb73f53af20f89e49dcac2b864a70fe0fe48ab"}]}