Thursday, 2020-02-20

cloudnullmwhahaha stevebaker sshnaidm|afk is there a doc on deploying nova-less ?00:00
* cloudnull would like to setup a local test env 00:01
mwhahahaI think it's in the docs00:01
stevebakercloudnull: https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/provisioning/baremetal_provision.html00:01
EmilienMit sounds like a friday project :D00:01
EmilienMTengu: and a patch in your tripleo-lab :P00:02
openstackgerritMerged openstack/tripleo-heat-templates stable/train: Force facts cache refreshing after OS upgrade.  https://review.opendev.org/70834300:02
openstackgerritMerged openstack/tripleo-heat-templates stable/train: Add swiftoperator role on ceph-rgw template  https://review.opendev.org/70862700:08
openstackgerritMerged openstack/tripleo-heat-templates stable/train: tripleo_container_manage: set tripleo_container_manage_check_puppet_config  https://review.opendev.org/70861900:09
openstackgerritMerged openstack/tripleo-upgrade master: Configure undercloud_enable_paunch when set  https://review.opendev.org/70824100:09
openstackgerritMerged openstack/tripleo-quickstart master: Disable paunch on fs050  https://review.opendev.org/70825600:09
*** ade_lee has joined #tripleo00:11
*** jmasud has quit IRC00:11
*** jmasud has joined #tripleo00:14
*** stevebaker has quit IRC00:36
*** ekultails has left #tripleo00:56
openstackgerritMerged openstack/python-tripleoclient master: Remove mistral from the list_available_roles deploy workflow  https://review.opendev.org/70827601:01
openstackgerritMerged openstack/python-tripleoclient master: Remove mistral from the create_container deploy workflow  https://review.opendev.org/70827901:01
*** stevebaker has joined #tripleo01:15
cloudnullstevebaker ping - i've been looking into https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/provisioning/baremetal_provision.html - but I don't see where I'd define node ipmi setting. where might that go?01:21
cloudnulland I was looking at the logs https://logserver.rdoproject.org/58/707658/5/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001/7734c27/logs/undercloud/home/zuul/01:21
cloudnulland i didn't see anything that seemed to standout01:22
stevebakercloudnull: isn't that something which is determined for the ironic node during node import? At node provision time, all nodes should be in an available state with things like ipmi already set up01:25
cloudnulloh , so we still need the instackenv ?01:25
cloudnulland need to do node import and introspection ?01:26
openstackgerritMerged openstack/tripleo-common master: Set PROFILE_TASKS_TASK_OUTPUT_LIMIT  https://review.opendev.org/70655001:29
openstackgerritMerged openstack/tripleo-heat-templates stable/stein: Swap tasks to avoid non-existing variable error.  https://review.opendev.org/70565001:29
stevebakercloudnull: yes, this just adds an extra provision step to boot the nodes, instead of having it done by heat->nova->ironic01:29
cloudnullah i see01:29
cloudnullcool , thanks!01:29
stevebakercloudnull: that doc page is at the end of the baremetal section, which all still applies https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/provisioning/index.html01:30
cloudnulland the ssh key, you said that's default to ~/.ssh/id_rsa01:30
cloudnull?01:30
cloudnullok this makes more sense now, thanks01:30
stevebakercloudnull: yes, unless you specify openstack overcloud node provision --overcloud-ssh-key ...01:31
cloudnullin the most recent job log -https://logserver.rdoproject.org/58/707658/5/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001/7734c27/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz I see the key being set to :01:31
cloudnull2020-02-19 22:46:11 | Using ssh user "heat-admin" for initial connection.01:31
cloudnull2020-02-19 22:46:11 | Using ssh key at "/home/zuul/.ssh/id_rsa_tripleo" for initial connection.01:31
cloudnullwhich would be the key used when none is defined and its auto-discovered01:31
cloudnulllooks like we just run - openstack overcloud node provision --stack overcloud --output /home/zuul/overcloud-baremetal-deployed.yaml /home/zuul/overcloud_baremetal_deploy.yaml01:32
stevebakercloudnull: oh. the provision command needs the exact same auto-discover logic, whatever that is01:33
cloudnullbefore that was just pulling from mistral , so we'll need to update that01:34
stevebaker        parser.add_argument(01:35
stevebaker            '--overcloud-ssh-key',01:35
stevebaker            default=os.path.join(01:35
stevebaker                os.path.expanduser('~'), '.ssh', 'id_rsa_tripleo'),01:35
stevebaker            help=_('Key path for ssh access to overcloud nodes.')01:35
stevebaker        )01:35
openstackgerritMerged openstack/tripleo-ansible master: tripleo_container_manage: disable logging for tasks using container_data  https://review.opendev.org/70873901:36
cloudnullso that default is the key used when the node is provisioned ?01:37
cloudnullhttps://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L603-L62501:38
cloudnullwhich looks like its used there?01:38
stevebakercloudnull: I think so yes. Also the provision command defaults to a public key, but the deploy command uses the private. So thats wrong too01:39
cloudnullso really, os.path.expanduser('~'), '.ssh', 'id_rsa_tripleo'), just needs to be set to the pub key ?01:40
cloudnullid_rsa_tripleo.pub01:41
stevebakercloudnull: I'm assuming the other way around, the provision command needs to default to the private id_rsa_tripleo01:41
cloudnulloh their different - https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_node.py#L619-L62301:43
cloudnulllooks like this is the function being used01:43
*** rfolco has quit IRC01:44
cloudnulland it seems that switch is used for different purposes across commands01:44
cloudnullboth private and public keys, in different methods01:45
cloudnullbut the use of baremetal.deploy_roles across modules, seems to expect different things ?01:46
cloudnullhttps://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_node.py#L635-L639 vs https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L614-L61801:48
cloudnullthe parsed arg between the two modules for parsed_args.overcloud_ssh_user is different01:48
stevebakercloudnull: yes they are. what does get_hosts_and_enable_ssh_admin do with parsed_args.overcloud_ssh_key?01:52
cloudnullit pulls a list of hosts and then runs the enable ssh admin workflow01:53
cloudnullwhich creates the tripleo-admin user01:53
stevebakercloudnull: using that key for initial access?01:54
cloudnullyes01:54
stevebakercloudnull: ok, here is what I think needs to happen. 1) provision command needs to default to id_rsa_tripleo.pub https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_node.py#L62101:55
stevebakercloudnull: 2) overcloud deploy _provision_baremetal needs to append '.pub' to parsed_args.overcloud_ssh_key (this code path is *only* hit if you want to do provision and deploy in one go. we're calling provision seperately)01:56
stevebakerhttps://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L61101:56
openstackgerritMerged openstack/tripleo-heat-templates master: placement: Remove Nova to Placement extraction step  https://review.opendev.org/70380401:59
cloudnullok02:00
stevebakercloudnull: heh, in my zuul generated node id_rsa and id_rsa_tripleo are the same key02:00
cloudnullyea they should be one in the same02:00
cloudnullwhich is confusing02:00
openstackgerritSteve Baker proposed openstack/python-tripleoclient master: Use correct default key file for baremetal provision  https://review.opendev.org/70878102:04
stevebakercloudnull: its a needed fix, but it may not fix your problem02:04
cloudnullyou beat me to it :D02:06
cloudnulldo we need to set that pub key entry here too https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L611-L61202:06
cloudnullhttps://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_node.py#L632-L63302:06
cloudnulllooks like the same code is in two palces.02:06
cloudnull**places02:06
stevebakercloudnull: for that command the argument is documented as being the public key, I've change the default to id_rsa_tripleo.pub instead02:08
cloudnullok02:08
cloudnullseems off that the same switch would have a different value and use the same underlying functions?02:09
cloudnullhttps://pasted.tech/pastes/20e4bb6026b7158afc110ece2dd893ddf5758636.raw02:10
cloudnullwould normalize the options, and should have the desired effect?02:10
cloudnullOK i gotta unplug for the evening. thanks for you help on this stuff stevebaker , i'll give this metalsmith things a better spin tomorrow02:14
stevebakercloudnull: yes, given that provision is a new command I'm fine with switching to a private key to make it consistent with other commands. A subset of that change would need to be backported to stable/train though, which is fine02:14
cloudnull++02:14
stevebakercloudnull: thanks very much, I'll abandon my change for yours02:14
openstackgerritKevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Use correct default key file for baremetal provision  https://review.opendev.org/70878102:15
cloudnull^ i just pushed to yours02:16
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Improve execution and add a port check  https://review.opendev.org/70765802:17
cloudnull^ see if that makes fs001 happier02:17
cloudnullok, now I go away for real.02:17
stevebakersweet02:17
stevebakerbye02:17
cloudnullhave a great night :D02:17
stevebakerstill day here :)02:17
cloudnullthanks again , I appreciate you answering my n00b questions.02:17
cloudnullexpect more soon :D02:17
openstackgerritSteve Baker proposed openstack/python-tripleoclient master: Use correct default key file for baremetal provision  https://review.opendev.org/70878102:20
stevebakeri just updated the commit message02:21
*** mschuppert has quit IRC02:29
*** Goneri has quit IRC02:34
openstackgerritEmilien Macchi proposed openstack/tripleo-upgrade stable/train: Configure undercloud_enable_paunch when set  https://review.opendev.org/70879002:47
*** openstackstatus has joined #tripleo02:56
*** ChanServ sets mode: +v openstackstatus02:56
*** rlandy|bbl is now known as rlandy03:03
openstackgerritwes hayutin proposed openstack/tripleo-quickstart-extras master: pip install contextlib2  https://review.opendev.org/70879103:13
openstackgerritwes hayutin proposed openstack/tripleo-quickstart master: ensure sudo is not called in ci.centos  https://review.opendev.org/70826403:14
openstackgerritDavid Vallee Delisle proposed openstack/os-net-config master: We need to use the MAC instead of PCI Address for Mellanox cards  https://review.opendev.org/70642903:16
*** psachin has joined #tripleo03:33
*** ykarel|away is now known as ykarel03:34
*** lbragstad has quit IRC03:39
*** stevebaker has quit IRC03:40
*** evrardjp has quit IRC03:42
*** evrardjp has joined #tripleo03:45
openstackgerritKevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Use correct default key file for baremetal provision  https://review.opendev.org/70878103:54
openstackgerritSagi Shnaidman proposed openstack/python-tripleoclient master: Use correct default key file for baremetal provision  https://review.opendev.org/70878103:58
sshnaidm|afkcloudnull, oops..03:58
openstackgerritSagi Shnaidman proposed openstack/python-tripleoclient master: Use correct default key file for baremetal provision  https://review.opendev.org/70878104:00
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Improve execution and add a port check  https://review.opendev.org/70765804:01
sshnaidm|afkcloudnull, returned back your patch04:02
*** jmasud has quit IRC04:12
*** jmasud has joined #tripleo04:14
*** udesale has joined #tripleo04:20
*** ramishra has joined #tripleo04:29
*** rlandy has quit IRC04:41
openstackgerritSagi Shnaidman proposed openstack/tripleo-ci master: DNM: test SA on c8 dry run  https://review.opendev.org/70880004:48
*** ramishra has quit IRC04:58
*** saneax has joined #tripleo05:01
*** ramishra has joined #tripleo05:06
openstackgerritRabi Mishra proposed openstack/tripleo-common master: Move plan container creation to utils  https://review.opendev.org/70880205:11
openstackgerritSagi Shnaidman proposed openstack/tripleo-ci master: Add centos8 dry run job for CI  https://review.opendev.org/70880305:12
openstackgerritSagi Shnaidman proposed openstack/tripleo-ci master: Add centos8 dry run job for CI  https://review.opendev.org/70880305:15
openstackgerritSagi Shnaidman proposed openstack/tripleo-ci master: DNM: test SA on c8 dry run  https://review.opendev.org/70880005:17
*** ccamacho has quit IRC05:17
*** mmedvede has quit IRC05:17
*** mmedvede has joined #tripleo05:18
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart master: Handle pip being installed as /usr/bin/pip3  https://review.opendev.org/70697905:21
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart-extras master: Use tripleo-operator-ansible for standalone  https://review.opendev.org/70577505:26
*** evrardjp has quit IRC05:34
*** evrardjp has joined #tripleo05:34
openstackgerritMerged openstack/tripleo-ansible stable/train: tripleo_container_manage: disable logging for tasks using container_data  https://review.opendev.org/70875405:38
*** ade_lee has quit IRC06:00
*** ade_lee has joined #tripleo06:00
*** udesale_ has joined #tripleo06:03
*** ratailor has joined #tripleo06:04
*** ratailor has quit IRC06:05
*** udesale has quit IRC06:06
*** ratailor has joined #tripleo06:07
*** marios|ruck has joined #tripleo06:08
*** lmiccini has joined #tripleo06:19
*** yolanda has joined #tripleo06:22
*** ramishra has quit IRC06:32
*** ramishra has joined #tripleo06:32
*** ahosam has joined #tripleo06:33
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Use CI mirrors for molecule jobs  https://review.opendev.org/70880806:47
*** jfrancoa has joined #tripleo06:49
*** abdysn has joined #tripleo06:51
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: DNM: test deps  https://review.opendev.org/70880906:53
sshnaidm|afkmarios|ruck, fyi https://review.opendev.org/#/c/708803/06:58
marios|ruckack sshnaidm|afk07:00
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-ci master: [WIP] Improve build images role for CentOS8  https://review.opendev.org/70859907:15
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Use CI mirrors for molecule jobs  https://review.opendev.org/70880807:21
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Use CI mirrors for molecule jobs  https://review.opendev.org/70880807:22
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: DNM: test deps  https://review.opendev.org/70880907:22
*** dpawlik has joined #tripleo07:31
*** paramite has joined #tripleo07:36
*** ratailor has quit IRC07:40
*** ratailor has joined #tripleo07:41
*** slaweq has joined #tripleo07:51
*** jpena|off is now known as jpena07:51
*** jbadiapa has joined #tripleo07:52
*** ykarel is now known as ykarel|lunch08:01
*** holser has joined #tripleo08:02
*** tesseract has joined #tripleo08:05
*** jmasud has quit IRC08:12
*** jmasud has joined #tripleo08:14
Tenguhello there08:25
TenguEmilienM: wow. yep, indeed, that's a Friday Patch for tripleo-lab I think :).08:25
*** rpittau|afk is now known as rpittau08:26
*** tosky has joined #tripleo08:29
*** iurygregory has joined #tripleo08:31
*** ccamacho has joined #tripleo08:32
*** bogdando has joined #tripleo08:35
*** tesseract has quit IRC08:38
*** tesseract has joined #tripleo08:39
openstackgerritJuan Badia Payno proposed openstack/tripleo-ansible master: BaR Rename the mysql_short_bootstrap_name  https://review.opendev.org/70881908:39
*** amoralej|off is now known as amoralej08:40
*** gfidente has joined #tripleo08:41
*** jpich has joined #tripleo08:51
openstackgerritC├ędric Jeanneret (Tengu) proposed openstack/tripleo-common master: Move away from "ss" and drop default verbose mode  https://review.opendev.org/70833908:54
*** avivgt has joined #tripleo08:55
*** pkopec has joined #tripleo09:01
*** shardy has joined #tripleo09:04
*** holser has quit IRC09:08
*** mschuppert has joined #tripleo09:08
*** holser has joined #tripleo09:08
openstackgerritJuan Badia Payno proposed openstack/tripleo-ansible master: BaR Rename the mysql_short_bootstrap_name  https://review.opendev.org/70881909:09
*** lucasagomes has joined #tripleo09:09
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Added overcloud images yaml for CentOS-8  https://review.opendev.org/70860609:12
*** ykarel|lunch is now known as ykarel09:13
*** suuuper has joined #tripleo09:16
*** tkajinam has quit IRC09:19
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart-extras master: Revert "Add tempest.api.compute.images test_delete_saving_image to skip"  https://review.opendev.org/70681709:25
*** derekh has joined #tripleo09:29
chandankumarcgoncalves, thanks for working on c8 support in dib09:39
chandankumarhttps://review.opendev.org/#/c/684308/09:39
chandankumarcgoncalves, I was looking for that09:39
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-ci master: [WIP] Improve build images role for CentOS8  https://review.opendev.org/70859909:40
*** xek_ has joined #tripleo09:46
*** StevenK has quit IRC09:52
*** StevenK has joined #tripleo09:52
openstackgerritJuan Badia Payno proposed openstack/tripleo-ansible master: BaR Rename the mysql_short_bootstrap_name  https://review.opendev.org/70881909:54
*** tesseract has quit IRC09:58
*** jaosorior has joined #tripleo09:59
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Added overcloud images yaml for CentOS-8  https://review.opendev.org/70860610:02
openstackgerritAbdallah Yasin proposed openstack/os-net-config master: Adding IPoIB pkeys interfaces to the os-net-config  https://review.opendev.org/70733110:03
*** alexmcleod has joined #tripleo10:07
*** tesseract has joined #tripleo10:11
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Fix dcn-hci resource mapping path  https://review.opendev.org/70883810:19
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Fix dcn-hci resource mapping path  https://review.opendev.org/70883810:38
openstackgerritMarios Andreou proposed openstack/tripleo-ci master: Pin kolla venv and package install workaround py2 queue bug  https://review.opendev.org/70884610:38
marios|ruckykarel: chandankumar: yuck ^^10:38
marios|ruckykarel: chandankumar: testing https://review.rdoproject.org/r/#/c/25080/10:41
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Use CI mirrors for molecule jobs  https://review.opendev.org/70880810:43
*** pbandark has joined #tripleo10:43
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: DNM: test deps  https://review.opendev.org/70880910:43
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-ci master: [WIP] Improve build images role for CentOS8  https://review.opendev.org/70859910:44
ykarelmarios|ruck, no need to pin package10:44
ykarelrpm10:44
ykarelit will be get from whatever is in repo10:44
ykarelfor repo u already proposed pin in rdoinfo, that will take care of it10:44
openstackgerritMerged openstack/tripleo-ci master: Fix scenario010 triggers  https://review.opendev.org/70872210:45
marios|ruckykarel: k i thought we wanted to pin there too.. will update after the test runs. also can make the git chckout conditional for py210:46
marios|ruckykarel: chandankumar: thanks10:46
openstackgerritMartin Kopec proposed openstack/ansible-role-collect-logs master: Dynamically generate a list of files to collect for rsync  https://review.opendev.org/70376510:50
*** pbandark has quit IRC11:01
*** shardy has quit IRC11:02
*** udesale_ has quit IRC11:03
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-ci master: [WIP] Improve build images role for CentOS8  https://review.opendev.org/70859911:03
*** shardy has joined #tripleo11:03
openstackgerritMarios Andreou proposed openstack/tripleo-ci master: Pin kolla venv install workaround py2 queue import error  https://review.opendev.org/70884611:04
*** rpittau is now known as rpittau|bbl11:20
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: WIP: try podman driver  https://review.opendev.org/70885111:23
*** sshnaidm|afk is now known as sshnaidm11:23
*** ykarel is now known as ykarel|afk11:24
*** jaosorior has quit IRC11:30
*** ansmith has joined #tripleo11:33
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: WIP: try podman driver  https://review.opendev.org/70885111:38
openstackgerritSagi Shnaidman proposed openstack/tripleo-ci master: DNM: c8 test  https://review.opendev.org/70885511:41
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-ci master: [WIP] Improve build images role for CentOS8  https://review.opendev.org/70859911:42
*** ansmith has quit IRC11:44
openstackgerritMichal Pryc proposed openstack/tripleo-ansible master: DNM - check ansible-playbook-3 on rhel8  https://review.opendev.org/70885611:48
*** dtantsur|afk is now known as dtantsur11:51
*** rfolco has joined #tripleo11:52
openstackgerritMerged openstack/tripleo-heat-templates master: NodeDataLookup utility should rely on python env  https://review.opendev.org/70871511:54
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: WIP: try podman driver  https://review.opendev.org/70885111:54
*** jpena is now known as jpena|lunch12:00
*** amoralej is now known as amoralej|lunch12:01
chandankumarcgoncalves, Hello, around?12:07
*** ccamacho has quit IRC12:14
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: WIP: try podman driver  https://review.opendev.org/70885112:24
*** ratailor has quit IRC12:26
openstackgerritSagi Shnaidman proposed openstack/tripleo-ci master: DNM: c8 test  https://review.opendev.org/70885512:26
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Fix dcn-hci resource mapping path  https://review.opendev.org/70883812:28
*** udesale has joined #tripleo12:34
*** beagles is now known as beagles_pto12:35
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Improve logic in introspection module  https://review.opendev.org/70728612:36
*** social has joined #tripleo12:45
*** numans has quit IRC12:49
*** psachin has quit IRC12:49
*** rlandy has joined #tripleo12:51
openstackgerritJose Luis Franco proposed openstack/tripleo-upgrade stable/train: Ensure bc package is installed for l3 ping tests.  https://review.opendev.org/70886212:51
*** shardy has quit IRC12:55
*** shardy has joined #tripleo12:56
abdysnTengu: hi, it seams https://review.opendev.org/#/c/705634/ is not added to the gate? does it need a recheck?12:57
Tenguabdysn: hmmm.... depends-on has merged, should have been triggered12:57
*** morazi has joined #tripleo12:58
openstackgerritJose Luis Franco proposed openstack/tripleo-heat-templates stable/stein: Force facts cache refreshing after OS upgrade.  https://review.opendev.org/70886312:58
*** raildo has joined #tripleo12:58
Tenguhmm... running recheck might drop the +1 from zuul in case we hit any weird things.12:59
Tengucomme on gate......12:59
Tenguah12:59
Tenguabdysn: starting gate.12:59
Tenguabdysn: I just added a second +W to trigger it.12:59
*** sshnaidm has quit IRC13:00
*** sshnaidm has joined #tripleo13:01
*** rh-jelabarre has joined #tripleo13:03
abdysnTengu: Great :)13:04
*** rpittau|bbl is now known as rpittau13:07
*** artom has quit IRC13:08
abdysnTengu: oh right, do you by chance know anyone who can provide the specs to build the haproxy container? i mean the haproxy version and configuration?13:09
Tenguabdysn: hm, that's all in kolla iirc13:09
abdysnah, should ask there then :) thanks.13:10
Tenguabdysn: so you have a partial answer here: https://opendev.org/openstack/tripleo-common/src/branch/master/container-images/overcloud_containers.yaml.j2#L26113:10
Tengushowing the possible overrides we're making13:11
abdysnThanks :)13:11
Tenguabdysn: and here's the kolla thing: https://opendev.org/openstack/kolla/src/branch/master/docker/haproxy13:12
*** marios|ruck is now known as marios|ruck|call13:12
*** ysandeep is now known as ysandeep|away13:13
*** mcornea has joined #tripleo13:14
abdysnTengu: great, appreciate your help :). we are trying to change the openssl version on the haproxy, but it seam that the haproxy is tightly coupled with the installed haproxy, so we want to try to build a new haproxy image that have what we want :).13:16
*** ccamacho has joined #tripleo13:17
Tenguabdysn: you might want to talk to bandini about that.13:19
abdysngreat, will do :)13:20
*** jpena|lunch is now known as jpena13:24
*** lbragstad has joined #tripleo13:26
*** amoralej|lunch is now known as amoralej13:28
*** pbandark has joined #tripleo13:30
*** paramite|clone has joined #tripleo13:33
*** ansmith has joined #tripleo13:34
*** paramite has quit IRC13:34
cloudnullmornings13:35
*** psachin has joined #tripleo13:37
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates stable/train: Remove BlockStorageCinderVolume service from certain DCN roles  https://review.opendev.org/70887813:47
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: paunch: fail early if containers weren't deployed by Paunch before  https://review.opendev.org/70888013:52
openstackgerritAthlan-Guyot sofer proposed openstack/paunch stable/train: Fallback to a rm -f action for podman  https://review.opendev.org/70888113:54
*** Goneri has joined #tripleo13:55
openstackgerritSorin Sbarnea proposed openstack/tripleo-ansible master: Upgrade molecule to 3.0a9  https://review.opendev.org/70802614:01
openstackgerritSorin Sbarnea proposed openstack/ansible-role-collect-logs master: Upgrade to molecule 3.0  https://review.opendev.org/70802014:02
*** abdysn has quit IRC14:03
openstackgerritSorin Sbarnea proposed openstack/tripleo-ansible master: Upgrade molecule to 3.0  https://review.opendev.org/70802614:04
*** marios|ruck|call is now known as marios|ruck14:06
*** ysandeep|away is now known as ysandeep14:09
openstackgerritJuan Badia Payno proposed openstack/tripleo-ansible master: BaR Rename the mysql_short_bootstrap_name  https://review.opendev.org/70881914:12
*** marios|ruck is now known as marios|ruck|afk14:13
chemmatbu: around ?14:14
openstackgerritMerged openstack/tripleo-validations master: Add basic molecule tests for uncovered roles  https://review.opendev.org/70415814:17
EmilienMcloudnull: hey14:17
cloudnullyo14:18
EmilienMcloudnull: have you already seen a race where14:18
EmilienMcloudnull: you create a systemd unit file with template module and right after you enable that service14:18
EmilienMand it fails to find the unit file14:18
EmilienMi can see it in a CI job14:18
EmilienMcloudnull: http://paste.openstack.org/show/789798/14:19
EmilienMwith more logs http://paste.openstack.org/show/789799/14:21
cloudnullEmilienM nope , ive not seen thta14:21
cloudnullcaused by async tasks maybe ?14:22
EmilienMno, I made the systemd tasks not using async for that reason14:22
EmilienMto avoid systemd races14:22
*** ykarel|afk is now known as ykarel14:23
*** marios|ruck|afk is now known as marios|ruck14:25
cloudnulli cant think of why there would be a race there then14:25
cloudnullmaybe that service was just not deployed ?14:25
*** leanderthal has joined #tripleo14:25
EmilienMunfortunately, I can't check if the systemd file actually exists, because CI logs don't collect it14:26
EmilienMcloudnull: the service is being upgraded:14:26
EmilienMhttps://b7fcf12d6809cf4f87e3-e088871e0bec453fba7cc9495a9471da.ssl.cf1.rackcdn.com/708790/1/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/9dd54ef/logs/undercloud/home/zuul/undercloud_upgrade.log14:26
openstackgerritGrzegorz Grasza proposed openstack/tripleo-ansible master: Make sure not to re-enroll already enrolled server  https://review.opendev.org/70440414:27
openstackgerritGrzegorz Grasza proposed openstack/tripleo-ansible master: Added files to allow cleanup when stack delete is called  https://review.opendev.org/70658814:27
openstackgerritJose Luis Franco proposed openstack/tripleo-upgrade master: Modify overcloud upgrade run loop.  https://review.opendev.org/70781414:28
cloudnullEmilienM maybe the daemon hadn't been reloaded?14:30
cloudnullso it doesn't see the new service after the container-rm ?14:30
openstackgerritGrzegorz Grasza proposed openstack/tripleo-ansible master: Add code to deregister nodes and services from freeipa  https://review.opendev.org/70022614:30
openstackgerritGrzegorz Grasza proposed openstack/tripleo-ansible master: Make sure not to re-enroll already enrolled server  https://review.opendev.org/70440414:30
openstackgerritGrzegorz Grasza proposed openstack/tripleo-ansible master: Added files to allow cleanup when stack delete is called  https://review.opendev.org/70658814:30
*** ekultails has joined #tripleo14:32
openstackgerritEmilien Macchi proposed openstack/tripleo-ci master: collect-logs: collect tripleo related systemd unit files  https://review.opendev.org/70888914:32
EmilienMzbr: ^ is it the right place to add more files?14:32
EmilienMor should I patch ansible-role-collect-logs ?14:33
EmilienMsshnaidm: ^14:33
sshnaidmEmilienM, yeah, it should be in tripleo-ci14:34
EmilienMthx14:34
sshnaidmEmilienM, no other changes needed14:34
EmilienMack14:34
*** jaosorior has joined #tripleo14:34
*** sshnaidm is now known as sshnaidm|afk14:34
zbryeah.14:36
EmilienMmatbu: hey, what does the tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates ?14:38
zbrEmilienM: sshnaidm|afk cloudnull ekultails : please read and comment on https://github.com/ansible-community/molecule/issues/2565 -- i want to make the mirror config more transparent.14:41
*** marios|ruck is now known as marios|ruck|call14:42
EmilienMzbr: awesome, would be a great addition indeed.14:42
zbri really do not like the volume mount approach, maybe we can find a less intrusive way of doing it14:42
zbrmost people do not know that molecule can also load a default molecule file, and is merging it with individual ones, that is another way to avoid copy/paste on each scenario.14:43
zbrapparently the volume mount is not so bad, seems to be ignored when file does not exist. still I will try to define it in a single place.14:45
openstackgerritAlex Schultz proposed openstack/tripleo-operator-ansible master: Add update run role  https://review.opendev.org/70874414:45
openstackgerritGiulio Fidente proposed openstack/tripleo-docs master: Extend external Swift docs for Ceph RadosGW  https://review.opendev.org/70889214:48
openstackgerritGrzegorz Grasza proposed openstack/tripleo-ansible master: Make sure not to re-enroll already enrolled server  https://review.opendev.org/70440414:48
openstackgerritGrzegorz Grasza proposed openstack/tripleo-ansible master: Added files to allow cleanup when stack delete is called  https://review.opendev.org/70658814:48
matbuEmilienM: it's doing a noop update14:51
EmilienMmatbu: ok so it's not updating containers right?14:51
matbuEmilienM: so it's just testing the framework14:51
matbuEmilienM: i think so, need to double check14:52
matbuEmilienM: for me the main interest is the validation of the python code mainly14:52
EmilienMok14:52
matbuEmilienM: very usefull for the removal of the mistral actions for example14:52
matbuEmilienM: but i can check, but if so, i think just removed and redeployed the same container14:53
matbuEmilienM: because it's deploying master to master (or release to release)14:53
openstackgerritKevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Use correct default key file and normalize the usage  https://review.opendev.org/70878114:53
EmilienMok14:53
EmilienMmatbu: is there a way to change an environment file or parameter before running the update?14:54
matbuEmilienM: in CI ?14:54
*** TrevorV has joined #tripleo14:54
EmilienMmatbu: i'm trying to see how we can test the transition from paunch managed containers to ansible in a minor update job14:54
matbuEmilienM: like inject new env file ?14:54
matbuEmilienM: hm i think at the tripleo-upgrade level14:55
EmilienMok, i'll take a look14:55
openstackgerritAlex Schultz proposed openstack/tripleo-ci master: Remove centos-7 container for U  https://review.opendev.org/70889314:55
matbuEmilienM: feel free to query if need .. we are a bit in the dark of the scale lab this week , because we need to finish before losing the lab14:56
bogdandoEmilienM: the similar situation was with upgrades from containers having always managed-by=paunch to custom values tripleo provides for new deployments14:56
EmilienMmatbu: ack, thanks14:56
bogdandowe had to add additional checks for paunch14:56
bogdandomaybe  the same approach for ansible may be taken14:57
bogdandoto not consider those "orphants" as lost or unmanaged14:57
ekultailszbr: I left a comment on the GitHub issue. I like the idea and I feel like we could make something like that work with minimal effort.14:57
*** abishop has joined #tripleo14:59
ade_leeEmilienM, sorry -- I'm confused.  according to https://review.opendev.org/#/c/701280/  we skip the scale_down tasks in the case that we have pre-provisioned nodes, right?15:01
ade_leeEmilienM, which is why my scale_down said "Ansible skipped."15:02
EmilienMbogdando: can you elaborate? i did some changes lately on how we treat orphans in tripleo_container_manage15:04
*** jdandrea has joined #tripleo15:04
openstackgerritMerged openstack/tripleo-upgrade stable/stein: Fix ping loss detection  https://review.opendev.org/70857515:05
*** jdandrea has quit IRC15:05
EmilienMade_lee: indeed15:06
EmilienMade_lee: I'll look at it today if I can15:06
*** jdandrea has joined #tripleo15:06
ade_leeEmilienM, cool thanks15:06
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add no_log to the podman login  https://review.opendev.org/70889515:07
EmilienMade_lee: yeah now I remember. We talked about it with sbaker 6 weeks ago and some work need to happen to lookup the physical resources from the stack and get the IP/hostname in split stack case15:08
bogdandoEmilienM: https://review.opendev.org/#/c/696589/15:08
bogdandoimagine the "managed-by: paunch" -> "managed-by: ansible" transition to follow that approach15:09
bogdando(as of Train containers will have managed-by tripleo-<RoleName> iirc)15:09
bogdandoor they may have that after applied minor update for any release15:10
openstackgerritMerged openstack/tripleo-operator-ansible master: Add update prepare role  https://review.opendev.org/70874215:10
bogdandoanyway, ansible can take those legacy values into account EmilienM15:11
*** social has quit IRC15:12
EmilienMbogdando: in the role, I made it "easier" and static: https://github.com/openstack/tripleo-ansible/blob/57cc4d80758d99bb0f9e11f479ef5a413425cccf/tripleo_ansible/roles/tripleo_container_manage/tasks/podman/create.yml#L5815:12
EmilienMi don't think we need to be able to change that15:12
bogdandoright, I mean you can use managed_by values to identify paunch-managed containers15:13
EmilienMbogdando: and also check out https://github.com/openstack/tripleo-ansible/blob/74996fba58c2efa5dc8f03551200a245a71ff3f3/tripleo_ansible/ansible_plugins/filter/helpers.py#L131-L13515:13
bogdandoright... maybe I didn't get the problem then15:13
EmilienMbogdando: managed-by tripleo-<RoleName> will disappear15:14
EmilienMit'll be managed-by tripleo-ansible15:14
bogdandoEmilienM: so what's the issue with "the transition from paunch managed containers to ansible in a minor update job"?15:16
bogdandoaside of https://review.opendev.org/#/c/708880/15:17
EmilienMbogdando: there is no issue so far. It's just that all containers previously managed by Paunch will be re-created15:17
EmilienMbogdando: the downtime should be minimal thanks to https://review.opendev.org/#/c/708247/15:17
bogdandomhm, for upgrades, not updates?15:17
EmilienMhttps://review.opendev.org/#/c/708247/1/tripleo_ansible/roles/tripleo-container-manage/tasks/podman/create.yml15:17
EmilienMbogdando: updates as well, managed_by changed.15:18
EmilienMwe could make it so it uses the same managed_by as before with paunch but to me it doesn't make sense to keep managed_by: "tripleo-{{ tripleo_role_name }}"15:19
bogdandoI'm not sure changing paunch to ansible within minor updates is good idea15:19
*** psachin has quit IRC15:22
openstackgerritMerged openstack/tripleo-upgrade master: Force error when "launch workload" task fails  https://review.opendev.org/70866115:25
openstackgerritGiulio Fidente proposed openstack/tripleo-docs master: Extend external Swift docs for Ceph RadosGW  https://review.opendev.org/70889215:26
openstackgerritJuan Badia Payno proposed openstack/tripleo-ansible master: BaR Rename the mysql_short_bootstrap_name  https://review.opendev.org/70881915:33
openstackgerritEmilien Macchi proposed openstack/tripleo-upgrade stable/train: Configure undercloud_enable_paunch when set  https://review.opendev.org/70879015:40
*** marios|ruck|call is now known as marios|ruck15:40
*** sshnaidm|afk is now known as sshnaidm15:43
*** jfrancoa has quit IRC15:44
*** artom has joined #tripleo15:47
openstackgerritMerged openstack/tripleo-heat-templates master: Generate /etc/hosts early on both under and overcloud  https://review.opendev.org/70563415:48
*** jpich has quit IRC15:52
*** jpich has joined #tripleo15:52
*** ratailor has joined #tripleo15:56
*** bogdando has quit IRC15:57
sshnaidmzbr, you comment here is incorrect: https://review.opendev.org/#/c/708808/15:57
openstackgerritMerged openstack/paunch stable/stein: Do not force remove containers  https://review.opendev.org/70466915:57
sshnaidmzbr, it's totally possible to run molecule jobs locally and anywhere15:57
sshnaidmzbr, you should try to run it locally before15:58
openstackgerritAlex Schultz proposed openstack/ansible-role-container-registry master: Don't log login  https://review.opendev.org/70890215:59
zbrsshnaidm: yep, i discovered that the volume does not generate an error when file is missing.15:59
sshnaidmzbr, if you have better solution right now - please write it, if not - let's go with that approach15:59
zbrsshnaidm: i am working on one right now, involves https://github.com/ansible-community/molecule/pull/2566 and the linked bug.15:59
sshnaidmEmilienM, cloudnull mwhahaha please take a look: https://review.opendev.org/#/c/708808/16:00
zbri am confident that we can avoid having to add it to >50 files.16:00
sshnaidmzbr, when it will be ready, you're more than welcome to try it..16:00
EmilienM      rdo_mirror: http://mirror.gra1.ovh.openstack.org:8080/rdo16:00
EmilienMhttps://36d0834365e8b1dc2848-76fc92e5e7c4e5a091c792a95503ad1d.ssl.cf1.rackcdn.com/708808/4/check/tripleo-ansible-centos-7-molecule-tripleo_packages/4db18b1/reports.html16:00
EmilienMnice!16:01
EmilienMzbr: it sounds like a great interim solution until we get something cleaner, isn't?16:01
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add no_log to the podman login  https://review.opendev.org/70889516:01
*** leanderthal has quit IRC16:03
EmilienMweshay|ruck, mwhahaha: https://review.opendev.org/#/c/708808/ will make our molecule more reliable and zbr proposed a plan to integrate a new interface in Molecule itself. https://github.com/ansible-community/molecule/issues/256516:03
zbrEmilienM: cloudnull: sshnaidm: are you aware of the existense of https://hub.docker.com/repository/docker/pycontribs/centos ?16:03
EmilienMnegative16:04
*** TrevorV has quit IRC16:04
zbron most molecule jobs, I use these because they are preloaded with python, avoiding the need to install it.16:04
cloudnullsshnaidm can you add that change here too https://github.com/openstack/tripleo-ansible/blob/master/_skeleton_role_/molecule/default/molecule.yml16:04
sshnaidmcloudnull, yeah, totally16:04
cloudnullotherwise your change looks good to me16:04
*** rfolco is now known as rfolco|doctor16:04
*** ykarel is now known as ykarel|afk16:05
zbrthis speeds up provisioning considerably, but there is a catch: you need to be careful when to use them, or go for pure ones.16:05
sshnaidmcloudnull, is it fine to add it on top?16:05
cloudnullsure16:05
zbr lets say that these are the container version of the nodepool images :D16:05
sshnaidmcloudnull, then let's merge it, and I adding the followup..16:06
cloudnullmerging16:07
EmilienM++16:07
EmilienMwe'll need that on train16:07
*** jpena is now known as jpena|off16:07
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Add CI proxy settings to role skeleton  https://review.opendev.org/70890316:08
sshnaidmcloudnull, ^^16:08
sshnaidmzbr, take a look also https://review.opendev.org/#/c/708851/16:12
zbrsshnaidm: tbh, I would be surprised to see podman driver working considering that I do not remember seeing it running on CI for more than month16:14
sshnaidmzbr, it worked for me today16:15
zbrthat is good to know16:15
zbrwe need to enable it back on CI16:15
sshnaidmzbr, but Ci version of ansible may be too old..16:15
sshnaidmAnsible Version: 2.8.816:15
zbrmy plan is to remove both docker and podman drivers from molecule core and replace them with generic containers driver, one that use first backend that it finds.16:16
zbrthis will allow us to control which one is used by the jobs from outside, without touching each molecule.yml file.16:16
sshnaidmzbr, "first backend that it finds" open a lot of options to fail..16:17
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: DNM - wait for systemd files to be on the host before reloading service  https://review.opendev.org/70890416:17
sshnaidmzbr, there should be option to configure a specific backend16:18
zbryep16:18
sshnaidmzbr, and also podman has little bit different options and behavior16:18
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible stable/train: DNM - wait for systemd files to be on the host before reloading service  https://review.opendev.org/70890616:18
sshnaidmso instead of replacing, I'd add additional "generic" driver16:19
zbri know, hiding the differences could prove an interesting challenge.16:19
sshnaidmI think it's gonna be also generic "container" module for podman and docker in ansible, so it will be possible to use it in molecule as is, instead of working on this in molecule itself16:19
zbri am sure we will still need to allow to pass-over backend specific options16:20
openstackgerritEmilien Macchi proposed openstack/tripleo-upgrade stable/train: DNM - test upgrades to ansible managed containers with a sleep  https://review.opendev.org/70890716:20
zbrtrue, nobody say we have to drop these16:20
sshnaidmzbr, because the less molecule deals with backends, the better for all16:20
zbrwe may have different oppinions here, if you like DIY, you probably need to use only delegated, or even better, do everyting in ansible, bash,... :D16:21
zbrthe max value I see on molecule is that it hides the provisioning of test hosts, so I do not have to waste my time doing that work.16:22
EmilienMmwhahaha: I found a new one :D16:23
mwhahahaorly16:23
EmilienMhttps://05badc18f82a08bb81f1-14103d98dfeab14adaa7d6f3b3cde1a9.ssl.cf1.rackcdn.com/700738/37/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/1c93ad4/logs/undercloud/home/zuul/undercloud_upgrade.log16:23
EmilienMundercloud upgrade failed16:23
EmilienMeverything is fine16:23
EmilienM"Deployment successful!"16:23
mwhahahathat's not new16:23
EmilienMhttps://review.opendev.org/#/c/702161/16:24
EmilienM?16:24
mwhahahai think so16:24
EmilienMhttps://bugs.launchpad.net/tripleo/+bug/185918216:24
openstackLaunchpad bug 1859182 in tripleo "tripleoclient report successful upgrade when ansible failed" [High,Fix released] - Assigned to Rabi Mishra (rabi)16:24
mwhahahasee not new16:25
mwhahahaDO TRY AND KEEP UP16:25
EmilienMwell it was "fixed"16:25
EmilienMbut i still see it16:25
EmilienMI'll check the client version16:26
mwhahahayea there's probably an incorrect return code being returned or something16:26
weshay|ruckha..16:26
EmilienMNone: AnsibleRunnerException: Ansible execution failed. playbook: deploy_steps_playbook.yaml, Run Status: failed, Return Code: 216:27
mwhahahai see it16:28
mwhahahado you want to know?16:28
mwhahahaor should i let you figure it out16:28
EmilienM:)16:28
EmilienMteach me plz16:29
mwhahahaNO16:29
*** bandini has quit IRC16:29
EmilienMa good news is that the upgrade isn't broken on other patches so we didn't miss a patch which break upgrades16:30
*** jfrancoa has joined #tripleo16:30
EmilienMmwhahaha: it's a standalone thing I think because in undercloud it catches the error16:30
EmilienMbut I haven't digged much yet16:30
* EmilienM thinks mwhahaha is writting the patch now16:31
mwhahahahttps://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L1313-L133216:31
mwhahahais the hint16:31
*** ccamacho has quit IRC16:32
zbrdo we use the same "platforms:" block on all/most of molecule.yml files from tripleo-ansible?16:32
mwhahahareally the problem starts on 1333, but anyway i'll poke at it later16:32
* mwhahaha is doing other things16:32
zbrif so, you will be impressed how much we can remove from molecule.yml....16:32
*** jpich has quit IRC16:39
*** raildo has quit IRC16:39
*** jpich has joined #tripleo16:39
*** morazi has quit IRC16:40
EmilienMplease all don't run check-rdo until https://bugs.launchpad.net/tripleo/+bug/1863920 is closed16:41
openstackLaunchpad bug 1863920 in tripleo "(intermittent?) cli-enable-ssh-admin.yaml fails during the overcloud deploy" [Critical,In progress] - Assigned to Sagi (Sergey) Shnaidman (sshnaidm)16:41
EmilienMOVB jobs fs001 at least will fail anyway until this bug is closed16:41
weshay|ruckyay16:42
weshay|ruckTHIS MAN SPEAKS TRUTH16:42
weshay|ruckand french16:42
EmilienMthe patch that will fix it is https://review.opendev.org/#/c/708781/16:42
*** weshay|ruck changes topic to "CI Status: GREEN'ish, OVB RED 1863920| community irc meeting Tues@1400 UTC - tripleo-ci-community meeting Tues@1330 UTC | https://docs.openstack.org/tripleo-docs/latest/"16:42
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Disable Paunch by default Standalone/Overcloud  https://review.opendev.org/70073816:43
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Disable Paunch by default Standalone/Overcloud  https://review.opendev.org/70073816:43
*** slaweq has quit IRC16:43
EmilienMweshay|ruck: thx16:43
*** jdandrea has quit IRC16:44
*** lmiccini has quit IRC16:44
*** raildo has joined #tripleo16:45
*** dtantsur is now known as dtantsur|afk16:51
*** TrevorV has joined #tripleo16:51
openstackgerritMarios Andreou proposed openstack/tripleo-docs master: WIP document the dlrn_hash_tag featureset_override testproject  https://review.opendev.org/70891416:52
*** rpittau is now known as rpittau|afk16:53
zbrrevote on https://review.opendev.org/#/c/708026/16:54
*** jaosorior has quit IRC16:55
*** jdandrea has joined #tripleo16:55
*** jdandrea has quit IRC16:57
ramishraEmilienM: https://bugs.launchpad.net/tripleo/+bug/185918216:57
openstackLaunchpad bug 1859182 in tripleo "tripleoclient report successful upgrade when ansible failed" [High,In progress] - Assigned to Rabi Mishra (rabi)16:57
*** jdandrea has joined #tripleo16:57
*** pbandark has quit IRC16:57
ramishraEmilienM: https://bugs.launchpad.net/tripleo/+bug/1859182 was fixed but now again broken by https://review.opendev.org/#/c/704390/3/tripleoclient/utils.py16:58
openstackLaunchpad bug 1859182 in tripleo "tripleoclient report successful upgrade when ansible failed" [High,In progress] - Assigned to Rabi Mishra (rabi)16:58
*** jdandrea has quit IRC16:58
*** lucasagomes has quit IRC16:58
*** jdandrea_ has joined #tripleo16:58
*** tesseract has quit IRC17:00
openstackgerritMerged openstack/tripleo-ansible master: Don't fail symlinking puppet modules if directory exists  https://review.opendev.org/70828517:01
openstackgerritMerged openstack/tripleo-ansible master: Use CI mirrors for molecule jobs  https://review.opendev.org/70880817:01
*** jdandrea_ has quit IRC17:03
*** marios|ruck is now known as marios|out17:04
*** abishop has quit IRC17:06
*** ratailor has quit IRC17:10
openstackgerritSorin Sbarnea proposed openstack/ansible-role-collect-logs master: Assure pyyaml is installed before running sova test  https://review.opendev.org/70378117:14
openstackgerritSorin Sbarnea proposed openstack/ansible-role-collect-logs master: Upgrade to molecule 3.0  https://review.opendev.org/70802017:15
openstackgerritRabi Mishra proposed openstack/python-tripleoclient master: Revert "Add raised exception when playbook fails"  https://review.opendev.org/70892617:16
ramishracloudnull: ^^, I think that patch brings back https://bugs.launchpad.net/tripleo/+bug/185918217:17
openstackLaunchpad bug 1859182 in tripleo "tripleoclient report successful upgrade when ansible failed" [High,In progress] - Assigned to Rabi Mishra (rabi)17:17
*** chem has quit IRC17:21
*** marios|out has quit IRC17:25
*** jpena|off is now known as jpena17:27
*** chem has joined #tripleo17:28
*** suuuper has quit IRC17:28
zbrEmilienM: cloudnull : revote https://review.opendev.org/#/c/708026/17:29
EmilienMramishra: nice catch17:30
*** jfrancoa has quit IRC17:31
*** udesale has quit IRC17:33
cloudnullzbr done17:34
*** evrardjp has quit IRC17:34
*** evrardjp has joined #tripleo17:35
weshay|ruckALL https://lists.rdoproject.org/pipermail/dev/2020-February/009279.html17:36
cloudnullramishra something else must be going on, if the playbook exits non-0 it raises an exception, without the fix being reverted the playbook will always return with the status code, even if that status code is >017:37
*** dking_desktop has joined #tripleo17:38
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible stable/train: Use CI mirrors for molecule jobs  https://review.opendev.org/70893017:38
EmilienMsshnaidm: ^ backport17:38
cloudnullramishra EmilienM maybe the patch has merged but its not in the current package ?17:39
ramishracloudnull: don't understand what you mean. it's not the playbook status code but the return from run_ansible_playbook() call which matters17:39
*** jpich has quit IRC17:40
cloudnullrun_ansible_playbook returns the playbook status code17:40
ramishrait won't if you raise an exception17:40
cloudnullif it fails we raise an exception17:41
cloudnullwhich would exit without any return17:41
cloudnulland halt the run17:41
ramishrayes, but we check the rc in https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L135117:41
*** shardy has quit IRC17:42
*** avivgt has quit IRC17:42
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: WIP: try podman driver  https://review.opendev.org/70885117:42
ramishrain the finally block.. If you want to raise an expection you've to refactor the finally block17:42
cloudnullhttps://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L1320-L132817:43
cloudnullthe rc object is set in that try block which traps Exception17:43
ramishracloudnull: https://review.opendev.org/#/c/702161/17:44
ramishrareturn and raise expection are mutually exclusive17:44
ramishrathat's what that fix was about17:44
cloudnulli understand that,17:44
ramishrathen how would you get rc when you raise an exception in run_ansible_playbook()17:45
cloudnullbut you will have to refactor all of the playbook runs to support returning the rc always17:45
cloudnullyou wouldn't you'd fix the except Exception17:45
weshay|ruckTengu, what's the story here?17:45
weshay|ruckFeb 20 17:20:44 standalone.localdomain healthcheck_metrics_qdr[289453]: Error: exec failed: container_linux.go:345: starting container process caused "exec: \"/openstack/healthcheck\": permission denied": OCI runtime permission denied error17:45
weshay|ruckFeb 20 17:20:43 standalone.localdomain systemd[1]: tripleo_metrics_qdr_healthcheck.service: main process exited, code=exited, status=126/n/a17:45
cloudnulland trap a good exception instead of everything under the sun17:45
weshay|ruckhttps://043e10ad019e85b50614-986c99e3b3126abac54dc88cbca785e8.ssl.cf1.rackcdn.com/707361/2/gate/tripleo-ci-centos-7-scenario001-standalone/190b36f/logs/undercloud/var/log/extra/failed_services.txt17:45
cloudnullif https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L1329 is hit rc != 0 and you can set it accordingly17:46
cloudnulllike we do here https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L122817:47
ramishracloudnull: I mentioned if you want to refactor the finaly block you can, but catching execption and then put some return code there would not be correct17:47
ramishrathe fact that you're expecting a return code when rasing an exception is inocrrect17:48
cloudnullthe fact we're traping Exception is incorrect :D17:48
ramishrathat's old code17:48
cloudnullold == still wrong17:48
cloudnullwe can change it17:49
cloudnullhttps://github.com/openstack/python-tripleoclient/search?q=run_ansible_playbook&unscoped_q=run_ansible_playbook17:49
ramishraI did not say it's not wrong..  but we should not probably break working old code with new code right;)17:49
cloudnullbut then we need to update all those to handle the rc info17:49
cloudnullso either way something is going to change17:50
cloudnulli.e. raised exception or implementing rc handling everytime the function is called17:51
*** gfidente has quit IRC17:52
ramishraI don't see any bug, if there was an issue your patch was fixing something17:52
ramishraWas https://review.opendev.org/#/c/704390/ fixing any bug?17:53
*** avivgt has joined #tripleo17:54
cloudnulli dont think there was a filed bug17:54
ramishraif not, I would prefer we revert it and then fix the way we want in a subsequent patch17:54
cloudnullit came up here in channel17:55
*** morazi has joined #tripleo17:56
cloudnullwe're already defining the rc here https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L1228 i think it would be better to just redefine it https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L1329 which fixes that rc issue in the finally.17:57
ramishrawe should definitely have bugs for issues17:57
cloudnullotherwise we have to add rc handling everywhere in the client17:57
ramishraI don't like that approach.. anyway time for me to hit bed17:58
cloudnullI don't like the other approach, so we're at an impasse :)17:58
*** avivgt has quit IRC17:59
ramishracloudnull: I mean we can leave it for more people to provide opinion.. What I'm saying is lets revert the patch.. Have a proper bug on what issues we've at other places without rasing an exception and then fix it the way majority agrees on18:00
ramishrabut I'll leave it to you, if you don't want to revert and fix it directly and keep that regression open18:01
*** derekh has quit IRC18:02
cloudnullyour revert means regression elsewhere. the majority can merge that review if they thinks its the way we need to go. my opinion is expressed in the review18:03
ramishraI'm really shutting shop now.. goodnight18:03
cloudnullhave a good one .18:03
ramishracloudnull: yeah, but we don't have a bug for that;) anyway bye18:03
*** holser has quit IRC18:09
*** chandankumar is now known as raukadah18:14
openstackgerritKevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Add handler for exception raising  https://review.opendev.org/70894218:16
cloudnullramishra EmilienM ^ should be a proper fix18:16
openstackgerritMerged openstack/paunch master: Fallback to a rm -f action for podman  https://review.opendev.org/70834118:16
EmilienMcloudnull: thx, I'll look at it shortly18:20
*** cfontain has joined #tripleo18:28
*** jpena is now known as jpena|off18:34
*** cfontain has quit IRC18:34
*** ykarel|afk is now known as ykarel|away18:34
*** amoralej is now known as amoralej|off18:36
zbrcloudnull: i release 3.0.1 few minutes ago, https://github.com/ansible-community/molecule/releases -- feel free to use it to remove ~3000-3500 lines of molecule.yml from tripleo-ansible18:42
cloudnull?18:43
zbrmany of the files could become empty18:43
zbrread https://github.com/ansible-community/molecule/pull/256618:43
cloudnullso we can store all of the configs in .config/molecule/config.yml ?18:44
zbrmolecule load the default values from the config file, so you don't have to write anything in individual scenarios, or just the stuff you want different than basic config.18:44
zbryep18:44
cloudnullneat18:44
zbrto common part18:44
zbrplatforms is inhereted too, if you do not alter it, is not "merged" because is a list.18:45
zbrthe original feature is quite old, i just made it work with in-repo configs.18:46
zbrthe user config was not of much use to me.18:46
openstackgerritEmilien Macchi proposed openstack/paunch stable/train: Fallback to a rm -f action for podman  https://review.opendev.org/70888118:53
*** saneax has quit IRC18:55
*** ansmith_ has joined #tripleo18:56
*** ansmith has quit IRC18:59
*** cfontain has joined #tripleo18:59
*** cfontain has quit IRC19:06
*** pbandark has joined #tripleo19:10
*** stevebaker has joined #tripleo19:15
openstackgerritSorin Sbarnea proposed openstack/ansible-role-container-registry master: Migrated to molecule 3.0  https://review.opendev.org/70895319:15
*** rlandy is now known as rlandy|mtg19:16
weshay|ruckthere I wf.. https://review.opendev.org/#/c/708930/ EmilienM :/19:18
weshay|ruck;P19:18
zbrcloudnull: i discovered that https://opendev.org/openstack/ansible-plugin-container-connection doesn't even have a .gitreview file19:20
cloudnullnot been a lot of traction there.19:20
cloudnullsadly19:20
zbrgerrit may not be the right place to see traction, is not github19:21
*** pbandark has quit IRC19:22
weshay|ruckzbr, did you find a fix to the stupid queens issue btw?19:23
EmilienMweshay|ruck: lol19:23
zbrweshay|ruck: i send you the link19:23
weshay|ruckzbr, but I'm too stoopid..19:24
zbrweshay|ruck: i am not sure where it needs to be done, but you need to https://github.com/ansible-community/molecule/blob/master/tools/test-setup.sh#L1119:24
zbr"zipp<0.6.0;python_version=='2.7'"19:24
weshay|ruckzbr, /me wonders if ansible deal w/ that19:25
weshay|ruckk19:25
zbror you risk having pip installing an incompatible newer version19:25
weshay|ruckso just make sure zipp is pinned under 0.6.019:25
zbrpick the exact condition!19:25
zbrthis needs to be only for old pythons19:25
zbrif you pre-install it, you should be fine.19:26
*** rlandy|mtg is now known as rlandy19:34
openstackgerritwes hayutin proposed openstack/tripleo-quickstart-extras master: pip install zipp to fix contextlib2 error  https://review.opendev.org/70879119:35
weshay|ruckzbr, ^19:35
*** cfontain has joined #tripleo19:35
openstackgerritwes hayutin proposed openstack/tripleo-ci master: update centos8 build containers to current-tripleo build  https://review.opendev.org/70895619:38
openstackgerritDincer Celik proposed openstack/tripleo-common master: Cleanup template overrides for rabbitmq  https://review.opendev.org/69788819:47
*** cfontain has quit IRC19:51
*** ansmith_ has quit IRC19:55
fultonjtripleo-ci-centos-7-containerized-undercloud-upgrades has been failing often with Failed to start tripleo_memcached.service: Unit not found20:06
fultonje.g. https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6d5/708513/1/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/6d5a3b8/logs/undercloud/home/zuul/undercloud-ansible-V8ZRHj/ansible.log20:06
fultonjhas anyone seen this ^ ?20:06
fultonjweshay|ruck: ^ ?20:08
weshay|rucklooking.. not familiar yet20:09
fultonji can open a LP bug20:09
fultonji've rechecked https://review.opendev.org/#/c/708513/ a few times20:10
fultonjdidn't want to recheck again if it's a known issue20:10
weshay|ruckfultonj, http://dashboard-ci.tripleo.org/d/jobs/jobs-exploration?orgId=1&var-influxdb_filter=job_name%7C%3D%7Ctripleo-ci-centos-7-containerized-undercloud-upgrades20:11
fultonjso if i recheck there's a 76% chance it will pass20:12
weshay|ruckit's mostly passing in the gate20:12
weshay|ruckhttp://dashboard-ci.tripleo.org/d/jobs/jobs-exploration?orgId=1&var-influxdb_filter=job_name%7C%3D%7Ctripleo-ci-centos-7-containerized-undercloud-upgrades&var-influxdb_filter=pipeline%7C%3D%7Cgate20:13
fultonjoh yeah20:13
fultonjweshay|ruck: i guess i'll recheck then20:14
fultonjthanks mr ptl :)20:15
* weshay|ruck wonders if the db connection is up and down20:16
fultonjdb connection?20:16
fultonjthe failure in this instance was a missing unit20:16
* weshay|ruck poking around20:17
weshay|ruckfultonj, http://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6d5/708513/1/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/6d5a3b8/logs/undercloud/var/log/extra/podman/podman_allinfo.log20:18
fultonjoh20:18
fultonjthe memcached container is up20:19
weshay|ruckand down20:19
fultonjflapping20:19
weshay|ruckaye20:19
*** jfrancoa has joined #tripleo20:20
fultonji have two independent patches stuck it20:21
fultonj2 rechecks each today20:21
weshay|ruckfultonj, ya.. let's get a bug on it20:21
fultonjweshay|ruck: ok, i'll open a bug20:21
fultonjgiven the passing rate i'll recheck my two jobs unless you object20:22
*** paramite|clone has quit IRC20:23
fultonjweshay|ruck: https://bugs.launchpad.net/tripleo/+bug/186410520:27
openstackLaunchpad bug 1864105 in tripleo "memcached container flapping on train upgrades job" [Undecided,Triaged]20:27
weshay|ruckfultonj, thanks20:30
fultonjsure20:30
openstackgerritMerged openstack/tripleo-heat-templates master: Fix dcn-hci resource mapping path  https://review.opendev.org/70883820:35
openstackgerritMerged openstack/tripleo-heat-templates stable/stein: Force facts cache refreshing after OS upgrade.  https://review.opendev.org/70886320:35
openstackgerritMerged openstack/ansible-role-collect-logs master: Fix idempotence of infrared scenario  https://review.opendev.org/70861320:35
openstackgerritMerged openstack/tripleo-ansible master: Upgrade molecule to 3.0  https://review.opendev.org/70802620:37
mwhahahacloudnull: did you figure out the ssh thing yet?20:41
cloudnullmwhahaha - https://review.opendev.org/#/c/708781 - i think so20:43
cloudnulli currently fighting the "how do i setup novaless environments" right now to verify20:43
cloudnullbut the gate looks promissing20:44
cloudnulltripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 - passed20:44
mwhahahayikes20:44
mwhahahaoh it's cause id_rsa_tripleo20:53
*** artom has quit IRC20:54
openstackgerritAlex Schultz proposed openstack/ansible-role-container-registry master: Bump molecule version  https://review.opendev.org/70896420:57
openstackgerritAlex Schultz proposed openstack/ansible-role-container-registry master: Don't log login  https://review.opendev.org/70890220:58
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add strategy optimization  https://review.opendev.org/70469120:59
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add additional strategy optimization  https://review.opendev.org/70444321:00
*** TrevorV has quit IRC21:03
*** ansmith_ has joined #tripleo21:06
*** jfrancoa has quit IRC21:06
openstackgerritwes hayutin proposed openstack/python-tripleoclient master: turn off temp dir cleanup in container builds  https://review.opendev.org/70896721:13
*** mcornea has quit IRC21:13
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Add release file for master CentOS 8  https://review.opendev.org/70896821:18
EmilienMfultonj: hey21:20
EmilienM[Bug 1864105] [NEW] memcached container flapping on train upgrades job21:20
openstackbug 1864105 in tripleo "memcached container flapping on train upgrades job" [Critical,Triaged] https://launchpad.net/bugs/186410521:20
fultonjEmilienM: hi21:20
EmilienMit's funny21:20
fultonjyeah21:20
EmilienMthat's what I've been investigating all day21:21
EmilienMlet's talk about it21:21
EmilienMso I added a task to make sure we have the file21:21
EmilienMcheck out http://paste.openstack.org/show/789811/21:21
EmilienMnormally, the module itself should do the reload before enabling the unit21:21
EmilienMbut I suspect a race21:21
EmilienMcloudnull suggested something like http://paste.openstack.org/show/789812/21:22
EmilienMto force the reload in a standalone task21:22
EmilienMI was going to try it next21:22
mwhahahaisn't there a bug around reload21:23
fultonjsec21:23
EmilienMI know nothing about a bug around reload21:23
*** Goneri has quit IRC21:25
*** xek_ has quit IRC21:26
* fultonj reads21:26
fultonj"Wait for /etc/systemd/system/tripleo_memcached.service to be created]" ack21:26
EmilienMyeah I tried this but it doesn't helop21:26
fultonjwhen container_sysd_name in containers_changed21:27
fultonjwe need to get a log of what made the container die21:27
EmilienMit didn't die21:28
EmilienMit restarted with a new image21:28
EmilienMit's an upgrade job21:28
EmilienMit was deleted, then re-created21:28
fultonjso it's not flapping21:28
EmilienMno I think systemd doesn't know that the unit file is there yet21:28
EmilienMwhich is weird21:28
*** raildo has quit IRC21:28
EmilienMsince it works fine in a regular deployment21:28
fultonji assumed it was flapping ; i assumed restart=always21:29
fultonjso it tries to start hits what i thought would be the root cause, fails, restarts etc21:29
* fultonj should look at the unit file though21:29
EmilienMmwhahaha: https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/system/systemd.py#L38821:30
EmilienMI haven't found any issue in the module21:30
EmilienMfultonj: unit file isn't collected until https://review.opendev.org/#/c/708889/21:30
* fultonj reading artificats of job21:31
*** jbadiapa has quit IRC21:31
fultonjoh21:31
EmilienMfultonj: but I added depends-on on some patches and I can see the unit file exists21:31
EmilienMfultonj: take https://review.opendev.org/#/c/708907/ for example21:31
EmilienMyou have the unit file and the same error21:31
EmilienMhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_574/708907/1/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/5749c07/21:31
mwhahahai vaguely recall a patch where the ansible bits didn't work. i found a 2.2 bug tho21:31
mwhahahaprobably an old issue21:32
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage/systemd: force systemd daemon reload  https://review.opendev.org/70890421:32
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Disable Paunch by default Standalone/Overcloud  https://review.opendev.org/70073821:32
EmilienMlet's see how that work with a manual relaod21:33
openstackgerritMerged openstack/tripleo-common stable/stein: Make ()'s around step|int optional  https://review.opendev.org/70839221:35
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: DNM - test https://review.opendev.org/#/c/708942  https://review.opendev.org/70897121:36
EmilienMcloudnull: ^ testing your patch21:36
EmilienMcloudnull: I put it on top of https://review.opendev.org/#/c/700738 on purpose21:36
EmilienMmhh wait21:36
EmilienMlet me change that21:37
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: DNM - test https://review.opendev.org/#/c/708942  https://review.opendev.org/70897121:37
EmilienMcloudnull: now ^ will fail if your tripleoclient patch works21:37
*** ansmith_ has quit IRC21:38
EmilienMcloudnull: since disabling Paunch seems to fail on the containerized undercloud job for now (systemd issue)21:38
* cloudnull fingers crossed 21:47
EmilienMstevebaker: I can't stay longer today but we need to chat about re: https://review.opendev.org/#/c/701280/21:51
EmilienMstevebaker: ade_lee was doing scale down with deployed servers and he apparently needs some tasks to run21:51
fultonjEmilienM: thanks for digging into that21:51
EmilienMfultonj: yeah no worries. I'll get this fixed before eow hopefully.21:52
EmilienMfultonj: I need this fixed so I can turn off paunch by default21:52
EmilienMand it won't break upgrades21:52
stevebakerEmilienM: ack, shall we talk your monday?21:52
EmilienMstevebaker: I'll read irc later but if you have ideas I'm open to hear about them21:52
EmilienMstevebaker: yeah, I'll put something in our calendar if you don't mind21:52
stevebakerEmilienM: sure thing21:53
*** avivgt has joined #tripleo21:56
*** pbandark has joined #tripleo21:56
openstackgerritMerged openstack/tripleo-ci master: collect-logs: collect tripleo related systemd unit files  https://review.opendev.org/70888922:07
openstackgerritMerged openstack/tripleo-heat-templates master: Remove comment about tripleo_container_manage being experimental  https://review.opendev.org/70852122:07
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: WIP: try podman driver  https://review.opendev.org/70885122:11
*** avivgt has quit IRC22:11
openstackgerritMerged openstack/tripleo-ansible master: Add no_log to the podman login  https://review.opendev.org/70889522:12
openstackgerritMerged openstack/tripleo-ansible stable/train: Use CI mirrors for molecule jobs  https://review.opendev.org/70893022:12
*** jmasud has quit IRC22:15
openstackgerritSteve Baker proposed openstack/tripleo-ansible master: module tripleo_baremetal_expand_roles  https://review.opendev.org/70641722:16
openstackgerritSteve Baker proposed openstack/tripleo-ansible master: module metalsmith_instances  https://review.opendev.org/70641822:16
openstackgerritSteve Baker proposed openstack/tripleo-ansible master: module tripleo_baremetal_check_existing  https://review.opendev.org/70704322:16
openstackgerritSteve Baker proposed openstack/tripleo-ansible master: module tripleo_baremetal_populate_environment  https://review.opendev.org/70729522:16
openstackgerritSteve Baker proposed openstack/tripleo-ansible master: Playbook for overcloud node provision  https://review.opendev.org/70704422:16
openstackgerritSteve Baker proposed openstack/python-tripleoclient master: WIP switch to ansible for overcloud node provision  https://review.opendev.org/70874922:16
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Add release file for master CentOS 8  https://review.opendev.org/70896822:19
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates stable/queens: NodeDataLookup utility should rely on python env  https://review.opendev.org/70897822:20
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates stable/train: NodeDataLookup utility should rely on python env  https://review.opendev.org/70897922:20
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates stable/stein: NodeDataLookup utility should rely on python env  https://review.opendev.org/70898022:21
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates stable/rocky: NodeDataLookup utility should rely on python env  https://review.opendev.org/70898122:22
openstackgerritMerged openstack/tripleo-ansible master: paunch: fail early if containers weren't deployed by Paunch before  https://review.opendev.org/70888022:24
openstackgerritAlex Schultz proposed openstack/tripleo-ansible stable/train: Add no_log to the podman login  https://review.opendev.org/70898222:25
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates stable/stein: [Stein-only] add no_log to the podman login  https://review.opendev.org/70898422:28
*** dpawlik has quit IRC22:33
openstackgerritAlex Schultz proposed openstack/tripleo-ansible master: Add tmpwatch to the bootstrap  https://review.opendev.org/70898622:35
*** iurygregory has quit IRC22:36
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates stable/stein: [Stein and older] Ensure tmpwatch installed  https://review.opendev.org/70898722:37
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart master: Sync fs035 job with fs001 settings  https://review.opendev.org/70898822:44
*** jmasud has joined #tripleo22:44
*** pkopec has quit IRC22:47
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Add release file for master CentOS 8  https://review.opendev.org/70896822:49
cloudnullEmilienM mwhahaha with the "enable-swap" template, if the host has a swap partition, will that template use it, or does it always try and create a swap file ?22:53
mwhahahaenable-swap.yaml is a file22:54
mwhahahathere's a different one for partitions22:54
mwhahahaenable-swap-partition.yaml22:54
cloudnulli see22:54
mwhahaha(fittingly named)22:54
cloudnulldoes enable-swap-partition.yaml require config or does it seek an existing partition ?22:55
mwhahahaneeds to be labled it looks like22:55
* mwhahaha shrugs22:55
mwhahahai only ever use enable-swap.yaml22:55
cloudnull++22:55
cloudnullI'm tinkering with metalsmith and it has an option swap_size_mb22:56
cloudnullwhich creates a swap partition, but it doesn't enable it22:56
cloudnullputs nothing in fstab, and doesn't seem to run swapon $DEV for the partition22:57
mwhahaharight the image itself would have to know about the partitions22:57
mwhahahaovercloud-full by default doesn't22:57
mwhahahathere's a way to do custom partitions and images but that's out of my scope22:58
*** tkajinam has joined #tripleo22:58
mwhahahalooks like you'd just have to specify swap_partition_label: <a label> in parameter_defaults somewhere. though it assumes the name is consistent across all devices22:58
mwhahahahttps://bugzilla.redhat.com/show_bug.cgi?id=1293408#c522:59
openstackbugzilla.redhat.com bug 1293408 in openstack-tripleo-heat-templates "Overcloud nodes are created without swap space" [High,Closed: errata] - Assigned to hbrock22:59
* cloudnull will give that a spin 23:00
openstackgerritAlex Schultz proposed openstack/ansible-role-container-registry master: Don't log login  https://review.opendev.org/70890223:02
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: WIP: try podman driver  https://review.opendev.org/70885123:02
*** sshnaidm is now known as sshnaidm|afk23:02
cloudnullbut in exciting news, i have a novaless environment deployed - https://pasted.tech/pastes/fd7ae3e111080ea0c65a76ce27b1e90542bc635923:02
cloudnulli dont think https://github.com/openstack/tripleo-heat-templates/blob/master/extraconfig/all_nodes/swap-partition.j2.yaml#L21 works23:06
cloudnullthese swap partitions don't have a label, https://pasted.tech/pastes/8ae221164d290a0f97e3e65e90f94058b91c575223:08
mwhahahaprobably23:08
mwhahahait's generally untested afaik23:08
mwhahahafeel free to improve :D23:08
* mwhahaha runs away for a bit23:09
cloudnull:D23:09
*** slaweq has joined #tripleo23:10
*** ahosam has quit IRC23:10
*** ahosam has joined #tripleo23:11
*** slaweq has quit IRC23:15
*** pbandark has quit IRC23:25
*** pbandark has joined #tripleo23:25
*** ahosam has quit IRC23:40
*** jmasud has quit IRC23:43
*** pbandark has quit IRC23:45
*** jmasud has joined #tripleo23:46
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Improve the ability to enable swap  https://review.opendev.org/70901523:48
*** jmasud has quit IRC23:50
*** alexmcleod has quit IRC23:52
*** jmasud has joined #tripleo23:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!