| openstackgerrit | Merged openstack/os-net-config stable/train: SRIOV VF: Add min_tx_rate and max_tx_rate https://review.opendev.org/717677 | 00:12 |
|---|---|---|
| openstackgerrit | Merged openstack/puppet-tripleo master: Remove undefined octavia::rabbit_password from default hieradata https://review.opendev.org/718126 | 00:37 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove client for unsupported services from lower-constraint https://review.opendev.org/718583 | 00:39 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove clients for unsupported services from lower-constraints https://review.opendev.org/718583 | 00:40 |
| *** sanjayu_ has quit IRC | 00:42 | |
| *** cdearborn has quit IRC | 00:50 | |
| openstackgerrit | Flavio Fernandes proposed openstack/tripleo-heat-templates master: Add Octavia OVN Provider configuration https://review.opendev.org/707695 | 00:51 |
| *** jmasud has quit IRC | 00:54 | |
| *** jmasud has joined #tripleo | 00:55 | |
| *** raildo has quit IRC | 01:24 | |
| *** ysandeep|away is now known as ysandeep|rover | 01:52 | |
| *** Goneri has quit IRC | 01:59 | |
| *** mrunge_ has joined #tripleo | 02:02 | |
| *** mrunge has quit IRC | 02:03 | |
| openstackgerrit | Rabi Mishra proposed openstack/tripleo-ansible master: WIP escape newline in nested json string https://review.opendev.org/717953 | 02:06 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage: optimize systemd services/healthcheck bootstrap https://review.opendev.org/718545 | 02:17 |
| *** jdandrea has quit IRC | 02:40 | |
| *** ysandeep|rover is now known as ysandeep|afk | 03:09 | |
| *** rh-jelabarre has quit IRC | 03:12 | |
| *** social has quit IRC | 03:16 | |
| openstackgerrit | Merged openstack/puppet-tripleo stable/train: Support for PowerMax Volume Config https://review.opendev.org/717793 | 03:25 |
| openstackgerrit | Merged openstack/tripleo-common stable/stein: Wire-in missing concurrency for deploy_instances https://review.opendev.org/717196 | 03:26 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove hieradata for TripleO UI https://review.opendev.org/718107 | 03:28 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Cleanup remaining items for Trove https://review.opendev.org/718095 | 03:32 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/train: Workaround for cinder A/A and etcd with TLS-everywhere https://review.opendev.org/717837 | 03:36 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Enable caching in heat https://review.opendev.org/716967 | 03:36 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove duplicate key https://review.opendev.org/717106 | 03:36 |
| *** skramaja has joined #tripleo | 03:58 | |
| *** ysandeep|afk is now known as ysandeep | 04:00 | |
| *** evrardjp has quit IRC | 04:37 | |
| *** evrardjp has joined #tripleo | 04:37 | |
| *** udesale has joined #tripleo | 04:40 | |
| *** ykarel|away is now known as ykarel | 04:47 | |
| *** ysandeep is now known as ysandeep|rover | 04:56 | |
| openstackgerrit | Rabi Mishra proposed openstack/tripleo-ansible master: WIP escape newline in nested json string https://review.opendev.org/717953 | 05:02 |
| *** dmacpher has quit IRC | 05:03 | |
| *** dmacpher has joined #tripleo | 05:03 | |
| ramishra | ykarel: Hey! I guess we need a promotion for https://review.opendev.org/#/c/718127/? ovb jobs still failing | 05:05 |
| ramishra | https://trunk.rdoproject.org/centos8/component/baremetal/40/be/40bed1ea88e1b0e2cddc380f24823390f6e3bba3_50ba4be6/ does not seem to have the latest packages | 05:06 |
| ykarel | ramishra, where are u seeing issue ? the issue shouldn't be seen in upstream jobs | 05:06 |
| ykarel | only promotion jobs should fail due to that | 05:07 |
| ramishra | ykarel: https://logserver.rdoproject.org/53/717953/2/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001/0305619/logs/undercloud/home/zuul/overcloud_prep_images.log.txt.gz | 05:08 |
| ykarel | ramishra, looking, may be it's false promoted again | 05:09 |
| *** ratailor has joined #tripleo | 05:10 | |
| *** marios_ has joined #tripleo | 05:11 | |
| ykarel | ramishra, checked, it's again false promotion, ovb failed and still promoted https://trunk.rdoproject.org/api-centos8-master-uc/api/civotes_agg_detail.html?ref_hash=33c72c65c61b68a92cd1e9ace947ed4c | 05:13 |
| ykarel | so we need another promotion with that ironic fix | 05:14 |
| ykarel | ysandeep|rover, fyi ^^ | 05:14 |
| ysandeep|rover | ykarel, ack | 05:15 |
| ramishra | ykarel: Is it someone ignoring those ovb failures and force promoting? | 05:16 |
| ykarel | ramishra, yes, it's ignored, yesterdays promotion needed with ignoring ovb, but then it was missed to revert the criteria | 05:19 |
| ykarel | so it false promoted again | 05:19 |
| *** jbadiapa has quit IRC | 05:20 | |
| ykarel | ramishra, https://review.rdoproject.org/r/#/c/26311/ | 05:21 |
| ykarel | chandankumar, can u get ^^ merged | 05:21 |
| chandankumar | ykarel, on it | 05:21 |
| ramishra | ykarel: Ah ok.. thanks! | 05:21 |
| ykarel | for now i am appliying it manually on promoter | 05:21 |
| ramishra | sounds good, thanks for looking | 05:23 |
| *** fultonj has quit IRC | 05:25 | |
| *** fultonj has joined #tripleo | 05:26 | |
| *** bandini has joined #tripleo | 05:31 | |
| openstackgerrit | Merged openstack/python-tripleoclient stable/train: Ensure the undercloud output dir constant is consistent https://review.opendev.org/718415 | 05:35 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Add check for bootstrap_node for downcase https://review.opendev.org/718194 | 05:35 |
| openstackgerrit | Merged openstack/instack-undercloud stable/rocky: Remove trove firewall rule https://review.opendev.org/699328 | 05:35 |
| *** dwilde has quit IRC | 05:45 | |
| *** sanjayu_ has joined #tripleo | 06:06 | |
| *** dpawlik has joined #tripleo | 06:07 | |
| *** sanjayu__ has joined #tripleo | 06:09 | |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates master: Move the haproxy iptables rules creation to host_prep_tasks https://review.opendev.org/718159 | 06:11 |
| *** sanjayu_ has quit IRC | 06:12 | |
| openstackgerrit | Merged openstack/tripleo-ansible master: tripleo_all_nodes_data: sort enabled_services (idempotency) https://review.opendev.org/718471 | 06:30 |
| ramishra | ykarel: I guess you'll hit https://bugs.launchpad.net/bugs/1871338 | 06:33 |
| openstack | Launchpad bug 1871338 in tripleo "overcloud deployment failing with msg: 'argument parameters is of type <class ''str''> and we were unable to convert to dict: unable to evaluate string as dictionary'." [Critical,In progress] - Assigned to Sagi (Sergey) Shnaidman (sshnaidm) | 06:33 |
| ramishra | with promotion | 06:33 |
| ykarel | ramishra, yes | 06:33 |
| ramishra | I was testing https://review.opendev.org/#/c/717953/ to see if it fixes or works around it.. but it seems Depends-On also does not work | 06:34 |
| ramishra | for the ovb jobs | 06:34 |
| ykarel | ramishra, depends-on not working? | 06:34 |
| ramishra | looks like it did not https://logserver.rdoproject.org/53/717953/3/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001/3ed6cb1/logs/undercloud/home/zuul/overcloud_prep_images.log.txt.gz | 06:35 |
| ykarel | ramishra, ack /me checks logs | 06:35 |
| *** slaweq_ has quit IRC | 06:36 | |
| ykarel | ramishra, depends on do not work on merged patches | 06:36 |
| *** dpawlik has quit IRC | 06:37 | |
| ykarel | ramishra, if u see ZUUL_CHANGES=openstack/tripleo-ansible:master:refs/changes/53/717953/3, ironic patch is not there as it's merged | 06:37 |
| ykarel | in job-output.txt | 06:37 |
| ramishra | Hmm.. not sure how we can test it.. | 06:38 |
| ykarel | ramishra, there can be two option | 06:38 |
| *** bogdando has joined #tripleo | 06:39 | |
| ramishra | force promote the ironic fix? | 06:39 |
| *** dpawlik has joined #tripleo | 06:39 | |
| ykarel | either test with promotion job where we have ironic patch | 06:39 |
| ykarel | or depends-on an ironic patch that is not merged | 06:39 |
| ramishra | ykarel: good idea, let me try the second one and see if it works | 06:40 |
| ykarel | ramishra, ack | 06:40 |
| openstackgerrit | Rabi Mishra proposed openstack/tripleo-ansible master: WIP escape newline in nested json string https://review.opendev.org/717953 | 06:47 |
| *** xek_ has joined #tripleo | 06:48 | |
| *** bogdando has quit IRC | 06:53 | |
| *** bogdando has joined #tripleo | 06:55 | |
| *** sshnaidm|afk is now known as sshnaidm | 06:55 | |
| *** jcapitao has joined #tripleo | 06:57 | |
| *** jtomasek has joined #tripleo | 06:57 | |
| *** jtomasek has quit IRC | 06:59 | |
| *** jtomasek has joined #tripleo | 06:59 | |
| *** slaweq_ has joined #tripleo | 07:00 | |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed openstack/python-tripleoclient stable/train: Fix misused deployment vs connection timeouts https://review.opendev.org/718347 | 07:02 |
| openstackgerrit | Francesco Pantano proposed openstack/tripleo-docs master: Customizing crushmap using device classes https://review.opendev.org/718109 | 07:03 |
| *** slaweq_ is now known as slaweq | 07:03 | |
| *** dtantsur|afk is now known as dtantsur | 07:06 | |
| *** maciejjozefczyk has joined #tripleo | 07:10 | |
| *** rfolco has quit IRC | 07:11 | |
| *** iurygregory has joined #tripleo | 07:11 | |
| *** rpittau|afk is now known as rpittau | 07:18 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-role-collect-logs master: Add timeout for containers logs collection https://review.opendev.org/717604 | 07:18 |
| openstackgerrit | Takashi Kajinami proposed openstack/instack-undercloud stable/queens: Remove trove firewall rule https://review.opendev.org/718605 | 07:20 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/train: Include {{step}} when setting facts https://review.opendev.org/717526 | 07:25 |
| *** florianf has joined #tripleo | 07:36 | |
| *** psachin has joined #tripleo | 07:38 | |
| *** tosky has joined #tripleo | 07:42 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-operator-ansible master: Adds overcloud roles generate role - creates roles_data.yaml https://review.opendev.org/716639 | 07:43 |
| *** marios_ is now known as marios | 07:43 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Refactor image build for downstream and image-build role https://review.opendev.org/717043 | 07:44 |
| *** cylopez has joined #tripleo | 07:46 | |
| *** zbr has quit IRC | 07:46 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Refactor image build for downstream and image-build role https://review.opendev.org/717043 | 07:46 |
| *** zbr has joined #tripleo | 07:47 | |
| *** suuuper has joined #tripleo | 07:48 | |
| *** jschlueter has quit IRC | 07:49 | |
| *** ratailor has quit IRC | 07:51 | |
| *** jschlueter has joined #tripleo | 07:51 | |
| *** abdysn has joined #tripleo | 07:54 | |
| *** jpich has joined #tripleo | 07:55 | |
| *** ratailor has joined #tripleo | 07:58 | |
| *** pkopec has joined #tripleo | 07:59 | |
| *** threestrands has quit IRC | 08:00 | |
| *** ysandeep|rover is now known as ysandeep|lunch | 08:10 | |
| *** xek_ is now known as xek | 08:17 | |
| *** florianf has quit IRC | 08:17 | |
| *** lucasagomes has joined #tripleo | 08:18 | |
| openstackgerrit | Mark Goddard proposed openstack/tripleo-ci master: Exclude rabbitmq-3.7.24 container image https://review.opendev.org/718615 | 08:19 |
| *** alexmcleod has joined #tripleo | 08:28 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Deprecate KeepalivedRestart https://review.opendev.org/718492 | 08:29 |
| dciabrin_ | whois jfrancoa | 08:36 |
| dciabrin_ | oops sorry guys :) | 08:37 |
| *** dciabrin_ is now known as dciabrin | 08:37 | |
| ykarel | ramishra, Depends-On on ironic worked and it moved forward. | 08:37 |
| ykarel | also it looks like ur patch is working as it's in overcloud deploy for more than 20+ minutes | 08:38 |
| ykarel | looking https://review.rdoproject.org/zuul/stream/587204b34e114382812b529db0484109?logfile=console.log | 08:38 |
| ykarel | sshnaidm, fyi ^^ | 08:40 |
| ykarel | patch is https://review.opendev.org/#/c/717953/ | 08:41 |
| sshnaidm | nice | 08:42 |
| sshnaidm | so it's multiline issue | 08:42 |
| ykarel | me not aware why it passes in some jobs and fails in some | 08:43 |
| sshnaidm | ykarel, it passes multiline params in ovb, but singleline in multinode | 08:43 |
| ykarel | sshnaidm, seen failures in multionode job too | 08:44 |
| ykarel | fs030 | 08:44 |
| sshnaidm | maybe it has multiline params too? | 08:44 |
| sshnaidm | like ssh keys | 08:45 |
| ykarel | sshnaidm, failure logs https://logserver.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-multinode-1ctlr-featureset030-master/7097e91/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz | 08:45 |
| ykarel | sshnaidm, where to check multiline? | 08:45 |
| ramishra | ykarel: looks like it failed, don't know if the same issue or something else in a later stage | 08:50 |
| ykarel | ramishra, possibly different issue | 08:51 |
| openstackgerrit | Merged openstack/os-apply-config master: Use unittest.mock instead of third party mock https://review.opendev.org/717411 | 08:51 |
| ykarel | let's wait for logs and confirm | 08:53 |
| sshnaidm | ykarel, rendered_parameters | 08:54 |
| sshnaidm | ykarel, banner text is multinode | 08:54 |
| sshnaidm | s/multinode/multiline | 08:55 |
| ykarel | sshnaidm, hmm got it | 08:55 |
| ykarel | so in featureset why it's not multiline | 08:55 |
| ykarel | featureset 10 | 08:55 |
| ykarel | me checks logs | 08:55 |
| *** derekh has joined #tripleo | 08:56 | |
| ykarel | hmm that parameter is not there https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_334/717865/6/check/tripleo-ci-centos-8-containers-multinode/334ec7d/logs/undercloud/home/zuul/overcloud_deploy.log | 08:57 |
| ykarel | ohhk it's set only for some scenarios http://codesearch.openstack.org/?q=BannerText&i=nope&files=&repos= | 08:58 |
| *** social has joined #tripleo | 08:59 | |
| ramishra | ykarel: yeah the patch worked.. it seems to have failed with "2020-04-09 08:44:13 | <13>Apr 9 08:44:11 puppet-user: Error: /Stage[main]/Pacemaker::Service/Service[pcsd]/ensure: change from 'stopped' to 'running' failed: Systemd start for pcsd failed!" | 09:00 |
| ykarel | ramishra, cool | 09:01 |
| *** tkajinam has quit IRC | 09:03 | |
| *** ykarel is now known as ykarel|lunch | 09:09 | |
| *** dpawlik has quit IRC | 09:10 | |
| *** holser has joined #tripleo | 09:11 | |
| openstackgerrit | amolkahat proposed openstack/tripleo-ci master: Removed centos7-scenario012 job. Added job tripleo-ci-centos-7-scenario012-standalone in depricated-jobs.yaml file. https://review.opendev.org/718346 | 09:12 |
| *** holser has joined #tripleo | 09:12 | |
| *** sanjayu__ is now known as saneax_lunch | 09:14 | |
| ramishra | Hmm...https://logserver.rdoproject.org/53/717953/4/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-vexxhost/587204b/logs/overcloud-controller-0/var/log/pcsd/pcsd.log.txt.gz | 09:17 |
| ramishra | bandini: ^^^ any idea what's wrong | 09:17 |
| *** ysandeep|lunch is now known as ysandeep|rover | 09:18 | |
| bandini | ramishra: we have seen that one one other time, tried to debug it and failed since it reproduces rarely and we did not have the env | 09:19 |
| ramishra | bandini: you mean https://bugs.launchpad.net/tripleo/+bug/1867602 | 09:20 |
| openstack | Launchpad bug 1867602 in tripleo "overcloud deploy failed due to Systemd start for pcsd failed" [Critical,Incomplete] | 09:20 |
| bandini | ramishra: yeah, ideally we'd add pacemaker::corosync::pcsd_debug: true' to CI so we can investigate more | 09:21 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-heat-templates master: WIP: refactor nova db config https://review.opendev.org/718552 | 09:21 |
| openstackgerrit | amolkahat proposed openstack/tripleo-ci master: Port tripleo-ci-centos-8-scenario012-standalone job to os_tempest https://review.opendev.org/717995 | 09:26 |
| owalsh | gchamoul: hey, getting issues with tripleo-validations lower constraints on stable/train... | 09:32 |
| owalsh | https://5e8b4515abd7c7f803b6-b71c90f67122d6f7b38b8442bc3a114a.ssl.cf2.rackcdn.com/718557/1/check/openstack-tox-lower-constraints/b78a02c/job-output.txt | 09:32 |
| owalsh | gchamoul: is that using oslo.service from master instead of train? | 09:33 |
| openstackgerrit | Piotr Kopec proposed openstack/tripleo-ansible master: Add new role to manage persistent memory devices https://review.opendev.org/717158 | 09:34 |
| *** udesale_ has joined #tripleo | 09:34 | |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Fixed typos https://review.opendev.org/718637 | 09:35 |
| *** udesale has quit IRC | 09:38 | |
| *** florianf has joined #tripleo | 09:39 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/tripleo-heat-templates stable/train: Properly place undercloud hosts record upgrade task https://review.opendev.org/718640 | 09:39 |
| *** mgoddard has joined #tripleo | 09:45 | |
| mgoddard | Hi | 09:45 |
| mgoddard | I'm trying to add an exclude for a container image that we are adding in kolla | 09:46 |
| mgoddard | https://review.opendev.org/#/c/718615/ | 09:46 |
| mgoddard | I can see that it has been picked up: https://logserver.rdoproject.org/24/715224/3/openstack-check/tripleo-build-containers-rhel-8-train/0c31d19/logs/build_containers.sh | 09:46 |
| mgoddard | but tripleo still tries to build it: https://logserver.rdoproject.org/24/715224/3/openstack-check/tripleo-build-containers-rhel-8-train/0c31d19/logs/containers-expected-to-build.log | 09:47 |
| mgoddard | anyone have any ideas? | 09:47 |
| mgoddard | cloudnull: ^ | 09:47 |
| *** dpawlik has joined #tripleo | 09:49 | |
| *** ykarel|lunch is now known as ykarel | 09:51 | |
| *** surpatil has joined #tripleo | 09:51 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: Drop eventlet from lower constraints https://review.opendev.org/718645 | 09:53 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: tripleo-validations failing on lower constraints https://review.opendev.org/718649 | 10:05 |
| *** jpich has quit IRC | 10:06 | |
| bandini | ade_lee: I have an environmnet with https://bugs.launchpad.net/tripleo/+bug/1871703/comments/3 | 10:06 |
| openstack | Launchpad bug 1871703 in tripleo " $ca_pkcs12 = '/var/lib/certmonger/local/creds' not found in centos-8 ussuri undercloud deployment" [High,Triaged] | 10:06 |
| *** jpich has joined #tripleo | 10:07 | |
| *** Abdallahyas has joined #tripleo | 10:07 | |
| owalsh | ramishra, gchamoul: hi, is this right? https://review.opendev.org/718649 | 10:08 |
| *** abdysn has quit IRC | 10:10 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: Fix multistack inventory https://review.opendev.org/718557 | 10:10 |
| *** gfidente|afk is now known as gfidente | 10:11 | |
| ramishra | owalsh: that's not true or stable/train.. eventlet is not bumped in oslo.service https://github.com/openstack/oslo.service/blob/stable/train/requirements.txt#L7 | 10:12 |
| owalsh | ramishra: hrmm, so we need to set constraints for oslo.service? | 10:13 |
| ramishra | What error do you see.. may be someting wrong with stable jobs for tripleo-validations | 10:13 |
| owalsh | ramishra: https://5e8b4515abd7c7f803b6-b71c90f67122d6f7b38b8442bc3a114a.ssl.cf2.rackcdn.com/718557/1/check/openstack-tox-lower-constraints/b78a02c/job-output.txt | 10:13 |
| ramishra | owalsh: removing that is fine.. but the reason is not true | 10:13 |
| *** rpittau is now known as rpittau|bbl | 10:13 | |
| ykarel | ramishra, wrt to pcsd start error, u posted some fix? | 10:14 |
| ykarel | i see u posted a recheck-rdo there | 10:14 |
| owalsh | ramishra: yea, didn't seem right to me, why I'm asking for sanity checks :-) | 10:14 |
| *** apetrich has quit IRC | 10:15 | |
| openstackgerrit | Sorin Sbarnea proposed openstack/ansible-role-collect-logs master: Convert flatten_nested_dict filter into a module https://review.opendev.org/717723 | 10:15 |
| ramishra | ykarel: no I don't yet know the issue.. Just checked if it's reproducing as https://bugs.launchpad.net/tripleo/+bug/1867602 has been marked incomplete | 10:16 |
| openstack | Launchpad bug 1867602 in tripleo "overcloud deploy failed due to Systemd start for pcsd failed" [Critical,Incomplete] | 10:16 |
| ramishra | which looks the same | 10:16 |
| ykarel | chandankumar, ohhk, two jobs failed with same error so seems a real issue | 10:16 |
| ykarel | me checks logs to see if can find something | 10:16 |
| chandankumar | ykarel, which one | 10:17 |
| ykarel | chandankumar, https://review.opendev.org/#/c/717953/ ovb failures | 10:18 |
| ykarel | i see E, [2020-04-09T08:49:51.199 #00000] ERROR -- : Unable to start pcsd daemon, exiting: [Errno 2] No such file or directory: '/var/lib/pcsd/pcsd.crt' in pcsd logs | 10:20 |
| chandankumar | ykarel, above bug then | 10:20 |
| openstackgerrit | amolkahat proposed openstack/tripleo-quickstart master: Ported featureset060.yml to os_tempest. https://review.opendev.org/718091 | 10:20 |
| ykarel | chandankumar, yes, | 10:21 |
| ykarel | chandankumar, from bandini suggestion is pacemaker::corosync::pcsd_debug: true tried | 10:21 |
| ykarel | and got something? | 10:21 |
| chandankumar | ykarel, last time, we failed to reproduce it | 10:22 |
| chandankumar | ykarel, https://review.opendev.org/#/c/713219/ | 10:22 |
| bandini | i don't think it has ever been tried? or i never got to see any debug logs | 10:22 |
| owalsh | ramishra: also had to cherry pick the master patch that bumps the keystonauth1/openstacksdk/os-service-types ... more concerned about that | 10:22 |
| ykarel | chandankumar, ohhk, let' see if it reproduces again, | 10:23 |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-heat-templates master: [DNM] pcs debug https://review.opendev.org/713219 | 10:23 |
| chandankumar | ykarel, ^^ | 10:23 |
| ykarel | chandankumar, ack /me adjusts | 10:23 |
| openstackgerrit | yatin proposed openstack/tripleo-heat-templates master: [DNM] pcs debug https://review.opendev.org/713219 | 10:24 |
| ykarel | chandankumar, updated with depends-on on fixes | 10:25 |
| ykarel | bandini, will let u know if it reproduces | 10:25 |
| bandini | ack thanks I cced myself there in the meantime | 10:25 |
| ykarel | okk thanks | 10:26 |
| ykarel | ramishra, fyi promoting that ironic fix | 10:28 |
| ykarel | https://trunk.rdoproject.org/api-centos8-master-uc/api/civotes_agg_detail.html?ref_hash=e979cbb265693180fcde894768701cda | 10:28 |
| *** xek has quit IRC | 10:32 | |
| openstackgerrit | Sorin Sbarnea proposed openstack/ansible-role-collect-logs master: Convert flatten_nested_dict filter into a module https://review.opendev.org/717723 | 10:32 |
| *** saneax_lunch is now known as saneax_ | 10:34 | |
| *** surpatil has quit IRC | 10:44 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Refactor image build for downstream and image-build role https://review.opendev.org/717043 | 10:52 |
| *** jcapitao is now known as jcapitao_lunch | 10:59 | |
| *** apetrich has joined #tripleo | 11:00 | |
| *** mgoddard has quit IRC | 11:02 | |
| ramishra | ykarel: ack | 11:07 |
| ramishra | owalsh: I don't know why you've to bump minimum version of libraries in stable branches unless it's being bumped in a dep.. May be something wrong with the job | 11:11 |
| *** rfolco has joined #tripleo | 11:34 | |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Make tripleo_shell_script more robust https://review.opendev.org/718675 | 11:34 |
| owalsh | ramishra: yea, missing upper constraints for train? | 11:36 |
| *** ratailor has quit IRC | 11:36 | |
| owalsh | ramishra: https://github.com/openstack/tripleo-validations/blob/stable/train/tox.ini#L25 | 11:37 |
| owalsh | ramishra: not used in the lower-constraints job though | 11:40 |
| ramishra | owalsh: that's an issue and should be fixed for stable branches but lower-constraints job uses https://github.com/openstack/tripleo-validations/blob/stable/train/tox.ini#L120 | 11:40 |
| ramishra | yeah | 11:40 |
| *** rh-jelabarre has joined #tripleo | 11:41 | |
| *** ysandeep|rover is now known as ysandeep|coffee | 11:47 | |
| owalsh | ramishra: weird, job just passed for me without bumping any versions, just removing evenlet | 11:48 |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Refactor image build for downstream and image-build role https://review.opendev.org/717043 | 11:48 |
| ykarel | ramishra, master promoted, ironic fix is available now in current-tripleo | 11:49 |
| ramishra | owalsh: yeah, that's what the error https://5e8b4515abd7c7f803b6-b71c90f67122d6f7b38b8442bc3a114a.ssl.cf2.rackcdn.com/718557/1/check/openstack-tox-lower-constraints/b78a02c/job-output.txt about | 11:51 |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Make tripleo_shell_script more robust https://review.opendev.org/718675 | 11:52 |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Ensure we don't pass empty params https://review.opendev.org/718678 | 11:52 |
| owalsh | ramishra: yes, earlier when I tested the fix locally I also had to cherry pick the other patch... but now it passes with just the eventlet patch | 11:52 |
| ramishra | I fail to understand where it's pulling that oslo.service that needs eventlet>=0.22.0 | 11:52 |
| ramishra | owalsh: I can look at it later | 11:52 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/train: Exclude /var/lib/ironic/* from container-puppet.sh rsync https://review.opendev.org/718548 | 11:53 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/train: Deprecate KeepalivedRestart https://review.opendev.org/718679 | 11:53 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: tripleo_all_nodes_data: sort enabled_services (idempotency) https://review.opendev.org/718680 | 11:53 |
| *** dpawlik has quit IRC | 11:56 | |
| *** rpittau|bbl is now known as rpittau | 11:57 | |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Ensure we don't pass empty params https://review.opendev.org/718678 | 11:59 |
| *** ysandeep|coffee is now known as ysandeep|rover | 12:00 | |
| EmilienM | chem: sorry I had to -2 https://review.opendev.org/#/c/717719 -- this is clearly not the solution | 12:00 |
| EmilienM | chem: we have had the issue at each cycle and each time it's the same thing, we need to update tripleoclient dependencies versions | 12:00 |
| EmilienM | chem: the path forward is the following : | 12:01 |
| EmilienM | chem: 1) make sure there is a recent tag in tripleo-ansible that contains the new ansible plugin | 12:01 |
| EmilienM | chem: 2) update tripleoclient-distgit to depends on tripleo-ansible newtag | 12:01 |
| EmilienM | that's it | 12:01 |
| chem | EmilienM: hey, so I have no problem for this review to go away, it's just that I opened it after a discussion (https://bugzilla.redhat.com/show_bug.cgi?id=1817457#c5) where it was said that maybe maintening that dependency inside the pkg was not a good idea | 12:03 |
| openstack | bugzilla.redhat.com bug 1817457 in python-tripleoclient "[osp16] Undercloud update fails, ansible cannot parse the generated playbook: ERROR! no action detected in task" [Medium,On_dev] - Assigned to sathlang | 12:03 |
| EmilienM | chem: discussion with who? | 12:04 |
| EmilienM | chem: that's how we have solved these (common) issues at every release | 12:04 |
| chem | EmilienM: I think jfrancoa mwhahaha and odyssey4me were around | 12:04 |
| chem | EmilienM: butagain no problem going that way if that's the usual solution, but I would need some mentoring/pointer as I never did that before | 12:05 |
| EmilienM | chem: I'm on it | 12:06 |
| EmilienM | chem: I'll show you what I did | 12:06 |
| chem | EmilienM: thanks, appreciated | 12:06 |
| owalsh | ramishra: tripleo-common in lower-constraints is ancient | 12:06 |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Make tripleo_shell_script more robust https://review.opendev.org/718675 | 12:07 |
| EmilienM | chem: step 1 https://review.opendev.org/#/c/718683 | 12:07 |
| *** Abdallahyas has quit IRC | 12:07 | |
| *** dpawlik has joined #tripleo | 12:08 | |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Use absolute path to include puppet classes https://review.opendev.org/718684 | 12:10 |
| owalsh | ramishra: and that version of tripleo-common requires mistral which requires oslo.service | 12:10 |
| EmilienM | chem: step2: https://review.rdoproject.org/r/26358 | 12:10 |
| EmilienM | chem: it was THT in fact, but same result | 12:10 |
| *** raildo has joined #tripleo | 12:10 | |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Use absolute name to include puppet classes https://review.opendev.org/718684 | 12:10 |
| owalsh | ramishra: can either pin mistral (which t-h-t lc does) or bump tripleo-common to a newer version that doesn't require mistral | 12:11 |
| *** jcapitao_lunch is now known as jcapitao | 12:13 | |
| *** florianf has quit IRC | 12:15 | |
| chem | EmilienM: sorry, just one last question abouth the train entry in the release file | 12:17 |
| chem | EmilienM: https://review.opendev.org/#/c/718683/1/deliverables/train/tripleo-ansible.yaml | 12:17 |
| chem | EmilienM: and then I just sit back and applause :) | 12:17 |
| EmilienM | chem: answered | 12:20 |
| ramishra | owalsh: Looks like tripleo-common 7.1.0 does not have lower constraints ..yeah bumping tripleo-common would probably be the right thing | 12:20 |
| openstackgerrit | Merged openstack/os-refresh-config master: Cleanup py27 support https://review.opendev.org/717568 | 12:20 |
| chem | EmilienM: thanks, all clear now. | 12:20 |
| cloudnull | mornings | 12:20 |
| owalsh | ramishra: ack, 11.0.0? | 12:21 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-operator-ansible master: Add missed variables to image upload command https://review.opendev.org/718687 | 12:23 |
| sshnaidm | mwhahaha, marios ^^ | 12:23 |
| ramishra | owalsh: may be 11.3.2 https://github.com/openstack/releases/blob/master/deliverables/train/tripleo-common.yaml#L30 | 12:24 |
| owalsh | ramishra: ack, thanks! | 12:25 |
| EmilienM | chem: I looked-up and found a few examples where we did the same thing: https://review.rdoproject.org/r/#/c/20986/ - https://review.rdoproject.org/r/#/c/20245/ - https://review.rdoproject.org/r/#/c/19747/ | 12:26 |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed openstack/paunch master: Process labels before building container run args https://review.opendev.org/718688 | 12:26 |
| *** psachin has quit IRC | 12:26 | |
| owalsh | ramishra: too new, metalsmith requires a newer openstacksdk, 11.3.1 is good | 12:28 |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed openstack/paunch stable/train: Process labels before building container run args https://review.opendev.org/718459 | 12:28 |
| ramishra | owalsh: ok | 12:29 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: tripleo-validations failing on lower constraints https://review.opendev.org/718649 | 12:31 |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Refactor image build for downstream and image-build role https://review.opendev.org/717043 | 12:31 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: Fix multistack inventory https://review.opendev.org/718557 | 12:32 |
| paramite | Greetings guys, I have a question. Does Keystone user creation during overcloud deployment changed recently? | 12:32 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-ci master: WIP: Modify conatiner build role to allow for internal builds https://review.opendev.org/717895 | 12:34 |
| *** pkopec has quit IRC | 12:35 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Refactor image build for downstream and image-build role https://review.opendev.org/717043 | 12:43 |
| *** iurygregory has quit IRC | 12:43 | |
| *** iurygregory has joined #tripleo | 12:44 | |
| *** hkominos has joined #tripleo | 12:49 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-heat-templates master: WIP: refactor nova db config https://review.opendev.org/718552 | 12:50 |
| *** lbragstad_ has joined #tripleo | 12:52 | |
| *** lbragstad has quit IRC | 12:54 | |
| *** jamesden_ has joined #tripleo | 12:55 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-heat-templates master: WIP: refactor nova db config https://review.opendev.org/718552 | 12:55 |
| openstackgerrit | Maciej Józefczyk proposed openstack/tripleo-heat-templates master: WIP: Change default dns domain in neutron https://review.opendev.org/699407 | 12:56 |
| openstackgerrit | Flavio Fernandes proposed openstack/tripleo-heat-templates master: Add Octavia OVN Provider configuration https://review.opendev.org/707695 | 12:56 |
| hkominos | Hi guys. During an attepted update from Rocky to Stein I get an "ResourceFailure: resources.Networks: Resource UPDATE failed: resources.ManagementNetwork: Updating a stack when it is deleting is not supported." My rocky templates had the default behaviour . (enabled but not in the role). Did something change in the way rendering happens for the | 12:57 |
| hkominos | network templates ? | 12:57 |
| *** jamesdenton has quit IRC | 12:57 | |
| ykarel | ramishra, bandini, it failed and logs with debug enabled patch- https://logserver.rdoproject.org/19/713219/3/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001/1b7bf6b/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz | 12:59 |
| *** cdearborn has joined #tripleo | 12:59 | |
| hkominos | nvm | 13:03 |
| owalsh | ramishra: I hate pip! tripleo-common 11.3.1 works for me but fails in CI | 13:07 |
| bandini | ykarel: ack will look after this call | 13:07 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Use absolute names to include puppet classes https://review.opendev.org/718684 | 13:07 |
| owalsh | tripleo-common requires metalsmith requires newer openstacksdk, but tripleo-common lower-constraints pins a lower openstacksdk but magically works (because pip is dumb I assume) | 13:08 |
| ykarel | bandini, okk | 13:11 |
| *** ykarel is now known as ykarel|afk | 13:11 | |
| *** mrunge_ is now known as mrunge | 13:14 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: tripleo-validations failing on lower constraints https://review.opendev.org/718649 | 13:16 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: Fix multistack inventory https://review.opendev.org/718557 | 13:17 |
| openstackgerrit | Luke Short proposed openstack/python-tripleoclient stable/train: Expose --limit, --skip-tags, and --tags on the CLI. https://review.opendev.org/716654 | 13:19 |
| weshay|ruck | mwhahaha, EmilienM as you guys have time.. understanding thoughts / vote on https://review.opendev.org/#/c/718468/ | 13:25 |
| openstackgerrit | Flavio Fernandes proposed openstack/tripleo-heat-templates stable/train: Add Octavia OVN Provider configuration https://review.opendev.org/714916 | 13:25 |
| *** pkopec has joined #tripleo | 13:26 | |
| marios | mwhahaha: should we nuke that file? https://review.opendev.org/#/c/716639/15/roles/tripleo_overcloud_roles_generate/tests/test.yml folks were asking about it but as far as we can see isn't used anywhere | 13:28 |
| marios | mwhahaha: do you want it there for merge or should we remove? | 13:28 |
| marios | mwhahaha: (its in all the roles though so perhaps you have some plan for those) | 13:28 |
| mwhahaha | just leave it, we have it in all because that's one of the default role things | 13:29 |
| marios | mwhahaha: k zbr fyi ^^^ | 13:29 |
| sshnaidm | mwhahaha, do you know why we have TripleOImagePrepareDefault https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/container_image.py#L885:L885 | 13:29 |
| sshnaidm | mwhahaha, how would I call it in operator with "... default" | 13:30 |
| mwhahaha | tripleo_container_image_prepare_default ? | 13:30 |
| mwhahaha | already exists? | 13:30 |
| EmilienM | dciabrin, bandini : can you guys look at https://review.opendev.org/#/c/718615/ ? | 13:30 |
| weshay|ruck | cloudnull, https://review.opendev.org/#/c/718468/ | 13:31 |
| mwhahaha | sshnaidm: https://github.com/openstack/tripleo-operator-ansible/blob/master/roles/tripleo_container_image_prepare_default/tasks/main.yml ? | 13:31 |
| sshnaidm | mwhahaha, oh, I didn't notice it's a different role, thanks | 13:32 |
| bandini | EmilienM: done. (dciabrin has his birthday today so he is probably not in front of the pc, I hope) | 13:34 |
| zbr | mwhahaha: marios: i doubt we need a tests/test.yaml as we we have molecule/default/converge.yml for that. Good idea to propose a wide removal and see what happens. | 13:35 |
| weshay|ruck | ade_lee, fyi https://bugs.launchpad.net/tripleo/+bug/1871703 looks like bandini hit it as well | 13:35 |
| openstack | Launchpad bug 1871703 in tripleo " $ca_pkcs12 = '/var/lib/certmonger/local/creds' not found in centos-8 ussuri undercloud deployment" [High,Triaged] | 13:35 |
| zbr | likely a leftover carried away from the template. | 13:35 |
| weshay|ruck | ade_lee, /me opens new tmate | 13:35 |
| mwhahaha | zbr: marios: i don't really care, we can remove if you feel strongly about it | 13:35 |
| zbr | test/ folder is used by testinfra, but is supposed to have python code inside. | 13:35 |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: Run container image prepare with operator https://review.opendev.org/718701 | 13:35 |
| marios | zbr: indeed it isn't used currently... | 13:36 |
| marios | zbr: i guess the main 'problem' is that it is in all the roles... so it will require a cleanup patch to remove all | 13:36 |
| marios | zbr: i can remove from this patch though ... k sec | 13:36 |
| bandini | weshay|ruck: yeah, I have an env ready to be accessed and debugged (fwiw) | 13:38 |
| EmilienM | bandini: thank you sir | 13:40 |
| openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [Q->T] Introduce Queens to Train Upgrade https://review.opendev.org/713482 | 13:40 |
| weshay|ruck | bandini, aye.. I'll grab you and ade in an internal channel where we can share tmates.. etc | 13:40 |
| bandini | weshay|ruck: ack let me drop the access info to you and ade | 13:40 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-operator-ansible master: Adds overcloud roles generate role - creates roles_data.yaml https://review.opendev.org/716639 | 13:40 |
| marios | mwhahaha: zbr: removed test.yaml ^^ | 13:40 |
| marios | sshnaidm: please revote when you get chance thanks ^ | 13:41 |
| zbr | marios: can you also open another change that removes all fake tests.yaml files? | 13:42 |
| openstackgerrit | Flavio Fernandes proposed openstack/puppet-tripleo master: Add Octavia OVN Provider configuration (2 of 2) https://review.opendev.org/713716 | 13:42 |
| marios | zbr: sure | 13:42 |
| ade_lee | weshay|ruck, I'm curious -- did the creds ever show up on your system? | 13:44 |
| weshay|ruck | ade_lee, bandini can you guys join #tls_help internal | 13:46 |
| ade_lee | EmilienM, mwhahaha hjensas cloudnull https://review.opendev.org/#/c/718120/ and https://review.opendev.org/#/c/718191/ please | 13:48 |
| *** dwilde has joined #tripleo | 13:48 | |
| openstackgerrit | Flavio Fernandes proposed openstack/puppet-tripleo stable/train: Add Octavia OVN Provider configuration (2 of 2) https://review.opendev.org/715960 | 13:52 |
| *** rcernin has quit IRC | 13:54 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage: optimize systemd services/healthcheck bootstrap https://review.opendev.org/718545 | 13:57 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage: optimize systemd services/healthcheck bootstrap https://review.opendev.org/718545 | 13:57 |
| sshnaidm | cloudnull, ramishra so we go with this patch https://review.opendev.org/#/c/717953/ and I can abandon mine? | 13:58 |
| ramishra | sshnaidm: though it worked when updating parameters, there is some issue later.. we're still looking at that | 14:00 |
| ramishra | don't know if it's related | 14:00 |
| sshnaidm | ramishra, ack | 14:00 |
| cloudnull | sshnaidm ramishra. | 14:01 |
| cloudnull | ++ | 14:01 |
| cloudnull | that's a good find | 14:01 |
| *** TrevorV has joined #tripleo | 14:01 | |
| *** ykarel|afk is now known as ykarel | 14:02 | |
| *** abdysn has joined #tripleo | 14:04 | |
| *** cloudnull has quit IRC | 14:08 | |
| openstackgerrit | Merged openstack/tripleo-operator-ansible master: Adds overcloud roles generate role - creates roles_data.yaml https://review.opendev.org/716639 | 14:09 |
| *** tkajinam has joined #tripleo | 14:13 | |
| *** Goneri has joined #tripleo | 14:14 | |
| *** pojadhav is now known as pojadhav|away | 14:15 | |
| openstackgerrit | Merged openstack/tripleo-ansible master: Don't call sdk.connect() again https://review.opendev.org/717952 | 14:20 |
| *** lbragstad_ is now known as lbragstad | 14:20 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: tripleo-validations failing on lower constraints https://review.opendev.org/718649 | 14:22 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-validations stable/train: Fix multistack inventory https://review.opendev.org/718557 | 14:23 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Use kolla tools for memcached configuration https://review.opendev.org/718565 | 14:25 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-operator-ansible master: Update tripleo-repos to match pattern https://review.opendev.org/718135 | 14:30 |
| *** rascasoft has joined #tripleo | 14:38 | |
| openstackgerrit | Merged openstack/tripleo-operator-ansible master: Add missed variables to image upload command https://review.opendev.org/718687 | 14:40 |
| ade_lee | EmilienM, mwhahaha https://review.opendev.org/#/c/718120/ please | 14:42 |
| openstackgerrit | Merged openstack/tripleo-operator-ansible master: Fixed typos https://review.opendev.org/718637 | 14:44 |
| openstackgerrit | Merged openstack/tripleo-operator-ansible master: Ensure we don't pass empty params https://review.opendev.org/718678 | 14:47 |
| openstackgerrit | Merged openstack/tripleo-operator-ansible master: Make tripleo_shell_script more robust https://review.opendev.org/718675 | 14:47 |
| *** bogdando has quit IRC | 14:49 | |
| *** ysandeep|rover is now known as ysandeep|afk | 14:51 | |
| *** abdysn has quit IRC | 14:54 | |
| *** abdysn has joined #tripleo | 14:54 | |
| weshay|ruck | chandankumar, bandini let's sync on some overcloud image build issues | 14:54 |
| weshay|ruck | chandankumar, https://bugs.launchpad.net/tripleo/+bug/1867602 https://bugs.launchpad.net/tripleo/+bug/1871703 | 14:55 |
| openstack | Launchpad bug 1867602 in tripleo "overcloud deploy failed due to Systemd start for pcsd failed" [Critical,Triaged] | 14:55 |
| openstack | Launchpad bug 1871703 in tripleo " $ca_pkcs12 = '/var/lib/certmonger/local/creds' not found in centos-8 ussuri undercloud deployment" [High,Triaged] | 14:55 |
| *** cloudnull has joined #tripleo | 14:55 | |
| weshay|ruck | bandini, not sure if I understand yet.. what is wrong w/ the centos-8 build .. yet | 14:55 |
| bandini | weshay|ruck: yeah trying to narrow down the prob a bit | 14:55 |
| bandini | let me update the pcsd bz in the meantime | 14:57 |
| *** abdysn has quit IRC | 15:00 | |
| *** cloudnull has quit IRC | 15:01 | |
| *** cloudnull3 has joined #tripleo | 15:01 | |
| *** cloudnull3 is now known as cloudnull | 15:02 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-ci master: WIP: Modify conatiner build role to allow for internal builds https://review.opendev.org/717895 | 15:04 |
| *** hkominos has quit IRC | 15:04 | |
| *** rh-jelabarre has quit IRC | 15:06 | |
| *** slaweq has quit IRC | 15:06 | |
| lbragstad | EmilienM i ended up testing a stable/train ovb environment last night (sans TLS or using FreeIPA for DNS) and the scale down tasks failed for me | 15:07 |
| lbragstad | is that a relatively new feature? or is not fully baked? | 15:07 |
| lbragstad | i wanted to see if i could successfully scale down a compute instance without any of the TLS changes we introduced | 15:08 |
| EmilienM | lbragstad: yeah me too it failed | 15:09 |
| EmilienM | because the scale down tasks only work if podman is enabled instead of docker | 15:09 |
| EmilienM | the default will switch in ussuri (podman vs docker) | 15:10 |
| lbragstad | bah... so that's why it's failing to find that service | 15:10 |
| EmilienM | yup | 15:10 |
| lbragstad | got it- that makes sense | 15:10 |
| EmilienM | so you have 2 options : | 15:10 |
| lbragstad | doesn't explain the connectivity issues we're seeing, but it helps | 15:10 |
| EmilienM | 1) deploy without HA (one controller) and enable podman (centos7) | 15:10 |
| EmilienM | 2) deploy on centos8, but i don't know if we have packages/containers | 15:11 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Add missing memcached_servers for undercloud placement https://review.opendev.org/715234 | 15:11 |
| openstackgerrit | Merged openstack/tripleo-ci master: Revert "make scenario001 non-voting until packages are sane" https://review.opendev.org/718477 | 15:11 |
| EmilienM | i would do 1) | 15:11 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Added scale_tasks to handle cleanup on scale down of nodes https://review.opendev.org/706564 | 15:11 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove Dell EMC PS Series Driver support https://review.opendev.org/717145 | 15:11 |
| EmilienM | lbragstad: I'm going to fix the scale_down tasks to be skipped when docker | 15:11 |
| lbragstad | EmilienM yeah - we haven't gotten our automation working with centos8 yet :( | 15:11 |
| openstackgerrit | Merged openstack/python-tripleoclient master: Fix ResourceWarning: unclosed file https://review.opendev.org/718214 | 15:11 |
| lbragstad | ok - so scale down will only be supported for podman deployments? | 15:12 |
| EmilienM | yup | 15:12 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: nova-compute: disable scale_tasks when docker is used https://review.opendev.org/718727 | 15:13 |
| EmilienM | lbragstad: ^ that will help with the error | 15:13 |
| openstackgerrit | Luke Short proposed openstack/python-tripleoclient master: Expose --limit, --skip-tags, and --tags on the CLI. https://review.opendev.org/692421 | 15:16 |
| lbragstad | EmilienM looking | 15:17 |
| lbragstad | EmilienM also - trying to figure out why docker doesn't find that container on the compute host we're scaling down | 15:18 |
| lbragstad | do we know why tripleo_nova_compute_healthcheck isn't on the compute host? | 15:18 |
| EmilienM | when docker is used, there is no systemd services managing the containers | 15:18 |
| EmilienM | that's why.. | 15:18 |
| lbragstad | ahhh... | 15:18 |
| *** rh-jelabarre has joined #tripleo | 15:21 | |
| *** slaweq has joined #tripleo | 15:22 | |
| lbragstad | EmilienM is the reason you're stopping the healthcheck and nova-compute service is so that you can delete it later (at line 1160)? | 15:24 |
| lbragstad | https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/nova/nova-compute-container-puppet.yaml#L1160 | 15:24 |
| lbragstad | can you not delete the nova compute service if it's running? | 15:24 |
| EmilienM | lbragstad: it's not clean | 15:24 |
| EmilienM | yes it's the reason | 15:24 |
| lbragstad | ok - cool | 15:24 |
| EmilienM | we followed reqs from nova folks | 15:24 |
| EmilienM | in any case we don't support docker anymore so | 15:25 |
| lbragstad | so - the compute container must be down before disabling the service | 15:25 |
| EmilienM | it's better, yes | 15:25 |
| EmilienM | a proper teardown | 15:25 |
| lbragstad | that makes sense | 15:25 |
| EmilienM | https://docs.openstack.org/operations-guide/ops-maintenance-compute.html#planned-maintenance | 15:27 |
| EmilienM | lbragstad: we follow that doc ^ | 15:27 |
| EmilienM | we disable the service, we stop the containers and delete the service | 15:27 |
| lbragstad | ++ | 15:28 |
| lbragstad | ok - cool | 15:28 |
| lbragstad | i see your point in using option #1 | 15:28 |
| lbragstad | we just shouldn't expect to use HA with docker on train | 15:28 |
| EmilienM | unless you don't care about scale down :D | 15:28 |
| EmilienM | but I'll fix it | 15:28 |
| EmilienM | so nobody else hits this | 15:29 |
| lbragstad | well - we care about scale down and wanted to test it without any of our patches to see if it was something we were doing | 15:29 |
| lbragstad | we do notice a different failure on scale down when we implement tls | 15:29 |
| lbragstad | our tls scale down executes successfully, but the very first nova scale down task fails | 15:30 |
| openstackgerrit | Dave Wilde (d34dh0r53) proposed openstack/tripleo-heat-templates stable/train: Added scale_tasks to handle cleanup on scale down of nodes https://review.opendev.org/718731 | 15:31 |
| *** ykarel is now known as ykarel|away | 15:33 | |
| lbragstad | speaking of - ^ that's the patch we were testing when we hit the nova scale down issue | 15:35 |
| openstackgerrit | Dave Wilde (d34dh0r53) proposed openstack/tripleo-heat-templates stable/train: Added scale_tasks to handle cleanup on scale down of nodes https://review.opendev.org/718731 | 15:35 |
| openstackgerrit | Dave Wilde (d34dh0r53) proposed openstack/tripleo-heat-templates stable/train: Added scale_tasks to handle cleanup on scale down of nodes https://review.opendev.org/718731 | 15:37 |
| owalsh | bandini, dciabrin: hey looking at https://bugs.launchpad.net/tripleo/+bug/1871795 ... | 15:43 |
| openstack | Launchpad bug 1871795 in tripleo "tripleo-upgrade playbook fails in task "store roles and register controller role name"" [High,New] | 15:43 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove manifests for Trove https://review.opendev.org/718097 | 15:43 |
| owalsh | bandini, dciabrin: doesn't look lke https://review.opendev.org/715491 is the culpit, inventory is empty so filters will do nothing | 15:45 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-docs master: node_config: document ExtraGroupVars https://review.opendev.org/718741 | 15:46 |
| owalsh | bandini, dciabrin: could be related to https://review.opendev.org/713919 but nothing obviously broken | 15:47 |
| bandini | owalsh: yeah not sure, I did not see it on my reviews, damien mentioned it to me before disappearing into PTO today, so I thought I'd bring it up | 15:47 |
| owalsh | bandini: happen to know what is generating /home/zuul/tripleo-ansible-inventory.yaml | 15:48 |
| *** maciejjozefczyk has quit IRC | 15:49 | |
| *** tkajinam has quit IRC | 15:49 | |
| bandini | owalsh: it used to be tripleo-ansible-inventory in tripleo-common repo, but I see now that it is part of the tripleo-validation rpm | 15:50 |
| owalsh | bandini: yea, I've landed patches for it too e.g https://review.opendev.org/713919. I mean where that's being run in CI, try to figure out how it resulted in an empty inventory | 15:51 |
| *** lucasagomes has quit IRC | 15:53 | |
| bandini | owalsh: ops sorry :) not sure where it is being run, sorry | 15:53 |
| owalsh | bandini: don't see how I could have broken it... 404 from heat might cause this though | 15:56 |
| bandini | you reckon it was a temporary hiccup thing? | 15:57 |
| bandini | or another review maybe? | 15:57 |
| *** rpittau is now known as rpittau|afk | 15:59 | |
| owalsh | bandini: recheck to see? | 16:00 |
| bandini | owalsh: sure thing let me try | 16:01 |
| EmilienM | lbragstad: I don't think we should spend time on docker support for scale down tasks in train | 16:03 |
| EmilienM | imo | 16:03 |
| *** iurygregory has quit IRC | 16:03 | |
| lbragstad | EmilienM yeah - that's fine | 16:03 |
| lbragstad | EmilienM i'll defer to what y'all want to do with it :) | 16:04 |
| *** marios is now known as marios|out | 16:05 | |
| *** dpawlik has quit IRC | 16:06 | |
| *** Trevor_V has joined #tripleo | 16:16 | |
| *** marios|out has quit IRC | 16:18 | |
| *** TrevorV has quit IRC | 16:19 | |
| openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient stable/train: Fix ResourceWarning: unclosed file https://review.opendev.org/718744 | 16:20 |
| openstackgerrit | amolkahat proposed openstack/tripleo-quickstart master: Ported featureset060.yml to os_tempest. https://review.opendev.org/718091 | 16:22 |
| owalsh | bandini: FWIW looks ok in other jobs e.g https://b2319ea300d930171925-35e07a4cf42cfaed2fcffa4bf0b16f1b.ssl.cf5.rackcdn.com/692421/20/check/tripleo-ci-centos-8-scenario000-multinode-oooq-container-updates/94dc35c/logs/undercloud/home/zuul/tripleo-ansible-inventory.yaml | 16:23 |
| bandini | owalsh: ack got you, let's see what the recheck does tonight | 16:25 |
| *** jpich has quit IRC | 16:28 | |
| *** cloudnull has quit IRC | 16:31 | |
| weshay|ruck | need eyes on https://bugs.launchpad.net/tripleo/+bug/1871338 | 16:32 |
| openstack | Launchpad bug 1871338 in tripleo "overcloud deployment failing with msg: 'argument parameters is of type <class ''str''> and we were unable to convert to dict: unable to evaluate string as dictionary'." [Critical,In progress] - Assigned to Sagi (Sergey) Shnaidman (sshnaidm) | 16:32 |
| *** weshay|ruck changes topic to "CI status: Upstream GREEN, 3rd Party RED lp#1871338 | community irc meeting Tues@1400 UTC - tripleo-ci-community meeting Tues@1330 UTC | https://docs.openstack.org/tripleo-docs" | 16:33 | |
| weshay|ruck | thanks for ops!!! | 16:33 |
| *** cloudnull has joined #tripleo | 16:34 | |
| openstackgerrit | Merged openstack/tripleo-docs master: node_config: document ExtraGroupVars https://review.opendev.org/718741 | 16:34 |
| *** bandini has quit IRC | 16:34 | |
| weshay|ruck | ade_lee, we're going to add rpm -V verification to the image builds.. to try and ensure this does not happen again.. if it is feasible of course | 16:35 |
| *** udesale_ has quit IRC | 16:35 | |
| *** cloudnull has quit IRC | 16:35 | |
| ade_lee | weshay|ruck, cool | 16:36 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Move the haproxy iptables rules creation to host_prep_tasks https://review.opendev.org/718159 | 16:36 |
| weshay|ruck | mwhahaha, you have a sec to look at a promotion blocker? https://bugs.launchpad.net/tripleo/+bug/1871338 | 16:36 |
| openstack | Launchpad bug 1871338 in tripleo "overcloud deployment failing with msg: 'argument parameters is of type <class ''str''> and we were unable to convert to dict: unable to evaluate string as dictionary'." [Critical,In progress] - Assigned to Sagi (Sergey) Shnaidman (sshnaidm) | 16:36 |
| weshay|ruck | sagi is gone to break unleavened bread | 16:36 |
| *** cloudnull has joined #tripleo | 16:36 | |
| *** evrardjp has quit IRC | 16:37 | |
| mwhahaha | isn't ekultails and cloudnull looking at the one? | 16:37 |
| mwhahaha | i can if they aren't | 16:37 |
| *** evrardjp has joined #tripleo | 16:37 | |
| * weshay|ruck asks now that cloudnull is back in his lounge | 16:37 | |
| * cloudnull missed a bunch of messages | 16:37 | |
| weshay|ruck | thanks mwhahaha .. | 16:37 |
| *** jcapitao has quit IRC | 16:37 | |
| cloudnull | my ISP is having a bad day | 16:37 |
| weshay|ruck | cloudnull, I was going to just give you $1400 of my bonus from gov cheese.. | 16:37 |
| weshay|ruck | but you weren't here | 16:38 |
| cloudnull | hahaha | 16:38 |
| cloudnull | mwhahaha what is ekultails looking at? | 16:38 |
| weshay|ruck | so instead of free $$ and cheese.. https://bugs.launchpad.net/tripleo/+bug/1871338 | 16:38 |
| openstack | Launchpad bug 1871338 in tripleo "overcloud deployment failing with msg: 'argument parameters is of type <class ''str''> and we were unable to convert to dict: unable to evaluate string as dictionary'." [Critical,In progress] - Assigned to Sagi (Sergey) Shnaidman (sshnaidm) | 16:38 |
| mwhahaha | -^ | 16:38 |
| weshay|ruck | any clue on dat one? | 16:38 |
| cloudnull | oh , sshnaidm ramishra and I were looking at that | 16:38 |
| weshay|ruck | cloudnull, sagi abandoned his change | 16:38 |
| cloudnull | there is a change, let me go find that | 16:39 |
| cloudnull | https://review.opendev.org/#/c/717953/ | 16:39 |
| cloudnull | looks like it needs WIP removed. | 16:39 |
| cloudnull | that should resolve the issue | 16:40 |
| ekultails | That's not something I've been working on. Would be happy to lend a hand if needed. | 16:40 |
| ekultails | It looks like it Depends-On a Do-Not-Merge patch. | 16:40 |
| * cloudnull goes to tune up that review | 16:40 | |
| ekultails | The DNM patch is no-op so I guess yeah we can remove the "WIP" and "Depends-On" from the commit message and it should be fine. | 16:41 |
| mwhahaha | seems an odd solution | 16:42 |
| * mwhahaha shrugs | 16:42 | |
| ekultails | Indeed | 16:42 |
| cloudnull | mwhahaha i think we can sanatize the input, instead of trying to deal with the issue in the playbook | 16:43 |
| cloudnull | **sanitize | 16:43 |
| cloudnull | but that's the problem, stupid escaped newlines | 16:44 |
| mwhahaha | indeed | 16:45 |
| mwhahaha | is it because we're pretty printing some json stuff | 16:45 |
| *** ysandeep|afk is now known as ysandeep|out | 16:46 | |
| *** dtantsur is now known as dtantsur|afk | 16:46 | |
| cloudnull | i'm not really sure... I think it comes out of heat? | 16:47 |
| cloudnull | https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L302 | 16:47 |
| mwhahaha | no | 16:47 |
| mwhahaha | probably from files | 16:47 |
| mwhahaha | not certain tho | 16:48 |
| mwhahaha | it's passed in via env | 16:48 |
| mwhahaha | and isn't this the thing that updates the plan for heat? | 16:48 |
| mwhahaha | so this is before heat runs | 16:48 |
| cloudnull | its an env, but the key is parameter_defaults and in some cases it seems to be a valid dictionary, but it others its a dictionary like string | 16:49 |
| mwhahaha | https://github.com/openstack/python-tripleoclient/blob/a54ea2d01a947c77bc6b9fabb474bab90a831598/tripleoclient/v1/overcloud_deploy.py#L460 | 16:49 |
| cloudnull | so you're probably right in that its coming from files | 16:50 |
| mwhahaha | comes from my favorit function process_multiple_environments | 16:50 |
| * cloudnull clicks the "do you want to know more" link | 16:50 | |
| mwhahaha | comes from heatclient.common | 16:51 |
| mwhahaha | actually | 16:51 |
| mwhahaha | files, env = template_utils.process_environment_and_files( | 16:52 |
| mwhahaha | https://github.com/openstack/python-heatclient/blob/master/heatclient/common/template_utils.py#L287 | 16:53 |
| mwhahaha | https://github.com/openstack/python-heatclient/blob/9dede4eb2b824b21c0031507bd986796f535810c/heatclient/common/template_format.py#L53 | 16:53 |
| mwhahaha | it's the yaml loader | 16:53 |
| mwhahaha | i think | 16:53 |
| mwhahaha | anyway | 16:53 |
| mwhahaha | sanitizing it in tripleoclient would probably be a good idea | 16:53 |
| cloudnull | ++ | 16:54 |
| mwhahaha | is it yaml with json in it? | 16:54 |
| mwhahaha | is that the problem? | 16:54 |
| cloudnull | it seems private keys are in that parameters defaults object. | 16:54 |
| mwhahaha | fun | 16:55 |
| cloudnull | which seem to have extra newlines | 16:55 |
| *** alexmcleod has quit IRC | 16:55 | |
| *** pojadhav|away is now known as pojadhav | 16:56 | |
| *** hjensas has quit IRC | 17:00 | |
| *** derekh has quit IRC | 17:01 | |
| *** sshnaidm is now known as sshnaidm|afk | 17:12 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [Q->T] Introduce Queens to Train Upgrade https://review.opendev.org/713482 | 17:18 |
| openstackgerrit | Grzegorz Grasza proposed openstack/tripleo-heat-templates stable/train: Skip both tenant and management networks when generating certs https://review.opendev.org/718756 | 17:19 |
| *** pkopec has quit IRC | 17:20 | |
| openstackgerrit | Grzegorz Grasza proposed openstack/tripleo-heat-templates stable/stein: Skip both tenant and management networks when generating certs https://review.opendev.org/718757 | 17:20 |
| openstackgerrit | Merged openstack/tripleo-ansible master: Hide passwords better https://review.opendev.org/718096 | 17:20 |
| *** panda is now known as panda|off | 17:27 | |
| *** hewbrocca is now known as hewbrocca_afk | 17:28 | |
| *** rascasoft has quit IRC | 17:33 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Add the ability to load vars from files https://review.opendev.org/718759 | 17:35 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Add the ability to load vars from files https://review.opendev.org/718759 | 17:36 |
| cloudnull | ramishra sshnaidm|afk ^ | 17:36 |
| cloudnull | -cc weshay|ruck | 17:36 |
| weshay|ruck | danka | 17:37 |
| cloudnull | we'll need to make sure fs001 is happy, but that should-tm take care of it | 17:38 |
| *** sshnaidm|afk is now known as sshnaidm|off | 18:02 | |
| *** suuuper has quit IRC | 18:10 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage: optimize systemd services/healthcheck bootstrap https://review.opendev.org/718545 | 18:20 |
| *** rfolco is now known as rfolco|bbl | 18:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart-extras master: Revert "Enable ConfigDebug for undercloud deployments" https://review.opendev.org/718766 | 18:25 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-quickstart-extras master: Revert "Enable ConfigDebug for undercloud deployments" https://review.opendev.org/718766 | 18:25 |
| EmilienM | cloudnull: hey, with mwhahaha we're looking at why we get so much logs into overcloud deploys in CI | 18:26 |
| EmilienM | I suspect the --verbose increases Ansible verbosity | 18:26 |
| EmilienM | normally we would get logs if tripleo_container_manage_debug is set to True | 18:27 |
| EmilienM | I'm trying to look where we set it to true in our CI configs | 18:27 |
| EmilienM | oh I found it | 18:28 |
| EmilienM | DockerPuppetDebug is set to true | 18:28 |
| EmilienM | cloudnull: unping | 18:28 |
| * cloudnull unpining | 18:29 | |
| * EmilienM blames mwhahaha | 18:29 | |
| EmilienM | https://review.opendev.org/#/c/531015/ | 18:29 |
| EmilienM | mwhahaha: should we partial revert it? ^ | 18:29 |
| openstackgerrit | Pooja Jadhav proposed openstack/tripleo-quickstart-extras master: DNM: Testing ipa multinode https://review.opendev.org/706373 | 18:31 |
| EmilienM | ekultails: btw on https://review.opendev.org/#/c/718545 - I replied. I couldn't find a good name, although I don't think it'll be easy to re-use that filter | 18:31 |
| mwhahaha | 2 years ago? | 18:32 |
| mwhahaha | but we don't see the puppet debug bits in the older versions | 18:33 |
| mwhahaha | or maybe it was partially eaten by the old scripts | 18:41 |
| mwhahaha | who knows | 18:41 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage: optimize systemd services/healthcheck bootstrap https://review.opendev.org/718545 | 18:45 |
| *** pojadhav is now known as pojadhav|off | 18:45 | |
| *** mgoddard has joined #tripleo | 18:46 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_bootstrap: do not fail if OVS rpm wasn't found https://review.opendev.org/717876 | 18:52 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_bootstrap: do not fail if OVS rpm wasn't found https://review.opendev.org/717876 | 18:52 |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-heat-templates master: WIP: refactor nova db config https://review.opendev.org/718552 | 18:56 |
| openstackgerrit | Merged openstack/tripleo-ansible stable/train: tripleo_all_nodes_data: sort enabled_services (idempotency) https://review.opendev.org/718680 | 18:58 |
| ekultails | EmilienM: A lot can change in a few minutes. ;-) | 19:04 |
| ekultails | +1 | 19:04 |
| openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates stable/train: Move the haproxy iptables rules creation to host_prep_tasks https://review.opendev.org/718201 | 19:07 |
| *** spsurya_ has quit IRC | 19:08 | |
| owalsh | fultonj: hi, just spotted something with the multistack inventory... | 19:11 |
| owalsh | fultonj: since https://review.opendev.org/713919 the inventory is alway a multistack inventory, even if it's a single stack... | 19:12 |
| weshay|ruck | ade_lee, it's working now :) stack@undercloud ~]$ sudo ls /var/lib/certmonger/local/creds | 19:13 |
| weshay|ruck | ade_lee, another packaging snafu | 19:13 |
| owalsh | fultonj: different structure but ultimately the same inventory, but just noticed this https://review.opendev.org/#/c/713750/9/tripleo_common/inventories.py@106 | 19:14 |
| ade_lee | weshay|ruck, sweet | 19:14 |
| ade_lee | mwhahaha, cloudnull https://review.opendev.org/#/c/718731/ please | 19:16 |
| owalsh | fultonj: I guess I should keep it in train at least | 19:16 |
| fultonj | owalsh: i think i'd need an example to be sure i understand what you're saying | 19:19 |
| fultonj | owalsh: i understand "inventory is alway a multistack inventory even if it's a single stack" | 19:20 |
| fultonj | owalsh: but if i had a diff of before/after then i'd understand more exactly what you mean | 19:20 |
| owalsh | fultonj: right, before with single stack the undercloud had a 'plan' var, now it has a 'plans' var | 19:21 |
| owalsh | well, it still has a plan var, but it's == '' | 19:22 |
| fultonj | https://review.opendev.org/#/c/713919/2/scripts/tripleo-ansible-inventory | 19:23 |
| fultonj | owalsh: when multistack inventory was introduced, it did exactly what was done in the past (triggered the same code) unless >1 stack was passed | 19:24 |
| fultonj | after this ^ review the new TripleInventories object is always used | 19:25 |
| owalsh | fultonj: yea, that's intentional | 19:25 |
| fultonj | sure | 19:26 |
| owalsh | and the new inventory is a superset of the old, except for this one var | 19:26 |
| fultonj | you made "inventories" handle the 1 stack scenario but perhaps these things you're noticing were overlooked | 19:26 |
| fultonj | i say this only because it sounds like you were not expecting the change | 19:27 |
| owalsh | fultonj: just missed that line | 19:27 |
| fultonj | except for this one var | 19:27 |
| fultonj | plan vs plans | 19:27 |
| fultonj | so a 3rd patch ? | 19:27 |
| fultonj | just to handle the plan vs plans? | 19:28 |
| fultonj | extgra log around line 106 | 19:28 |
| fultonj | "extra logic" around line 106 | 19:29 |
| fultonj | then again, but why merge one inventory | 19:29 |
| owalsh | much better to have a single code path | 19:31 |
| owalsh | or they will diverge/rot | 19:31 |
| fultonj | owalsh: ok, i'll accept that | 19:32 |
| owalsh | I'll push a fix just in case anybody/thing uses that | 19:34 |
| fultonj | owalsh: ok, thanks. please add me to the review | 19:35 |
| openstackgerrit | Alex Schultz proposed openstack/python-tripleoclient master: Use undercloud upgrade to update all required pkgs https://review.opendev.org/718784 | 19:48 |
| mwhahaha | chem, odyssey4me -^ fyi | 19:49 |
| *** gfidente is now known as gfidente|afk | 19:53 | |
| *** rfolco|bbl is now known as rfolco | 20:15 | |
| *** cylopez has quit IRC | 20:15 | |
| openstackgerrit | Oliver Walsh proposed openstack/tripleo-common master: Include uc plan var when TripleoInventories contains a single plan https://review.opendev.org/718789 | 20:21 |
| *** raildo_ has joined #tripleo | 20:24 | |
| *** raildo has quit IRC | 20:26 | |
| *** jtomasek has quit IRC | 20:41 | |
| *** owalsh is now known as owalsh_afk | 20:43 | |
| weshay|ruck | cloudnull++ https://review.opendev.org/#/c/718759/ | 20:53 |
| weshay|ruck | thank you!! it passed | 20:54 |
| weshay|ruck | mwhahaha, can we get a review ^ | 20:54 |
| weshay|ruck | passes here https://logserver.rdoproject.org/59/718759/2/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001/805bbf2/ | 20:54 |
| cloudnull | awesome! | 20:54 |
| mwhahaha | k | 20:56 |
| weshay|ruck | check is passing too | 20:59 |
| weshay|ruck | will hit the gate in a few | 20:59 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage: optimize systemd services/healthcheck bootstrap https://review.opendev.org/718545 | 21:03 |
| *** Trevor_V has quit IRC | 21:05 | |
| weshay|ruck | check it out ya'll.. our new cloud is slower than our old one :) | 21:09 |
| weshay|ruck | tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001SUCCESS in 3h 10m 36s | 21:09 |
| weshay|ruck | tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-vexxhostSUCCESS in 3h 30m 02s | 21:09 |
| weshay|ruck | weeeee | 21:09 |
| mwhahaha | so the usual | 21:09 |
| mwhahaha | got it | 21:09 |
| EmilienM | weshay|ruck: is it using local mirror and stuff? | 21:10 |
| weshay|ruck | EmilienM, ya | 21:12 |
| weshay|ruck | EmilienM, only diff is there is a monthly charge | 21:12 |
| EmilienM | lol | 21:12 |
| *** fultonj has quit IRC | 21:12 | |
| *** fultonj has joined #tripleo | 21:14 | |
| *** fultonj has quit IRC | 21:16 | |
| *** fultonj has joined #tripleo | 21:17 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: molecule/podman_*: enable check mode testing https://review.opendev.org/718811 | 21:17 |
| *** fultonj has quit IRC | 21:18 | |
| EmilienM | sshnaidm|off: ^ FYI I'm going to push this for all molecule testing, to cover the dry runs aka check mode in our CI | 21:20 |
| EmilienM | it very often breaks in TripleO and these tests will get us coverage in tripleo-ansible. | 21:20 |
| EmilienM | we'll miss coverage in tasks defined in THT but it's another story ... | 21:20 |
| sshnaidm|off | EmilienM, ack | 21:20 |
| weshay|ruck | sshnaidm|off, CHAG | 21:21 |
| weshay|ruck | sshnaidm|off, CHAG | 21:21 |
| weshay|ruck | sshnaidm|off, CHAG | 21:21 |
| * sshnaidm|off off | 21:21 | |
| EmilienM | a likely story | 21:21 |
| weshay|ruck | gotta go deal w/ kids.. bbl | 21:23 |
| *** hberaud has quit IRC | 21:25 | |
| *** hberaud has joined #tripleo | 21:26 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: standalone/overcloud: enable the HA deployment by default https://review.opendev.org/359060 | 21:26 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: standalone/overcloud: enable the HA deployment by default https://review.opendev.org/359060 | 21:27 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Fix incorrect spec file name https://review.opendev.org/718575 | 21:36 |
| openstackgerrit | Merged openstack/puppet-tripleo master: Add missing unit tests for placement manifests https://review.opendev.org/718574 | 21:37 |
| openstackgerrit | Slawek Kaplonski proposed openstack/tripleo-heat-templates master: Add support for resource provider bandwidth in Neutron config https://review.opendev.org/714464 | 21:45 |
| *** cloudnull has quit IRC | 21:51 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/tripleo-heat-templates master: Add support for resource provider bandwidth in Neutron config https://review.opendev.org/714464 | 21:53 |
| *** ramishra has quit IRC | 22:03 | |
| *** slaweq has quit IRC | 22:14 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [Q->T] Introduce Queens to Train Upgrade https://review.opendev.org/713482 | 22:20 |
| *** raildo_ has quit IRC | 22:21 | |
| rajinir | https://review.opendev.org/#/c/712533/9 - the rdo job keeps failing on this review. The error is not related , any idea? | 22:27 |
| rajinir | another one - same error - https://review.opendev.org/#/c/716638/ | 22:29 |
| *** cloudnull has joined #tripleo | 22:35 | |
| *** jmasud has quit IRC | 22:40 | |
| openstackgerrit | Merged openstack/tripleo-repos master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/717405 | 22:42 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Add placement to testing matric https://review.opendev.org/718108 | 22:42 |
| *** jmasud has joined #tripleo | 22:43 | |
| *** tkajinam has joined #tripleo | 22:53 | |
| *** hjensas has joined #tripleo | 23:00 | |
| mwhahaha | it's not related | 23:03 |
| *** tosky has quit IRC | 23:08 | |
| openstackgerrit | Merged openstack/tripleo-ansible stable/train: 'tripleo_ovs_dpdk_lcore_list' should be optional https://review.opendev.org/717303 | 23:32 |
| *** rfolco has quit IRC | 23:37 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!