Monday, 2022-08-15

-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307601:15
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307601:26
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307601:31
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307601:35
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307602:04
-@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/zuul] 853073: zuul_console: do not use f-strings https://review.opendev.org/c/zuul/zuul/+/85307302:48
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307603:37
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307604:23
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307604:49
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307604:50
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307605:01
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307605:11
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307605:18
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307605:22
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307606:25
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307606:37
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 853144: Force set repo remote URL https://review.opendev.org/c/zuul/zuul/+/85314407:32
-@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 853151: Treat k8s mem limits as Megabytes instead of Mebibytes https://review.opendev.org/c/zuul/nodepool/+/85315109:38
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 853144: Force set repo remote URL https://review.opendev.org/c/zuul/zuul/+/85314409:51
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307610:15
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 853144: Force set repo remote URL https://review.opendev.org/c/zuul/zuul/+/85314410:25
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 853144: Force set repo remote URL https://review.opendev.org/c/zuul/zuul/+/85314410:26
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307610:52
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307611:07
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: [wip] zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307611:21
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853076: zuul-console: run python 2.7 regression test https://review.opendev.org/c/zuul/zuul/+/85307611:37
-@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 853168: Add config option to limit ephemeral storage on K8s Pod labels https://review.opendev.org/c/zuul/nodepool/+/85316812:51
@jpew:matrix.orgOk, just because I'm really paranoid and want to make sure before I try this and potentially mess things up: If I have a project with branches named "master" and "foo" and both have this configuration:16:18
```
- job:
name: bar
- project:
gate:
jobs:
- bar:
branches:
- master
```
That variant of `bar` defined on the `foo` branch would never be selected as the variant to run on the `master` branch because it is an anonymous job that only gets "created" when processing a change for a given branch. And this is different from
```
- job:
name bar
branches:
- master
- project:
gate:
jobs:
- bar
```
which doesn't work for me because it seems to pick at random which variant of bar because those jobs are always defined (not anonymous)
@jim:acmegating.comjpew: if you have multiple matching variants, zuul doesn't pick one at random, it combines them as described in https://zuul-ci.org/docs/zuul/latest/config/job.html#job16:59
@jpew:matrix.orgcorvus: Sure, sorry. "random" is imprecise; there is a defined method to how it combines the jobs, it's just not what I would have expected. Apologies. `job.branches` is very hard for a mere mortal like me to wrap my head around17:03
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 853190: Ensure-java: update apt cache https://review.opendev.org/c/zuul/zuul-jobs/+/85319017:54
@clarkb:matrix.orgcorvus: ^ small thing on that17:56
@jim:acmegating.comClark: i thought redhat had a different package name17:57
@clarkb:matrix.orgoh hrm that may be17:58
@jim:acmegating.comwe also haven't ever tested that on anything other than debian17:58
@clarkb:matrix.orgya `java-17-openjdk` seems to be the format for fedora and friends17:58
@jim:acmegating.com(i mean, it rightly *should* support more than one platform, i'm just trying to minimally fix breakage)17:59
@clarkb:matrix.orgYa, I assumed that since it wasn't using the apt module to install it was a sufficiently portable package name. But I agree that doesn't seem to be the case. I'll update my review17:59
@jim:acmegating.comcool thx17:59
@clarkb:matrix.orgif any other reviewer has a moment for 853190 that will help fix issues with Gerrit's CI system18:41
@clarkb:matrix.orgianw: I left a thought on https://review.opendev.org/c/zuul/zuul/+/853076 let me know what you think about that18:50
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 853190: Ensure-java: update apt cache https://review.opendev.org/c/zuul/zuul-jobs/+/85319020:19
-@gerrit:opendev.org- Joshua Watt proposed: [zuul/zuul] 853197: Add branch-filter for zuul jobs https://review.opendev.org/c/zuul/zuul/+/85319720:28
@jpew:matrix.org^^ corvus This is kinda what I want. I know it needs tests and documentation, but does it seem sane?20:31
@jim:acmegating.comjpew: i don't think we should add that.  zuul has a method of controlling this, and it is is complex, but also very powerful.  i'm not convinced you can't achieve what you want using what is on offer, and i'm reluctant to make it more complex.21:18
@jpew:matrix.orgcorvus: That's fine; I just can't figure out how to actually do it :/21:19
@jim:acmegating.comjpew: maybe take those suggestions from last week and your idea above from earlier and try it out on some test projects or a test zuul instance to make sure you've got the hang of it?21:22
@jpew:matrix.orgcorvus: Ya, I'll do that. I was hoping someone would be able to short-cut it with "Oh ya, you can do X with Y" :)21:27
@iwienand:matrix.orgClark: yeah, you are right.  those versions don't install for python 2.7 ...21:49
@jim:acmegating.comjpew: i hear you.  i don't feel comfortable signing off on "yes do that on your production instance" in a non-professional capacity without knowing all the details, but i do think you're on the right track and it's worth experimentation/confirming.21:51
@clarkb:matrix.org> <@iwienand:matrix.org> Clark: yeah, you are right.  those versions don't install for python 2.7 ...21:59
Hrm so I guess the only way to properly test what I suggest is via ansible itself with its remote code copying system? That is probably overkill
@iwienand:matrix.orgClark: i think what i really want to do is somehow start a 2.7 container that is ansible-managable and run against it from the extant stream jobs.  figure out how to get it into the inventory and run against it.  that is annoying because it is completely orthogonal to my first attempt21:59
@iwienand:matrix.orgheh, yes jinx basically we're saying the same thing :)  22:00
@iwienand:matrix.orgi guess that would also allow testing 2.7, 3.5 -> 3.10 which is nice.  22:01
-@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 852859: Fix zoned executor metric when unzoned is allowed https://review.opendev.org/c/zuul/zuul/+/85285922:59
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 853208: [wip] zuul-stream : 2.7 testing https://review.opendev.org/c/zuul/zuul/+/85320823:54

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!