| -@gerrit:opendev.org- Tristan Cacqueray proposed: | 00:03 | |
| - [zuul/zuul-operator] 845851: Update CRD apiVersion to v1 (from v1beta) https://review.opendev.org/c/zuul/zuul-operator/+/845851 | ||
| - [zuul/zuul-operator] 847830: Update to the latest version of kopf https://review.opendev.org/c/zuul/zuul-operator/+/847830 | ||
| @tristanc_:matrix.org | corvus: 845851 finally got a verify+1 \o/ | 00:35 |
|---|---|---|
| @blaisep-sureify:matrix.org | Ok, so I think what it boils down to is that the OpendDev docs are useful for using the OpenDev deployment. | 01:01 |
| (eg. namespaced repos, etc) | ||
| I'm trying to make software factory work. | ||
| @q:fricklercloud.de | "nodepool image-pause foo" throws NoNodeError for a non-existing image, but unpause just passes happily, would it make sense to change that? | 05:17 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 846806: Support deleting DIB images while builder is offline https://review.opendev.org/c/zuul/nodepool/+/846806 | 06:12 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848093: zuul-web: raise 404s when tenant isn't found https://review.opendev.org/c/zuul/zuul/+/848093 | 10:11 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848093: zuul-web: raise 404s when tenant isn't found https://review.opendev.org/c/zuul/zuul/+/848093 | 11:44 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: Add gobased CRD spec definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 12:27 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848109: WIP: add OLM documentation https://review.opendev.org/c/zuul/zuul-operator/+/848109 | 13:26 | |
| @tristanc_:matrix.org | sean-k-mooney: corvus: here are my notes about using OLM to install operators. I was able to setup cert-manager and pxc from the operatorhub catalog, unfortunately there was rbac issues on my cluster which prevent me from using them for the zuul-operator. Thus I think using vendored bundle is a good solution for now. | 13:35 |
| @tristanc_:matrix.org | Though it looked promising because the install command let you set a namespace scope | 13:37 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848132: zuul-web: return errors as JSON https://review.opendev.org/c/zuul/zuul/+/848132 | 13:50 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848093: zuul-web: raise 404s when tenant isn't found https://review.opendev.org/c/zuul/zuul/+/848093 | 13:56 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848132: zuul-web: return errors as JSON https://review.opendev.org/c/zuul/zuul/+/848132 | 13:56 | |
| @jrosser:matrix.org | Does anyone know how to run this in it's CLI mode? https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/upload-logs-base/library/zuul_swift_upload.py | 13:57 |
| @jrosser:matrix.org | I'm getting a bunch of import errors https://paste.opendev.org/show/bdXfjX8VWqtENkHYdVdD/ which could be due to https://review.opendev.org/742736 | 13:58 |
| @jim:acmegating.com | jrosser: there's a hint here: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/upload-logs-base/library/zuul_ibm_upload.py#L29 | 13:59 |
| @jrosser:matrix.org | corvus: fabulous! thanks | 14:02 |
| -@gerrit:opendev.org- Jonathan Rosser proposed: [zuul/zuul-jobs] 848135: Add cli running example for s3 and swift log uploaders https://review.opendev.org/c/zuul/zuul-jobs/+/848135 | 14:09 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: Add gobased CRD spec definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 14:09 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 848137: Fix exception in blob store cleanup https://review.opendev.org/c/zuul/zuul/+/848137 | 14:17 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 846442: Update ZooKeeper class connection methods https://review.opendev.org/c/zuul/nodepool/+/846442 | 14:46 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: WIP Add gobased CRD spec definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 15:01 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: WIP Add gobased CRD spec definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 15:10 | |
| @jpew:matrix.org | We had nodepool-builder (DIB) generate and upload a bad image to OpenStack, which broke our builds. Is there some way to test the images it creates before they are uploaded (or used by nodepool-launcher)? | 15:11 |
| @clarkb:matrix.org | jpew: there is a spec up to add image building to zuul itself which would support that similar to how zuul already supports building, use and validation of other artifacts. However, I'm not sure it is straightforward to do with nodepool today. Nodepool does keep the current and previous versions of images though with the idea being rolling back should be easy. You can pause the image build, then delete the latest build thati s broken and nodepool will start using the older one. | 15:15 |
| @jpew:matrix.org | Clark: Ya, that's what I did | 15:15 |
| @clarkb:matrix.org | You can also likely build some sorts of verification directly into your dib process. But that is probably clunky | 15:15 |
| @tristanc_:matrix.org | it might be possible to do wrap the dib command by adding some sanity check after the build, before the upload | 15:16 |
| @clarkb:matrix.org | ya or add an element that runs late enough to do verification (though maybe you can't get an element to run late enough to see the final image itself I don't recall) | 15:17 |
| @jim:acmegating.com | also https://zuul-ci.org/docs/nodepool/latest/openstack.html#attr-providers.[openstack].post-upload-hook | 15:17 |
| @clarkb:matrix.org | ah that seems most useful for this. And ya just checked dib run phases and they all run before the final image artifact is produced | 15:18 |
| @tristanc_:matrix.org | ha, that is interesting, thanks @corvus! | 15:18 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848093: zuul-web: raise 404s when tenant isn't found https://review.opendev.org/c/zuul/zuul/+/848093 | 15:19 | |
| @tristanc_:matrix.org | is there a script that demonstrate how the hook can be used? | 15:19 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: Add gobased CRD spec definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 15:25 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848132: zuul-web: return errors as JSON https://review.opendev.org/c/zuul/zuul/+/848132 | 15:42 | |
| -@gerrit:opendev.org- Jonathan Rosser proposed: [zuul/zuul-jobs] 848153: Create the log container and show the url before uploading logs https://review.opendev.org/c/zuul/zuul-jobs/+/848153 | 16:25 | |
| -@gerrit:opendev.org- Jonathan Rosser proposed wip: [zuul/zuul-jobs] 848153: Create the log container and show the url before uploading logs https://review.opendev.org/c/zuul/zuul-jobs/+/848153 | 16:39 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: Add gobased CRD spec definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 16:41 | |
| @tobias.henkel:matrix.org | tristanC: what's unclear in the docs about this hook? | 16:45 |
| @tobias.henkel:matrix.org | it just gets the documented params and can do anything | 16:45 |
| @tobias.henkel:matrix.org | if it returns with 0 nodepool continues, if it fails nodepool deletes it (and re-uploads the image) | 16:45 |
| @tristanc_:matrix.org | tobiash: i meant, an example of openstack server create + ssh test would be useful to have | 16:46 |
| @tobias.henkel:matrix.org | ah, so you wanted a real world validation example | 16:46 |
| @tobias.henkel:matrix.org | maybe we could add such an example to the docs | 16:46 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 17:48 | |
| - [zuul/nodepool] 848171: Fix ignore_provider_quota in statemachine drivers https://review.opendev.org/c/zuul/nodepool/+/848171 | ||
| - [zuul/nodepool] 848172: AWS multi quota support https://review.opendev.org/c/zuul/nodepool/+/848172 | ||
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 847790: Add support for namespaced deployment https://review.opendev.org/c/zuul/zuul-operator/+/847790 | 18:19 | |
| @tristanc_:matrix.org | corvus: could you please have a look at https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 18:20 |
| @tristanc_:matrix.org | and if that's ok with you, i'd like to propose a bump to v1beta with https://review.opendev.org/c/zuul/zuul-operator/+/785985 | 18:21 |
| @jim:acmegating.com | tristanC: yes, but probably not right away | 18:22 |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: Use golang struct for the CRD definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 18:24 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848093: zuul-web: raise 404s when tenant isn't found https://review.opendev.org/c/zuul/zuul/+/848093 | 18:45 | |
| @ecsantos:matrix.org | Hello folks, I have a question regarding pipeline.trigger configuration | 20:06 |
| @ecsantos:matrix.org | Is there any difference of behavior between these two configurations? [1 vs. 2] | 20:06 |
| [1] https://paste.opendev.org/show/btMNYFjFuoQM0JHzSUik/ | ||
| [2] https://paste.opendev.org/show/bmYQlVenNl6ilD3d2wNP/ | ||
| @ecsantos:matrix.org | That is, in [2], will it require a single comment to set both labels or else jobs won't be triggered? (logical "OR" behavior) | 20:07 |
| @ecsantos:matrix.org | s/OR/AND/ | 20:07 |
| @clarkb:matrix.org | > <@ecsantos:matrix.org> That is, in [2], will it require a single comment to set both labels or else jobs won't be triggered? (logical "OR" behavior) | 20:09 |
| https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/gerrit/gerritmodel.py#L475-L483 the code there implies they should be ANDed | ||
| @ecsantos:matrix.org | Doing some tests I noticed that if I use [1], when I vote W+1, if the change already has CR+1, the jobs trigger. If I use [2], they only trigger if a single comment votes on both labels at the same time | 20:24 |
| @clarkb:matrix.org | Yes, that reflects my understanding of the code I linked | 20:25 |
| @clarkb:matrix.org | ecsantos: each event is processed separately. If a single event matches a single event trigger under the pipeline then the pipeline is triggered | 20:29 |
| @clarkb:matrix.org | separately there are requirements you can set on a pipeline and those must always be satisfied regardless of the trigger event | 20:29 |
| @clarkb:matrix.org | For gate we require things like it being the current patchset, a +1 workflow vote and a positive verified vote. | 20:30 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 846442: Update ZooKeeper class connection methods https://review.opendev.org/c/zuul/nodepool/+/846442 | 20:40 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 847646: Fix merging with submitWholeTopic https://review.opendev.org/c/zuul/zuul/+/847646 | 21:29 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 847646: Fix merging with submitWholeTopic https://review.opendev.org/c/zuul/zuul/+/847646 | 22:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 847646: Fix merging with submitWholeTopic https://review.opendev.org/c/zuul/zuul/+/847646 | 22:33 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-operator] 848103: Use golang struct for the CRD definition https://review.opendev.org/c/zuul/zuul-operator/+/848103 | 23:59 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!