*** tosky has quit IRC | 00:11 | |
*** rh-jelabarre has quit IRC | 02:13 | |
*** spatel has joined #openstack-ansible | 03:08 | |
*** ianychoi_ has joined #openstack-ansible | 03:40 | |
*** ianychoi has quit IRC | 03:43 | |
*** jamesdenton has quit IRC | 03:59 | |
*** jamesdenton has joined #openstack-ansible | 03:59 | |
*** spatel has quit IRC | 04:00 | |
*** shyamb has joined #openstack-ansible | 04:31 | |
*** shyam89 has joined #openstack-ansible | 04:31 | |
*** Underknowledge has quit IRC | 04:40 | |
*** Underknowledge has joined #openstack-ansible | 04:40 | |
*** gyee has quit IRC | 05:20 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-ansible | 05:33 | |
*** gokhani has joined #openstack-ansible | 06:13 | |
*** shyam89 has quit IRC | 06:20 | |
*** shyamb has quit IRC | 06:20 | |
*** pto has joined #openstack-ansible | 07:09 | |
*** fresta has quit IRC | 07:33 | |
*** fresta has joined #openstack-ansible | 07:33 | |
*** luksky has joined #openstack-ansible | 07:36 | |
gokhani | Good 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 IRC | 07:47 | |
*** fresta has joined #openstack-ansible | 07:50 | |
ebbex | jrosser: Are you certain hostvars[] being part of the 'ansible_facts[] vs ansible_{{ lookup }}' problem? | 07:51 |
*** rpittau|afk is now known as rpittau | 08:08 | |
*** andrewbonney has joined #openstack-ansible | 08:25 | |
*** hindret_ has joined #openstack-ansible | 08:29 | |
*** gixx_ has joined #openstack-ansible | 08:29 | |
*** fyx_ has joined #openstack-ansible | 08:29 | |
*** johnsom_ has joined #openstack-ansible | 08:30 | |
*** persia has joined #openstack-ansible | 08:31 | |
*** rpittau_ has joined #openstack-ansible | 08:33 | |
*** sc__ has joined #openstack-ansible | 08:34 | |
*** jrollen has joined #openstack-ansible | 08:34 | |
*** tosky has joined #openstack-ansible | 08:36 | |
jrosser | ebbex: there seems some reasonable evidence from the tripleo folk on the GitHub issue I think | 08:41 |
jrosser | though I suspect that gathering minimum rather than all facts makes as much or more difference | 08:42 |
jrosser | on my lab for two network nodes to gather all facts is 2mins, and just a couple of seconds for minimum facts | 08:43 |
*** persia_ has quit IRC | 08:43 | |
*** rpittau has quit IRC | 08:43 | |
*** fyx has quit IRC | 08:43 | |
*** johnsom has quit IRC | 08:43 | |
*** hindret has quit IRC | 08:43 | |
*** sc has quit IRC | 08:43 | |
*** gixx has quit IRC | 08:43 | |
*** jroll has quit IRC | 08:43 | |
*** gixx_ is now known as gixx | 08:43 | |
*** hindret_ is now known as hindret | 08:43 | |
*** johnsom_ is now known as johnsom | 08:43 | |
*** rpittau_ is now known as rpittau | 08:43 | |
*** fyx_ is now known as fyx | 08:43 | |
ebbex | Yeah, reducing fact gathering makes more sense. | 08:47 |
*** openstackgerrit has joined #openstack-ansible | 09:15 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in docs https://review.opendev.org/c/openstack/openstack-ansible/+/778000 | 09:15 |
*** shyamb has joined #openstack-ansible | 09:17 | |
*** shyamb has quit IRC | 09:19 | |
*** shyamb has joined #openstack-ansible | 09:19 | |
*** shyamb has quit IRC | 09:21 | |
*** shyamb has joined #openstack-ansible | 09:22 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in playbooks https://review.opendev.org/c/openstack/openstack-ansible/+/777996 | 09:22 |
*** shyamb has quit IRC | 09:23 | |
*** shyamb has joined #openstack-ansible | 09:23 | |
*** shyamb has quit IRC | 09:24 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in group_vars https://review.opendev.org/c/openstack/openstack-ansible/+/777997 | 09:35 |
jrosser | ebbex: which hostvars thing are you referring to in your review on ^ that | 09:35 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in playbooks https://review.opendev.org/c/openstack/openstack-ansible/+/777996 | 09:36 |
*** sshnaidm|afk is now known as sshnaidm|aoff | 09:41 | |
*** sshnaidm|aoff is now known as sshnaidm|off | 09:41 | |
ebbex | jrosser: hostvars[physical_host].ansible_facts, prefer ['ansible_facts'] | 09:46 |
jrosser | ahha i see | 09:46 |
jrosser | sure | 09:46 |
jrosser | anywhere i've forgotton to change that is a mistake | 09:47 |
openstackgerrit | Merged openstack/openstack-ansible-ops stable/victoria: Fix regex_replace mapping https://review.opendev.org/c/openstack/openstack-ansible-ops/+/778096 | 09:56 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Gather minimal facts for dynamic_address_fact https://review.opendev.org/c/openstack/openstack-ansible/+/777989 | 09:58 |
jrosser | ebbex: also that json_query stuff is horrible i agree | 10:04 |
jrosser | it does seem like someone has gone to really huge effort to achieve all that in one task | 10:04 |
jrosser | and 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 understandable | 10:05 |
openstackgerrit | Merged 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/+/778222 | 10:08 |
openstackgerrit | Merged openstack/openstack-ansible-os_gnocchi master: Move gnocchi pip packages from constraints to requirements https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/772281 | 10:08 |
ebbex | jrosser: 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_queries | 10:14 |
jrosser | except that container_networks[network_address] may not be defined :( | 10:14 |
ebbex | network_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 |
jrosser | from 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.j2 | 10:22 |
jrosser | this all comes from an effort some time ago to make container_networks and the dynamic inventory be optional | 10:23 |
jrosser | my 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 undefined | 10:26 |
openstackgerrit | Merged openstack/openstack-ansible-os_cinder master: Use ansible_facts[] instead of fact variables https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/777178 | 10:31 |
openstackgerrit | Merged openstack/openstack-ansible-os_keystone master: Use ansible_facts[] instead of fact variables https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/777180 | 10:33 |
openstackgerrit | Merged openstack/openstack-ansible-os_placement master: Use ansible_facts[] instead of fact variables https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/777181 | 10:35 |
openstackgerrit | Ebbex proposed openstack/openstack-ansible master: Gather minimal facts for dynamic_address_fact https://review.opendev.org/c/openstack/openstack-ansible/+/777989 | 10:36 |
*** admin0 has quit IRC | 11:18 | |
*** pto has quit IRC | 11:53 | |
openstackgerrit | Daniel 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/+/768117 | 11:54 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_octavia stable/victoria: Use barbican for certificates storage https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/778678 | 12:33 |
*** jbadiapa has joined #openstack-ansible | 12:45 | |
*** rh-jelabarre has joined #openstack-ansible | 13:33 | |
*** spatel has joined #openstack-ansible | 13:48 | |
*** admin0 has joined #openstack-ansible | 14:00 | |
openstackgerrit | Jonathan 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/+/776959 | 14:23 |
openstackgerrit | Jonathan 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/+/776959 | 14:25 |
openstackgerrit | Marc Gariépy proposed openstack/openstack-ansible-os_neutron master: Fix conditional for ovs https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/778075 | 14:30 |
openstackgerrit | James Denton proposed openstack/openstack-ansible master: Add Neutron Role Testing Overrides https://review.opendev.org/c/openstack/openstack-ansible/+/778008 | 14:38 |
jamesdenton | jrosser i've just rolled the o_u_c template change into this patch, so getting rid of those neutron_provider overrides | 14:45 |
*** ianychoi__ has joined #openstack-ansible | 14:46 | |
jrosser | jamesdenton: nice! | 14:46 |
*** ianychoi_ has quit IRC | 14:49 | |
openstackgerrit | Jonathan 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/+/776201 | 14:51 |
openstackgerrit | Jonathan 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/+/778713 | 14:51 |
*** kleini_ has quit IRC | 15:01 | |
jrosser | ebbex: with https://review.opendev.org/c/openstack/openstack-ansible/+/777989 - shall we put it back to patchset 11? | 15:05 |
mgariepy | hmm stale facts can cause issue when re-running playbook. | 15:06 |
mgariepy | having stale facts for the host will fail with something like that: http://paste.openstack.org/show/803231/ | 15:07 |
mgariepy | this was a run for os_glance.yml | 15:07 |
jrosser | mgariepy: i had this recently too, the facts lifetime is 24hours | 15:12 |
mgariepy | maybe we should refresh them | 15:12 |
jrosser | could you try this https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/777559 | 15:13 |
mgariepy | it can happen that i need to run playbook for various service just to update the config . | 15:13 |
mgariepy | nice. | 15:14 |
jrosser | are you on V with that failing? | 15:14 |
mgariepy | yep | 15:14 |
mgariepy | v + a few patches. | 15:14 |
jrosser | hmm thats not a clean cherry-pick, i'll try to backport it | 15:17 |
mgariepy | it needs 1904e6e991559a10cd19f14b37af27ed1497715c | 15:18 |
mgariepy | hmm | 15:19 |
mgariepy | it did cherry-pick for me on my branch. | 15:20 |
mgariepy | https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/777908 | 15:20 |
mgariepy | already merged haha | 15:20 |
jrosser | doh! | 15:33 |
jrosser | unhelpful message from gerrit UI then for me | 15:33 |
mgariepy | well you have to be used to that. openstack is always giving unhelpful message also. | 15:37 |
mgariepy | changing ceph mons is not fun. | 15:37 |
*** gyee has joined #openstack-ansible | 15:52 | |
*** spatel has quit IRC | 15:55 | |
openstackgerrit | James Denton proposed openstack/openstack-ansible master: Add Neutron Role Testing Overrides https://review.opendev.org/c/openstack/openstack-ansible/+/778008 | 16:02 |
*** pcaruana has quit IRC | 16:36 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Collect keystone apache federation files https://review.opendev.org/c/openstack/openstack-ansible-ops/+/741236 | 16:48 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Gather minimal facts for dynamic_address_fact https://review.opendev.org/c/openstack/openstack-ansible/+/777989 | 16:59 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Gather minimal facts in common playbooks https://review.opendev.org/c/openstack/openstack-ansible/+/777990 | 17:00 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in AIO config https://review.opendev.org/c/openstack/openstack-ansible/+/777999 | 17:00 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in playbooks https://review.opendev.org/c/openstack/openstack-ansible/+/777996 | 17:01 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in docs https://review.opendev.org/c/openstack/openstack-ansible/+/778000 | 17:01 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in group_vars https://review.opendev.org/c/openstack/openstack-ansible/+/777997 | 17:01 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: Use ansible_facts[] instead of fact variables in tests https://review.opendev.org/c/openstack/openstack-ansible/+/777998 | 17:01 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: WIP - Gather minimal facts and disable fact variables https://review.opendev.org/c/openstack/openstack-ansible/+/778396 | 17:02 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: [reno] Stop publishing release notes https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/772014 | 17:14 |
*** rpittau is now known as rpittau|afk | 17:27 | |
admin0 | does osa support redhat 8 ? | 17:36 |
*** pcaruana has joined #openstack-ansible | 17:42 | |
jrosser | admin0: strictly speaking no, but i imagine the delta might be relatively small | 17:59 |
jrosser | given we've now had a successful run on Centos-8 Stream, i would expect something broadly similar to work on RHEL | 17:59 |
jrosser | but as usual small things can turn into big blockers, you'd have to try it, and be prepared to fix up whatever was needed | 18:00 |
*** jbadiapa has quit IRC | 18:00 | |
*** andrewbonney has quit IRC | 18:23 | |
openstackgerrit | Jonathan Rosser proposed openstack/ansible-hardening master: Use ansible_facts[] instead of fact variables https://review.opendev.org/c/openstack/ansible-hardening/+/778483 | 18:34 |
openstackgerrit | James Denton proposed openstack/openstack-ansible master: Add Neutron Role Testing Overrides https://review.opendev.org/c/openstack/openstack-ansible/+/778008 | 18:38 |
*** ioni has quit IRC | 18:55 | |
*** mgagne has quit IRC | 18:56 | |
*** mgagne has joined #openstack-ansible | 18:57 | |
*** juanoterocas has joined #openstack-ansible | 19:48 | |
openstackgerrit | Marc 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/+/778771 | 19:55 |
*** gokhani has quit IRC | 20:17 | |
*** klamath_atx has quit IRC | 20:31 | |
*** Jeffrey4l has quit IRC | 20:39 | |
*** Jeffrey4l has joined #openstack-ansible | 20:39 | |
admin0 | stuck in powertools :) centos has powertools .. redhat does not | 20:45 |
*** ioni has joined #openstack-ansible | 20:49 | |
*** openstackgerrit has quit IRC | 21:05 | |
*** luksky has quit IRC | 21:28 | |
*** luksky has joined #openstack-ansible | 21:28 | |
*** luksky has quit IRC | 21:39 | |
*** luksky has joined #openstack-ansible | 21:53 | |
jrosser | admin0: https://access.redhat.com/discussions/5417621 | 22:15 |
admin0 | yeah .. i already got that | 22:20 |
admin0 | which is why i reported the line above .. as my journey in redhat8 +osa got stuck there | 22:20 |
admin0 | sent it to IT to have approval for centos8 .. | 22:21 |
*** klamath_atx has joined #openstack-ansible | 22:28 | |
*** juanoterocas has quit IRC | 22:50 | |
*** openstackgerrit has joined #openstack-ansible | 23:04 | |
openstackgerrit | Merged openstack/openstack-ansible stable/victoria: Bump SHAs for stable/victoria https://review.opendev.org/c/openstack/openstack-ansible/+/775521 | 23:04 |
*** luksky has quit IRC | 23:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!