)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"36396aa290ff6d61d475020a72b27c5f465ea1fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"078b0864_9b11def3","updated":"2021-10-13 08:48:43.000000000","message":"and rebasing now the parent is merged ","commit_id":"2dfe74e2ed1fd62867b804bdfd57cf4f1f0718f8"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"522220e9e77c660b6ba4c38ee42642d2d93b53a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"3f6d33e7_9f467f94","updated":"2021-10-14 11:39:40.000000000","message":"Downstream checked!","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"708ab13873b5ccd2c3edef5239e68f27a474ac70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"e068a25e_cfc66b94","updated":"2021-10-14 14:56:38.000000000","message":"Few comments in line  ok to merge as is and fix afterwards","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c164b57cc4858c139e7b8648495e16fe59ecae81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"63dee898_2d0796a3","updated":"2021-10-14 06:00:54.000000000","message":"Looks good!\nhttps://review.rdoproject.org/r/c/testproject/+/35465\n\nand\n\ntripleo-stream9-development-buildimage-overcloud-hardened-uefi-full-master\tSUCCESS 31m 34s\ntripleo-stream9-development-buildimage-overcloud-full-master\tSUCCESS 32m 30s\ntripleo-stream9-development-buildimage-ironic-python-agent-master\tSUCCESS 19m 09s","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"110eb59ba9320ca4a4340c1d36c08cff4a213f78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"974cb17c_d06dd841","updated":"2021-10-14 10:41:14.000000000","message":"Looks good.","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1aad985a4dc362c15bfdc1ee46d9a3e0b41e30f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"20460e74_f6c140c4","updated":"2021-10-19 09:09:59.000000000","message":"thanks for review sagi\n\nsince we\u0027re waiting for depends on anyway  lets try your suggestion i think it should be OK\n","commit_id":"1d33d8db187f8d628cb797c5d7bb20067202897a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"68d074e5f626e5b08a10fd263c04f4855e58fed9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"c175dd7f_6e9213a6","updated":"2021-10-19 10:25:04.000000000","message":"Look Sane, Thank you Marios for Picking it up :-)","commit_id":"86274966bf8f2722b8233f9999f0edeb579e4260"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c97b99af82d1508ad059083cecfbdfe3267d19a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"be13734d_2d013486","updated":"2021-10-20 15:51:34.000000000","message":"recheck","commit_id":"86274966bf8f2722b8233f9999f0edeb579e4260"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"00e56e6c3ee762a22750918d84d0fc8214427028","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"0b5eaf82_54550d55","updated":"2021-10-19 12:04:32.000000000","message":"recheck \n\ntest looks ok @ https://review.rdoproject.org/r/c/testproject/+/35465/14#message-cdf6b236123b9c1182e15b0f1acc64d7874150dc\n\nfailed here for repos issue?\n\n        * https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f71/808177/18/check/tripleo-tox-molecule/f71686a/job-output.txt\n        * 2021-10-19 09:33:17.044720 | primary | \u001b[31mFailed to download metadata for repo \u0027tripleo-centos-powertools\u0027: Yum repo downloading error: Downloading error(s): repodata/09c4e2d68abe42e93672468b1683deb335ff4b2ec94c29658e3828282767d1c9-primary.xml.gz - Cannot download, all mirrors were already tried without success; repodata/22749fd3ae87bcbfa017a39ff7dfae45e3e3b2caa07dda57ff536795a3662105-filelists.xml.gz - Cannot download, all mirrors were already tried without success; repodata/ffaa722867a37a786b1e1c62d567b4c07b6060ac7385ab7a0e432b29e7587858-comps-PowerTools.x86_64.xml - Cannot download, all mirrors were already tried without success; repodata/1c7d95defcd30519c22300f0359a3232c538ec8e2cc76eee4c9333a604c84aef-modules.yaml.xz - Cannot download, all mirrors were already tried without success\u001b[0m","commit_id":"86274966bf8f2722b8233f9999f0edeb579e4260"}],"playbooks/tripleo-buildimages/pre.yaml":[{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"b614eab91d78660d4364b7d19fa3c32571239420","unresolved":true,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    - name: Include EL9 taks"},{"line_number":31,"context_line":"      when: ansible_distribution_major_version is version(9, \u0027\u003d\u003d\u0027)"},{"line_number":32,"context_line":"      include: pre9.yaml"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    - name: Include bindep role"},{"line_number":35,"context_line":"      include_role:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"1ecf722e_ad81f9be","line":32,"updated":"2021-10-06 20:03:25.000000000","message":"I know this was a pattern used in the containers build but idk if keeping a 9 pre is the best way. What happens when we hit 10?","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9d7cee6879c8a148ccaca1e42f51e019469a85a0","unresolved":true,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    - name: Include EL9 taks"},{"line_number":31,"context_line":"      when: ansible_distribution_major_version is version(9, \u0027\u003d\u003d\u0027)"},{"line_number":32,"context_line":"      include: pre9.yaml"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    - name: Include bindep role"},{"line_number":35,"context_line":"      include_role:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"b26b1c28_0518574a","line":32,"in_reply_to":"1ecf722e_ad81f9be","updated":"2021-10-07 11:53:11.000000000","message":"well the pre9.yaml here is for including the workaround repos like the HA repo and delorean/depos from centos9-master-preview\nper your comments below sounds like we are ready to switch over so we might just remove this special case here altogether\n\nwill try on the next run and see what happens.","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"36396aa290ff6d61d475020a72b27c5f465ea1fb","unresolved":true,"context_lines":[{"line_number":25,"context_line":"      vars:"},{"line_number":26,"context_line":"        override_repos: \"{{ build_override_repos | default(\u0027\u0027) }}\""},{"line_number":27,"context_line":"        tripleo_repos_repository: \"{{ ansible_user_dir }}/{{ zuul.projects[\u0027opendev.org/openstack/tripleo-repos\u0027].src_dir }}\""},{"line_number":28,"context_line":"      when: ansible_distribution_major_version is version(8, \u0027\u003d\u003d\u0027)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    - name: Include EL9 taks"},{"line_number":31,"context_line":"      when: ansible_distribution_major_version is version(9, \u0027\u003d\u003d\u0027)"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"4ea0cf56_c75b045a","line":28,"range":{"start_line":28,"start_character":62,"end_line":28,"end_character":63},"updated":"2021-10-13 08:48:43.000000000","message":"this should be \u003c\u003d for the c7 jobs","commit_id":"2dfe74e2ed1fd62867b804bdfd57cf4f1f0718f8"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"af71ff398aad7624a24001f37b57c9daa6d7aa78","unresolved":true,"context_lines":[{"line_number":24,"context_line":"        name: tripleo-repos"},{"line_number":25,"context_line":"      vars:"},{"line_number":26,"context_line":"        override_repos: \"{{ build_override_repos | default(\u0027\u0027) }}\""},{"line_number":27,"context_line":"        tripleo_repos_repository: \"{{ ansible_user_dir }}/{{ zuul.projects[\u0027opendev.org/openstack/tripleo-repos\u0027].src_dir }}\""},{"line_number":28,"context_line":"      when: ansible_distribution_major_version is version(8, \u0027\u003c\u003d\u0027)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    - name: Include EL9 taks"},{"line_number":31,"context_line":"      when: ansible_distribution_major_version is version(9, \u0027\u003d\u003d\u0027)"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"1bf50a27_1121dad8","line":28,"range":{"start_line":27,"start_character":1,"end_line":28,"end_character":66},"updated":"2021-10-18 09:23:26.000000000","message":"we can move this task after \"Include EL9 tasks\" and remove this condition","commit_id":"1d33d8db187f8d628cb797c5d7bb20067202897a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1aad985a4dc362c15bfdc1ee46d9a3e0b41e30f1","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        name: tripleo-repos"},{"line_number":25,"context_line":"      vars:"},{"line_number":26,"context_line":"        override_repos: \"{{ build_override_repos | default(\u0027\u0027) }}\""},{"line_number":27,"context_line":"        tripleo_repos_repository: \"{{ ansible_user_dir }}/{{ zuul.projects[\u0027opendev.org/openstack/tripleo-repos\u0027].src_dir }}\""},{"line_number":28,"context_line":"      when: ansible_distribution_major_version is version(8, \u0027\u003c\u003d\u0027)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    - name: Include EL9 taks"},{"line_number":31,"context_line":"      when: ansible_distribution_major_version is version(9, \u0027\u003d\u003d\u0027)"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"b0e89fbc_366814f4","line":28,"range":{"start_line":27,"start_character":1,"end_line":28,"end_character":66},"in_reply_to":"1bf50a27_1121dad8","updated":"2021-10-19 09:09:59.000000000","message":"Ack","commit_id":"1d33d8db187f8d628cb797c5d7bb20067202897a"}],"playbooks/tripleo-buildimages/pre9.yaml":[{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"b614eab91d78660d4364b7d19fa3c32571239420","unresolved":true,"context_lines":[{"line_number":10,"context_line":"    gpgcheck: false"},{"line_number":11,"context_line":"    gpgkey:"},{"line_number":12,"context_line":"      - file:///usr/share/distribution-gpg-keys/centos/RPM-GPG-KEY-CentOS-Official"},{"line_number":13,"context_line":"    enabled: true"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- name: get extra repos delorean and deps"},{"line_number":16,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"57aefad6_87d1acf0","line":13,"updated":"2021-10-06 20:03:25.000000000","message":"We don\u0027t need this anymore - HA repo should be laid down with the rest of the centos repos","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"b614eab91d78660d4364b7d19fa3c32571239420","unresolved":true,"context_lines":[{"line_number":20,"context_line":"  loop:"},{"line_number":21,"context_line":"    - \"https://trunk.rdoproject.org/centos9-master-preview/{{ promote_source | default(\u0027consistent\u0027) }}/delorean.repo\""},{"line_number":22,"context_line":"    - https://trunk.rdoproject.org/centos9-master-preview/delorean-deps.repo"},{"line_number":23,"context_line":"  changed_when: true"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Add missing /etc/pki/rpm-gpg dir"},{"line_number":26,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"d939669e_a050f35a","line":23,"updated":"2021-10-06 20:03:25.000000000","message":"we centos9-master  - we don\u0027t have to use centos9-master-preview","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9d7cee6879c8a148ccaca1e42f51e019469a85a0","unresolved":true,"context_lines":[{"line_number":20,"context_line":"  loop:"},{"line_number":21,"context_line":"    - \"https://trunk.rdoproject.org/centos9-master-preview/{{ promote_source | default(\u0027consistent\u0027) }}/delorean.repo\""},{"line_number":22,"context_line":"    - https://trunk.rdoproject.org/centos9-master-preview/delorean-deps.repo"},{"line_number":23,"context_line":"  changed_when: true"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"- name: Add missing /etc/pki/rpm-gpg dir"},{"line_number":26,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"91e6521a_e9d02962","line":23,"in_reply_to":"d939669e_a050f35a","updated":"2021-10-07 11:53:11.000000000","message":"k i will try and remove this and see how we go without special casing 9","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"708ab13873b5ccd2c3edef5239e68f27a474ac70","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    cmd: |"},{"line_number":6,"context_line":"      dnf config-manager --add-repo {{ item }}"},{"line_number":7,"context_line":"  loop:"},{"line_number":8,"context_line":"    - \"https://trunk.rdoproject.org/centos9-master/{{ promote_source | default(\u0027consistent\u0027) }}/delorean.repo\""},{"line_number":9,"context_line":"    - https://trunk.rdoproject.org/centos9-master/delorean-deps.repo"},{"line_number":10,"context_line":"  changed_when: true"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"3ebb4abd_1e81f517","line":8,"updated":"2021-10-14 14:56:38.000000000","message":"centos9-{{release}} - going to need wallaby here as well","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"8c66e81e61f0c8c8d2a1c70b6ccd10a89dcd9507","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    cmd: |"},{"line_number":6,"context_line":"      dnf config-manager --add-repo {{ item }}"},{"line_number":7,"context_line":"  loop:"},{"line_number":8,"context_line":"    - \"https://trunk.rdoproject.org/centos9-master/{{ promote_source | default(\u0027consistent\u0027) }}/delorean.repo\""},{"line_number":9,"context_line":"    - https://trunk.rdoproject.org/centos9-master/delorean-deps.repo"},{"line_number":10,"context_line":"  changed_when: true"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"6b0d922f_4899d2e0","line":8,"in_reply_to":"03349eb5_1ddfee40","updated":"2021-10-15 11:50:14.000000000","message":"more info on \u0027fails image_sanity\u0027:\n\n\nOn the overcloud-full job it fails/hangs during the pxe boot\n\nhttps://review.rdoproject.org/r/c/testproject/+/35465/11#message-a6f9dffd6cbb257a3c070ed893e93485a4744488\nhttps://logserver.rdoproject.org/65/35465/11/check/tripleo-stream9-development-buildimage-overcloud-full-master/651d0d0/job-output.txt\n\n{code:java}\n2021-10-14 04:45:21.186188 | primary | \\x1bc\\x1b[?7l\\x1b[2J\\x1b[0mSeaBIOS (version 1.14.0-6.el9)\n2021-10-14 04:45:21.208626 | primary |\n2021-10-14 04:45:21.208659 | primary |\n2021-10-14 04:45:21.224878 | primary | iPXE (http://ipxe.org) 00:05.0 C000 PCI2.10 PnP PMM+7FF8C4C0+7FECC4C0 C000\n2021-10-14 04:45:21.243473 | primary | Press Ctrl-B to configure iPXE (PCI 00:05.0)...\n2021-10-14 04:45:21.243491 | primary |\n2021-10-14 04:45:21.243497 | primary |\n2021-10-14 09:53:37.532063 | RUN END RESULT_TIMED_OUT: [untrusted : opendev.org/openstack/tripleo-ci/playbooks/tripleo-buildimages/run-v3.yaml@master]\n{code}\n\nIf there is some problem with libguestfs/virt-customize for C9 then this is less of a problem.\nSo we really need a test job that uses the build image - trying to put fs1 job there: \n\n https://review.rdoproject.org/r/c/testproject/+/36267 Test c9 fs1 job with image and containers build [","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"102961c55c057e5b0ad7c4d6256eeb8a9892d989","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    cmd: |"},{"line_number":6,"context_line":"      dnf config-manager --add-repo {{ item }}"},{"line_number":7,"context_line":"  loop:"},{"line_number":8,"context_line":"    - \"https://trunk.rdoproject.org/centos9-master/{{ promote_source | default(\u0027consistent\u0027) }}/delorean.repo\""},{"line_number":9,"context_line":"    - https://trunk.rdoproject.org/centos9-master/delorean-deps.repo"},{"line_number":10,"context_line":"  changed_when: true"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"03349eb5_1ddfee40","line":8,"in_reply_to":"3ebb4abd_1e81f517","updated":"2021-10-15 11:31:13.000000000","message":"ack may as well since we are waiting on the depends-on will update\n\nam trying to dig at the failing image_sanity - really we want a test job using the built images ... will try and post a testproject for that but not sure it can work yet lets see what i\u0027ll miss ;) will add links in a bit\n\n\nfailing image sanity there: https://logserver.rdoproject.org/65/35465/11/check/tripleo-stream9-development-buildimage-overcloud-full-master/651d0d0/job-output.txt","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"708ab13873b5ccd2c3edef5239e68f27a474ac70","unresolved":true,"context_lines":[{"line_number":6,"context_line":"      dnf config-manager --add-repo {{ item }}"},{"line_number":7,"context_line":"  loop:"},{"line_number":8,"context_line":"    - \"https://trunk.rdoproject.org/centos9-master/{{ promote_source | default(\u0027consistent\u0027) }}/delorean.repo\""},{"line_number":9,"context_line":"    - https://trunk.rdoproject.org/centos9-master/delorean-deps.repo"},{"line_number":10,"context_line":"  changed_when: true"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Enable supported container tools version"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"094fbcfc_ae9c5220","line":9,"updated":"2021-10-14 14:56:38.000000000","message":"same here","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"102961c55c057e5b0ad7c4d6256eeb8a9892d989","unresolved":false,"context_lines":[{"line_number":6,"context_line":"      dnf config-manager --add-repo {{ item }}"},{"line_number":7,"context_line":"  loop:"},{"line_number":8,"context_line":"    - \"https://trunk.rdoproject.org/centos9-master/{{ promote_source | default(\u0027consistent\u0027) }}/delorean.repo\""},{"line_number":9,"context_line":"    - https://trunk.rdoproject.org/centos9-master/delorean-deps.repo"},{"line_number":10,"context_line":"  changed_when: true"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Enable supported container tools version"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"f6d847c8_8f004bac","line":9,"in_reply_to":"094fbcfc_ae9c5220","updated":"2021-10-15 11:31:13.000000000","message":"Ack","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"110eb59ba9320ca4a4340c1d36c08cff4a213f78","unresolved":true,"context_lines":[{"line_number":15,"context_line":"     {{ ansible_pkg_mgr }} module switch-to container-tools:latest -y;"},{"line_number":16,"context_line":"     {{ ansible_pkg_mgr }} clean metadata"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"- name: Install vitualenv"},{"line_number":19,"context_line":"  become: true"},{"line_number":20,"context_line":"  package:"},{"line_number":21,"context_line":"    name: python3-virtualenv"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"3a32df44_83affe95","line":18,"range":{"start_line":18,"start_character":16,"end_line":18,"end_character":25},"updated":"2021-10-14 10:41:14.000000000","message":"typo.","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"af71ff398aad7624a24001f37b57c9daa6d7aa78","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    name: python3-virtualenv"},{"line_number":22,"context_line":"    state: latest"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: Include tripleo-repos"},{"line_number":25,"context_line":"  include_role:"},{"line_number":26,"context_line":"    name: tripleo-repos"},{"line_number":27,"context_line":"  vars:"},{"line_number":28,"context_line":"    override_repos: \"{{ buildcontainers_override_repos | default(\u0027\u0027) }}\""},{"line_number":29,"context_line":"    tripleo_repos_repository: \"{{ ansible_user_dir }}/{{ zuul.projects[\u0027opendev.org/openstack/tripleo-repos\u0027].src_dir }}\""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"9a3e4318_3bcaa032","line":29,"range":{"start_line":24,"start_character":0,"end_line":29,"end_character":121},"updated":"2021-10-18 09:23:26.000000000","message":"this seems exactly as task in main.\nWhat if we include here only centos9 tasks and will run tripleo-repos in main for all centos?","commit_id":"1d33d8db187f8d628cb797c5d7bb20067202897a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1aad985a4dc362c15bfdc1ee46d9a3e0b41e30f1","unresolved":true,"context_lines":[{"line_number":21,"context_line":"    name: python3-virtualenv"},{"line_number":22,"context_line":"    state: latest"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"- name: Include tripleo-repos"},{"line_number":25,"context_line":"  include_role:"},{"line_number":26,"context_line":"    name: tripleo-repos"},{"line_number":27,"context_line":"  vars:"},{"line_number":28,"context_line":"    override_repos: \"{{ buildcontainers_override_repos | default(\u0027\u0027) }}\""},{"line_number":29,"context_line":"    tripleo_repos_repository: \"{{ ansible_user_dir }}/{{ zuul.projects[\u0027opendev.org/openstack/tripleo-repos\u0027].src_dir }}\""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"a63f84f0_1692869b","line":29,"range":{"start_line":24,"start_character":0,"end_line":29,"end_character":121},"in_reply_to":"9a3e4318_3bcaa032","updated":"2021-10-19 09:09:59.000000000","message":"ok lets try it","commit_id":"1d33d8db187f8d628cb797c5d7bb20067202897a"}],"roles/oooci-build-images/tasks/main.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"ca9f50018923317109d76da46eee26635097350c","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    - ansible_distribution_major_version \u003d\u003d \u00277\u0027"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"- name: Include OS specific variables"},{"line_number":12,"context_line":"  include_vars: centos-9.yaml"},{"line_number":13,"context_line":"  when: ansible_distribution_major_version \u003d\u003d \u00279\u0027"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- name: Make sure logs directory exists"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"4ee13aab_81070cf0","line":12,"updated":"2021-10-06 08:27:52.000000000","message":"going to try combining this with the previous task","commit_id":"9f0e4af3a9991540d7be0d0b94a26a9b1469bd89"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"b614eab91d78660d4364b7d19fa3c32571239420","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  when:"},{"line_number":8,"context_line":"    - ansible_distribution \u003d\u003d \"CentOS\""},{"line_number":9,"context_line":"    - ansible_distribution_major_version is version(\u00277\u0027, \u0027\u003d\u003d\u0027) or"},{"line_number":10,"context_line":"      ansible_distribution_major_version is version(\u00279\u0027, \u0027\u003d\u003d\u0027)"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Make sure logs directory exists"},{"line_number":13,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"666b3bff_f5665b55","line":10,"updated":"2021-10-06 20:03:25.000000000","message":"hmmm - so 8 is default?","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9d7cee6879c8a148ccaca1e42f51e019469a85a0","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  when:"},{"line_number":8,"context_line":"    - ansible_distribution \u003d\u003d \"CentOS\""},{"line_number":9,"context_line":"    - ansible_distribution_major_version is version(\u00277\u0027, \u0027\u003d\u003d\u0027) or"},{"line_number":10,"context_line":"      ansible_distribution_major_version is version(\u00279\u0027, \u0027\u003d\u003d\u0027)"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"- name: Make sure logs directory exists"},{"line_number":13,"context_line":"  file:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"3e0602f4_54da251c","line":10,"in_reply_to":"666b3bff_f5665b55","updated":"2021-10-07 11:53:11.000000000","message":"yeah i think for now, at least until we have things working. i know at some point we will switch to 9 being default but i don\u0027t think that is here yet","commit_id":"83ca03188b28011899ab146234179c417dce1228"}],"roles/oooci-build-images/tasks/pre.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"037da05ae1803a0cb059d9a87f51a24a85827efe","unresolved":true,"context_lines":[{"line_number":97,"context_line":"      pip:"},{"line_number":98,"context_line":"        name: setuptools"},{"line_number":99,"context_line":"        virtualenv: \"{{ workspace }}/.tmp\""},{"line_number":100,"context_line":"        virtualenv_command: \"virtualenv-3\""},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"    - name: Borrow activate_this.py"},{"line_number":103,"context_line":"      copy:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"bf629c4b_6550c6b7","side":"PARENT","line":100,"range":{"start_line":100,"start_character":39,"end_line":100,"end_character":41},"updated":"2021-10-13 11:00:58.000000000","message":"this breaks the c7 job will have to add conditional\n\nhttps://9d2ae92fac454099dddf-680a475b84522975485d6aab5feabd34.ssl.cf1.rackcdn.com/808177/15/check/tripleo-buildimage-overcloud-full-centos-7-train/b7ab873/job-output.txt\n\n2021-10-13 09:57:21.650808 | TASK [oooci-build-images : Create temp venv w/ virtualenv command]\n2021-10-13 09:57:22.693886 | primary | ERROR\n2021-10-13 09:57:22.694254 | primary | {\n2021-10-13 09:57:22.694381 | primary |   \"msg\": \"Failed to find required executable virtualenv in paths: /usr/local/bin:/usr/bin:/sbin:/usr/sbin:/usr/local/sbin\"\n2021-10-13 09:57:22.694461 | primary | }","commit_id":"5e294fc0975b07f5ad04d5be52692d8515fe531e"}],"roles/oooci-build-images/vars/centos-7.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6e91b487744f306c05e2eb2be39461560f6adf9e","unresolved":true,"context_lines":[{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"7\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"b649c27b_ac974f8c","line":8,"range":{"start_line":8,"start_character":24,"end_line":8,"end_character":25},"updated":"2021-09-30 08:41:53.000000000","message":"whaaa?","commit_id":"e23430d2204c894209d1d5be8ac4e53b3d958b1f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"a5bc605866d2ae772012e30ad3c4026729116d1a","unresolved":true,"context_lines":[{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"7\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"8308d551_d929401f","line":8,"range":{"start_line":8,"start_character":24,"end_line":8,"end_character":25},"in_reply_to":"b649c27b_ac974f8c","updated":"2021-09-30 08:43:57.000000000","message":"Ah Nice catch! Got added by mistake. we can remove it as got added here https://review.opendev.org/c/openstack/tripleo-ci/+/808177/4/roles/oooci-build-images/vars/centos-9.yaml#8","commit_id":"e23430d2204c894209d1d5be8ac4e53b3d958b1f"}],"roles/oooci-build-images/vars/centos-9.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"5823c749f0f1c5401c04dab9ca8045962f158a65","unresolved":true,"context_lines":[{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/centos*"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"9\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"container-tools:latest\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"37c9617c_d8884456","line":6,"range":{"start_line":6,"start_character":13,"end_line":6,"end_character":16},"updated":"2021-09-30 08:53:48.000000000","message":"also going to update this to be \u00279-stream\u0027 per comments at https://review.opendev.org/c/openstack/diskimage-builder/+/806819/3/diskimage_builder/elements/centos/root.d/10-centos-cloud-image","commit_id":"e23430d2204c894209d1d5be8ac4e53b3d958b1f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c5233c2045a4e6c4ac3183341bcc8542253158ef","unresolved":true,"context_lines":[{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/centos*"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"9\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"container-tools:latest\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ce148958_387ad40f","line":6,"range":{"start_line":6,"start_character":13,"end_line":6,"end_character":16},"in_reply_to":"37c9617c_d8884456","updated":"2021-10-01 08:52:21.000000000","message":"nope we keep it as \u00279\u0027 see https://review.opendev.org/c/openstack/diskimage-builder/+/806819/3..4#message-ba8f3ccdd4fb90aee0196141cea8d383198d8b19","commit_id":"e23430d2204c894209d1d5be8ac4e53b3d958b1f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"592014e8df6684ff6f7c46a9a6bf7b71a5578e82","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"tripleo_image_source: https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Stream-GenericCloud-9-20210830.0.x86_64.qcow2"},{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/centos9.repo"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"9\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"container-tools:latest\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"4ab8e020_48aed4c0","line":5,"range":{"start_line":3,"start_character":16,"end_line":5,"end_character":28},"updated":"2021-10-08 08:32:43.000000000","message":"so this also needs update now we are no longer creating the centos9.repo as we are trying to remove https://review.opendev.org/c/openstack/tripleo-ci/+/808177/8/playbooks/tripleo-buildimages/pre9.yaml","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"b614eab91d78660d4364b7d19fa3c32571239420","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  - /etc/yum.repos.d/centos9.repo"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"9\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"container-tools:latest\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fcb5fab8_45dada3f","line":7,"updated":"2021-10-06 20:03:25.000000000","message":"I understand we can\u0027t use conatner-tools: 3.0 but latest is risky","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c2e3520b95f1b77b0cbe715ab39fa6029d7b25c6","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  - /etc/yum.repos.d/centos9.repo"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"9\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"container-tools:latest\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"1ca6979d_7c406a03","line":7,"in_reply_to":"100364fb_ec7b9021","updated":"2021-10-07 12:30:56.000000000","message":"container-tools:3.0 is not shipped for CS9, we are using container-tools:latest.","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"d68cacf13a310d06a405e9ec2b0f320221d6ed0c","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  - /etc/yum.repos.d/centos9.repo"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"9\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"container-tools:latest\""},{"line_number":8,"context_line":"tripleo_image_os: centos9"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"100364fb_ec7b9021","line":7,"in_reply_to":"fcb5fab8_45dada3f","updated":"2021-10-07 11:59:16.000000000","message":"but do you have any suggestion about which version i should use? i just did some digging but i can\u0027t find something newer than tools-3.0 @ https://kojihub.stream.centos.org/koji/packageinfo?packageID\u003d3458 (for example)\n\nso i think for now we have to stay with :latest?","commit_id":"83ca03188b28011899ab146234179c417dce1228"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1142117d03821ab5a9e8f8a11731490aef52c166","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"tripleo_image_source: https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Stream-GenericCloud-9-20210830.0.x86_64.qcow2"},{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/centos*"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"},{"line_number":6,"context_line":"dib_release: \"9\""},{"line_number":7,"context_line":"dib_dnf_module_streams: \"container-tools:latest\""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"3ca334c0_90330e41","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":2},"updated":"2021-10-08 10:47:11.000000000","message":"now we\u0027re using tripleo-repos for the setup (with the parent review https://review.opendev.org/c/openstack/tripleo-ci/+/800577 ) the repos are more like CentOS-Stream-9-*","commit_id":"07c9dbb52577d489d96f3eea308f4bc4da90562c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c164b57cc4858c139e7b8648495e16fe59ecae81","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"tripleo_image_source: https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Stream-GenericCloud-9-20210830.0.x86_64.qcow2"},{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/CentOS-Stream-9-*"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"5351de6a_7ef78ec5","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":129},"updated":"2021-10-14 06:00:54.000000000","message":"The image name is keep changing. I have asked dpwalik to upload this image to RDO image server and keep a generic name. Once this is available, we will update that.","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"708ab13873b5ccd2c3edef5239e68f27a474ac70","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"tripleo_image_source: https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Stream-GenericCloud-9-20210830.0.x86_64.qcow2"},{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/CentOS-Stream-9-*"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"ccb7c358_b4472698","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":129},"in_reply_to":"3c733d9e_b2d7eaac","updated":"2021-10-14 14:56:38.000000000","message":"Agree - let\u0027s keep a consistent link that works","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b77b4e5c9b487d96dcf6ee23ba5e45e1eaf622c0","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"tripleo_image_source: https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Stream-GenericCloud-9-20210830.0.x86_64.qcow2"},{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/CentOS-Stream-9-*"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"a3cc09f1_c0b6bee1","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":129},"in_reply_to":"5351de6a_7ef78ec5","updated":"2021-10-14 08:01:21.000000000","message":"ack","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"0640f7bc18543265cadae9e8c95eb1c4f6629f43","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"tripleo_image_source: https://cloud.centos.org/centos/9-stream/x86_64/images/CentOS-Stream-GenericCloud-9-20210830.0.x86_64.qcow2"},{"line_number":3,"context_line":"dib_yum_repo_conf:"},{"line_number":4,"context_line":"  - /etc/yum.repos.d/CentOS-Stream-9-*"},{"line_number":5,"context_line":"  - /etc/yum.repos.d/delorean*"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"3c733d9e_b2d7eaac","line":2,"range":{"start_line":2,"start_character":22,"end_line":2,"end_character":129},"in_reply_to":"a3cc09f1_c0b6bee1","updated":"2021-10-14 08:08:43.000000000","message":"Here is the latest uploaded image: http://images.rdoproject.org/CentOS-9-Stream-x86_64-GenericCloud.qcow2 We can update it to this url.","commit_id":"5b7d17638104af6b0738f990d79142535b78a60e"}]}