| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 830541: Only process pipelines if there are outstanding events https://review.opendev.org/c/zuul/zuul/+/830541 | 01:00 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 01:06 | |
| - [zuul/nodepool] 830525: Update AWS driver to use statemachine framework https://review.opendev.org/c/zuul/nodepool/+/830525 | ||
| - [zuul/nodepool] 830526: Refactor AWS driver tests https://review.opendev.org/c/zuul/nodepool/+/830526 | ||
| - [zuul/nodepool] 830527: Add additional tests to the aws driver https://review.opendev.org/c/zuul/nodepool/+/830527 | ||
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 830554: Look up worker_zone for log streaming from executor https://review.opendev.org/c/zuul/zuul/+/830554 | 07:22 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 830555: Don't fail on missing change_queues key in status json https://review.opendev.org/c/zuul/zuul/+/830555 | 07:31 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 830554: Look up worker_zone for log streaming from executor https://review.opendev.org/c/zuul/zuul/+/830554 | 08:58 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 830554: Look up worker_zone for log streaming from executor https://review.opendev.org/c/zuul/zuul/+/830554 | 09:01 | |
| @tristanc_:matrix.org | Is storyboard a good place to create request for Zuul feature ? | 13:14 |
|---|---|---|
| @tristanc_:matrix.org | For example, as a job author, I would like to add an artifact directly to the summary page (for example Unit Test Report) so that the user can access it directly. Where this kind of things can be triaged? | 13:17 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 810955: GUI: Add tenant dropdown to top menu https://review.opendev.org/c/zuul/zuul/+/810955 | 13:26 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 830541: Only process pipelines if there are outstanding events https://review.opendev.org/c/zuul/zuul/+/830541 | 14:19 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 829384: Add buildset start/end db columns https://review.opendev.org/c/zuul/zuul/+/829384 | 14:25 | |
| @jpew:matrix.org | I'm trying to re-enqueue a buildset triggered by a timer from the website, but it's not doing anything; is there something I'm missing? | 14:37 |
| @mhuin:matrix.org | > <@jpew:matrix.org> I'm trying to re-enqueue a buildset triggered by a timer from the website, but it's not doing anything; is there something I'm missing? | 14:39 |
| From the React App you mean? | ||
| @jpew:matrix.org | Ya | 14:39 |
| @mhuin:matrix.org | jpew: can you open the dev tools in your browser, usually Ctrl+I - and check the network call when you do the re-enqueue | 14:40 |
| @mhuin:matrix.org | it should be something like https://zuu/path/to/api/tenant/XXX/enqueue | 14:40 |
| @mhuin:matrix.org | hm there might be project somewhere in the url but it definitely ends with enqueue | 14:42 |
| @jpew:matrix.org | Ya, it is that | 14:43 |
| @jpew:matrix.org | It's the same as when I re-enqueue another (e.g. gerrit triggered) change | 14:44 |
| @jpew:matrix.org | I can't seem to manually trigger the change from the command line anymore since gearman went away | 14:44 |
| @jpew:matrix.org | But when I did, I had to use enqueue-ref to trigger a normally timer triggered change | 14:44 |
| @mhuin:matrix.org | jpew: you have to set the [web-client] section in the config file used by the zuul client | 14:45 |
| @mhuin:matrix.org | the web GUI should be able to figure out when to call the REST API equivalent of enqueue or enqueue-ref - basically it checks whether the change param looks like a gerrit change or a commit hash | 14:46 |
| @mhuin:matrix.org | anyhoo have you checked the network trace in the dev console when you try to run a re-enqueue? | 14:47 |
| @jpew:matrix.org | mhu: Ya, I can see the POST; is there something else I can look at? (Sorry, I'm not a Web Developer) | 14:55 |
| @mhuin:matrix.org | Do you see the response code ? any error returned ? | 14:56 |
| @jpew:matrix.org | It just says "true" | 14:57 |
| @jpew:matrix.org | The request it sent in was `{"pipeline":"periodic-daily","change":null}` | 14:57 |
| @mhuin:matrix.org | that null value may be the problem | 14:59 |
| @mhuin:matrix.org | but I'm surprised by the return value | 14:59 |
| @mhuin:matrix.org | do you have access to zuul's components logs? | 14:59 |
| @jpew:matrix.org | Ya | 14:59 |
| @mhuin:matrix.org | I'd have a look at the zuul-web and zuul-scheduler logs at request's time | 15:00 |
| @mhuin:matrix.org | if I were you | 15:00 |
| @jpew:matrix.org | Hmm, the trigger that it's createing has oldrev=00...00 newrev=00.00 and ref=null | 15:02 |
| @jpew:matrix.org | That doens't look correct | 15:02 |
| @mhuin:matrix.org | looks like a bug there, I'll have a look. You re-enqueued from the buildset page right? | 15:04 |
| @jpew:matrix.org | Ya | 15:04 |
| @mhuin:matrix.org | jpew: I sent you a message for debug | 15:10 |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 830628: Items in extra paths should be loaded in dependent changes https://review.opendev.org/c/zuul/zuul/+/830628 | 15:13 | |
| @tobias.henkel:matrix.org | zuul-maint: this would be an easy review: https://review.opendev.org/c/zuul/zuul/+/827327 (we were just bitten by a hanging gerrit blocking all schedulers) | 15:16 |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 830628: Items in extra paths should be loaded in dependent changes https://review.opendev.org/c/zuul/zuul/+/830628 | 15:16 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 830628: Items in extra paths should be loaded in dependent changes https://review.opendev.org/c/zuul/zuul/+/830628 | 15:49 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 829829: Don't reconfigure after every gitlab merge https://review.opendev.org/c/zuul/zuul/+/829829 | 16:56 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 830668: Add a debug log entry about reconfiguration with no file list https://review.opendev.org/c/zuul/zuul/+/830668 | 17:17 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 17:23 | |
| - [zuul/nodepool] 807806: Add "slots" to static node driver https://review.opendev.org/c/zuul/nodepool/+/807806 | ||
| - [zuul/nodepool] 811016: Expose slot number from metastatic driver https://review.opendev.org/c/zuul/nodepool/+/811016 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 829384: Add buildset start/end db columns https://review.opendev.org/c/zuul/zuul/+/829384 | 17:43 | |
| @clarkb:matrix.org | corvus: super minor thing on https://review.opendev.org/c/zuul/zuul/+/830668 but probably worth changing now just to avoid it causing problems later ? | 18:12 |
| @jim:acmegating.com | Clark: ++ | 18:24 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 830668: Add a debug log entry about reconfiguration with no file list https://review.opendev.org/c/zuul/zuul/+/830668 | 18:24 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Felix Edel: [zuul/zuul] 830555: Don't fail on missing change_queues key in status json https://review.opendev.org/c/zuul/zuul/+/830555 | 18:24 | |
| -@gerrit:opendev.org- Zuul merged on behalf of likui: [zuul/nodepool] 819614: Replace deprecated assertEquals https://review.opendev.org/c/zuul/nodepool/+/819614 | 18:27 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 830679: Increase mysql read timeout in tests https://review.opendev.org/c/zuul/zuul/+/830679 | 18:57 | |
| @jim:acmegating.com | felixedeltobiash Clark : ^ i think that's somewhat urgent | 18:58 |
| @jim:acmegating.com | both failures of https://review.opendev.org/826549 in the gate were due to that | 18:58 |
| @clarkb:matrix.org | corvus: can you check my questions on https://review.opendev.org/c/zuul/nodepool/+/807806 I think I must be missing something there. Or maybe we do leak the _node_slots datastructure contents | 19:02 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 19:10 | |
| - [zuul/zuul] 827327: Add ssh connection timeout https://review.opendev.org/c/zuul/zuul/+/827327 | ||
| - [zuul/zuul] 827328: Improve "zuul" driver debug log messages https://review.opendev.org/c/zuul/zuul/+/827328 | ||
| @jim:acmegating.com | Clark: i'll get back to you on that :) | 19:19 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 19:29 | |
| - [zuul/zuul] 830082: Restore merge failure reporting tests https://review.opendev.org/c/zuul/zuul/+/830082 | ||
| - [zuul/zuul] 830012: Add pipeline timing metrics https://review.opendev.org/c/zuul/zuul/+/830012 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 830013: Add "repo state" waiting status https://review.opendev.org/c/zuul/zuul/+/830013 | 19:36 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 830409: Add max retries to refresh of pipeline change list https://review.opendev.org/c/zuul/zuul/+/830409 | 19:38 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 830014: Add waiting status to queued https://review.opendev.org/c/zuul/zuul/+/830014 | 19:38 | |
| @jim:acmegating.com | Clark: ^ that required a small change due to a merge conflict | 19:39 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 830668: Add a debug log entry about reconfiguration with no file list https://review.opendev.org/c/zuul/zuul/+/830668 | 20:17 | |
| @clarkb:matrix.org | corvus: https://review.opendev.org/c/zuul/zuul/+/817825 seems to fail consistently so I'm going to assume that is related to the change for now. But heads up | 20:51 |
| @jim:acmegating.com | Clark: yes, i suspect it's flawed. it's low-priority for me; maybe i'll wip it. | 20:56 |
| @tobias.henkel:matrix.org | corvus: q on 830541 | 21:12 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 830083: Report which repo failed initial merge ops https://review.opendev.org/c/zuul/zuul/+/830083 | 21:23 | |
| @tobias.henkel:matrix.org | zuul-maint: 752022 would be an easy review which adds a zookeeper-timeout setting to nodepool which is necessary in our system to avoid zk disconnects | 21:33 |
| @jim:acmegating.com | tobiash: replied on 083; what do you think? | 21:34 |
| @jim:acmegating.com | tobiash: 022 could use docs i think | 21:36 |
| @tobias.henkel:matrix.org | whoops | 21:36 |
| @tobias.henkel:matrix.org | corvus: replied on 830541 | 21:38 |
| @jim:acmegating.com | sounds good, thx | 21:40 |
| @clarkb:matrix.org | I should review 830541 | 21:42 |
| @clarkb:matrix.org | corvus: tobiash I'm having a hard time parsing whether or not we want to implement the dirty flag before landing https://review.opendev.org/c/zuul/zuul/+/830541 ? | 21:51 |
| @clarkb:matrix.org | It seems like everyone agrees this is necesasry but then we are +2'ing? Do we think it is sufficient to do this in a followup? or should we go ahead and update not? | 21:51 |
| @clarkb:matrix.org | * It seems like everyone agrees this is necesasry but then we are +2'ing? Do we think it is sufficient to do this in a followup? or should we go ahead and update now? | 21:51 |
| @jim:acmegating.com | my read is not necessary but probably desirable. i can either add it right now, or probably in a few hours :) | 21:52 |
| @clarkb:matrix.org | I'm trying to reason about how to deal with lost pipeline handling if that were to happen to say opendev. I think for check/gate it is easy. You just reapprove or whatever. For timer triggered queues its a bit more complicated? maybe for that reason it is better to update and address it properly? | 21:53 |
| @jim:acmegating.com | any event for the pipeline whatsoever would wake it up again, including builds completing. | 21:54 |
| @jim:acmegating.com | if all builds are already complete and there's something stuck in there, you could probably do a tenant reconfig or something... | 21:54 |
| @tobias.henkel:matrix.org | Clark: I think it's quite unlikely to have a pipeline get into this state without any further events, that's why I'm +2 on that | 21:54 |
| @clarkb:matrix.org | ah ok it isn't just trigger events it is everything. Which makes sense as it looks at all three queues | 21:55 |
| @clarkb:matrix.org | ok I'll approve and we can improve it in a followup | 21:56 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 821481: Allow disabling host-key-checking in statemachine https://review.opendev.org/c/zuul/nodepool/+/821481 | 21:56 | |
| -@gerrit:opendev.org- Tobias Henkel proposed: [zuul/nodepool] 752022: Add zookeeper-timeout connection config https://review.opendev.org/c/zuul/nodepool/+/752022 | 22:01 | |
| @tobias.henkel:matrix.org | corvus: +2 on 830525 (aws driver rework) with an inline thought about a potential future improvement of quota handling | 23:03 |
| @jim:acmegating.com | tobiash: i like that idea. i think it *should* "just work" because of the way you set up the quotainformation class -- but would be a good standalone change since it could trip us up if we accidentally added assumptions about it. | 23:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!