openstackgerrit | Merged openstack/diskimage-builder master: Don't use hardcode while override base image file https://review.opendev.org/c/openstack/diskimage-builder/+/771978 | 00:03 |
---|---|---|
*** jmasud has quit IRC | 00:16 | |
*** lbragstad has quit IRC | 00:23 | |
openstackgerrit | Sergii Golovatiuk proposed openstack/tripleo-validations master: Rename Dockerfile.j2 to Dockerfile https://review.opendev.org/c/openstack/tripleo-validations/+/778596 | 00:26 |
*** tosky has quit IRC | 00:28 | |
openstackgerrit | Merged openstack/tripleo-ci master: Deprecate non content provider undercloud zuul template layout https://review.opendev.org/c/openstack/tripleo-ci/+/778700 | 00:30 |
openstackgerrit | Merged openstack/python-tripleoclient stable/victoria: Add --disable-container-prepare flag https://review.opendev.org/c/openstack/python-tripleoclient/+/769739 | 00:30 |
openstackgerrit | John Fulton proposed openstack/tripleo-ansible master: Only create a user if they do not exist https://review.opendev.org/c/openstack/tripleo-ansible/+/779358 | 00:46 |
*** jmasud has joined #tripleo | 00:47 | |
openstackgerrit | John Fulton proposed openstack/tripleo-ansible master: Deploy RGW, MDS, Ganesha and Ceph Dashboard using cephadm https://review.opendev.org/c/openstack/tripleo-ansible/+/773364 | 00:48 |
*** lbragstad has joined #tripleo | 00:52 | |
*** ha1f3mpty has quit IRC | 01:38 | |
openstackgerrit | Merged openstack/diskimage-builder master: Add aarch64 support for rhel https://review.opendev.org/c/openstack/diskimage-builder/+/770703 | 01:52 |
openstackgerrit | Merged openstack/diskimage-builder master: Use the same bootloader pkg-map for all redhat family https://review.opendev.org/c/openstack/diskimage-builder/+/779105 | 02:00 |
openstackgerrit | John Fulton proposed openstack/tripleo-ansible master: Introduce ceph_spec_bootstrap module https://review.opendev.org/c/openstack/tripleo-ansible/+/777767 | 02:05 |
openstackgerrit | Merged openstack/diskimage-builder master: Add efibootmgr utility for UEFI boot menu management https://review.opendev.org/c/openstack/diskimage-builder/+/779106 | 02:11 |
*** jmasud has quit IRC | 02:14 | |
*** jmasud has joined #tripleo | 02:17 | |
*** rh-jelabarre has quit IRC | 02:23 | |
openstackgerrit | Merged openstack/puppet-tripleo master: Remove unnecessary usage of hiera https://review.opendev.org/c/openstack/puppet-tripleo/+/777111 | 02:25 |
*** rcernin has quit IRC | 02:25 | |
openstackgerrit | John Fulton proposed openstack/tripleo-ansible master: Introduce ceph_spec_bootstrap module https://review.opendev.org/c/openstack/tripleo-ansible/+/777767 | 02:30 |
openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Add CephDynamicSpec, CephSpecPath, CephOsdSpec, CephSpecFqdn https://review.opendev.org/c/openstack/tripleo-heat-templates/+/779112 | 02:32 |
*** rcernin has joined #tripleo | 02:42 | |
*** rcernin has quit IRC | 02:48 | |
*** rcernin has joined #tripleo | 02:51 | |
*** Yarboa has quit IRC | 02:53 | |
*** Yarboa has joined #tripleo | 02:56 | |
*** tmazur has quit IRC | 02:58 | |
openstackgerrit | Steve Baker proposed openstack/diskimage-builder master: Don't run grub2-install for efi block devices https://review.opendev.org/c/openstack/diskimage-builder/+/779107 | 03:07 |
*** rcernin has quit IRC | 03:20 | |
*** dmacpher_ has quit IRC | 03:23 | |
*** dmacpher has joined #tripleo | 03:27 | |
*** psachin has joined #tripleo | 03:30 | |
*** jmasud has quit IRC | 03:31 | |
*** rcernin has joined #tripleo | 03:40 | |
*** lbragstad has quit IRC | 03:43 | |
*** zzzeek has quit IRC | 03:46 | |
*** zzzeek has joined #tripleo | 03:47 | |
*** dpawlik6 has joined #tripleo | 03:51 | |
*** rcernin_ has joined #tripleo | 03:51 | |
*** social_ has joined #tripleo | 03:54 | |
*** raukadah has joined #tripleo | 03:54 | |
*** khyr0n_ has joined #tripleo | 03:54 | |
*** dpeacock_ has joined #tripleo | 03:54 | |
*** owalsh_ has joined #tripleo | 03:58 | |
*** rcernin has quit IRC | 03:59 | |
*** hoonetorg has quit IRC | 03:59 | |
*** mjturek has quit IRC | 03:59 | |
*** openstackgerrit has quit IRC | 03:59 | |
*** amoralej|off has quit IRC | 03:59 | |
*** khyr0n has quit IRC | 03:59 | |
*** jbadiapa has quit IRC | 03:59 | |
*** dpeacock has quit IRC | 03:59 | |
*** dmellado has quit IRC | 03:59 | |
*** social has quit IRC | 03:59 | |
*** supamatt has quit IRC | 03:59 | |
*** chandankumar has quit IRC | 03:59 | |
*** leifmadsen has quit IRC | 03:59 | |
*** tbarron has quit IRC | 03:59 | |
*** cgoncalves has quit IRC | 03:59 | |
*** dpawlik has quit IRC | 03:59 | |
*** owalsh has quit IRC | 03:59 | |
*** baha has quit IRC | 03:59 | |
*** dpawlik6 is now known as dpawlik | 03:59 | |
*** leifmadsen has joined #tripleo | 04:01 | |
*** skramaja has joined #tripleo | 04:02 | |
*** dmellado has joined #tripleo | 04:02 | |
*** Yarboa has quit IRC | 04:13 | |
*** Yarboa has joined #tripleo | 04:15 | |
*** jmasud has joined #tripleo | 04:19 | |
*** ykarel has joined #tripleo | 04:23 | |
*** udesale has joined #tripleo | 04:31 | |
*** jmasud has quit IRC | 04:34 | |
*** ratailor has joined #tripleo | 04:34 | |
*** jmasud has joined #tripleo | 04:40 | |
*** saneax has joined #tripleo | 04:48 | |
*** raukadah is now known as chandankumar | 04:51 | |
*** jmasud has quit IRC | 05:12 | |
*** ysandeep|away is now known as ysandeep | 05:16 | |
*** mmethot_ has joined #tripleo | 05:24 | |
*** poopcat has quit IRC | 05:24 | |
*** mmethot has quit IRC | 05:25 | |
*** poopcat has joined #tripleo | 05:26 | |
*** iurygregory has quit IRC | 05:26 | |
*** jmasud has joined #tripleo | 05:40 | |
*** rcernin_ has quit IRC | 05:48 | |
*** rcernin_ has joined #tripleo | 05:49 | |
*** marios has joined #tripleo | 06:06 | |
*** lmiccini has joined #tripleo | 06:26 | |
*** jpodivin has joined #tripleo | 06:40 | |
*** jpodivin has quit IRC | 06:40 | |
*** hakhande has joined #tripleo | 06:40 | |
*** jpodivin has joined #tripleo | 06:40 | |
*** slaweq has joined #tripleo | 06:50 | |
*** bogdando has joined #tripleo | 06:52 | |
*** _mmethot_ has joined #tripleo | 07:00 | |
*** dhill has quit IRC | 07:03 | |
*** mmethot_ has quit IRC | 07:03 | |
*** dhill has joined #tripleo | 07:04 | |
*** amoralej has joined #tripleo | 07:06 | |
*** cgoncalves has joined #tripleo | 07:09 | |
*** jmasud has quit IRC | 07:11 | |
*** xek has joined #tripleo | 07:48 | |
*** rdopiera has joined #tripleo | 08:11 | |
*** jcapitao has joined #tripleo | 08:12 | |
*** gfidente has joined #tripleo | 08:21 | |
*** rpittau|afk is now known as rpittau | 08:24 | |
*** rcernin_ has quit IRC | 08:24 | |
*** cylopez has joined #tripleo | 08:25 | |
*** jbadiapa has joined #tripleo | 08:26 | |
*** apetrich has joined #tripleo | 08:39 | |
*** jpich has joined #tripleo | 08:53 | |
*** jpena|off is now known as jpena | 08:55 | |
*** derekh has joined #tripleo | 09:00 | |
*** tosky has joined #tripleo | 09:02 | |
*** ccamacho has joined #tripleo | 09:03 | |
*** lucasagomes has joined #tripleo | 09:05 | |
marios | ykarel: o/ replied at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778880/1//COMMIT_MSG#1 please check https://review.opendev.org/c/openstack/tripleo-ci/+/779447 | 09:46 |
marios | ykarel: whenever you next have time thank you ! | 09:46 |
*** udesale_ has joined #tripleo | 09:55 | |
ykarel | marios, looking | 09:56 |
*** udesale has quit IRC | 09:59 | |
*** rcernin_ has joined #tripleo | 10:27 | |
*** iurygregory_ has joined #tripleo | 10:45 | |
*** iurygregory_ is now known as iurygregory | 10:46 | |
*** frenzy_friday has quit IRC | 10:55 | |
*** frenzy_friday has joined #tripleo | 10:55 | |
*** marios has quit IRC | 10:56 | |
*** jpich has quit IRC | 10:57 | |
*** jpich has joined #tripleo | 10:58 | |
*** marios has joined #tripleo | 11:03 | |
*** ysandeep is now known as ysandeep|afk | 11:11 | |
sshnaidm | marios, chandankumar https://review.opendev.org/c/openstack/tripleo-repos/+/779489 | 11:23 |
*** dtantsur|afk is now known as dtantsur | 11:25 | |
chandankumar | sshnaidm: thanks :-) | 11:25 |
*** rcernin_ has quit IRC | 11:26 | |
chandankumar | sshnaidm: Can We create a seperate branch for tripleo-repos to push those changes? | 11:30 |
sshnaidm | chandankumar, probably, but I think it's an overhead | 11:31 |
sshnaidm | it's just one-file-script | 11:31 |
chandankumar | sshnaidm: ok | 11:31 |
*** marios has quit IRC | 11:31 | |
*** marios has joined #tripleo | 11:37 | |
*** artom has quit IRC | 11:51 | |
*** slaweq_ has joined #tripleo | 11:52 | |
*** jcapitao is now known as jcapitao_lunch | 11:52 | |
*** slaweq has quit IRC | 11:59 | |
*** ysandeep|afk is now known as ysandeep | 12:01 | |
*** ykarel_ has joined #tripleo | 12:17 | |
*** ykarel has quit IRC | 12:20 | |
*** ykarel_ is now known as ykarel | 12:21 | |
*** artom has joined #tripleo | 12:25 | |
*** jpena is now known as jpena|lunch | 12:34 | |
*** slaweq_ is now known as slaweq | 12:38 | |
*** ayoung has quit IRC | 12:38 | |
*** psachin has quit IRC | 12:47 | |
*** zzzeek has quit IRC | 13:00 | |
*** zzzeek has joined #tripleo | 13:02 | |
*** jcapitao_lunch is now known as jcapitao | 13:03 | |
*** lucasagomes has quit IRC | 13:04 | |
*** rh-jelabarre has joined #tripleo | 13:12 | |
*** amoralej is now known as amoralej|lunch | 13:17 | |
*** lbragstad has joined #tripleo | 13:24 | |
*** tmazur has joined #tripleo | 13:28 | |
*** jpena|lunch is now known as jpena | 13:36 | |
*** Yarboa has quit IRC | 13:44 | |
*** Yarboa has joined #tripleo | 13:45 | |
*** ratailor has quit IRC | 13:53 | |
*** tkajinam has quit IRC | 13:53 | |
*** amoralej|lunch is now known as amoralej | 14:01 | |
spotz | Just took the WIP off https://review.opendev.org/c/openstack/tripleo-docs/+/774281 not sure why it didn't send a message to channel though | 14:02 |
*** ade_lee has quit IRC | 14:06 | |
*** ade_lee has joined #tripleo | 14:06 | |
*** lmiccini has quit IRC | 14:10 | |
*** lucasagomes has joined #tripleo | 14:16 | |
*** lucasagomes has quit IRC | 14:16 | |
*** lucasagomes has joined #tripleo | 14:17 | |
*** jpich has quit IRC | 14:17 | |
*** jpich has joined #tripleo | 14:18 | |
*** pcaruana has quit IRC | 14:19 | |
fultonj | gfidente: i'm looking at https://review.opendev.org/c/openstack/tripleo-ansible/+/779358 w/ fmount | 14:26 |
fultonj | Nothing about the ansible user module is idempotent, regardless of if the local paramter is true or false. Thus, the "check if user exists" task on line 18 was introduced to wrap the non-idempotent ask in a conditional. Is there a reason we can't use local true? | 14:26 |
fultonj | Prior to adding local:true this was failing AND /home/ didn't have a ceph-admin director but /etc/passwd did. | 14:26 |
gfidente | fultonj no I think local:true was also an option to fix the permissions issue | 14:27 |
gfidente | I missed the other check to "workaround" idempotency | 14:27 |
fultonj | https://review.opendev.org/c/openstack/tripleo-ansible/+/779358/3/tripleo_ansible/roles/tripleo_create_admin/tasks/create_user.yml#17 | 14:27 |
fultonj | the behavior is strange | 14:28 |
gfidente | no wait a sec | 14:28 |
fmount | gfidente: fultonj local:true is meant for user creation, not for permissions | 14:29 |
gfidente | fmount yeah but as side effect it also didn't have the permissions issue | 14:29 |
gfidente | anyway | 14:29 |
fmount | yeah | 14:29 |
fmount | I mean, my question is: can we use local: true to fix the "user exists" issue? | 14:30 |
fmount | without having additional tasks ? | 14:30 |
fmount | """This will check /etc/passwd for an existing account before invoking commands. If the local account database exists somewhere other than /etc/passwd, this setting will not work properly.""" | 14:30 |
gfidente | right but I don't think it worked in the scenario in which the user existed already | 14:31 |
fultonj | also, the goal for this user is to have a working ssh connection and key in their home dir | 14:31 |
gfidente | sorry, it didn't work | 14:31 |
fmount | I run it multiple times (and on a multinode) | 14:31 |
fmount | gfidente: https://paste.centos.org/view/0061dc4f | 14:31 |
fultonj | do we know what ansible strategy this task is running with? | 14:31 |
fmount | sorry: https://paste.centos.org/view/655c5fd3 | 14:32 |
mwhahaha | cloudnull: is openstack_ports a fact? https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_134/779320/1/check/tripleo-ci-centos-8-containers-multinode/134f327/logs/undercloud/home/zuul/overcloud_deploy.log | 14:33 |
fultonj | fmount: as per https://paste.centos.org/view/655c5fd3, are you saying local:true seems to make it idempotent? | 14:34 |
fmount | yeah | 14:34 |
fmount | because as per the local parameter definition (in the doc), it checks /etc/passwd | 14:35 |
fmount | before running the cmd | 14:35 |
fultonj | makes sense | 14:35 |
fultonj | :) | 14:35 |
fultonj | let's try removing the id command and when | 14:35 |
fultonj | we'll just add local:true | 14:35 |
fultonj | and recheck 294 | 14:35 |
fmount | makes sense | 14:36 |
fmount | gfidente: wdyt | 14:36 |
cloudnull | mwhahaha WAT... reading | 14:36 |
mwhahaha | 2021-03-08 21:53:47 | 2021-03-08 21:53:47.377909 | | DEPRECATED | The 'os_port_facts' module has been renamed to 'os_port_info', and the renamed one no longer returns ansible_facts | 14:37 |
mwhahaha | might be related | 14:37 |
fmount | fultonj: 586 merged, we can rebase 358 then | 14:37 |
fultonj | yay | 14:37 |
cloudnull | mwhahaha what's interesting is that "openstack_ports" object isn't used in the calling playbook, https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml | 14:39 |
cloudnull | so I suspect its something internal to a module ? | 14:39 |
mwhahaha | cloudnull: it's no longer in master, that's stable | 14:39 |
mwhahaha | stable/victoria and ussuri | 14:39 |
cloudnull | ah | 14:39 |
fultonj | fmount: i'm submitting the change and regbasing it | 14:39 |
fultonj | gfidente: ack? | 14:39 |
cloudnull | OIC https://github.com/openstack/tripleo-ansible/blob/stable/ussuri/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L185 | 14:40 |
cloudnull | so I guess the stable modules are anything but :P | 14:40 |
fmount | fultonj: ack +1 | 14:41 |
mwhahaha | cloudnull: so i think it is a fact, just a weird naming | 14:41 |
cloudnull | https://docs.ansible.com/ansible/2.9/modules/os_port_facts_module.html | 14:42 |
cloudnull | so even the alias should return the openstack_ports fact | 14:43 |
cloudnull | unless we're pulling the collection for 2.10+ somehow? | 14:43 |
cloudnull | https://docs.ansible.com/ansible/2.9/modules/os_port_info_module.html#os-port-info-module | 14:43 |
mwhahaha | cloudnull: that failure was on the patch to drop the inject vars | 14:43 |
mwhahaha | cloudnull: so it didn't show up as something needing converting, will fix | 14:43 |
cloudnull | oh | 14:43 |
mwhahaha | though I guess the best solution is to not use the fact but a register? | 14:44 |
cloudnull | ++ | 14:44 |
*** artom has quit IRC | 14:49 | |
*** ysandeep is now known as ysandeep|afk | 14:50 | |
mwhahaha | fultonj: gfidente, https://review.opendev.org/c/openstack/tripleo-ansible/+/777586 broke rdo | 14:54 |
mwhahaha | though i question why it's still compiling under py2.7 | 14:55 |
fmount | mwhahaha: exactly, that doesn't work w/ py2.7 but I assumed it wasn't used anymore | 14:56 |
mwhahaha | looks like stein is not correctly configured | 14:57 |
mwhahaha | it's been foillowing master | 14:57 |
mwhahaha | even though the package wasn't used until train | 14:57 |
mwhahaha | will need to fix in rdo later i guess | 14:57 |
fmount | I think so ^ | 14:57 |
fultonj | mwhahaha: i'm reading this as we don't need to revert 777586 since rdo should be changed. is that ok? | 15:01 |
mwhahaha | yea | 15:01 |
fultonj | thanks mwhahaha | 15:02 |
fultonj | since we're chatting do you see issues with this | 15:02 |
fultonj | https://review.opendev.org/c/openstack/tripleo-ansible/+/779358 | 15:02 |
fmount | mwhahaha: the local:true story ^ | 15:02 |
fmount | on user creation | 15:02 |
mwhahaha | should be ok? | 15:03 |
fultonj | mwhahaha: ok, thanks | 15:03 |
*** pcaruana has joined #tripleo | 15:04 | |
fultonj | fmount: i'm rebasing the stack on 779358 and we'll recheck the 001 change | 15:04 |
fmount | fultonj: ack I agree | 15:05 |
fmount | let's do that and recheck 294 | 15:05 |
fmount | thanks fultonj | 15:05 |
csibbitt | cloudnull, any idea what I should try next with this package/user problem on https://review.opendev.org/c/openstack/tripleo-ansible/+/772920 ? Did you manage to prove that it's podman 3 vs 2 that is the problem? | 15:05 |
cloudnull | csibbitt I had it running as I signed off yesterday, let me check to see what's what | 15:12 |
*** mjturek has joined #tripleo | 15:13 | |
*** ysandeep|afk is now known as ysandeep|away | 15:34 | |
*** ykarel has quit IRC | 15:43 | |
marios | Tengu: gchamoul: do we have a validation framework core group in gerrit? | 15:44 |
gchamoul | marios: nope I don't think so | 15:45 |
*** supamatt has joined #tripleo | 15:45 | |
marios | gchamoul: ack thanks for sanity check I couldn't find one | 15:46 |
marios | gchamoul: can you reply here so it is clear please http://lists.openstack.org/pipermail/openstack-discuss/2021-March/020949.html | 15:47 |
marios | gchamoul: i think you mean what i wrote there | 15:47 |
zbr | cloudnull: can you please look at https://review.opendev.org/c/openstack/tripleo-ansible/+/779180 ? (before we need to rebase it again) | 15:47 |
*** udesale_ has quit IRC | 15:48 | |
Tengu | marios: nope. it's one of those "trust the people" case :) | 15:48 |
marios | Tengu: ack yeah i thought so but i think to be fair we should be explicit about what we mean | 15:48 |
Tengu | yup | 15:48 |
cloudnull | zbr done | 15:48 |
cloudnull | csibbitt still hacking, trying to figure out what is going on. | 15:49 |
csibbitt | cloudnull, thanks again for the assistance. I'll see if I can fit in some time today to get my own clean podman3 environment so I can test that theory as well and bang on it if it's broken. | 15:50 |
zbr | cloudnull: thanks. I have a change adding tripleo-ansible in the canary testing of ansible-lint, so we prevent surprises. | 15:50 |
cloudnull | ++ | 15:50 |
gchamoul | marios: answered! | 15:52 |
marios | gchamoul: thank you | 15:52 |
*** openstackgerrit has joined #tripleo | 16:04 | |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: Make UpgradeInitCommand and UpgradeLeapp{ToRemove,ToInstall,CommandOptions} per-role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/779522 | 16:04 |
*** frenzy_friday is now known as frenzyfriday|brb | 16:07 | |
*** strattao has joined #tripleo | 16:08 | |
openstackgerrit | Jiri Podivin proposed openstack/tripleo-validations master: Test folder structure rearranged. https://review.opendev.org/c/openstack/tripleo-validations/+/778986 | 16:09 |
*** jpodivin has quit IRC | 16:09 | |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Set tags on all OS::Neutron::Port resources https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778792 | 16:13 |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Add tripleo_network_name tag hint to networks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/779502 | 16:13 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: Make UpgradeInitCommand and UpgradeLeapp{ToRemove,ToInstall,CommandOptions} per-role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/779522 | 16:16 |
*** cylopez has quit IRC | 16:17 | |
openstackgerrit | Merged openstack/tripleo-ci master: Moves files: overrides from tripleo-common to tripleo-ci template https://review.opendev.org/c/openstack/tripleo-ci/+/778901 | 16:17 |
openstackgerrit | Carlos Gonçalves proposed openstack/tripleo-heat-templates master: Add FRR service https://review.opendev.org/c/openstack/tripleo-heat-templates/+/763657 | 16:20 |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Set tags on all OS::Neutron::Port resources https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778792 | 16:23 |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Add tripleo_network_name tag hint to networks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/779502 | 16:23 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config master: Bump hacking https://review.opendev.org/c/openstack/os-collect-config/+/779527 | 16:24 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/victoria: Bump hacking https://review.opendev.org/c/openstack/os-collect-config/+/779528 | 16:25 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/victoria: Update .gitreview for stable/ussuri https://review.opendev.org/c/openstack/os-collect-config/+/779529 | 16:25 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/c/openstack/os-collect-config/+/779530 | 16:25 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/victoria: Bump hacking https://review.opendev.org/c/openstack/os-collect-config/+/779528 | 16:27 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/c/openstack/os-collect-config/+/770772 | 16:27 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/c/openstack/os-collect-config/+/770773 | 16:27 |
*** hakhande has quit IRC | 16:28 | |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/c/openstack/os-collect-config/+/773653 | 16:33 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/c/openstack/os-collect-config/+/773654 | 16:33 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/ussuri: Bump hacking https://review.opendev.org/c/openstack/os-collect-config/+/779531 | 16:33 |
openstackgerrit | Alex Schultz proposed openstack/os-refresh-config stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/c/openstack/os-refresh-config/+/770774 | 16:37 |
openstackgerrit | Alex Schultz proposed openstack/os-refresh-config stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/c/openstack/os-refresh-config/+/770775 | 16:37 |
openstackgerrit | Alex Schultz proposed openstack/os-refresh-config stable/victoria: Remove tripleo-multinode-container-minimal from layout + fix test https://review.opendev.org/c/openstack/os-refresh-config/+/779532 | 16:37 |
openstackgerrit | Alex Schultz proposed openstack/os-collect-config stable/victoria: Add Python3 victoria unit tests https://review.opendev.org/c/openstack/os-collect-config/+/779533 | 16:38 |
*** jmasud has joined #tripleo | 16:39 | |
*** amoralej is now known as amoralej|off | 16:42 | |
*** jpich has quit IRC | 16:45 | |
*** jpich has joined #tripleo | 16:46 | |
*** jcapitao has quit IRC | 16:47 | |
openstackgerrit | Bhagyashri Shewale proposed openstack/python-tripleoclient master: Revert "Revert "Revert "Add environment variables for better var handling""" https://review.opendev.org/c/openstack/python-tripleoclient/+/779474 | 16:48 |
mwhahaha | bhagyashri|rover: -^ what's failing? | 16:49 |
openstackgerrit | Bhagyashri Shewale proposed openstack/python-tripleoclient master: [DNM] Revert "Revert "Revert "Add environment variables for better var handling""" https://review.opendev.org/c/openstack/python-tripleoclient/+/779474 | 16:49 |
bhagyashri|rover | mwhahaha, https://logserver.rdoproject.org/51/29351/23/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-master/4043efe/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz | 16:50 |
mwhahaha | i don't think that's related | 16:50 |
mwhahaha | we've seen keystone endpoint problems elsewhere | 16:50 |
mwhahaha | but sure let's see if that affects that | 16:51 |
bhagyashri|rover | mwhahaha, ack, just let me test once if the issue is still then will abandon my patch ^ | 16:52 |
bhagyashri|rover | if the issue will still there ...* | 16:53 |
mwhahaha | nice to see that's leaking the passwords in failure too | 16:53 |
* mwhahaha sighs | 16:53 | |
openstackgerrit | James Slagle proposed openstack/python-tripleoclient master: Add HeatPodLauncher https://review.opendev.org/c/openstack/python-tripleoclient/+/774573 | 16:53 |
openstackgerrit | James Slagle proposed openstack/python-tripleoclient master: Update tripleo launch heat for HeatPodLauncher https://review.opendev.org/c/openstack/python-tripleoclient/+/774574 | 16:53 |
openstackgerrit | James Slagle proposed openstack/python-tripleoclient master: Support using ephemeral Heat https://review.opendev.org/c/openstack/python-tripleoclient/+/769984 | 16:53 |
openstackgerrit | James Slagle proposed openstack/python-tripleoclient master: Add enable_neutron and enable_heat https://review.opendev.org/c/openstack/python-tripleoclient/+/777256 | 16:53 |
bhagyashri|rover | mwhahaha, yeah | 16:53 |
openstackgerrit | James Slagle proposed openstack/python-tripleoclient master: Automatically run overcloud export when using ephemeral heat https://review.opendev.org/c/openstack/python-tripleoclient/+/777494 | 16:53 |
*** ade_lee has quit IRC | 16:53 | |
bhagyashri|rover | that point i also surprised | 16:53 |
mwhahaha | i'm not, ansible likes to dump stuff | 16:53 |
*** ade_lee has joined #tripleo | 16:53 | |
bhagyashri|rover | hmm | 16:54 |
*** marios is now known as marios|out | 16:56 | |
*** frenzyfriday|brb is now known as frenzy_friday | 16:58 | |
*** lucasagomes has quit IRC | 17:02 | |
openstackgerrit | Merged openstack/tripleo-common stable/ussuri: Removes unneeded zuul layout overrides - templates for consistency https://review.opendev.org/c/openstack/tripleo-common/+/779208 | 17:03 |
openstackgerrit | Merged openstack/tripleo-common stable/train: Removes unneeded zuul layout overrides - templates for consistency https://review.opendev.org/c/openstack/tripleo-common/+/779212 | 17:04 |
openstackgerrit | Merged openstack/tripleo-quickstart master: Updating promotion testing hash victoria to c8 stream repos and baseos image https://review.opendev.org/c/openstack/tripleo-quickstart/+/778557 | 17:04 |
*** rpittau is now known as rpittau|afk | 17:05 | |
openstackgerrit | Carlos Gonçalves proposed openstack/tripleo-heat-templates master: Add FRR service https://review.opendev.org/c/openstack/tripleo-heat-templates/+/763657 | 17:05 |
*** langdon has quit IRC | 17:06 | |
*** langdon has joined #tripleo | 17:11 | |
*** saneax has quit IRC | 17:12 | |
*** jpich has quit IRC | 17:24 | |
openstackgerrit | Merged openstack/tripleo-ansible stable/train: Modifying playbooks to support python-tripleoclient B&R commands https://review.opendev.org/c/openstack/tripleo-ansible/+/778214 | 17:28 |
openstackgerrit | Merged openstack/tripleo-ansible stable/victoria: Modifying playbooks to support python-tripleoclient B&R commands https://review.opendev.org/c/openstack/tripleo-ansible/+/777539 | 17:29 |
fultonj | [5~ | 17:30 |
openstackgerrit | Daniel Bengtsson proposed openstack/tripleo-upgrade master: [WIP] Added validations to updates https://review.opendev.org/c/openstack/tripleo-upgrade/+/773181 | 17:37 |
*** gfidente is now known as gfidente|afk | 17:47 | |
cloudnull | csibbitt - Failed to mount container qdrtest: b'Error: cannot run command "podman mount" in rootless mode, must execute `podman unshare` first' -- So i think this is the root cause of the failure to install the packages. | 17:51 |
cloudnull | running debug on a local execution w/ podman 2 | 17:51 |
openstackgerrit | Merged openstack/tripleo-specs master: remove unicode from code https://review.opendev.org/c/openstack/tripleo-specs/+/769286 | 17:52 |
cloudnull | so I think this may be an issue with molecule | 17:53 |
cloudnull | -cc zbr | 17:53 |
csibbitt | cloudnull, so this should be affecting every role that uses test_deps and I'm just the first canary? | 17:54 |
cloudnull | i guess so ? | 17:54 |
cloudnull | I have a reproducer I can start a tmate if you're interested? | 17:54 |
mwhahaha | i didn't think we ran molecule rootless? | 17:55 |
openstackgerrit | Merged openstack/tripleo-common master: Removes unneeded zuul layout overrides - templates for consistency https://review.opendev.org/c/openstack/tripleo-common/+/778900 | 17:56 |
cloudnull | I think we do w/ zuul | 17:56 |
cloudnull | or at least I'm able to reproduce it w/ rootless locally | 17:56 |
mwhahaha | it shouldn't be rootless? | 17:56 |
cloudnull | probably not. | 17:57 |
cloudnull | let me retry it w/ root | 17:57 |
*** derekh has quit IRC | 18:00 | |
*** artom has joined #tripleo | 18:01 | |
*** jpena is now known as jpena|off | 18:02 | |
*** dprince has joined #tripleo | 18:03 | |
*** jmasud has quit IRC | 18:11 | |
*** jmasud has joined #tripleo | 18:12 | |
*** dtantsur is now known as dtantsur|afk | 18:13 | |
cloudnull | so privileged containers are failing a different way - Status code was -1 and not [200]: Request failed: <urlopen error _ssl.c:880: The handshake operation timed out> | 18:18 |
cloudnull | which is exciting | 18:18 |
cloudnull | and podman 3 has yet another failure | 18:18 |
cloudnull | podman 3 Error: container_linux.go:370: starting container process caused: process_linux.go:459: container init caused: rootfs_linux.go:59: mounting "sysfs" to rootfs at "/sys" caused: operation not permitted: OCI permission denied | 18:19 |
cloudnull | so its a good time all around | 18:19 |
*** tovchinn has joined #tripleo | 18:22 | |
*** bogdando has quit IRC | 18:23 | |
*** tmazur has quit IRC | 18:23 | |
*** marios|out has quit IRC | 18:26 | |
openstackgerrit | Alex Schultz proposed openstack/os-apply-config master: Add Python3 wallaby unit tests https://review.opendev.org/c/openstack/os-apply-config/+/769124 | 18:42 |
*** frenzy_friday has quit IRC | 18:44 | |
*** frenzy_friday has joined #tripleo | 18:44 | |
csibbitt | I saw holser asking about an "Error: failed to mount shm tmpfs" that was permission denied as well. I wonder if they are related | 18:53 |
weshay|ruck | mwhahaha, looking at some upgrade jobs that one of your patches is failing on.. | 19:17 |
weshay|ruck | https://fee09b73e50977935895-b79c57b376466cab8e443243a2295837.ssl.cf2.rackcdn.com/779319/2/check/tripleo-ci-centos-8-standalone-upgrade-victoria/f5bc7fa/job-output.txt | 19:17 |
weshay|ruck | "msg": "Timeout when waiting for 198.72.124.102:8766" | 19:18 |
weshay|ruck | that may be what is biting you | 19:18 |
holser | csibbitt weshay|ruck let me have a look | 19:18 |
mwhahaha | weshay|ruck: fun | 19:18 |
weshay|ruck | it's a content-provider thing | 19:18 |
weshay|ruck | holser, ^ | 19:19 |
holser | weshay|ruck Should I close a bug? | 19:19 |
weshay|ruck | don't think there is a bug on this issue yet | 19:19 |
*** jmasud has quit IRC | 19:27 | |
weshay|ruck | https://bugs.launchpad.net/tripleo/+bug/1918320 | 19:28 |
openstack | Launchpad bug 1918320 in tripleo "tripleo content-providers repo check failing" [Critical,Triaged] | 19:28 |
weshay|ruck | looking into it | 19:28 |
*** mandre has joined #tripleo | 19:30 | |
*** rdopiera has quit IRC | 19:30 | |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: wait a little longer while checking content-provider ip https://review.opendev.org/c/openstack/tripleo-ci/+/779569 | 19:39 |
openstackgerrit | Emma Foley proposed openstack/tripleo-ansible master: Ansible for collectd https://review.opendev.org/c/openstack/tripleo-ansible/+/776709 | 19:41 |
openstackgerrit | Emma Foley proposed openstack/tripleo-ansible master: [WIP] Remove dependency on github repo for tripleo_collectd https://review.opendev.org/c/openstack/tripleo-ansible/+/777435 | 19:41 |
*** lbragstad has quit IRC | 20:01 | |
*** jmasud has joined #tripleo | 20:08 | |
openstackgerrit | Emma Foley proposed openstack/tripleo-ansible master: [WIP] Remove dependency on github repo for tripleo_collectd https://review.opendev.org/c/openstack/tripleo-ansible/+/777435 | 20:14 |
*** lbragstad has joined #tripleo | 20:15 | |
openstackgerrit | Merged openstack/tripleo-ansible stable/victoria: Fixed and rename task name, which does the opposite https://review.opendev.org/c/openstack/tripleo-ansible/+/777333 | 20:20 |
*** jamesden_ is now known as jamesdenton | 20:20 | |
*** jamesdenton has quit IRC | 20:21 | |
*** jamesden_ has joined #tripleo | 20:21 | |
*** jamesden_ is now known as jamesdenton | 20:22 | |
openstackgerrit | Carlos Gonçalves proposed openstack/tripleo-ansible master: WIP: tripleo_ovn_cluster role https://review.opendev.org/c/openstack/tripleo-ansible/+/776969 | 20:28 |
openstackgerrit | Carlos Gonçalves proposed openstack/tripleo-heat-templates master: WIP: OVN DB clustering https://review.opendev.org/c/openstack/tripleo-heat-templates/+/775444 | 20:29 |
*** jamesdenton has quit IRC | 20:29 | |
*** dmacpher has quit IRC | 20:30 | |
*** dmacpher has joined #tripleo | 20:30 | |
*** irclogbot_1 has quit IRC | 20:31 | |
*** irclogbot_3 has joined #tripleo | 20:34 | |
*** dprince has quit IRC | 20:35 | |
*** jamesdenton has joined #tripleo | 20:38 | |
openstackgerrit | Merged openstack/os-net-config master: Updating sriov_config service to run after openibd service https://review.opendev.org/c/openstack/os-net-config/+/779445 | 20:40 |
openstackgerrit | Merged openstack/tripleo-ci master: Add ansible-generated to collect logs list https://review.opendev.org/c/openstack/tripleo-ci/+/779094 | 20:48 |
openstackgerrit | waleed mousa proposed openstack/os-net-config stable/victoria: Updating sriov_config service to run after openibd service https://review.opendev.org/c/openstack/os-net-config/+/779599 | 20:51 |
*** jamesdenton has quit IRC | 20:51 | |
openstackgerrit | waleed mousa proposed openstack/os-net-config stable/ussuri: Updating sriov_config service to run after openibd service https://review.opendev.org/c/openstack/os-net-config/+/779600 | 20:52 |
openstackgerrit | waleed mousa proposed openstack/os-net-config stable/train: Updating sriov_config service to run after openibd service https://review.opendev.org/c/openstack/os-net-config/+/779601 | 20:52 |
*** bandini has quit IRC | 20:55 | |
weshay|ruck | this is also busting up victoria upgrades https://bugs.launchpad.net/tripleo/+bug/1918325 | 21:03 |
openstack | Launchpad bug 1918325 in tripleo "victoria updates fail w/ Problem: package libknet1-compress-bzip2-plugin-1.16-1.el8.x86_64 requires libknet1(x86-64) = 1.16-1.el8, but none of the providers can be installed" [Critical,Triaged] - Assigned to Sofer Athlan-Guyot (sofer-athlan-guyot) | 21:03 |
weshay|ruck | ussuri -> victoria | 21:03 |
*** Yarboa has quit IRC | 21:13 | |
openstackgerrit | Merged openstack/tripleo-ansible master: Baremetal deploy - add network_config schema https://review.opendev.org/c/openstack/tripleo-ansible/+/772359 | 21:14 |
*** Yarboa has joined #tripleo | 21:16 | |
*** jmasud has quit IRC | 21:18 | |
fultonj | https://bugs.launchpad.net/tripleo/+bug/1918188 | 21:19 |
openstack | Launchpad bug 1918188 in tripleo "tripleo-ansible role tripleo_create_admin needs idempotence protection" [High,In progress] - Assigned to John Fulton (jfulton-org) | 21:19 |
fultonj | cloudnull: i think i found the root cause of ^ but I'd like your advice | 21:19 |
fultonj | could I get your opinion on what i've discovered so far? | 21:20 |
*** jmasud has joined #tripleo | 21:26 | |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: Add code comments on why content-provider check https://review.opendev.org/c/openstack/tripleo-ci/+/779569 | 21:27 |
cloudnull | looking | 21:28 |
fultonj | cloudnull: https://bugs.launchpad.net/tripleo/+bug/1918188/comments/2 | 21:31 |
openstack | Launchpad bug 1918188 in tripleo "tripleo-ansible role tripleo_create_admin needs idempotence protection" [High,In progress] - Assigned to John Fulton (jfulton-org) | 21:31 |
fultonj | i just added a comment on what i'm seeing | 21:31 |
*** ccamacho has quit IRC | 21:32 | |
fultonj | cloudnull: the output of the playbook run is host1: failed=0 and host2: failed=1 but it doesn't know that host1 IS host2 | 21:35 |
cloudnull | where is the task? | 21:35 |
fultonj | https://review.opendev.org/c/openstack/tripleo-ansible/+/779358/10/tripleo_ansible/roles/tripleo_create_admin/tasks/create_user.yml | 21:36 |
fultonj | which is called by https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_run_cephadm/tasks/enable_ceph_admin_user.yml | 21:40 |
openstackgerrit | Brian Haley proposed openstack/openstack-tempest-skiplist master: DNM: do not skip Octavia master jobs https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/778235 | 21:41 |
fultonj | maybe this is the intented behavior though ? | 21:42 |
fultonj | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L17 | 21:42 |
fultonj | what is the purpose of the tripleo_target_host variable? | 21:43 |
fultonj | not sure if you're still looking at this | 21:46 |
fultonj | if there's something important about it including localhost, then I can go back to adding an extra task to check | 21:47 |
mwhahaha | you're hitting standalone where undercloud == standalone | 21:54 |
fultonj | mwhahaha: yes, exactly | 21:55 |
mwhahaha | because we invoke the creation on the udnercloud and overcloud | 21:55 |
mwhahaha | i really am not sure that cli-enable-ssh-admin should be consumed anywhere but in tripleoclient | 21:56 |
mwhahaha | tbh i don't think we should have ever made cli playbooks but that's already passed | 21:57 |
fultonj | mwhahaha: at the PTG we talked about transitioning from deploying ceph _during_ deployment to _before_ | 21:58 |
mwhahaha | yea i get that, but why are we using that playbook | 21:58 |
mwhahaha | at all | 21:59 |
mwhahaha | cli-* playbooks should only be run from within tripleoclient | 21:59 |
mwhahaha | not via a role | 21:59 |
mwhahaha | because then you don't run into this target problem | 21:59 |
fultonj | that playbook creates users on the overcloud so ansible can SSH | 22:00 |
fultonj | and i need to create a user on the overcloud so ansible can SSH | 22:00 |
fultonj | so i used it | 22:00 |
fultonj | when we move to "before deployment" the plan is to use it there too | 22:01 |
mwhahaha | cli-* are wrapped in a tripleoclient command | 22:01 |
fultonj | but since we're still on "during" we're invoking it | 22:01 |
mwhahaha | and should not be invoked outside of that context | 22:01 |
cloudnull | its odd that its running twice for the same host, like the inventory is being redefined ? | 22:02 |
mwhahaha | cloudnull: we defined undercloud and standalone as seperate hosts | 22:02 |
cloudnull | i.e. host1 IS host2 with different names ? | 22:02 |
mwhahaha | because reusability | 22:02 |
cloudnull | ah . | 22:02 |
mwhahaha | so what you have to do is target a specific host, not all | 22:02 |
cloudnull | ++ | 22:02 |
mwhahaha | you're not qualifying just the stack (which we do in the cli) | 22:02 |
mwhahaha | which is why you are hitting this | 22:02 |
mwhahaha | so it's being run against all | 22:03 |
mwhahaha | https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/workflows/deployment.py#L215-L230 | 22:04 |
mwhahaha | we run it against specific hosts | 22:04 |
mwhahaha | which isn't what you are doing in the role | 22:04 |
openstackgerrit | David Vallee Delisle proposed openstack/tripleo-heat-templates master: WIP: Define TSX flag on Computes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/775729 | 22:06 |
fultonj | "target a specific host, not all" | 22:06 |
fultonj | you mean set tripleo_target_host? | 22:07 |
fultonj | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L17 | 22:07 |
cloudnull | fultonj I think you need to set ssh_servers to make it go | 22:07 |
fultonj | i am setting that | 22:07 |
cloudnull | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_run_cephadm/tasks/enable_ceph_admin_user.yml#L27 | 22:07 |
fultonj | https://f14d9b704a9d947abb07-39d40a512ecd567406090b9a01c1d98a.ssl.cf1.rackcdn.com/767294/64/check/tripleo-ci-centos-8-scenario001-standalone/5891a47/logs/undercloud/home/zuul/standalone-ansible-nt6zd6np/cephadm/ceph-admin.yml | 22:08 |
fultonj | ssh_servers: [standalone] | 22:08 |
fultonj | target_host is defaulting to localhost | 22:08 |
cloudnull | but with the same name matching two hosts in inventory I think you would have to set it to standalone[0] | 22:09 |
fultonj | wow | 22:09 |
mwhahaha | i think you need ip addresses | 22:09 |
mwhahaha | not hostname | 22:09 |
mwhahaha | yea ssh_servers is expected to be IPs | 22:11 |
mwhahaha | https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/workflows/deployment.py#L104 | 22:11 |
cloudnull | with the inclusion of the inventory the names should work , | 22:12 |
cloudnull | though yes, that object does expect ip addresses | 22:12 |
mwhahaha | we invoke it with IPs | 22:12 |
mwhahaha | so no guarantee that inventory will work | 22:12 |
mwhahaha | that's just setting up for bugs later | 22:12 |
cloudnull | fair | 22:13 |
openstackgerrit | wes hayutin proposed openstack/openstack-tempest-skiplist master: add victoria TestSnapShotPattern to skip https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/779617 | 22:13 |
*** Yarboa has quit IRC | 22:14 | |
fultonj | you still get the double effect with an IP | 22:14 |
mwhahaha | you shouldn't | 22:14 |
fultonj | http://paste.openstack.org/show/803409/ | 22:15 |
fultonj | so tripleo_target_host is not defined | 22:16 |
fultonj | so hosts is localhost | 22:16 |
fultonj | hosts: "{{ tripleo_target_host | default('localhost') }}" | 22:16 |
mwhahaha | yea this is the problem with invoking the playbook via a playbook | 22:16 |
cloudnull | it does run on localhost which is used to interact with the openstack clients | 22:16 |
*** Yarboa has joined #tripleo | 22:16 | |
cloudnull | but when that playbook runs, it does it with a dynamic inventory | 22:16 |
fultonj | i didn't invoke it with a playbook just now though | 22:16 |
cloudnull | so if you set the ip address, it should only target the one node ? | 22:17 |
fultonj | ansible-playbook -i standalone_inventory.yml /home/stack/tripleo-ansible/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml -e @ceph-admin.yml | 22:17 |
fultonj | distribute_private_key: true | 22:17 |
fultonj | ssh_servers: | 22:17 |
fultonj | - 192.168.24.1 | 22:17 |
fultonj | tripleo_admin_user: ceph-admin | 22:17 |
fultonj | those 4 lines were in @ceph-admin.yml | 22:18 |
fultonj | should i be setting tripleo_target_host to something other than localhost ? | 22:18 |
cloudnull | oh https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L185 | 22:18 |
fultonj | i will switch to IPs like you said | 22:18 |
cloudnull | maybe set ssh_servers: [] | 22:19 |
cloudnull | becase in this case you only ever want it to run once on the local host | 22:19 |
fultonj | cloudnull: that did it | 22:19 |
fultonj | ssh_servers: [] | 22:19 |
cloudnull | ++ | 22:19 |
fultonj | this is strange | 22:20 |
fultonj | for standalone ci we can override it to ssh_servers [] | 22:20 |
mwhahaha | i don't think that's a good idea | 22:20 |
cloudnull | we generate the group here - https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L171-L181 | 22:20 |
mwhahaha | like i really don't think we should be reusing that playbook | 22:20 |
mwhahaha | like really really don't think it's a good idea | 22:20 |
cloudnull | and the target is always "localhost:$dynamic_group" | 22:21 |
cloudnull | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L185 | 22:21 |
fultonj | when tripleo-clinet invokes it for tripleo-admin | 22:21 |
fultonj | it could be invoked then for ceph-admin too if ceph is being deployed | 22:21 |
fultonj | mwhahaha: how about that ^ ? | 22:22 |
mwhahaha | so it's happening because we don't run that playbook for standalone | 22:22 |
fultonj | right! | 22:22 |
mwhahaha | it's done via workflow/deployment.py because it's *only* for multinode | 22:22 |
fultonj | right | 22:22 |
mwhahaha | which is why we don't have this elsewhere | 22:22 |
fultonj | yeah | 22:22 |
mwhahaha | while you can get it working, it points to an issue with the implementation | 22:22 |
cloudnull | ++ its not the most stable interface do to it only being run with the client | 22:23 |
mwhahaha | i think that likely needs to get addressed | 22:23 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates master: Check Ceph cluster healthy state before starting FS to BS playbook https://review.opendev.org/c/openstack/tripleo-heat-templates/+/775600 | 22:24 |
*** rcernin has joined #tripleo | 22:24 | |
fultonj | [17:21] <fultonj> when tripleo-clinet invokes it for tripleo-admin | 22:24 |
fultonj | [17:21] <fultonj> it could be invoked then for ceph-admin too if ceph is being deployed | 22:24 |
fultonj | [17:22] <fultonj> mwhahaha: how about that ^ ? | 22:24 |
fultonj | mwhahaha: for multinode, you ok with ^ as it would be consistent with the existing usage? | 22:25 |
mwhahaha | but we don't invoke it for standalone | 22:26 |
mwhahaha | but sure? | 22:26 |
fultonj | i know, i'm talking about multinode | 22:26 |
mwhahaha | i'm concerned about the additional user tho | 22:26 |
fultonj | for standalone i'll do something different | 22:26 |
hjensas | bhagyashri|rover: I don't see a job running in rdo zuul for https://review.opendev.org/c/openstack/python-tripleoclient/+/779474 | 22:26 |
mwhahaha | fultonj: why can't we reuse the tripleo-admin user again? | 22:27 |
fultonj | private key | 22:27 |
fultonj | ceph orchestrator runs on overcloud, not undercloud and uses a private key to mange the other nodes | 22:27 |
mwhahaha | because we also don't want to always create the ceph user | 22:27 |
fultonj | of course not | 22:27 |
hjensas | bhagyashri|rover: I assyme its the check keystone endpoint issue your trying to figure out? | 22:28 |
fultonj | i wasn't going to always create ceph user | 22:28 |
mwhahaha | it feels like we need to make the user creation bits generic such that it's not contained in the playbook but rather a role | 22:28 |
openstackgerrit | Merged openstack/openstack-tempest-skiplist master: add victoria TestSnapShotPattern to skip https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/779617 | 22:28 |
*** jbadiapa has quit IRC | 22:29 | |
*** rcernin has quit IRC | 22:30 | |
*** rcernin has joined #tripleo | 22:30 | |
fultonj | i'm not sure when the user creation bits will become generic in a role instead of a playbook | 22:33 |
mwhahaha | looks like user creation is a role, but it's the other ssh keys tuff that issn't | 22:33 |
mwhahaha | and the problem is the localhost:target_queue in the playbook itself | 22:34 |
mwhahaha | so maybe it's being able to override that? not sure | 22:34 |
fultonj | it's in both | 22:34 |
fultonj | https://review.opendev.org/c/openstack/tripleo-ansible/+/768365 | 22:35 |
fultonj | the ssh key stuff ^ | 22:35 |
fultonj | i could invokes tasks_from the generic role from with new ceph role | 22:36 |
mwhahaha | i mean if you're only doing, https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L206-L210 then it you don't need the playbook? | 22:36 |
fultonj | yeah, that's what i'm thinking | 22:36 |
fultonj | (since you pointed it out) | 22:37 |
fultonj | maybe that will work | 22:37 |
fultonj | "the problem is the localhost:target_queue in the playbook itself" --> so skip the playbook | 22:39 |
mwhahaha | well if it's an external deploy task you need a playbook | 22:39 |
mwhahaha | but you could have your own | 22:39 |
fultonj | if i have my own playbook which borrows heafily from cli-enable-ssh-admin.yaml but omits the problematic localhost:target_queue then i can invoke it instead of cli-enable-ssh-admin.yaml and it can work with standalone or multinode | 22:44 |
mwhahaha | i don't think you need most of that playbook tho? | 22:45 |
fultonj | s/heafily/lightly :) | 22:45 |
fultonj | maybe lines 206-210 | 22:45 |
mwhahaha | aternatively fix that https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-enable-ssh-admin.yaml#L185 and make it a var | 22:45 |
mwhahaha | but i think you only ned the role inclusion | 22:45 |
fultonj | so we figured out why we have the double behavior | 22:46 |
fultonj | thanks mwhahaha cloudnull | 22:46 |
cloudnull | ++ | 22:47 |
openstackgerrit | John Fulton proposed openstack/tripleo-heat-templates master: Revert "Hard code jobs to Nautilus before setting new default to Octopus" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/779605 | 23:00 |
*** tkajinam has joined #tripleo | 23:00 | |
*** ccamacho has joined #tripleo | 23:10 | |
openstackgerrit | Sergii Golovatiuk proposed openstack/tripleo-validations stable/train: Add validation to verify undercloud.conf content. https://review.opendev.org/c/openstack/tripleo-validations/+/775668 | 23:24 |
*** Yarboa has quit IRC | 23:24 | |
*** Yarboa has joined #tripleo | 23:35 | |
*** mandre has quit IRC | 23:38 | |
*** tovchinn has quit IRC | 23:40 | |
*** bdodd has quit IRC | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!