)]}'
{"zuul.d/system-config-run.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9bb7acbffb53af61f7bee54397c75d233f015d1e","unresolved":true,"context_lines":[{"line_number":118,"context_line":"      - name: github.com/ansible-collections/community.general"},{"line_number":119,"context_line":"        override-checkout: main"},{"line_number":120,"context_line":"    vars:"},{"line_number":121,"context_line":"      install_ansible_ara_enable: true"},{"line_number":122,"context_line":"      install_ansible_requirements:"},{"line_number":123,"context_line":"        # Zuul checkout of Ansible devel"},{"line_number":124,"context_line":"        - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"30072476_228f4d40","line":121,"updated":"2024-07-09 02:02:39.000000000","message":"Doesn\u0027t the existing enable in the `bridge99.opendev.org` variables from `system-config-run-base` do this already?  I don\u0027t think it needs to be a global variable (neither does install_ansible_requirements really i guess either)","commit_id":"6afe1f1fb747cbe272057f31503f34596a124097"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"f5f4cb03f27848a837ddcd73277aa628e253d23c","unresolved":false,"context_lines":[{"line_number":118,"context_line":"      - name: github.com/ansible-collections/community.general"},{"line_number":119,"context_line":"        override-checkout: main"},{"line_number":120,"context_line":"    vars:"},{"line_number":121,"context_line":"      install_ansible_ara_enable: true"},{"line_number":122,"context_line":"      install_ansible_requirements:"},{"line_number":123,"context_line":"        # Zuul checkout of Ansible devel"},{"line_number":124,"context_line":"        - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4646dc23_efe6b386","line":121,"in_reply_to":"27fd82d6_cd3417e0","updated":"2024-07-11 20:36:35.000000000","message":"Done","commit_id":"6afe1f1fb747cbe272057f31503f34596a124097"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"aaf57a82bd2fa9551176eaed19da52048beb2b18","unresolved":true,"context_lines":[{"line_number":118,"context_line":"      - name: github.com/ansible-collections/community.general"},{"line_number":119,"context_line":"        override-checkout: main"},{"line_number":120,"context_line":"    vars:"},{"line_number":121,"context_line":"      install_ansible_ara_enable: true"},{"line_number":122,"context_line":"      install_ansible_requirements:"},{"line_number":123,"context_line":"        # Zuul checkout of Ansible devel"},{"line_number":124,"context_line":"        - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"27fd82d6_cd3417e0","line":121,"in_reply_to":"30072476_228f4d40","updated":"2024-07-09 02:35:43.000000000","message":"It does indeed.\n\nI can remove this and move install_ansible_requirements down to a bridge99 host-var","commit_id":"6afe1f1fb747cbe272057f31503f34596a124097"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9bb7acbffb53af61f7bee54397c75d233f015d1e","unresolved":true,"context_lines":[{"line_number":124,"context_line":"        - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"},{"line_number":125,"context_line":"        - \u0027{{ ansible_user_dir }}/src/opendev.org/openstack/openstacksdk\u0027"},{"line_number":126,"context_line":"        - \u0027tzdata\u0027"},{"line_number":127,"context_line":"        - \u0027ara[server]\u0027"},{"line_number":128,"context_line":"      # These are required because we are not install the pypi"},{"line_number":129,"context_line":"      # \"ansible\" bundle here, but the upstream devel branch"},{"line_number":130,"context_line":"      install_ansible_collections:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f47c2c3d_e8ea8a8c","line":127,"updated":"2024-07-09 02:02:39.000000000","message":"huh, so in Ideb5d6f52a7c471520b5cbba33aa5c9ca4e550e0 we stopped installing ARA from source.  i agree we needed this since then.  Part of the reason for the -devel job was to run _all_ the bits from main checkouts to find early integration problems.\n\nSince the original problem was using a Zuul checkout -- perhaps we should use a `git+https` install here to use the current main branch?  although that\u0027s less reliable, putting a dependency on pulling from github right into the job, as this is a best-effort thing it\u0027s probably enough?","commit_id":"6afe1f1fb747cbe272057f31503f34596a124097"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"5fce7ef214877ff8f744d34306e6ae15831578b7","unresolved":false,"context_lines":[{"line_number":124,"context_line":"        - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"},{"line_number":125,"context_line":"        - \u0027{{ ansible_user_dir }}/src/opendev.org/openstack/openstacksdk\u0027"},{"line_number":126,"context_line":"        - \u0027tzdata\u0027"},{"line_number":127,"context_line":"        - \u0027ara[server]\u0027"},{"line_number":128,"context_line":"      # These are required because we are not install the pypi"},{"line_number":129,"context_line":"      # \"ansible\" bundle here, but the upstream devel branch"},{"line_number":130,"context_line":"      install_ansible_collections:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b42a2782_6c051a2e","line":127,"in_reply_to":"aa7f449f_62baafa5","updated":"2024-07-11 20:35:16.000000000","message":"I think for now.  We\u0027ll just install the released ARA from pypi.  We can add a change to the end of the list as a discussion point for a ARA master vs release","commit_id":"6afe1f1fb747cbe272057f31503f34596a124097"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"aaf57a82bd2fa9551176eaed19da52048beb2b18","unresolved":true,"context_lines":[{"line_number":124,"context_line":"        - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"},{"line_number":125,"context_line":"        - \u0027{{ ansible_user_dir }}/src/opendev.org/openstack/openstacksdk\u0027"},{"line_number":126,"context_line":"        - \u0027tzdata\u0027"},{"line_number":127,"context_line":"        - \u0027ara[server]\u0027"},{"line_number":128,"context_line":"      # These are required because we are not install the pypi"},{"line_number":129,"context_line":"      # \"ansible\" bundle here, but the upstream devel branch"},{"line_number":130,"context_line":"      install_ansible_collections:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"aa7f449f_62baafa5","line":127,"in_reply_to":"f47c2c3d_e8ea8a8c","updated":"2024-07-09 02:35:43.000000000","message":"We could do something like that.  Or we could add tasks to shallow clone the ara repo and install from that.  I don\u0027t have a strong opinion about which is better.  Or if we do need to include ara@master in this job.\n\n\nAs you say that was the original intent, but the change you found implies that it isn\u0027t needed now.","commit_id":"6afe1f1fb747cbe272057f31503f34596a124097"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"0f2c3a80194f0b1ad26a1424fbf5e1e08ac8c67e","unresolved":true,"context_lines":[{"line_number":124,"context_line":"          - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"},{"line_number":125,"context_line":"          - \u0027{{ ansible_user_dir }}/src/opendev.org/openstack/openstacksdk\u0027"},{"line_number":126,"context_line":"          - \u0027ara[server]\u0027"},{"line_number":127,"context_line":"          - \u0027tzdata\u0027"},{"line_number":128,"context_line":"          # These are required because we are not install the pypi"},{"line_number":129,"context_line":"          # \"ansible\" bundle here, but the upstream devel branch"},{"line_number":130,"context_line":"        install_ansible_collections:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5403effd_cd89cf72","line":127,"updated":"2024-07-11 23:05:22.000000000","message":"If this is the case, then maybe we don\u0027t explicitly install ara here,but modify playbooks/roles/install-ansible/tasks/main.yaml to just add the ara packages unconditionally?","commit_id":"51850a92b9c188d0fe514a33afe8ac6ab9e09e0f"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"a8b28bfa4405f3e010d349d9081dad9417710d18","unresolved":true,"context_lines":[{"line_number":124,"context_line":"          - \u0027{{ ansible_user_dir}}/src/github.com/ansible/ansible\u0027"},{"line_number":125,"context_line":"          - \u0027{{ ansible_user_dir }}/src/opendev.org/openstack/openstacksdk\u0027"},{"line_number":126,"context_line":"          - \u0027ara[server]\u0027"},{"line_number":127,"context_line":"          - \u0027tzdata\u0027"},{"line_number":128,"context_line":"          # These are required because we are not install the pypi"},{"line_number":129,"context_line":"          # \"ansible\" bundle here, but the upstream devel branch"},{"line_number":130,"context_line":"        install_ansible_collections:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c8dfad8c_75d0f566","line":127,"in_reply_to":"5403effd_cd89cf72","updated":"2024-07-18 02:45:30.000000000","message":"I think for now, I\u0027d like to keep this here.  We can discuss the intent for ARA in https://review.opendev.org/c/opendev/system-config/+/924012","commit_id":"51850a92b9c188d0fe514a33afe8ac6ab9e09e0f"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1fe7171f698516f70437f916be48e66a76da9145","unresolved":true,"context_lines":[{"line_number":97,"context_line":"      Run the base playbook with the latest ansible."},{"line_number":98,"context_line":"    nodeset:"},{"line_number":99,"context_line":"      nodes:"},{"line_number":100,"context_line":"        - \u003c\u003c: *bridge_node_x86"},{"line_number":101,"context_line":"        - name: focal"},{"line_number":102,"context_line":"          label: ubuntu-focal"},{"line_number":103,"context_line":"        - name: jammy"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9d630a93_37aaa4f7","line":100,"updated":"2024-08-05 07:14:48.000000000","message":"Looking at this again, I\u0027m pretty iffy if installing out-of-tree python versions is really the right way forward for this.  That our bridge bits run correctly on more recent hosts is also the point of this test.\n\nI think what this is telling us is that Jammy (which is the current bridge_node) is not the right thing to be testing this on?  and that\u0027s OK, because the idea is that we\u0027re trying to keep things clean(ish) so that we\u0027re not surprised in the future when stuff doesn\u0027t work.\n\n\"Hey, the latest versions of ansible aren\u0027t going to work on jammy\" is a pretty valid outcome of this test -- and probably giving us a red flag that we should move this testing to noble or whatever platform is realistically going to be the one if we every need to replace the \"real\" bridge?","commit_id":"7449032f76d835815ab2fcd855b71643b859a080"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"22927cbd0507363a7d9ab8aee5d348051e2eb874","unresolved":true,"context_lines":[{"line_number":97,"context_line":"      Run the base playbook with the latest ansible."},{"line_number":98,"context_line":"    nodeset:"},{"line_number":99,"context_line":"      nodes:"},{"line_number":100,"context_line":"        - \u003c\u003c: *bridge_node_x86"},{"line_number":101,"context_line":"        - name: focal"},{"line_number":102,"context_line":"          label: ubuntu-focal"},{"line_number":103,"context_line":"        - name: jammy"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ba696aef_e278bdda","line":100,"in_reply_to":"9d630a93_37aaa4f7","updated":"2024-08-05 22:51:20.000000000","message":"One small clarification, this isn\u0027t installing a python packaged and shipped by canonical which doesn\u0027t match my view of \"out of tree\".  That being said.\n\nIan has a valid and fair view.  I don\u0027t have a view on the the right solution.  Use Python 3.11 on Jammy or switch to Noble","commit_id":"7449032f76d835815ab2fcd855b71643b859a080"}]}