| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 877341: Fix variable name in job request queue log message https://review.opendev.org/c/zuul/zuul/+/877341 | 07:37 | |
| -@gerrit:opendev.org- Benedikt Löffler proposed on behalf of Tobias Henkel: [zuul/nodepool] 877310: Improve image build checking with many diskimages https://review.opendev.org/c/zuul/nodepool/+/877310 | 09:31 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 877178: Conditionally render ANSI console text with a black background https://review.opendev.org/c/zuul/zuul/+/877178 | 14:36 | |
| @dpawlik:matrix.org | Hello folks o/ May I ask you for review/reply for Tengu comment in review: https://review.opendev.org/c/zuul/zuul-jobs/+/876081 ? | 15:23 |
|---|---|---|
| @dpawlik:matrix.org | We would like to move the role from zuul-jobs to dedicated project in oko org and zuul-jobs will just use it. I will cover it with proper zuul job | 15:24 |
| @clarkb:matrix.org | My initial thoughts are that this role was never in zuul-jobs properly so there isn't anything preventing you from doing that (backward compat for example). That said if you do move it most zuul users will likely only consume it via nested ansible. Which is fine but something to be aware of | 15:25 |
| @clarkb:matrix.org | I think that most zuul users are also unlikely to be aware the role exists if it isn't in zuul-jobs. So it probably comes down to what the intent is here. Is this is a general microshift role or is this specifically for consumption by zuul ci systems | 15:25 |
| @clarkb:matrix.org | * I think that most zuul users are also unlikely to be aware the role exists if it isn't in zuul-jobs. So it probably comes down to what the intent is here. Is this a general microshift role or is this specifically for consumption by zuul ci systems | 15:26 |
| @dpawlik:matrix.org | it's a good question. On the beginning I was considering to move the role into zuul-jobs, due few our project will use the same role and some project are in separate tenant etc. so it was hard to maintain and also community can use the role for tests. Right now, I guess it will be used in Openstack Zuul CI, but can not say if it will be "tomorrow" or in few weeks after merging | 15:31 |
| @clarkb:matrix.org | if you host the role in https://github.com/openstack-k8s-operators/ci-framework I'm pretty sure you'll need to run it with nested ansible in openstack's zuul system. | 15:32 |
| @clarkb:matrix.org | This is doable (opendev runs nested ansible to test its deployments for example) but something to keep in mind | 15:32 |
| @dpawlik:matrix.org | ack | 15:34 |
| @jim:acmegating.com | dpawlik: you're welcome to host it wherever you want. i think hosting it in zuul-jobs would be fine and would make it easier for other microshift users to find and use the role. i think there are a few things to consider: | 15:49 |
| 1) if you're not using it in a zuul job directly, (ie you will always use it with nested ansible) where it's hosted doesn't matter very much (but the opendev community has found that cloning from github can be unreliable at the scale opendev operates). | ||
| 2) if you want to use it in a zuul job directly (not via nested ansible) in an opendev tenant, it must be hosted in opendev (in zuul-jobs). | ||
| 3) zuul itself (via nodepool) has support for openshift and uses the ensure-openshift role to perform functional testing -- if there is eventually a community build of microshift, that could be a nice replacement -- but only if it's in zuul-jobs. that's an example of some cross-community collaboration that could potentially happen in zuul-jobs. | ||
| @jim:acmegating.com | 4) if the ultimate goal is not really a role "for zuul jobs", then it's probably better not to be in zuul-jobs (getting at what Clark was saying). | 15:54 |
| -@gerrit:opendev.org- Guillaume Chauvel proposed: | 15:57 | |
| - [zuul/zuul] 875057: quick-start: run additional tutorials using var run_playbooks https://review.opendev.org/c/zuul/zuul/+/875057 | ||
| - [zuul/zuul] 875639: quick-start: Change Gerrit wait method & increase Scheduler gerrit wait time https://review.opendev.org/c/zuul/zuul/+/875639 | ||
| - [zuul/zuul] 875640: quick-start: recheck as PATCHSET_LEVEL comment https://review.opendev.org/c/zuul/zuul/+/875640 | ||
| - [zuul/zuul] 732067: tutorial: Add "gate your first patch" https://review.opendev.org/c/zuul/zuul/+/732067 | ||
| @dpawlik:matrix.org | corvus: thanks for reply. I need to go in few min, so I try to replay you later or tomorrow. Sorry! | 16:04 |
| @jim:acmegating.com | dpawlik: no worries -- just trying to provide info to help you make the decision :) | 16:13 |
| @jim:acmegating.com | fyi, we started a conversation in #_oftc_#opendev:matrix.org about the dockerhub announcement which may affect our hosting of docker.io/zuul container images. we don't have enough information to make decisions yet, but we started recording info and brainstorming some options on this etherpad: https://etherpad.opendev.org/p/MJTzrNTDMFyEUxi1ReSo | 17:44 |
| @jim:acmegating.com | (link to the announcement and other background info in that etherpad) | 17:44 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:54 | |
| - [zuul/nodepool] 877431: Use image cache when launching nodes https://review.opendev.org/c/zuul/nodepool/+/877431 | ||
| - [zuul/nodepool] 877432: Use node cache in node deleter https://review.opendev.org/c/zuul/nodepool/+/877432 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:57 | |
| - [zuul/nodepool] 877431: Use image cache when launching nodes https://review.opendev.org/c/zuul/nodepool/+/877431 | ||
| - [zuul/nodepool] 877432: Use node cache in node deleter https://review.opendev.org/c/zuul/nodepool/+/877432 | ||
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!