| *** cdearborn has quit IRC | 00:10 | |
| cloudnull | EmilienM so i'm really not sure why the build process is not failing, maybe there's some try/except block in an underlying caller? | 00:18 |
|---|---|---|
| cloudnull | when testing with oslo the lib is behaving correctly | 00:19 |
| cloudnull | https://pasted.tech/pastes/f4847e3b421455fbc6acc3804e6bfcb2691a5a1e | 00:19 |
| cloudnull | even when testing with a valid Dockerfile, which has a `RUN false` in it, it builds then fails correctly - https://pasted.tech/pastes/c858e1f48f0e9ec56d814847da859f752394cc20 | 00:19 |
| cloudnull | in both cases the exception ProcessExecutionError is raised via oslo | 00:20 |
| cloudnull | when using the client a proper exception should be raised here https://github.com/openstack/python-tripleoclient/blob/c6d441b24c2f367a6d95772a7d83e4f6fcd8caa0/tripleoclient/v1/container_image.py#L200-L232 | 00:24 |
| cloudnull | the only thing i can think is that - https://github.com/openstack/tripleo-common/blob/master/tripleo_common/image/builder/buildah.py#L164-L169 which is threading the build command is masking the return codes | 00:25 |
| *** betherly has joined #tripleo | 00:26 | |
| *** betherly has quit IRC | 00:30 | |
| *** dmsimard4 is now known as dmsimard | 00:32 | |
| *** rcernin has quit IRC | 00:34 | |
| *** rcernin has joined #tripleo | 00:35 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/tripleo-common master: Ensure we're raising proper exceptions https://review.opendev.org/670408 | 00:38 |
| cloudnull | mwhahaha weshay|rover EmilienM - https://review.opendev.org/#/c/670408 ok that should do it | 00:39 |
| cloudnull | that will ensure our build process raises an exception when there's an erro r | 00:39 |
| cloudnull | and with that, im going to go uncloud for a while. talk to you all soon | 00:40 |
| *** betherly has joined #tripleo | 00:46 | |
| *** panda has quit IRC | 00:48 | |
| *** panda has joined #tripleo | 00:50 | |
| *** betherly has quit IRC | 00:51 | |
| openstackgerrit | Merged openstack/puppet-tripleo stable/rocky: Support TLS deployments with KernelDisableIPv6 enabled https://review.opendev.org/669901 | 00:55 |
| *** irclogbot_2 has joined #tripleo | 00:56 | |
| *** irclogbot_2 has quit IRC | 01:00 | |
| *** betherly has joined #tripleo | 01:06 | |
| *** betherly has quit IRC | 01:11 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-tripleo master: Enable service token in nova and cinder https://review.opendev.org/670424 | 01:17 |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Enable service token in nova and cinder https://review.opendev.org/666744 | 01:18 |
| *** irclogbot_3 has joined #tripleo | 01:26 | |
| *** betherly has joined #tripleo | 01:27 | |
| *** betherly has quit IRC | 01:32 | |
| *** irclogbot_3 has quit IRC | 01:34 | |
| openstackgerrit | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Enable service token in nova and cinder https://review.opendev.org/666744 | 01:38 |
| *** dsneddon has quit IRC | 01:41 | |
| *** dsneddon has joined #tripleo | 01:41 | |
| *** dsneddon has quit IRC | 01:46 | |
| *** betherly has joined #tripleo | 01:48 | |
| *** mschuppert has quit IRC | 01:50 | |
| *** betherly has quit IRC | 01:53 | |
| *** apetrich has quit IRC | 01:58 | |
| *** dsneddon has joined #tripleo | 02:07 | |
| *** brault has joined #tripleo | 02:18 | |
| openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Make instances name/hostname optional https://review.opendev.org/669019 | 02:19 |
| openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Changes to baremetal provision format https://review.opendev.org/669774 | 02:19 |
| openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Break out hostname format logic into functions https://review.opendev.org/670432 | 02:19 |
| *** brault has quit IRC | 02:22 | |
| *** irclogbot_2 has joined #tripleo | 02:26 | |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/tripleo-common master: Ensure we're raising proper exceptions https://review.opendev.org/670408 | 02:28 |
| *** irclogbot_2 has quit IRC | 02:30 | |
| *** betherly has joined #tripleo | 02:39 | |
| openstackgerrit | Merged openstack/tripleo-common master: Remove the tripleo.plan_management.v1.download_logs workflow https://review.opendev.org/669161 | 02:41 |
| openstackgerrit | Merged openstack/tripleo-common master: Remove the tripleo.plan_management.v1.publish_ui_logs_to_swiftworkflow https://review.opendev.org/669162 | 02:41 |
| *** betherly has quit IRC | 02:44 | |
| *** vinaykns has quit IRC | 02:46 | |
| *** altlogbot_3 has joined #tripleo | 02:48 | |
| *** altlogbot_3 has quit IRC | 02:52 | |
| *** betherly has joined #tripleo | 03:00 | |
| *** betherly has quit IRC | 03:05 | |
| *** rlandy has quit IRC | 03:16 | |
| *** betherly has joined #tripleo | 03:20 | |
| *** irclogbot_3 has joined #tripleo | 03:22 | |
| *** dsneddon has quit IRC | 03:24 | |
| *** betherly has quit IRC | 03:25 | |
| *** irclogbot_3 has quit IRC | 03:26 | |
| *** betherly has joined #tripleo | 03:42 | |
| *** psachin has joined #tripleo | 03:45 | |
| *** betherly has quit IRC | 03:46 | |
| *** dsneddon has joined #tripleo | 03:47 | |
| *** irclogbot_0 has joined #tripleo | 03:52 | |
| EmilienM | cloudnull: nicely done | 03:53 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove service_names hieradata file https://review.opendev.org/669424 | 03:55 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: deploy-steps: remove unused hieradata_templates_list https://review.opendev.org/669917 | 03:55 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove config_step hieradata https://review.opendev.org/669924 | 03:55 |
| *** irclogbot_0 has quit IRC | 03:56 | |
| *** betherly has joined #tripleo | 04:02 | |
| *** ykarel|away has joined #tripleo | 04:02 | |
| *** dsneddon has quit IRC | 04:03 | |
| *** udesale has joined #tripleo | 04:05 | |
| *** betherly has quit IRC | 04:07 | |
| openstackgerrit | yatin proposed openstack/tripleo-common master: DNM: test tripleo-ansible https://review.opendev.org/670276 | 04:07 |
| openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Make instances name/hostname optional https://review.opendev.org/669019 | 04:10 |
| openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Changes to baremetal provision format https://review.opendev.org/669774 | 04:10 |
| openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Break out hostname format logic into functions https://review.opendev.org/670432 | 04:10 |
| openstackgerrit | Steve Baker proposed openstack/tripleo-common master: Allow mixing of count and instances https://review.opendev.org/670437 | 04:10 |
| *** betherly has joined #tripleo | 04:33 | |
| *** dsneddon has joined #tripleo | 04:34 | |
| *** betherly has quit IRC | 04:38 | |
| *** dsneddon has quit IRC | 04:39 | |
| *** ratailor has joined #tripleo | 04:52 | |
| *** poopcat has quit IRC | 04:52 | |
| *** zbitter has quit IRC | 04:53 | |
| *** zbitter has joined #tripleo | 04:53 | |
| *** rcernin has quit IRC | 04:54 | |
| *** ykarel|away has quit IRC | 04:59 | |
| *** irclogbot_1 has joined #tripleo | 05:00 | |
| *** dsneddon has joined #tripleo | 05:03 | |
| *** pdeore has joined #tripleo | 05:04 | |
| *** dsneddon has quit IRC | 05:07 | |
| *** pcaruana has joined #tripleo | 05:13 | |
| *** betherly has joined #tripleo | 05:14 | |
| *** jistr has quit IRC | 05:15 | |
| *** irclogbot_1 has quit IRC | 05:16 | |
| *** jistr has joined #tripleo | 05:18 | |
| *** betherly has quit IRC | 05:19 | |
| *** ykarel|away has joined #tripleo | 05:24 | |
| *** ykarel|away is now known as ykarel | 05:25 | |
| *** holser_ has joined #tripleo | 05:32 | |
| *** ratailor_ has joined #tripleo | 05:33 | |
| *** ratailor has quit IRC | 05:35 | |
| Tengu | hello there | 05:36 |
| *** holser_ has quit IRC | 05:36 | |
| *** pdeore has quit IRC | 05:39 | |
| *** dsneddon has joined #tripleo | 05:40 | |
| ykarel | o/ | 05:40 |
| * Tengu reading the latest CIX for rocky promotion | 05:41 | |
| *** yprokule has joined #tripleo | 05:43 | |
| *** dsneddon has quit IRC | 05:45 | |
| *** mburned is now known as mburned_out | 05:45 | |
| *** aedc has quit IRC | 05:47 | |
| *** mburned_out is now known as mburned | 05:48 | |
| *** ratailor_ has quit IRC | 05:48 | |
| Tengu | hmmm.... that's not very nice: http://logs.openstack.org/54/670254/2/check/tripleo-ci-centos-7-scenario010-standalone/2c44a77/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz#_2019-07-11_19_33_19 | 05:49 |
| Tengu | transcient, or.. ? | 05:49 |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Move tripleo-ci-centos-7-containers-multinode to os_tempest https://review.opendev.org/667390 | 05:49 |
| *** zaneb has joined #tripleo | 05:51 | |
| *** ratailor has joined #tripleo | 05:51 | |
| ykarel | with that error it doesn't look transcient | 05:52 |
| *** zbitter has quit IRC | 05:53 | |
| Tengu | ykarel: :/ | 05:53 |
| Tengu | guess we're good for some new CIX, for Network/Octavia then | 05:53 |
| Tengu | lemme create an LP. | 05:53 |
| *** jbadiapa has quit IRC | 05:55 | |
| Tengu | ykarel: would that be a promotion blocker? | 05:56 |
| ykarel | Tengu, i can see good amount of SUCCESS http://zuul.openstack.org/builds?job_name=tripleo-ci-centos-7-scenario010-standalone recently | 05:56 |
| ykarel | seems it's fixed | 05:57 |
| ykarel | i see two failures though | 05:57 |
| Tengu | let's do a recheck | 05:57 |
| Tengu | ykarel: 9h ago for the failing job. maybe it was fixed in the meanwhile. | 05:57 |
| ykarel | Tengu, yes | 05:57 |
| Tengu | I keep the LP open, ready to submit. | 05:57 |
| Tengu | #coffee | 05:58 |
| *** marios has joined #tripleo | 05:59 | |
| ykarel | Tengu, so https://review.opendev.org/#/c/667247/ and https://review.opendev.org/#/c/665256/ | 06:00 |
| ykarel | ^^ both changes needed together to work, so possibly failing job had only one of them | 06:00 |
| *** skramaja has joined #tripleo | 06:00 | |
| openstackgerrit | Rajesh Tailor proposed openstack/tripleo-puppet-elements stable/queens: Stop and disable libvirtd service on controller https://review.opendev.org/670448 | 06:05 |
| Tengu | ykarel: ok! let's see with the recheck | 06:05 |
| ykarel | yes recheck will work | 06:06 |
| *** betherly has joined #tripleo | 06:06 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/rocky: Rocky: enable container auth support https://review.opendev.org/670349 | 06:09 |
| *** altlogbot_2 has joined #tripleo | 06:10 | |
| *** betherly has quit IRC | 06:11 | |
| *** dsneddon has joined #tripleo | 06:12 | |
| *** altlogbot_2 has quit IRC | 06:14 | |
| *** dsneddon has quit IRC | 06:17 | |
| *** amoralej|off is now known as amoralej | 06:18 | |
| *** dsneddon has joined #tripleo | 06:22 | |
| *** jtomasek has joined #tripleo | 06:25 | |
| *** Goneri has joined #tripleo | 06:25 | |
| *** dsneddon has quit IRC | 06:27 | |
| *** mschuppert has joined #tripleo | 06:27 | |
| *** irclogbot_1 has joined #tripleo | 06:36 | |
| *** betherly has joined #tripleo | 06:37 | |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-heat-templates stable/rocky: Allow logrotate to access container_file_t files https://review.opendev.org/670451 | 06:37 |
| Tengu | damn... forget the -x -.- | 06:38 |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-heat-templates stable/rocky: Allow logrotate to access container_file_t files https://review.opendev.org/670451 | 06:38 |
| *** irclogbot_1 has quit IRC | 06:40 | |
| *** betherly has quit IRC | 06:42 | |
| *** saneax has joined #tripleo | 06:53 | |
| *** aedc has joined #tripleo | 06:58 | |
| *** yprokule has quit IRC | 07:02 | |
| *** yprokule has joined #tripleo | 07:02 | |
| *** holser_ has joined #tripleo | 07:02 | |
| *** dsneddon has joined #tripleo | 07:02 | |
| *** dsneddon has quit IRC | 07:07 | |
| *** holser_ has quit IRC | 07:12 | |
| *** holser_ has joined #tripleo | 07:13 | |
| *** jpich has joined #tripleo | 07:17 | |
| *** Goneri has quit IRC | 07:17 | |
| *** Goneri has joined #tripleo | 07:18 | |
| *** rpittau|afk is now known as rpittau | 07:18 | |
| openstackgerrit | mathieu bultel proposed openstack/python-tripleoclient master: Make mistral optional for Ansible Update and Upgrade https://review.opendev.org/670295 | 07:18 |
| *** tosky has joined #tripleo | 07:19 | |
| openstackgerrit | mathieu bultel proposed openstack/python-tripleoclient master: DNM CI test https://review.opendev.org/670314 | 07:21 |
| *** jbadiapa has joined #tripleo | 07:22 | |
| *** bogdando has joined #tripleo | 07:24 | |
| *** jbadiapa has quit IRC | 07:27 | |
| *** iurygregory has joined #tripleo | 07:30 | |
| *** dsneddon has joined #tripleo | 07:35 | |
| *** xek has joined #tripleo | 07:36 | |
| *** florianf has joined #tripleo | 07:37 | |
| *** dsneddon has quit IRC | 07:40 | |
| *** slaweq has joined #tripleo | 07:41 | |
| *** lucasagomes has joined #tripleo | 07:43 | |
| *** mcornea has joined #tripleo | 07:47 | |
| *** aedc has quit IRC | 07:48 | |
| *** shyamb has joined #tripleo | 07:49 | |
| *** dsneddon has joined #tripleo | 07:50 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-tripleo master: Enable service token in nova and cinder https://review.opendev.org/670424 | 07:51 |
| *** aedc has joined #tripleo | 07:54 | |
| *** dsneddon has quit IRC | 07:54 | |
| *** bhagyashris has joined #tripleo | 07:57 | |
| *** ccamacho has joined #tripleo | 07:59 | |
| *** ykarel is now known as ykarel|lunch | 07:59 | |
| *** shyamb has quit IRC | 07:59 | |
| *** altlogbot_2 has joined #tripleo | 08:03 | |
| *** altlogbot_2 has quit IRC | 08:04 | |
| *** yolanda has quit IRC | 08:08 | |
| *** yolanda has joined #tripleo | 08:09 | |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed openstack/tripleo-heat-templates master: Fix generating Apache configs by container-puppet https://review.opendev.org/669178 | 08:10 |
| Tengu | arxcruz|ruck: heya! can you tell me if https://bugs.launchpad.net/tripleo/+bug/1835718 is sorted out so that we can move the CIX card to "done" ? | 08:12 |
| openstack | Launchpad bug 1835718 in tripleo "Rocky promotion jobs fail on undercloud install" [Critical,Triaged] - Assigned to Alex Schultz (alex-schultz) | 08:12 |
| Tengu | the patch merged, so it should be fine | 08:12 |
| *** altlogbot_0 has joined #tripleo | 08:12 | |
| *** gkadam has joined #tripleo | 08:15 | |
| *** altlogbot_0 has quit IRC | 08:16 | |
| *** gkadam has quit IRC | 08:17 | |
| *** altlogbot_0 has joined #tripleo | 08:18 | |
| *** tkajinam has quit IRC | 08:19 | |
| *** altlogbot_0 has quit IRC | 08:22 | |
| *** dsneddon has joined #tripleo | 08:24 | |
| *** altlogbot_0 has joined #tripleo | 08:24 | |
| *** pkopec has joined #tripleo | 08:27 | |
| *** dsneddon has quit IRC | 08:28 | |
| *** altlogbot_0 has quit IRC | 08:29 | |
| *** derekh has joined #tripleo | 08:32 | |
| *** rascasoft has quit IRC | 08:33 | |
| *** rascasoft has joined #tripleo | 08:34 | |
| *** bogdando has quit IRC | 08:40 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-ci master: Rename tripleo-ci build-images to oooci-build-images for clash https://review.opendev.org/669888 | 08:43 |
| *** bogdando has joined #tripleo | 08:43 | |
| *** shyamb has joined #tripleo | 08:46 | |
| *** aedc has quit IRC | 08:48 | |
| *** ykarel|lunch is now known as ykarel | 08:48 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-ci master: Tidy up tripleo-standalone-scenarios-full template with anchors https://review.opendev.org/663018 | 08:51 |
| *** rascasoft has quit IRC | 08:55 | |
| *** rascasoft has joined #tripleo | 08:58 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-ci master: Add new scenario007-standalone job definition & add to check https://review.opendev.org/631497 | 08:58 |
| *** betherly has joined #tripleo | 09:00 | |
| *** irclogbot_1 has joined #tripleo | 09:00 | |
| *** dsneddon has joined #tripleo | 09:03 | |
| *** irclogbot_1 has quit IRC | 09:04 | |
| *** betherly has quit IRC | 09:05 | |
| *** dsneddon has quit IRC | 09:08 | |
| *** Goneri has quit IRC | 09:14 | |
| arxcruz|ruck | Tengu, i don't have the results yet, i'll update as soon as I get okay? :) | 09:15 |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates stable/stein: Clean docker and podman after executing an update or upgrade https://review.opendev.org/670478 | 09:16 |
| Tengu | arxcruz|ruck: ;_; ok... hoped to get rid of the spam-bot :/ | 09:16 |
| marios | reviews please https://review.opendev.org/#/c/669329/ & https://review.opendev.org/#/c/670290/ needed by https://review.rdoproject.org/r/#/c/21357/ rdo rhel 8 image build https://tree.taiga.io/project/tripleo-ci-board/task/1151 thank you! | 09:19 |
| *** Goneri has joined #tripleo | 09:27 | |
| *** Goneri has quit IRC | 09:39 | |
| *** ratailor_ has joined #tripleo | 09:40 | |
| *** dsneddon has joined #tripleo | 09:42 | |
| *** ratailor has quit IRC | 09:43 | |
| *** dsneddon has quit IRC | 09:47 | |
| *** dmacpher has quit IRC | 09:48 | |
| *** dmacpher has joined #tripleo | 09:48 | |
| *** dmacpher has quit IRC | 09:49 | |
| *** dmacpher has joined #tripleo | 09:49 | |
| openstackgerrit | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service. https://review.opendev.org/666578 | 09:58 |
| ykarel | marios, adding ironic in standalone 007 was intentional? | 09:59 |
| ykarel | https://review.opendev.org/#/c/631496/17/ci/environments/scenario007-standalone.yaml | 10:00 |
| marios | ykarel: its been a while but i took what we had in scen7 multinode and copied over ... let me check | 10:00 |
| ykarel | ack, but multinode one don't have ironic | 10:01 |
| ykarel | and i don't think it was there ever, but good to check | 10:01 |
| marios | ykarel: ack checking | 10:01 |
| ykarel | ack | 10:02 |
| marios | ykarel: thanks adding comment. this review was started a long time ago and there was/still is some issue with neutron sidecar containers that beagles was looking at at some point | 10:03 |
| marios | ykarel: so i haven't been focusing on it for a while/other things have priority. i missed that ironic not sure where it came from | 10:04 |
| ykarel | marios, ack, good to revise it as per multinode- 007 file, that will help it move forward | 10:04 |
| marios | ykarel: earlier versions didn't have that so not sure (eg https://review.opendev.org/#/c/631496/4/ci/environments/scenario007-standalone.yaml | 10:04 |
| marios | ykarel: not sure where/why i added that | 10:05 |
| ykarel | ^^ was for ovn | 10:05 |
| ykarel | may be u took scenario012 as reference | 10:05 |
| *** shyamb has quit IRC | 10:05 | |
| openstackgerrit | Sergii Golovatiuk proposed openstack/tripleo-heat-templates master: Refactor docker stop on upgrade tasks https://review.opendev.org/670484 | 10:06 |
| marios | ykarel: ah so thats why | 10:06 |
| marios | ykarel: see comments on v8/9 apr 3 from folco and me | 10:06 |
| * ykarel looks | 10:06 | |
| ykarel | marios, okk, u compared undercloud containers with standalone :( | 10:09 |
| ykarel | should have compared overcloud ones | 10:09 |
| openstackgerrit | Martin Schuppert proposed openstack/tripleo-common master: Create <service> _cell_node_names if nova_additional_cell https://review.opendev.org/670486 | 10:09 |
| marios | ykarel: :D /me facepalm | 10:10 |
| marios | ykarel: in my defence folco started it :D | 10:10 |
| ykarel | :) | 10:10 |
| marios | rfolco: ^^ | 10:10 |
| openstackgerrit | Martin Schuppert proposed openstack/tripleo-heat-templates master: Set nova_additional_cell as global_vars https://review.opendev.org/670487 | 10:10 |
| marios | ykarel: ok will try and look at that again today and fixup those containers | 10:11 |
| marios | ykarel: thank you! | 10:11 |
| ykarel | marios, ack | 10:11 |
| marios | ykarel: pasted irc into the review fyi | 10:12 |
| ykarel | ack | 10:12 |
| *** dsneddon has joined #tripleo | 10:17 | |
| *** dsneddon has quit IRC | 10:21 | |
| *** ratailor_ has quit IRC | 10:23 | |
| *** ratailor has joined #tripleo | 10:24 | |
| *** bogdando has quit IRC | 10:27 | |
| *** holser_ is now known as holser|lunch | 10:28 | |
| *** avivgta has joined #tripleo | 10:30 | |
| *** yolanda has quit IRC | 10:31 | |
| *** yolanda has joined #tripleo | 10:32 | |
| *** irclogbot_3 has joined #tripleo | 10:32 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates master: Add scenario007-standalone environment file https://review.opendev.org/631496 | 10:36 |
| marios | ykarel: thanks when you next have time ^ i rechecked on the ci patch lets see | 10:37 |
| ykarel | marios, ack /me looks | 10:38 |
| *** irclogbot_3 has quit IRC | 10:38 | |
| ykarel | marios, looks better | 10:38 |
| marios | ykarel: yeah we have to wait for the job i restricted it so it will only run that one | 10:39 |
| *** shyamb has joined #tripleo | 10:40 | |
| openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-common master: Remove pika from container builds https://review.opendev.org/670398 | 10:40 |
| marios | simple rename if someone has time please thanks https://review.opendev.org/#/c/669888/ | 10:43 |
| openstackgerrit | Merged openstack/tripleo-common master: Added support of overcloud os_cloud https://review.opendev.org/664568 | 10:43 |
| *** bhagyashris has quit IRC | 10:43 | |
| *** irclogbot_1 has joined #tripleo | 10:43 | |
| *** irclogbot_1 has quit IRC | 10:44 | |
| *** Goneri has joined #tripleo | 10:53 | |
| *** dsneddon has joined #tripleo | 10:53 | |
| *** dsneddon has quit IRC | 10:58 | |
| *** altlogbot_3 has joined #tripleo | 11:04 | |
| *** shyam89 has joined #tripleo | 11:08 | |
| *** altlogbot_3 has quit IRC | 11:08 | |
| *** shyamb has quit IRC | 11:08 | |
| *** tesseract has joined #tripleo | 11:08 | |
| *** marios has quit IRC | 11:09 | |
| *** altlogbot_2 has joined #tripleo | 11:14 | |
| *** dsneddon has joined #tripleo | 11:15 | |
| *** altlogbot_2 has quit IRC | 11:16 | |
| *** bogdando has joined #tripleo | 11:17 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/tripleo-quickstart-extras master: WIP Add playbook and role to run tobiko https://review.opendev.org/655423 | 11:19 |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates master: Stop services for unupgraded controllers https://review.opendev.org/669095 | 11:26 |
| *** avivgta has quit IRC | 11:26 | |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-quickstart master: Add support for running tempest against Octavia featuresets https://review.opendev.org/669666 | 11:29 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-quickstart master: Add support for running tempest against Octavia featuresets https://review.opendev.org/669666 | 11:31 |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates stable/stein: Stop services for unupgraded controllers https://review.opendev.org/669628 | 11:32 |
| openstackgerrit | Carlos Camacho proposed openstack/tripleo-heat-templates master: Stop services for unupgraded controllers https://review.opendev.org/669095 | 11:36 |
| openstackgerrit | Martin Schuppert proposed openstack/tripleo-heat-templates master: Set nova_additional_cell as global_vars https://review.opendev.org/670487 | 11:36 |
| *** dtrainor has quit IRC | 11:37 | |
| *** dtrainor has joined #tripleo | 11:37 | |
| *** apetrich has joined #tripleo | 11:38 | |
| openstackgerrit | Martin Schuppert proposed openstack/tripleo-heat-templates master: Set nova_additional_cell as global_vars https://review.opendev.org/670487 | 11:39 |
| *** shyam89 has quit IRC | 11:39 | |
| openstackgerrit | Martin Schuppert proposed openstack/tripleo-common master: Create <service> _cell_node_names if nova_additional_cell https://review.opendev.org/670486 | 11:39 |
| openstackgerrit | Martin Mágr proposed openstack/tripleo-heat-templates master: Rsyslog composable service https://review.opendev.org/624330 | 11:42 |
| *** paramite has joined #tripleo | 11:44 | |
| paramite | panda, we still seem to have rsyslog image issues: http://logs.openstack.org/30/624330/11/check/tripleo-ci-centos-7-scenario002-standalone/59ac239/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz#_2019-07-08_22_01_55 | 11:45 |
| paramite | panda, any idea why https://review.opendev.org/#/c/624325 was not merged? | 11:45 |
| paramite | EmilienM, maybe you will know ^ | 11:46 |
| EmilienM | paramite: probably was in a gate reset or something | 11:47 |
| EmilienM | I did recheck and it'll land today | 11:47 |
| paramite | EmilienM, thanks a lot! | 11:48 |
| *** shyam89 has joined #tripleo | 11:48 | |
| panda | yeah a recheck should fix it. Also I mistakenly added a +1W | 11:48 |
| panda | I was sure I put just a +1CR there ... probably a mistyping in gertty | 11:48 |
| paramite | panda, I thought that it was on purpose :) | 11:49 |
| panda | paramite: well, I told you I was giving a +1, but I meat a +1 in core review, not in workflow, which actually gives the green flag to merge .. | 11:49 |
| panda | that was a mistake :( | 11:50 |
| *** holser|lunch is now known as holser_ | 11:50 | |
| *** saneax has quit IRC | 11:52 | |
| *** aedc has joined #tripleo | 11:54 | |
| *** udesale has quit IRC | 11:59 | |
| *** udesale has joined #tripleo | 12:00 | |
| *** marios has joined #tripleo | 12:10 | |
| *** morazi has joined #tripleo | 12:16 | |
| *** Goneri has quit IRC | 12:21 | |
| *** derekh has quit IRC | 12:26 | |
| *** aedc has quit IRC | 12:27 | |
| *** dsneddon has quit IRC | 12:28 | |
| *** Goneri has joined #tripleo | 12:29 | |
| *** mcornea has quit IRC | 12:29 | |
| *** ekultails has joined #tripleo | 12:30 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient master: Add --parameters and --create-vars-file arguments to the list subcommand https://review.opendev.org/670352 | 12:31 |
| EmilienM | ykarel: please check with cloudnull when he's online, maybe file a bug if you can | 12:31 |
| *** afazekas_ has joined #tripleo | 12:31 | |
| ykarel | EmilienM, ack, was there already bug against it? i see there was revert | 12:32 |
| ykarel | hmm found https://bugs.launchpad.net/tripleo/+bug/1835657 | 12:32 |
| openstack | Launchpad bug 1835657 in tripleo "tripleo-ci-centos-7-standalone-upgrade-stein is broken by container registry login task" [Critical,Fix released] - Assigned to Kevin Carter (kevin-carter) | 12:32 |
| marios | simple rename if someone has time please thanks https://review.opendev.org/#/c/669888/ | 12:32 |
| ykarel | EmilienM, i think same bug can be reused | 12:33 |
| EmilienM | ok | 12:34 |
| ykarel | as issue is same | 12:34 |
| ykarel | ack | 12:34 |
| *** afazekas_ is now known as afazekas | 12:34 | |
| *** amoralej is now known as amoralej|lunch | 12:35 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/tripleo-quickstart-extras master: WIP Add playbook and role to run tobiko https://review.opendev.org/655423 | 12:37 |
| *** rlandy has joined #tripleo | 12:37 | |
| *** skramaja has quit IRC | 12:39 | |
| openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-heat-templates master: DNM: login-defs in 004 standalone https://review.opendev.org/670034 | 12:40 |
| *** paramite has quit IRC | 12:41 | |
| *** paramite has joined #tripleo | 12:42 | |
| mwhahaha | how are we not reproducing this in ci? | 12:44 |
| openstackgerrit | Luke Short proposed openstack/tripleo-ansible master: Import the tripleo-persist role from tripleo-common. https://review.opendev.org/670362 | 12:49 |
| marios | easy one tidy up files: for scen-standalone 1-4 https://review.opendev.org/#/c/663018/ if you have time please thanks | 12:53 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: WIP generate addition drop-in dependencies for podman containers https://review.opendev.org/669910 | 12:56 |
| *** dsneddon has joined #tripleo | 12:58 | |
| *** aedc has joined #tripleo | 13:00 | |
| *** dsneddon has quit IRC | 13:03 | |
| *** paramite has quit IRC | 13:03 | |
| *** raildo has joined #tripleo | 13:03 | |
| ykarel | marios, so scenario007 moved forward, and failed at os_tempest https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/playbooks/tempest.yml#L32 | 13:04 |
| ykarel | i would say good to go without os_tempest at first stage | 13:05 |
| *** derekh has joined #tripleo | 13:06 | |
| marios | ykarel: ack thanks I will try and look again on monday see if we can get this merged... sounds like the neutron issue was fixed | 13:06 |
| marios | ykarel: thank you for making me look at that today! | 13:06 |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient master: Add --parameters and --create-vars-file arguments to the list subcommand https://review.opendev.org/670352 | 13:06 |
| ykarel | marios, yes, it's better now, i also commented on template file, there is one extra entry in that | 13:07 |
| ykarel | related to ironic | 13:07 |
| ykarel | so once u get change update that also | 13:07 |
| marios | ykarel: ack will check thanks (might do it after a call in a bit otherwise monday) | 13:07 |
| ykarel | ack, np | 13:07 |
| *** aedc has quit IRC | 13:12 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient master: Add --parameters and --create-vars-file arguments to the list subcommand https://review.opendev.org/670352 | 13:13 |
| *** amoralej|lunch is now known as amoralej | 13:19 | |
| openstackgerrit | David Peacock proposed openstack/python-tripleoclient master: WIP - Implement tripleo container image show command https://review.opendev.org/667988 | 13:26 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-ci master: Add roles/playbooks copy in setup.cfg https://review.opendev.org/670551 | 13:28 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-quickstart master: Add support for running tempest against Octavia featuresets https://review.opendev.org/669666 | 13:30 |
| *** sshnaidm|off has quit IRC | 13:30 | |
| *** florianf_ has joined #tripleo | 13:31 | |
| *** shyam89 has quit IRC | 13:33 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient master: Add --parameters and --create-vars-file arguments to the list subcommand https://review.opendev.org/670352 | 13:33 |
| *** florianf has quit IRC | 13:34 | |
| *** sshnaidm has joined #tripleo | 13:35 | |
| *** sshnaidm is now known as sshnaidm|off | 13:36 | |
| *** dsneddon has joined #tripleo | 13:36 | |
| *** florianf_ has quit IRC | 13:37 | |
| *** irclogbot_0 has joined #tripleo | 13:38 | |
| *** irclogbot_0 has quit IRC | 13:38 | |
| openstackgerrit | Sergii Golovatiuk proposed openstack/tripleo-heat-templates stable/stein: Upgrade fixes for RabbitMQ and Pacemaker https://review.opendev.org/670555 | 13:40 |
| *** dsneddon has quit IRC | 13:41 | |
| openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-common master: Correct how we fetch External and InternalApi networks name https://review.opendev.org/670557 | 13:41 |
| *** florianf has joined #tripleo | 13:42 | |
| openstackgerrit | Sergii Golovatiuk proposed openstack/tripleo-heat-templates stable/stein: Upgrade fixes for RabbitMQ and Pacemaker https://review.opendev.org/670555 | 13:44 |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-ci master: Add roles/playbooks copy in setup.cfg https://review.opendev.org/670551 | 13:45 |
| *** paramite has joined #tripleo | 13:49 | |
| *** sabedevops has joined #tripleo | 13:51 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart master: Add tripleo-ci to the extra-requirements https://review.opendev.org/670563 | 13:52 |
| marios | reviews please https://review.opendev.org/#/c/669329/ & https://review.opendev.org/#/c/670290/ needed by https://review.rdoproject.org/r/#/c/21357/ rdo rhel 8 image build https://tree.taiga.io/project/tripleo-ci-board/task/1151 thank you! | 13:54 |
| *** ykarel is now known as ykarel|afk | 13:54 | |
| *** aakarsh has quit IRC | 13:56 | |
| *** Vorrtex has joined #tripleo | 13:56 | |
| *** ykarel|afk has quit IRC | 13:59 | |
| cloudnull | mornings | 14:03 |
| slagle | ekultails: hey, what's the difference between molecule/default and molecule/legacy_vars? i guess both need to be updated the same way? | 14:04 |
| slagle | looks like CI is using legacy_vars. but when I run molecule locally, default is used | 14:04 |
| *** vinaykns has joined #tripleo | 14:05 | |
| ekultails | slagle: The first level of directories below molecule are the different scenarios. | 14:07 |
| ekultails | Based on what needs to be tested, there may be one or more scenarios. | 14:07 |
| ekultails | By default, the "default" scenario will run. | 14:07 |
| ekultails | You can run `molecule test --all` to run the mall or `molecule test --scenario legacy_vars` to cover only the other scenario. | 14:08 |
| *** dsneddon has joined #tripleo | 14:08 | |
| slagle | ekultails: ok, that makes sense. but why do we need both if the contents are identical? | 14:08 |
| ekultails | run them all* not run the mall. Sorry, I've been watching Strangers Things 3. ;-) | 14:09 |
| ekultails | Hmm, if they really are exactly the same then it sounds like we can get ride of the legacy_vars. | 14:09 |
| ekultails | And just keep the default scenario. | 14:10 |
| slagle | they're the same, at least for the role i'm currently working on (tripleo-ssh-known-hosts) | 14:10 |
| slagle | i'll try just rm'ing legacy_vars, see if CI is happy with that :) | 14:10 |
| *** irclogbot_2 has joined #tripleo | 14:10 | |
| ekultails | cloudnull: Do you know why there is a legacy_vars scenario? Looks to be used in a handful of roles. | 14:12 |
| cloudnull | yes its testing using non-namespaced variables | 14:13 |
| cloudnull | slagle the ci tests all of the scenarios. if you wanted to run them all you can do so with `molecule test --all` | 14:13 |
| cloudnull | you could also run just that one scenario with `molecule test -s $NAME` | 14:14 |
| *** chandankumar is now known as raukadah | 14:14 | |
| cloudnull | we also have some tools to mimic what zuul is doing - https://docs.openstack.org/tripleo-ansible/latest/contributing.html#local-testing-of-new-roles | 14:14 |
| cloudnull | if you are running things in a clean vm | 14:14 |
| *** altlogbot_0 has joined #tripleo | 14:15 | |
| *** dsneddon has quit IRC | 14:15 | |
| ekultails | Namespaced variables meaning, for example, being prefixed with tripleo_ssh_known_hosts? | 14:15 |
| cloudnull | yes, instead of ssh_known_hosts | 14:15 |
| cloudnull | as an example | 14:16 |
| * cloudnull has no idea if that was a var | 14:16 | |
| ekultails | Cool, thanks for the clarification. | 14:16 |
| cloudnull | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo-container-tag/defaults/main.yml#L22-L31 | 14:16 |
| cloudnull | is a good example | 14:16 |
| openstackgerrit | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service. https://review.opendev.org/666578 | 14:17 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: tripleo-hieradata: merge j2 hiera templates https://review.opendev.org/669916 | 14:17 |
| ekultails | Yeah, back in Consulting I ran into a lot of customer use cases that had very generic and conflicting names lol. Namespaces/prefixes are definitely the way to go. | 14:18 |
| ekultails | ++ | 14:18 |
| cloudnull | ++ | 14:18 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: tripleo-hieradata: include config_step into all_nodes_merged https://review.opendev.org/669921 | 14:18 |
| *** ykarel|afk has joined #tripleo | 14:24 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: Remove the tripleo.plan_management.v1.list_sample_plans workflow https://review.opendev.org/669163 | 14:24 |
| *** aakarsh has joined #tripleo | 14:25 | |
| EmilienM | abishop: thank you for the reply on cinder/netapp | 14:27 |
| slagle | cloudnull: ok. so since i plan to remove the usage of the non-namespaced var, i can also remove all of legacy_vars | 14:27 |
| *** jcoufal has joined #tripleo | 14:27 | |
| cloudnull | +1 | 14:27 |
| cloudnull | I just added tests so we'd catch that usage due to us having both non-namespaced and namespaced vars when we did the role import | 14:28 |
| cloudnull | but if we dont need that anymore, remove it :) | 14:28 |
| * cloudnull is always happy to delete things :D | 14:28 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-ansible master: Remove non-template mode from tripleo-ssh-known-hosts role https://review.opendev.org/669966 | 14:28 |
| abishop | EmilienM: sure, thank you for your first one! | 14:29 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/rocky: Revert "Revert "Set proper setype for service directories"" https://review.opendev.org/644258 | 14:31 |
| *** bnemec is now known as beekneemech | 14:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/rocky: Revert "Revert "docker: wire SELinuxMode with Ansible vars"" https://review.opendev.org/644257 | 14:35 |
| slagle | cloudnull: ekultails : out of curiosity, is there any effort for podman support in molecule? | 14:38 |
| cloudnull | I think zbr just got that added ? | 14:39 |
| openstackgerrit | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service. https://review.opendev.org/666578 | 14:39 |
| slagle | yea? cool | 14:39 |
| * cloudnull would like to switch to that instead of docker if at all possible. | 14:39 | |
| cloudnull | though I've not actually given it a spin yet | 14:39 |
| cloudnull | https://github.com/ansible/molecule/pull/2098 | 14:40 |
| *** hoonetorg has quit IRC | 14:40 | |
| cloudnull | sorry sshnaidm|off made the initial commit | 14:40 |
| cloudnull | but, yes, it lookslike its in ? | 14:40 |
| cloudnull | maybe just not in a released branch | 14:40 |
| zbr | cloudnull: yep sshnaidm|off added it but its is experimental, is is on master. | 14:40 |
| cloudnull | slagle ^ | 14:41 |
| zbr | but we do not run CI on it and i doubt is usable in jobs. | 14:41 |
| slagle | ok, i may give it a try locally | 14:41 |
| cloudnull | ++ I'd be keen on switching our jobs around to run podman instead of docker | 14:42 |
| zbr | i would also be very unhappy to see it activated in jobs because it will prevent me from running molecule locally. my desktop is a mac, and I do run molecule very well now with docker. | 14:42 |
| zbr | no remoting support in podman at all (docker has good DOCKER_HOST=ssh://), also other problems. More likely to start using libvirt or openstack driver than podman driver. | 14:44 |
| *** dsneddon has joined #tripleo | 14:47 | |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-common master: Set container=oci for our containers https://review.opendev.org/670574 | 14:49 |
| *** Goneri has quit IRC | 14:52 | |
| *** rpittau is now known as elfosardo | 14:52 | |
| openstackgerrit | Harald Jensås proposed openstack/python-tripleoclient master: Ensure files are closed, ensure no ResourceWarning on console https://review.opendev.org/670576 | 14:54 |
| *** hoonetorg has joined #tripleo | 14:56 | |
| *** dsneddon has quit IRC | 14:57 | |
| *** Goneri has joined #tripleo | 14:57 | |
| *** ratailor has quit IRC | 14:58 | |
| *** ayoung has quit IRC | 15:12 | |
| *** ayoung has joined #tripleo | 15:12 | |
| *** Goneri has quit IRC | 15:15 | |
| *** ykarel|afk is now known as ykarel|away | 15:16 | |
| openstackgerrit | Merged openstack/tripleo-ci master: Rename tripleo-ci build-images to oooci-build-images for clash https://review.opendev.org/669888 | 15:18 |
| *** sabedevops has quit IRC | 15:19 | |
| *** amoralej is now known as amoralej|off | 15:19 | |
| *** ansmith has quit IRC | 15:23 | |
| *** dsneddon has joined #tripleo | 15:27 | |
| *** iurygregory is now known as skolt | 15:27 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/tripleo-quickstart-extras master: WIP Add playbook and role to run tobiko https://review.opendev.org/655423 | 15:27 |
| *** holser_ has quit IRC | 15:30 | |
| *** jcoufal has quit IRC | 15:34 | |
| *** yprokule has quit IRC | 15:35 | |
| *** morazi_ has joined #tripleo | 15:35 | |
| *** morazi has quit IRC | 15:36 | |
| *** marios is now known as marios|out | 15:39 | |
| openstackgerrit | Gael Chamoulaud proposed openstack/python-tripleoclient master: Add --parameters and --create-vars-file arguments to the list subcommand https://review.opendev.org/670352 | 15:40 |
| *** ayoung has quit IRC | 15:44 | |
| *** jcoufal has joined #tripleo | 15:45 | |
| openstackgerrit | Harald Jensås proposed openstack/python-tripleoclient master: Ensure files are closed, ensure no ResourceWarning on console https://review.opendev.org/670576 | 15:45 |
| *** marios|out has quit IRC | 15:46 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: Correct how we fetch External and InternalApi networks name https://review.opendev.org/670557 | 15:47 |
| *** lucasagomes has quit IRC | 15:47 | |
| *** nkinder has joined #tripleo | 15:47 | |
| EmilienM | damn | 15:47 |
| *** ayoung has joined #tripleo | 15:48 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: Correct how we fetch External and InternalApi networks name https://review.opendev.org/670557 | 15:48 |
| cloudnull | EmilienM mwhahaha weshay|rover when you get a chance - https://review.opendev.org/#/c/670408/ - could use some review . | 15:48 |
| cloudnull | regarding our image build tools not raising an exception | 15:49 |
| EmilienM | cloudnull: please update unit testds | 15:49 |
| cloudnull | will do | 15:49 |
| *** bcafarel has quit IRC | 15:50 | |
| cloudnull | doesn't look like build_all has a unit test | 15:51 |
| * cloudnull will go make one | 15:51 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: tripleo-hieradata: merge j2 hiera templates https://review.opendev.org/669916 | 15:51 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: tripleo-hieradata: merge j2 hiera templates https://review.opendev.org/669916 | 15:52 |
| EmilienM | slagle: this log confused me for the bootstrap_nodeid thing: http://logs.openstack.org/17/669917/10/check/tripleo-ci-centos-7-undercloud-containers/d47c1e4/logs/undercloud/home/zuul/undercloud_install.log.txt.gz#_2019-07-12_04_36_38 | 15:52 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-common master: tripleo-hieradata: include config_step into all_nodes_merged https://review.opendev.org/669921 | 15:53 |
| slagle | EmilienM: i can tell you why | 15:54 |
| slagle | EmilienM: bootstrap_nodeid comes from this file: http://logs.openstack.org/17/669917/10/check/tripleo-ci-centos-7-undercloud-containers/d47c1e4/logs/undercloud/home/zuul/undercloud-ansible-yf2YcF/Undercloud/hieradata_vars.yaml.txt.gz | 15:55 |
| slagle | EmilienM: you can see where i pass that in with vars_files on the "Hieradata from vars" task | 15:55 |
| slagle | EmilienM: however, since you moved bootstrap_nodeid to the "render all_nodes" tasks, the var is undefined there | 15:55 |
| weshay|rover | EmilienM I'll ask rfolco to include that patch in our manual test | 15:56 |
| weshay|rover | rfolco https://review.opendev.org/#/c/670408/4/tripleo_common/image/builder/buildah.py | 15:56 |
| weshay|rover | we may need to rebase that patch on top of some other tripleo-common patches to test it | 15:56 |
| weshay|rover | cloudnull EmilienM is that ok? | 15:57 |
| slagle | EmilienM: i would actually like to get rid of the "render all_nodes data as group_vars..." tasks. we'd just need to find all the other tasks that are using those vars. i think there's just a few | 15:57 |
| * weshay|rover looks at which ones | 15:57 | |
| EmilienM | slagle: ok, let me look that | 15:57 |
| *** ykarel|away has quit IRC | 15:57 | |
| *** ansmith has joined #tripleo | 15:57 | |
| weshay|rover | https://review.opendev.org/#/c/670398/ that one | 15:57 |
| cloudnull | weshay|rover feel free to rebase / change / modify anything I ever work on. | 15:57 |
| weshay|rover | thanks | 15:58 |
| weshay|rover | rfolco you seeing this? | 15:58 |
| slagle | EmilienM: the other thing we could do is move what's in hieradata_vars.yaml to be group vars, then we wouldn't need to use the vars_files | 15:58 |
| weshay|rover | rfolco multiple patches in tripleo-common :) | 15:58 |
| slagle | EmilienM: i had originally thought it might be a lot of data, and I didn't want to set all that as host/group vars. but it turned out to be pretty small | 15:59 |
| EmilienM | slagle: yes, let's do that and if it becomes too big we can still roll-back... | 15:59 |
| slagle | b/c anything you set as a host var persists in memory for that host for the whole playbook run | 15:59 |
| EmilienM | right | 15:59 |
| slagle | ok, i can look at that | 15:59 |
| openstackgerrit | Merged openstack/tripleo-puppet-elements stable/queens: Stop and disable libvirtd service on controller https://review.opendev.org/670448 | 15:59 |
| openstackgerrit | Merged openstack/tripleo-heat-templates stable/rocky: Allow logrotate to access container_file_t files https://review.opendev.org/670451 | 16:00 |
| EmilienM | cool thanks | 16:00 |
| EmilienM | weshay|rover: yes, if you want to test out the errors it's the patch to use | 16:00 |
| * EmilienM lunch | 16:00 | |
| rfolco | weshay|rover, ok will apply both patches in tripleo-common | 16:00 |
| *** adrianreza_ has joined #tripleo | 16:01 | |
| *** jpich has quit IRC | 16:01 | |
| *** ykarel|away has joined #tripleo | 16:09 | |
| *** pkopec has quit IRC | 16:10 | |
| *** sabedevops has joined #tripleo | 16:13 | |
| *** Vorrtex has quit IRC | 16:13 | |
| openstackgerrit | Merged openstack/tripleo-common master: Remove pika from container builds https://review.opendev.org/670398 | 16:17 |
| weshay|rover | rfolco fyi.. https://review.opendev.org/#/c/670398/ just merged | 16:23 |
| *** hjensas is now known as hjensas|afk | 16:24 | |
| *** elfosardo is now known as rpittau|afk | 16:27 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Set HieradataVars as group vars instead https://review.opendev.org/670596 | 16:30 |
| slagle | EmilienM: ^ | 16:30 |
| slagle | only light testing done so far :) my deploy is still in progress | 16:31 |
| openstackgerrit | Brent Eagles proposed openstack/tripleo-heat-templates master: Configure amphora for upload and run tempest https://review.opendev.org/669669 | 16:35 |
| *** bogdando has quit IRC | 16:42 | |
| ykarel|away | cloudnull, can u please check https://bugs.launchpad.net/tripleo/+bug/1835657 | 16:46 |
| openstack | Launchpad bug 1835657 in tripleo "tripleo-ci-centos-7-standalone-upgrade-stein is broken by container registry login task" [Critical,Fix released] - Assigned to Kevin Carter (kevin-carter) | 16:46 |
| ykarel|away | may be u already on it | 16:46 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/tripleo-common master: Ensure we're raising proper exceptions https://review.opendev.org/670408 | 16:47 |
| cloudnull | EmilienM ^ | 16:47 |
| *** derekh has quit IRC | 16:48 | |
| cloudnull | ykarel|away I've been looking at that, and sadly I've not been able to recreate the error. though I've found a couple suspect things I've love to chat more about. I'll post them in the LP issue so folks can think about it asynchronously | 16:49 |
| ykarel|away | cloudnull, ack, atleast it's failing consistantly in CI | 16:51 |
| cloudnull | ykarel|away when I look at http://logs.openstack.org/25/654925/9/check/tripleo-ci-centos-7-standalone-upgrade-stein/44390d5/logs/undercloud/home/zuul/standalone_upgrade.log.txt.gz#_2019-07-11_10_11_16 I see it skipping the fact set, which would mean the logins option is initially undefined? | 16:55 |
| cloudnull | and when I look at the internal CI which I assume is a multi-node test I see the facts being set on controller-0 but not compute-1 which I think is why it's raising the error there | 16:56 |
| cloudnull | 2019-07-12 09:03:13,120 p=301 u=mistral | ok: [controller-0] => {"ansible_facts": {"container_registry_insecure_registries": ["192.168.24.1:8787"], "container_registry_login": false, "container_registry_logins": {}, "container_registry_logins_json": {}}, "changed": false} | 16:56 |
| cloudnull | 2019-07-12 09:03:15,738 p=301 u=mistral | fatal: [compute-0]: FAILED! => {"msg": "An unhandled exception occurred while running the lookup plugin 'dict'. Error was a <class 'ansible.errors.AnsibleError'>, original message: with_dict expects a dict"} | 16:56 |
| cloudnull | ^ Log entries I was looking at earlier | 16:56 |
| *** udesale has quit IRC | 16:59 | |
| *** morazi_ has quit IRC | 17:00 | |
| *** betherly has joined #tripleo | 17:00 | |
| cloudnull | ykarel|away I'm going to go grab a bite to eat but I'll continue looking at it | 17:01 |
| *** psachin has quit IRC | 17:03 | |
| ykarel|away | cloudnull, let me try to reproduce, if i can | 17:04 |
| *** jtomasek has quit IRC | 17:04 | |
| *** betherly has quit IRC | 17:05 | |
| EmilienM | slagle: thanks | 17:09 |
| *** jcoufal has quit IRC | 17:09 | |
| EmilienM | cloudnull: ack | 17:09 |
| *** morazi_ has joined #tripleo | 17:13 | |
| ykarel|away | cloudnull, i am able to reproduce it locally | 17:13 |
| ykarel|away | cloudnull, try running playbook locally: http://paste.openstack.org/show/754351/ | 17:15 |
| openstackgerrit | Natal Ngétal proposed openstack/paunch master: [Configuration] Switch to stestr. https://review.opendev.org/629421 | 17:17 |
| openstackgerrit | Clark Boylan proposed openstack/diskimage-builder master: Only enable dbus-daemon on fedora-29 https://review.opendev.org/670606 | 17:23 |
| *** florianf has quit IRC | 17:33 | |
| openstackgerrit | Merged openstack/puppet-tripleo master: Replace hiera('service_names') by hiera('enabled_services') https://review.opendev.org/669414 | 17:36 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Remove service_names hieradata file https://review.opendev.org/669424 | 17:36 |
| *** xek has quit IRC | 17:53 | |
| *** xek has joined #tripleo | 17:53 | |
| *** poopcat has joined #tripleo | 17:54 | |
| *** betherly has joined #tripleo | 18:01 | |
| *** betherly has quit IRC | 18:06 | |
| EmilienM | slagle: I commented on https://review.opendev.org/#/c/670596/1/puppet/role.role.j2.yaml | 18:16 |
| *** betherly has joined #tripleo | 18:21 | |
| *** betherly has quit IRC | 18:26 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Set HieradataVars as group vars instead https://review.opendev.org/670596 | 18:27 |
| slagle | EmilienM: fixed | 18:27 |
| EmilienM | slagle: +2 | 18:28 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: deploy-steps: remove unused hieradata_templates_list https://review.opendev.org/669917 | 18:29 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove config_step hieradata https://review.opendev.org/669924 | 18:29 |
| slagle | dang, that going to merge conflict my own patch | 18:35 |
| *** sabedevops has quit IRC | 18:37 | |
| *** betherly has joined #tripleo | 18:42 | |
| EmilienM | slagle: you can rebase on tops of the ssh things | 18:43 |
| EmilienM | or vice versa if yours can wait | 18:44 |
| *** rascasoft has quit IRC | 18:44 | |
| *** mburned is now known as mburned_out | 18:46 | |
| *** betherly has quit IRC | 18:46 | |
| *** rascasoft has joined #tripleo | 18:47 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Set HieradataVars as group vars instead https://review.opendev.org/670596 | 18:47 |
| *** morazi_ has quit IRC | 18:47 | |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Set HieradataVars as group vars instead https://review.opendev.org/670596 | 18:48 |
| *** ccamacho has quit IRC | 18:48 | |
| slagle | EmilienM: ok, i did | 18:48 |
| *** irclogbot_2 has quit IRC | 18:49 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: deploy-steps: remove unused hieradata_templates_list https://review.opendev.org/669917 | 18:50 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove config_step hieradata https://review.opendev.org/669924 | 18:50 |
| EmilienM | no more conflict :) | 18:50 |
| *** irclogbot_1 has joined #tripleo | 18:53 | |
| *** slaweq has quit IRC | 19:17 | |
| *** mburned_out is now known as mburned | 19:21 | |
| cloudnull | ykarel|away that playbook works for me | 19:25 |
| cloudnull | https://pasted.tech/pastes/20cae44b777be2458e32301f0d0c02b54f44386c.raw | 19:25 |
| ykarel|away | cloudnull, strange, can u try with ansible 2.8.0 | 19:26 |
| cloudnull | sure | 19:26 |
| cloudnull | installing now | 19:26 |
| ykarel|away | cloudnull, looks like u have not used complete playbook | 19:27 |
| ykarel|away | let me recheck what i shared | 19:28 |
| cloudnull | same result | 19:28 |
| cloudnull | https://pasted.tech/pastes/cce87c6d75054d2b513889ac6cd8084414925302.raw | 19:28 |
| ykarel|away | cloudnull, try to use same playbook i shared | 19:28 |
| ykarel|away | u missed last conditionals | 19:28 |
| ykarel|away | when: step|int ==1 | 19:29 |
| ykarel|away | to the block | 19:29 |
| cloudnull | oic | 19:29 |
| cloudnull | that failed | 19:29 |
| cloudnull | https://pasted.tech/pastes/07714b556f63c5855ee7dd294838084fae8cec3c.raw | 19:30 |
| cloudnull | and that's because the look is evaluated but those facts are not set | 19:30 |
| cloudnull | so those facts need to be set outside of the block | 19:30 |
| ykarel|away | cloudnull, yup | 19:31 |
| cloudnull | ykarel|away mind trying https://pasted.tech/pastes/5a2850e51b83035d5ed9e95055920a15f69048b7 | 19:33 |
| cloudnull | just to make sure we're on the same page | 19:33 |
| *** tesseract has quit IRC | 19:33 | |
| cloudnull | if that works for you I'll smash together another review | 19:33 |
| ykarel|away | cloudnull, ^^ looks same i shared, u changed something | 19:34 |
| ykarel|away | ? | 19:34 |
| *** slaweq has joined #tripleo | 19:34 | |
| cloudnull | yes its slightly different | 19:34 |
| ykarel|away | okk got it u moved block: | 19:35 |
| cloudnull | ++ | 19:35 |
| ykarel|away | yes ^^ should work | 19:35 |
| cloudnull | k | 19:36 |
| mwhahaha | login shouldn't be stepwise | 19:37 |
| mwhahaha | i thought it was host prep | 19:37 |
| ykarel|away | hmm right | 19:38 |
| *** skolt has quit IRC | 19:39 | |
| ykarel|away | EmilienM, see ^^, looks like u are chasing same thing over email, re. original message: with_dict expects a dict" | 19:39 |
| mwhahaha | additionally it hit on the overcloud but not undercloud | 19:40 |
| mwhahaha | which uses the same code | 19:40 |
| mwhahaha | i wodner if overcloud injects bad credentials somewhere | 19:40 |
| * mwhahaha is slowly building an environment | 19:40 | |
| EmilienM | yeah me too | 19:40 |
| ykarel|away | ack | 19:41 |
| *** mburned is now known as mburned_out | 19:41 | |
| mwhahaha | which is exactly what i want to be doing on a friday afternoon | 19:42 |
| * mwhahaha cries | 19:42 | |
| EmilienM | lol | 19:43 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move container logins facts outside of the login block https://review.opendev.org/670627 | 19:44 |
| cloudnull | ykarel|away ^ | 19:45 |
| ykarel|away | ohhk so install_and_configure_podman anchor was adding that if condition | 19:47 |
| cloudnull | Though, after seeing what mwhahaha just said, and rereading the tasks, there's no conditional on that block | 19:48 |
| cloudnull | ykarel|away is that a heat'ism ? | 19:48 |
| ykarel|away | so with this host_prep_tasks has condition when: step|int == 1 | 19:48 |
| ykarel|away | cloudnull, see Line 136 | 19:48 |
| cloudnull | OHIC | 19:49 |
| ykarel|away | and here is that in the log http://logs.openstack.org/25/654925/9/check/tripleo-ci-centos-7-standalone-upgrade-stein/44390d5/logs/undercloud/home/zuul/undercloud-ansible-1WMDAE/Standalone/upgrade_tasks.yaml.txt.gz | 19:49 |
| cloudnull | in the upgrade tasks... | 19:49 |
| cloudnull | so I guess those facts need to be set there too | 19:50 |
| mwhahaha | ugh | 19:50 |
| * mwhahaha throws things at ansible for evaluating things that should be skipped | 19:50 | |
| cloudnull | ^ | 19:50 |
| *** slaweq has quit IRC | 19:51 | |
| mwhahaha | if we do something stupid like wrap it in a block, does the loop stop being evaluated? | 19:51 |
| mwhahaha | or is it always evaulated no matter what (wtf) | 19:51 |
| cloudnull | the lookup is being evaluated no matter what | 19:52 |
| mwhahaha | meh | 19:52 |
| *** morazi has joined #tripleo | 19:53 | |
| mwhahaha | so sounds like we need to move it to an ansible role? | 19:53 |
| mwhahaha | and then it'll stop being evaluated? | 19:53 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move container logins facts outside of the login block https://review.opendev.org/670627 | 19:53 |
| cloudnull | ^ that will make the login facts available to both the hostprep and upgrade tasks k | 19:54 |
| mwhahaha | cloudnull: i don't think that's fixes it because we take that whole section and alap a step on it | 19:54 |
| mwhahaha | so it won't end up different | 19:54 |
| cloudnull | I added the facts block as a pre-step | 19:54 |
| cloudnull | which should be run before the failing block with the step conditional | 19:55 |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Set HieradataVars as group vars instead https://review.opendev.org/670596 | 19:55 |
| cloudnull | however, yes, if we stuck these things in a role ansible would not evaluate any of it until its executed. | 19:55 |
| mwhahaha | i don't think moving it solves it | 19:55 |
| mwhahaha | we'll have to check the generated upgrade tasks | 19:56 |
| mwhahaha | but we'll see | 19:56 |
| mwhahaha | because when step|int == 2, it's undefined | 19:56 |
| * mwhahaha blames EmilienM for not creating a podman role. also cause it's friday | 19:56 | |
| cloudnull | https://review.opendev.org/#/c/670627/2/deployment/podman/podman-baremetal-ansible.yaml@133 should run regardless of step? | 19:57 |
| mwhahaha | oh maybe | 19:58 |
| mwhahaha | yaml id's are hard to follow sometimes | 19:58 |
| * ykarel|away too late, need to go | 19:58 | |
| EmilienM | what | 19:58 |
| EmilienM | why you always end up blaming me | 19:58 |
| mwhahaha | EmilienM: because you end up writing 1/3 of everything | 19:58 |
| cloudnull | EmilienM tl;dr its friday and your fault | 19:58 |
| mwhahaha | plus podman is your fault | 19:59 |
| mwhahaha | like ntp/chrony is my fault | 19:59 |
| EmilienM | climate change as well? | 19:59 |
| mwhahaha | or config download is slagle's fault | 19:59 |
| mwhahaha | sure we can blame you for climate change | 19:59 |
| * ykarel|away :D | 20:00 | |
| cloudnull | ykarel|away are you able to pull that patch and give it a spin ? | 20:00 |
| *** betherly has joined #tripleo | 20:02 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: deploy-steps: remove unused hieradata_templates_list https://review.opendev.org/669917 | 20:03 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Remove config_step hieradata https://review.opendev.org/669924 | 20:03 |
| *** ykarel|away has quit IRC | 20:04 | |
| cloudnull | mwhahaha EmilienM is there a DNM review up we can update to see if the latest change resolves the container login issues? | 20:05 |
| EmilienM | afik my DNM in oooq was useless | 20:06 |
| mwhahaha | not really i don't think we've been able to reproduce it in stream? | 20:06 |
| mwhahaha | i've got an env i'll test shortly | 20:06 |
| EmilienM | same | 20:06 |
| EmilienM | my standalone deployed just fine btw | 20:06 |
| * mwhahaha does the introspection dance | 20:06 | |
| openstackgerrit | Merged openstack/tripleo-ci master: Run periodic standalone jobs https://review.opendev.org/669977 | 20:07 |
| *** betherly has quit IRC | 20:07 | |
| cloudnull | EmilienM ++ standalone deployment seem to work just fine, I think the missing link was these facts were not getting set in that upgrade_tasks section. | 20:09 |
| mwhahaha | yea but we're seeing it elsewhere on normal overcloud deployments | 20:09 |
| EmilienM | yeah | 20:09 |
| mwhahaha | so unless we're running upgrade tasks on deploys (which would be bad), something else is weird | 20:09 |
| cloudnull | I think that issue was due to the lookup being evaluated even when the task shouldn't run | 20:10 |
| * cloudnull guessing | 20:10 | |
| *** holser_ has joined #tripleo | 20:11 | |
| mwhahaha | we're all just guessing at this point :D | 20:13 |
| openstackgerrit | Merged openstack/diskimage-builder master: Only enable dbus-daemon on fedora-29 https://review.opendev.org/670606 | 20:13 |
| *** mburned_out is now known as mburned | 20:14 | |
| *** slaweq has joined #tripleo | 20:15 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/tripleo-heat-templates master: [dnm] Test manila without pacemaker https://review.opendev.org/642926 | 20:19 |
| openstackgerrit | Corey Bryant proposed openstack/os-net-config master: Add Python 3 Train unit tests https://review.opendev.org/670643 | 20:19 |
| vkmc | gouthamr, (o/ \o) \o/ | 20:19 |
| openstackgerrit | Corey Bryant proposed openstack/paunch master: Add Python 3 Train unit tests https://review.opendev.org/670644 | 20:20 |
| gouthamr | vkmc: ^ rebased that old patch :) if it fails, let's debug with the fresh logs :P | 20:20 |
| openstackgerrit | John Dennis proposed openstack/tripleo-quickstart master: Create supplemental node volume in correct volume pool https://review.opendev.org/670646 | 20:20 |
| *** slaweq has quit IRC | 20:20 | |
| vkmc | gouthamr, yes, cool | 20:21 |
| vkmc | thanks | 20:21 |
| openstackgerrit | Corey Bryant proposed openstack/tripleo-image-elements master: Add Python 3 Train unit tests https://review.opendev.org/670647 | 20:21 |
| openstackgerrit | James Slagle proposed openstack/tripleo-heat-templates master: Set HieradataVars as group vars instead https://review.opendev.org/670596 | 20:22 |
| openstackgerrit | Corey Bryant proposed openstack/os-refresh-config master: Add Python 3 Train unit tests https://review.opendev.org/670649 | 20:22 |
| openstackgerrit | Jeremy Freudberg proposed openstack/tripleo-validations master: Support multi-arch in deploy image validations https://review.opendev.org/666730 | 20:22 |
| openstackgerrit | Jeremy Freudberg proposed openstack/tripleo-validations master: [wip] Loosen ironic-boot-configuration https://review.opendev.org/670650 | 20:22 |
| openstackgerrit | Corey Bryant proposed openstack/openstack-virtual-baremetal master: Add Python 3 Train unit tests https://review.opendev.org/670651 | 20:23 |
| *** betherly has joined #tripleo | 20:23 | |
| *** slagle is now known as slagle-afk | 20:23 | |
| openstackgerrit | Corey Bryant proposed openstack/os-collect-config master: Add Python 3 Train unit tests https://review.opendev.org/670652 | 20:24 |
| openstackgerrit | Corey Bryant proposed openstack/tripleo-repos master: Add Python 3 Train unit tests https://review.opendev.org/670653 | 20:24 |
| *** betherly has quit IRC | 20:27 | |
| openstackgerrit | Corey Bryant proposed openstack/tripleo-validations master: Add Python 3 Train unit tests https://review.opendev.org/670655 | 20:28 |
| openstackgerrit | Corey Bryant proposed openstack/tripleo-common-tempest-plugin master: Add Python 3 Train unit tests https://review.opendev.org/670656 | 20:29 |
| openstackgerrit | Corey Bryant proposed openstack/python-tripleoclient master: Add Python 3 Train unit tests https://review.opendev.org/670657 | 20:29 |
| *** slaweq has joined #tripleo | 20:31 | |
| *** slaweq has quit IRC | 20:35 | |
| openstackgerrit | Corey Bryant proposed openstack/tripleo-common master: Add Python 3 Train unit tests https://review.opendev.org/670658 | 20:36 |
| openstackgerrit | Corey Bryant proposed openstack/tripleo-puppet-elements master: Add Python 3 Train unit tests https://review.opendev.org/670659 | 20:38 |
| openstackgerrit | Corey Bryant proposed openstack/tripleo-heat-templates master: Add Python 3 Train unit tests https://review.opendev.org/670660 | 20:39 |
| openstackgerrit | Merged openstack/openstack-virtual-baremetal master: Add Python 3 Train unit tests https://review.opendev.org/670651 | 20:39 |
| mwhahaha | cloudnull: so interesting thing is that the login task is coming from the controller tasks rather than the compute (but is failing on compute) | 20:40 |
| openstackgerrit | Corey Bryant proposed openstack/os-apply-config master: Add Python 3 Train unit tests https://review.opendev.org/670661 | 20:40 |
| mwhahaha | cloudnull, EmilienM: it's failing because we're running the controller host prep tasks so the compute node is skipped when we run the set facts, but we still evaluate the query | 20:43 |
| mwhahaha | http://paste.openstack.org/show/754360/ | 20:44 |
| mwhahaha | i think we need to stop using query() or lookup() | 20:45 |
| mwhahaha | on facts | 20:46 |
| mwhahaha | because they get skipped | 20:46 |
| * mwhahaha goes back to the drawing board | 20:46 | |
| cloudnull | :'( | 20:50 |
| cloudnull | mwhahaha what if we slam that together in a new role and just include it ? | 20:51 |
| mwhahaha | so we could fix it for train but we need to backport it | 20:52 |
| cloudnull | then it'd work similar to what we're doing with the older docker login | 20:52 |
| cloudnull | hum | 20:52 |
| mwhahaha | so it'd have to go tripleo-ansible to tripleo-common | 20:52 |
| mwhahaha | i'd like to figure out how to solve this in THT if possible, but yes a role in the long term would be more ideal | 20:52 |
| cloudnull | ++ | 20:53 |
| mwhahaha | i think i figured it out | 20:53 |
| * mwhahaha sighs | 20:54 | |
| * cloudnull holds mwhahaha's beer | 20:54 | |
| mwhahaha | it's simplier than we were making it | 20:54 |
| mwhahaha | i also don't think we need the json conversion stuff either | 20:55 |
| mwhahaha | but i'll follow up with that later | 20:55 |
| *** chem has quit IRC | 20:56 | |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Specify a default for container_registry_logins https://review.opendev.org/670663 | 20:56 |
| mwhahaha | cloudnull: -^ | 20:56 |
| mwhahaha | that's it | 20:56 |
| * cloudnull trying | 20:57 | |
| *** pcaruana has quit IRC | 20:59 | |
| mwhahaha | oops needs an extra ) | 21:00 |
| cloudnull | ++ works | 21:00 |
| openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: Specify a default for container_registry_logins https://review.opendev.org/670663 | 21:00 |
| mwhahaha | but yea that's it | 21:00 |
| mwhahaha | so we don't need to move anything around either | 21:00 |
| *** raildo has quit IRC | 21:01 | |
| openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: DNM: RHEL8 support for containers build https://review.opendev.org/669487 | 21:01 |
| cloudnull | https://pasted.tech/pastes/09908ff3e620d8cf3cf225b9da26ef654c42909f <- that was the reproducer playbook I was tinkering with and with the default it works, which sucks because it means the lookup is being run no matter. | 21:01 |
| mwhahaha | right | 21:02 |
| cloudnull | but +1 from me, that would be easy to backport | 21:02 |
| openstackgerrit | Alex Schultz proposed openstack/ansible-role-container-registry master: Specify a default for container_registry_logins https://review.opendev.org/670665 | 21:02 |
| cloudnull | I abandoned https://review.opendev.org/#/c/670627/ | 21:03 |
| mwhahaha | i think the more concerting part is that I don't think it ever runs correctly on the nova compute | 21:03 |
| * mwhahaha chooses to ignore this and moves on with life | 21:03 | |
| *** betherly has joined #tripleo | 21:04 | |
| openstackgerrit | Merged openstack/tripleo-ansible master: Add template mode to tripleo-ssh-known-hosts https://review.opendev.org/669960 | 21:05 |
| cloudnull | hahaha | 21:05 |
| openstackgerrit | Kevin Carter (cloudnull) proposed openstack/tripleo-common master: Ensure we're raising proper exceptions https://review.opendev.org/670408 | 21:08 |
| *** betherly has quit IRC | 21:09 | |
| mwhahaha | I think this didn't show up in CI because it only affects podman | 21:12 |
| mwhahaha | ovb runs docker | 21:12 |
| cloudnull | interesting. why is that? | 21:12 |
| * mwhahaha shakes his fist at the lack of centos8 striking yet again | 21:12 | |
| mwhahaha | docker is an include of role tasks | 21:13 |
| cloudnull | ha | 21:13 |
| mwhahaha | the upgrade is the upgrade from docker to podman | 21:13 |
| cloudnull | i see | 21:13 |
| mwhahaha | i reproduced it on a podman overcloud | 21:13 |
| mwhahaha | standalone won't hit it because it's 1 node | 21:14 |
| mwhahaha | all the upstream jobs are 1 node | 21:14 |
| mwhahaha | it triggers on >1 | 21:14 |
| * mwhahaha sighs | 21:14 | |
| cloudnull | ok, le sigh | 21:14 |
| mwhahaha | also why the undercloud install was fine but the overcloud hit it | 21:15 |
| openstackgerrit | Ronelle Landy proposed openstack/openstack-virtual-baremetal master: Add requirements for python 3 installs https://review.opendev.org/670667 | 21:21 |
| openstackgerrit | Merged openstack/tripleo-heat-templates master: Lookup pre/post deployment vars with hostvars https://review.opendev.org/669964 | 21:29 |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-upgrade master: Change Upgrade Run Script Generation to Per Host https://review.opendev.org/667610 | 21:29 |
| *** ekultails has quit IRC | 21:32 | |
| *** cfontain has joined #tripleo | 21:34 | |
| * mwhahaha give sup for teh week | 21:35 | |
| cloudnull | did it not work ? | 21:36 |
| *** cfontain has quit IRC | 21:37 | |
| *** holser_ has quit IRC | 21:46 | |
| openstackgerrit | Merged openstack/python-tripleoclient master: Ensure files are closed, ensure no ResourceWarning on console https://review.opendev.org/670576 | 21:47 |
| *** rlandy has quit IRC | 21:54 | |
| *** cfontain has joined #tripleo | 21:55 | |
| openstackgerrit | Ronelle Landy proposed openstack/tripleo-ci master: Add roles/playbooks copy in setup.cfg https://review.opendev.org/670551 | 21:55 |
| *** xek has quit IRC | 22:05 | |
| *** betherly has joined #tripleo | 22:17 | |
| *** cfontain_ has joined #tripleo | 22:19 | |
| *** betherly has quit IRC | 22:21 | |
| *** vinaykns has quit IRC | 22:25 | |
| *** cfontain_ has quit IRC | 22:25 | |
| *** aakarsh has quit IRC | 22:32 | |
| *** cfontain_ has joined #tripleo | 22:41 | |
| *** holser_ has joined #tripleo | 22:41 | |
| *** cfontain_ has quit IRC | 22:45 | |
| *** aakarsh has joined #tripleo | 22:51 | |
| *** cfontain has joined #tripleo | 23:14 | |
| *** betherly has joined #tripleo | 23:19 | |
| *** tosky has quit IRC | 23:20 | |
| *** betherly has quit IRC | 23:24 | |
| *** betherly has joined #tripleo | 23:39 | |
| *** betherly has quit IRC | 23:44 | |
| *** artom has quit IRC | 23:45 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!