| *** pmannidi is now known as pmannidi|brb | 00:58 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Manila: Deprecate ineffctive *DriverHandlesShareServers https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807226 | 01:31 |
|---|---|---|
| *** pmannidi|brb is now known as pmannidi | 01:34 | |
| *** pmannidi is now known as pmannidi|Lunch | 02:20 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Manila: Deprecate ineffctive *DriverHandlesShareServers https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807226 | 02:45 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Manila: Remove usage of deprecated driver_handles_share_servers https://review.opendev.org/c/openstack/puppet-tripleo/+/807227 | 02:47 |
| opendevreview | Merged openstack/python-tripleoclient stable/train: Fix edit authorized_keys file in-place https://review.opendev.org/c/openstack/python-tripleoclient/+/806993 | 02:57 |
| opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Restore kernel and ramdisk arguments to node configure https://review.opendev.org/c/openstack/python-tripleoclient/+/805893 | 03:34 |
| *** pmannidi|Lunch is now known as pmannidi | 04:03 | |
| opendevreview | Merged openstack/tripleo-common stable/wallaby: Add dnsmasq to ironic-pxe container https://review.opendev.org/c/openstack/tripleo-common/+/806074 | 04:46 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 05:39 |
| chandankumar | Tengu: \o. I think tcib-extra is working fine | 05:40 |
| Tengu | chandankumar: hey, good to hear! | 05:44 |
| chandankumar | Tengu: I need feedback on this https://review.opendev.org/c/openstack/tripleo-common/+/800580 basically shell hacks | 05:45 |
| Tengu | lemme check, I just booted the laptop, still need to auth against aaaall of the services :D | 05:45 |
| chandankumar | sure take your time :-) | 05:45 |
| Tengu | at least coffee is already making its effect :] More or less. | 05:46 |
| Tengu | chandankumar: commented out | 05:54 |
| Tengu | main concern is about the ceph related packages you remove - aren't they of any use for CS8? | 05:54 |
| chandankumar | Tengu: good point, I will add the ceph packages with conditionals | 05:56 |
| Tengu | :) | 05:57 |
| chandankumar | Tengu: regarding which command galera container needs it, may be bandini can tell better on that | 05:57 |
| chandankumar | I got your point regarding which | 05:58 |
| Tengu | chandankumar: yeah, I'm a bit worried RHEL/CS are on the same path as Debian in order to deprecate that command... | 05:59 |
| Tengu | it would be a really, really bad move imho, since `command -v' doesn't do the same... | 05:59 |
| dciabrin | Tengu, o/ so I was told you're working on something on TCIB to support conditional builds for centos9? | 06:00 |
| Tengu | dciabrin: yep, with chandankumar | 06:01 |
| Tengu | dciabrin: https://review.opendev.org/c/openstack/python-tripleoclient/+/807098 | 06:01 |
| Tengu | you want to vote on it and make it merge | 06:01 |
| Tengu | dciabrin: that one is the key to the magic within TCIB | 06:01 |
| *** ysandeep|out is now known as ysandeep | 06:04 | |
| dciabrin | +2 guys, it turns out we need it to fix the mariadb container image for centos9 | 06:06 |
| Tengu | dciabrin: please, check with chandankumar and https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 06:06 |
| Tengu | dciabrin: he's already working on some container stuff, so you probably want to collaborate in order to ensure you're not doing the same and losing time ;) | 06:06 |
| chandankumar | Tengu: https://review.opendev.org/c/openstack/tripleo-common/+/800580/21/container-images/tcib/base/mariadb/mariadb.yaml#3 | 06:07 |
| chandankumar | dciabrin: ^^ | 06:07 |
| Tengu | :) | 06:07 |
| chandankumar | dciabrin: do we need the same for redis also? | 06:07 |
| Tengu | chandankumar: don't you have core on tripleo? or only tripleo-ci? I don't remember... | 06:07 |
| chandankumar | Tengu: I have core on tripleo but restricted to tripleo-ci | 06:08 |
| Tengu | chandankumar: ok. sigh... dciabrin maybe you can +W the patch? chandankumar has tested it already (you can maybe vote your +1, chandankumar, just for the show :)) | 06:08 |
| chandankumar | Done | 06:10 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/807174 | 06:10 |
| dciabrin | ack, one less in the queue :) | 06:12 |
| dciabrin | chandankumar, I'll push an update to https://review.opendev.org/c/openstack/tripleo-common/+/800580/21/container-images/tcib/base/mariadb/mariadb.yaml, we found out yesterday with bandini that we need a tweak for mariadb 10.5 | 06:12 |
| chandankumar | dciabrin: Can you suggest there in the patch itself | 06:13 |
| dciabrin | chandankumar, ack even better let me push that | 06:13 |
| * dciabrin prepares it | 06:13 | |
| Tengu | thanks dciabrin, chandankumar :) | 06:17 |
| * Tengu loves seeing successful collaboration | 06:17 | |
| chandankumar | dciabrin: my changes are also ready, let me know when to push | 06:30 |
| *** amoralej|off is now known as amoralej | 06:45 | |
| *** ykarel_ is now known as ykarel | 06:56 | |
| *** pojadhav is now known as pojadhav|afk | 07:00 | |
| opendevreview | Martin Schuppert proposed openstack/python-tripleoclient stable/victoria: Remove name parameter from cell export https://review.opendev.org/c/openstack/python-tripleoclient/+/807250 | 07:10 |
| Tengu | chandankumar: wondering...... this patck of mine should hit wallaby, shouldn't it? | 07:10 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: temp DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/807241 | 07:10 |
| opendevreview | Martin Schuppert proposed openstack/python-tripleoclient stable/ussuri: Remove name parameter from cell export https://review.opendev.org/c/openstack/python-tripleoclient/+/807251 | 07:11 |
| opendevreview | Martin Schuppert proposed openstack/python-tripleoclient stable/train: Remove name parameter from cell export https://review.opendev.org/c/openstack/python-tripleoclient/+/807252 | 07:11 |
| chandankumar | Tengu: Since it is a unique feature, we can backport it to till train, it is not going to break anything | 07:12 |
| Tengu | chandankumar: train has no actual use imho. | 07:13 |
| Tengu | wallaby, on the other hand... it's needed for osp-17 | 07:13 |
| Tengu | I'll start the backport once it's merged on master. | 07:13 |
| chandankumar | Tengu: sure | 07:17 |
| *** bhagyashris__ is now known as bhagyashris | 07:27 | |
| *** jpena|off is now known as jpena | 07:37 | |
| chandankumar | bandini: dciabrin new tag with which in base container a8b72470998eaa3e1039457bf1eb53f2 | 07:52 |
| chandankumar | pushed on registry | 07:52 |
| dciabrin | chandankumar, ack thx. i'm still preparing the tweak for mariadb | 07:52 |
| chandankumar | Tengu: https://logserver.rdoproject.org/53/18953/102/check/tripleo-build-containers-stream9-development/a06b19b/logs/container-builds/ff30c13b-d8b8-4050-ac3b-eebd8f2e9eae/base/os/horizon/Dockerfile | 07:53 |
| chandankumar | tcib working flawlessly | 07:53 |
| chandankumar | dciabrin: no problem, I created a new patch out of main one for testing | 07:53 |
| chandankumar | dciabrin: you can update the main patch, I will sync later on | 07:53 |
| bandini | chandankumar: excellent!! | 07:53 |
| bandini | chandankumar: I added issue 16 on the doc (usr/share/zoneinfo missing from images) | 07:54 |
| chandankumar | dciabrin: forked patch https://review.opendev.org/c/openstack/tripleo-common/+/807241 | 07:54 |
| chandankumar | bandini: looking into that | 07:54 |
| bandini | w00t | 07:54 |
| bandini | chandankumar++ | 07:54 |
| chandankumar | Tengu++ | 07:54 |
| chandankumar | bandini++ dciabrin++ for testing back and forth | 07:55 |
| Tengu | too easy :) | 07:56 |
| Tengu | bandini, dciabrin I'll backport that patch to wallaby so that it will exist in osp-17 | 07:57 |
| Tengu | and I guess the patches you're making now will also be needed in wallaby for the same reasons | 07:57 |
| bandini | yep yep | 07:57 |
| Tengu | at least, that's one of your issues solved, bandini :) | 07:58 |
| Tengu | or, at least, on the right track :) | 07:58 |
| bandini | w00t! | 07:59 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: DNR gross hack workaround for issue 16 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807246 | 07:59 |
| dciabrin | chandankumar, I added my comments on the mariadb TCIB config. we need to import an additional file from kolla to support mariadb 10.5. | 08:01 |
| chandankumar | dciabrin: awesome, thanks :-) | 08:02 |
| chandankumar | dciabrin: good point on == and = | 08:05 |
| dciabrin | it's a bit pedantic really... but hey | 08:06 |
| Tengu | yup - didn't see it.. | 08:06 |
| Tengu | dciabrin: you're French, it's in your nature :D | 08:06 |
| * Tengu runs and hides | 08:06 | |
| dciabrin | lol | 08:06 |
| * Tengu raises a «FRIDAY» sign | 08:07 | |
| dciabrin | friday indeed :) | 08:08 |
| chandankumar | sorry | 08:10 |
| chandankumar | https://logserver.rdoproject.org/53/18953/102/check/tripleo-build-containers-stream9-development/a06b19b/logs/container-builds/ff30c13b-d8b8-4050-ac3b-eebd8f2e9eae/base/base-build.log | 08:10 |
| chandankumar | It is working | 08:10 |
| chandankumar | it's friday | 08:10 |
| chandankumar | last time I checked at wrong place | 08:10 |
| Tengu | bandini: what's supposed to provide that zoneinfo? | 08:13 |
| Tengu | tzdata... ok. so it's missing... | 08:13 |
| bandini | yeah tzdata rpm is installed but /usr/share/zoneinfo (part of tzdata) is missing | 08:15 |
| Tengu | o_O | 08:16 |
| Tengu | incomplete package? that's... well. | 08:16 |
| Tengu | pfrrtt | 08:16 |
| Tengu | maybe it's installed in some other location? | 08:16 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: [WIP] Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 08:16 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Dependency reorganization https://review.opendev.org/c/openstack/validations-common/+/807248 | 08:18 |
| opendevreview | Adriano Petrich proposed openstack/tripleo-common stable/victoria: Add a retry to the instropection waiting workbook https://review.opendev.org/c/openstack/tripleo-common/+/807249 | 08:18 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/807174 | 08:19 |
| jpodivin | zbr: hi, do you have a moment? | 08:20 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: [WIP] Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 08:21 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates master: WIP DNM/DNR Add support for ovn bgp agent https://review.opendev.org/c/openstack/tripleo-heat-templates/+/785666 | 08:24 |
| opendevreview | Marios Andreou proposed openstack/tripleo-quickstart master: Use tripleo.repos.get_hash in repo-setup role https://review.opendev.org/c/openstack/tripleo-quickstart/+/791486 | 08:32 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Dependency reorganization https://review.opendev.org/c/openstack/validations-common/+/807248 | 08:33 |
| *** ykarel is now known as ykarel|lunch | 08:36 | |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: [WIP] Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 08:36 |
| Tengu | bandini: I think your https://review.opendev.org/c/openstack/tripleo-ansible/+/806837 isn't working due to a hard-coded ConmonPidFile at the container creation. | 08:37 |
| Tengu | bandini: not 100% sure, since /var/run is supposed to be a symlink to /run, therefore content should match and be equal, but with podman/conmon....... we never know. | 08:38 |
| bandini | yeah it must be something silly at the lower levels | 08:38 |
| bandini | i mean it is low prio stuff, just annoying as it gets logged quite a bit | 08:38 |
| Tengu | yup | 08:38 |
| * bandini errand | 08:38 | |
| Tengu | guess we'll have to do some silly jumps at some point to change this. | 08:39 |
| chandankumar | bandini: Tengu https://gist.githubusercontent.com/raukadah/96a57cddb6ffd2129e3568c4414a4b6b/raw/568168290b7f262187c1cd1e0a10fc3f1cdc59ce/gistfile1.txt | 08:39 |
| chandankumar | tzdata is in base image | 08:39 |
| chandankumar | the dir itself does not exist there | 08:40 |
| Tengu | weird. | 08:40 |
| chandankumar | in rpm it is there | 08:40 |
| Tengu | did they remove that location during the image build? | 08:40 |
| Tengu | that would be plain stupid. | 08:40 |
| chandankumar | need to check the logs | 08:42 |
| dciabrin | chandankumar, Tengu a rpm -V shows that the package was installed but the files got removed at some point at container build time. | 08:44 |
| *** pojadhav|afk is now known as pojadhav | 08:46 | |
| chandankumar | time to open a bug then | 08:47 |
| dciabrin | chandankumar, I'll open it with the details | 08:47 |
| Tengu | dciabrin: thanks :) | 08:48 |
| chandankumar | dciabrin: https://gist.github.com/raukadah/96a57cddb6ffd2129e3568c4414a4b6b | 08:48 |
| chandankumar | more info here also | 08:48 |
| chandankumar | it will help | 08:48 |
| Tengu | do you know the location of the Dockerfile? | 08:48 |
| dciabrin | I have no idea how those containers are built lately :( | 08:49 |
| chandankumar | checking on #centos-devel | 08:50 |
| Tengu | https://github.com/CentOS/sig-cloud-instance-build/ | 08:50 |
| Tengu | hmmm. nope. no centos9 thing there. | 08:51 |
| Tengu | dang. | 08:51 |
| chandankumar | https://gitlab.com/redhat/centos-stream | 08:51 |
| chandankumar | need to dig where it is | 08:51 |
| Tengu | not sure the container image is built from there... ? | 08:52 |
| dciabrin | https://bugs.launchpad.net/tripleo/+bug/1942590 | 08:52 |
| Tengu | dciabrin: guess it should be open against CentOS instead | 08:53 |
| Tengu | but at least we have a trace on tripleo side | 08:53 |
| dciabrin | Tengu, i think the centos package itself is fine, because we see rpm complaining about missing files | 08:53 |
| chandankumar | https://gitlab.com/redhat/centos-stream/release-engineering/kickstarts/-/blob/main/CentOS-Stream-9-container-base.ks | 08:53 |
| dciabrin | i mean missing files in the container vs what is shipped in the rpm | 08:54 |
| chandankumar | https://gitlab.com/redhat/centos-stream/release-engineering/kickstarts | 08:54 |
| Tengu | chandankumar: oh! | 08:54 |
| Tengu | https://gitlab.com/redhat/centos-stream/release-engineering/kickstarts/-/blob/main/CentOS-Stream-9-container-base.ks#L49-50 | 08:54 |
| Tengu | seriously... | 08:54 |
| Tengu | so it is intended. | 08:54 |
| dciabrin | gah :( | 08:54 |
| Tengu | that's so fucking wrong | 08:54 |
| chandankumar | dciabrin: send a pull request | 08:55 |
| Tengu | not as if the size of that location is huge.. | 08:55 |
| Tengu | ... ok. yes. it is. 4.9M /usr/share/zoneinfo/ | 08:56 |
| Tengu | almist 5Mo for files.. | 08:56 |
| Tengu | *almost | 08:56 |
| Tengu | chandankumar: maybe the right thing to do then is to reinstall tzdata with tcib? | 08:56 |
| Tengu | or bind-mount from the host and be done with it | 08:56 |
| Tengu | after all, bind-mounting is a fine solution as well. | 08:56 |
| dciabrin | either way will work. bandini started the bind-mount dance before we figured out why the files were gone | 08:57 |
| chandankumar | let me do that | 08:57 |
| Tengu | yeah... well. bind-mount is probably better imho, chandankumar | 08:57 |
| Tengu | iirc we can even make that bind-mount global. | 08:57 |
| Tengu | directly in deployment/containers-common.yaml | 08:58 |
| Tengu | using the volumes_base maybe | 08:58 |
| Tengu | dciabrin, bandini, chandankumar -^^ wdyt? bind-mount /usr/share/zoneinfo in all the containers, read-only, using deployment/containers-common.yaml "volumes_base". better than installing 5Mo of duplicated data? | 08:59 |
| chandankumar | Tengu: sounds good to me | 09:00 |
| Tengu | lemme give it a try | 09:00 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: DNR gross hack workaround for issue 16 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807246 | 09:00 |
| Tengu | we already have /etc/localtime and some other things. | 09:00 |
| bandini | why not I guess | 09:01 |
| Tengu | lemme push | 09:02 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Bind-mount /usr/share/zoneinfo https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807275 | 09:02 |
| Tengu | bandini: -^ | 09:02 |
| Tengu | of course.... feel free to update your own hack and drop mine ;) | 09:02 |
| bandini | Tengu: I'll move to yours once I figure out this new other issue ;) | 09:03 |
| Tengu | and chandankumar++ for finding that kickstart file! | 09:03 |
| Tengu | bandini: "new other"? | 09:04 |
| * chandankumar scratch my mind what I was doing last. | 09:04 | |
| bandini | yeah issue 18 is the next one, still looking at it | 09:04 |
| Tengu | ah, let me open that doc again | 09:04 |
| Tengu | ah. no content :D | 09:05 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: temp DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 09:05 |
| chandankumar | building the container again with mariadb hack | 09:06 |
| chandankumar | bandini: Tengu for right now, will the build the container will reinstallation of tzdata in base container? | 09:08 |
| chandankumar | till Tengu cedric fix lands | 09:09 |
| chandankumar | ? | 09:09 |
| Tengu | maybe? that would help dropping a hacky patch | 09:09 |
| chandankumar | let me update then | 09:09 |
| Tengu | but I have to double check the volumes_base are used in nova (and others) just to be sure. | 09:09 |
| bandini | we can do either i guess | 09:09 |
| Tengu | seems to be used everywhere. yay! | 09:10 |
| Tengu | ok, most are using {get_attr: [ContainersCommon, volumes]} - that "volumes" includes the volumes_base | 09:11 |
| Tengu | so we're on the safe side. | 09:11 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: temp DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 09:13 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Dependency reorganization https://review.opendev.org/c/openstack/validations-common/+/807248 | 09:23 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: [WIP] Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 09:27 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: [WIP] Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 09:28 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/807174 | 09:29 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: temp DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 09:35 |
| opendevreview | Merged openstack/python-tripleoclient master: Allow to pass arbitrary parameters to TCIB image config https://review.opendev.org/c/openstack/python-tripleoclient/+/807098 | 09:36 |
| opendevreview | Merged openstack/tripleo-ansible master: Fix molecule job for tripleo_ha_wrapper https://review.opendev.org/c/openstack/tripleo-ansible/+/807027 | 09:36 |
| opendevreview | Michele Baldessari proposed openstack/puppet-pacemaker master: Make sure we test for Promotable when looking for constraints https://review.opendev.org/c/openstack/puppet-pacemaker/+/807280 | 09:39 |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Updating the queries https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/807281 | 09:44 |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient stable/wallaby: Allow to pass arbitrary parameters to TCIB image config https://review.opendev.org/c/openstack/python-tripleoclient/+/807257 | 09:48 |
| Tengu | chandankumar, bandini -^^ | 09:48 |
| bandini | oh excellent | 09:48 |
| Tengu | backport on its way! | 09:48 |
| Tengu | master just merged. | 09:48 |
| Tengu | bandini, chandankumar I think we should use a common "hashtag" on our cs9-related patches. That would make the search/tracking easier, and we'll be able to quickly know about potential missing things in Wallaby. | 09:50 |
| Tengu | something like "cs9": https://review.opendev.org/q/hashtag:%22cs9%22+(status:open%20OR%20status:merged) | 09:51 |
| bandini | cs9 works for me | 09:51 |
| Tengu | perfect. | 09:51 |
| Tengu | lemme tag my things already! | 09:51 |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Updating the queries https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/807281 | 09:51 |
| Tengu | bandini: added a note in bold in your doc. | 09:53 |
| Tengu | bold and red. bam. | 09:54 |
| Tengu | a pity there's no "blink" nor "pre" XD | 09:54 |
| Tengu | hmmm... or was it "marquee"... damn '90s Internet :D | 09:55 |
| bandini | ack +1 | 09:55 |
| bandini | dciabrin: ^ set cs9 as tag in your reviews for this stuff | 09:55 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 09:56 |
| dciabrin | bandini, ack | 09:57 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 09:57 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 09:59 |
| Tengu | bandini: I also just dropped a mail on the ML. That will make everything easier. | 10:01 |
| bandini | ok | 10:02 |
| *** ykarel|lunch is now known as ykarel | 10:07 | |
| opendevreview | Giulio Fidente proposed openstack/tripleo-heat-templates stable/ussuri: Enable Ceph RGW public port into firewall when using SSL https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807259 | 10:14 |
| jpodivin | sshnaidm[m]: hi, I've no idea if you are there but if you are, I need to talk to you about the https://review.opendev.org/c/openstack/validations-common/+/806681 | 10:33 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: WIP - Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 10:44 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates stable/wallaby: Add base Neutron service plugins to BGPVPN https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807260 | 10:55 |
| *** jcapitao is now known as jcapitao_lunch | 10:59 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Consider GlanceMultistoreConfig when setting glance_pool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/806901 | 11:03 |
| Tengu | chandankumar: care to add the "cs9" hashtag to https://review.opendev.org/c/openstack/tripleo-common/+/800580 ? | 11:04 |
| Tengu | even if it's a temporary hack, at least it will show up in the query. | 11:05 |
| Tengu | oh. or maybe it's even on Ronelle only. Not sure who can set those tags.. | 11:05 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: WIP - Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 11:05 |
| *** dviroel|out is now known as dviroel | 11:21 | |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: Extend fencing to hosts using fence_kubevirt agent. https://review.opendev.org/c/openstack/puppet-tripleo/+/807067 | 11:24 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Dependency reorganization https://review.opendev.org/c/openstack/tripleo-validations/+/807292 | 11:24 |
| opendevreview | Merged openstack/puppet-tripleo master: haproxy: frontend/backend syntax in config https://review.opendev.org/c/openstack/puppet-tripleo/+/801075 | 11:27 |
| chandankumar | Tengu: ok will do that | 11:31 |
| Tengu | chandankumar: thanks :) | 11:33 |
| chandankumar | dciabrin: \o /usr/share/openstack-tripleo-common-containers/container-images/kolla/mariadb/security_reset.expect.debian does not exists on c9 | 11:35 |
| chandankumar | dciabrin: https://logserver.rdoproject.org/53/18953/104/check/tripleo-build-containers-stream9-development/61164f5/logs/container-builds/e8ca4560-dd91-402b-91bf-d0a5d2337852/base/mariadb/mariadb-build.log | 11:35 |
| chandankumar | or am i missing something? | 11:35 |
| chandankumar | dciabrin: sorry my bad | 11:37 |
| chandankumar | misread the comment | 11:37 |
| *** jpena is now known as jpena|lunch | 11:38 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Enforce root user at the beginning of container build https://review.opendev.org/c/openstack/tripleo-ansible/+/807297 | 11:38 |
| *** rlandy is now known as rlandy|rover | 11:39 | |
| opendevreview | Martin Schuppert proposed openstack/puppet-pacemaker master: Support fence_kubevirt apiversion parameter https://review.opendev.org/c/openstack/puppet-pacemaker/+/807299 | 11:39 |
| *** pojadhav is now known as pojadhav|brb | 11:39 | |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: temp DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 11:40 |
| chandankumar | bandini: Tengu: collectd container is coming to c9 | 11:42 |
| Tengu | chandankumar: that's a really good news for mrunge and his team! | 11:42 |
| bandini | w00t | 11:44 |
| *** pojadhav|brb is now known as pojadhav | 11:48 | |
| opendevreview | Rajesh Tailor proposed openstack/puppet-tripleo master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/puppet-tripleo/+/803911 | 11:55 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: DNM test https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807301 | 11:57 |
| *** jcapitao_lunch is now known as jcapitao | 12:27 | |
| Tengu | bandini: oh, btw - I've tested to update a systemd unit, moving the PidFile of a running container, and I could just stop'n'start it... | 12:28 |
| Tengu | so there's probably something else somewhere hitting some dark corners. | 12:28 |
| bandini | Tengu: aha interesting | 12:30 |
| opendevreview | Michele Baldessari proposed openstack/puppet-pacemaker master: Make sure we test for Promoted when looking for constraints https://review.opendev.org/c/openstack/puppet-pacemaker/+/807280 | 12:33 |
| Tengu | bandini: still - I think your patch is missing one change, for the ConmonPidFile value | 12:36 |
| *** amoralej is now known as amoralej|lunch | 12:36 | |
| bandini | Tengu: feel free to amend and we see if ci loves it? | 12:36 |
| Tengu | bandini: I'll push a thing in a few then :) | 12:37 |
| bandini | w00t, am chasing another thing atm | 12:38 |
| bandini | thanks! | 12:38 |
| *** jpena|lunch is now known as jpena | 12:39 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Systemd PIDFile needs to be in /run not in /var/run https://review.opendev.org/c/openstack/tripleo-ansible/+/806837 | 12:42 |
| Tengu | bandini: -^ | 12:42 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Update validations for Ansible 2.11 https://review.opendev.org/c/openstack/validations-common/+/806681 | 12:44 |
| opendevreview | Kevin Carter proposed openstack/tripleo-ansible stable/wallaby: Update stack delete to support local stacks https://review.opendev.org/c/openstack/tripleo-ansible/+/807264 | 12:45 |
| opendevreview | Kevin Carter proposed openstack/tripleo-heat-templates stable/wallaby: Fix playbook_dir Ansible Warning https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807265 | 12:45 |
| opendevreview | Kevin Carter proposed openstack/tripleo-heat-templates stable/wallaby: Fix playbook_dir Ansible Warning https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807265 | 12:49 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Added pre from centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 12:51 |
| *** chem is now known as Guest6174 | 13:01 | |
| opendevreview | Rajesh Tailor proposed openstack/puppet-tripleo master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/puppet-tripleo/+/803911 | 13:12 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-docs master: Document adding files into TCIB build environment https://review.opendev.org/c/openstack/tripleo-docs/+/807173 | 13:14 |
| waleedm | Hi guys, I'm facing a problem with installing tripleo using quickstart because unstable repos for Centos, any one faing it and what the solution for it ? | 13:19 |
| *** amoralej|lunch is now known as amoralej | 13:25 | |
| opendevreview | Rajesh Tailor proposed openstack/puppet-tripleo master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/puppet-tripleo/+/803911 | 13:35 |
| rlandy|rover | waleedm: " unstable repos for Centos" do you mean moving a changing centos 8 stream? | 13:40 |
| rlandy|rover | if that is your issue, there are composes for centos 8 stream https://composes.centos.org/ - you can choose to install centos from a particular compose you know works | 13:42 |
| waleedm | ratailor, not its failing to run dnf update or dnf repolist, for connection refused, I tried to run it manullay sometime it's ok and sometime same error | 13:43 |
| waleedm | for example this oneErrors during downloading metadata for repository 'centos-rabbitmq-38': | 13:46 |
| waleedm | - Curl error (7): Couldn't connect to server for http://mirror.centos.org/centos/8/messaging/x86_64/rabbitmq-38/repodata/repomd.xml [Failed to connect to mirror.centos.org port 80: Connection refused] | 13:46 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Added pre from centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 13:48 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 13:49 |
| rlandy|rover | waleedm: oh mirrors | 13:50 |
| rlandy|rover | waleedm: we do hit mirror issue from time to time ... we don't have a fix for broken centos mirrors | 13:51 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/wallaby: NovaLiveMigrationPermitAutoConverge should default to True https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807266 | 14:02 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/victoria: NovaLiveMigrationPermitAutoConverge should default to True https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807267 | 14:02 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/ussuri: NovaLiveMigrationPermitAutoConverge should default to True https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807268 | 14:02 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: NovaLiveMigrationPermitAutoConverge should default to True https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807269 | 14:02 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: [DNM] Disable sanity test https://review.opendev.org/c/openstack/tripleo-upgrade/+/807339 | 14:15 |
| tkajinam | rlandy|rover, are we using an external mirror instead of the one hosted inside infra ? | 14:20 |
| tkajinam | looking at the url waleedm, seems like so | 14:20 |
| tkajinam | we might be able to use internal mirror instead. Infra provides the one hosted internally, like inside ovh | 14:21 |
| tkajinam | like http://mirror.gra1.ovh.opendev.org/centos/ | 14:21 |
| mwhahaha | we use mirrors | 14:21 |
| mwhahaha | i don't know if waleedm is talking about CI | 14:22 |
| tkajinam | ah, ok | 14:22 |
| tkajinam | I should scroll up the log for a bit, my bad | 14:22 |
| rlandy|rover | tkajinam: yes - the ci jobs reference the mirror from the cloud provider if availble | 14:23 |
| tkajinam | rlandy|rover, ack, thanks. sorry I misunderstood the discussion | 14:25 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: temp DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 14:25 |
| chandankumar | bandini: Tengu mrunge I am working on half baked collected c9 container https://review.opendev.org/c/openstack/tripleo-common/+/800580/26/container-images/tcib/base/collectd/collectd.yaml once it is available I will let you know. | 14:30 |
| bandini | ack, thanks! | 14:30 |
| opendevreview | alexey-mr proposed openstack/os-net-config master: Allow linux tap to use routes https://review.opendev.org/c/openstack/os-net-config/+/807160 | 14:33 |
| waleedm | rlandy|rover, Is there any workaround for now for solving this issue | 14:37 |
| *** ykarel is now known as ykarel|away | 14:44 | |
| rlandy|rover | waleedm: not really - unless you try other mirrors that may be available when one is not | 14:50 |
| rlandy|rover | this should not be a consistent problem | 14:50 |
| waleedm | rlandy|rover, so the other solution to install the tripleo not using quickstart | 14:51 |
| rlandy|rover | waleedm; you'll still run into mirror issues if you need anything from centos | 14:52 |
| ysandeep | hjensas: hey o/ small query regarding network-datav2, It is neccessary to pass network-data.yaml using --networks-file in deploy command, even if we are doing network/vip/node provision with --network-config in prior steps? | 15:05 |
| hjensas | ysandeep: yes, the networks-file is used to render THT jinja2. | 15:05 |
| ysandeep | hjensas, ah thanks, and whats the difference between node provision with --network-ports vs --network-config | 15:08 |
| ysandeep | , My understanding is os-net-config only run when we pass --network-config in node provision step. | 15:08 |
| ysandeep | ^^ correct? | 15:09 |
| opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: Added stack_name variable to overcloud-delete defaults https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/805260 | 15:09 |
| hjensas | ysandeep: --network-ports creates the ports/ip allocation and populate the environments requred for the Heat stack. '--network-config' enable running os-net-config on the nodes via the ansible role tripleo_network_config prior to creating the stack. | 15:11 |
| hjensas | ysandeep: --network-ports is deprecated, it is enabled by default since some time. | 15:11 |
| opendevreview | Merged openstack/tripleo-repos master: Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 15:11 |
| ysandeep | hjensas, thanks! We are passing --network-ports in ci.. If its default.. I think we remove that additional param. | 15:12 |
| ysandeep | hjensas++ | 15:13 |
| hjensas | ysandeep: yes that makes sense, both master and W enables it by default - https://review.opendev.org/q/Ia0df205ea568f089c7cb7e5464bde76d294f4c36 | 15:14 |
| opendevreview | Merged openstack/puppet-pacemaker master: Support fence_kubevirt apiversion parameter https://review.opendev.org/c/openstack/puppet-pacemaker/+/807299 | 15:48 |
| *** amoralej is now known as amoralej|off | 16:06 | |
| opendevreview | Ade Lee proposed openstack/tripleo-specs master: Add spec for secret management https://review.opendev.org/c/openstack/tripleo-specs/+/807361 | 16:23 |
| *** hjensas is now known as hjensas|afk | 16:29 | |
| ade_lee | jpodivin,hey - how did the certmnger thing go? were you able to get the right version? | 16:31 |
| rlandy|rover | ade_lee: hey ... do we have a legit failure here? https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8ac/806627/2/gate/tripleo-ci-centos-8-standalone-on-multinode-ipa/8acb05e/logs/undercloud/home/zuul/standalone_deploy.log | 16:31 |
| ade_lee | rlandy|rover, /me going back to hiding .. | 16:32 |
| rlandy|rover | gate failuer | 16:32 |
| rlandy|rover | ade_lee: no, no ... we have an agreement here | 16:33 |
| ade_lee | :) | 16:33 |
| rlandy|rover | https://zuul.opendev.org/t/openstack/builds?job_name=tripleo-ci-centos-8-standalone-on-multinode-ipa | 16:33 |
| rlandy|rover | history is somewhat spotty - checking if it's all the same failure | 16:34 |
| ade_lee | rlandy|rover, is this a consistent failure? | 16:34 |
| rlandy|rover | that is what I am checking the history for | 16:34 |
| ade_lee | rlandy|rover, this is on master? | 16:34 |
| rlandy|rover | ade_lee: yes | 16:34 |
| rlandy|rover | ade_lee: the same job passed on check | 16:36 |
| ade_lee | rlandy|rover, so it seems to be intermittent | 16:36 |
| rlandy|rover | so will rekick | 16:36 |
| rlandy|rover | but just keeping on eye on this one | 16:36 |
| rlandy|rover | master promoted yesterday - so we'll check the trend | 16:36 |
| ade_lee | yeah -- I have to just into a meeting - but I'll try to dig in later today | 16:36 |
| rlandy|rover | oh the joys of life with a voting/gate job | 16:37 |
| ade_lee | rlandy|rover, ack let me know if its happening a lot | 16:37 |
| rlandy|rover | ade_lee: no yet - I'll ping you if you need to investigate | 16:37 |
| ade_lee | ok - we've seen this error before - and it might be a timing thing | 16:37 |
| *** ysandeep is now known as ysandeep|out | 16:41 | |
| *** jpena is now known as jpena|off | 16:41 | |
| opendevreview | Ronelle Landy proposed openstack/openstack-tempest-skiplist master: Revert "Skip test_attach_volume_negative tempest test" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/806520 | 17:26 |
| *** odyssey4me is now known as Guest6198 | 18:03 | |
| *** dviroel is now known as dviroel|af | 19:35 | |
| *** dviroel|af is now known as dviroel|afk | 19:35 | |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: Revert "tripleo-podman: Enable podman.socket" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807376 | 19:52 |
| opendevreview | Lance Bragstad proposed openstack/tripleo-heat-templates master: DNM: Provide a template to enable project personas https://review.opendev.org/c/openstack/tripleo-heat-templates/+/781571 | 21:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!