*** rh-jelabarre has quit IRC | 00:50 | |
*** kleini has quit IRC | 01:16 | |
*** kleini has joined #openstack-ansible | 01:16 | |
*** mcarden has quit IRC | 01:17 | |
*** d34dh0r53 has quit IRC | 01:52 | |
*** ebbex has quit IRC | 01:52 | |
*** bverschueren has quit IRC | 01:52 | |
*** NewJorg has quit IRC | 01:52 | |
*** grabes has quit IRC | 01:52 | |
*** chandankumar has quit IRC | 01:52 | |
*** corvus has quit IRC | 01:53 | |
*** d34dh0r53 has joined #openstack-ansible | 01:54 | |
*** ebbex has joined #openstack-ansible | 01:54 | |
*** bverschueren has joined #openstack-ansible | 01:54 | |
*** NewJorg has joined #openstack-ansible | 01:54 | |
*** grabes has joined #openstack-ansible | 01:54 | |
*** chandankumar has joined #openstack-ansible | 01:54 | |
*** corvus has joined #openstack-ansible | 01:54 | |
*** evrardjp has quit IRC | 02:33 | |
*** evrardjp has joined #openstack-ansible | 02:33 | |
*** macz_ has joined #openstack-ansible | 02:43 | |
*** macz_ has quit IRC | 02:47 | |
openstackgerrit | YuehuiLei proposed openstack/openstack-ansible-os_designate master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/789987 | 02:51 |
---|---|---|
openstackgerrit | YuehuiLei proposed openstack/openstack-ansible-openstack_openrc master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/789988 | 02:53 |
*** openstackgerrit has quit IRC | 03:31 | |
*** miloa has joined #openstack-ansible | 05:14 | |
*** miloa has quit IRC | 05:29 | |
*** pto_ has joined #openstack-ansible | 06:21 | |
*** pto has quit IRC | 06:25 | |
*** shyamb has joined #openstack-ansible | 06:28 | |
*** shyamb has quit IRC | 06:46 | |
*** shyamb has joined #openstack-ansible | 06:50 | |
*** pto_ has quit IRC | 06:53 | |
*** pto has joined #openstack-ansible | 06:55 | |
*** oleksandry has joined #openstack-ansible | 07:06 | |
*** andrewbonney has joined #openstack-ansible | 07:13 | |
*** rpittau|afk is now known as rpittau | 07:14 | |
*** pto has quit IRC | 07:26 | |
*** pto_ has joined #openstack-ansible | 07:26 | |
*** pto_ has quit IRC | 07:27 | |
*** pto has joined #openstack-ansible | 07:27 | |
*** oleksandry has quit IRC | 07:30 | |
*** oleksandry has joined #openstack-ansible | 07:46 | |
*** openstackgerrit has joined #openstack-ansible | 07:47 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/790006 | 07:47 |
*** tosky has joined #openstack-ansible | 07:47 | |
jrosser | noonedeadpunk: interested to know what you think of that ^ | 07:47 |
jrosser | we have a choice of adding more and more stuff on the end of transport_url (it's already pretty ugly) or breaking it out into variables | 07:48 |
noonedeadpunk | I think that's actually good in case it's working as expected (I think it does). This won;'t work for nova though, so we should be careful there | 07:50 |
jrosser | what trouble will we have there (i've been testing this with nova-compute btw) | 07:51 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/790006 | 07:54 |
jrosser | what i would like to do is patch enough roles that we can put together a big stack with the pki patches and turn the ssl back on | 07:56 |
jrosser | it's a shame that currently its needing an extra var for tls1.2 | 07:56 |
*** pto has quit IRC | 08:07 | |
*** pto has joined #openstack-ansible | 08:07 | |
noonedeadpunk | will bring trouble with conductor | 08:09 |
noonedeadpunk | It's mostly because of https://opendev.org/openstack/openstack-ansible-os_nova/commit/c6d4c6207fa904f30e471c598884b7bce66cbc8f | 08:10 |
noonedeadpunk | I've migrated to cells template - it parse URL in config and place into the conductor during runtime | 08:11 |
*** shyamb has quit IRC | 08:12 | |
noonedeadpunk | so in case we don't have `{query}` for cell, I'm not sure that conductor will be happy with having setting in oslo_messaging_rabbit | 08:23 |
jrosser | perhaps we have to split the behaviour there | 08:29 |
jrosser | leave the query for ssl=1 / ssl=0 | 08:29 |
jrosser | and put the ssl_version in a variable | 08:30 |
noonedeadpunk | yeah, I think this way it should work | 08:36 |
*** Premkumarar has joined #openstack-ansible | 08:43 | |
*** shyamb has joined #openstack-ansible | 08:45 | |
*** shyam89 has joined #openstack-ansible | 08:53 | |
*** shyamb has quit IRC | 08:54 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Correct shibboleth apache module name for ubuntu 20.04 https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/790018 | 09:02 |
*** Premkumarar has quit IRC | 09:25 | |
*** pto_ has joined #openstack-ansible | 09:31 | |
*** pto has quit IRC | 09:31 | |
*** shyam89 has quit IRC | 09:35 | |
*** shyamb has joined #openstack-ansible | 09:40 | |
*** shyamb has quit IRC | 10:05 | |
*** shyamb has joined #openstack-ansible | 10:06 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/790006 | 10:13 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/790034 | 10:15 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/790035 | 10:17 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/790036 | 10:18 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_nova master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/790037 | 10:22 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible master: WIP - Test PKI role https://review.opendev.org/c/openstack/openstack-ansible/+/788031 | 10:27 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/790006 | 10:31 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/790034 | 10:32 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/790035 | 10:32 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/790036 | 10:33 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_nova master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/790037 | 10:34 |
jonher | https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/789267 is an easy +W | 11:03 |
jrosser | jonher: done! | 11:07 |
jonher | thanks! | 11:07 |
*** recyclehero has quit IRC | 11:24 | |
openstackgerrit | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Gather minimal facts in CI https://review.opendev.org/c/openstack/openstack-ansible/+/790042 | 11:30 |
noonedeadpunk | jrosser: so, yestarday I come up with https://review.opendev.org/c/openstack/openstack-ansible/+/789776 and on the contrary we might have solve this withhttps://review.opendev.org/c/openstack/openstack-ansible/+/789784 | 11:30 |
jrosser | its tricky isnt it | 11:36 |
jrosser | becasue 6x the amount of data really is a lot | 11:36 |
jrosser | but balanced with actually how much difference it makes | 11:36 |
noonedeadpunk | yeah... | 11:37 |
noonedeadpunk | I'm also not really sure. And considering, that setup runs each time despite cache is valid or not... | 11:37 |
jrosser | i think in general what bothers me a bit is there are bound to be places were some extra fact is needed and we've not spotted it | 11:40 |
openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty stable/victoria: Fix wsgi_venv path for cloudkitty-api https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/789267 | 11:58 |
*** pto has joined #openstack-ansible | 12:03 | |
*** pto has quit IRC | 12:04 | |
*** pto has joined #openstack-ansible | 12:05 | |
*** pto_ has quit IRC | 12:06 | |
*** pto has quit IRC | 12:07 | |
openstackgerrit | Damian Dąbrowski proposed openstack/openstack-ansible master: Increase backend_fall value for haproxy_letsencrypt_service https://review.opendev.org/c/openstack/openstack-ansible/+/790051 | 12:10 |
openstackgerrit | Damian Dąbrowski proposed openstack/openstack-ansible master: Increase backend_fall value for haproxy_letsencrypt_service https://review.opendev.org/c/openstack/openstack-ansible/+/790051 | 12:11 |
*** pto has joined #openstack-ansible | 12:13 | |
*** rh-jelabarre has joined #openstack-ansible | 12:29 | |
*** shyamb has quit IRC | 12:36 | |
*** pto has quit IRC | 12:38 | |
*** pto has joined #openstack-ansible | 12:39 | |
*** pto has quit IRC | 12:43 | |
*** pto has joined #openstack-ansible | 12:43 | |
openstackgerrit | Merged openstack/openstack-ansible-os_zun master: Use ansible_facts[] instead of fact variables https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/780733 | 12:53 |
*** pto has quit IRC | 13:02 | |
*** pto has joined #openstack-ansible | 13:03 | |
*** pto has joined #openstack-ansible | 13:03 | |
*** oleksandry has quit IRC | 13:32 | |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Allow to configure corosync ports https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/789241 | 13:54 |
*** vakuznet has joined #openstack-ansible | 13:55 | |
vakuznet | hi, having keystone issue with recent train releses: http://paste.openstack.org/show/804999/ | 13:59 |
openstackgerrit | Merged openstack/openstack-ansible-os_keystone master: Correct shibboleth apache module name for ubuntu 20.04 https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/790018 | 14:00 |
openstackgerrit | Merged openstack/openstack-ansible-os_cloudkitty master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/780874 | 14:01 |
*** oleksandry has joined #openstack-ansible | 14:04 | |
*** dave-mccowan has joined #openstack-ansible | 14:05 | |
noonedeadpunk | no idea if that's deployment issue or keystone itself. feels actually more like code issue | 14:06 |
jrosser | vakuznet: i am taking a guess that the version of python-ldap is not compatible with the train version of keystone | 14:07 |
jrosser | maybe there is a missing constraint (outside OSA) and an update to python-ldap has broken things | 14:07 |
noonedeadpunk | but I'd expect it to be constrained... | 14:07 |
*** akahat is now known as akahat|ruck | 14:07 | |
noonedeadpunk | `python-ldap===3.2.0` | 14:07 |
noonedeadpunk | vakuznet: can you check that it's the version you have in venv? | 14:08 |
jrosser | noonedeadpunk: you were right about transport_url template being problematic http://paste.openstack.org/show/805002/ | 14:11 |
vakuznet | version did not chage since 20.2.0 http://paste.openstack.org/show/805001 | 14:11 |
jrosser | oh wait - it's my error in the code, not the template | 14:11 |
openstackgerrit | Merged openstack/openstack-ansible-galera_server master: Add galera devel packages installation https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/789786 | 14:13 |
*** lemko is now known as lemko1 | 14:14 | |
*** lemko1 is now known as lemko | 14:14 | |
*** lemko is now known as lemko_1 | 14:15 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/790006 | 14:15 |
openstackgerrit | Merged openstack/openstack-ansible-os_glance master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/780749 | 14:15 |
*** lemko_1 is now known as lemko | 14:15 | |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Replace deprecated host param for monitors https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/789243 | 14:16 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/790034 | 14:16 |
openstackgerrit | Merged openstack/openstack-ansible-os_masakari master: Add masakari-introspectiveinstancemonitor support https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/789244 | 14:16 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/790035 | 14:16 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/790036 | 14:17 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_nova master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/790037 | 14:18 |
openstackgerrit | Jonathan Rosser proposed openstack/ansible-role-pki master: WIP - Create server certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/788021 | 14:19 |
vakuznet | jrosser: https://opendev.org/openstack/keystone/commit/105f95795f661f8106b3f33b87662024e5bf6dcb might be a reason | 14:21 |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/780844 | 14:25 |
vakuznet | noonedeadpunk: commit references python-ldap-3.3.0 in commit message | 14:27 |
noonedeadpunk | for U it's already 3.2.0.... | 14:29 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-haproxy_server master: WIP - Use external PKI role to manage haproxy self-signed certificates https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/790078 | 14:29 |
noonedeadpunk | but still not 3.3.0 | 14:29 |
noonedeadpunk | so it's has been not wises backport I guess.... | 14:30 |
noonedeadpunk | I think worth reaching them in #openstack-keystone | 14:31 |
fridtjof[m] | hey again, in the setup-openstack stage of osa victoria right now, and I think I've hit a bug or misconfiguration | 14:31 |
jrosser | noonedeadpunk: serious patch here https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/790078 | 14:31 |
jrosser | noonedeadpunk: it needs to support 1) user supplied certs 2) standalone use of haproxy_server role 3) haproxy_server role used in OSA 4) Certbot | 14:32 |
* jrosser head hurts | 14:32 | |
noonedeadpunk | should it have depends-on? | 14:32 |
fridtjof[m] | it gets past the keystone bootstrap step, and then waits for keystone to come up by checking http://<internal lb vip>:5000 for a valid response | 14:32 |
jrosser | yeah :) well spotted! | 14:32 |
fridtjof[m] | haproxy only serves https though, so this never succeeds - why would it try to talk HTTP here? | 14:33 |
vakuznet | /join #openstack-keystone | 14:33 |
fridtjof[m] | looking at the relevant variable here: https://opendev.org/openstack/openstack-ansible-os_keystone/src/branch/stable/victoria/defaults/main.yml#L167 | 14:34 |
jrosser | fridtjof[m]: in the standard configuration haproxy serves http on the internal endpoint | 14:34 |
jrosser | and https on the external endpoint | 14:34 |
fridtjof[m] | ....oh | 14:34 |
fridtjof[m] | let's continue with another question - do i need both an external and internal lb VIP? | 14:35 |
jrosser | so i think here the debug steps are to try to replicate the request to internal_vip:5000 with curl | 14:35 |
fridtjof[m] | In my environment there's not really a distinction | 14:35 |
jrosser | short answer is yes | 14:35 |
fridtjof[m] | (my issue here is that I set both to the same IP then) | 14:35 |
jrosser | ah right, thats not going to work | 14:35 |
jrosser | you can't bind to the same port on the same IP for both the http and https services | 14:36 |
fridtjof[m] | alright, i'll just change the external one to be another IP then | 14:36 |
fridtjof[m] | of course, yeah | 14:36 |
fridtjof[m] | thanks a lot for the pointer in the right direction :D | 14:37 |
jrosser | no worries | 14:38 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-haproxy_server master: WIP - Use external PKI role to manage haproxy self-signed certificates https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/790078 | 14:42 |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/786842 | 14:47 |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: [reno] Stop publishing release notes https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/772023 | 14:47 |
noonedeadpunk | vakuznet: in the meanwhile I believe you can try setting higher python-ldap version in `global-requirement-pins.txt` | 14:48 |
openstackgerrit | Merged openstack/openstack-ansible-specs master: Protecting plaintext configs https://review.opendev.org/c/openstack/openstack-ansible-specs/+/788829 | 14:49 |
vakuznet | noonedeadpunk: i'll try. python-ldap bumped to 3.3.1 in victoria. ussuri might have the same issue. | 14:56 |
noonedeadpunk | oh... what python version do you have in venv? | 15:00 |
noonedeadpunk | vakuznet: are you running centos? (sorry mixed channels) | 15:02 |
noonedeadpunk | vakuznet: you can try setting `openstack_venv_python_executable: python3` as well, since nowadays centos 7 shipps py36 along with libselinux modules | 15:07 |
vakuznet | 2.7 /openstack/venvs/keystone-20.2.5/lib/python2.7 | 15:09 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/790006 | 15:13 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/790034 | 15:14 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/790035 | 15:15 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/790036 | 15:15 |
jrosser | vakuznet: if you want to upgrade to ussri then you'd get a full python3 deployment out of the box even on centos-7 | 15:17 |
*** gyee has joined #openstack-ansible | 15:26 | |
*** macz_ has joined #openstack-ansible | 15:35 | |
*** macz_ has quit IRC | 15:35 | |
*** macz_ has joined #openstack-ansible | 15:35 | |
*** rpittau is now known as rpittau|afk | 15:42 | |
jrosser | noonedeadpunk: seems haproxy_server role still uses functional tests, thoughs on that? | 15:44 |
*** sshnaidm is now known as sshnaidm|afk | 15:44 | |
noonedeadpunk | yeah, I didn't really patched things for infra jobs :( | 15:45 |
jrosser | ok, ill change it | 15:47 |
*** oleksandry has quit IRC | 15:48 | |
*** oleksandry has joined #openstack-ansible | 15:49 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-haproxy_server master: WIP - Use external PKI role to manage haproxy self-signed certificates https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/790078 | 15:50 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-haproxy_server master: Use integrated tests for haproxy_server https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/790090 | 15:50 |
*** oleksandry has quit IRC | 15:56 | |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Add variables for rabbitmq ssl configuration https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/790035 | 16:02 |
openstackgerrit | Jonathan Rosser proposed openstack/openstack-ansible-haproxy_server master: WIP - Use external PKI role to manage haproxy self-signed certificates https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/790078 | 16:15 |
openstackgerrit | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Gather minimal facts in CI https://review.opendev.org/c/openstack/openstack-ansible/+/790042 | 16:24 |
openstackgerrit | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Don't collect virtual facts https://review.opendev.org/c/openstack/openstack-ansible/+/789926 | 16:26 |
noonedeadpunk | ok, yes, that's kind of real situyation what we have out of ci right now https://review.opendev.org/c/openstack/openstack-ansible/+/790042/2 | 16:27 |
fridtjof[m] | huh. I have separate external/internal VIPs now, but only the internal one works (but with HTTPS??). I can see on both infra hosts that neither of them has the external IP configured | 17:09 |
fridtjof[m] | Both IPs are in the same subnet, and their _interface vars both have the same interface set. Could this be the problem, aka does it 100% expect there to be two interfaces? | 17:10 |
*** ThiagoCMC has joined #openstack-ansible | 17:11 | |
*** andrewbonney has quit IRC | 17:13 | |
*** macz_ has quit IRC | 17:15 | |
fridtjof[m] | (or could it be caused by some idempotency issue? I just ran the playbooks again instead of reinstalling all half-deployed hosts) | 17:15 |
noonedeadpunk | it can be the same interface, but must be different ip addresses | 17:18 |
noonedeadpunk | worth checking keepalived, since it's responsible for handling VIPs | 17:18 |
fridtjof[m] | ah, restarting keepalived did it | 17:26 |
fridtjof[m] | looks to be working! :) | 17:28 |
fridtjof[m] | (should maybe wait for system testing tomorrow :P hope it's smooth sailing for that) | 17:30 |
*** macz_ has joined #openstack-ansible | 17:34 | |
openstackgerrit | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Split keepalived liveness checks for internal/external networks https://review.opendev.org/c/openstack/openstack-ansible/+/782374 | 17:36 |
jrosser | noonedeadpunk: i'm not really knowing what to suggest next here https://review.opendev.org/c/openstack/openstack-ansible/+/788031 | 17:37 |
jrosser | argh | 17:37 |
jrosser | here http://lists.openstack.org/pipermail/openstack-discuss/2021-May/022313.html | 17:37 |
jrosser | i can't really decide if it's user error / ansible bug / failing to restart apache after disabling the shib mod....... | 17:39 |
noonedeadpunk | oh, well I tried to follow it | 17:40 |
openstackgerrit | Merged openstack/openstack-ansible master: Increase backend_fall value for haproxy_letsencrypt_service https://review.opendev.org/c/openstack/openstack-ansible/+/790051 | 17:40 |
noonedeadpunk | haven't seen this specific reply though | 17:40 |
jrosser | i'm not sure if --extra-vars '{"keystone_sp_distro_packages":["libapache2-mod-auth-openidc"]}'” is on a fresh deploy or somehow that fixes an exsiting one | 17:46 |
noonedeadpunk | I'd bet that would be fresh one | 17:46 |
jrosser | yeah, i think you're right | 17:46 |
jrosser | i replicated the logic in a test playbook to make sure we didnt have some var_foo | ternary('true', 'false') thing always evaulating to true | 17:47 |
jrosser | and it looked ok | 17:47 |
noonedeadpunk | ok, so. despite mod-auth-openidc and mod-shib are not conflicting based on the curl, they still should not be enabled at the same time, right? | 17:47 |
jrosser | the complexity is on bionic where you cannot have both becasue of libcurl3/4 | 17:48 |
jrosser | but on focal it is fine for both to be installed | 17:48 |
jrosser | we've not tested that here (not got a focal deploy yet) | 17:48 |
jrosser | i guess i could drop our keystone OIDC config into an AIO though | 17:49 |
noonedeadpunk | oh, ok, so this should disable module https://opendev.org/openstack/openstack-ansible-os_keystone/src/branch/master/tasks/keystone_apache.yml#L48-L57 | 17:49 |
jrosser | yes, thats right | 17:49 |
jrosser | and it should restart apache when it's done that | 17:50 |
noonedeadpunk | oh, hm | 17:55 |
noonedeadpunk | First thing I don't like is shibd.service which is enabled. I'm not sure we disable it somewhere later? | 17:59 |
noonedeadpunk | the second is that when I try to remove module, after apache installation, it jsut fails | 18:00 |
noonedeadpunk | "Error executing /usr/sbin/apache2ctl: AH00526: Syntax error on line 6 of /etc/apache2/conf-enabled/shib.conf:\nInvalid command 'ShibCompatValidUser', perhaps misspelled or defined by a module not included in the server configuration\n"} | 18:00 |
noonedeadpunk | so feels like we need to clean up /etc/apache2/conf-enabled/shib.conf as well when keystone_sp_apache_mod_shib is false | 18:01 |
noonedeadpunk | somewhere right after installation | 18:01 |
jrosser | there doesnt seem to be an ansible module for that | 18:03 |
jrosser | is there some extra dependancy we're getting aside from the apache module? | 18:03 |
noonedeadpunk | no, not really. I think it's part of the package | 18:08 |
noonedeadpunk | https://packages.ubuntu.com/focal/amd64/libapache2-mod-shib/filelist | 18:09 |
jrosser | ah no shibd.service | 18:10 |
noonedeadpunk | ah, wait, yes, extra requirements | 18:10 |
noonedeadpunk | http://paste.openstack.org/show/805009/ | 18:11 |
noonedeadpunk | I think main issue we have is indeed presence of /etc/apache2/conf-enabled/shib.conf once module is disabled | 18:12 |
noonedeadpunk | so apache fails to restart | 18:12 |
jrosser | there doesnt seem to be an ansible a2enconf type module | 18:12 |
* jrosser away | 18:13 | |
noonedeadpunk | let's probably just unlink then... dunno... | 18:16 |
*** zul_ has joined #openstack-ansible | 18:56 | |
openstackgerrit | Damian Dąbrowski proposed openstack/openstack-ansible stable/victoria: Increase backend_fall value for haproxy_letsencrypt_service https://review.opendev.org/c/openstack/openstack-ansible/+/790158 | 19:56 |
*** recyclehero has joined #openstack-ansible | 20:02 | |
*** dirk has quit IRC | 20:31 | |
*** zul_ has quit IRC | 21:55 | |
*** macz_ has quit IRC | 23:14 | |
*** tosky has quit IRC | 23:17 | |
*** rh-jelabarre has quit IRC | 23:37 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!