)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fae98b65bce6ff35b5404e2ca0a97569d5fa71e5","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Zuul will be removing support for ansible 2.8 and 2.9 in the near"},{"line_number":10,"context_line":"future. This changes removes the pins of jobs to those versions which"},{"line_number":11,"context_line":"will cause the jobs to run against the current zuul tenant default"},{"line_number":12,"context_line":"ansible (version 5). Eventually this will update to Ansible version 6"},{"line_number":13,"context_line":"and so on."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I410ff0205a317a7655657a8acb6a95c3b61144d8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4b963004_855d2b56","line":12,"range":{"start_line":12,"start_character":9,"end_line":12,"end_character":18},"updated":"2022-09-22 05:14:08.000000000","message":"Zuul tenant default ansible version.\nhttps://opendev.org/openstack/project-config/src/branch/master/zuul/main.yaml#L89\n```\ndefault-ansible-version: \u00275\u0027\n```\nand\nfrom logs:\nhttps://d6341d649c554a5c4ee6-1f83b5006b42c57e1aaf7265351cee4a.ssl.cf5.rackcdn.com/858768/1/check/tripleo-ci-centos-9-scenario001-standalone/82786f2/zuul-info/inventory.yaml\n```\n      ansible_version: \u00275\u0027\n```","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"33c77d95538f5b7ae72c3242bc6c07078109b29b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4fb7764d_e4524c6d","updated":"2022-09-22 02:24:46.000000000","message":"+1 to remove the pin and took zuul defaults, but I have a query, commented inline:-","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"672f4e1b55a0892160925e12a9a725cbc716321f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c48c6882_89f334ac","updated":"2022-09-22 07:00:22.000000000","message":"adding tentative -1 for my previous comment about updating the pin to a supported version instead of removing it altogether","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"b62959737647bad0c9c19eea561de9704ee27c8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"181c4d1e_549fcc5c","updated":"2022-09-22 15:57:00.000000000","message":"https://review.opendev.org/c/openstack/tripleo-heat-templates/+/858889/ looks good","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"f25cbdf6f8e9310989ee668ef03f717517af2294","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5ba4a8cc_be3ca9ab","updated":"2022-09-23 05:27:03.000000000","message":"recheck \n\n\nhttps://309a1cbcb2627a3c0a2a-85473db09f2bbfe700fdb13c6018230b.ssl.cf1.rackcdn.com/858768/1/gate/tripleo-ci-centos-9-standalone-on-multinode-ipa/67d6f60/job-output.txt\n\n2022-09-22 17:57:59.883032 | primary | TASK [os_tempest : Execute tempest tests] **************************************\n2022-09-22 17:57:59.883214 | primary | Thursday 22 September 2022  17:57:59 +0000 (0:00:00.050)       0:37:20.400 ****\n2022-09-22 18:10:41.340479 | primary | fatal: [undercloud]: FAILED! \u003d\u003e {\"changed\": false, \"cmd\": \"set -e\\nif [ -d /openstack/venvs/tempest-untagged/bin ];\\nthen\\n. /openstack/venvs/tempest-untagged/bin/activate\\nfi\\ntempest run   --concurrency 2   --exclude-list /home/zuul/tempest/etc/tempest_excludelist.txt   --include-list /home/zuul/tempest/etc/tempest_includelist.txt \u003e /var/log/tempest/tempest_run.log\\n\", \"delta\": \"0:12:38.851104\", \"end\": \"2022-09-22 18:10:40.619737\", \"msg\": \"non-zero return code\", \"rc\": 1, \"start\": \"2022-09-22 17:58:01.768633\", \"stderr\": \"\", \"stderr_lines\": [], \"stdout\": \"\", \"stdout_lines\": []}","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"891e3f073720849b2a70c7ea436398487754d39f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d534e974_6b5b5133","updated":"2022-09-22 11:20:46.000000000","message":"removing -1 as we are discussing with the ci team right now ","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"832dcb94632281cf71a9c7ec24d85f7a8b89f8e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f4f66b45_eca2c0e6","updated":"2022-09-22 11:29:36.000000000","message":"testing train and wallaby - then will approve","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e292123281d74d9bfaa21174ec03950a16a9e6bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0432c904_2cc7ee2f","in_reply_to":"f4f66b45_eca2c0e6","updated":"2022-09-22 11:32:06.000000000","message":"tests there\nhttps://review.opendev.org/c/openstack/tripleo-heat-templates/+/858888\nhttps://review.opendev.org/c/openstack/tripleo-heat-templates/+/858889","commit_id":"1a953737629406e021f92cfe357883f30a19ea68"}],"zuul.d/base.yaml":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"33c77d95538f5b7ae72c3242bc6c07078109b29b","unresolved":true,"context_lines":[{"line_number":54,"context_line":"    parent: tripleo-ci-base-common-required-projects"},{"line_number":55,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":56,"context_line":"    ansible-version: 2.8"},{"line_number":57,"context_line":"    description: |"},{"line_number":58,"context_line":"        Base abstract job for all TripleO CI zuulv3 jobs"},{"line_number":59,"context_line":"    timeout: 10800"},{"line_number":60,"context_line":"    post-timeout: 3600"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5419df0d_05f222c8","side":"PARENT","line":57,"updated":"2022-09-22 02:24:46.000000000","message":"I was looking at one of the jobs which ran in check for this patch:-\n\ntripleo-ci-centos-9-scenario001-standalone which has tripleo-ci-base-standard as indirect parent, checked via https://zuul.openstack.org/jobs\n\n~~~\ntripleo-ci-base-standardBase abstract job for all TripleO CI zuulv3 jobs\ntripleo-ci-base-standalone-standardBase abstract job for standalone TripleO CI zuulv3 jobs\ntripleo-ci-base-required-projects-standalone-upstreamBase abstract job to add required-projects for Upstream Standalone jobs\ntripleo-ci-base-standalone-centos-9Base abstract job for standalone TripleO CI Centos-9 zuulv3 jobs.\ntripleo-ci-centos-9-scenario001-standalone (branched)\n~~~\n\n\nThe jobs is now picking Ansible Version: 2.12.9\n\nhttps://d6341d649c554a5c4ee6-1f83b5006b42c57e1aaf7265351cee4a.ssl.cf5.rackcdn.com/858768/1/check/tripleo-ci-centos-9-scenario001-standalone/82786f2/job-output.txt\n~~~\n2022-09-21 16:42:35.357909 | TASK [emit-job-header : Print job information]\n2022-09-21 16:42:35.420589 | # Job Information\n2022-09-21 16:42:35.420903 | Ansible Version: 2.12.9\n~~~\n\n\u003e\u003e based on the commit - This changes removes the pins of jobs to those versions which will cause the jobs to run against the current zuul tenant default ansible (version 5) \n\nWant to crosscheck were we expecting to see V5 here now and not Ansible Version: 2.12.9?","commit_id":"d9d9747203f233a747d1f9dd9e7aa2c6ac88aa93"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fae98b65bce6ff35b5404e2ca0a97569d5fa71e5","unresolved":true,"context_lines":[{"line_number":54,"context_line":"    parent: tripleo-ci-base-common-required-projects"},{"line_number":55,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":56,"context_line":"    ansible-version: 2.8"},{"line_number":57,"context_line":"    description: |"},{"line_number":58,"context_line":"        Base abstract job for all TripleO CI zuulv3 jobs"},{"line_number":59,"context_line":"    timeout: 10800"},{"line_number":60,"context_line":"    post-timeout: 3600"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"aa543e88_a99fe5c0","side":"PARENT","line":57,"in_reply_to":"5419df0d_05f222c8","updated":"2022-09-22 05:14:08.000000000","message":"I have added links for the zuul tenant default version in the commit message comment. I also have the same question from where Ansible Version: 2.12.9 is coming?","commit_id":"d9d9747203f233a747d1f9dd9e7aa2c6ac88aa93"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a008628cfd600da826a93b73247a291766de00a7","unresolved":true,"context_lines":[{"line_number":54,"context_line":"    parent: tripleo-ci-base-common-required-projects"},{"line_number":55,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":56,"context_line":"    ansible-version: 2.8"},{"line_number":57,"context_line":"    description: |"},{"line_number":58,"context_line":"        Base abstract job for all TripleO CI zuulv3 jobs"},{"line_number":59,"context_line":"    timeout: 10800"},{"line_number":60,"context_line":"    post-timeout: 3600"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"509ff5f5_f4ecf9f8","side":"PARENT","line":57,"in_reply_to":"8162135c_35c5ffd7","updated":"2022-09-22 14:42:47.000000000","message":"The version info that ansible reports is confusing and Zuul has had to do its best here. The `ansible` package which includes `ansible-core` and other useful collections had versions like 2.8.x, 2.9.x, 4.x, 5.x, and now 6.x. This is what Zuul installs into its ansible runtime environments for jobs. This is why we use those versions for specifying the runtime in the Zuul config. But when you ask ansible to report its version it reports the ansible-core version which is 2.8.x, 2.9.x, 2.10.x and so on. In this case ansible 5.x maps to ansible-core 2.12.x.\n\nAs for your package requirements they should be completely independent of this. This attribute describes the ansible version zuul should use to execute your test jobs. Then within that test job you can manage your package dependencies in a separate ansible install however you like.\n\nI would advice against strictly managing this in your jobs. No one else is doing it, and it hasn\u0027t appeared to be necessary. Instead it creates this unnecessary headache when Zuul drops old versions. You should only set ansible-version in your zuul jobs if there is a strict requirement for that version in your jobs for some reason.","commit_id":"d9d9747203f233a747d1f9dd9e7aa2c6ac88aa93"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"d6a991d01758f561508b38bc145429aa7fafcc67","unresolved":true,"context_lines":[{"line_number":54,"context_line":"    parent: tripleo-ci-base-common-required-projects"},{"line_number":55,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":56,"context_line":"    ansible-version: 2.8"},{"line_number":57,"context_line":"    description: |"},{"line_number":58,"context_line":"        Base abstract job for all TripleO CI zuulv3 jobs"},{"line_number":59,"context_line":"    timeout: 10800"},{"line_number":60,"context_line":"    post-timeout: 3600"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"8162135c_35c5ffd7","side":"PARENT","line":57,"in_reply_to":"aa543e88_a99fe5c0","updated":"2022-09-22 06:59:49.000000000","message":"i don\u0027t have an answer for the where are we getting it from but it seems we have related pins in requirements.txt/test-requirements across tripleo repos eg [1][2] \n\nI think we should consider updating our pin to a supported version, rather than removing it altogether so we are using a version of ansible that the tripleo repos expect. \n\n\n[1] https://opendev.org/openstack/tripleo-ansible/src/commit/7898061a317d38cee421e5d185538589ed274ac0/ansible-requirements.txt#L1\n\n[2] https://opendev.org/openstack/tripleo-heat-templates/src/commit/db8e72f9fc3ea4ff3dc9bb12d058543987c71c3d/test-ansible-requirements.txt#L5","commit_id":"d9d9747203f233a747d1f9dd9e7aa2c6ac88aa93"}]}