| -@gerrit:opendev.org- Ian Wienand proposed: | 00:09 | |
| - [zuul/zuul] 829617: merger: report missing object as ERROR https://review.opendev.org/c/zuul/zuul/+/829617 | ||
| - [zuul/zuul] 829842: Unit testing for ref going missing https://review.opendev.org/c/zuul/zuul/+/829842 | ||
| @iwienand:matrix.org | corvus: Clark ^ i'm just spinning in circles on the unit-test at this point, i've split it out for now. hopefully it's something obvious i'm missing | 00:12 |
|---|---|---|
| -@gerrit:opendev.org- Ian Wienand proposed: | 00:17 | |
| - [zuul/zuul] 829617: merger: report missing object as ERROR https://review.opendev.org/c/zuul/zuul/+/829617 | ||
| - [zuul/zuul] 829842: Unit testing for ref going missing https://review.opendev.org/c/zuul/zuul/+/829842 | ||
| @clarkb:matrix.org | The shape of that latest patchset looks about right to me. I wonder what the repo_state will look like in the test log (I'm assuming it will fail based on what you said) | 00:21 |
| @clarkb:matrix.org | Its possible we need some additional external factor to cause the repo_state to be called with the additional info and that is lacking (just based on what I had seen in the log before) | 00:22 |
| @jim:acmegating.com | agree, test looks generally correct | 00:32 |
| @fungicide:matrix.org | hey everybody, fatih reminded me that the cdcon cfp is closing in a little over 24 hours, and it could be a good place to present about zuul to people who aren't familiar with it. i sent a note about it to the mailing list too just now, but proposals can be submitted here: https://events.linuxfoundation.org/cdcon/program/cfp/ | 00:44 |
| @clarkb:matrix.org | Note the event conflicts with open infra summit so if you are like me and already have obligations to that event... | 00:45 |
| @fungicide:matrix.org | yep, same here. but i figure maybe there's someone who's not going to berlin yet doesn't mind austin | 00:46 |
| @clarkb:matrix.org | yup just calling that out here too in case people submit before reading your email :) | 00:47 |
| @fungicide:matrix.org | indeed. thanks! | 00:47 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 828429: Make a global component registry https://review.opendev.org/c/zuul/zuul/+/828429 | 00:54 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 818300: Add support for adding and removing labels in gitlab https://review.opendev.org/c/zuul/zuul/+/818300 | 01:38 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Niklas Borg: [zuul/zuul] 824748: Required gitlab labels is subset of set labels https://review.opendev.org/c/zuul/zuul/+/824748 | 01:38 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 829853: encrypt-file: always import expiring keys https://review.opendev.org/c/zuul/zuul-jobs/+/829853 | 06:30 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 829853: encrypt-file: always import expiring keys https://review.opendev.org/c/zuul/zuul-jobs/+/829853 | 06:49 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 829853: encrypt-file: always import expiring keys https://review.opendev.org/c/zuul/zuul-jobs/+/829853 | 06:58 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 829853: encrypt-file: always import expiring keys https://review.opendev.org/c/zuul/zuul-jobs/+/829853 | 07:31 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 829853: encrypt-file: always import expiring keys https://review.opendev.org/c/zuul/zuul-jobs/+/829853 | 07:45 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 829853: encrypt-file: always import expiring keys https://review.opendev.org/c/zuul/zuul-jobs/+/829853 | 08:56 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 829310: Release tenant read lock early on pending reconfig https://review.opendev.org/c/zuul/zuul/+/829310 | 09:15 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 829867: Report gross/total tenant resource usage stats https://review.opendev.org/c/zuul/zuul/+/829867 | 09:31 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 829009: Remove SQL reporter deprecation from configloader https://review.opendev.org/c/zuul/zuul/+/829009 | 11:50 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Benjamin Schanzel: [zuul/nodepool] 825129: Fix tenant-quota for providers without label quota infos https://review.opendev.org/c/zuul/nodepool/+/825129 | 12:33 | |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of Ian Wienand: [zuul/zuul] 829842: Unit testing for ref going missing https://review.opendev.org/c/zuul/zuul/+/829842 | 17:47 | |
| @clarkb:matrix.org | corvus: ianw ^ I noticed that the situation we ran into with opendev wasn't using depends on but required-projects. I've udpated the test to try and mimic that more closely in case there are specific behaviors in required-projects that are necessary to trip over this | 17:48 |
| @clarkb:matrix.org | I haevn't run that locally as I'm not really set up for that right now so it might explode spectacularly but I think the shape is a bit closer to what we saw in prod and we can improve from there | 17:48 |
| @clarkb:matrix.org | hrm that still results in a repo_state of none. | 19:23 |
| @iwienand:matrix.org | Clark: ahh, yes that's a good insight too. i do have the local setup, i can try iterating on that a bit when i get a chance | 20:49 |
| -@gerrit:opendev.org- Ian Wienand proposed: | 21:06 | |
| - [zuul/zuul-jobs] 828818: encrypt-file : role to encrypt a file https://review.opendev.org/c/zuul/zuul-jobs/+/828818 | ||
| - [zuul/zuul-jobs] 829853: encrypt-file: always import expiring keys https://review.opendev.org/c/zuul/zuul-jobs/+/829853 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Joan Gilabert: [zuul/zuul-jobs] 829063: Add patch to list of build dependencies https://review.opendev.org/c/zuul/zuul-jobs/+/829063 | 21:36 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!