Tuesday, 2022-12-13

*** rlandy is now known as rlandy|out00:28
*** dviroel|rover|out is now known as dviroel|rover01:12
*** dviroel|rover is now known as dviroel|rover|out01:36
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove leftovers of cinder Dell SC backend  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86719002:29
*** blarnath is now known as d34dh0r5303:37
*** blarnath is now known as d34dh0r5303:52
opendevreviewchandan kumar proposed openstack/tripleo-ci master: Revert "Set tripleo-ci-centos-9-undercloud-upgrade to non-voting"  https://review.opendev.org/c/openstack/tripleo-ci/+/86729104:18
opendevreviewMerged openstack/tripleo-ansible master: bnr - Make the galera desync retries and delay configurable  https://review.opendev.org/c/openstack/tripleo-ansible/+/86196604:42
opendevreviewMerged openstack/tripleo-ansible master: Block deletion of existing snapshots executing the creation command  https://review.opendev.org/c/openstack/tripleo-ansible/+/86644904:42
*** yadnesh|away is now known as yadnesh04:47
opendevreviewMerged openstack/python-tripleoclient master: bnr - Deprecation note  https://review.opendev.org/c/openstack/python-tripleoclient/+/86633405:32
opendevreviewKarthik S proposed openstack/os-net-config stable/train: Avoid reconfiguring SR-IOV devices during stack update  https://review.opendev.org/c/openstack/os-net-config/+/86729206:11
opendevreviewKarthik S proposed openstack/os-net-config stable/train: Avoid reconfiguring SR-IOV devices during stack update  https://review.opendev.org/c/openstack/os-net-config/+/86729206:31
opendevreviewKarthik S proposed openstack/os-net-config stable/train: Avoid reconfiguring SR-IOV devices during stack update  https://review.opendev.org/c/openstack/os-net-config/+/86729206:32
opendevreviewKarthik S proposed openstack/os-net-config master: WIP: Add ethtool_opts config for sriov VF  https://review.opendev.org/c/openstack/os-net-config/+/86731906:52
opendevreviewFernando Díaz Bravo proposed openstack/tripleo-ansible stable/zed: Block deletion of existing snapshots executing the creation command  https://review.opendev.org/c/openstack/tripleo-ansible/+/86729307:17
opendevreviewFernando Díaz Bravo proposed openstack/tripleo-ansible stable/zed: bnr - Make the galera desync retries and delay configurable  https://review.opendev.org/c/openstack/tripleo-ansible/+/86729507:18
opendevreviewTakashi Kajinami proposed openstack/puppet-pacemaker stable/1.1.x: Stable-only: Run stable/wallaby job  https://review.opendev.org/c/openstack/puppet-pacemaker/+/86695407:27
opendevreviewMerged openstack/puppet-pacemaker stable/1.5.x: Update .gitreview for stable/1.5.x  https://review.opendev.org/c/openstack/puppet-pacemaker/+/86337507:29
opendevreviewMerged openstack/puppet-pacemaker stable/1.5.x: Update TOX_CONSTRAINTS_FILE for stable/1.5.x  https://review.opendev.org/c/openstack/puppet-pacemaker/+/86337607:29
opendevreviewMerged openstack/puppet-pacemaker stable/1.5.x: Add fence_watchdog (sbd) stonith agent  https://review.opendev.org/c/openstack/puppet-pacemaker/+/86665007:29
lmiccini\o/07:31
lmiccinio/ tkajinam maybe we can unblock https://review.opendev.org/c/openstack/puppet-tripleo/+/866462 ?07:35
tkajinamlmiccini, done07:36
lmiccinithanks!07:36
tkajinamlmiccini, because it uses change id for depends-on, the patch is not merged unless we merge stable/1.1 backport of the puppet-pacemaker patch07:37
lmiccinithat's ok I guess, we need to get that merged anyway07:38
tkajinamyeah07:38
opendevreviewCedric Jeanneret proposed openstack/tripleo-quickstart-extras master: Refactor a bit the script called to deploy external compute  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/86686407:57
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Configure firewall for standalone deploy  https://review.opendev.org/c/openstack/tripleo-ansible/+/86314607:57
*** sandy__ is now known as ysandeep07:57
opendevreviewFernando Díaz Bravo proposed openstack/python-tripleoclient stable/zed: bnr - Deprecation note  https://review.opendev.org/c/openstack/python-tripleoclient/+/86729608:09
*** amoralej|off is now known as amoralej08:11
*** jpena|off is now known as jpena08:27
opendevreviewFernando Díaz Bravo proposed openstack/tripleo-ansible master: Only do snapshot on inactive nodes  https://review.opendev.org/c/openstack/tripleo-ansible/+/86279508:28
opendevreviewFernando Díaz Bravo proposed openstack/tripleo-ansible master: Only do snapshot on inactive nodes  https://review.opendev.org/c/openstack/tripleo-ansible/+/86279508:29
opendevreviewFernando Díaz Bravo proposed openstack/tripleo-ansible master: Only do snapshot on inactive nodes  https://review.opendev.org/c/openstack/tripleo-ansible/+/86279508:33
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Configure firewall for standalone deploy  https://review.opendev.org/c/openstack/tripleo-ansible/+/86314608:36
opendevreviewFernando Díaz Bravo proposed openstack/tripleo-ansible master: Only do snapshot on inactive nodes  https://review.opendev.org/c/openstack/tripleo-ansible/+/86279508:38
opendevreviewBrendan Shephard proposed openstack/tripleo-ansible master: Remove the infra_image from containers.conf  https://review.opendev.org/c/openstack/tripleo-ansible/+/86707408:58
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: Swap puppet for ansible config - iscsid  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86063109:13
opendevreviewDaniel Bengtsson proposed openstack/python-tripleoclient stable/zed: Fix upgrade playbook path.  https://review.opendev.org/c/openstack/python-tripleoclient/+/86732309:56
damanii would like to know if i can have code review https://review.opendev.org/c/openstack/python-tripleoclient/+/867323 10:03
damaniand this one too please https://review.opendev.org/c/openstack/tripleo-ansible/+/865514 ?10:04
opendevreviewamolkahat proposed openstack/tripleo-quickstart master: Removed C7 based job and release files.  https://review.opendev.org/c/openstack/tripleo-quickstart/+/86732510:13
opendevreviewamolkahat proposed openstack/tripleo-quickstart master: Change trunk.registry to quay registry  https://review.opendev.org/c/openstack/tripleo-quickstart/+/86523810:14
Tengudamani: checked.10:15
opendevreviewamolkahat proposed openstack/tripleo-quickstart master: Change trunk.registry to quay registry  https://review.opendev.org/c/openstack/tripleo-quickstart/+/86523810:17
damaniTengu, thanks a lot 10:31
opendevreviewMerged openstack/tripleo-heat-templates master: Get rid of jinja2 templating in conditional statements  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86592910:35
*** ysandeep is now known as ysandeep|brb10:37
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Remove centos8 wallaby jobs from zuul layouts  https://review.opendev.org/c/openstack/tripleo-ci/+/86735110:39
lmiccinitkajinam: still around? shall we push the remaining stuff?10:45
opendevreviewMerged openstack/tripleo-ci master: Remove reference to IBM BM2 provider  https://review.opendev.org/c/openstack/tripleo-ci/+/86727610:47
opendevreviewMarios Andreou proposed openstack/tripleo-common stable/wallaby: Remove centos8 wallaby jobs from zuul layouts  https://review.opendev.org/c/openstack/tripleo-common/+/86735410:57
yadneshrequesting code review for https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86668811:03
opendevreviewMarios Andreou proposed openstack/tripleo-upgrade stable/wallaby: Remove centos8 wallaby jobs from zuul layouts  https://review.opendev.org/c/openstack/tripleo-upgrade/+/86735511:07
*** ysandeep|brb is now known as ysandeep11:09
*** rlandy|out is now known as rlandy11:17
*** dviroel|rover|out is now known as dviroel|rover11:18
*** frenzy_friday is now known as frenzy_friday|doc11:34
opendevreviewMartin Magr proposed openstack/tripleo-ansible stable/wallaby: Add podman_socket role  https://review.opendev.org/c/openstack/tripleo-ansible/+/86404111:56
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Remove centos8 wallaby jobs from zuul layouts  https://review.opendev.org/c/openstack/tripleo-ci/+/86735112:13
opendevreviewFernando Díaz Bravo proposed openstack/python-tripleoclient stable/wallaby: bnr - Deprecation note  https://review.opendev.org/c/openstack/python-tripleoclient/+/86729712:25
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Configure firewall for standalone deploy  https://review.opendev.org/c/openstack/tripleo-ansible/+/86314612:34
Tengupfrrt.12:57
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Configure firewall for standalone deploy  https://review.opendev.org/c/openstack/tripleo-ansible/+/86314612:57
opendevreviewKarthik S proposed openstack/os-net-config master: Add ethtool_opts config for sriov VF  https://review.opendev.org/c/openstack/os-net-config/+/86731912:58
opendevreviewJesse Pretorius proposed openstack/tripleo-heat-templates stable/zed: DNM test bug/1998501  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86702612:58
*** amoralej is now known as amoralej|lunch12:59
opendevreviewJuan Badia Payno proposed openstack/tripleo-upgrade master: WIP undercloud_os_upgrade  https://review.opendev.org/c/openstack/tripleo-upgrade/+/86748613:20
*** yadnesh is now known as yadnesh|away13:34
beaglesif anyone has a sec, this has been sitting around awhile https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85944913:36
Tengubeagles: wow, sorry, I didn't see tkajinam mention on it - I just voted and pushed an answer.13:50
Tengu(about "old rules cleanup")13:50
beaglesTengu, thanks!13:56
Tengubeagles: so with the new nftables thingy, there shouldn't be any dangling rules anymore.13:56
beaglesTengu, that's cool!13:56
Tengui.e. you remove it from t-h-t or env files, it should be removed.13:56
Tenguthat was one if the goal of generating the rules in a serie of files, using dedicated chains.13:57
Tenguthat way: we can empty our custom tripleo_* chains, and be clean.13:57
Tengubeagles: oh, btw, also, there's now a new way to push rules. tripleo_nftables now supports file snippets.13:58
Tengumeaning we can push files in a specific location, and they will be loaded, parsed, and ordered according to the rule_name (as usual) globally, and used to generate the rulesets.13:59
Tengutook some time to get it, and I just stumbled on a stupid issue (that shouldn't happen by default - it crash when we push an empty file X( ). will allow to remove many things from tripleo-heat-templates :)13:59
*** dasm|off is now known as dasm14:00
*** amoralej|lunch is now known as amoralej14:00
opendevreviewManojkatari proposed openstack/tripleo-ansible master: Add configure tasks for iscsid role  https://review.opendev.org/c/openstack/tripleo-ansible/+/86061114:14
lmicciniTengu: bogdando could you guys help with https://review.opendev.org/c/openstack/puppet-pacemaker/+/866717 and https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954 please?14:15
Tengunudged.14:17
lmiccini\o/ thank you!14:18
rlandylmiccini: you all set on the quickstart side?14:26
lmiccinirlandy: hopefully, but now I need to figure out why gate jobs are not triggered (again :( )14:26
rlandybranches?14:26
lmiccinirlandy: I wonder if I shouldn't add the overrides to https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954/7/.zuul.yaml#38 as well14:27
lmicciniwe didn't add that to 1.5 so I am not sure14:30
rlandylmiccini: iirc, templates may not apply w/o merge14:33
rlandywould it be possible to merge template depend-on first14:33
rlandyand then try?14:33
lmiccinirlandy: it is merged14:33
lmiccinihttps://review.opendev.org/c/openstack/tripleo-ci/+/86672214:33
rlandyhmmm - ok forget that then14:34
rlandyyeah -  then branches would my next guess14:35
lmiccinirlandy: you mean something like this https://review.opendev.org/c/openstack/puppet-pacemaker/+/863375/5/.gitreview ?14:38
rlandyum ... not there ... getting14:43
rlandylmiccini: to clarify  ... jobs defined ran in check?14:49
lmiccinirlandy: yes14:49
rlandypuppet-pacemaker-lint-ubuntu-focal14:49
rlandy        - puppet-pacemaker-syntax-6-ubuntu-focal14:49
rlandy        - puppet-pacemaker-unit-6.21-centos-8-stream14:49
rlandy        - puppet-pacemaker-unit-6.21-ubuntu-focal14:49
rlandy        - tripleo-ci-centos-8-scenario004-standalone: &scen4_vars14:49
rlandy            override-checkout: stable/wallaby14:49
rlandyas expected14:49
rlandybut just not in gate?14:49
rlandyand you expect these jobs to run in the tripleo gate?14:50
rlandyproject:14:50
rlandy    queue: tripleo14:50
rlandy    templates:14:50
lmicciniI see they ran for https://review.opendev.org/c/openstack/puppet-pacemaker/+/866717 (check)14:50
lmiccinisorry for https://review.opendev.org/c/openstack/puppet-pacemaker/+/86695414:51
rlandyok - so the jobs kick fine in check - they just don't run in gate14:53
lmicciniyep14:54
*** frenzy_friday|doc is now known as frenzy_friday14:57
rlandyI guess this is a tkajinam question - would we expect puppet jobs in the tripleo gate14:58
rlandyhttps://review.opendev.org/c/openstack/puppet-pacemaker/+/856233 did work though14:58
lmiccinirlandy: yes, seems to be "just" this 1.1.x branch14:59
*** marios is now known as marios|out15:05
opendevreviewMerged openstack/python-tripleoclient stable/zed: bnr - Deprecation note  https://review.opendev.org/c/openstack/python-tripleoclient/+/86729615:14
*** pojadhav is now known as pojadhav|dinner15:23
csibbittchkumar|ruck, Hiya. Can I bother you for another set of eyes on this bugfix? It's been passing for awhile and I've addressed what Takashi was asking for. Should just need a WF flag I think https://review.opendev.org/c/openstack/tripleo-ansible/+/86560115:26
chkumar|ruckcsibbitt: thank you for fixing it. :-)15:34
chkumar|ruckcc tkajinam workflowed this review https://review.opendev.org/c/openstack/tripleo-ansible/+/865601 as comments got addressed.15:34
lmiccinirlandy: any idea where to look next? 15:41
csibbittchkumar|ruck, No problem, thanks for the WF! I still have some more open patches for these molecule tests on zed and wallaby. 15:42
csibbittIt looks like the zed one is ready to go, but I'll need to take another look at wallaby now that the bugfix has merged. I think it will be okay too though. Zed: https://review.opendev.org/c/openstack/tripleo-ansible/+/86588815:43
*** eliadcohen_ is now known as eliadcohen15:43
csibbittActually I'm not 100% sure it's even needed on Wallaby at this point. I'll take a closer look15:44
*** dviroel|rover is now known as dviroel|rover|lunch15:57
opendevreviewMerged openstack/python-tripleoclient stable/zed: Fix upgrade playbook path.  https://review.opendev.org/c/openstack/python-tripleoclient/+/86732316:09
beaglesTengu, is there a flag that causes container-puppet-<insert service name> here containers to be removed. I vaguely recall re-running these manually but they seem to have been removed in the deployment I'm looking at16:11
opendevreviewLuca Miccini proposed openstack/puppet-pacemaker stable/1.1.x: Add fence_watchdog (sbd) stonith agent  https://review.opendev.org/c/openstack/puppet-pacemaker/+/86671716:12
paramiteGreetings guys, can somebody please help me to move on with my backports before everyone flees for Christmas? > https://review.opendev.org/c/openstack/tripleo-ansible/+/864041 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865156 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865155 https://review.opendev.org/c/openstack/puppet-tripleo/+/864174 https://review.opendev.org/c/openstack/puppet-tripleo/+/85889716:13
paramite https://review.opendev.org/c/openstack/puppet-tripleo/+/86516816:13
Tengubeagles: hmmm.... I don't know.16:13
beaglesTengu, cool.. I'll keep looking16:14
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: mock patch to test ci  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86733216:21
opendevreviewCedric Jeanneret proposed openstack/tripleo-quickstart-extras master: Refactor a bit the script called to deploy external compute  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/86686416:23
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Configure firewall for standalone deploy  https://review.opendev.org/c/openstack/tripleo-ansible/+/86314616:23
opendevreviewOwen McGonagle proposed openstack/tripleo-upgrade master: [WIP] osp 17.x change overcloud ssh user from heat-admin to tripleo-admin  https://review.opendev.org/c/openstack/tripleo-upgrade/+/86753816:32
rlandylmiccini; sorry - meetings ... I'm looking at the template where the job did run in gate16:33
lmiccinirlandy: no worries. I am trying to workaround forcing the gates to be triggered by https://review.opendev.org/c/openstack/puppet-pacemaker/+/86671716:33
lmicciniI have added a depends-on and workflow'd that16:33
rlandylmiccini: the diff I see is https://opendev.org/openstack/puppet-openstack-integration/src/branch/master/zuul.d/layout.yaml#L716:34
rlandyno project defined16:34
rlandyhttps://github.com/openstack/puppet-pacemaker/blob/master/.zuul.yaml - working one16:36
rlandywhere the project is there16:36
rlandystill looking16:36
rlandyif it were a branch thing, it would not run on check16:37
rlandybut you have that working16:37
lmicciniI think we define the project via https://review.opendev.org/c/openstack/puppet-pacemaker/+/866954/7/.zuul.yaml ?16:37
lmicciniI have added this as depends-on, let's see if this is enough16:38
rlandyyep - it's fine16:40
rlandymay I recheck this?16:40
lmiccinirlandy: https://zuul.opendev.org/t/openstack/status#866717 it is already running afaics16:40
rlandyok - watching16:40
rlandyso ... looking at this chain of patches ... https://review.opendev.org/c/openstack/puppet-pacemaker/+/866717/ has a depends-on https://review.opendev.org/c/openstack/puppet-pacemaker/+/86695416:46
rlandylmiccini: ^^ can you help me understand this series of patches16:46
opendevreviewMerged openstack/tripleo-validations stable/zed: Adjusting node-disks validation for new authentication procedures  https://review.opendev.org/c/openstack/tripleo-validations/+/86687016:47
opendevreviewMerged openstack/tripleo-validations stable/wallaby: Adjusting node-disks validation for new authentication procedures  https://review.opendev.org/c/openstack/tripleo-validations/+/86686916:47
opendevreviewMerged openstack/validations-libs stable/1.6: Tox docs job moved to ubuntu-focal nodeset  https://review.opendev.org/c/openstack/validations-libs/+/86698716:47
opendevreviewMerged openstack/validations-common stable/1.6: Tox docs job moved to ubuntu-focal nodeset  https://review.opendev.org/c/openstack/validations-common/+/86670016:47
lmiccinirlandy: I was under the impression that we needed 866954 to have check and gate jobs working for 1.1.x ?16:47
rlandyhttps://review.opendev.org/c/openstack/puppet-pacemaker/+/866954 is rebased on https://review.opendev.org/c/openstack/puppet-pacemaker/+/866717/16:47
lmicciniuhm16:48
opendevreviewOwen McGonagle proposed openstack/tripleo-upgrade stable/zed: [WIP] osp 17.x change overcloud ssh user from heat-admin to tripleo-admin  https://review.opendev.org/c/openstack/tripleo-upgrade/+/86730116:48
rlandyit depends on a patch that it is in a rebase order with?16:48
rlandyam I reading that wrong?16:48
lmiccinirlandy: I haven't noticed that rebase tbh16:49
opendevreviewOwen McGonagle proposed openstack/tripleo-upgrade stable/wallaby: [WIP] osp 17.x change overcloud ssh user from heat-admin to tripleo-admin  https://review.opendev.org/c/openstack/tripleo-upgrade/+/86730216:49
rlandya patch won't hit gate until the depends on does16:49
rlandyand it can't hit gate until the rebase does16:49
rlandyso none hit gate?16:50
* rlandy confused16:50
*** dviroel|rover|lunch is now known as dviroel|rover16:50
lmiccinirlandy: I added the depends-on a few minutes ago and despite both patches being +w'd none of them hit the gate so I am not sure I made it worse :D16:50
rlandylmiccini: maybe we can chat for 5 and sort out the right order here16:51
lmiccinirlandy: sure16:51
rlandygchat?16:51
lmiccinirlandy: yup16:51
rlandyhttps://meet.google.com/mon-qrwr-ogo?pli=1&authuser=016:51
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: Swap puppet for ansible config - iscsid  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86063116:52
opendevreviewLuca Miccini proposed openstack/puppet-pacemaker stable/1.1.x: Stable-only: Run stable/wallaby job  https://review.opendev.org/c/openstack/puppet-pacemaker/+/86695417:00
opendevreviewRoberto Alfieri proposed openstack/tripleo-ansible master: Changed the way how we gather interface and mtu info and fixed indentation inside block  https://review.opendev.org/c/openstack/tripleo-ansible/+/86720117:00
opendevreviewJuan Badia Payno proposed openstack/tripleo-heat-templates master: Bug#1998501 fixed ansible conditional evaluation  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86638117:23
*** jpena is now known as jpena|off17:36
bogdandobeagles: I can also recall such a flag, but couldn't locate any traces of it :)17:41
bogdandotried all possible regexes :D17:41
bogdandoideally, need to do the same but in git log -S now :/17:41
bogdandoin tht and tripleo-ansible repos17:41
bogdandoslagle: perchance17:42
opendevreviewJuan Badia Payno proposed openstack/tripleo-heat-templates stable/zed: Bug#1998501 fixed ansible conditional evaluation  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86702617:42
bogdandocan you recall where is that flag now? which cleans up or not puppet config containers after completion17:42
*** amoralej is now known as amoralej|off17:44
*** ysandeep is now known as ysandeep|out17:50
*** pojadhav|dinner is now known as pojadhav18:10
*** dasm is now known as dasm|off19:26
rlandywoohoo  lmiccini job hit the gate19:51
opendevreviewAlan Bishop proposed openstack/tripleo-heat-templates master: Ensure glance-api services are deployed correctly  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86607519:58
opendevreviewMerged openstack/tripleo-common master: New constant for THT system path and THT roles directory  https://review.opendev.org/c/openstack/tripleo-common/+/86632420:28
opendevreviewMerged openstack/tripleo-common master: Preserve environment variables with buildah  https://review.opendev.org/c/openstack/tripleo-common/+/86708021:09
*** dviroel|rover is now known as dviroel|out21:58
*** rlandy is now known as rlandy|out22:43
opendevreviewIan Wienand proposed openstack/diskimage-builder master: tox jobs: pin to correct nodesets  https://review.opendev.org/c/openstack/diskimage-builder/+/86757923:58
opendevreviewClark Boylan proposed openstack/diskimage-builder master: tox jobs: pin to correct nodesets  https://review.opendev.org/c/openstack/diskimage-builder/+/86757923:59

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