Thursday, 2021-03-04

*** tosky has quit IRC00:11
*** rh-jelabarre has quit IRC02:13
*** spatel has joined #openstack-ansible03:08
*** ianychoi_ has joined #openstack-ansible03:40
*** ianychoi has quit IRC03:43
*** jamesdenton has quit IRC03:59
*** jamesdenton has joined #openstack-ansible03:59
*** spatel has quit IRC04:00
*** shyamb has joined #openstack-ansible04:31
*** shyam89 has joined #openstack-ansible04:31
*** Underknowledge has quit IRC04:40
*** Underknowledge has joined #openstack-ansible04:40
*** gyee has quit IRC05:20
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-ansible05:33
*** gokhani has joined #openstack-ansible06:13
*** shyam89 has quit IRC06:20
*** shyamb has quit IRC06:20
*** pto has joined #openstack-ansible07:09
*** fresta has quit IRC07:33
*** fresta has joined #openstack-ansible07:33
*** luksky has joined #openstack-ansible07:36
gokhaniGood morning folks, is there any one running on Ussuri ? ı have problems with rabbitmq. Nova-api intermittently gives timeout errors on rabbitmq. ıf yes, can you share with me your oslo.messaging, nova and py-amqp versions? Also I can not change heartbeat timeout treshold for rabbitmq.07:38
*** fresta has quit IRC07:47
*** fresta has joined #openstack-ansible07:50
ebbexjrosser: Are you certain hostvars[] being part of the 'ansible_facts[] vs ansible_{{ lookup }}' problem?07:51
*** rpittau|afk is now known as rpittau08:08
*** andrewbonney has joined #openstack-ansible08:25
*** hindret_ has joined #openstack-ansible08:29
*** gixx_ has joined #openstack-ansible08:29
*** fyx_ has joined #openstack-ansible08:29
*** johnsom_ has joined #openstack-ansible08:30
*** persia has joined #openstack-ansible08:31
*** rpittau_ has joined #openstack-ansible08:33
*** sc__ has joined #openstack-ansible08:34
*** jrollen has joined #openstack-ansible08:34
*** tosky has joined #openstack-ansible08:36
jrosserebbex: there seems some reasonable evidence from the tripleo folk on the GitHub issue I think08:41
jrosserthough I suspect that gathering minimum rather than all facts makes as much or more difference08:42
jrosseron my lab for two network nodes to gather all facts is 2mins, and just a couple of seconds for minimum facts08:43
*** persia_ has quit IRC08:43
*** rpittau has quit IRC08:43
*** fyx has quit IRC08:43
*** johnsom has quit IRC08:43
*** hindret has quit IRC08:43
*** sc has quit IRC08:43
*** gixx has quit IRC08:43
*** jroll has quit IRC08:43
*** gixx_ is now known as gixx08:43
*** hindret_ is now known as hindret08:43
*** johnsom_ is now known as johnsom08:43
*** rpittau_ is now known as rpittau08:43
*** fyx_ is now known as fyx08:43
ebbexYeah, reducing fact gathering makes more sense.08:47
*** openstackgerrit has joined #openstack-ansible09:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in docs  https://review.opendev.org/c/openstack/openstack-ansible/+/77800009:15
*** shyamb has joined #openstack-ansible09:17
*** shyamb has quit IRC09:19
*** shyamb has joined #openstack-ansible09:19
*** shyamb has quit IRC09:21
*** shyamb has joined #openstack-ansible09:22
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in playbooks  https://review.opendev.org/c/openstack/openstack-ansible/+/77799609:22
*** shyamb has quit IRC09:23
*** shyamb has joined #openstack-ansible09:23
*** shyamb has quit IRC09:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in group_vars  https://review.opendev.org/c/openstack/openstack-ansible/+/77799709:35
jrosserebbex: which hostvars thing are you referring to in your review on ^ that09:35
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in playbooks  https://review.opendev.org/c/openstack/openstack-ansible/+/77799609:36
*** sshnaidm|afk is now known as sshnaidm|aoff09:41
*** sshnaidm|aoff is now known as sshnaidm|off09:41
ebbexjrosser: hostvars[physical_host].ansible_facts, prefer ['ansible_facts']09:46
jrosserahha i see09:46
jrossersure09:46
jrosseranywhere i've forgotton to change that is a mistake09:47
openstackgerritMerged openstack/openstack-ansible-ops stable/victoria: Fix regex_replace mapping  https://review.opendev.org/c/openstack/openstack-ansible-ops/+/77809609:56
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Gather minimal facts for dynamic_address_fact  https://review.opendev.org/c/openstack/openstack-ansible/+/77798909:58
jrosserebbex: also that json_query stuff is horrible i agree10:04
jrosserit does seem like someone has gone to really huge effort to achieve all that in one task10:04
jrosserand maybe really the answer is to split it onto a set of conditional tasks, maybe first splitting is_metal=true and is_metal=false apart would make it more understandable10:05
openstackgerritMerged openstack/openstack-ansible-os_gnocchi master: Provide different package lists for install and wheel build  https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/77822210:08
openstackgerritMerged openstack/openstack-ansible-os_gnocchi master: Move gnocchi pip packages from constraints to requirements  https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/77228110:08
ebbexjrosser: Yeah, just setfact: ansible_facts[ hostvars[inventory_hostname]['container_networks'][network_address]['bridge'] | replace('-', '_'') ]['ipv4']['address'] when is_metal and the simpler setfact when not is_metal. No json_queries10:14
jrosserexcept that container_networks[network_address] may not be defined :(10:14
ebbexnetwork_address (container_address, storage_address, tunnel_address) is always passed in as a vars to the task. if there are nova_hosts, cinder_hosts or neutron_hosts without the container_networks we have a bigger problem, no?10:19
jrosserfrom a neutron POV currently you can completely ignore container_networks and provide your own neutron_provider_networks like we do right now in the OVN tests https://review.opendev.org/c/openstack/openstack-ansible/+/778008/6/tests/roles/bootstrap-host/templates/user_variables_neutron_ovn.yml.j210:22
jrosserthis all comes from an effort some time ago to make container_networks and the dynamic inventory be optional10:23
jrossermy patch to dynamic_address_fact.yml is in fact failing so far because i've not accounted for the case where container_networks['tunnel_address'] is undefined10:26
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Use ansible_facts[] instead of fact variables  https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/77717810:31
openstackgerritMerged openstack/openstack-ansible-os_keystone master: Use ansible_facts[] instead of fact variables  https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/77718010:33
openstackgerritMerged openstack/openstack-ansible-os_placement master: Use ansible_facts[] instead of fact variables  https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/77718110:35
openstackgerritEbbex proposed openstack/openstack-ansible master: Gather minimal facts for dynamic_address_fact  https://review.opendev.org/c/openstack/openstack-ansible/+/77798910:36
*** admin0 has quit IRC11:18
*** pto has quit IRC11:53
openstackgerritDaniel Meloy proposed openstack/openstack-ansible-os_nova master: Add Virtual GPU Config to nova.conf template  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/76811711:54
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_octavia stable/victoria: Use barbican for certificates storage  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/77867812:33
*** jbadiapa has joined #openstack-ansible12:45
*** rh-jelabarre has joined #openstack-ansible13:33
*** spatel has joined #openstack-ansible13:48
*** admin0 has joined #openstack-ansible14:00
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Use ansible_facts[] instead of fact variables  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/77695914:23
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Use ansible_facts[] instead of fact variables  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/77695914:25
openstackgerritMarc Gariépy proposed openstack/openstack-ansible-os_neutron master: Fix conditional for ovs  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/77807514:30
openstackgerritJames Denton proposed openstack/openstack-ansible master: Add Neutron Role Testing Overrides  https://review.opendev.org/c/openstack/openstack-ansible/+/77800814:38
jamesdentonjrosser i've just rolled the o_u_c template change into this patch, so getting rid of those neutron_provider overrides14:45
*** ianychoi__ has joined #openstack-ansible14:46
jrosserjamesdenton: nice!14:46
*** ianychoi_ has quit IRC14:49
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Optionally build container image as a local chroot  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/77620114:51
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Switch all base images to be built locally rather than downloaded  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/77871314:51
*** kleini_ has quit IRC15:01
jrosserebbex: with https://review.opendev.org/c/openstack/openstack-ansible/+/777989 - shall we put it back to patchset 11?15:05
mgariepyhmm stale facts can cause issue when re-running playbook.15:06
mgariepyhaving stale facts for the host will fail with something like that: http://paste.openstack.org/show/803231/15:07
mgariepythis was a run for os_glance.yml15:07
jrossermgariepy: i had this recently too, the facts lifetime is 24hours15:12
mgariepymaybe we should refresh them15:12
jrossercould you try this https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/77755915:13
mgariepyit can happen that i need to run playbook for various service just to update the config .15:13
mgariepynice.15:14
jrosserare you on V with that failing?15:14
mgariepyyep15:14
mgariepyv + a few patches.15:14
jrosserhmm thats not a clean cherry-pick, i'll try to backport it15:17
mgariepyit needs 1904e6e991559a10cd19f14b37af27ed1497715c15:18
mgariepyhmm15:19
mgariepyit did cherry-pick for me on my branch.15:20
mgariepyhttps://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/77790815:20
mgariepyalready merged haha15:20
jrosserdoh!15:33
jrosserunhelpful message from gerrit UI then for me15:33
mgariepywell you have to be used to that. openstack is always giving unhelpful message also.15:37
mgariepychanging ceph mons is not fun.15:37
*** gyee has joined #openstack-ansible15:52
*** spatel has quit IRC15:55
openstackgerritJames Denton proposed openstack/openstack-ansible master: Add Neutron Role Testing Overrides  https://review.opendev.org/c/openstack/openstack-ansible/+/77800816:02
*** pcaruana has quit IRC16:36
openstackgerritMerged openstack/openstack-ansible-ops master: Collect keystone apache federation files  https://review.opendev.org/c/openstack/openstack-ansible-ops/+/74123616:48
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Gather minimal facts for dynamic_address_fact  https://review.opendev.org/c/openstack/openstack-ansible/+/77798916:59
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Gather minimal facts in common playbooks  https://review.opendev.org/c/openstack/openstack-ansible/+/77799017:00
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in AIO config  https://review.opendev.org/c/openstack/openstack-ansible/+/77799917:00
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in playbooks  https://review.opendev.org/c/openstack/openstack-ansible/+/77799617:01
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in docs  https://review.opendev.org/c/openstack/openstack-ansible/+/77800017:01
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in group_vars  https://review.opendev.org/c/openstack/openstack-ansible/+/77799717:01
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in tests  https://review.opendev.org/c/openstack/openstack-ansible/+/77799817:01
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: WIP - Gather minimal facts and disable fact variables  https://review.opendev.org/c/openstack/openstack-ansible/+/77839617:02
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: [reno] Stop publishing release notes  https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/77201417:14
*** rpittau is now known as rpittau|afk17:27
admin0does osa support redhat 8 ?17:36
*** pcaruana has joined #openstack-ansible17:42
jrosseradmin0: strictly speaking no, but i imagine the delta might be relatively small17:59
jrossergiven we've now had a successful run on Centos-8 Stream, i would expect something broadly similar to work on RHEL17:59
jrosserbut as usual small things can turn into big blockers, you'd have to try it, and be prepared to fix up whatever was needed18:00
*** jbadiapa has quit IRC18:00
*** andrewbonney has quit IRC18:23
openstackgerritJonathan Rosser proposed openstack/ansible-hardening master: Use ansible_facts[] instead of fact variables  https://review.opendev.org/c/openstack/ansible-hardening/+/77848318:34
openstackgerritJames Denton proposed openstack/openstack-ansible master: Add Neutron Role Testing Overrides  https://review.opendev.org/c/openstack/openstack-ansible/+/77800818:38
*** ioni has quit IRC18:55
*** mgagne has quit IRC18:56
*** mgagne has joined #openstack-ansible18:57
*** juanoterocas has joined #openstack-ansible19:48
openstackgerritMarc Gariépy proposed openstack/openstack-ansible-ceph_client master: Fix Remove revoked ceph apt-keys  https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/77877119:55
*** gokhani has quit IRC20:17
*** klamath_atx has quit IRC20:31
*** Jeffrey4l has quit IRC20:39
*** Jeffrey4l has joined #openstack-ansible20:39
admin0stuck in powertools :) centos has powertools .. redhat does not20:45
*** ioni has joined #openstack-ansible20:49
*** openstackgerrit has quit IRC21:05
*** luksky has quit IRC21:28
*** luksky has joined #openstack-ansible21:28
*** luksky has quit IRC21:39
*** luksky has joined #openstack-ansible21:53
jrosseradmin0: https://access.redhat.com/discussions/541762122:15
admin0yeah .. i already got that22:20
admin0which is why i reported the line above .. as my journey in redhat8 +osa got stuck there22:20
admin0sent it to IT to have approval for centos8 ..22:21
*** klamath_atx has joined #openstack-ansible22:28
*** juanoterocas has quit IRC22:50
*** openstackgerrit has joined #openstack-ansible23:04
openstackgerritMerged openstack/openstack-ansible stable/victoria: Bump SHAs for stable/victoria  https://review.opendev.org/c/openstack/openstack-ansible/+/77552123:04
*** luksky has quit IRC23:28

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