| -@gerrit:opendev.org- Michal Nasiadka proposed: [zuul/zuul-jobs] 973423: Use mirror_info in configure-mirrors role for pypi/wheel - take 2 https://review.opendev.org/c/zuul/zuul-jobs/+/973423 | 06:23 | |
| @priteau:matrix.org | Hello. Do you know why this change is not going through the gate? I don't even see it in the Zuul status dashboard: https://review.opendev.org/c/openstack/kolla-ansible/+/974029 | 11:30 |
|---|---|---|
| @mnasiadka:matrix.org | It didn't have Backport Candidate set, we require that in Kolla/Kolla-Ansible | 13:51 |
| @priteau:matrix.org | I always forget about this rule 🤦 | 15:50 |
| @clarkb:matrix.org | Pierre Riteau: mnasiadka in the Gerrit web UI you can hover over each submit requirement list in the top left area of the change page and it will tell if it is satisfied or not and there is a little expander button to see the underlying rule | 15:53 |
| @clarkb:matrix.org | Zuul takes care of the Verified submit requirement when gating succeeds but all others will likely require input from reviewers | 15:53 |
| @mnasiadka:matrix.org | I've noticed that after W+1 is set, when we set BC (to satisfy merging rules) Zuul doesn't always pick up gating - or maybe that's a Gerrit thing | 15:54 |
| @clarkb:matrix.org | mnasiadka: zuul gate is only triggered by workflow votes | 15:55 |
| @mnasiadka:matrix.org | That explains | 15:55 |
| @clarkb:matrix.org | https://opendev.org/openstack/project-config/src/branch/master/zuul.d/pipelines.yaml#L83-L91 (verified +1 is there to handle rechecks going from check to gate automatically) | 15:55 |
| @clarkb:matrix.org | If anyone has a chance to review https://review.opendev.org/c/opendev/system-config/+/972001 and test via the held node today i should be able to keep an eye on an etherpad upgrade. The held node is at 23.253.20.107 with a clarkb-test pad in it | 15:57 |
| @mnasiadka:matrix.org | Clark: not that's a big pain (with post W+1 BC votes) - would adding BC to trigger be doable, or would that increase Zuul load in a great manner? | 16:01 |
| @clarkb:matrix.org | I think the main concern would be that that may not necessarily be standard behavior expected by all projects in the tenant? That pipeline config applies to all of the projects in the openstack tenant so it is important that we think about behavior consistency | 16:02 |
| @clarkb:matrix.org | I wonder if there is a way to express "could merge except for a missing Verified +2" and let that be a trigger on any new vote | 16:02 |
| @clarkb:matrix.org | https://review.opendev.org/c/opendev/system-config/+/973540 is the other change in the backlog which will attempt mysqldumps over 127.0.0.1 instead of the unix socket in hopes that behaves better if mariadb is restarted during a backup | 16:04 |
| @clarkb:matrix.org | There is a way to query detailed submit requirements status for each change. So theoretically zuul could do something like this where it gets a vote event for a change and then it could query to see if all requirements are satisfied except for Verified +2 then gate it. But even Verified +2 is convention so figuring out the right way to express this might be a challenge | 16:10 |
| @mnasiadka:matrix.org | Yeah, I’m not currently looking for more work :) | 16:57 |
| @clarkb:matrix.org | I haven't heard any concerns about the proposed service coordinator election timing proposed in yesterday's meeting. I think I'll go ahead and send an email for that today and any further feedback can be sent to the thread | 18:03 |
| @clarkb:matrix.org | we have time if there are concerns to adjust, but getting it on the mailing list to a broader audience seems like a good next step | 18:03 |
| @fungicide:matrix.org | heading out to an appointment, but i'll try to take a look at the etherpad upgrade when i get back | 18:17 |
| @clarkb:matrix.org | thanks | 18:18 |
| @clarkb:matrix.org | Service coordinator election planning email has been sent. We can followup on that thread if there are any adjustments that need to be made | 18:54 |
| @mnasiadka:matrix.org | lists.openstack.org seems to be unresponsive/extremely slow - is that only for me? | 19:45 |
| @fungicide:matrix.org | mnasiadka: load average is fairly low this time at least | 19:47 |
| @fungicide:matrix.org | mariadbd seems fairly busy though | 19:48 |
| @fungicide:matrix.org | oh, and there's a uwsgi process with a whole cpu pegged a lot of the time | 19:48 |
| @fungicide:matrix.org | might be some bot army has exhausted the apache worker slots again, checking... | 19:48 |
| @clarkb:matrix.org | a busy db may also indicate reuqests that are expensive are being made. Tailing the apache logs can give a sense for that | 19:49 |
| @clarkb:matrix.org | as well as bot army behaviors in general | 19:49 |
| -@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed: [opendev/system-config] 974243: Don't rewrite the mod_status path in Apache https://review.opendev.org/c/opendev/system-config/+/974243 | 19:59 | |
| @fungicide:matrix.org | we'll need that ^ to reach /server-status on lists.o.o | 19:59 |
| @clarkb:matrix.org | fungi: is the localhost only access encoded elsewhere? | 20:00 |
| @clarkb:matrix.org | oh huh maybe that is part of the ubuntu supplied config as I see us excludign server-status similarly for services like etherpad without setting up the server-status location in config | 20:01 |
| @fungicide:matrix.org | Clark: yes, in /etc/apache2/mods-enabled/status.conf there's a `Require local` on the server-status handler in the /server-status location block | 20:04 |
| @fungicide:matrix.org | that file also includes a commented-out example of how to grant access to other addresses | 20:05 |
| @clarkb:matrix.org | thanks. I kinda figured once I realized other services do similar wihtout configuring mod status directly | 20:06 |
| -@gerrit:opendev.org- Zuul merged on behalf of Dmitriy Rabotyagov: [openstack/project-config] 973383: Add template to the os_freezer repo https://review.opendev.org/c/openstack/project-config/+/973383 | 20:46 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Dmitriy Rabotyagov: [openstack/project-config] 973388: Add os_watcher zuul templates https://review.opendev.org/c/openstack/project-config/+/973388 | 20:49 | |
| -@gerrit:opendev.org- Doug Goldstein proposed: [openstack/project-config] 974261: update from ironic-core to ironic-reviewers for openstacksdk https://review.opendev.org/c/openstack/project-config/+/974261 | 22:52 | |
| -@gerrit:opendev.org- Doug Goldstein proposed: [openstack/project-config] 974261: update from ironic-core to ironic-reviewers for openstacksdk https://review.opendev.org/c/openstack/project-config/+/974261 | 22:57 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!