| @clarkb:matrix.org | corvus: left a couple of questions if you have a moment to look | 00:04 |
|---|---|---|
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 820079: Fix mutation while iterating over queues https://review.opendev.org/c/zuul/zuul/+/820079 | 00:28 | |
| @jim:acmegating.com | Clark: thanks, replied/updated | 00:28 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 820080: Handle more than 1024 changes in the pipeline change list https://review.opendev.org/c/zuul/zuul/+/820080 | 00:30 | |
| @jim:acmegating.com | mordred: ^ updated 079 if you want to take a second look | 00:30 |
| @clarkb:matrix.org | corvus: hrm there may be one more spot in the supercedent pipeline | 00:31 |
| @clarkb:matrix.org | corvus: in getChangeQueue() but that one doesn't seem to do dequeues so maybe it is fine | 00:32 |
| @clarkb:matrix.org | I +2'd because I don't think that can deuque | 00:33 |
| @clarkb:matrix.org | and I can't type | 00:33 |
| @jim:acmegating.com | yeah, i added [:] then removed it from that because of the return statement | 00:43 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 820079: Fix mutation while iterating over queues https://review.opendev.org/c/zuul/zuul/+/820079 | 01:51 | |
| -@gerrit:opendev.org- Albin Vass proposed: [zuul/nodepool] 820024: Openshift: Enable configuring runtimeClassName https://review.opendev.org/c/zuul/nodepool/+/820024 | 08:41 | |
| @avass:vassast.org | corvus: Updated ^ but I'm setting -w until I have the possibility to test the change in an environment with sandboxed containers available. | 08:42 |
| @avass:vassast.org | Now with working tests and linting :) | 13:10 |
| -@gerrit:opendev.org- Albin Vass proposed: [zuul/nodepool] 820024: Openshift: Enable configuring runtimeClassName https://review.opendev.org/c/zuul/nodepool/+/820024 | 13:10 | |
| @mhuin:matrix.org | Hello, do you have any tips to debug "QueueItem XXX for Change YYY is a failing item because ['it has an invalid configuration']" ? | 13:21 |
| @mhuin:matrix.org | The change comes from a depends-on on another change | 13:21 |
| @mhuin:matrix.org | I can find the line and commit on the repo for that change as the executor prepares it: "Create reference refs/zuul/4f26aeafdb2367620a393c973eddbe8f8b846ebd at 4bc91c7fa94594046e5781d75894a58e3f654c71 in /var/lib/zuul/executor/YYY" | 13:24 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 820184: Web: fix whitelabel tenants with no auth https://review.opendev.org/c/zuul/zuul/+/820184 | 15:26 | |
| @jim:acmegating.com | mhu: zuul-maint: ^ that should fix the error we're seeing on zuul.openstack.org now | 15:27 |
| @mhuin:matrix.org | ah, good catch | 15:28 |
| @jim:acmegating.com | mhu: generally that message means there's a zuul config error, which should be reported to the change (it might be the dependency?) | 15:28 |
| @mhuin:matrix.org | corvus: the thing is, nothing was reported back to gerrit. The change was dropped silently from the pipeline, except for the logs I pasted | 15:29 |
| @mhuin:matrix.org | we ended up figuring out the problem, we were missing some roles from another dependency | 15:30 |
| @mhuin:matrix.org | is it possible to have a more informative message than 'invalid configuration' when this happens? | 15:30 |
| @jim:acmegating.com | yes, they should be in the report :) | 15:31 |
| @jim:acmegating.com | clearly something wasn't working as expected, but it's certainly the case that all errors should eventually be reported | 15:31 |
| @jim:acmegating.com | probably worth looking in the debug logs to find out what went wrong | 15:32 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 820194: REST API: add idx_min, idx_max params to getBuilds, getBuildsets https://review.opendev.org/c/zuul/zuul/+/820194 | 16:11 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 820204: Log zkobject paths with errors https://review.opendev.org/c/zuul/zuul/+/820204 | 17:11 | |
| @jim:acmegating.com | i've been digging into the issue that came up after my "chaos monkey" event yesterday; i can not determine the cause, or even confirm that it was actually related to my shutting down the scheduler (they both successfully processed the pipeline after the issue). but it's still a striking coincidence that it affected the pipeline that was being processed when i shut it down. anyway, that error info would have helped significantly ^ | 17:13 |
| @jim:acmegating.com | Clark, tobiash, tristanC ^ | 17:13 |
| @clarkb:matrix.org | Noted. I'll review after gerrit user summit | 17:14 |
| @jim:acmegating.com | oh wait, i want to change something there.... i'm going to upload a new ps | 17:14 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 820204: Log zkobject paths with errors https://review.opendev.org/c/zuul/zuul/+/820204 | 17:16 | |
| @jim:acmegating.com | okay, that's better | 17:16 |
| @jim:acmegating.com | (just wanted to make sure i was using log.error and log.exception in the right places) | 17:16 |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul] 800664: Improve performance of _saveRepoState https://review.opendev.org/c/zuul/zuul/+/800664 | 17:17 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 820210: Put pipeline icon to the left of the name https://review.opendev.org/c/zuul/zuul/+/820210 | 17:32 | |
| @jim:acmegating.com | mhu, tristanC : ^ not very important, but i think that may fit in with existing style a bit better | 17:33 |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul] 755929: [web][config] move timezone component to preferences https://review.opendev.org/c/zuul/zuul/+/755929 | 17:44 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul] 755929: [web][config] move timezone component to preferences https://review.opendev.org/c/zuul/zuul/+/755929 | 17:49 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul] 755929: [web][config] move timezone component to preferences https://review.opendev.org/c/zuul/zuul/+/755929 | 17:49 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 820184: Web: fix whitelabel tenants with no auth https://review.opendev.org/c/zuul/zuul/+/820184 | 18:15 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 820204: Log zkobject paths with errors https://review.opendev.org/c/zuul/zuul/+/820204 | 18:54 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 820194: REST API: add idx_min, idx_max params to getBuilds, getBuildsets https://review.opendev.org/c/zuul/zuul/+/820194 | 21:00 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul] 755929: [web][config] move timezone component to preferences https://review.opendev.org/c/zuul/zuul/+/755929 | 21:11 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 819745: Add a keycloak tutorial https://review.opendev.org/c/zuul/zuul/+/819745 | 21:48 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!