| -@gerrit:opendev.org- Simon Westphahl proposed on behalf of Benjamin Schanzel: [zuul/nodepool] 868090: Adapt load-test to new zk structure https://review.opendev.org/c/zuul/nodepool/+/868090 | 06:23 | |
| -@gerrit:opendev.org- Simon Westphahl proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 862522: Convert openstack driver to statemachine https://review.opendev.org/c/zuul/nodepool/+/862522 | 06:23 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 869653: dnm: Debug Github event preprocessing exception https://review.opendev.org/c/zuul/zuul/+/869653 | 07:32 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 867137: Update openshift client install version https://review.opendev.org/c/zuul/zuul/+/867137 | 17:12 | |
| @clarkb:matrix.org | Turns out hey rename the stable client under the stable-4.11/ dir. This update should make it more stable for zuul image builds | 17:12 |
|---|---|---|
| @clarkb:matrix.org | I checked and kubectl and oc have the same hashes under 4.11.20 so that is still good | 17:12 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 862522: Convert openstack driver to statemachine https://review.opendev.org/c/zuul/nodepool/+/862522 | 17:13 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Benjamin Schanzel: [zuul/nodepool] 868090: Adapt load-test to new zk structure https://review.opendev.org/c/zuul/nodepool/+/868090 | 17:13 | |
| @clarkb:matrix.org | haha the nodepool tox config isn't work which is preventing the stack to switch to nox from landing | 17:20 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/nodepool] 869749: Fix nodepool's tox.ini so that nox changes can land https://review.opendev.org/c/zuul/nodepool/+/869749 | 17:22 | |
| @clarkb:matrix.org | I think ^ will address it | 17:22 |
| @jim:acmegating.com | Clark: we're totally going to get a nodepool change merged today | 17:24 |
| @clarkb:matrix.org | eventually it is bound to happen | 17:27 |
| @clarkb:matrix.org | corvus the nodepool tox.ini fix hit a retry limit on the nodepool k8s job. Do we need to squash some changes? | 18:26 |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of Ian Wienand: | 18:30 | |
| - [zuul/nodepool] 866955: nodepool-functional-k8s: switch to microk8s https://review.opendev.org/c/zuul/nodepool/+/866955 | ||
| - [zuul/nodepool] 867745: Unpin microk8s https://review.opendev.org/c/zuul/nodepool/+/867745 | ||
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of Benjamin Schanzel: [zuul/nodepool] 868090: Adapt load-test to new zk structure https://review.opendev.org/c/zuul/nodepool/+/868090 | 18:30 | |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 18:30 | |
| - [zuul/nodepool] 868438: Switch to nox https://review.opendev.org/c/zuul/nodepool/+/868438 | ||
| - [zuul/nodepool] 862522: Convert openstack driver to statemachine https://review.opendev.org/c/zuul/nodepool/+/862522 | ||
| @clarkb:matrix.org | yes squashed the base of that stack into the tox.ini fixup | 18:30 |
| -@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed wip: [zuul/zuul-website] 869756: Remove the Summit CFP banner now that it is closed https://review.opendev.org/c/zuul/zuul-website/+/869756 | 19:28 | |
| -@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed wip: [zuul/zuul-website] 869756: Remove the Summit CFP banner now that it is closed https://review.opendev.org/c/zuul/zuul-website/+/869756 | 19:43 | |
| -@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed wip: [zuul/zuul-website] 869756: Remove the Summit CFP banner now that it is closed https://review.opendev.org/c/zuul/zuul-website/+/869756 | 19:53 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: | 19:58 | |
| - [zuul/nodepool] 866955: nodepool-functional-k8s: switch to microk8s https://review.opendev.org/c/zuul/nodepool/+/866955 | ||
| - [zuul/nodepool] 867745: Unpin microk8s https://review.opendev.org/c/zuul/nodepool/+/867745 | ||
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul-jobs] 869760: use-buildset-registry: Respect docker_mirror https://review.opendev.org/c/zuul/zuul-jobs/+/869760 | 20:15 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul-jobs] 869761: use-docker-mirror: Also run task if docker_mirror is defined https://review.opendev.org/c/zuul/zuul-jobs/+/869761 | 20:16 | |
| -@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed: [zuul/zuul-website] 869756: Remove the Summit CFP banner now that it is closed https://review.opendev.org/c/zuul/zuul-website/+/869756 | 20:18 | |
| @jpew:matrix.org | I noticed that our docker builds using the buildset registry fail due to not using our docker mirror failing; I attempted to fix it https://review.opendev.org/c/zuul/zuul-jobs/+/869760 . Maybe it would be better to have the code prepend the existing `registry-mirrors` list instead? | 20:22 |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul-jobs] 869760: use-buildset-registry: Prepend buildset registry to mirrors https://review.opendev.org/c/zuul/zuul-jobs/+/869760 | 20:36 | |
| @jpew:matrix.org | I have a job that Depends-on a change to a role to test it, but Zuul is not using the change when it runs.... is there a reason why? | 20:47 |
| @jpew:matrix.org | Oddly, the job does seem to pick up and use https://review.opendev.org/c/zuul/zuul-jobs/+/869761, but it doesn't use https://review.opendev.org/c/zuul/zuul-jobs/+/869760 and it's annoying me as to why :) | 20:48 |
| @jpew:matrix.org | (both are in Depends-on of the test gerrit) | 20:48 |
| @fungicide:matrix.org | i don't have time to look closely at it, but a common reason is if the job/playbook using that role is defined in a trusted config repo, in which case it won't use speculative versions in order to avoid creating a security risk (attacker proposing a malicious change to a role which then runs automatically in a security-sensitive part of a job, perhaps exposing log upload credentials or something) | 20:52 |
| @jpew:matrix.org | fungi: Ah, that make sense | 20:53 |
| @jpew:matrix.org | That would be why then | 20:53 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 868438: Switch to nox https://review.opendev.org/c/zuul/nodepool/+/868438 | 21:21 | |
| @michael_kelly_anet:matrix.org | Question about using repos with LFS in them - I sort of hacked up a job for this myself, but it's a bit ugly - the base workspace prep jobs don't seem to support this though, afiact? | 21:37 |
| @jim:acmegating.com | i am not aware of any lfs support in zuul-jobs, no | 21:40 |
| @michael_kelly_anet:matrix.org | Any objection if I try to figure this out? | 21:42 |
| @michael_kelly_anet:matrix.org | The limitation I ran into is that the underlying repo URL (which is needed for us to fetch LFS objects) doesn't appear to be present in the job environment. Is there a reason for this? | 21:43 |
| @michael_kelly_anet:matrix.org | (and we configure the repos s.t. origin = file:///dev/null) | 21:43 |
| @michael_kelly_anet:matrix.org | Also, corvus I had some open questions on https://review.opendev.org/c/zuul/zuul-operator/+/853695/18 I has hoping to get your opinion on | 21:44 |
| @michael_kelly_anet:matrix.org | (just been distracted by a few other things) | 21:45 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 869771: Add release notes for 8.0.1 https://review.opendev.org/c/zuul/nodepool/+/869771 | 21:48 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!