Friday, 2020-09-11

*** zzzeek has joined #tripleo00:02
*** sanjayu__ has quit IRC00:19
*** rcernin has quit IRC00:28
*** rcernin has joined #tripleo00:31
*** ayoung has quit IRC00:49
*** ayoung has joined #tripleo00:50
*** cfontain has joined #tripleo00:57
*** cfontain has quit IRC01:03
*** ayoung has quit IRC01:10
*** ayoung has joined #tripleo01:11
*** lbragstad_ has joined #tripleo01:33
*** lbragstad has quit IRC01:36
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Consume ceph, prom, grafana containers from quay.io  https://review.opendev.org/75011901:47
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common stable/ussuri: Consume ceph, prom, grafana containers from quay.io  https://review.opendev.org/75027801:47
*** ayoung has quit IRC01:54
*** ayoung has joined #tripleo01:55
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart-extras master: Fetch non tripleo containers in provider job and consume it  https://review.opendev.org/74964501:59
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart-extras master: Fetch non tripleo containers in provider job and consume it  https://review.opendev.org/74964501:59
*** ekultails has quit IRC02:04
*** artom has quit IRC02:12
openstackgerritlikui proposed openstack/diskimage-builder master: Stop install unnecessary packages when run tox -edocs  https://review.opendev.org/75115002:15
*** lbragstad_ has quit IRC02:17
*** zzzeek has quit IRC02:27
*** zzzeek has joined #tripleo02:33
*** ayoung has quit IRC02:47
*** ayoung has joined #tripleo03:00
*** udesale has joined #tripleo03:12
*** jmasud has quit IRC03:21
*** jmasud has joined #tripleo03:23
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart-extras master: Fetch non tripleo containers in provider job and consume it  https://review.opendev.org/74964503:28
*** skramaja has joined #tripleo03:42
*** rcernin has quit IRC03:53
*** ykarel|away has joined #tripleo03:54
*** udesale has quit IRC04:13
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-common master: Move get network configs action to utils  https://review.opendev.org/73784704:15
*** rcernin has joined #tripleo04:15
*** ratailor has joined #tripleo04:18
*** ysandeep|away is now known as ysandeep04:24
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-common master: Move get network configs action to utils  https://review.opendev.org/73784704:27
*** evrardjp has quit IRC04:33
*** evrardjp has joined #tripleo04:33
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-common master: Move get network configs action to utils  https://review.opendev.org/73784704:37
*** ykarel|away is now known as ykarel04:38
*** skramaja has quit IRC04:38
*** skramaja has joined #tripleo04:39
*** cloudnull has quit IRC04:51
*** cloudnull has joined #tripleo04:52
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-common master: Move get network configs action to utils  https://review.opendev.org/73784704:56
*** bandini has joined #tripleo05:04
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates master: Make sure we noop the Mysql_ providers  https://review.opendev.org/75067805:05
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates master: Make sure we noop the Rabbitmq_ providers  https://review.opendev.org/75076305:05
*** ratailor has quit IRC05:06
*** ratailor has joined #tripleo05:07
*** marios has joined #tripleo05:08
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-common master: Move get network configs action to utils  https://review.opendev.org/73784705:08
*** ratailor has quit IRC05:11
jpalanis_chandankumar, hi05:20
TenguEmilienM: matter of broken CI, and CI not failing on the patches that were moving installation paths for two packages. promotion is the only place where all those patches were together, apparently.05:31
chandankumarjpalanis_: Hello05:32
*** xek has joined #tripleo05:35
TenguEmilienM: (broken due to docker.io - that made us lose about 2 weeks of rechecks and all)05:36
openstackgerritCédric Jeanneret (Tengu) proposed openstack/python-tripleoclient stable/ussuri: Allow to pass a static inventory to the validator CLI  https://review.opendev.org/75033105:38
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-validations stable/ussuri: New validation: ensure we have access to the right package version  https://review.opendev.org/75062605:44
openstackgerritMarios Andreou proposed openstack/tripleo-quickstart-extras master: WIP pass build_repo_dir + fix tempest cloud name for standalone upgrade  https://review.opendev.org/75044305:47
jpalanis_chandankumar, https://review.opendev.org/#/c/737847/ failing cross repo dependency merge conflict issue but i think rebased correctly.05:49
chandankumarjpalanis_: I am also hitting against my review not sure why05:50
jpalanis_chandankumar, ok05:50
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Consume ceph, prom, grafana containers from quay.io  https://review.opendev.org/75011905:51
*** lmiccini has joined #tripleo05:51
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Consume ceph, prom, grafana containers from quay.io  https://review.opendev.org/75011905:52
*** slaweq has joined #tripleo06:02
openstackgerritBhagyashri Shewale proposed openstack/openstack-tempest-skiplist master: Add octavia test to skip list  https://review.opendev.org/75116206:04
Tenguis the CI running on Windows? "2020-09-10 09:17:06.162987 | secondary | ERROR: All items complete"   sounds like the nice "ERROR: no error" to me -.-06:13
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-validations master: Correct how the image_serve validation works  https://review.opendev.org/74395606:14
openstackgerritCédric Jeanneret (Tengu) proposed openstack/python-tripleoclient master: Do not relabel /etc/yum.repos.d nor /etc/pki/rpm-gpg  https://review.opendev.org/74905106:16
*** slaweq has quit IRC06:18
*** zzzeek has quit IRC06:23
*** zzzeek has joined #tripleo06:24
*** udesale has joined #tripleo06:24
*** xek has quit IRC06:27
*** rcernin has quit IRC06:28
openstackgerritRabi Mishra proposed openstack/tripleo-ansible master: Adds filter for relative file paths  https://review.opendev.org/75116506:33
openstackgerrityatin proposed openstack/tripleo-puppet-elements stable/ussuri: Remove overcloud-opstools element  https://review.opendev.org/75116606:33
*** rcernin has joined #tripleo06:46
*** zzzeek has quit IRC06:51
*** apetrich has joined #tripleo06:53
*** zzzeek has joined #tripleo06:53
*** jcapitao has joined #tripleo06:55
*** apetrich has quit IRC06:55
*** apetrich has joined #tripleo06:56
*** pcaruana has joined #tripleo07:00
*** jtomasek has joined #tripleo07:02
*** slaweq has joined #tripleo07:03
*** jcapitao has quit IRC07:04
*** jcapitao has joined #tripleo07:06
*** amoralej|off is now known as amoralej07:07
*** jcapitao has quit IRC07:21
*** saneax has joined #tripleo07:24
*** jcapitao has joined #tripleo07:26
*** bogdando has joined #tripleo07:28
*** ysandeep is now known as ysandeep|lunch07:34
*** saneax has quit IRC07:44
*** jpena|off is now known as jpena07:44
bandinianyone seen '2020-09-11T06:17:57.203400262+00:00 stderr F <13>Sep 11 06:17:57 puppet-user: Error: Evaluation Error: Error while evaluating a Resource Statement, Duplicate declaration: Nova_config[DEFAULT/max_concurrent_builds] is already declared at (file: /etc/puppet/modules/nova/manifests/compute.pp, line: 370); cannot redeclare (file: /etc/puppet/modules/nova/manifests/compute/ironic.pp, line:07:45
bandini32) (file: /etc/puppet/modules/nova/manifests/compute/ironic.pp, line: 32, column: 5) on node undercloud.ooo.test' already? (rdo https://logserver.rdoproject.org/78/750678/3/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp_1supp-featureset039/fd5bb97/logs/undercloud/var/log/containers/stdouts/container-puppet-nova.log.txt.gz)07:45
*** jcapitao has quit IRC07:45
Tenguhmmm nope.07:45
Tengunot yet07:45
bandiniseems I73bbca9caf733c8b11335c988b8dbcc33c354150 broke it07:46
ramishrabandini: https://review.opendev.org/#/c/750967/ will fix it I guess07:47
*** shyamb has joined #tripleo07:47
Tenguif tempest stops being nasty07:48
bandiniramishra: aha thanks Rabi!07:49
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-validations stable/train: New validation: Ensure sVirt is active  https://review.opendev.org/75118008:01
*** jpich has joined #tripleo08:03
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates master: python3-novaclient installation via THT  https://review.opendev.org/74674508:04
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo master: Revert "Make sure python3-novaclient is installed before creating fence_compute"  https://review.opendev.org/75118208:04
*** gfidente has joined #tripleo08:05
*** hoonetorg has joined #tripleo08:11
*** xaban has joined #tripleo08:13
xabanHi, via which template file or environments can we configure additional static routes for the overcloud nodes? Any documentation regarding it?08:14
*** pcaruana has quit IRC08:19
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: haproxy bundle  https://review.opendev.org/75045508:22
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: mysql  https://review.opendev.org/75045608:22
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: manila-share  https://review.opendev.org/75045708:22
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: cinder backup/volume  https://review.opendev.org/75045808:22
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: ovn dbs  https://review.opendev.org/75045908:22
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: redis bundle  https://review.opendev.org/75046008:22
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: rabbitmq  https://review.opendev.org/75046108:22
ykarelxaban, see https://opendev.org/openstack/tripleo-heat-templates/commit/f5f655379725ed72cbc00005be3417fde82a195308:25
ykarelas per it u can set using <NetworkName>InterfaceRoutes environment for the network u want to add08:25
xabanykarel: may I show you an example that I am using, which is not working?08:26
ykarelyes share08:27
openstackgerritRabi Mishra proposed openstack/tripleo-ansible master: Add filter for absolute path of file/dir  https://review.opendev.org/75116508:28
*** shyamb has quit IRC08:28
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/ussuri: pcs commands on host: rabbitmq  https://review.opendev.org/75046108:28
xabanykarel: https://nopaste.xyz/?bfecc32e65c9e41c#6YHdek3e7q71pmGMvidkBU9ykGjhhNrFJEFiV2dU8Tpc08:29
xabanshouldn't this add the static route: 10.26.15.0/24 via 10.26.15.254 ??08:29
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/train: Bind mount /var/lib/container-config-scripts inside the restart bundles  https://review.opendev.org/75118408:29
*** jcapitao has joined #tripleo08:30
*** pcaruana has joined #tripleo08:31
ykarelxaban, can u check os-net-config on controller node to see if that config landed there08:34
ykarelyes if config is right, routes should have been set08:34
ykarelat /etc/os-net-config/config.json08:35
*** ysandeep|lunch is now known as ysandeep08:36
xabanUnfortunately, answer is negative.08:36
xabanChange: 2018-03-20, hmm, the file is not changed for quite some time08:37
ykarelohh so you are doing update not a fresh deploy?08:38
*** ssbarnea has joined #tripleo08:38
xabanykarel: indeed08:38
ykareli think for network config, update needs to be enabled by flag, ramishra hjensas__ may have more idea on it08:39
ramishraxaban: use NetworkDeploymentActions: ['CREATE', 'UPDATE'] in one of your env files08:39
ykarelyes ^ flag, thanks ramishra08:39
xabanHmm08:39
ramishrayou can use {{role.name}}NetworkDeploymentActions if you want to limit it to a role08:41
ramishraex. ControllerNetworkDeploymentActions: ['CREATE', 'UPDATE'] for controllers08:42
*** derekh has joined #tripleo08:42
xabanramishra and ykarel: thanks! if it correct that an update - by default - doesn't update the networking config on the nodes?08:43
*** ccamacho has joined #tripleo08:44
ramishraxaban: yes08:45
xabanGood to know, thanks!08:45
*** tosky has joined #tripleo08:47
*** ccamacho has quit IRC08:48
hjensas__xaban: it's not updated by default because it's not always safe. Some services especially on controllers (pacemaker, galera) does'nt always handle a network interface being brought down and up again.08:51
*** hjensas__ is now known as hjensas08:51
*** xaban has quit IRC08:51
hjensasxaban: os-net-config did get better since it was put in place, it will for many cases now update running config without bringing down/up interfaces. When it's possible ... It's a good idea to use the role specific parameter to enable the update, if you know you only need to update Compute's for example.08:53
*** xek has joined #tripleo08:53
*** shyamb has joined #tripleo08:55
*** pojadhav is now known as pojadhav|afk08:55
*** ccamacho has joined #tripleo09:02
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/tripleo-upgrade stable/ussuri: Add neutron-ovs inclusion before other user environments  https://review.opendev.org/75119009:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/tripleo-upgrade stable/train: Add neutron-ovs inclusion before other user environments  https://review.opendev.org/75119109:06
*** ssbarnea has quit IRC09:06
*** pcaruana has quit IRC09:07
*** pcaruana has joined #tripleo09:08
*** rcernin has quit IRC09:10
*** shyam89 has joined #tripleo09:11
openstackgerritHarald Jensås proposed openstack/python-tripleoclient master: Convert to 'raw' images when uploading  https://review.opendev.org/74946609:12
*** shyamb has quit IRC09:15
*** ratailor has joined #tripleo09:16
*** xek has quit IRC09:17
*** cyberpear has quit IRC09:17
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Add support for Glance disk_formats configuration  https://review.opendev.org/75120009:17
*** chem has joined #tripleo09:18
*** jtomasek has quit IRC09:28
shyam89Hi owalsh: Let me know once you get online09:33
shyam89We need your advice on what approach we should use to add trilio mysql calls in puppet-tripleo module09:35
*** saneax has joined #tripleo09:35
shyam89https://github.com/openstack/puppet-tripleo/blob/stable/train/manifests/profile/base/database/mysql.pp09:35
*** ykarel is now known as ykarel|lunch09:36
bhagyashri|roverTengu, Hi09:36
bhagyashri|roverTengu, master jobs are still failing :(09:37
openstackgerritRajesh Tailor proposed openstack/tripleo-heat-templates master: Fix typos  https://review.opendev.org/75121209:38
*** sanjayu_ has joined #tripleo09:41
*** saneax has quit IRC09:42
Tengubhagyashri|rover: yeah - we're talking about it on #rdo and #oooq - the correction patch is blocked due to validations-common not being up-to-date in CI. we might get a way out.09:45
openstackgerritRajesh Tailor proposed openstack/tripleo-heat-templates master: Complete missing description  https://review.opendev.org/75122209:46
Tengubhagyashri|rover: sorry for the whole sh*t - this kind of things are hard to catch (and correct, seeing how things are pushed to promotion)09:46
bhagyashri|roverTengu, ok np looking at discussion.09:46
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Consume ceph, prom, grafana containers from quay.io  https://review.opendev.org/75011909:47
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Consume ceph, prom, grafana containers from quay.io  https://review.opendev.org/75011909:53
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-ansible master: OvS DPDK and SRIOV derive parameters  https://review.opendev.org/73748209:59
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Consume ceph, prom, grafana containers from quay.io  https://review.opendev.org/75011910:05
*** shyam89 has quit IRC10:08
*** ratailor_ has joined #tripleo10:09
*** ratailor has quit IRC10:11
*** d34dh0r53 has quit IRC10:11
*** mnaser has quit IRC10:11
*** donnyd has quit IRC10:11
*** d34dh0r53 has joined #tripleo10:11
*** donnyd has joined #tripleo10:11
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart-extras master: Fetch non tripleo containers in provider job and consume it  https://review.opendev.org/74964510:14
*** ykarel|lunch is now known as ykarel10:16
*** mnaser has joined #tripleo10:16
*** mnaser has quit IRC10:17
*** mnaser has joined #tripleo10:17
*** jmasud has quit IRC10:20
*** tkajinam has quit IRC10:22
*** jmasud has joined #tripleo10:22
*** shyamb has joined #tripleo10:30
openstackgerritlikui proposed openstack/diskimage-builder master: replace imp module  https://review.opendev.org/75123610:31
*** shyamb has quit IRC10:33
*** jpich has quit IRC10:39
*** jpich has joined #tripleo10:40
*** jcapitao is now known as jcapitao_lunch10:47
*** chem has quit IRC10:50
openstackgerritMerged openstack/tripleo-ansible stable/ussuri: Use bool filter when checking use_default_templates var  https://review.opendev.org/75108810:51
openstackgerritMerged openstack/tripleo-ansible master: Don't hide logs in molecule  https://review.opendev.org/75105010:51
*** shyamb has joined #tripleo10:52
*** chem has joined #tripleo10:53
openstackgerritSandeep Yadav proposed openstack/tripleo-quickstart-extras master: Use osp_release flag to set selinux mode  https://review.opendev.org/75124310:56
*** artom has joined #tripleo11:01
openstackgerritmathieu bultel proposed openstack/tripleo-validations master: Revert "Change path for validation Ansible files"  https://review.opendev.org/75124411:03
*** sanjayu__ has joined #tripleo11:03
openstackgerritmathieu bultel proposed openstack/tripleo-validations master: Revert "Change path for validation Ansible files"  https://review.opendev.org/75124411:04
*** sanjayu_ has quit IRC11:05
*** rcernin has joined #tripleo11:06
*** cyberpear has joined #tripleo11:08
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Add support for Glance disk_formats configuration  https://review.opendev.org/75120011:10
*** shyam89 has joined #tripleo11:11
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-validations master: Revert "Change path for validation Ansible files"  https://review.opendev.org/75124411:12
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Add support for Glance disk_formats configuration  https://review.opendev.org/75120011:12
*** shyamb has quit IRC11:15
*** rcernin has quit IRC11:20
openstackgerritGiulio Fidente proposed openstack/tripleo-ansible master: Add tripleo-lvmfilter role to restrict visible block devices for LVM2  https://review.opendev.org/69799011:24
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-validations master: Ensure we get a prefix for validation roles  https://review.opendev.org/75124711:25
Tengumatbu: -^11:25
*** sanjayu__ has quit IRC11:28
*** jtomasek has joined #tripleo11:28
*** chem has quit IRC11:29
*** shyam89 has quit IRC11:30
*** shyam89 has joined #tripleo11:30
*** chem has joined #tripleo11:31
openstackgerritGiulio Fidente proposed openstack/tripleo-ansible master: Add tripleo-lvmfilter role to restrict visible block devices for LVM2  https://review.opendev.org/69799011:32
shyam89owalsh: Hi11:38
owalshshyam89: hi11:40
*** jpena is now known as jpena|lunch11:42
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Run tripleo_lvmfilter role to restrict block devices visible to LVM2  https://review.opendev.org/69799711:42
shyam89owalsh: Trilio service needs to create it's database and keystone resources. For that we need to add it's call in https://github.com/openstack/puppet-tripleo/blob/stable/train/manifests/profile/base/database/mysql.pp11:43
shyam89and https://github.com/openstack/puppet-tripleo/blob/stable/train/manifests/profile/base/keystone.pp11:43
shyam89We are planning to merge the changes in upstream. But for this release, can we do changes in puppet-tripleo on undercloud and upload that to overcloud?11:44
shyam89Using upload-puppet-module tool11:44
shyam89owalsh: Let me know it's pros and cons11:44
shyam89and if it will work11:44
*** xek has joined #tripleo11:45
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Run tripleo_lvmfilter role to restrict block devices visible to LVM2  https://review.opendev.org/69799711:47
*** raildo has joined #tripleo11:48
owalshshyam89: keystone/db are not really my area... I'd ask EmilienM or mwhahaha for advice on this11:51
owalshshyam89: given the time zones maybe better to discuss on openstack-discuss mailing list (start the subject with [tripleo])11:52
shyam89owalsh: okay11:53
shyam89thanks11:53
shyam89EmilienM: mwhahaha: Let me know your thoughts on my query ^^11:54
ykarelshyam89, btw did the addition of db and keystone calls in tht worked for you?11:55
ykareljust for testing11:55
shyam89ykarel: No, I did not seen their execution in any log file. Also no effect on the keystone and db resources11:56
ykarelshyam89, at what place u added the calls?11:57
ykarelexecution should be there if added at correct place11:57
*** jcapitao_lunch is now known as jcapitao11:59
*** rfolco|ruck has joined #tripleo12:01
*** rlandy has joined #tripleo12:01
shyam89ykarel: I added it here for db: https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/database/mysql-pacemaker-puppet.yaml#L17812:02
shyam89ykarel: like this: http://paste.openstack.org/show/797770/12:03
shyam89I observed that puppet module got loaded, but not executed12:03
shyam89maybe puppet tags issue?12:03
shyam89I observed that this call uses 'file' puppet tag. https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/database/mysql-pacemaker-puppet.yaml#L17212:04
ykarelshyam89, where u obsolerved puppet module got loaded?12:05
shyam89in container-puppet-mysql.log file12:06
shyam89ykarel: ^^12:06
ykarelshyam89, check logs for mysql_init_bundle12:09
ykareluhhk12:10
ykarelshyam89, so u need to add call at https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/database/mysql-pacemaker-puppet.yaml#L27612:10
ykarelinstead of the earlier place12:10
ykarelas there is puppet tags file, and only file resources will be applied12:10
shyam89okay12:11
shyam89I see12:12
shyam89ykarel: I will give a try12:12
shyam89But before that, I will test upload puppet module for tripleo puppet module12:12
*** xek has quit IRC12:12
shyam89ykarel: thanks. This is helpful.12:13
*** Goneri has joined #tripleo12:16
rlandyEmilienM: hi - how do we know where we pull the ubi8 image from in container builds? which registry?12:16
*** amoralej is now known as amoralej|lunch12:17
openstackgerritMerged openstack/tripleo-heat-templates master: Make sure we noop the Rabbitmq_ providers  https://review.opendev.org/75076312:18
openstackgerritMerged openstack/tripleo-common master: Run tripleo-build-containers-ubi-8 on BuildahBuilder code  https://review.opendev.org/75080012:18
EmilienMrlandy: good question12:18
EmilienMlet me find out12:18
EmilienMbut ubi8 comes from redhat registry12:18
ykarelregistry.redhat12:18
EmilienMyeah ^ that probably12:18
ykarelto be specific registry.redhat.io12:19
EmilienMregistry.access.redhat.com/ubi812:19
EmilienMor that ^12:19
EmilienMon my computer, it pulls it from registry.access.redhat.com/ubi812:19
rlandyEmilienM: talking with Shreshtha12:19
rlandywe're trying to sync up where we take ubi8 vs the ondeck image brew uses12:20
ykarelshyam89, for keystone i think u need to use ansible way because of https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/keystone/keystone-container-puppet.yaml#L38212:20
ykarelhttps://github.com/openstack/puppet-tripleo/blob/stable/train/manifests/profile/base/keystone.pp#L203-L21312:20
shyam89ykarel: yes, I used ansible way. Working fine on 16.1. But for RHOSP13 support, we need to use old way.12:21
ykarelshyam89, ok for 13 it will work12:22
EmilienMrlandy, ykarel : i think it really depends on the /etc/containers/registries.conf12:22
EmilienMand the "unqualified-search-registries"12:22
EmilienM:q!12:22
EmilienMi finally left VI !12:23
ykarelEmilienM, yes it will look in order in registeries defined in ^12:23
ykareland ubi8 will come from redhat registeris12:23
ykarelshyam89, for queens u can add at https://github.com/openstack/tripleo-heat-templates/blob/stable/queens/puppet/services/keystone.yaml#L51112:25
shyam89ykarel: okay, let me check12:27
shyam89ykarel: good one, thanks12:28
*** shyam89 has quit IRC12:29
*** pojadhav|afk is now known as pojadhav12:34
*** raildo_ has joined #tripleo12:35
*** raildo has quit IRC12:35
*** ratailor_ has quit IRC12:42
*** skramaja has quit IRC12:43
*** jpena|lunch is now known as jpena12:44
openstackgerritMerged openstack/tripleo-upgrade stable/ussuri: Add neutron-ovs inclusion before other user environments  https://review.opendev.org/75119012:45
*** lbragstad_ has joined #tripleo12:49
*** zzzeek has quit IRC12:50
*** lbragstad_ is now known as lbragstad12:51
*** zzzeek has joined #tripleo12:51
*** raildo has joined #tripleo12:53
*** raildo_ has quit IRC12:53
openstackgerritEmilien Macchi proposed openstack/tripleo-common stable/train: Add tripleo_states to default callback whitelist  https://review.opendev.org/74101512:56
*** ekultails has joined #tripleo13:00
*** zzzeek has quit IRC13:04
*** zzzeek has joined #tripleo13:07
*** matbu has quit IRC13:09
*** matbu has joined #tripleo13:10
openstackgerritMerged openstack/tripleo-docs master: Add a note about verifying container images after hotfix  https://review.opendev.org/75080713:11
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-ansible master: OvS DPDK and SRIOV derive parameters  https://review.opendev.org/73748213:14
openstackgerritwes hayutin proposed openstack/ansible-role-collect-logs master: use sova to check for selinux denials  https://review.opendev.org/75127313:19
weshay|ruckTengu, https://review.opendev.org/#/c/751273/13:19
*** amoralej|lunch is now known as amoralej13:19
weshay|ruckTengu, this patch caused a selinux denial btw https://review.opendev.org/#/c/739481/13:20
Tenguweshay|ruck: linked to podman version upstream I guess.13:20
Tenguiirc ykarel worked in order to get 1.6.4-15 in there.13:20
weshay|ruckaye13:20
Tengubut nice thing adding selinux in the parsing.13:21
Tenguwhat's sova?13:21
openstackgerritJohn Fulton proposed openstack/python-tripleoclient master: WIP/DNM: stub for ceph-export  https://review.opendev.org/75081213:21
Tenguweshay|ruck: since you're here - care to add some ptl ack on https://review.opendev.org/#/c/750970/ and https://review.opendev.org/#/c/750964/  ? That will help with the blocked promotion..13:22
Tenguweshay|ruck: (I've update the cix card with the plan, and some details about the path taken to get in this situation..)13:23
weshay|rucksova is a little tool to parse logs for common errors.. you see it drop a log file in each job.. in logs.. https://logserver.rdoproject.org/81/739481/8/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-vexxhost/9788565/logs/13:24
weshay|ruckNo_failure_reason_found.log13:24
weshay|ruckdon't ask about the names there13:24
weshay|rucknot my cup of tea13:24
weshay|ruckhttps://1dc19a8f26b6d8fa6b0f-1ea15234ebc61f9ad6f390769a79bf22.ssl.cf2.rackcdn.com/750071/1/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/06adabe/logs/13:24
weshay|ruckin that log you'll find.. _Undercloud_install_failed.log13:24
weshay|ruckand we capture that info in our monitoring http://dashboard-ci.tripleo.org/d/wb8HBhrWk/cockpit?orgId=113:25
weshay|ruckFailure cause13:25
* weshay|ruck looks at reviews13:25
*** tmazur has joined #tripleo13:27
*** TrevorV has joined #tripleo13:27
ykarelweshay|ruck, Tengu hmm i didn't removed DNM https://review.opendev.org/#/c/746111/13:27
ykareltested train at https://review.rdoproject.org/r/#/c/28958/13:27
Tenguweshay|ruck: ah, that's the tool providing those files. thanks!13:28
Tenguykarel: ah. well. guess we can drop the DNM right?13:28
ykarelbtw where u seen denials?13:28
Tenguprobably when tempest starts a VM.13:29
weshay|ruckykarel, https://logserver.rdoproject.org/81/739481/8/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-vexxhost/9788565/logs/overcloud-novacompute-0/var/log/audit/audit.log.txt.gz13:29
ykarelare we running with enforcing?13:29
Tenguhmmm13:29
weshay|ruckykarel, enforcing is not allowed upstream or 3rd party13:29
Tenguweshay|ruck: https://logserver.rdoproject.org/81/739481/8/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-vexxhost/9788565/logs/overcloud-novacompute-0/var/log/extra/denials.txt.gz  only one denial.13:29
weshay|ruckit was caught on our osp runs13:30
Tengunot linked to the patch13:30
ykarelweshay|ruck, hmm looking for that only13:30
weshay|ruckyup.. that's all it takes13:30
ykarelhow it got caught13:30
weshay|rucksame job runs on osp w/ enforcing13:30
ykarelok good13:30
ykarelbut upstream patch dnm will not help the osp jobs13:31
Tenguthe chronyd denial isn't linked to anything with nova, and it's a "temporary" one actually, until chronyd is restarted or something13:31
Tenguosp should have 1.6.4-15 already.13:31
Tenguunless the wrong container-tools stream is activated?13:31
openstackgerritRonelle Landy proposed openstack/tripleo-ci master: Add option to pull a custom base dockerfile - ubi8  https://review.opendev.org/75127713:36
openstackgerritChristian Schwede proposed openstack/tripleo-heat-templates master: Decrease Swift proxy timeouts for GET/HEAD requests  https://review.opendev.org/75127913:36
*** udesale_ has joined #tripleo13:37
Tengufolks, can we get +A on https://review.opendev.org/#/c/751244/ so that we unlock the promotion pipeline?13:38
Tenguthanks - and sorry for that failure -.-'13:38
Tenguthanks weshay|ruck !13:38
Tenguweshay|ruck: once that one is merged, any ETA for a promotion on master?13:38
Tenguprovided, of course, nothing else breaks :)13:39
openstackgerritmathieu bultel proposed openstack/tripleo-validations master: Remove objects migrated to validations-common  https://review.opendev.org/71320413:39
*** udesale has quit IRC13:39
weshay|ruckTengu, we've been dark for 2 days or so.. all jobs failing13:39
weshay|ruckTengu, so.. not sure.. let me check the components13:39
* Tengu ducks in a really dark, dark corner13:39
weshay|rucklooks pretty good http://dashboard-ci.tripleo.org/d/FE8Hf29Wz/component-pipeline?orgId=113:40
weshay|ruckeverything there is promoting quickly13:40
Tenguso that's a good news then13:40
weshay|ruckooof .. except for tempest13:40
Tenguyeah, was about to talk about it13:40
*** saneax has joined #tripleo13:40
openstackgerritAlex Schultz proposed openstack/python-tripleoclient master: Fix pathing for container image prepare  https://review.opendev.org/75110013:41
weshay|ruckit's only two days old13:41
Tengufinger crossed everything will be OK - meaning we can finish the checklist Monday.13:44
openstackgerritAlex Schultz proposed openstack/python-tripleoclient master: Fix pathing for container image prepare  https://review.opendev.org/75110013:48
*** errantekarmico has joined #tripleo13:50
*** matbu has quit IRC13:51
*** matbu has joined #tripleo13:53
*** gmann is now known as gmann_afk14:05
fultonjTengu: what's the lastest version of ansible you're using with tripleo-lab ?14:15
fultonje.g. are you using it with ansible-2.9.13-1 ?14:16
Tengufultonj: ansible 2.9.1314:17
fultonjack, thanks14:18
Tenguansible-2.9.13-1.fc32.noarch  yeè14:18
Tengufultonj: are you hitting any issue?14:20
fultonjpermissions issue during undercloud install14:20
fultonjbut i'm running an old version of tripleo lab from jul114:20
fultonjwith latest main from tripleo14:21
fultonjso i'm going to upgrade tripleo-lab first14:21
fultonji had upgrded ansible right before though so i wanted to rule that out14:21
fultonjwith all the permissions from from 2.9.12, 2.9.11 had to ask14:21
Tenguoh, ok14:24
jpalanis_fultonj, hi14:34
openstackgerritJaganathan Palanisamy proposed openstack/tripleo-ansible master: OvS DPDK and SRIOV derive parameters  https://review.opendev.org/73748214:34
fultonjhi jpalanis_14:34
jpalanis_fultonj, planning to do the separate patch for molecule and documentation changes14:34
jpalanis_fultonj, we can work and merge this two patches to remove the mistral dependencies14:35
fultonjjpalanis_: is it working in your env?14:35
jpalanis_fultonj, yes14:35
fultonjin a real deploy?14:35
fultonj'openstack overcloud deploy -p ...' sets the dpdk params right?14:36
jpalanis_fultonj, it was working with my code ovs dpdk and sriov. but not tested yet with latest code14:36
jpalanis_fultonj, mean using hci merged code14:37
jpalanis_fultonj, should work. will test and let you know14:37
jpalanis_fultonj, already added modules and filters test cases, only molecule NFV mock and document parts pending14:38
fultonjjpalanis_: that's great14:38
fultonjjpalanis_: is there mock data i could try the ansible module with?14:39
fultonjthat's not essential but would be a nice to have14:39
jpalanis_fultonj, for ansible modules, already used mock data in test cases.14:40
jpalanis_fultonj, just need to run the test cases14:40
*** errantekarmico has quit IRC14:40
jpalanis_fultonj, each test cases, different mock data needed14:40
fultonjjpalanis_: so you think it could merge once CI is green?14:41
fultonjjpalanis_: https://review.opendev.org/#/c/578947/14:42
jpalanis_fultonj, ok. will do the additional patches for enhancements and incases any issues14:42
fultonjshould ^ be abandoned?14:42
jpalanis_fultonj, yes, will do it. need to rework14:43
fultonjhttps://review.opendev.org/#/c/737847/ looks like it's getting close14:43
fultonjyou know why it failed tripleo-ci-centos-8-containers-multinode ?14:43
fultonjhttps://zuul.opendev.org/t/openstack/build/e291a6ca67bb400d9e5ea3386137619414:44
fultonji kind of doubt it's related but it should be looked into and if it's not related rechecked14:44
jpalanis_fultonj, https://zuul.opendev.org/t/openstack/build/e291a6ca67bb400d9e5ea3386137619414:45
jpalanis_fultonj, i think its not related to my changes.14:45
mwhahahafultonj: it oom'd14:46
mwhahahaEmilienM, cloudnull -^ containers-multinode in master oom'd too https://review.opendev.org/#/c/737847/14:46
fultonjack14:46
fultonjok14:46
fultonjjust wanted to be sure before rechdcking14:46
jpalanis_fultonj, ok14:46
mwhahahasomethings changed recently where we're oom'ing14:47
EmilienMlet me look14:47
EmilienMdo we have a logstash query?14:48
mwhahahaprobably not14:48
mwhahahalogstash is always days behind anyway14:49
*** jtomasek has quit IRC14:49
EmilienMi used to have a magic gerrit query to find what landed in tripleo very recently14:49
weshay|ruckheh.. this one https://review.opendev.org/#/q/status:merged+tripleo+age:3week14:50
mwhahahait's ooming in train+plus so i wonder if we have some base os changes that are now consuming more ram14:51
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Remove objects migrated to validations-common  https://review.opendev.org/71320414:52
EmilienMyeah let's rpm diff14:53
EmilienMmy favorite friday morning game14:53
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Remove objects migrated to validations-common  https://review.opendev.org/71320414:53
fultonjTengu: still hitting permissions issue14:54
fultonjhttp://paste.openstack.org/show/797780/14:54
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Run tripleo_lvmfilter role to restrict block devices visible to LVM2  https://review.opendev.org/69799714:54
Tengufultonj: did you update the tripleo-operator-ansible ?14:55
Tengufultonj: you can run "ansible-playbook config-host.yaml"14:55
Tenguthat will update and deploy the operator.14:56
fultonj ansible-playbook -i inventory.yaml config-host.yaml14:56
fultonji ran before running14:56
Tenguweird...14:56
* fultonj just re-ran and chaged=014:56
Tenguthought.... I don't think this is due to tripleo-lab. Are you re-using an existing undercloud VM, or is it a brand new one?14:57
fultonji had tripleo lab destroy and rebuild eveything14:58
Tengumeh......14:58
fultonji'll get to the bottom of it eventually14:58
Tenguwhat are the permission on that ansible.log ?14:58
EmilienMmwhahaha: first of, new ansible (2.9.13 vs 2.9.11)14:58
EmilienMhttps://www.diffchecker.com/0H5Tfsb514:58
fultonjhmmmm14:59
fultonjlol14:59
fultonj-rw-r--r--. 1 root  root   5710 Sep 11 14:36 ansible.log14:59
EmilienMmwhahaha: i took a failing job from today on master and a working job on master 3 days ago14:59
mwhahahawhy dowe have lower ansible-runner now?14:59
mwhahaha1.4.5 vs 1.4.615:00
Tengufultonj: wondering if it's not coming from the prior container prepare command....15:00
EmilienMmwhahaha: no, it's newer now15:00
mwhahahaoh inverse15:00
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates master: Run tripleo_lvmfilter role to restrict block devices visible to LVM2  https://review.opendev.org/69799715:00
mwhahahaso either ansible or ansible runner mabe15:00
EmilienMi mean there are a few python deps which could be the reason too but not sure15:00
bandinimmh ussuri jobs are failing with containers 404 but only on the overcloud lesigh15:05
ykarelboth ansible and ansible-runner were updated recently on master15:05
*** xek has joined #tripleo15:05
ykarelis seen issue master only?15:05
EmilienMI'm looking at ansible release notes15:06
mwhahahaykarel: no there's a bug around train too15:07
EmilienMdid we also bump ansible in train?15:07
ykarelmwhahaha, in train we didn't updated so may be something else15:07
ykarelrecently not updated in Train15:08
mwhahahawe haven't touched this code in a while, so i was wondering if it was the recent switch to tcib containers15:08
EmilienMno pushups today please15:09
weshay|ruckPUSH UPS15:10
weshay|ruckone for each container15:10
EmilienMmwhahaha: https://www.diffchecker.com/msqVQgZv15:12
EmilienMprofile_tasks_limit ?15:13
mwhahahano15:13
mwhahahai don't think at least15:14
*** bandini has quit IRC15:14
EmilienMand it would have failed on master before15:15
*** ysandeep is now known as ysandeep|away15:15
ykarelmwhahaha, i see ur comment wrt swap size15:15
ykarelhttps://opendev.org/openstack/tripleo-ci/src/branch/master/roles/prepare-node/tasks/main.yaml#L315:15
mwhahahayea i'd recommend 4g15:15
ykarelrole var configure_swap_size15:15
ykarelcurrently 1024 is used15:15
mwhahahai run 8+4 in my dev env15:16
mwhahahahaven't oom'd in a very very very long time15:16
mwhahahait's likely the stock undercloud is using all 8 and when we run the prepare process for teh overcloud, it's blowing up15:16
mwhahahacontainer image prepare uses a decent amount of ram15:17
ykarelcan try swap bump, but would be good to know why suddenly it starting oom15:18
*** xek has quit IRC15:19
*** lmiccini has quit IRC15:19
mwhahahaEmilienM: might also compare dstat files15:19
EmilienMbrb15:21
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add release version package  https://review.opendev.org/75131715:21
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add release version package  https://review.opendev.org/75131715:25
*** gfidente has quit IRC15:26
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add release version package  https://review.opendev.org/75131715:30
*** zzzeek has quit IRC15:32
*** zzzeek has joined #tripleo15:34
openstackgerritMerged openstack/python-tripleoclient master: Convert to 'raw' images when uploading  https://review.opendev.org/74946615:39
openstackgerritPaul Leimer proposed openstack/tripleo-heat-templates master: Gather more extra stats with the collectd virt plugin  https://review.opendev.org/75133015:42
*** ykarel is now known as ykarel|away15:42
mwhahahai wonder if it's crashing during the updating15:44
*** udesale_ has quit IRC15:45
*** jcapitao has quit IRC15:46
*** marios is now known as marios|out15:50
*** zzzeek has quit IRC15:55
*** ekultails has left #tripleo15:56
*** ekultails has joined #tripleo15:56
*** zzzeek has joined #tripleo15:56
*** bogdando has quit IRC15:58
*** marios|out has quit IRC16:00
*** zzzeek has quit IRC16:03
openstackgerritRafael Folco proposed openstack/tripleo-quickstart master: Fix containers config file on fs037  https://review.opendev.org/75134016:05
*** zzzeek has joined #tripleo16:05
openstackgerritMerged openstack/tripleo-validations master: Revert "Change path for validation Ansible files"  https://review.opendev.org/75124416:06
*** ykarel|away has quit IRC16:06
openstackgerritJohn Fulton proposed openstack/tripleo-operator-ansible master: Operator does not need to become for image prepare  https://review.opendev.org/75134416:14
*** derekh has quit IRC16:23
*** zzzeek has quit IRC16:26
*** zzzeek has joined #tripleo16:26
openstackgerritRonelle Landy proposed openstack/tripleo-common master: WIP: Add variable to override default labels on base dockerfile  https://review.opendev.org/75134816:29
*** saneax has quit IRC16:31
*** jpich has quit IRC16:34
*** artom has quit IRC16:35
*** artom has joined #tripleo16:35
*** rfolco|ruck is now known as rfolco|ruck|brb16:39
*** raildo_ has joined #tripleo16:46
*** raildo has quit IRC16:49
cloudnullmwhahaha EmilienM re: oom - wasn't there changes ekultails and dpeacock were working on, adjusting the memlimits for mariadb? did those land?17:00
cloudnullcould they be pusing us over the edge17:00
EmilienMhttps://review.opendev.org/#/c/747992/17:01
EmilienMit wasn't backported17:01
EmilienMekultails: ^ btw https://review.opendev.org/#/c/747992/8/deployment/database/mysql-base.yaml@6517:01
EmilienMekultails: please send a patch to address this comment17:01
cloudnullok.17:01
cloudnullmaybe we're just landing on new nodes w/ less ram?17:01
cloudnulllike a new provide just came online ?17:02
cloudnullor the swapiness is set to 0?17:02
chandankumarjpalanis_: patch in merge conflict issue is fixed now by infra side17:06
ekultailsEmilienM, aschultz: Thanks for the catch. I'll take care of it.17:09
ekultailscloudnull: We're looking into making the MariaDB tuning somewhat dynamic and potentially expanding that to other services.17:10
openstackgerritLuke Short proposed openstack/tripleo-heat-templates master: Use consistent naming for MysqlInnodbBufferPoolSize  https://review.opendev.org/75136117:14
*** jpena is now known as jpena|off17:34
*** gmann_afk is now known as gmann17:42
*** apetrich has quit IRC17:44
*** ekultails has quit IRC17:45
*** rfolco|ruck|brb is now known as rfolco|ruck17:46
*** ccamacho has quit IRC17:50
mwhahahacloudnull: i don't think we've changed anything really. i also don't think we do any swapiness/oom tuning17:57
mwhahahacloudnull: EmilienM: I was wondering if we should think about exposing OOMScoreAdjust in the podman systemd service units17:57
EmilienMehhhhh not sure i want to go into that business17:58
mwhahahawe probably should have been in that business but yea17:58
EmilienMwhat value would you put?17:59
EmilienM-500 ?17:59
mwhahahaanything? just to make it lower priority18:02
mwhahaha-10018:02
EmilienMworth giving a try18:03
mwhahahait's not going to solve this problem18:04
mwhahahabut might prevent other shit18:04
matbuweshay|ruck: can you merged that https://review.opendev.org/#/c/713204/ to see if it can unblock the promotion18:10
weshay|rucklooking18:11
openstackgerritRafael Folco proposed openstack/tripleo-quickstart master: Fix containers config file on fs037  https://review.opendev.org/75134018:11
openstackgerritMerged openstack/tripleo-heat-templates master: Make sure we noop the Mysql_ providers  https://review.opendev.org/75067818:26
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add release version package  https://review.opendev.org/75131718:30
*** rh-jelabarre has quit IRC18:36
*** chem has quit IRC18:36
*** rh-jelabarre has joined #tripleo18:36
*** chem has joined #tripleo18:38
rlandycloudnull: https://review.opendev.org/#/c/751277 Add option to pull a custom base dockerfile - ubi818:38
openstackgerritLance Bragstad proposed openstack/tripleo-heat-templates master: Update AdminToken description  https://review.opendev.org/75138018:40
EmilienMrlandy: +218:40
lbragstadqq - is there a way to explicitly override a default set by tht on older releases?18:40
rlandythanks18:40
mwhahaharlandy, cloudnull: not exactly a good variable name18:41
rlandymwhahaha: base_image?18:42
mwhahahaprobably better18:42
mwhahahacontainers_base_image18:43
mwhahahasomething like that18:43
rlandyok - updating18:43
openstackgerritRonelle Landy proposed openstack/tripleo-ci master: Add option to pull a custom base dockerfile - ubi8  https://review.opendev.org/75127718:43
cloudnullanyone want to give https://review.opendev.org/#/c/751074/ a nudge18:46
*** ekultails has joined #tripleo18:58
weshay|ruckfolks.. all the validations folks are away19:10
weshay|ruckdo we still need tripleo-validations in master.. if we have validations-common and validations-lib ?19:11
openstackgerritAlex Schultz proposed openstack/python-tripleoclient master: Add --temp-dir to overcloud image build  https://review.opendev.org/75138819:11
mwhahahai was wondering the same thing, if we don't then validations-common could obsolete the tripleo version but i think the tripleo version is still required19:12
weshay|ruckk.. I sent them an email19:18
weshay|ruckwe'll find out19:18
weshay|ruckmwhahaha, at the very least we should move tripleo-validations to the validations component19:18
* mwhahaha shrugs19:19
*** social has joined #tripleo19:24
*** xek has joined #tripleo19:47
*** pcaruana has quit IRC19:55
openstackgerritMerged openstack/tripleo-operator-ansible master: Add comment specifying tripleo_overcloud_node_delete_nodes as a list  https://review.opendev.org/75073719:56
lbragstadit looks like the AdminToken is set by default on train deployments, even though it was removed in later releases https://review.opendev.org/#/c/717421/20:00
lbragstadi have a patch to update the docs https://review.opendev.org/#/c/751380/ but is there a way to explicitly unset that variable at deploy time?20:01
mwhahahaadmintoken is reused for somethign isn't it?20:04
mwhahahalbragstad: AdminToken is still used as the keystone db password, so you wouldn't want to unset it20:05
lbragstadahhh20:12
lbragstadi'm missing the part where tht hands that off to puppet-keystone then20:12
mwhahahakeystone::db::mysql::password: {get_param: AdminToken}20:12
lbragstadyeah - that part makes sense20:12
lbragstadwait - so there isn't a way to disable AdminToken without disabling database authentication on train20:13
mwhahahayou can just not set it for the admin token part20:13
* mwhahaha looks20:14
mwhahaha            keystone::admin_token: {get_param: AdminToken}20:15
mwhahaharemoving that20:15
mwhahahaor blanking that as necessary20:15
lbragstadControllerExtraConfig20:15
lbragstadi wonder if that would work?20:15
lbragstadotherwise - i think i'd need to back port https://review.opendev.org/#/c/717421/20:15
mwhahahaif you set keystone::admin_token that way, it should override the THT one20:15
mwhahahathough backporting that is probably more correct20:15
mwhahahai don't know when we stopped relying on the admin token everywhere20:16
lbragstadyou think? i wasn't sure if that would be accepted since it would break things20:16
mwhahahawhat would it break?20:16
lbragstadi have a stable/train env up and you can technically use the admin token still20:16
mwhahahait's more of what does the puppet-keystone bits require20:16
lbragstadbecause it's set - even though `keystone-manage bootstrap` is used20:16
mwhahahai want to say we stopped using that with containers tho20:17
mwhahahawe had to backport some fixes around that stuff20:17
mwhahahabut it hink we did that in train20:17
mwhahahahttps://review.opendev.org/#/c/699017/1/deployment/keystone/keystone-container-puppet.yaml20:18
mwhahahaso i think you can just cherry pick that change back20:18
openstackgerritLance Bragstad proposed openstack/tripleo-heat-templates stable/train: Remove deprecated puppet-keystone modules related to admin auth  https://review.opendev.org/75142320:20
lbragstadtht parameters support nil, right? keystone::admin_token: nil for example20:23
openstackgerritMerged openstack/tripleo-validations master: Remove objects migrated to validations-common  https://review.opendev.org/71320420:24
mwhahahano20:26
mwhahaharemoving it will cause it to fall back to the default from the puppet module which in theory it should drop from the configs (if the default value is $::os_service_default)20:29
lbragstadyou mean removing it here - https://review.opendev.org/#/c/751423/1/deployment/keystone/keystone-container-puppet.yaml@394 ?20:31
mwhahahayes20:32
lbragstadyeah - ok, that makes sense20:32
lbragstadin the mean time, setting it to empty string using ControllerExtraConfig should work around the issue (i'm testing this now)?20:32
mwhahahamaybe?20:33
mwhahahai'd have to look what happens in the puppet module20:33
mwhahahaset it to <SERVICE DEFAULT>20:35
mwhahahahax20:35
mwhahahakeystone::admin_token: "<SERVICE DEFAULT>" in an extraconfig20:36
mwhahahashould reproduce unsetting it20:36
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: Set install of ipa packages to false - use local forwarder  https://review.opendev.org/74103520:36
lbragstadmwhahaha ok - trying that now20:37
*** TrevorV has quit IRC20:37
lbragstadside note; is anyone else noticing issues deploying stable/train rdo packages, specifically with the overcloud container starting in step 3?21:03
* lbragstad is able to recreate on ovb and deployed-server environments21:04
lbragstadovercloud containers*21:04
*** raildo_ has quit IRC21:06
*** rlandy is now known as rlandy|afk21:09
*** rfolco|ruck has quit IRC21:20
openstackgerritMerged openstack/tripleo-ansible master: Add guard to ensure a mapping has a valid key  https://review.opendev.org/75107421:33
openstackgerritJohn Fulton proposed openstack/python-tripleoclient master: Add openstack overcloud export ceph  https://review.opendev.org/75081221:35
*** fultonj has quit IRC21:38
*** xek has quit IRC21:49
*** rlandy|afk is now known as rlandy21:53
*** cfontain has joined #tripleo22:02
*** rcernin has joined #tripleo22:02
*** rcernin has quit IRC22:03
*** rcernin has joined #tripleo22:03
*** tmazur has quit IRC22:53
*** tosky has quit IRC23:20
*** chem has quit IRC23:39
*** chem has joined #tripleo23:41
*** ekultails has quit IRC23:42

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