Tuesday, 2022-01-04

*** rlandy|ruck is now known as rlandy|ruck|out01:00
*** rlandy|ruck|out is now known as rlandy|out01:00
opendevreviewBrendan Shephard proposed openstack/python-tripleoclient master: WIP - Move baremetal CLI commands  https://review.opendev.org/c/openstack/python-tripleoclient/+/82046302:53
opendevreviewBrendan Shephard proposed openstack/python-tripleoclient master: Move baremetal CLI commands  https://review.opendev.org/c/openstack/python-tripleoclient/+/82046302:55
*** pojadhav|pto is now known as pojadhav05:02
opendevreviewMerged openstack/tripleo-heat-templates master: Use public endpoint for [keystone_authtoken] www_authenticate_uri  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/82230605:34
*** akahat|PTO is now known as akahat06:02
*** akahat is now known as akahat|ruck06:30
opendevreviewchandan kumar proposed openstack/openstack-tempest-skiplist master: Move test_trunk_subport_lifecycle to skip list  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/82333206:53
*** ysandeep is now known as ysandeep|afk07:14
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: set NovaLibvirtNumPciePorts to 12  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/82234407:51
opendevreviewchandan kumar proposed openstack/tripleo-quickstart-extras master: Use Python3 based http.server command  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/82278308:01
opendevreviewamolkahat proposed openstack/tripleo-quickstart-extras master: set NovaLibvirtNumPciePorts to 12  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/82234408:08
*** amoralej|off is now known as amoralej08:09
*** ykarel_ is now known as ykarel08:23
*** bhagyashris_ is now known as bhagyashris08:45
opendevreviewShnaidman Sagi (Sergey) proposed openstack/tripleo-repos master: Install priorities of CentOS7 only on CentOS7 host  https://review.opendev.org/c/openstack/tripleo-repos/+/82183709:11
*** ysandeep|afk is now known as ysandeep09:16
opendevreviewchandan kumar proposed openstack/tripleo-quickstart-extras master: Inject gating gating repo during image build  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/82289709:25
opendevreviewJiri Podivin proposed openstack/validations-libs master: Redundant conditional removal  https://review.opendev.org/c/openstack/validations-libs/+/80273709:32
opendevreviewRicardo Diaz Campos proposed openstack/tripleo-quickstart master: Set FQDN hostname to undercloud.qcow2  https://review.opendev.org/c/openstack/tripleo-quickstart/+/82332110:03
opendevreviewJiri Podivin proposed openstack/validations-libs master: Validation actions refactor  https://review.opendev.org/c/openstack/validations-libs/+/80290110:10
opendevreviewchandan kumar proposed openstack/tripleo-common master: Remove /etc/yum.repos.d/ubi.repo  https://review.opendev.org/c/openstack/tripleo-common/+/82337010:11
opendevreviewMerged openstack/tripleo-repos master: Limit stream validation for CentOS 8 only  https://review.opendev.org/c/openstack/tripleo-repos/+/82183410:14
*** ysandeep is now known as ysandeep|coffee10:15
*** jcapitao is now known as jcapitao_afk10:34
*** chem is now known as Guest1021810:37
*** rlandy|out is now known as rlandy|ruck11:02
*** jcapitao_afk is now known as jcapitao11:15
*** dviroel_ is now known as dviroel|rover11:16
*** ysandeep|coffee is now known as ysandeep11:23
opendevreviewMerged openstack/tripleo-quickstart master: Sync collections requirements from tripleo-ci  https://review.opendev.org/c/openstack/tripleo-quickstart/+/82169111:50
opendevreviewBrendan Shephard proposed openstack/tripleo-quickstart master: Allow for more ganular Python3 version control  https://review.opendev.org/c/openstack/tripleo-quickstart/+/82258612:00
Tenguchandankumar, rlandy|ruck hello there! apparently, Change-Id: I23c64637e161c8054332c4e4761cd82802f3ae1f introduced an issue when we want to build containers based on centos:stream8 instead of ubi8. The "redhat-release" package may be absent, and the following code will crash the "base" build: https://paste.opendev.org/show/811909/12:17
Tenguchandankumar, rlandy|ruck imho, instead of checking for the release version, we should check the presence of the package. Would be safer. WDYT?12:17
rlandy|ruckon meeting - will look in a few12:17
Tengurlandy|ruck: np12:17
Tengujust hitting it on my new container build thing :)12:18
chandankumarTengu: redhat-release utility is a part of centos-stream-release i think for cs912:18
Tenguchandankumar: cs9? so here it's removing it for cs812:19
Tenguchandankumar: still - checking if the package is installed would be better12:19
Tenguif [ rpm -qa redhat-release ]; then rpm -e redhat-release; fi    or something like that12:19
Tenguwe're covering every cases for centos based builds.12:19
Tenguand we might want to `dnf install --exclude redhat-release centos-stream-release' - though not 100% sure it will work, that "--exclude" option is terrible.12:20
chandankumarTengu: for cs9 yes, sounds good for checking presence of package12:21
Tengu'k. lemme push a proposal.12:21
chandankumarTengu: we have got cs9 and c8 content provider in check12:22
chandankumarwe will know about that12:22
Tenguchandankumar: ah. no cs8 then? might explain my issues.12:22
Tengu...... `rpm -qa redhat-release' will return 0 in any cases.12:23
Tengudumb software.12:23
Tenguagain.12:23
chandankumarTengu: we have c8 and c9 both currently https://zuul.openstack.org/status#82337012:24
Tenguchandankumar: let's say I'm a bit ahead again :).12:25
Tenguif [ -n "$(rpm -qa redhat-release)" ]; then rpm -e ....  seems fine.12:26
opendevreviewCedric Jeanneret proposed openstack/tripleo-common master: Remove redhat-release only if installed  https://review.opendev.org/c/openstack/tripleo-common/+/82338412:27
Tenguchandankumar, rlandy|ruck -^^12:27
Tengulemme know if the commit message is correct - I'm lost with what I read in the code, and what chandankumar just said about the cs9 thing...12:27
rlandy|ruckTengu: ok - I remember we had some fun with redhat-release and container builds12:29
rlandy|ruckhence sshnaidm added that line12:29
rlandy|ruckTengu: why the switch away from ubi8?12:29
Tengurlandy|ruck: oh, well, I'm making a thing locally that will build containers for both centos8 and cs9 base, and it was easier to put "centos:stream${release}" instead of doing some kind of switching :).12:31
Tengurlandy|ruck: that's "just me" doing potentially silly thing I guess :)12:31
rlandy|ruckTengu: the aim was to use the same base for centos and rhel12:31
rlandy|ruckubi9 was not available12:32
rlandy|ruckand is only beta now12:32
rlandy|ruckso we were using stream12:32
Tenguyeah...12:32
Tenguso I'll switch back to ubi8 for centos-8 builds, if it makes things better. though my patch would prevent the need to change anything when we switch back to ubi9 I think?12:32
chandankumarTengu: for ubi9 you need this one first https://review.opendev.org/c/openstack/tripleo-common/+/82337012:34
Tenguchandankumar: ah. yay :D12:34
Tenguchandankumar: reviewed12:38
opendevreviewMerged openstack/tripleo-quickstart-extras master: Install ansible-core in CentOS 9 for github PRs  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/82325112:43
chandankumarTengu: thanks!, will take a look12:44
*** jcapitao is now known as jcapitao_lunch12:50
opendevreviewMerged openstack/tripleo-ci master: Allow hash override for emit releases script for minor update  https://review.opendev.org/c/openstack/tripleo-ci/+/82248212:51
*** beagles is now known as beagles-afk13:06
*** amoralej is now known as amoralej|lunch13:09
cloudnullmornings 13:14
*** jcapitao_lunch is now known as jcapitao13:33
opendevreviewMerged openstack/openstack-tempest-skiplist master: Move test_trunk_subport_lifecycle to skip list  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/82333213:38
slaglei see no proposed meeting agenda items, so I propose we cancel. any objections?13:56
fultonjno objection13:56
*** ykarel__ is now known as ykarel13:57
ykarelchandankumar, sshnaidm when you get chance please check https://review.opendev.org/c/openstack/tripleo-quickstart/+/82271214:03
ykareland depends-on14:03
opendevreviewchandan kumar proposed openstack/tripleo-common master: Remove /etc/yum.repos.d/ubi.repo  https://review.opendev.org/c/openstack/tripleo-common/+/82337014:05
*** amoralej|lunch is now known as amoralej14:14
chandankumardciabrin: hello, please have a look at https://logserver.rdoproject.org/openstack-periodic-integration-main-cs9/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-scenario004-standalone-master/d7bde3d/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz error={"changed": false, "msg": "Failed containers: rabbitmq_wait_bundle"}14:17
chandankumarit keeps coming time to time14:17
dciabrinchandankumar, hey o/ looking into it14:18
*** ysandeep is now known as ysandeep|afk14:41
*** beagles-afk is now known as beagles14:43
opendevreviewAnanya proposed openstack/tripleo-docs master: Updating promotion/CIX related info in ruck rover doc  https://review.opendev.org/c/openstack/tripleo-docs/+/82339914:43
opendevreviewAnanya proposed openstack/tripleo-docs master: Updating promotion/CIX/ci-health related info in ruck rover doc  https://review.opendev.org/c/openstack/tripleo-docs/+/82339914:47
*** dviroel|rover is now known as dviroel|rover|lunch14:57
mwhahaharlandy|ruck: does the quickstart reproducer still work? like if i wanted to reproduce tha ipa job myself somewhere15:11
opendevreviewMerged openstack/tripleo-ci master: [CS9] Add missing CS9 jobs defintion  https://review.opendev.org/c/openstack/tripleo-ci/+/82249615:11
opendevreviewMerged openstack/tripleo-ci master: Enable remaining CS9 jobs in tripleo  https://review.opendev.org/c/openstack/tripleo-ci/+/82249715:11
rlandy|ruckmwhahaha: it should - sshnaidm just got it working for c915:16
rlandy|ruckso c8 should work if you use cloud server nodes15:16
sshnaidmmwhahaha, ipa - multinode, standalone or ovb?15:17
mwhahahatripleo-ci-centos-8-standalone-on-multinode-ipa15:18
mwhahahalooking into that whole failure thing w/ ntp validation15:18
opendevreviewAlex Schultz proposed openstack/tripleo-ci master: Revert "Make standalone-ipa job non-voting during debug"  https://review.opendev.org/c/openstack/tripleo-ci/+/82279215:18
sshnaidmyeah, standalone works fine in reproducer15:18
mwhahahahttps://bfff4db8bf407324525b-0442bcedc0a9bf69aaea3fa941004c5a.ssl.cf5.rackcdn.com/822948/2/check/tripleo-ci-centos-8-standalone-on-multinode-ipa/8f96695/logs/reproducer-quickstart/index.html that thing?15:19
mwhahahai haven't used the reproducer in years at this point15:19
rlandy|ruckhttps://github.com/rdo-infra/ansible-role-tripleo-ci-reproducer/blob/master/README.md15:21
rlandy|ruckthat should be the latest15:21
mwhahahak i'll give it a whirl15:21
mwhahahahappy fake monday15:21
*** ysandeep|afk is now known as ysandeep15:21
rlandy|ruckI had real monday yesterday15:21
mwhahahai saw15:21
rlandy|ruckhad no idea we had a hokiday :(15:21
mwhahahai double checked on sunday cause i had a feeling :D15:22
sshnaidmmwhahaha, bogdan has a big experience with running it with libvirt: https://github.com/bogdando/oooq-warp/blob/master/docs/CI-reproducer.md15:23
mwhahahapffft docker15:23
sshnaidmmwhahaha, yeah, but reproducer can run as a pod now15:25
sshnaidmidk why he uses docker..15:26
mwhahahaprobably ubuntu15:26
mwhahahaAborting, target uses selinux but python bindings (libselinux-python) aren't installed!15:31
mwhahahaoh my favorite15:31
*** dviroel|rover|lunch is now known as dviroel|rover15:48
opendevreviewMerged openstack/tripleo-repos master: Install priorities of CentOS7 only on CentOS7 host  https://review.opendev.org/c/openstack/tripleo-repos/+/82183716:04
mwhahaharlandy|ruck: sshnaidm: thoughts on moving reproducer containers off docker.io?16:36
opendevreviewamolkahat proposed openstack/tripleo-ci master: CentOS-9 build containers base job  https://review.opendev.org/c/openstack/tripleo-ci/+/82085216:37
sshnaidmmwhahaha, +1, will do16:37
rlandy|rucksounds reasonable16:38
mwhahahamy laptop is highly unammused 16:38
rlandy|ruckI gather you are not amused either 16:39
mwhahahaprobably16:40
mwhahahasome of teh zuul services puke becuase no zuul.conf16:42
mwhahahajfyi16:42
mwhahahai have a feeling this isn't going to work16:43
mwhahaharlandy|ruck: https://paste.opendev.org/show/811912/ jfiy look slike logs/fingergw/launcher don't work out of the box16:45
* mwhahaha goes digging16:46
sshnaidmmwhahaha, did you run setup playbook like here: https://github.com/rdo-infra/ansible-role-tripleo-ci-reproducer/blob/master/README.md#setup-playbook16:47
mwhahahathat's what i'm running16:47
sshnaidmmwhahaha, seems like files are not generated16:47
mwhahahaIt was stuck ansible-role-tripleo-ci-reproducer : Get link of the tenant]16:48
mwhahahaand i went looking and those three containers weren't ok16:48
mwhahahasincce i've not done this before, it was a fresh install on f35 rootless podman16:49
sshnaidmmwhahaha, I run like this usually: https://paste.opendev.org/show/811913/16:49
sshnaidmmwhahaha, your key on f35 should be the key the Gerrit knows about16:49
sshnaidmsince it's used for connection to gerrit by zuul16:50
sshnaidmcheck if you have it in .ssh/id_rsa(.pub)16:50
sshnaidmmwhahaha, set also setup_nodepool: true16:51
mwhahahayea that's basically what i had16:53
mwhahahai'll try again16:53
mwhahahayea so that's exactly what i had but those three containers came up bad16:56
mwhahahacause setup_nbdepool is true by default16:56
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph spec"  https://review.opendev.org/c/openstack/python-tripleoclient/+/82283516:59
mwhahahasshnaidm: probably selinux'd as httpd.conf access was deined17:00
sshnaidmmwhahaha, better switch it off17:00
mwhahaha:D17:00
sshnaidmshould be the first line in readme..17:01
mwhahahayea setting to permissive seems to at least let those containers start17:11
opendevreviewAlan Bishop proposed openstack/tripleo-heat-templates stable/ussuri: [train,ussuri] Set cinder's __DEFAULT__ vol type description  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/82341117:17
abishoprlandy|ruck: ^^ shouldn't break the periodic upgrade job that my previous backport broke17:17
abishopis there a way to verify this?17:17
* rlandy|ruck looks17:17
rlandy|ruckabishop: we can try testproject that job17:18
* rlandy|ruck tries to remember which job broke17:18
abishoprlandy|ruck: https://bugs.launchpad.net/tripleo/+bug/195443117:19
abishopmentions the jobs17:19
rlandy|ruckabishop: k - setting up testproject for that17:20
abishopgreat, thx!17:20
abishopdon't want to repeat history by breaking cinder again :-/17:20
rlandy|ruckhttps://review.rdoproject.org/r/c/testproject/+/3715017:20
rlandy|ruck^^ testproject marios used to test the revrt17:21
rlandy|ruckrevert17:21
rlandy|ruckso we can switch the depends-on to current patch17:21
mwhahahafatal: [localhost]: FAILED! => {"msg": "An unhandled exception occurred while running the lookup plugin 'pipe'. Error was a <class 'ansible.errors.AnsibleError'>, original message: lookup_plugin.pipe(ssh-keygen -y -f /var/ssh/id_rsa) returned 255"}17:22
mwhahahameh17:22
rlandy|ruckabishop: rekicked with depends-on: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/82341117:23
rlandy|rucklet's see17:23
abishopthx again!17:23
opendevreviewRafael Castillo proposed openstack/tripleo-quickstart-extras master: Add compose-repos to dlrn config  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/82007317:32
*** rlandy|ruck is now known as rlandy|ruck|lunch17:46
*** cloudnull1 is now known as cloudnull17:50
*** rlandy|ruck|lunch is now known as rlandy|ruck18:14
*** jm1 is now known as jm1|afk18:48
*** jm1|afk is now known as jm118:48
*** jm1 is now known as jm1|afk18:51
*** jm1|afk is now known as jm118:51
*** jm1 is now known as jm1|afk18:53
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph spec"  https://review.opendev.org/c/openstack/python-tripleoclient/+/82283518:55
*** jm1 is now known as jm1|afk18:58
*** jm1|afk is now known as jm118:58
*** jm1 is now known as jm1|afk19:00
*** jm1 is now known as jm1|afk19:07
opendevreviewRafael Castillo proposed openstack/tripleo-common stable/wallaby: Add configs for CentOS 9 image  https://review.opendev.org/c/openstack/tripleo-common/+/82336319:45
rlandy|ruckabishop: https://review.rdoproject.org/r/c/testproject/+/37150 - passes ... you can check if the changes look to be included20:12
abishoprlandy|ruck: yes, good to see it passed, and I do see my changes20:18
rlandy|ruckabishop: k - cool20:19
*** dviroel|rover is now known as dviroel|rover|afk20:25
*** tosky_ is now known as tosky20:32
opendevreviewRafael Castillo proposed openstack/tripleo-puppet-elements stable/wallaby: Add config for CentOS 9 in yum_plugin_priorities  https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/82336420:37
mwhahaharlandy|ruck: who can i poke about reproducer questions? It doesn't seem like the ipa job works. I don't see it defined in the zuul instance that gets spun up21:18
rlandy|ruckmwhahaha: sshnaidm probably knows the reproducer best atm21:22
mwhahahak i'll ask him tomorrow then21:22
rlandy|rucktbh - I haven't touched it in a while21:22
mwhahahai got it up at least but the job isn't triggered21:22
rlandy|ruckbut it's a good reminder to get back into that21:22
mwhahahai threw up a review to un-ansible some of the containers which sped it up21:22
mwhahahahttps://review.rdoproject.org/r/c/rdo-infra/ansible-role-tripleo-ci-reproducer/+/3772421:23
rlandy|ruckk - will review - thanks21:24
mwhahahapip installing ansible is expensive but i left it in there for the gerrit config for now21:24
rlandy|rucknot sure at which point your failure happens but once you have a zuul instance and a project - you should be able to define the job manually21:25
rlandy|ruckbut it's been a while admittedly21:25
mwhahahai was trying to use the zuul_yaml bits  per the docs21:25
opendevreviewMerged openstack/tripleo-heat-templates stable/train: GaneshaNetwork - fallback -> external -> ctlplane  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/82329821:25
mwhahahait doesn't seem to twork21:26
rlandy|ruckok - will ping sshnaidm about the review tomorrow - if you have a paste of the error or file output you see, can add that21:31
mwhahahaansible-role-tripleo-ci-reproducer : Fail if job didn't start fails21:31
mwhahahai'll ask him about i tomorrow21:32
mwhahahai tried manually adding the job defs to test1 but it didn't seem to get picked up21:32
mwhahahathis thing has always been way more complicated than the old one :(21:32
*** dviroel|rover|afk is now known as dviroel|rover21:46
*** dviroel|rover is now known as dviroel|out21:47
rlandy|ruckstevebaker[m]: hi - happy new year ... 21:57
rlandy|ruckstevebaker[m]: could you please review patches listed in the Hardware prove/TripleO CI chat>21:58
rlandy|rucklooks like Ian is out for a while still21:58
rlandy|ruckthanks21:58
opendevreviewSteve Baker proposed openstack/python-tripleoclient stable/wallaby: Don't assume json network config in validations  https://review.opendev.org/c/openstack/python-tripleoclient/+/82343321:59
opendevreviewSteve Baker proposed openstack/python-tripleoclient stable/ussuri: Don't assume json network config in validations  https://review.opendev.org/c/openstack/python-tripleoclient/+/82343422:00
opendevreviewSteve Baker proposed openstack/python-tripleoclient stable/victoria: Don't assume json network config in validations  https://review.opendev.org/c/openstack/python-tripleoclient/+/82343522:01
opendevreviewSteve Baker proposed openstack/python-tripleoclient stable/train: Don't assume json network config in validations  https://review.opendev.org/c/openstack/python-tripleoclient/+/82343622:02
rlandy|ruckTengu: hey - left you some LADA notes in pvt chat22:10
rlandy|ruckthanks22:10
opendevreviewMerged openstack/tripleo-common master: Remove /etc/yum.repos.d/ubi.repo  https://review.opendev.org/c/openstack/tripleo-common/+/82337022:36
opendevreviewBrent Eagles proposed openstack/tripleo-heat-templates master: Include subnet in unbound colloc port creation to ensure IP allocation  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/82185323:11
opendevreviewTom Barron proposed openstack/tripleo-ansible stable/train: Use service_net_map for ganesha_vip  https://review.opendev.org/c/openstack/tripleo-ansible/+/82204423:21
opendevreviewTom Barron proposed openstack/tripleo-ansible stable/train: Use service_net_map for ganesha_vip  https://review.opendev.org/c/openstack/tripleo-ansible/+/82204423:22
opendevreviewTom Barron proposed openstack/tripleo-ansible stable/train: Use service_net_map for ganesha_vip  https://review.opendev.org/c/openstack/tripleo-ansible/+/82204423:29

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!