| -@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/nodepool] 936015: Openshift/pods drivers: refactor, fix doc https://review.opendev.org/c/zuul/nodepool/+/936015 | 01:48 | |
| @csatari:matrix.org | Hey ho, are there any community meetings for Zuul? | 06:57 |
|---|---|---|
| @mhuin:matrix.org | thanks everyone! any chance for a minor release of Nodepool to include that fix? | 07:50 |
| -@gerrit:opendev.org- Zuul merged on behalf of Tudor-Stefan Tabacel-Manea: [zuul/zuul] 929641: node model: adapt to node_properties https://review.opendev.org/c/zuul/zuul/+/929641 | 10:43 | |
| @fungicide:matrix.org | csatari: no, nothing scheduled. we just work asynchronously through matrix, mailing lists and code review | 12:32 |
| @csatari:matrix.org | Okey, thanks for the info. Which one of these is the best place to introduce some chnages what my colleagues would like to bring upstream? | 13:09 |
| @fungicide:matrix.org | csatari: if they're already written patches against the current state of the master branch, then uploading them to gerrit for review is one way to get discussion started. if the scope is large new features, then proposing one or more specifications first is a good way to start more involved design discussions: https://review.opendev.org/q/project:zuul/zuul+file:%5Edoc/source/developer/specs/.* https://zuul-ci.org/docs/zuul/latest/developer/specs/ | 13:31 |
| @fungicide:matrix.org | mentioning your changes in here and/or in a post to zuul-discuss@lists.zuul-ci.org are also good ways to try drawing more attention to them, if they seem to go unnoticed | 13:33 |
| -@gerrit:opendev.org- Karolina Kula proposed on behalf of Joel Capitao: [zuul/zuul-jobs] 936023: DNM Switch to KVM https://review.opendev.org/c/zuul/zuul-jobs/+/936023 | 13:55 | |
| @jim:acmegating.com | mhu: a release would need a release note. if you add one, i think we could make a minor release. | 14:38 |
| @jim:acmegating.com | zuul-maint: I'd like to make this release of zuul: commit ab30d104ba01b3733bdffacff14619ea871fac06 (tag: 11.2.0) | 14:55 |
| it's what opendev is running on schedulers, and one commit later than what it's running on executors (the one commit is a revert that shouldn't affect executors) | ||
| @fungicide:matrix.org | plan lgtm, 128 changes merged since 11.1.0, skimming i don't see anything that should warrant a major version increment | 15:02 |
| @fungicide:matrix.org | we have ansible 8 deprecation in there, but it's not a breaking change | 15:04 |
| @fungicide:matrix.org | oh, maybe dropping support for github enterprise <2.21 could be considered a backward-incompatibility, but looking at the corresponding commit message github themselves haven't supported that for several years so effectively unlikely to impact anyone | 15:08 |
| @jim:acmegating.com | yeah that's my view | 15:09 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 934187: Handle early AWS spot instance reclamations https://review.opendev.org/c/zuul/nodepool/+/934187 | 15:40 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 16:20 | |
| - [zuul/zuul] 935729: Batch fake build zookeeper additions https://review.opendev.org/c/zuul/zuul/+/935729 | ||
| - [zuul/zuul] 935730: Batch fake build database additions https://review.opendev.org/c/zuul/zuul/+/935730 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 935574: Add mirror-container-images role and job https://review.opendev.org/c/zuul/zuul-jobs/+/935574 | 21:12 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 935574: Add mirror-container-images role and job https://review.opendev.org/c/zuul/zuul-jobs/+/935574 | 21:15 | |
| @clarkb:matrix.org | I've been asked to remind everyone that the open infrastructure board election process has begun. Currently we are nominating candidates. If you are interested in running yourself or would like to endorse a candidate or candidates you can do so via https://openinfra.dev/election/2025-individual-director-election | 21:37 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 935574: WIP: Add mirror-container-images role and job https://review.opendev.org/c/zuul/zuul-jobs/+/935574 | 21:48 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 935574: WIP: Add mirror-container-images role and job https://review.opendev.org/c/zuul/zuul-jobs/+/935574 | 22:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 935574: WIP: Add mirror-container-images role and job https://review.opendev.org/c/zuul/zuul-jobs/+/935574 | 23:07 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 935574: WIP: Add mirror-container-images role and job https://review.opendev.org/c/zuul/zuul-jobs/+/935574 | 23:26 | |
| @clarkb:matrix.org | corvus: re ^ is the dest_repository vs repository things I called out not a problem? | 23:46 |
| @clarkb:matrix.org | looking at the testing I'm trying to see where we set the inputs to the test to cover that | 23:47 |
| @clarkb:matrix.org | I don't think we cover that in the test because we don't supply the optional matcher just a username and password in the credentials | 23:48 |
| @jim:acmegating.com | did i not fix that, sorry | 23:53 |
| @clarkb:matrix.org | ack I was trying to sort out if that could be a source of problems but since we don't cover it it shouldn't matter for now | 23:54 |
| @jim:acmegating.com | okay, i'm looking at your comment -- the line of code is correct | 23:54 |
| @jim:acmegating.com | and i think the readme for that role is correct and says dest_repository now | 23:54 |
| @jim:acmegating.com | i may have updated that? | 23:54 |
| @jim:acmegating.com | so maybe that's the issue -- i updated the doc, not the code where you left the comment | 23:55 |
| @clarkb:matrix.org | corvus: line 64 in https://review.opendev.org/c/zuul/zuul-jobs/+/935574/17/roles/mirror-container-images/README.rst | 23:55 |
| @clarkb:matrix.org | its confusing because you ahve two dicts one of which uses dest_repository and the other just repository | 23:56 |
| @jim:acmegating.com | ooooh that's the one you meant | 23:56 |
| @clarkb:matrix.org | but I think the one using just repository is using a "standard" data structure for those credentials so probably don't want to change it and the new datastructure needs to differentiate between source and dest so that one also shouldn't change | 23:57 |
| @jim:acmegating.com | yep line 11 should change | 23:57 |
| @jim:acmegating.com | i think you may have actually highlighted the token that should change in your comment; though everytime i move my mouse gerrit highlights all of them so it's confusing | 23:57 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 935574: Add mirror-container-images role and job https://review.opendev.org/c/zuul/zuul-jobs/+/935574 | 23:58 | |
| @jim:acmegating.com | Clark: ^ okay that should be the final version | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!