| opendevreview | Merged openstack/tripleo-heat-templates master: Align defaults for SoftwareConfigTransport https://review.opendev.org/c/openstack/tripleo-heat-templates/+/829667 | 01:35 |
|---|---|---|
| *** rlandy|ruck|bbl is now known as rlandy|out | 02:06 | |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Add openstack_git_root for Ansible repo path https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/830673 | 02:36 |
| *** pojadhav|out is now known as pojadhav|ruck | 02:52 | |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-ansible stable/wallaby: Creating a default molecule config https://review.opendev.org/c/openstack/tripleo-ansible/+/830589 | 03:00 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-ansible stable/wallaby: Removing playbooks section from default molecule config https://review.opendev.org/c/openstack/tripleo-ansible/+/830710 | 03:00 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-ansible stable/wallaby: Removing playbooks section from default molecule config https://review.opendev.org/c/openstack/tripleo-ansible/+/830710 | 03:05 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Don't bind host's /run in multipathd container https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830219 | 03:38 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830363 | 03:38 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Update of OVN controllers as an external task. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/829393 | 04:04 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Exclude /etc/openldap to avoid overriding ro file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830574 | 04:04 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Add CephNfs service on roles providing "external" network connectivity https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830137 | 04:04 |
| opendevreview | Merged openstack/python-tripleoclient master: Add Python 3.8 to supported runtime https://review.opendev.org/c/openstack/python-tripleoclient/+/828352 | 04:25 |
| opendevreview | Merged openstack/python-tripleoclient master: Mark Python 3.9 as supported runtime https://review.opendev.org/c/openstack/python-tripleoclient/+/829452 | 04:25 |
| opendevreview | Merged openstack/tripleo-common stable/wallaby: HA: do not rotate cluster passwords https://review.opendev.org/c/openstack/tripleo-common/+/830063 | 04:25 |
| *** frenzy_friday is now known as frenzyfriday|rover | 04:26 | |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Add tripleo-ci-centos-9-content-provider-wallaby job definition https://review.opendev.org/c/openstack/tripleo-ci/+/829677 | 04:27 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Add tripleo-ci-centos-9-undercloud-upgrade job https://review.opendev.org/c/openstack/tripleo-ci/+/829655 | 04:27 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart master: DNM cs9 undercloud upgrade testing https://review.opendev.org/c/openstack/tripleo-quickstart/+/829656 | 04:27 |
| *** soniya29 is now known as soniya29|afk | 04:44 | |
| *** ysandeep|out is now known as ysandeep | 04:53 | |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/victoria: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830733 | 05:29 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/ussuri: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830735 | 05:31 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/train: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830736 | 05:32 |
| shyamb | Hi | 05:37 |
| shyamb | centos8 stream repo files are not available in wallaby tripleo nova compute container images | 05:38 |
| shyamb | Have we intentionally cleaned it? | 05:38 |
| shyamb | @ykarel @Tengu | 05:38 |
| ykarel | shyamb, repo files are not bundled in container and overcloud images | 05:41 |
| ykarel | earlier ubi.repo used to exist but those also removed with https://review.opendev.org/c/openstack/tripleo-common/+/823451 | 05:41 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/train: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830736 | 05:50 |
| shyamb | ykarel: Okay | 05:52 |
| shyamb | If someone wants to add those repos, any documented steps? | 05:52 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/victoria: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830733 | 05:52 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/ussuri: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830735 | 05:54 |
| *** ysandeep is now known as ysandeep|away | 05:55 | |
| ykarel | shyamb, not aware about documented steps, it's like copy required repos in image, use those and remove after usage | 06:00 |
| shyamb | ykarel: Okay, thanks | 06:10 |
| shyamb | ykarel: Looks like we have removed something else too. | 06:27 |
| shyamb | I copied stream 8 repos. | 06:27 |
| shyamb | But it's not working. Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist | 06:27 |
| shyamb | gpg keys? | 06:27 |
| shyamb | or something else? | 06:27 |
| ykarel | you must be using $stream var in repo files that will not work in images directly | 06:28 |
| ykarel | you can create /etc/dnf/vars/stream with 8-stream as content or fix repo files to not used $stream | 06:28 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/ussuri: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830735 | 06:30 |
| shyamb | ykarel: Okay, got it | 06:30 |
| shyamb | thanks | 06:30 |
| shyamb | ykarel: I see kolla images have these repos and everything in place | 06:31 |
| shyamb | Only tripleo images are having this change | 06:32 |
| ykarel | that can be possible | 06:32 |
| shyamb | ykarel: Maybe I need to find some standard approach to get this back | 06:32 |
| shyamb | ykarel: all dnf var files are missing | 06:33 |
| shyamb | avdir avstream contentdir infra nfvsigdist stream | 06:34 |
| ykarel | the vars files are shipped as part for release rpms, so if those are not there vars file will not be there | 06:34 |
| shyamb | ykarel: Yes, I also think so. I think we removed those release rpms during tripleo container build | 06:36 |
| shyamb | ykarel: Can you please point me to tripleo dockerfile repo? | 06:36 |
| shyamb | I will take it from there | 06:36 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/victoria: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830733 | 06:36 |
| ykarel | shyamb, dockerfiles are generated on the fly | 06:36 |
| shyamb | yeah, those jinja templates? where we keep those | 06:37 |
| shyamb | in which repo? | 06:37 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/victoria: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830733 | 06:37 |
| ykarel | shyamb, https://github.com/openstack/tripleo-common/blob/master/container-images/tcib/base/os/nova-base/nova-compute/nova-compute.yaml | 06:38 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-ansible stable/ussuri: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/830735 | 06:38 |
| shyamb | ykarel: thanks | 06:39 |
| *** soniya29|afk is now known as soniya29 | 06:40 | |
| opendevreview | chandan kumar proposed openstack/tripleo-heat-templates master: Added NovaDisableComputeServiceCheckForFfu var https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830740 | 06:56 |
| *** ysandeep|away is now known as ysandeep | 07:04 | |
| *** amoralej|off is now known as amoralej | 07:09 | |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: [DNM] standalone upgrade ffu https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/830741 | 07:22 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade master: Add the update of the OVN controllers before Update Run step. https://review.opendev.org/c/openstack/tripleo-upgrade/+/830743 | 08:04 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Run centos-9 jobs for stable/wallaby check/gate layouts https://review.opendev.org/c/openstack/tripleo-ci/+/830132 | 08:33 |
| marios | ysandeep: chandankumar: ykarel: ^^ can you revote please - zuul failed so i updated for ykarel comment https://review.opendev.org/c/openstack/tripleo-ci/+/830132/4..5 thank you | 08:33 |
| *** jpena|off is now known as jpena | 08:34 | |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Enable fips test for metrics_qdr https://review.opendev.org/c/openstack/tripleo-ansible/+/828423 | 08:37 |
| opendevreview | Merged openstack/tripleo-ansible stable/victoria: Fix how tripleo_multipathd updates multipath.conf https://review.opendev.org/c/openstack/tripleo-ansible/+/830220 | 08:37 |
| opendevreview | Merged openstack/tripleo-common stable/victoria: Ensure failures on the undercloud leads to a complete stop https://review.opendev.org/c/openstack/tripleo-common/+/829205 | 08:38 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Add IronicNeutronAgentReportInterval parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/828559 | 08:40 |
| ysandeep | marios: minor nit - https://review.opendev.org/c/openstack/tripleo-ci/+/830132/5/zuul.d/build-image.yaml#167 - i think we can get rid of additional blank line here | 08:43 |
| marios | ysandeep: i left it intentionally i think it is clearer with the extra line | 08:44 |
| ysandeep | marios: ack if its intentional - looks like there is no pattern, For ex. we have one blank line after #195 but not after#220. | 08:46 |
| marios | ysandeep: ack thanks for checking | 08:46 |
| ykarel | marios, ack | 08:49 |
| marios | thanks chandankumar ykarel ysandeep | 08:58 |
| *** ysandeep is now known as ysandeep|lunch | 09:07 | |
| opendevreview | Marios Andreou proposed openstack/tripleo-validations master: Add centos-9 jobs for master zuul layout - master is now 9 only https://review.opendev.org/c/openstack/tripleo-validations/+/830790 | 09:19 |
| marios | jpodivin: fyi ^^^ | 09:19 |
| marios | jpodivin: but someone needs to sort out the molecule jobs didn't touch those ;) | 09:20 |
| jpodivin | marios: I see, weren't there some issues with 9 recently? | 09:20 |
| marios | jpodivin: probably many issues with 9 recently ;) but do you remember something more specific? | 09:21 |
| marios | hmm zuul didn't like it must have missed something in there... | 09:21 |
| jpodivin | marios: yeah but that shouldn't be hard to fix | 09:21 |
| jpodivin | anyway I seem to recall some failures in other projects when they tried to introduce these. | 09:22 |
| jpodivin | marios: but there's a review for centos9 jobs already in tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/828211 | 09:23 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-common stable/ussuri: Ensure failures on the undercloud leads to a complete stop https://review.opendev.org/c/openstack/tripleo-common/+/830711 | 09:25 |
| marios | jpodivin: ah... i went looking for it and couldn't find it ! | 09:25 |
| marios | jpodivin: thanks... looking | 09:25 |
| marios | jpodivin: k ... my one is a bit different (tried to also fix the coverchange job ) but lets go with +/828211 | 09:26 |
| jpodivin | marios: the coverchange is on back burner now. I have patches to fix it, but they have been stalled in review. We are stretched a bit thin right now. | 09:28 |
| marios | jpodivin: k thanks | 09:28 |
| marios | jpodivin: k the change i posted above (first one which i abandoned) will actualy be needed for wallaby - doing | 09:32 |
| marios | jpodivin: (let's merge https://review.opendev.org/c/openstack/tripleo-validations/+/828211 for master though) | 09:32 |
| jpodivin | marios: unless someone has reason not to we should. matbu ? | 09:34 |
| marios | jpodivin: ack doesn't have to be immediately but please bring to team i replied to david's question there | 09:35 |
| marios | jpodivin: i'll merge it if it isn't soon anyway (e.g. tomorrow) | 09:35 |
| jpodivin | marios: ack, we have a meeting today I'll bring it up. | 09:35 |
| marios | jpodivin: but because of our changes you are now without coverage on master e.g. see https://review.opendev.org/c/openstack/tripleo-validations/+/830502/9#message-bf1033f9e0bd55ef39100ad06dc5074c90d4646f | 09:35 |
| marios | jpodivin: cos no more master centos8 so you don't get the 8 standalone there any more | 09:36 |
| marios | jpodivin: because https://opendev.org/openstack/tripleo-ci/src/commit/c2d8892d8aadc45519be905df7ede7b5fd9ef423/zuul.d/standalone-jobs.yaml#L36 | 09:36 |
| jpodivin | ok that sucks | 09:36 |
| jpodivin | we have to do something about it. | 09:37 |
| marios | jpodivin: ack | 09:37 |
| jpodivin | marios: hm no, this isn't going to work. We'll have to rework https://review.opendev.org/c/openstack/tripleo-validations/+/828211 so that we don't loose coverage | 09:38 |
| marios | jpodivin: k please comment on the review though you only put -1 without why what's the problem? | 09:43 |
| jpodivin | marios: As you said we are going to loose coverage on master if we merge this. | 09:44 |
| marios | jpodivin: no you already lost master | 09:44 |
| marios | jpodivin: because of 11:36 < marios> jpodivin: because | 09:44 |
| marios | https://opendev.org/openstack/tripleo-ci/src/commit/c2d8892d8aadc45519be905df7ede7b5fd9ef423/zuul.d/standalone-jobs.yaml#L36 | 09:44 |
| marios | jpodivin: which merged already | 09:44 |
| opendevreview | Merged openstack/tripleo-quickstart master: Extend repo-setup role to use yum-config cli when needed https://review.opendev.org/c/openstack/tripleo-quickstart/+/825307 | 09:44 |
| marios | jpodivin: with https://review.opendev.org/c/openstack/tripleo-validations/+/828211 you get back the coverage | 09:44 |
| jpodivin | marios: I don't quite fllow | 09:44 |
| marios | jpodivin: but with centos9 | 09:44 |
| marios | jpodivin: centos8 jobs are no longer running on master because of this branches: match 11:44 < marios> https://opendev.org/openstack/tripleo-ci/src/commit/c2d8892d8aadc45519be905df7ede7b5fd9ef423/zuul.d/standalone-jobs.yaml#L36 | 09:44 |
| marios | jpodivin: ie. no master in branches: ^(stable/(train|ussuri|victoria|wallaby)).*$ | 09:45 |
| jpodivin | that regex no longer matches yes | 09:45 |
| marios | jpodivin: so you no longer have any 8 jobs running on your master branch | 09:45 |
| marios | jpodivin: if you merge https://review.opendev.org/c/openstack/tripleo-validations/+/828211 then you'll get the 9 standalone job | 09:45 |
| jpodivin | marios: so we already lost coverage because some patches were merged before the replacement was done. | 09:46 |
| jpodivin | is that correct? | 09:46 |
| marios | jpodivin: clearer? | 09:46 |
| marios | jpodivin: yeah i though this was merged already https://review.opendev.org/c/openstack/tripleo-validations/+/828211 but it seems we missed that one | 09:47 |
| marios | jpodivin: though/thought | 09:47 |
| marios | jpodivin: so we need to merge that asap so you can have 9 standalone running | 09:47 |
| marios | jpodivin: i workflowed it | 09:47 |
| opendevreview | chandan kumar proposed openstack/tripleo-heat-templates master: Added NovaDisableComputeServiceCheckForFfu var https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830740 | 09:48 |
| marios | no need to wait makes no difference i convinced myself ;) | 09:48 |
| jpodivin | marios: thanks we don't have anyone to that apparently. | 09:48 |
| jpodivin | with matbu unavailable and dpeacock asleep :D | 09:48 |
| opendevreview | Marios Andreou proposed openstack/tripleo-validations stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/tripleo-validations/+/830796 | 09:53 |
| marios | jpodivin: ^^^ k that one is for wallaby (similar to the earlier patch i posted by mistake) adding 9 alongside 8 for W ^^^ | 09:53 |
| opendevreview | Marios Andreou proposed openstack/tripleo-validations stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/tripleo-validations/+/830796 | 09:54 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Set NovaDisableComputeServiceCheckForFfu to true https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/830741 | 09:55 |
| opendevreview | Merged openstack/diskimage-builder master: rhel: work around RHEL-9 BLS issues https://review.opendev.org/c/openstack/diskimage-builder/+/829620 | 10:04 |
| opendevreview | Merged openstack/diskimage-builder master: Detect boot and EFI partitions in extract-image https://review.opendev.org/c/openstack/diskimage-builder/+/828617 | 10:04 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade stable/wallaby: Add the update of the OVN controllers before Update Run step. https://review.opendev.org/c/openstack/tripleo-upgrade/+/830797 | 10:15 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade stable/victoria: Add the update of the OVN controllers before Update Run step. https://review.opendev.org/c/openstack/tripleo-upgrade/+/830798 | 10:16 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade stable/ussuri: Add the update of the OVN controllers before Update Run step. https://review.opendev.org/c/openstack/tripleo-upgrade/+/830799 | 10:17 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade stable/train: Add the update of the OVN controllers before Update Run step. https://review.opendev.org/c/openstack/tripleo-upgrade/+/829394 | 10:18 |
| *** ysandeep|lunch is now known as ysandeep | 10:19 | |
| opendevreview | Will Szumski proposed openstack/diskimage-builder master: Always Use linuxefi with DIB_BLOCK_DEVICE=efi https://review.opendev.org/c/openstack/diskimage-builder/+/830801 | 10:20 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Add mds, nfs and ingress daemons to deployed ceph https://review.opendev.org/c/openstack/tripleo-ansible/+/825440 | 10:26 |
| opendevreview | Francesco Pantano proposed openstack/python-tripleoclient master: Add --daemons option to deployed ceph https://review.opendev.org/c/openstack/python-tripleoclient/+/830421 | 10:26 |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates master: Bootstrap DB with auth_ed25519 when it is enabled https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822188 | 10:27 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Centos 9 jobs for validations libs https://review.opendev.org/c/openstack/validations-libs/+/830802 | 10:35 |
| *** rlandy|out is now known as rlandy|ruck | 11:18 | |
| *** dviroel_ is now known as dviroel | 11:28 | |
| opendevreview | Marios Andreou proposed openstack/tripleo-validations stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/tripleo-validations/+/830796 | 11:33 |
| opendevreview | amolkahat proposed openstack/tripleo-ci master: Rename jobs with common names https://review.opendev.org/c/openstack/tripleo-ci/+/821849 | 11:46 |
| *** bhagyashris_ is now known as bhagyashris | 11:57 | |
| opendevreview | James Slagle proposed openstack/python-tripleoclient master: Add AuthCloudName to export data https://review.opendev.org/c/openstack/python-tripleoclient/+/830511 | 11:57 |
| *** amoralej is now known as amoralej|lunch | 12:19 | |
| *** sshnaidm is now known as sshnaidm|off | 12:23 | |
| opendevreview | Marios Andreou proposed openstack/tripleo-common master: Revert "Auto-detect partition types for mounting more images" https://review.opendev.org/c/openstack/tripleo-common/+/830715 | 12:25 |
| marios | frenzyfriday|rover: rlandy|ruck: ^^ | 12:25 |
| marios | pojadhav|ruck: ^ | 12:26 |
| marios | stevebaker[m]: ^^ | 12:26 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Centos 9 jobs for validations libs https://review.opendev.org/c/openstack/validations-libs/+/830802 | 12:26 |
| *** ysandeep is now known as ysandeep|afk | 12:33 | |
| pojadhav|ruck | marios, ack | 12:36 |
| marios | stevebaker[m]: o/ https://bugs.launchpad.net/tripleo/+bug/1962168 | 12:37 |
| opendevreview | Marios Andreou proposed openstack/tripleo-common master: Revert "Auto-detect partition types for mounting more images" https://review.opendev.org/c/openstack/tripleo-common/+/830715 | 12:38 |
| *** pmannidi is now known as pmannidi|Away | 12:50 | |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Centos 9 jobs for validations common https://review.opendev.org/c/openstack/validations-common/+/830814 | 12:51 |
| *** ysandeep|afk is now known as ysandeep | 12:54 | |
| opendevreview | Merged openstack/tripleo-validations master: Move zuul jobs layout to centos9 only for master branch https://review.opendev.org/c/openstack/tripleo-validations/+/828211 | 12:55 |
| *** carloss is now known as carloss|afk | 12:56 | |
| rlandy|ruck | hjensas: hello ... could you take a look at https://bugs.launchpad.net/tripleo/+bug/1962168 and related https://review.opendev.org/c/openstack/tripleo-common/+/830715 with marios comments | 13:00 |
| rlandy|ruck | thank you | 13:00 |
| *** amoralej|lunch is now known as amoralej | 13:19 | |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] parenting https://review.opendev.org/c/openstack/tripleo-ci/+/830818 | 13:22 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible stable/wallaby: Add tripleo_cephadm_default_container boolean https://review.opendev.org/c/openstack/tripleo-ansible/+/830716 | 13:26 |
| opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: Fix positional timeout arg https://review.opendev.org/c/openstack/python-tripleoclient/+/830819 | 13:26 |
| *** pdeore_ is now known as pdeore | 13:31 | |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] parenting https://review.opendev.org/c/openstack/tripleo-ci/+/830818 | 13:32 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible stable/wallaby: Add tripleo_cephadm_default_container boolean https://review.opendev.org/c/openstack/tripleo-ansible/+/830716 | 13:33 |
| opendevreview | Riccardo Pittau proposed openstack/diskimage-builder master: Revert "Detect boot and EFI partitions in extract-image" https://review.opendev.org/c/openstack/diskimage-builder/+/830717 | 13:42 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] parenting https://review.opendev.org/c/openstack/tripleo-ci/+/830818 | 13:55 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] parenting https://review.opendev.org/c/openstack/tripleo-ci/+/830818 | 13:58 |
| *** dviroel is now known as dviroel|brb | 14:01 | |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: DNM testing /diskimage-builder/+/830717 https://review.opendev.org/c/openstack/tripleo-ci/+/830825 | 14:17 |
| marios | rlandy|ruck: frenzyfriday|rover: pojadhav|ruck: ^^^ (https://zuul.openstack.org/status#830825) | 14:18 |
| rlandy|ruck | marios: thank you | 14:19 |
| *** carloss|afk is now known as carloss | 14:22 | |
| *** dviroel|brb is now known as dviroel | 14:23 | |
| *** pojadhav|ruck is now known as pojadhav|brb | 14:28 | |
| *** pojadhav|brb is now known as pojadhav|afk | 14:32 | |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/wallaby: oslo_config_validator: Nova port data changes settings have wrong values https://review.opendev.org/c/openstack/tripleo-validations/+/830719 | 14:32 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/victoria: oslo_config_validator: Nova port data changes settings have wrong values https://review.opendev.org/c/openstack/tripleo-validations/+/830720 | 14:33 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/ussuri: oslo_config_validator: Nova port data changes settings have wrong values https://review.opendev.org/c/openstack/tripleo-validations/+/830721 | 14:33 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/train: oslo_config_validator: Nova port data changes settings have wrong values https://review.opendev.org/c/openstack/tripleo-validations/+/830722 | 14:34 |
| opendevreview | John Fulton proposed openstack/tripleo-heat-templates stable/wallaby: Expose tripleo_cephadm_default_container boolean https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830723 | 14:51 |
| opendevreview | John Fulton proposed openstack/python-tripleoclient stable/wallaby: Add option to bootstrap cephadm without --image https://review.opendev.org/c/openstack/python-tripleoclient/+/830724 | 14:53 |
| opendevreview | John Fulton proposed openstack/python-tripleoclient stable/wallaby: Add option to bootstrap cephadm without --image https://review.opendev.org/c/openstack/python-tripleoclient/+/830724 | 14:57 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Centos 9 jobs for validations common https://review.opendev.org/c/openstack/validations-common/+/830814 | 14:58 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [WIP] Add a seperate parent for common required projects https://review.opendev.org/c/openstack/tripleo-ci/+/830836 | 15:05 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [WIP] Add a seperate parent for common required projects https://review.opendev.org/c/openstack/tripleo-ci/+/830836 | 15:06 |
| marios | rlandy|ruck: Tengu: cloudnull: slagle: o/ fyi http://lists.openstack.org/pipermail/openstack-discuss/2022-February/027403.html - would be great if you can join our community call on tuesday? | 15:13 |
| Tengu | not for me, PTO next week | 15:14 |
| cloudnull | marios sure ! | 15:14 |
| marios | Tengu: ;) ack thanks | 15:14 |
| marios | Tengu: enjoy | 15:14 |
| marios | cloudnull: o/ thx | 15:14 |
| fultonj | Tengu: have you seen this in tripleo-lab? https://paste.opendev.org/show/bHIHExn9jcaIiFJNggSX/ | 15:31 |
| fultonj | I'm wondering if I should increase a timeout | 15:32 |
| * fultonj is using the new UEFI default | 15:32 | |
| Tengu | fultonj: master? https://review.opendev.org/c/openstack/python-tripleoclient/+/830819 | 15:32 |
| fultonj | Tengu: I'm using master branch but i see that hasn't merged yet | 15:33 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [WIP] Add a seperate parent for common required projects https://review.opendev.org/c/openstack/tripleo-ci/+/830836 | 15:33 |
| Tengu | fultonj: we need that to correct the issue | 15:34 |
| fultonj | ok, thanks. i'll try that patch | 15:34 |
| * fultonj looks for tripleo-lab option to "use this patch" | 15:34 | |
| Tengu | fultonj: tested it here, it works fine :) | 15:34 |
| Tengu | fultonj: the best is to `git review -d 830836' and use the synchronize: param in tripleo-lab | 15:35 |
| Tengu | fultonj: basically, I passed an env file with https://paste.openstack.org/show/bYrYCOl4Qdr32idpyfa6/ | 15:35 |
| Tengu | and the right content in the repository (hence the `git review -d ...') | 15:35 |
| fultonj | /home/cjeanner/work/gerrit | 15:35 |
| * fultonj assumes ^ is on the machine you're running ansible from | 15:36 | |
| Tengu | fultonj: point to the root dir containing the python-tripleoclient directory | 15:36 |
| Tengu | and yep, that's on my laptop | 15:36 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Remove Nova from undercloud during upgrades https://review.opendev.org/c/openstack/tripleo-heat-templates/+/827260 | 15:36 |
| fultonj | thanks, i'll use that patch | 15:36 |
| Tengu | :) | 15:37 |
| opendevreview | Marios Andreou proposed openstack/tripleo-validations stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/tripleo-validations/+/830796 | 15:47 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Add mds, nfs and ingress daemons to deployed ceph https://review.opendev.org/c/openstack/tripleo-ansible/+/825440 | 15:55 |
| *** dviroel is now known as dviroel|lunvh | 16:03 | |
| *** dviroel|lunvh is now known as dviroel|lunch | 16:03 | |
| *** ysandeep is now known as ysandeep|out | 16:17 | |
| *** pojadhav|afk is now known as pojadhav|out | 16:25 | |
| *** ykarel is now known as ykarel|away | 16:31 | |
| opendevreview | Merged openstack/diskimage-builder master: Revert "Detect boot and EFI partitions in extract-image" https://review.opendev.org/c/openstack/diskimage-builder/+/830717 | 16:45 |
| opendevreview | Rafael Castillo proposed openstack/tripleo-quickstart master: Ensure ifcfg-eth0 script exists for all centos releases https://review.opendev.org/c/openstack/tripleo-quickstart/+/830870 | 16:47 |
| *** marios is now known as marios|out | 16:51 | |
| *** dviroel|lunch is now known as dviroel | 16:58 | |
| opendevreview | Rafael Castillo proposed openstack/tripleo-quickstart master: Ensure ifcfg-eth0 script exists for all centos releases https://review.opendev.org/c/openstack/tripleo-quickstart/+/830870 | 17:01 |
| opendevreview | Merged openstack/ansible-role-collect-logs master: Also collect container-config-scripts https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/830343 | 17:11 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: BnR only wait for galera when is enabled on the host https://review.opendev.org/c/openstack/tripleo-ansible/+/828573 | 17:13 |
| *** amoralej is now known as amoralej|off | 17:42 | |
| *** jpena is now known as jpena|off | 17:53 | |
| *** rlandy|ruck is now known as rlandy|ruck|mtg | 17:59 | |
| opendevreview | Douglas Viroel proposed openstack/tripleo-quickstart master: DNM - Combine tempest-conf overrides in profile generation https://review.opendev.org/c/openstack/tripleo-quickstart/+/830884 | 18:18 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-heat-templates master: WIP/DNM - test scenario 4 without ceph rgw https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830310 | 18:26 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: WIP - Add fips jobs for centos 9 https://review.opendev.org/c/openstack/tripleo-ci/+/824479 | 18:28 |
| opendevreview | Alan Bishop proposed openstack/tripleo-heat-templates stable/ussuri: Don't bind host's /run in multipathd container https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830888 | 18:49 |
| opendevreview | Alan Bishop proposed openstack/tripleo-ansible stable/ussuri: Fix how tripleo_multipathd updates multipath.conf https://review.opendev.org/c/openstack/tripleo-ansible/+/830889 | 18:49 |
| *** rlandy|ruck|mtg is now known as rlandy|ruck | 19:19 | |
| *** dviroel is now known as dviroel|afk | 20:54 | |
| opendevreview | Steve Baker proposed openstack/diskimage-builder master: Revert "Revert "Detect boot and EFI partitions in extract-image"" https://review.opendev.org/c/openstack/diskimage-builder/+/830900 | 20:54 |
| opendevreview | Merged openstack/tripleo-upgrade master: Add the update of the OVN controllers before Update Run step. https://review.opendev.org/c/openstack/tripleo-upgrade/+/830743 | 20:57 |
| opendevreview | Merged openstack/tripleo-upgrade stable/wallaby: Add the update of the OVN controllers before Update Run step. https://review.opendev.org/c/openstack/tripleo-upgrade/+/830797 | 20:57 |
| rlandy|ruck | cloudnull: hey - not sure if anyone other than takashi is working on puppet-tripleo but there are a bunch of failures: | 21:12 |
| rlandy|ruck | https://zuul.opendev.org/t/openstack/builds?job_name=puppet-openstack-unit-6.21-ubuntu-focal&skip=0 | 21:12 |
| mwhahaha | [2022-02-24 20:45:31 - ERROR] Puppetfiles cannot contain duplicate module names. Remove the duplicates of the following modules: redis | 21:12 |
| mwhahaha | i think he added redis to the puppet-openstack puppet file | 21:12 |
| mwhahaha | it needs to be removed from the puppet-tripleo extras | 21:12 |
| rlandy|ruck | killing our gates | 21:13 |
| mwhahaha | https://opendev.org/openstack/puppet-tripleo/src/branch/master/Puppetfile_extras#L45 | 21:13 |
| mwhahaha | nuke that | 21:13 |
| rlandy|ruck | mwhahaha: :) still saving us | 21:14 |
| mwhahaha | https://opendev.org/openstack/puppet-openstack-integration/commit/bf1a8bda6294494c6686234b57637e4e5d3b402a | 21:14 |
| mwhahaha | broke it | 21:14 |
| slagle | takashi already put up a patch for that, which i reviewed yesterday, one sec | 21:15 |
| slagle | rlandy|ruck: https://review.opendev.org/c/openstack/puppet-tripleo/+/829888 | 21:16 |
| rlandy|ruck | slagle: thanks - can we get a w+ on that | 21:16 |
| rlandy|ruck | then we can rebase the current gate patches | 21:17 |
| rlandy|ruck | gate will reset | 21:17 |
| opendevreview | Chris Sibbitt proposed openstack/tripleo-ansible stable/wallaby: Enable fips test for collectd https://review.opendev.org/c/openstack/tripleo-ansible/+/828424 | 21:18 |
| * rlandy|ruck will abandon/ restore other puppet patches | 21:18 | |
| slagle | rlandy|ruck: done | 21:18 |
| rlandy|ruck | great | 21:18 |
| rlandy|ruck | cleaning gate | 21:18 |
| rlandy|ruck | one more killing the gate: https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py37&project=openstack%2Ftripleo-ansible&branch=stable%2Ftrain&skip=0 | 21:51 |
| rlandy|ruck | 2022-02-24 21:40:44.981956 | ubuntu-bionic | File "/home/zuul/src/opendev.org/openstack/tripleo-ansible/.tox/py37/lib/python3.7/site-packages/setuptools/_vendor/importlib_metadata/__init__.py", line 886, in find_distributions | 21:53 |
| rlandy|ruck | 2022-02-24 21:40:44.981980 | ubuntu-bionic | found = self._search_paths(context.name, context.path) | 21:53 |
| rlandy|ruck | 2022-02-24 21:40:44.981994 | ubuntu-bionic | AttributeError: 'str' object has no attribute 'name' | 21:53 |
| rlandy|ruck | bshephar: ^^ when it's your daylight hours | 21:53 |
| cloudnull | rlandy|ruck that looks like something core to setuptools? | 22:04 |
| cloudnull | File "/home/zuul/src/opendev.org/openstack/tripleo-ansible/.tox/py37/lib/python3.7/site-packages/setuptools/_vendor/importlib_metadata/__init__.py", line 886, in find_distributions | 22:05 |
| cloudnull | error there | 22:05 |
| cloudnull | maybe something we can address in our tox environment ? | 22:06 |
| opendevreview | Steve Baker proposed openstack/diskimage-builder master: Revert "Revert "Detect boot and EFI partitions in extract-image"" https://review.opendev.org/c/openstack/diskimage-builder/+/830900 | 22:17 |
| stevebaker[m] | marios: hey, I'd like to test ^^ with that centos-7 job | 22:17 |
| cloudnull | rlandy|ruck on the passing jobs from https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py37&project=openstack%2Ftripleo-ansible&branch=stable%2Ftrain&skip=0 I can see that we had setuptools==60.6.0 while on the failing jobs we have setuptools==60.9.3 | 22:23 |
| cloudnull | so I suspect we've a bugged version of setuptools causing that failure | 22:23 |
| cloudnull | importlib_metadata is 0.20 on both | 22:24 |
| cloudnull | so that's the only outlier I can see in that job that would be causing the tox issues | 22:24 |
| rlandy|ruck | stevebaker[m]: hi ... you can use the same depends job | 22:27 |
| rlandy|ruck | getting | 22:27 |
| rlandy|ruck | stevebaker[m]: https://review.opendev.org/c/openstack/tripleo-ci/+/830825 | 22:28 |
| rlandy|ruck | that will kick c7 train image build jobs | 22:28 |
| rlandy|ruck | cloudnull: want to set a limit on those versions? | 22:29 |
| rlandy|ruck | or should I bug and pass to brendan who owns the job? | 22:29 |
| stevebaker[m] | rlandy|ruck: thank you! | 22:32 |
| cloudnull | I'll happily limit it, though I'm not sure how we'd go about that? | 22:33 |
| cloudnull | aren't we just running https://github.com/openstack/tripleo-ansible/blob/stable/train/zuul.d/layout.yaml#L12 | 22:33 |
| cloudnull | the global tox linters ? | 22:33 |
| rlandy|ruck | there are upper constraints somewhere | 22:35 |
| rlandy|ruck | ack - must just be a buggy version of setuptools | 22:35 |
| rlandy|ruck | but only on train? | 22:35 |
| cloudnull | https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L83 | 22:36 |
| cloudnull | i guess we could override that with something of our own? | 22:36 |
| rlandy|ruck | need to check why it's only hitting tripleo-ansible | 22:37 |
| cloudnull | https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L640 | 22:37 |
| cloudnull | that's the upper constraint file and the version of setuptools we're seeing on the failing job | 22:38 |
| rlandy|ruck | https://opendev.org/openstack/requirements/src/branch/stable/train/upper-constraints.txt | 22:38 |
| rlandy|ruck | train branch | 22:38 |
| cloudnull | so train doesn't have setuptools defined | 22:38 |
| rlandy|ruck | so we need that | 22:38 |
| rlandy|ruck | backport? | 22:39 |
| cloudnull | https://pypi.org/project/setuptools/ - seems like this version was just released the other day | 22:39 |
| rlandy|ruck | you could get lucky and they will fix the version | 22:41 |
| rlandy|ruck | but if master is locked | 22:41 |
| cloudnull | https://github.com/pypa/setuptools/issues/3113 | 22:41 |
| rlandy|ruck | train should be as well | 22:41 |
| rlandy|ruck | ok so we should LP bug this | 22:41 |
| rlandy|ruck | refer to this issue | 22:41 |
| cloudnull | ++ | 22:41 |
| rlandy|ruck | and then lock the version | 22:42 |
| cloudnull | ++ | 22:42 |
| rlandy|ruck | per master | 22:42 |
| rlandy|ruck | with closes-bug | 22:42 |
| * rlandy|ruck creates bug ... sec | 22:42 | |
| rlandy|ruck | cloudnull: https://bugs.launchpad.net/tripleo/+bug/1962232 | 22:48 |
| rlandy|ruck | now adding patch | 22:48 |
| cloudnull | https://review.opendev.org/c/openstack/requirements/+/830913 | 22:49 |
| cloudnull | done | 22:49 |
| cloudnull | :D | 22:49 |
| cloudnull | team work makes the dream wowrk | 22:49 |
| rlandy|ruck | nice | 22:50 |
| rlandy|ruck | hopefully gate behaves now | 22:51 |
| opendevreview | Kevin Carter proposed openstack/tripleo-ansible stable/train: Carry an upper constraint file to unblock the gate https://review.opendev.org/c/openstack/tripleo-ansible/+/830914 | 22:58 |
| cloudnull | ^ that should help us unblock the gates while we wait for governance to land the change in global requirements or we await a fix in setuptools | 22:59 |
| opendevreview | Kevin Carter proposed openstack/tripleo-ansible stable/train: Carry an upper constraint file to unblock the gate https://review.opendev.org/c/openstack/tripleo-ansible/+/830914 | 23:00 |
| *** dviroel|afk is now known as dviroel | 23:37 | |
| *** rlandy|ruck is now known as rlandy|out | 23:38 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!