| Tengu | Hello there! I was wondering if anything has been done to the proxy configuration related to the name, especially for the ansible galaxy case? | 00:41 |
|---|---|---|
| Tengu | For the record, galaxy has a bad practice of pushing the full uri with hostname in its answer, and that hostname is then used as a caching key in ansible-galaxy. | 00:42 |
| Tengu | Iirc, it creates issue because httpd doesn't replace the name by the correct/expected value, leading to crash in the ansible-galaxy cli | 00:43 |
| Tengu | Last status I remember was "we need to expose some new var to the proxy configuration and generate the httpd.comf with the correct replacement". Iirc that was from you, fungi :) | 00:44 |
| *** dasm is now known as dasm|off | 01:49 | |
| opendevreview | Vishal Manchanda proposed openstack/project-config master: Rename x/xstatic-angular-fileupload to openstack/xstatic-angular-fileupload https://review.opendev.org/c/openstack/project-config/+/873843 | 06:43 |
| *** blarnath is now known as d34dh0r53 | 06:47 | |
| opendevreview | Vishal Manchanda proposed openstack/project-config master: Rename x/xstatic-angular-fileupload->openstack/xstatic-angular-fileupload https://review.opendev.org/c/openstack/project-config/+/873843 | 06:54 |
| *** jpena|off is now known as jpena | 08:31 | |
| fungi | Tengu: it's been a while, but i think there was a change for that? i'll look | 12:56 |
| Tengu | fungi: thanks ! And sorry for that old zombie, I got "a little bit" sidetracked. | 12:57 |
| fungi | okay, so the offending lines are at https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/mirror/templates/mirror.vhost.j2#L583-L588 | 13:00 |
| fungi | there's only one open review proposing a change to that file, and it is unrelated | 13:01 |
| fungi | Tengu: i think what we were talking about was adding an extra var in the files like https://opendev.org/opendev/system-config/src/branch/master/inventory/service/host_vars/mirror01.bhs1.ovh.opendev.org.yaml | 13:05 |
| fungi | maybe call it something like primary_mirror_fqdn | 13:06 |
| fungi | so in that example we'd add a line like "primary_mirror_fqdn: mirror.bhs1.ovh.opendev.org" | 13:07 |
| fungi | the rackspace mirrors are multi-homed, and we should prefer their inside name since that's what the nodes are told to connect to, so in https://opendev.org/opendev/system-config/src/branch/master/inventory/service/host_vars/mirror01.iad.rax.opendev.org.yaml we'd do "primary_mirror_fqdn: mirror-int.iad.rax.opendev.org" | 13:08 |
| fungi | then in the vhost substitute lines we'd put primary_mirror_fqdn instead of apache_server_name | 13:09 |
| fungi | we do have some magic in playbooks/roles/mirror/tasks/main.yaml to generate apache_server_alias by stripping the digits out of the first part of inventory_hostname but that doesn't take the mirror-int interfaces on the rackspace mirrors into account, and i'm not even all that sure what we're relying on ServerAlias for in the vhost config anyway | 13:12 |
| fungi | so i'm hesitant to try leveraging that, a straight up hostvar is easier to reason about | 13:13 |
| fungi | probably what we should be doing is generating the ServerAlias line from the letsencrypt_certs data so we include all the possible names as aliases, but even then that leaves us needing some way to indicate a preferred one | 13:14 |
| opendevreview | Corey Bryant proposed openstack/openstack-zuul-jobs master: Add openstack-python3-charm-antelope-jobs https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/873907 | 13:20 |
| *** jpena is now known as jpena|off | 13:52 | |
| *** dasm|off is now known as dasm | 14:01 | |
| Tengu | fungi: yeah - it's a really tricky situation, especially since the module called to substitute name doesn't support httpd vars :(. | 14:18 |
| Tengu | if it supported them, we'd be all good...... | 14:18 |
| Tengu | :] can't we bump to httpd 2.5? :D | 14:19 |
| * Tengu ducks | 14:19 | |
| fungi | Tengu: anyway, what i described (adding primary_mirror_fqdn) should be an easy way to solve it if you want to propose a change for that | 14:25 |
| Tengu | fungi: not sure I'll be able to do it fast enough, with the "tiny" thing I have on my plate :/. I'll be off most of next week, sooo... :( | 15:09 |
| *** d34dh0r53 is now known as d34dh0r5- | 15:24 | |
| *** d34dh0r5- is now known as d34dh0r53 | 15:27 | |
| *** d34dh0r53 is now known as blarnath | 15:28 | |
| *** blarnath is now known as d34dh0r53 | 15:28 | |
| fungi | noted | 15:58 |
| clarkb | 2.5 isn't released yet right? | 16:15 |
| clarkb | its a dev version and 2.6 would be the resulting stable release? | 16:15 |
| fungi | correct | 16:20 |
| fungi | 2.4 is still the current release version | 16:20 |
| fungi | 2.5 is in beta since years | 16:20 |
| opendevreview | Vishal Manchanda proposed openstack/project-config master: Step 3: Remove project from infra System - Retire xstatic-font-awesome https://review.opendev.org/c/openstack/project-config/+/872835 | 16:35 |
| *** dasm is now known as dasm|off | 21:56 | |
| opendevreview | Merged openstack/project-config master: nodepool: size linaro max-servers to subnet https://review.opendev.org/c/openstack/project-config/+/871674 | 23:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!