Monday, 2022-11-14

opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Support for Cinder IBM Svf driver  https://review.opendev.org/c/openstack/puppet-tripleo/+/86235902:47
opendevreviewMerged openstack/tripleo-docs master: Fix typos in doc pages  https://review.opendev.org/c/openstack/tripleo-docs/+/85851603:43
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Use ansible variable to detect bootstrap node  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85771404:03
akahat|rucko/04:31
akahat|ruckHello, we are seeing lot's of tripleo jobs are failing on lp: https://bugs.launchpad.net/tripleo/+bug/199648204:31
akahat|ruckIt is currently hitting master and wallaby.04:32
tkajinamakahat|ruck, did we bump ansible recently ?04:36
tkajinamhttps://github.com/ansible/ansible/pull/73908/files04:37
akahat|rucktkajinam, may be this cryptography which is moved to ansible-requirements: https://github.com/openstack/tripleo-ansible/commit/c0bc395ab494ee18e77fba608cdd40762c7b8c5104:39
bshephartkajinam: Maybe this one? 04:41
bshephartripleo_ansible/ansible_plugins/strategy/tripleo_base.py04:41
bshephar136:                                        task._valid_attrs['any_errors_fatal'],04:41
bshepharOh, you already found it04:41
* tkajinam is writing a potential fix04:41
tkajinambshephar, yeah04:41
bshephartkajinam++ Too fast04:41
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible  https://review.opendev.org/c/openstack/tripleo-ansible/+/86438304:43
tkajinamlet's see how this works04:43
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible  https://review.opendev.org/c/openstack/tripleo-ansible/+/86438304:47
*** soniya29 is now known as soniya29|rover04:49
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible  https://review.opendev.org/c/openstack/tripleo-ansible/+/86438305:05
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible  https://review.opendev.org/c/openstack/tripleo-ansible/+/86438305:05
*** soniya29|rover is now known as soniya29|rover|afk05:34
opendevreviewKeigo Noha proposed openstack/tripleo-heat-templates master: Fix tripleo_delegate_to parameter in heat-engine  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86438706:03
akahat|rucktkajinam, hey.. i tried depends-on on your patch .. it working. https://review.rdoproject.org/r/c/testproject/+/4146506:41
*** ysandeep|out is now known as ysandeep06:44
tkajinamlooks like we need https://review.opendev.org/c/openstack/tripleo-ansible/+/860516 ?06:45
tkajinamhmm. ok so ansible 2.14 killed a lot of things06:46
*** soniya29|rover|afk is now known as soniya29|rover06:49
akahat|rucktkajinam, yeah.. we need that also.. ansible-2.14 changed lots of things.07:01
akahat|ruck testing with 860516 here: https://review.rdoproject.org/r/c/testproject/+/4146507:01
mariostkajinam: do you want to squash them into one or rebase? 07:02
tkajinamakahat|ruck, marios if we find out these two are enough then I can squash these two07:03
mariostkajinam: k ... akahat|ruck can you try with the other one too in depends-on? 07:03
tkajinammarios, I think he already did that07:03
tkajinamhttps://review.rdoproject.org/r/c/testproject/+/4146507:03
tkajinamhas two depends-on07:04
akahat|rucktkajinam, ack.. 07:04
mariosah k thanks akahat|ruck 07:04
mariosk cool different file so build-test-packages should not have any merge/rebase issue07:04
tkajinamif we find out more problems then we probably want to pin ansigle to 2.13 now07:09
tkajinamand test 2.14 in test project before removing the pin07:09
akahat|rucktkajinam, we need to change all the attributes which are referring to PlayIterator.07:29
akahat|rucktkajinam, https://logserver.rdoproject.org/65/41465/25/check/periodic-tripleo-ci-centos-9-standalone-master/9a72730/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz07:29
akahat|ruckhttps://codesearch.opendev.org/?q=PlayIterator&i=nope&literal=nope&files=&excludeFiles=&repos=07:30
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Replace deprecated PlayIterator attributes  https://review.opendev.org/c/openstack/tripleo-ansible/+/86051607:36
ramishratkajinam. akahat|ruck: Hmm.. wallaby should not be using ansible-core 2.14, looks like it's coming from quickstart-centos-appstreams, we can change the code to adapt in master, but have to pin in older branches In think07:37
ramishrahttps://logserver.rdoproject.org/openstack-component-clients/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-scenario000-multinode-oooq-container-updates-clients-wallaby/02f79b1/logs/undercloud/var/log/extra/package-list-installed.txt.gz07:37
tkajinamramishra, yeah07:38
tkajinamalso we have to know which ansible version we want in zed07:38
opendevreviewKeigo Noha proposed openstack/tripleo-heat-templates master: Fix tripleo_delegate_to parameter in heat-engine  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86438707:53
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Modify how tripleo_nftables gets its configurations  https://review.opendev.org/c/openstack/tripleo-ansible/+/86387207:56
*** soniya29 is now known as soniya29|rover08:09
xekTengu: o/08:11
xekTengu: please take a look at Marios comments on https://review.opendev.org/c/openstack/tripleo-ansible/+/86416508:12
Tenguhey xek 08:12
xekTengu: should I propose a patch to remove cryptography from molecule-requirements.txt?08:12
Tengumarios: xek: patch -^^  is a squash of 2 patches08:13
xekTengu: it's not a squash08:13
Tenguor maybe you're just missing a backport, actually.08:14
mariosakahat|ruck: tkajinam: ugh ...  looks like another one https://logserver.rdoproject.org/65/41465/25/check/periodic-tripleo-ci-centos-9-scenario000-multinode-oooq-container-updates-master/2b1834d/logs/undercloud/home/zuul/undercloud_install.log.txt.gz "The conditional check 'rpm_query_result.rc > 0' failed. The error was: error while evaluating conditional (rpm_query_result.rc > 0): 'dict object' has no 08:14
mariosattribute 'rc'.08:14
mariosTengu: xek: o/08:15
Tengumarios: heya :)08:16
xekTengu: the way I understand it, adding cryptography with the version limit in ansible-requirements.txt is what fixes the issue08:16
xekmarios: o/08:16
mariosTengu: xek: i can revisit the patch but as it stands didn't make sense to me as a 'backport' since there is no cryptography in requirements to pin... we are adding it. i saw comments about other patches missing so do you want to add those first?08:17
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: tripleo_boostrap: Fix undefined .rc attribute  https://review.opendev.org/c/openstack/tripleo-ansible/+/86439008:17
xekmarios: I think previously it wasn't a direct dependency, so that's why it wasn't there08:18
tkajinamnot sure why we are facing that issue now. If that is caused by any change in ansible 2.14 then it's likely to be giant killer08:18
mariostkajinam: akahat|ruck: updated ^^^ https://review.rdoproject.org/r/c/testproject/+/41465/26#message-096a90e3ca197222c880220d53b52b06087a12da lets see08:18
*** soniya29|rover is now known as soniya29|rover|lunch08:19
mariostkajinam: akahat|ruck: hold on... we might want to squash those changes togther08:20
mariostkajinam: akahat|ruck: checking again sec08:21
mariostkajinam: akahat|ruck: yeah see https://logserver.rdoproject.org/65/41465/25/check/periodic-tripleo-ci-centos-9-scenario000-multinode-oooq-container-updates-master/2b1834d/job-output.txt ... looks like only one of those depends-on was included 08:22
mariosakahat|ruck: tkajinam: grep for artg_change_list in ^^^ 08:22
mariosakahat|ruck: tkajinam: only 2022-11-14 02:50:32.024762 | primary |             "refspec": "refs/changes/83/864383/4" included there i think08:23
* marios re-coffee brb08:23
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14  https://review.opendev.org/c/openstack/tripleo-ansible/+/86439208:25
*** amoralej|off is now known as amoralej08:26
xekTengu: marios: if I do a squash of https://review.opendev.org/c/openstack/tripleo-ansible/+/864044 and https://review.opendev.org/c/openstack/tripleo-ansible/+/864010 I only get an issue with 'deleted by us:   zuul.d/playbooks/run-role-addition.yml'08:29
xekTengu: the changes there look unrelated08:29
*** jpena|off is now known as jpena08:38
opendevreviewDaniel Bengtsson proposed openstack/tripleo-heat-templates master: Use python -m instead pip directly.  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85565108:44
opendevreviewJiri Podivin proposed openstack/tripleo-quickstart master: Removing workaround providing no longer necessary 'plan-environment.yaml'  https://review.opendev.org/c/openstack/tripleo-quickstart/+/86416708:51
opendevreviewJiri Podivin proposed openstack/tripleo-quickstart master: Removing workaround providing no longer necessary 'plan-environment.yaml'  https://review.opendev.org/c/openstack/tripleo-quickstart/+/86416708:56
*** soniya29|rover|lunch is now known as soniya29|rover08:57
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14  https://review.opendev.org/c/openstack/tripleo-ansible/+/86439209:17
*** ysandeep is now known as ysandeep|lunch09:35
akahat|rucktkajinam, This again failed: https://logserver.rdoproject.org/65/41465/27/check/periodic-tripleo-ci-centos-9-standalone-master/300ca8b/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz09:58
akahat|rucktkajinam, i think we need to go with ansible version pin same thing suggested by ramishra 09:59
*** jpodivin_ is now known as jpodivin10:01
akahat|rucktkajinam, i mean it is failing on the other tasks; https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_bootstrap/tasks/bootstrap.yml#L8710:01
akahat|rucksorry.. didn't noticed it at first.10:01
akahat|ruckhttps://codesearch.opendev.org/?q=rpm_query_result&i=nope&literal=nope&files=&excludeFiles=&repos=10:02
opendevreviewGiulio Fidente proposed openstack/tripleo-ansible stable/zed: Support setting ovn-ofctrl-wait-before-clear  https://review.opendev.org/c/openstack/tripleo-ansible/+/86439610:17
akahat|rucktkajinam, updating your patch with the fix.10:19
xekramishra: hey, can you help us sort out what's needed for https://review.opendev.org/c/openstack/tripleo-ansible/+/864165 ?10:25
opendevreviewamolkahat proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14  https://review.opendev.org/c/openstack/tripleo-ansible/+/86439210:25
xekramishra: the patch itself works, because the gate failed initially and got fixed when I rebased the subsequent patch on top10:25
Tengumaybe we need a direct backport from master->wallaby of https://review.opendev.org/c/openstack/tripleo-ansible/+/863508 - for Zed, it's squashed in another, non-backportable patch (https://review.opendev.org/c/openstack/tripleo-ansible/+/863463)10:28
opendevreviewGrzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Fix molecule jobs broken with latest cryptography  https://review.opendev.org/c/openstack/tripleo-ansible/+/86418310:29
xekTengu: ^ I cherry-picked it (without conflict), let's see if it passes CI (it failed on train)10:32
Tenguxek: you got my +2 on the "move cryptography in".10:34
Tengu*pin10:35
opendevreviewArx Cruz proposed openstack/ansible-role-collect-logs master: Update ansible core version  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/86439910:48
*** ysandeep|lunch is now known as ysandeep10:52
opendevreviewArx Cruz proposed openstack/ansible-role-collect-logs master: Update ansible core version  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/86439911:03
*** soniya29 is now known as soniya29|rover11:04
*** frenzy_friday is now known as frenzy_friday|doc_appt11:05
xekTengu: cool, thanks for looking into it!11:12
opendevreviewamolkahat proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14  https://review.opendev.org/c/openstack/tripleo-ansible/+/86439211:13
*** dviroel_ is now known as dviroel11:15
*** mciecierski_ is now known as mciecierski11:32
opendevreviewLukas Bezdicka proposed openstack/tripleo-upgrade master: FFWD3 Refactor ffwd upgrade for train to wallaby  https://review.opendev.org/c/openstack/tripleo-upgrade/+/84968211:59
Tenguxek: on you to chase down others, especially to get the -1 sorted.12:11
Tengumarios, ramishra soooo is this the kind of issue we're seeing with newer ansible 2.14 or something like that? https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5d5/863872/10/check/tripleo-ci-centos-9-containers-multinode/5d5d523/logs/undercloud/home/zuul/undercloud_install.log12:13
Tenguah, seeing amolkahat's patch, it is.12:14
Tenguguess we'll need to do some changes in the tripleo-operator-ansible part.12:27
Tengulet's play.12:27
akahat|ruckmarios, soniya29|rover tkajinam Tengu ramishra please take a look when you are free: testproject: https://review.rdoproject.org/r/c/testproject/+/41465, patch: https://review.opendev.org/c/openstack/tripleo-ansible/+/86439212:31
Tenguakahat|ruck: checked your proposal for the fix - ramishra added some comment, would be good to address them imho12:32
akahat|ruckTengu, yeah.. just checked. working on it. Thanks ramishra!12:33
Tenguakahat|ruck: ++ thanks for working that fast on those issues.12:33
Tengubrb, need to make some tea.12:34
opendevreviewamolkahat proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14  https://review.opendev.org/c/openstack/tripleo-ansible/+/86439212:36
*** pojadhav- is now known as pojadhav12:38
opendevreviewArx Cruz proposed openstack/ansible-role-collect-logs master: Update ansible core version  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/86439912:38
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Modify how tripleo_nftables gets its configurations  https://review.opendev.org/c/openstack/tripleo-ansible/+/86387212:40
Tenguakahat|ruck: -^^  will allow to test "in real life".12:40
Tenguit died today with the new ansible version.12:41
opendevreviewArx Cruz proposed openstack/ansible-role-collect-logs master: Fix ansible sanity job  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/86439912:46
*** soniya is now known as soniya|rover12:46
opendevreviewRajesh Tailor proposed openstack/tripleo-heat-templates master: Allows configuring unified quotas in nova  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/86419713:02
mariosakahat|ruck: tkajinam: and now hitting the gate https://61ca0e1c634914faa769-63d8266434a9921ead6c82cc1d17cb33.ssl.cf2.rackcdn.com/829711/9/gate/tripleo-ci-centos-9-undercloud-containers/dfccb12/logs/undercloud/home/zuul/undercloud_install.log 13:04
mariosERROR! Unexpected Exception, this is probably a bug: 'Task' object has no attribute '_valid_attrs'13:04
mariosgood job you were working on the fix already tkajinam 13:05
akahat|ruckmarios, waiting for +1 from zuul.13:06
lmiccinihey folks can you give https://review.opendev.org/c/openstack/tripleo-ansible/+/863625 a push please? the other cherry pick merged, only missing wallaby13:10
opendevreviewCedric Jeanneret proposed openstack/tripleo-operator-ansible master: Ansible-2.14 compatibility  https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/86440913:15
opendevreviewArx Cruz proposed openstack/ansible-role-collect-logs master: Fix ansible sanity job  https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/86439913:29
opendevreviewGrzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Move cryprography pin  https://review.opendev.org/c/openstack/tripleo-ansible/+/86416513:46
opendevreviewGrzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Make creating/updating users idempotent  https://review.opendev.org/c/openstack/tripleo-ansible/+/86403413:47
opendevreviewMerged openstack/tripleo-docs master: Removing irrelavant references to 'plan-environment.yaml'  https://review.opendev.org/c/openstack/tripleo-docs/+/86364213:47
xekTengu: ^13:49
Tenguxek: voted - now that we have the complete chain it should be all good for marios :)13:50
opendevreviewGrzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Make creating/updating users idempotent  https://review.opendev.org/c/openstack/tripleo-ansible/+/86403413:51
opendevreviewMerged openstack/tripleo-docs master: Node scaling now covered by baremetal_provision  https://review.opendev.org/c/openstack/tripleo-docs/+/86192713:51
*** rcastillo|rover is now known as rcastillo13:57
opendevreviewMerged openstack/tripleo-docs master: Update documentation about deployment status  https://review.opendev.org/c/openstack/tripleo-docs/+/86319514:00
*** amoralej is now known as amoralej|lunch14:07
akahat|ruckmarios_, Tengu ramishra soniya|rover tkajinam please take a look when you are free, thanks! https://review.opendev.org/c/openstack/tripleo-ansible/+/864392, https://review.rdoproject.org/r/c/testproject/+/4146514:13
*** marios_ is now known as marios14:23
opendevreviewGrzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Make creating/updating users idempotent  https://review.opendev.org/c/openstack/tripleo-ansible/+/86403414:24
*** frenzy_friday|doc_appt is now known as frenzy_friday14:33
*** dasm|off is now known as dasm14:46
opendevreviewDavid Sedgmen proposed openstack/puppet-tripleo stable/train: Current change will not work reliably.  https://review.opendev.org/c/openstack/puppet-tripleo/+/86418614:47
opendevreviewDavid Sedgmen proposed openstack/puppet-tripleo stable/train: [Train ONLY] remove if statement from local ca check Current change will not work reliably. Removing this if statement will just cause the script to extract and update the CA trust on each renewal  https://review.opendev.org/c/openstack/puppet-tripleo/+/86418614:50
opendevreviewDavid Sedgmen proposed openstack/puppet-tripleo stable/train: [Train ONLY] remove if statement from local ca check Current change will not work reliably. Removing this if statement will just cause the script to extract and update the CA trust on each renewal  https://review.opendev.org/c/openstack/puppet-tripleo/+/86418614:52
*** amoralej|lunch is now known as amoralej15:06
opendevreviewTakashi Kajinami proposed openstack/tripleo-ansible master: Remove redundant check of result.rc  https://review.opendev.org/c/openstack/tripleo-ansible/+/86441715:07
*** ysandeep is now known as ysandeep|out15:10
*** soniya|rover is now known as soniya|out15:16
*** dviroel is now known as dviroel|lunch15:28
opendevreviewCedric Jeanneret proposed openstack/tripleo-operator-ansible master: Ansible-2.14 compatibility  https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/86440915:33
mariosdasm: o/ can you please help us get that merged in your day https://review.opendev.org/c/openstack/tripleo-ansible/+/864392/ (should go to the gate soon) - this is a gate blocker cc tkajinam ramishra akahat|ruck 15:46
mariosdasm: like if it needs a recheck 15:46
marioskind of thing15:46
dasmmarios: i added myself to cc. i'll keep an eye on that15:47
mariosthank you dasm appreciate15:47
dasmno worries15:47
marioscontent providers timed out there checking15:49
opendevreviewJiri Podivin proposed openstack/tripleo-validations master: Refactoring default-node-count validation to use openstack-collection modules  https://review.opendev.org/c/openstack/tripleo-validations/+/86442115:53
marioshttps://lists.openstack.org/pipermail/openstack-discuss/2022-November/031186.html [tripleo] gate blocker /tripleo/+bug/1996482 - please hold rechecks 16:09
mariostkajinam: ^ just fyi 16:09
*** dviroel|lunch is now known as dviroel16:26
*** marios is now known as marios|out16:31
opendevreviewDouglas Viroel proposed openstack/tripleo-ci master: Add new nodepool_provider for ibm cloud  https://review.opendev.org/c/openstack/tripleo-ci/+/86413417:01
*** amoralej is now known as amoralej|off17:37
*** jpena is now known as jpena|off17:39
*** slagle-out is now known as slagle18:54
*** dviroel is now known as dviroel|afk20:01
opendevreviewMerged openstack/os-net-config stable/train: Updating Zuul CI jobs for os-net-config repo  https://review.opendev.org/c/openstack/os-net-config/+/86359220:29
opendevreviewRoberto Alfieri proposed openstack/tripleo-docs master: Switch from Centos Stream 8 to Centos Stream 9  https://review.opendev.org/c/openstack/tripleo-docs/+/86444121:16
opendevreviewSteve Baker proposed openstack/tripleo-ansible stable/wallaby: Don't use meta:end_play after growvols check  https://review.opendev.org/c/openstack/tripleo-ansible/+/86444521:36
*** dasm is now known as dasm|off22:44

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