opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: libvirt: Fix invalid dependencies when virtlogd wrapper is enabled https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835768 | 00:03 |
---|---|---|
*** rlandy is now known as rlandy|out | 00:36 | |
opendevreview | Steve Baker proposed openstack/openstack-virtual-baremetal master: Make ipxe-boot image EFI capable https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/834813 | 00:40 |
*** dasm|afk is now known as dasm | 01:17 | |
*** pmannidi is now known as pmannidi|brb | 01:48 | |
opendevreview | Steve Baker proposed openstack/openstack-virtual-baremetal master: Make ipxe-boot image EFI capable https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/834813 | 02:16 |
*** pmannidi|brb is now known as pmannidi | 02:27 | |
*** dasm is now known as dasm|gone | 03:41 | |
*** soniya29 is now known as soniya29|rover | 03:41 | |
*** ysandeep|out is now known as ysandeep | 04:26 | |
opendevreview | Steve Baker proposed openstack/tripleo-common master: Support multiple mounts without explicit nbd devices https://review.opendev.org/c/openstack/tripleo-common/+/828412 | 04:47 |
opendevreview | Steve Baker proposed openstack/tripleo-common master: Support standard partition type GUIDs https://review.opendev.org/c/openstack/tripleo-common/+/835774 | 04:47 |
opendevreview | Merged openstack/tripleo-ci master: Bump black to 22.3.0 to fix deps issue (click) https://review.opendev.org/c/openstack/tripleo-ci/+/835547 | 05:02 |
*** marios is now known as marios|ruck | 05:06 | |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835677 | 06:03 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835678 | 06:03 |
marios|ruck | fyi ysandeep ^^ https://bugs.launchpad.net/tripleo/+bug/1966382/comments/27 \o/ ;) | 06:04 |
ysandeep | marios|ruck: reading | 06:04 |
ysandeep | ahh, stable branches as well. | 06:05 |
marios|ruck | yeah :/ | 06:05 |
marios|ruck | lets hope it "just works" (tm) and we don't need something special there | 06:05 |
ysandeep | marios|ruck, and job freeze fun continues :) https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835678 | 06:07 |
marios|ruck | well fantastic then | 06:08 |
marios|ruck | ysandeep: its ok its the upgrade jobs now... i guess we'll get through all the templates eventually :) | 06:08 |
marios|ruck | ysandeep: k will post something sec let me dig a bit | 06:08 |
marios|ruck | ysandeep: we have to deal for now and fix better in the next sprint as discussed yesterday | 06:09 |
ysandeep | +1 | 06:09 |
* ysandeep checking if other tripleo-* repos also need cherry-picks in the mean time | 06:09 | |
marios|ruck | thank you ysandeep | 06:09 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ansible stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835680 | 06:14 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ansible stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835681 | 06:15 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ansible stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835682 | 06:16 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ansible stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835683 | 06:16 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Add irrelevant-files for victoria upgrades template freeze issue https://review.opendev.org/c/openstack/tripleo-ci/+/835776 | 06:17 |
ysandeep | marios|ruck: fyi.. git ui append branch name in topic.. will be hard to track.. I am changing topic for cherry-picks to lp/1966382 as well | 06:18 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Add irrelevant-files for victoria upgrades template freeze issue https://review.opendev.org/c/openstack/tripleo-ci/+/835776 | 06:22 |
marios|ruck | ysandeep: sure thanks | 06:22 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-image-elements stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-image-elements/+/835684 | 06:22 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-image-elements stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-image-elements/+/835685 | 06:23 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Ensure we clean the /etc/machine-id https://review.opendev.org/c/openstack/tripleo-ci/+/835536 | 06:23 |
marios|ruck | Tengu: rebased onto master ^^^ the patch on top (black thing) merged | 06:23 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-image-elements stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-image-elements/+/835686 | 06:24 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-image-elements stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-image-elements/+/835687 | 06:26 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Add irrelevant-files upgrade/standalone template for freeze issue https://review.opendev.org/c/openstack/tripleo-ci/+/835777 | 06:26 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Ensure we clean the /etc/machine-id https://review.opendev.org/c/openstack/tripleo-ci/+/835536 | 06:27 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Add irrelevant-files upgrade/standalone template for freeze issue https://review.opendev.org/c/openstack/tripleo-ci/+/835776 | 06:28 |
marios|ruck | Tengu: sorry had to rebase again to deal with the freeze issue ... one sec will rebase again and will be ok | 06:28 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Ensure we clean the /etc/machine-id https://review.opendev.org/c/openstack/tripleo-ci/+/835536 | 06:29 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835688 | 06:29 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835688 | 06:30 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835689 | 06:31 |
marios|ruck | Tengu: fyi https://review.opendev.org/c/openstack/tripleo-ci/+/835536/8#message-de41d7cf9127e95814384a0c3ee24a55d48a7cf4 | 06:31 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835790 | 06:32 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835678 | 06:33 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-upgrade stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835791 | 06:34 |
marios|ruck | ysandeep: please vote when you get a chance https://review.opendev.org/c/openstack/tripleo-ci/+/835776 (https://zuul.openstack.org/status#835776) | 06:42 |
ysandeep | marios|ruck,: ack, when you have a second can you review this also https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/835537 so that I can create its cherry-picks | 06:42 |
ysandeep | marios|ruck, fyi.. I have created https://hackmd.io/dCpGo9LGSGGC3oFyFtKnnQ for tracking tripleo repos cherry-picks | 06:43 |
opendevreview | chandan kumar proposed openstack/tripleo-docs master: Added Docs for TripleO Zuul Job parenting https://review.opendev.org/c/openstack/tripleo-docs/+/835782 | 06:44 |
opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates master: Move permissions from neutron to ovn-bgp https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835783 | 06:44 |
marios|ruck | done ysandeep thanks | 06:45 |
ysandeep | marios|ruck, voted on https://review.opendev.org/c/openstack/tripleo-ci/+/835776 , we can merge it when zuul reports | 06:46 |
opendevreview | Jiri Podivin proposed openstack/tripleo-ci master: WIP - Add fips jobs for centos 9 https://review.opendev.org/c/openstack/tripleo-ci/+/824479 | 06:50 |
opendevreview | Merged openstack/tripleo-puppet-elements master: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/835537 | 06:56 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-puppet-elements stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/835792 | 07:05 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-puppet-elements stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/835793 | 07:06 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-puppet-elements stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/835794 | 07:06 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-puppet-elements stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/835795 | 07:07 |
*** arxcruz is now known as arxcruz|out | 07:15 | |
ysandeep | marios|ruck: https://hackmd.io/dCpGo9LGSGGC3oFyFtKnnQ - We should be good now for tripleo-repos, tht need cherry-picks to u/t as well but since you were working on cherry-picking tht patches I didn't touched that. | 07:18 |
ysandeep | I will let tkajinam and jpodivin decides if they need cherry-picks for puppet-* repos as well. | 07:19 |
*** amoralej|off is now known as amoralej | 07:19 | |
ysandeep | tkajinam, jparker regarding this is regarding "setuptools: Disable auto discovery" | 07:19 |
ysandeep | jparker, unping sry it was for jpodivin | 07:19 |
jpodivin | ysandeep: I don't know about those. It is possible that the behavior will be demonstrated there as well | 07:20 |
jpodivin | ... but unless there is an actual failure I don't have any strong opinions one way or another. | 07:21 |
jpodivin | tkajinam, it's up to you. | 07:21 |
ysandeep | jpodivin, yes we hit issues in our job for ex tox-txt that's why we posted cherry-picks.. but if you are not seeing any failure.. I don't think there is a need. | 07:22 |
marios|ruck | ysandeep: k thanks let me do the t/u now sec | 07:23 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835796 | 07:24 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835797 | 07:24 |
ysandeep | marios|ruck: ++ thanks! | 07:26 |
marios|ruck | ysandeep: holding breath for job freeze ;) | 07:26 |
ysandeep | :D | 07:27 |
marios|ruck | ysandeep: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835796/1#message-b7fbe608055b93f9aeafd16b1629f7729e44aef8 (kind of expected it.. sec fixing and will adn the depends on) | 07:27 |
ysandeep | ack | 07:28 |
opendevreview | Merged openstack/tripleo-docs master: Document the --cluster option for deployed ceph https://review.opendev.org/c/openstack/tripleo-docs/+/835757 | 07:31 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Add irrelevant-files upgrade/standalone template for freeze issue https://review.opendev.org/c/openstack/tripleo-ci/+/835776 | 07:33 |
opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade stable/queens: Improve ping test coverage during update. https://review.opendev.org/c/openstack/tripleo-upgrade/+/835785 | 07:33 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835796 | 07:34 |
tkajinam | ysandeep, jpodivin I think we don't need backports. as earlier mentioned, the file is used to generate release notes and I don't think we need to deal with newer version of setuptools in these jobs | 07:34 |
marios|ruck | tkajinam: we were hitting it in victoria/wallaby gates ... but you mean 'lets remove setup.py' instead of the patch? | 07:34 |
ysandeep | marios|ruck: i think tkajinam meant that only for puppet-* repos | 07:35 |
tkajinam | marios|ruck, which job was broken ? | 07:35 |
marios|ruck | ysandeep: ak i see | 07:35 |
marios|ruck | tkajinam: sec i commented on bug with links | 07:36 |
ysandeep | for ex: tox-tht | 07:36 |
tkajinam | if that is release note job then that is the problem | 07:36 |
marios|ruck | tkajinam: https://bugs.launchpad.net/tripleo/+bug/1966382/comments/27 | 07:36 |
tkajinam | if not, it doesn't affect puppet repos | 07:36 |
marios|ruck | tkajinam: yeah didn't realise you were referring to puppet ACK thanks | 07:36 |
*** jpena|off is now known as jpena | 07:37 | |
tkajinam | marios|ruck, ah, ok. thank you, too | 07:37 |
tkajinam | I think we've not yet hit the issue yet in puppet repos because we use upper-constraints | 07:39 |
tkajinam | in that release note job | 07:39 |
tkajinam | upper-constraints.txt has setuptools===60.9.3 now | 07:39 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Ensure we clean the /etc/machine-id https://review.opendev.org/c/openstack/tripleo-ci/+/835536 | 07:40 |
tkajinam | I don't think they bump that in stable branches so I think we are all good now | 07:40 |
jpodivin | tkajinam: You mean central upper-constraints ? | 07:40 |
tkajinam | yeah | 07:40 |
tkajinam | in the requirements repo | 07:40 |
jpodivin | tkajinam: I believe those are bound to python runtime version. | 07:40 |
ysandeep | tkajinam: ah thanks, make sense now | 07:40 |
jpodivin | unless you plan on changing that you should be cool on stable releases | 07:41 |
tkajinam | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L641-L642 | 07:41 |
tkajinam | just curious to know how it behaves with python 3.9 :-P | 07:41 |
marios|ruck | please revote when you get a sec ysandeep https://zuul.openstack.org/status#835776 (https://review.opendev.org/c/openstack/tripleo-ci/+/835776) we should be good now | 07:41 |
tkajinam | we might need to fix every single repos until the newer version is pulled to the requirements | 07:42 |
tkajinam | that's ... huge :-) | 07:42 |
jpodivin | tkajinam: by default, if there is no rule for the runtime, the other constraints are considered. | 07:42 |
jpodivin | tkajinam: in this case there are none | 07:42 |
ysandeep | marios|ruck, okay added same for tripleo-upgrades-ussuri-pipeline template.. voted | 07:43 |
tkajinam | maybe it doesn't affect release note job, as it uses Python 3.8 iirc | 07:43 |
tkajinam | which is the default runtime | 07:43 |
tkajinam | but python 3.9 jobs we have might be affected... or might have been affected already | 07:43 |
jpodivin | tkajinam: as I said, so long as the runtime stays the same, you are safe | 07:43 |
tkajinam | yeah | 07:43 |
opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade stable/queens: Improve ping test coverage during update. https://review.opendev.org/c/openstack/tripleo-upgrade/+/835785 | 07:43 |
tkajinam | I'm not concerned about puppet repos | 07:43 |
marios|ruck | ysandeep: thanks | 07:43 |
jpodivin | tkajinam: as for the 3.9 we can keep this in as a back up https://review.opendev.org/c/openstack/requirements/+/835215 | 07:44 |
jpodivin | or similar. | 07:44 |
tkajinam | jpodivin, yeah | 07:44 |
jpodivin | the point is that patching new constraint shouldn't be hard. | 07:44 |
tkajinam | that makes clear sense | 07:44 |
tkajinam | it shouldn't be hard, its muuuuuuuuuuuuuuuuch less hard than fixing ~100 repos | 07:44 |
jpodivin | exactly. | 07:47 |
Tengu | marios|ruck: checking - sorry, was on doggo walk duty :) | 07:53 |
Tengu | marios|ruck: ok - well, no problem at all ;). It happens - and thank you for taking care of those shenanigans ;) | 07:54 |
marios|ruck | Tengu: thanks should be OK now (had to rebase again few mins ago but should be done ) | 07:55 |
Tengu | :) | 07:56 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ci master: Make cs9 undercloud upgrade job voting https://review.opendev.org/c/openstack/tripleo-ci/+/835541 | 07:58 |
*** ysandeep is now known as ysandeep|lunch | 07:59 | |
*** pojadhav is now known as pojadhav|afk | 08:08 | |
*** pojadhav|afk is now known as pojadhav | 08:39 | |
opendevreview | Bogdan Dobrelya proposed openstack/puppet-tripleo master: WIP override oslo.log config for involved services https://review.opendev.org/c/openstack/puppet-tripleo/+/834846 | 08:41 |
*** ysandeep|lunch is now known as ysandeep | 08:58 | |
*** soniya29|rover is now known as soniya29|rover|lunch | 09:05 | |
jpodivin | Does anyone know if atop is going to get into centos9? | 09:07 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-ci master: Make cs9 undercloud upgrade job voting https://review.opendev.org/c/openstack/tripleo-ci/+/835541 | 09:16 |
jpodivin | marios|ruck: about the issue we had with black, do you think we should bump the dependency proactively? | 09:45 |
jpodivin | because I would be there are other repos using it. | 09:45 |
marios|ruck | jpodivin: which dependency? we updated to newer release of black | 09:50 |
marios|ruck | jpodivin: you mean also update click dependency? or something else | 09:50 |
jpodivin | marios|ruck: oh no, I mean black itself | 09:50 |
jpodivin | the version was bumped in one repo. | 09:50 |
marios|ruck | jpodivin: do you mean bump it for validations are you using black ? | 09:50 |
jpodivin | marios|ruck: I don't think we are using it, but others might be. I'm going run a quick search . | 09:51 |
opendevreview | Fernando Díaz Bravo proposed openstack/python-tripleoclient stable/wallaby: bnr - overcloud restore - Handle correctly the ~ on the path https://review.opendev.org/c/openstack/python-tripleoclient/+/835798 | 09:51 |
jpodivin | marios|ruck: ok, suspicions confirmed https://codesearch.openstack.org/?q=repo%3A%20https%3A%2F%2Fgithub.com%2Fpython%2Fblack.git&i=nope&literal=nope&files=&excludeFiles=&repos= | 09:51 |
marios|ruck | jpodivin: k i think it is just tripleo-ci but cant hurt to check thank you | 09:51 |
jpodivin | marios|ruck: so, I guess we should patch these right? | 09:51 |
marios|ruck | jpodivin: k if you restrict just for the tripleo world then it is just ci | 09:52 |
marios|ruck | https://codesearch.opendev.org/?q=repo:.*https://github.com/python/black.git&repos=openstack/tripleo-ansible,openstack/tripleo-ci,openstack/tripleo-common,openstack/tripleo-common-tempest-plugin,openstack/tripleo-docs,openstack/tripleo-ha-utils,openstack/tripleo-heat-templates,openstack/tripleo-image-elements,openstack/tripleo-ipsec,openstack/tripleo-operator-ansible,openstack/tripleo-puppet-elements, | 09:52 |
marios|ruck | openstack/tripleo-quickstart,openstack/tripleo-quickstart-extras,openstack/tripleo-repos,openstack/tripleo-specs,openstack/tripleo-upgrade,openstack/tripleo-validations,openstack/python-tripleoclient,openstack/puppet-tripleo | 09:52 |
jpodivin | marios|ruck: that makes sense. | 09:53 |
jpodivin | the only one that uses tripleo-* namespace would be the openstack / tripleo-ci-health-queries | 09:54 |
jpodivin | marios|ruck: are we using that ^ for anything ? | 09:54 |
marios|ruck | jpodivin: yeah at least that was being worked on by sorin don't know if it is still a thing/actively worked on right now. ananya may know but i think she is out | 09:55 |
jpodivin | marios|ruck: the activity seems rather ... moderate for that repo | 09:56 |
jpodivin | marios|ruck: last review I see is from December . | 09:56 |
jpodivin | Well, it's not really that big of a time investment, I'm going to submit a review for it and see what happens. | 09:57 |
marios|ruck | ack thanks jpodivin | 09:59 |
*** marios|ruck is now known as marios|ruck|lunch | 10:01 | |
*** soniya29|rover|lunch is now known as soniya29|rover | 10:07 | |
*** ysandeep is now known as ysandeep|afk | 10:09 | |
opendevreview | Jiri Podivin proposed openstack/tripleo-ci-health-queries master: Bumping black to 22.3.0 https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/835813 | 10:10 |
opendevreview | Jiri Podivin proposed openstack/ansible-role-collect-logs master: Bumping black to 22.3.0 https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/835814 | 10:10 |
*** rlandy|out is now known as rlandy | 10:22 | |
*** marios|ruck|lunch is now known as marios|ruck | 10:30 | |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart-extras master: WIP disable extras common for build-test-packages https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835815 | 10:45 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates stable/train: DNM Testing https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835762 | 10:57 |
*** soniya29|rover is now known as soniya29|rover|brb | 11:09 | |
opendevreview | Merged openstack/tripleo-heat-templates master: Expose new paramter `NovaMDEVTypes` https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835439 | 11:15 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: Revert "Make tripleo-ci-centos-9-standalone-on-multinode-ipa non voting till 1963675 is fixed" https://review.opendev.org/c/openstack/tripleo-ci/+/835674 | 11:17 |
*** dviroel|out is now known as dviroel | 11:17 | |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: Revert "Make tripleo-ci-centos-9-standalone-on-multinode-ipa non voting till 1963675 is fixed" https://review.opendev.org/c/openstack/tripleo-ci/+/835674 | 11:18 |
*** ysandeep|afk is now known as ysandeep | 11:18 | |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: Enable tripleo-ceph-periodic run for tripleo-ci project https://review.opendev.org/c/openstack/tripleo-ci/+/835755 | 11:19 |
*** soniya29|rover|brb is now known as soniya29|rover | 11:42 | |
opendevreview | Jiri Podivin proposed openstack/ansible-role-collect-logs master: Bumping black to 22.3.0 https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/835814 | 11:45 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Revert "Make tripleo-ci-centos-9-standalone-on-multinode-ipa non voting till 1963675 is fixed" https://review.opendev.org/c/openstack/tripleo-ci/+/835674 | 11:48 |
*** amoralej is now known as amoralej|lunch | 12:16 | |
opendevreview | Douglas Viroel proposed openstack/tripleo-quickstart-extras master: DNM - Set dependency vars in subnodes https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834972 | 12:16 |
*** ysandeep is now known as ysandeep|afk | 12:20 | |
opendevreview | Merged openstack/tripleo-upgrade stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835791 | 12:23 |
opendevreview | Merged openstack/tripleo-image-elements stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-image-elements/+/835687 | 12:23 |
opendevreview | Merged openstack/tripleo-ansible stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835683 | 12:23 |
opendevreview | Merged openstack/tripleo-image-elements stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-image-elements/+/835686 | 12:23 |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835677 | 12:23 |
opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Adds compose-repos pre playbook on base upstream jobs https://review.opendev.org/c/openstack/tripleo-ci/+/834515 | 12:26 |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart-extras master: Disable extras common for build-test-packages https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835815 | 12:43 |
opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/wallaby: Move permissions from neutron to ovn-bgp https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835805 | 12:57 |
opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/wallaby: Move permissions from neutron to ovn-bgp https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835805 | 12:58 |
*** ysandeep|afk is now known as ysandeep | 12:58 | |
*** amoralej|lunch is now known as amoralej | 13:03 | |
opendevreview | Merged openstack/tripleo-ci master: Add irrelevant-files upgrade/standalone template for freeze issue https://review.opendev.org/c/openstack/tripleo-ci/+/835776 | 13:08 |
opendevreview | Merged openstack/tripleo-puppet-elements stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/835795 | 13:08 |
opendevreview | Merged openstack/tripleo-heat-templates stable/train: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835797 | 13:08 |
*** dasm|gone is now known as dasm | 13:26 | |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Add new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 13:49 |
opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates master: Designate: unbound resolver port information available to service configuration https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835833 | 14:10 |
opendevreview | Merged openstack/tripleo-upgrade stable/wallaby: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835688 | 14:15 |
opendevreview | Merged openstack/tripleo-ansible stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835682 | 14:15 |
opendevreview | Merged openstack/tripleo-ansible stable/victoria: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-ansible/+/835681 | 14:15 |
cloudnull | rlandy following up on the namespace things I mentioning yesterday. If we don't set anything in the undercloud.conf (Just assuming the defaults) on C9-stream the build will use containers from tripleomaster, which are ubi8, and doesn't quite work right when deploying the overcloud. | 14:22 |
cloudnull | do you think we should switch the default namespace over to tripleomastercentos9? | 14:22 |
opendevreview | mbu proposed openstack/validations-libs stable/1.6: Checking inventory file presence and format https://review.opendev.org/c/openstack/validations-libs/+/835807 | 14:23 |
cloudnull | its hard coded in the client | 14:23 |
cloudnull | -- /usr/lib/python3.9/site-packages/tripleoclient/constants.py:43:DEFAULT_CONTAINER_NAMESPACE = "tripleomaster" | 14:23 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Add new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 14:24 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Setting default inventory path for VF interface with ooo client. https://review.opendev.org/c/openstack/python-tripleoclient/+/835835 | 14:27 |
*** pojadhav is now known as pojadhav|afk | 14:31 | |
*** ysandeep is now known as ysandeep|out | 14:37 | |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Run control plane update first, next compute and cephstorage last https://review.opendev.org/c/openstack/tripleo-upgrade/+/791949 | 14:38 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Setting default inventory path for VF interface with ooo client. https://review.opendev.org/c/openstack/python-tripleoclient/+/835835 | 14:39 |
rlandy | cloudnull: how are you running jobs? if you use our release files: https://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-9/master.yml#L26 you should hit the right namespace | 14:40 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/victoria: Setting default inventory path for VF interface with ooo client. https://review.opendev.org/c/openstack/python-tripleoclient/+/835837 | 14:41 |
rlandy | also on libvirt deploy: https://github.com/openstack/tripleo-quickstart/blob/master/config/release/centosci/CentOS-9/master-current-tripleo.yml#L9 | 14:41 |
cloudnull | rlandy just running through with defaults | 14:41 |
cloudnull | trying to run things like a community user / customer would w/ master | 14:42 |
cloudnull | the default namespace, which is hardcoded into the client, pulls tripleomaster. | 14:43 |
rlandy | cloudnull: then we need to change the client defaults | 14:43 |
rlandy | or the default namespace | 14:44 |
rlandy | cloudnull> do you think we should switch the default namespace over to tripleomastercentos9? | 14:44 |
rlandy | so yes, for master client | 14:44 |
rlandy | cloudnull: if we consider the current workflow | 14:44 |
rlandy | where master moves only every three releases | 14:44 |
rlandy | we will hit the same workflow when we need to go to centOS stream 10 | 14:45 |
rlandy | so we need to start differenciating | 14:45 |
rlandy | going forward that would have to stay the extendable pattern | 14:45 |
rlandy | cloudnull: ^^ objections?? | 14:45 |
opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352 | 14:46 |
opendevreview | Kevin Carter proposed openstack/python-tripleoclient master: Update the default namespace https://review.opendev.org/c/openstack/python-tripleoclient/+/835839 | 14:46 |
cloudnull | rlandy++ | 14:47 |
cloudnull | no objections from me | 14:47 |
rlandy | cloudnull: thanks for above patch - voted | 14:48 |
bogdando | rlandy: hi, since you mentioned've namespaces, did you have a chance to take a look the mixed versions spec? | 14:53 |
bogdando | https://review.opendev.org/c/openstack/tripleo-specs/+/834076 | 14:53 |
rlandy | bogdando: yes .. I did - I need to relook at it after comments | 14:53 |
bogdando | there is surely a lot of things to version all that, including namespaces, for your team | 14:53 |
rlandy | we were planning on seeing the discussion at PTG | 14:53 |
bogdando | ack | 14:53 |
bogdando | thanks | 14:53 |
rlandy | bogdando: I also asked chandan to look into your comment about modify image | 14:54 |
rlandy | and getting the gating repo | 14:54 |
rlandy | from his investigation, we are picking up the gating repo rpms | 14:54 |
bogdando | I wonder if CI infra would be the most work items taker in that regard, more than core tripleo team :) | 14:54 |
rlandy | bogdando: if you see that again, pls LP bug it so we can track your case | 14:54 |
rlandy | in case it's a particular gating repo we are missing | 14:54 |
bogdando | I see, thanks | 14:55 |
rlandy | bogdando: for mixed release? yep - a lot of CI load there | 14:55 |
bogdando | yea | 14:55 |
rlandy | bogdando: initial questions around upstream vs donwstream support on mixed mode | 14:55 |
rlandy | raised that at the osp 18 planning meetings | 14:55 |
bogdando | even for upstream only, would be quite a few | 14:56 |
rlandy | we did discuss at the CI team meetings about how we would set up multinode to do check testimg | 14:56 |
rlandy | with different centos versions | 14:56 |
rlandy | it should be possible on our side | 14:56 |
rlandy | obviously standalone is not an option here | 14:56 |
bogdando | also the reproducer support, it's the best way to deploy and debug | 14:57 |
rlandy | but ovb and multinode would be a possibility | 14:57 |
rlandy | we will lean on jm1 to add that reproducer support | 14:57 |
bogdando | standalone could at least test cs8 with cs9 containers compat | 14:57 |
rlandy | ack on that | 14:57 |
bogdando | or vice versa | 14:57 |
rlandy | but multinode would give real separation | 14:57 |
bogdando | indeed | 14:57 |
rlandy | so the CI code makes some assumptions | 14:58 |
rlandy | based on installed OS | 14:58 |
rlandy | those will need to be able to be overridden | 14:58 |
bogdando | well, mixed nova mode, would assume running different container OS versions vs host versions iirc | 14:58 |
rlandy | ^^ tricky to catch those all | 14:58 |
bogdando | anyways, so standalone would help here | 14:58 |
rlandy | so - yeah - I looked at it :) | 14:58 |
opendevreview | Kevin Carter proposed openstack/python-tripleoclient master: update env options to resolve deprecation warnings https://review.opendev.org/c/openstack/python-tripleoclient/+/835842 | 14:58 |
rlandy | but would like to get a fuller idea of what we need to support for testing after PTG | 14:59 |
bogdando | yes, see you there | 15:00 |
bogdando | virtually, again :( | 15:00 |
*** marios|ruck is now known as marios | 15:00 | |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: [Train-only] Setting default inventory path for VF interface with ooo client. https://review.opendev.org/c/openstack/python-tripleoclient/+/835844 | 15:03 |
*** dviroel is now known as dviroel_ | 15:04 | |
*** dviroel_ is now known as dviroel | 15:04 | |
opendevreview | Kevin Carter proposed openstack/tripleo-common master: Remove "deprecated" key in inventory https://review.opendev.org/c/openstack/tripleo-common/+/835845 | 15:06 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/victoria: Memcached: Allow puppet to be aware IPv6 is used https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834595 | 15:06 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/ussuri: Memcached: Allow puppet to be aware IPv6 is used https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834596 | 15:08 |
hjensas | tkajinam++ , yes :) | 15:08 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/train: Memcached: Allow puppet to be aware IPv6 is used https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834597 | 15:09 |
*** marios is now known as marios|ruck | 15:10 | |
opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Explicitly cast boolean variables in cephadm playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/835848 | 15:10 |
tkajinam | hjensas, thanks for pointing that :-) | 15:13 |
opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates master: Nova: Disable instance usage audit by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822789 | 15:19 |
tkajinam | beagles, thanks for fixing that typo ! | 15:22 |
opendevreview | yatin proposed openstack/tripleo-quickstart-extras master: [DNM] enable ovn debug https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835597 | 15:35 |
opendevreview | Douglas Viroel proposed openstack/tripleo-quickstart-extras master: DNM - Set dependency vars in subnodes https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834972 | 15:40 |
opendevreview | Merged openstack/tripleo-heat-templates master: Move permissions from neutron to ovn-bgp https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835783 | 15:43 |
*** dviroel is now known as dviroel|lunch | 15:44 | |
opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Explicitly cast boolean variables in cephadm playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/835848 | 15:46 |
*** amoralej is now known as amoralej|off | 15:50 | |
*** bogdando_ is now known as bogdando | 16:00 | |
marios|ruck | folks if you are having 8 content provider fails it should be fixed once https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835815/ merges | 16:09 |
beagles | tkajinam: np! | 16:10 |
bogdando | marios|ruck: thanks | 16:13 |
*** marios|ruck is now known as marios|out | 16:20 | |
*** jpena is now known as jpena|off | 16:28 | |
opendevreview | Kevin Carter proposed openstack/tripleo-common master: Update the default namespace https://review.opendev.org/c/openstack/tripleo-common/+/835873 | 16:28 |
opendevreview | Kevin Carter proposed openstack/tripleo-ansible master: Update the default namespace https://review.opendev.org/c/openstack/tripleo-ansible/+/835874 | 16:30 |
cloudnull | rlandy - https://review.opendev.org/q/topic:default%257Cnamespace couple more default namespace changes | 16:31 |
rlandy | looking | 16:35 |
rlandy | cloudnull: left some comments in https://review.opendev.org/c/openstack/tripleo-ansible/+/835874 - I think those should reference quay.io now | 16:43 |
rlandy | unless you have some reason to stay with dicker | 16:43 |
rlandy | docker | 16:43 |
rlandy | we hope to get rid of that registry eventually | 16:43 |
cloudnull | ++ | 16:48 |
cloudnull | will update that | 16:48 |
opendevreview | Kevin Carter proposed openstack/tripleo-ansible master: Update the default namespace https://review.opendev.org/c/openstack/tripleo-ansible/+/835874 | 16:50 |
opendevreview | Kevin Carter proposed openstack/python-tripleoclient master: Update the default namespace https://review.opendev.org/c/openstack/python-tripleoclient/+/835839 | 16:52 |
rlandy | slagle: hello ... wrt PTG schedule ... https://etherpad.opendev.org/p/tripleo-zed-ptg-topics - probably makes more sense to have the EL9/Centos9-stream topic before stream pinning | 16:54 |
rlandy | EL9/Centos9-stream topic will give a status of where we are | 16:55 |
rlandy | before we chat about the failures there | 16:55 |
*** dviroel|lunch is now known as dviroel | 17:00 | |
opendevreview | Merged openstack/tripleo-ci master: Enable tripleo-ceph-periodic run for tripleo-ci project https://review.opendev.org/c/openstack/tripleo-ci/+/835755 | 17:21 |
opendevreview | Merged openstack/puppet-tripleo master: Revert "Pin puppet-systemd" https://review.opendev.org/c/openstack/puppet-tripleo/+/827974 | 17:21 |
opendevreview | Kevin Carter proposed openstack/tripleo-ansible master: Remove missing stat warning https://review.opendev.org/c/openstack/tripleo-ansible/+/835883 | 17:58 |
opendevreview | Kevin Carter proposed openstack/tripleo-ansible master: Remove missing stat warning https://review.opendev.org/c/openstack/tripleo-ansible/+/835883 | 17:59 |
opendevreview | Douglas Viroel proposed openstack/tripleo-quickstart-extras master: DNM - Set dependency vars in subnodes https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834972 | 18:09 |
opendevreview | Merged openstack/tripleo-quickstart-extras master: Disable extras common for build-test-packages https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835815 | 18:21 |
opendevreview | Merged openstack/tripleo-image-elements master: remove unicode literal from code https://review.opendev.org/c/openstack/tripleo-image-elements/+/830545 | 18:21 |
rlandy | centos-8 content providers should be passing now | 18:25 |
rlandy | can recheck any failed patches on that | 18:25 |
hjensas | error: Multiple top-level packages discovered in a flat-layout: | 18:36 |
hjensas | seeing on Victoria https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834595 | 18:36 |
hjensas | I guess we need - https://review.opendev.org/c/openstack/puppet-tripleo/+/835589 | 18:38 |
rlandy | hjensas: there are a bunch of fixes for those error | 18:49 |
rlandy | getting | 18:49 |
rlandy | 'https://review.opendev.org/q/topic:lp%252F1966382 | 18:50 |
rlandy | probably need one of those | 18:50 |
hjensas | rlandy: ack, I'll start by merging https://review.opendev.org/c/openstack/puppet-tripleo/+/835589 ... | 19:21 |
hjensas | and then look at the rest. | 19:22 |
rlandy | hjensas: I w+ what I could | 19:23 |
rlandy | I don't have merge rights on all those repos | 19:23 |
hjensas | rlandy: I guess I'll have to wait for the backports proposed by Marios and Sandeep there. Hopefully they will be happier in CI now that content provider job works. | 19:37 |
hjensas | I see you rechecked them already. *thumbs up* | 19:37 |
rlandy | some are failing the content provider and some random tox | 19:41 |
opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Add ceph-install playbook to install ceph before overcloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352 | 20:35 |
opendevreview | Merged openstack/tripleo-ci master: Revert "Make tripleo-ci-centos-9-standalone-on-multinode-ipa non voting till 1963675 is fixed" https://review.opendev.org/c/openstack/tripleo-ci/+/835674 | 21:09 |
opendevreview | Douglas Viroel proposed openstack/tripleo-quickstart-extras master: DNM - Set dependency vars in subnodes https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834972 | 21:15 |
*** dviroel is now known as dviroel|out | 21:16 | |
*** dasm is now known as dasm|off | 21:43 | |
*** rlandy is now known as rlandy|bbl | 22:11 | |
*** ianw_pto is now known as ianw | 22:24 | |
opendevreview | Merged openstack/tripleo-upgrade stable/train: Run control plane update first, next compute and cephstorage last https://review.opendev.org/c/openstack/tripleo-upgrade/+/791949 | 23:32 |
opendevreview | Merged openstack/tripleo-upgrade stable/ussuri: Disable auto discovery https://review.opendev.org/c/openstack/tripleo-upgrade/+/835790 | 23:32 |
opendevreview | Merged openstack/puppet-tripleo master: setuptools: Disable auto discovery https://review.opendev.org/c/openstack/puppet-tripleo/+/835589 | 23:32 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove unused puppet-ptp https://review.opendev.org/c/openstack/puppet-tripleo/+/828635 | 23:35 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!