| @jim:acmegating.com | it looks like https://quay.io/repository/openshift/origin?tab=tags&tag=latest is empty which means that we can't install openshift 3 from quay.io | 00:20 |
|---|---|---|
| @jim:acmegating.com | i see two outstanding changes in the zuul-jobs repo related to running openshift 4: https://review.opendev.org/c/zuul/zuul-jobs/+/854047 and https://review.opendev.org/c/zuul/zuul-jobs/+/876081/15 | 00:21 |
| @jim:acmegating.com | neither of those appear to be close to mergeable at this point | 00:21 |
| @jim:acmegating.com | is anyone else aware of any other options we might have in order to continue functional testing of openshift? or should we drop that test and treat it like the proprietary/black-box offerings and add an openshift fake? | 00:22 |
| @clarkb:matrix.org | I think tony.breeds had insight as well as sean-k-mooney | 00:27 |
| @clarkb:matrix.org | My personal feeling on it is that if openshift wants to make it difficult to test against then we shouldn't feel obligated to test against it. | 00:28 |
| @jim:acmegating.com | yeah, i don't think the openshift fake is objectionable, just sad | 00:28 |
| @tony.breeds:matrix.org | I don't have a a lot of context or insights really. I can certainly look at what'd it'd take to get it over the line. So I guess for now that means we need to treat it like a blackbox for now and disable that testing. :/ | 01:03 |
| @tony.breeds:matrix.org | How much of OpenShift/K8s would we actually need t fake? | 01:04 |
| -@gerrit:opendev.org- Dmitriy Rabotyagov proposed: [zuul/zuul-jobs] 899230: Add role for uploading Ansible collections to Galaxy https://review.opendev.org/c/zuul/zuul-jobs/+/899230 | 08:12 | |
| @muneerefx:matrix.org | hi everyone. iam new to zuul. i need a help from you all. Iam trying fetch the uuid that is auto genarated by zuul.But i donkt know how to fetch it.Can you give a solution please | 08:33 |
| @noonedeadpunk:matrix.org | uuid of what actually? | 08:35 |
| @clarkb:matrix.org | corvus: I'm looking at `Change 360219: Delete vote now fails for an already deleted vote` more closely. I agree that there doesn't seem to be special handling of that in Zuul. We do seem to support the event type and lookup the deleter account in the event properly. I think that means it may be possible for zuul pipelines to be constructed that match those events. That said I don't think OpenDev is doing that anywhere. As you say we reset votes by applying a new vote not deleting existing votes.Long story short I expect this to be fine for OpenDev but other Gerrit users may need to accomodate this if they trigger on deleted votes in Zuul. Event then I don't think | 17:07 |
| @clarkb:matrix.org | * corvus: I'm looking at `Change 360219: Delete vote now fails for an already deleted vote` more closely. I agree that there doesn't seem to be special handling of that in Zuul. We do seem to support the event type and lookup the deleter account in the event properly. I think that means it may be possible for zuul pipelines to be constructed that match those events. That said I don't think OpenDev is doing that anywhere. As you say we reset votes by applying a new vote not deleting existing votes. | 17:08 |
| Long story short I expect this to be fine for OpenDev but other Gerrit users may need to accomodate this if they trigger on deleted votes in Zuul. Event then I don't think zuul really needs to do anything. Zuul users just need to be aware of the chagne in Gerrit. | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 17:44 | |
| - [zuul/zuul] 899213: Add tests for admin api token usage with access-rules https://review.opendev.org/c/zuul/zuul/+/899213 | ||
| - [zuul/zuul] 899291: Tidy some auth exceptions https://review.opendev.org/c/zuul/zuul/+/899291 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 899292: Use server supplied error strings https://review.opendev.org/c/zuul/zuul-client/+/899292 | 17:47 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 899308: Remove openshift functional testing https://review.opendev.org/c/zuul/nodepool/+/899308 | 20:04 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 899291: Tidy some auth exceptions https://review.opendev.org/c/zuul/zuul/+/899291 | 20:09 | |
| @jim:acmegating.com | it looks like we have pretty good fake coverage of openshift already, so i don't think there's anything else to do other than remove the func job. | 21:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!