opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Support for Cinder IBM Svf driver https://review.opendev.org/c/openstack/puppet-tripleo/+/862359 | 02:47 |
---|---|---|
opendevreview | Merged openstack/tripleo-docs master: Fix typos in doc pages https://review.opendev.org/c/openstack/tripleo-docs/+/858516 | 03:43 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Use ansible variable to detect bootstrap node https://review.opendev.org/c/openstack/tripleo-heat-templates/+/857714 | 04:03 |
akahat|ruck | o/ | 04:31 |
akahat|ruck | Hello, we are seeing lot's of tripleo jobs are failing on lp: https://bugs.launchpad.net/tripleo/+bug/1996482 | 04:31 |
akahat|ruck | It is currently hitting master and wallaby. | 04:32 |
tkajinam | akahat|ruck, did we bump ansible recently ? | 04:36 |
tkajinam | https://github.com/ansible/ansible/pull/73908/files | 04:37 |
akahat|ruck | tkajinam, may be this cryptography which is moved to ansible-requirements: https://github.com/openstack/tripleo-ansible/commit/c0bc395ab494ee18e77fba608cdd40762c7b8c51 | 04:39 |
bshephar | tkajinam: Maybe this one? | 04:41 |
bshephar | tripleo_ansible/ansible_plugins/strategy/tripleo_base.py | 04:41 |
bshephar | 136: task._valid_attrs['any_errors_fatal'], | 04:41 |
bshephar | Oh, you already found it | 04:41 |
* tkajinam is writing a potential fix | 04:41 | |
tkajinam | bshephar, yeah | 04:41 |
bshephar | tkajinam++ Too fast | 04:41 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/864383 | 04:43 |
tkajinam | let's see how this works | 04:43 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/864383 | 04:47 |
*** soniya29 is now known as soniya29|rover | 04:49 | |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/864383 | 05:05 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Handle new FieldAttributes in core Ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/864383 | 05:05 |
*** soniya29|rover is now known as soniya29|rover|afk | 05:34 | |
opendevreview | Keigo Noha proposed openstack/tripleo-heat-templates master: Fix tripleo_delegate_to parameter in heat-engine https://review.opendev.org/c/openstack/tripleo-heat-templates/+/864387 | 06:03 |
akahat|ruck | tkajinam, hey.. i tried depends-on on your patch .. it working. https://review.rdoproject.org/r/c/testproject/+/41465 | 06:41 |
*** ysandeep|out is now known as ysandeep | 06:44 | |
tkajinam | looks like we need https://review.opendev.org/c/openstack/tripleo-ansible/+/860516 ? | 06:45 |
tkajinam | hmm. ok so ansible 2.14 killed a lot of things | 06:46 |
*** soniya29|rover|afk is now known as soniya29|rover | 06:49 | |
akahat|ruck | tkajinam, 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/+/41465 | 07:01 |
marios | tkajinam: do you want to squash them into one or rebase? | 07:02 |
tkajinam | akahat|ruck, marios if we find out these two are enough then I can squash these two | 07:03 |
marios | tkajinam: k ... akahat|ruck can you try with the other one too in depends-on? | 07:03 |
tkajinam | marios, I think he already did that | 07:03 |
tkajinam | https://review.rdoproject.org/r/c/testproject/+/41465 | 07:03 |
tkajinam | has two depends-on | 07:04 |
akahat|ruck | tkajinam, ack.. | 07:04 |
marios | ah k thanks akahat|ruck | 07:04 |
marios | k cool different file so build-test-packages should not have any merge/rebase issue | 07:04 |
tkajinam | if we find out more problems then we probably want to pin ansigle to 2.13 now | 07:09 |
tkajinam | and test 2.14 in test project before removing the pin | 07:09 |
akahat|ruck | tkajinam, we need to change all the attributes which are referring to PlayIterator. | 07:29 |
akahat|ruck | tkajinam, https://logserver.rdoproject.org/65/41465/25/check/periodic-tripleo-ci-centos-9-standalone-master/9a72730/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz | 07:29 |
akahat|ruck | https://codesearch.opendev.org/?q=PlayIterator&i=nope&literal=nope&files=&excludeFiles=&repos= | 07:30 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Replace deprecated PlayIterator attributes https://review.opendev.org/c/openstack/tripleo-ansible/+/860516 | 07:36 |
ramishra | tkajinam. 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 think | 07:37 |
ramishra | https://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.gz | 07:37 |
tkajinam | ramishra, yeah | 07:38 |
tkajinam | also we have to know which ansible version we want in zed | 07:38 |
opendevreview | Keigo Noha proposed openstack/tripleo-heat-templates master: Fix tripleo_delegate_to parameter in heat-engine https://review.opendev.org/c/openstack/tripleo-heat-templates/+/864387 | 07:53 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Modify how tripleo_nftables gets its configurations https://review.opendev.org/c/openstack/tripleo-ansible/+/863872 | 07:56 |
*** soniya29 is now known as soniya29|rover | 08:09 | |
xek | Tengu: o/ | 08:11 |
xek | Tengu: please take a look at Marios comments on https://review.opendev.org/c/openstack/tripleo-ansible/+/864165 | 08:12 |
Tengu | hey xek | 08:12 |
xek | Tengu: should I propose a patch to remove cryptography from molecule-requirements.txt? | 08:12 |
Tengu | marios: xek: patch -^^ is a squash of 2 patches | 08:13 |
xek | Tengu: it's not a squash | 08:13 |
Tengu | or maybe you're just missing a backport, actually. | 08:14 |
marios | akahat|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 |
marios | attribute 'rc'. | 08:14 |
marios | Tengu: xek: o/ | 08:15 |
Tengu | marios: heya :) | 08:16 |
xek | Tengu: the way I understand it, adding cryptography with the version limit in ansible-requirements.txt is what fixes the issue | 08:16 |
xek | marios: o/ | 08:16 |
marios | Tengu: 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 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_boostrap: Fix undefined .rc attribute https://review.opendev.org/c/openstack/tripleo-ansible/+/864390 | 08:17 |
xek | marios: I think previously it wasn't a direct dependency, so that's why it wasn't there | 08:18 |
tkajinam | not 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 killer | 08:18 |
marios | tkajinam: akahat|ruck: updated ^^^ https://review.rdoproject.org/r/c/testproject/+/41465/26#message-096a90e3ca197222c880220d53b52b06087a12da lets see | 08:18 |
*** soniya29|rover is now known as soniya29|rover|lunch | 08:19 | |
marios | tkajinam: akahat|ruck: hold on... we might want to squash those changes togther | 08:20 |
marios | tkajinam: akahat|ruck: checking again sec | 08:21 |
marios | tkajinam: 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 |
marios | akahat|ruck: tkajinam: grep for artg_change_list in ^^^ | 08:22 |
marios | akahat|ruck: tkajinam: only 2022-11-14 02:50:32.024762 | primary | "refspec": "refs/changes/83/864383/4" included there i think | 08:23 |
* marios re-coffee brb | 08:23 | |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14 https://review.opendev.org/c/openstack/tripleo-ansible/+/864392 | 08:25 |
*** amoralej|off is now known as amoralej | 08:26 | |
xek | Tengu: 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 |
xek | Tengu: the changes there look unrelated | 08:29 |
*** jpena|off is now known as jpena | 08:38 | |
opendevreview | Daniel Bengtsson proposed openstack/tripleo-heat-templates master: Use python -m instead pip directly. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855651 | 08:44 |
opendevreview | Jiri Podivin proposed openstack/tripleo-quickstart master: Removing workaround providing no longer necessary 'plan-environment.yaml' https://review.opendev.org/c/openstack/tripleo-quickstart/+/864167 | 08:51 |
opendevreview | Jiri Podivin proposed openstack/tripleo-quickstart master: Removing workaround providing no longer necessary 'plan-environment.yaml' https://review.opendev.org/c/openstack/tripleo-quickstart/+/864167 | 08:56 |
*** soniya29|rover|lunch is now known as soniya29|rover | 08:57 | |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14 https://review.opendev.org/c/openstack/tripleo-ansible/+/864392 | 09:17 |
*** ysandeep is now known as ysandeep|lunch | 09:35 | |
akahat|ruck | tkajinam, 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.gz | 09:58 |
akahat|ruck | tkajinam, i think we need to go with ansible version pin same thing suggested by ramishra | 09:59 |
*** jpodivin_ is now known as jpodivin | 10:01 | |
akahat|ruck | tkajinam, 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#L87 | 10:01 |
akahat|ruck | sorry.. didn't noticed it at first. | 10:01 |
akahat|ruck | https://codesearch.opendev.org/?q=rpm_query_result&i=nope&literal=nope&files=&excludeFiles=&repos= | 10:02 |
opendevreview | Giulio Fidente proposed openstack/tripleo-ansible stable/zed: Support setting ovn-ofctrl-wait-before-clear https://review.opendev.org/c/openstack/tripleo-ansible/+/864396 | 10:17 |
akahat|ruck | tkajinam, updating your patch with the fix. | 10:19 |
xek | ramishra: hey, can you help us sort out what's needed for https://review.opendev.org/c/openstack/tripleo-ansible/+/864165 ? | 10:25 |
opendevreview | amolkahat proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14 https://review.opendev.org/c/openstack/tripleo-ansible/+/864392 | 10:25 |
xek | ramishra: the patch itself works, because the gate failed initially and got fixed when I rebased the subsequent patch on top | 10:25 |
Tengu | maybe 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 |
opendevreview | Grzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Fix molecule jobs broken with latest cryptography https://review.opendev.org/c/openstack/tripleo-ansible/+/864183 | 10:29 |
xek | Tengu: ^ I cherry-picked it (without conflict), let's see if it passes CI (it failed on train) | 10:32 |
Tengu | xek: you got my +2 on the "move cryptography in". | 10:34 |
Tengu | *pin | 10:35 |
opendevreview | Arx Cruz proposed openstack/ansible-role-collect-logs master: Update ansible core version https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/864399 | 10:48 |
*** ysandeep|lunch is now known as ysandeep | 10:52 | |
opendevreview | Arx Cruz proposed openstack/ansible-role-collect-logs master: Update ansible core version https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/864399 | 11:03 |
*** soniya29 is now known as soniya29|rover | 11:04 | |
*** frenzy_friday is now known as frenzy_friday|doc_appt | 11:05 | |
xek | Tengu: cool, thanks for looking into it! | 11:12 |
opendevreview | amolkahat proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14 https://review.opendev.org/c/openstack/tripleo-ansible/+/864392 | 11:13 |
*** dviroel_ is now known as dviroel | 11:15 | |
*** mciecierski_ is now known as mciecierski | 11:32 | |
opendevreview | Lukas Bezdicka proposed openstack/tripleo-upgrade master: FFWD3 Refactor ffwd upgrade for train to wallaby https://review.opendev.org/c/openstack/tripleo-upgrade/+/849682 | 11:59 |
Tengu | xek: on you to chase down others, especially to get the -1 sorted. | 12:11 |
Tengu | marios, 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.log | 12:13 |
Tengu | ah, seeing amolkahat's patch, it is. | 12:14 |
Tengu | guess we'll need to do some changes in the tripleo-operator-ansible part. | 12:27 |
Tengu | let's play. | 12:27 |
akahat|ruck | marios, 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/+/864392 | 12:31 |
Tengu | akahat|ruck: checked your proposal for the fix - ramishra added some comment, would be good to address them imho | 12:32 |
akahat|ruck | Tengu, yeah.. just checked. working on it. Thanks ramishra! | 12:33 |
Tengu | akahat|ruck: ++ thanks for working that fast on those issues. | 12:33 |
Tengu | brb, need to make some tea. | 12:34 |
opendevreview | amolkahat proposed openstack/tripleo-ansible master: Fix compatibility with Ansible 2.14 https://review.opendev.org/c/openstack/tripleo-ansible/+/864392 | 12:36 |
*** pojadhav- is now known as pojadhav | 12:38 | |
opendevreview | Arx Cruz proposed openstack/ansible-role-collect-logs master: Update ansible core version https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/864399 | 12:38 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Modify how tripleo_nftables gets its configurations https://review.opendev.org/c/openstack/tripleo-ansible/+/863872 | 12:40 |
Tengu | akahat|ruck: -^^ will allow to test "in real life". | 12:40 |
Tengu | it died today with the new ansible version. | 12:41 |
opendevreview | Arx Cruz proposed openstack/ansible-role-collect-logs master: Fix ansible sanity job https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/864399 | 12:46 |
*** soniya is now known as soniya|rover | 12:46 | |
opendevreview | Rajesh Tailor proposed openstack/tripleo-heat-templates master: Allows configuring unified quotas in nova https://review.opendev.org/c/openstack/tripleo-heat-templates/+/864197 | 13:02 |
marios | akahat|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 |
marios | ERROR! Unexpected Exception, this is probably a bug: 'Task' object has no attribute '_valid_attrs' | 13:04 |
marios | good job you were working on the fix already tkajinam | 13:05 |
akahat|ruck | marios, waiting for +1 from zuul. | 13:06 |
lmiccini | hey folks can you give https://review.opendev.org/c/openstack/tripleo-ansible/+/863625 a push please? the other cherry pick merged, only missing wallaby | 13:10 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-operator-ansible master: Ansible-2.14 compatibility https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/864409 | 13:15 |
opendevreview | Arx Cruz proposed openstack/ansible-role-collect-logs master: Fix ansible sanity job https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/864399 | 13:29 |
opendevreview | Grzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Move cryprography pin https://review.opendev.org/c/openstack/tripleo-ansible/+/864165 | 13:46 |
opendevreview | Grzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Make creating/updating users idempotent https://review.opendev.org/c/openstack/tripleo-ansible/+/864034 | 13:47 |
opendevreview | Merged openstack/tripleo-docs master: Removing irrelavant references to 'plan-environment.yaml' https://review.opendev.org/c/openstack/tripleo-docs/+/863642 | 13:47 |
xek | Tengu: ^ | 13:49 |
Tengu | xek: voted - now that we have the complete chain it should be all good for marios :) | 13:50 |
opendevreview | Grzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Make creating/updating users idempotent https://review.opendev.org/c/openstack/tripleo-ansible/+/864034 | 13:51 |
opendevreview | Merged openstack/tripleo-docs master: Node scaling now covered by baremetal_provision https://review.opendev.org/c/openstack/tripleo-docs/+/861927 | 13:51 |
*** rcastillo|rover is now known as rcastillo | 13:57 | |
opendevreview | Merged openstack/tripleo-docs master: Update documentation about deployment status https://review.opendev.org/c/openstack/tripleo-docs/+/863195 | 14:00 |
*** amoralej is now known as amoralej|lunch | 14:07 | |
akahat|ruck | marios_, 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/+/41465 | 14:13 |
*** marios_ is now known as marios | 14:23 | |
opendevreview | Grzegorz Grasza proposed openstack/tripleo-ansible stable/wallaby: Make creating/updating users idempotent https://review.opendev.org/c/openstack/tripleo-ansible/+/864034 | 14:24 |
*** frenzy_friday|doc_appt is now known as frenzy_friday | 14:33 | |
*** dasm|off is now known as dasm | 14:46 | |
opendevreview | David Sedgmen proposed openstack/puppet-tripleo stable/train: Current change will not work reliably. https://review.opendev.org/c/openstack/puppet-tripleo/+/864186 | 14:47 |
opendevreview | David 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/+/864186 | 14:50 |
opendevreview | David 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/+/864186 | 14:52 |
*** amoralej|lunch is now known as amoralej | 15:06 | |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Remove redundant check of result.rc https://review.opendev.org/c/openstack/tripleo-ansible/+/864417 | 15:07 |
*** ysandeep is now known as ysandeep|out | 15:10 | |
*** soniya|rover is now known as soniya|out | 15:16 | |
*** dviroel is now known as dviroel|lunch | 15:28 | |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-operator-ansible master: Ansible-2.14 compatibility https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/864409 | 15:33 |
marios | dasm: 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 |
marios | dasm: like if it needs a recheck | 15:46 |
marios | kind of thing | 15:46 |
dasm | marios: i added myself to cc. i'll keep an eye on that | 15:47 |
marios | thank you dasm appreciate | 15:47 |
dasm | no worries | 15:47 |
marios | content providers timed out there checking | 15:49 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Refactoring default-node-count validation to use openstack-collection modules https://review.opendev.org/c/openstack/tripleo-validations/+/864421 | 15:53 |
marios | https://lists.openstack.org/pipermail/openstack-discuss/2022-November/031186.html [tripleo] gate blocker /tripleo/+bug/1996482 - please hold rechecks | 16:09 |
marios | tkajinam: ^ just fyi | 16:09 |
*** dviroel|lunch is now known as dviroel | 16:26 | |
*** marios is now known as marios|out | 16:31 | |
opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Add new nodepool_provider for ibm cloud https://review.opendev.org/c/openstack/tripleo-ci/+/864134 | 17:01 |
*** amoralej is now known as amoralej|off | 17:37 | |
*** jpena is now known as jpena|off | 17:39 | |
*** slagle-out is now known as slagle | 18:54 | |
*** dviroel is now known as dviroel|afk | 20:01 | |
opendevreview | Merged openstack/os-net-config stable/train: Updating Zuul CI jobs for os-net-config repo https://review.opendev.org/c/openstack/os-net-config/+/863592 | 20:29 |
opendevreview | Roberto Alfieri proposed openstack/tripleo-docs master: Switch from Centos Stream 8 to Centos Stream 9 https://review.opendev.org/c/openstack/tripleo-docs/+/864441 | 21:16 |
opendevreview | Steve Baker proposed openstack/tripleo-ansible stable/wallaby: Don't use meta:end_play after growvols check https://review.opendev.org/c/openstack/tripleo-ansible/+/864445 | 21:36 |
*** dasm is now known as dasm|off | 22:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!