| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [opendev/zone-zuul-ci.org] 880910: Remove old nameservers https://review.opendev.org/c/opendev/zone-zuul-ci.org/+/880910 | 01:53 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 880874: WIP: Add ZK load testing script https://review.opendev.org/c/zuul/zuul/+/880874 | 16:56 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 874718: Add GitHub pipeline trigger requirements https://review.opendev.org/c/zuul/zuul/+/874718 | 18:46 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 875790: Add Gerrit pipeline trigger requirements https://review.opendev.org/c/zuul/zuul/+/875790 | 18:50 | |
| @jim:acmegating.com | Clarkianw https://zuul.opendev.org/t/zuul/build/e56e9640e46241a88e8ed5a5446ea7dc failed the upload image job | 21:57 |
|---|---|---|
| @jim:acmegating.com | looks like it's due to the quay create | 21:58 |
| @jim:acmegating.com | (the upload error is a red herring; it failed after the pre-playbook failed -- that's the real error) | 21:58 |
| @clarkb:matrix.org | ya that confused me until I opened the console | 21:58 |
| @clarkb:matrix.org | the only thing you should need is an api token with create perms. Do you know if the api token in the secret for those repos has create perms? | 21:59 |
| @clarkb:matrix.org | oh wait it might need the descriptin and name stuff | 21:59 |
| @clarkb:matrix.org | hrm this I didn't anticipate as Ifugred that would all be present going forward. I can push a change up that adds that and we can see if it ifxed it? | 22:00 |
| @jim:acmegating.com | sure! | 22:00 |
| @jim:acmegating.com | i'm trying to figure out the api question in the mean time | 22:00 |
| @jim:acmegating.com | this is the thing where we generate an oauth2 token and that's a one-time event and we lose the information about how it was generated, right? | 22:01 |
| @clarkb:matrix.org | yes, via the web ui under applications. I suspect you did create it with create perms since I brought it up when you were doing this iirc | 22:03 |
| @jim:acmegating.com | yeah, probably best to assume that's correct for now, and try your idea of adding more fields, and if that doesn't work, try a manual curl with it using the same token and if that doesn't work, make a new app with a new token and see if it works then. and only if that works, should we generate a new token for prod. because changing it is not easy. | 22:04 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 881892: Add container creation details https://review.opendev.org/c/zuul/zuul/+/881892 | 22:05 | |
| @clarkb:matrix.org | I think that is what is missing. Apologies for not catching that. I could also write a change to ensure-quay-rpeo to skip the creation if those are not set | 22:06 |
| @clarkb:matrix.org | I think both things may be worthwile? I'll work on the other change now too | 22:06 |
| @jim:acmegating.com | ++ | 22:06 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 881893: Skip quay repo creation if necessary info is missing https://review.opendev.org/c/zuul/zuul-jobs/+/881893 | 22:09 | |
| @clarkb:matrix.org | That should have it gracefully skip and continue on without attempting if the info is missing | 22:09 |
| @jim:acmegating.com | +3 on both | 22:16 |
| @fungicide:matrix.org | is someone working on the equivalent of 881892 for nodepool's image jobs, or shall i? | 22:19 |
| @clarkb:matrix.org | fungi: not yet but 881893 should prevent it from being a problem. I think we can see how zuul does and figure it out before we try and retrofit others | 22:20 |
| @fungicide:matrix.org | k | 22:20 |
| @clarkb:matrix.org | just to rule out other potential problems (or rule them in) first | 22:21 |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul-jobs] 881893: Skip quay repo creation if necessary info is missing https://review.opendev.org/c/zuul/zuul-jobs/+/881893 | 22:22 | |
| @clarkb:matrix.org | corvus: I think that fixed zuul as the job is paused and we pause after we get to that point? | 22:36 |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 881892: Add container creation details https://review.opendev.org/c/zuul/zuul/+/881892 | 23:49 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!