| *** dviroel|afk is now known as dviroel | 00:08 | |
| *** dviroel is now known as dviroel|out | 01:33 | |
| opendevreview | OpenStack Proposal Bot proposed openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/866243 | 02:22 |
|---|---|---|
| *** yadnesh|away is now known as yadnesh | 04:00 | |
| *** akekane is now known as abhishekk | 04:59 | |
| *** jpena|off is now known as jpena | 08:31 | |
| *** yadnesh is now known as yadnesh|afk | 09:00 | |
| ade_lee | clarkb, yeah thanks - sorry I didn't realize I still had a hold on something | 09:51 |
| *** yadnesh|afk is now known as yadnesh | 09:58 | |
| frickler | elodilles: seems the oslo cleanup has increased the number of config errors again :-( "Job oslo.messaging-src-dsvm-* not defined" in multiple devstack-plugin-* repos and branches | 10:01 |
| elodilles | frickler: ohh, i see :/ | 10:03 |
| elodilles | frickler: it seems that there was a job defined in oslo.messaging's (?) queens branch that is used in devstack-plugin-* | 10:04 |
| elodilles | oslo.messaging-src-dsvm-full-amqp1-dual-centos-7 | 10:05 |
| elodilles | centos-7... seems quite old, i doubt it works nowadays | 10:05 |
| elodilles | from newer branches probably it's easy to remove these job definitons | 10:05 |
| elodilles | frickler: https://review.opendev.org/q/topic:remove-oslo-messaging-src-dsvm | 10:25 |
| Tengu | \o/ my patch to correct how ansible-galaxy is proxified works! | 10:26 |
| frickler | elodilles: thx, there is also devstack-plugin-kafka affected, but I neither wanted to blame you nor assume that you are doing all the cleanup, just share my sadness about the error count moving into the wrong direction | 10:30 |
| frickler | s/are doing/are responsible for doing/ | 10:31 |
| elodilles | frickler: i'm preparing a patch right now for devstack-plugin-kafka as well right now. no problem :) yes, that is sad, that as we remove branches, we not just decrease config errors, but sometimes also increase them :/ | 10:32 |
| *** dviroel|out is now known as dviroel | 10:43 | |
| *** rlandy|out is now known as rlandy | 11:20 | |
| Tengu | fungi, clarkb heya! when around, care to re-check the ansible-galaxy proxy thing? All is green now, even with the new tests \o/ | 12:44 |
| opendevreview | Merged openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/866243 | 12:54 |
| *** rlandy is now known as rlandy|brb | 13:03 | |
| Tengu | fungi, clarkb also, I'm trying to understand how to access the "zuul_site_mirror_fqdn" parameter - is it something pushed in some parameter file we can then load from without our own playbooks? if so, where is it located? I'd love to switch to that var instead of relying on the "old" bash thing. | 13:05 |
| *** akekane is now known as abhishekk | 13:07 | |
| fungi | Tengu: it should be a generally available ansible variable for the roles/playbooks zuul runs, but https://codesearch.opendev.org/?q=zuul_site_mirror_fqdn is a quick way to get some examples | 13:12 |
| Tengu | fungi: so I guess it's pushed in some default ansible variable location, such as /etc/ansible/host_vars or the like? | 13:12 |
| fungi | Tengu: looks like tripleo-quickstart-extras is already using it | 13:14 |
| Tengu | guess I can give it a try. That would be smarter/more elegant than making a "shell" call | 13:14 |
| Tengu | hmmm good catch. roles/create-zuul-based-reproducer/templates/launcher-playbook.yaml.j2: zuul_site_mirror_fqdn: "{{ zuul_site_mirror_fqdn | default(mirror_path) }}" | 13:14 |
| Tengu | (and other occurrences) | 13:15 |
| Tengu | let's see how it would work in tripleo-ansible/zuul.d/playbooks/pre.yml | 13:15 |
| *** rlandy|brb is now known as rlandy | 13:18 | |
| fungi | Tengu: so for example, if you need to set it in some /etc/ansible/galaxy.conf file, you could make a galaxy.conf.j2 template and then just embed "my_galaxy_url = https://{{ zuul_site_mirror_fqdn }}:4448/" in it or whatever | 13:18 |
| Tengu | oh, ini_file | 13:18 |
| Tengu | that's even better :) | 13:18 |
| Tengu | like that: https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 13:18 |
| fungi | sure | 13:18 |
| Tengu | I've been poking at that for a while, using the bash thing, but since we're supposed to get the env var... | 13:18 |
| fungi | lots easier than templatizing an entire conffile | 13:18 |
| Tengu | I'll see if it crashes or not - should work, if I understand well, so I'll be able to remove the comment on the "when" condition in order to make it OK on RDO as well (not sure they expose this var as well) | 13:19 |
| Tengu | and otherwise, it's better in case one is running it outside of the opendev infra. | 13:19 |
| Tengu | ~> if it works, as soon as the proxy config is corrected/deployed, we'll be good | 13:20 |
| Tengu | thanks for the help, reviews and pointers! | 13:20 |
| *** frenzy_friday|rover is now known as frenzy_friday|rover|lunch | 13:33 | |
| *** dasm|off is now known as dasm | 13:46 | |
| fungi | sure thing | 13:51 |
| Tengu | cool! the zuul var is indeed available out of the box. how lovely is that :) | 13:52 |
| Tengu | "just" a matter of getting the config in then. | 13:53 |
| fungi | yeah, we tried to keep it simple and convenient | 13:54 |
| *** yadnesh is now known as yadnesh|away | 13:55 | |
| *** frenzy_friday|rover|lunch is now known as frenzy_friday|rover | 14:01 | |
| Tengu | fungi: I guess you won't get the patch merged right before the weekend, correct? | 14:33 |
| fungi | clarkb will likely be around soon, so if he has time to review it again we might get it merged in the next few hours | 14:34 |
| fungi | it's still thursday after all | 14:34 |
| Tengu | oh, cool! and is it directly applied? | 14:34 |
| fungi | yes, the jobs which run in the deploy pipeline triggered by the change merging will roll it out to all the mirror servers | 14:35 |
| Tengu | that's what I call a well managed infra-as-a-code :) | 14:36 |
| fungi | zuul as a continuous deployment system, right | 14:36 |
| Tengu | less work for humans, more load on the machines :) | 14:36 |
| fungi | at least until the uprising | 14:38 |
| Tengu | :) | 14:38 |
| Tengu | hmmm. interesting. ansible-galaxy allows to get a list of servers. maybe I can put the proxy first, and if for any reason it fails, I can then fallback on the original galaxy.ansible.com... | 14:39 |
| *** blarnath is now known as d34dh0r53 | 15:06 | |
| clarkb | sorry very slow start today | 17:05 |
| clarkb | Tengu: fungi: there is a bug in the port specifications for that change. Otherwise lgtm | 17:26 |
| *** jpena is now known as jpena|off | 17:27 | |
| fungi | oh, good catch | 17:27 |
| Tengu | oh? | 17:36 |
| Tengu | I'll check tomorrow... | 17:36 |
| Tengu | oh. dang. stupid me. | 17:38 |
| Tengu | so merge on Monday to avoid weekend things then ;) | 17:38 |
| fungi | Tengu: clarkb: i just pushed a revision as penance for missing those copy-paste errors | 17:41 |
| fungi | but i agree it's interesting the way we've got the tests rigged up it worked on the other listening port | 17:41 |
| fungi | probably because it was just the ServerName/ServerAlias which aren't critical | 17:42 |
| fungi | because, as you say, sni already has that worked out by the time it's reached | 17:42 |
| opendevreview | Jay Faulkner proposed openstack/project-config master: Allow Ironic/Sushy cores to toggle WIP state https://review.opendev.org/c/openstack/project-config/+/863931 | 19:54 |
| *** dviroel is now known as dviroel|out | 21:02 | |
| clarkb | ade_lee: I've finally got around to cleaning up that held node today and it seems zuul forgot about it but nodepool didn't. SO this may have been ab ug on our side | 21:56 |
| clarkb | should be all cleaned up now though | 21:56 |
| *** dasm is now known as dasm|off | 22:07 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!