| *** sai_p_ has joined #puppet-openstack | 01:53 | |
| *** ykarel|away is now known as ykarel | 04:49 | |
| *** amoralej|off is now known as amoralej | 06:57 | |
| *** sai_p_ has quit IRC | 07:03 | |
| *** rcernin has quit IRC | 07:45 | |
| *** tosky has joined #puppet-openstack | 07:46 | |
| *** dtantsur|afk is now known as dtantsur | 07:47 | |
| *** jpena|off is now known as jpena | 07:56 | |
| *** lbragstad_ has joined #puppet-openstack | 08:17 | |
| *** lbragstad has quit IRC | 08:20 | |
| *** gfidente has joined #puppet-openstack | 08:44 | |
| *** hjensas|afk is now known as hjensas | 08:46 | |
| *** derekh has joined #puppet-openstack | 08:49 | |
| *** tkajinam has quit IRC | 09:09 | |
| *** gfidente is now known as gfidente|ffu | 09:42 | |
| *** owalsh_ is now known as owalsh | 09:50 | |
| *** cgfbee has quit IRC | 10:51 | |
| *** rajinir has quit IRC | 11:00 | |
| *** jpena is now known as jpena|lunch | 11:43 | |
| *** amoralej is now known as amoralej|lunch | 12:00 | |
| zigo | tobias-urdin: I've found a big bug in puppet-module-openstacklib where os_transport_url is adding a trailing / at the end of the string, for only 1 of the 3 hosts I'm giving as parameter !!! | 12:19 |
|---|---|---|
| zigo | It's a horrible result where components can't communicated with each other. | 12:19 |
| zigo | -d | 12:19 |
| tobias-urdin | can u give an example | 12:25 |
| *** jpena|lunch is now known as jpena | 12:42 | |
| *** amoralej|lunch is now known as amoralej | 13:14 | |
| zigo | tobias-urdin: If I set something like this: | 13:21 |
| zigo | $nova_notif_transport_url = os_transport_url({ | 13:21 |
| zigo | 'transport' => $messaging_notify_proto, | 13:21 |
| zigo | 'hosts' => fqdn_rotate($all_masters), | 13:21 |
| zigo | 'port' => $messaging_notify_port, | 13:21 |
| zigo | 'username' => 'nova', | 13:21 |
| zigo | 'password' => $pass_nova_messaging, | 13:21 |
| zigo | }) | 13:21 |
| zigo | then I get something like this: | 13:21 |
| zigo | transport_url=rabbit://u:p@host:port,u:p@host:port,u:p@host:port/ | 13:21 |
| zigo | instead of: | 13:21 |
| zigo | transport_url=rabbit://u:p@host:port,u:p@host:port,u:p@host:port | 13:21 |
| zigo | Then whith a virtual_host param set, it should be: | 13:21 |
| zigo | transport_url=rabbit://u:p@host:port/virtual-host,u:p@host:port/virtual-host,u:p@host:port/virtual-host | 13:21 |
| zigo | but what openstacklib does is: | 13:21 |
| zigo | transport_url=rabbit://u:p@host:port,u:p@host:port,u:p@host:port/virtual-host | 13:21 |
| zigo | tobias-urdin: I believe I have a working patch to fix the wrong behavior. | 13:21 |
| zigo | Of course, this doesn't affect people using a single rabbit host, for example using haproxy, but that's not what I'm doing. | 13:21 |
| zigo | tobias-urdin: My patch: https://salsa.debian.org/openstack-team/puppet/puppet-module-openstacklib/-/blob/debian/ussuri/debian/patches/fix-multiple-rabbitmq-transport_url.patch | 13:22 |
| tobias-urdin | zigo: based on oslo documentation it's correct https://docs.openstack.org/oslo.messaging/latest/reference/transport.html | 13:23 |
| zigo | tobias-urdin: As much as I could see, that's not how it's working. | 13:23 |
| zigo | Are we sure the doc is correct? | 13:23 |
| tobias-urdin | you need to check with oslo team then, or directly in the code | 13:24 |
| tobias-urdin | it probably depends on what driver is used | 13:24 |
| zigo | Right. | 13:24 |
| zigo | I'll go and ask. | 13:24 |
| tobias-urdin | im doing this transport_url=rabbit://u:p@host:port,u:p@host:port,u:p@host:port/myvhost | 13:24 |
| tobias-urdin | without any issues | 13:24 |
| tobias-urdin | is it possible that the issue is that the "/" vhost is used, so maybe it should be transport_url=rabbit://u:p@host:port,u:p@host:port,u:p@host:port// | 13:25 |
| tobias-urdin | or smth like that | 13:25 |
| zigo | I don't set any vhost, and then it just uses /, which works only if the / is ommited in the URL. | 13:25 |
| zigo | Rabbit's weirdness... | 13:26 |
| zigo | I'll try to more carefully check what's going on. | 13:26 |
| tobias-urdin | zigo: all integration testing uses the "/" vhost, so doesn't look like there is an issue atleast there | 13:28 |
| zigo | tobias-urdin: They use a single vhost, so it's not relevant as a test. | 13:29 |
| tobias-urdin | well the testing tests with a single, i have every project split up in our deployment, so i dont think it's puppets fault atleast | 13:30 |
| zigo | tobias-urdin: This all started because I am having neutron-bgp-dragent getting printing some timeout in my logs ... | 13:32 |
| *** dpawlik2 has joined #puppet-openstack | 15:12 | |
| *** irclogbot_1 has quit IRC | 15:14 | |
| *** redrobot has quit IRC | 15:14 | |
| *** dpawlik4 has quit IRC | 15:14 | |
| *** tonyb has quit IRC | 15:15 | |
| *** tonyb has joined #puppet-openstack | 15:16 | |
| *** jtomasek_ has quit IRC | 15:17 | |
| *** ykarel is now known as ykarel|away | 15:17 | |
| *** mkarpin has joined #puppet-openstack | 15:18 | |
| *** irclogbot_2 has joined #puppet-openstack | 15:19 | |
| *** lbragstad_ is now known as lbragstad | 16:22 | |
| *** amoralej is now known as amoralej|off | 16:27 | |
| *** dtantsur is now known as dtantsur|afk | 16:28 | |
| *** derekh has quit IRC | 16:34 | |
| lbragstad | EmilienM i'm revisiting https://review.opendev.org/#/c/729364/5 and i know we were able to get it working after an initial all-in-one.sh deploy by modifying the code directly | 16:49 |
| lbragstad | i'm able to reproduce the failures locally by mimicking what zuul does and installing the current patch set | 16:50 |
| lbragstad | i think there is something wrong with how i'm declaring the provider for keystone_config and it's relationship with keystone_domain_config because i see this all over in the logs http://paste.openstack.org/show/795505/ | 16:51 |
| lbragstad | i suspect something in failing in puppet-keystone that prevents it from configuring keystone properly, resulting in the 500s | 16:51 |
| lbragstad | i applied http://paste.openstack.org/raw/795506/ on top of https://review.opendev.org/#/c/729364/5 - but that doesn't seem to fix the issue | 16:54 |
| *** jpena is now known as jpena|off | 17:14 | |
| openstackgerrit | Alan Bishop proposed openstack/puppet-cinder master: Move the keymgr_XXX parameters from cinder::api to init https://review.opendev.org/739126 | 17:49 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-keystone master: WIP: support mutli-line config opts https://review.opendev.org/729364 | 17:49 |
| *** gfidente|ffu is now known as gfidente|afk | 18:29 | |
| openstackgerrit | Lance Bragstad proposed openstack/puppet-keystone master: WIP: support mutli-line config opts https://review.opendev.org/729364 | 20:03 |
| openstackgerrit | Merged openstack/puppet-placement master: Use workers instead of threads in wsgi by default https://review.opendev.org/738630 | 20:05 |
| lbragstad | EmilienM i updated with everything i've tried today https://review.opendev.org/#/c/729364/7 | 21:56 |
| lbragstad | i'd appreciate any pointers if there is something blatantly obvious that i'm missing | 21:57 |
| *** rcernin has joined #puppet-openstack | 22:42 | |
| *** tosky has quit IRC | 22:54 | |
| *** tkajinam has joined #puppet-openstack | 22:55 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!