Monday, 2022-08-29

@iwienand:matrix.orgoh nice, https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=101574000:45
@iwienand:matrix.orgthat merged the fix, so we should be able to revert the pulling of the unstable glibc into the container and just use glibc_2.31-13+deb11u4.dsc00:46
-@gerrit:opendev.org- Ian Wienand proposed:01:08
- [zuul/zuul-jobs] 854335: ansible-lint: pin to < 6.5 https://review.opendev.org/c/zuul/zuul-jobs/+/854335
- [zuul/zuul-jobs] 854933: linter: Use capitals for names https://review.opendev.org/c/zuul/zuul-jobs/+/854933
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 854939: Revert "Use debian bookworm version of libc-bin" https://review.opendev.org/c/zuul/zuul/+/85493903:57
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 854505: Add Ansible version to a job's Zuul vars https://review.opendev.org/c/zuul/zuul/+/85450506:08
-@gerrit:opendev.org- Marvin Becker proposed: [zuul/nodepool] 853993: Add hold command to disable nodes https://review.opendev.org/c/zuul/nodepool/+/85399312:48
@mbecker12:matrix.orgvlotorev: ^ I added some documentation now12:49
-@gerrit:opendev.org- Tobias Henkel proposed: [zuul/nodepool] 855036: Enable ENA support on uploaded AWS images https://review.opendev.org/c/zuul/nodepool/+/85503614:07
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 855048: Add ENA support option on uploaded AWS images https://review.opendev.org/c/zuul/nodepool/+/85504816:20
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 855048: Add ENA support option on uploaded AWS images https://review.opendev.org/c/zuul/nodepool/+/85504816:21
-@gerrit:opendev.org- Tobias Henkel proposed: [zuul/nodepool] 855049: Allow specification of architecture in aws diskimages https://review.opendev.org/c/zuul/nodepool/+/85504916:31
@tobias.henkel:matrix.orgcorvus: nodepool rejects specification of the architecture of an aws diskimage, fix and documentation ^16:31
@tobias.henkel:matrix.orgoh, your change drive-by fixes that16:33
@tobias.henkel:matrix.orgI'll abandon mine then16:33
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com:17:10
- [zuul/nodepool] 852466: Allow specifying diskimage metadata/tags https://review.opendev.org/c/zuul/nodepool/+/852466
- [zuul/nodepool] 852670: Add support for dynamic tags https://review.opendev.org/c/zuul/nodepool/+/852670
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com:21:16
- [zuul/nodepool] 853372: Convert GCE to state machine driver and remove simple https://review.opendev.org/c/zuul/nodepool/+/853372
- [zuul/nodepool] 854632: Use per-instance LRU caches on statemachine adapters https://review.opendev.org/c/zuul/nodepool/+/854632
- [zuul/nodepool] 854438: Add missing cloud/region/az/host_id info to nodes https://review.opendev.org/c/zuul/nodepool/+/854438
@clarkb:matrix.orgZuulians https://review.opendev.org/c/zuul/zuul-jobs/+/854335 is a straightforward test fixup for zuul-jobs21:18
@clarkb:matrix.orgianw: I'm looking at https://zuul.opendev.org/t/zuul/build/fdcb285423fe4b96939cf58870fe97d9/console#2/0/18/ubuntu-jammy and comparing that to the site preview for https://review.opendev.org/c/zuul/zuul/+/854555/ and I'm not sure I'm seeing the difference? it seems the package task there has a proper set of output?21:31
-@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/zuul-jobs] 854335: ansible-lint: pin to < 6.5 https://review.opendev.org/c/zuul/zuul-jobs/+/85433521:44
@iwienand:matrix.orgClark: hrm, interesting, i wonder if it's more specifically packages only on rpm distros, i was looking at fedora21:48
@jim:acmegating.comyes it is21:48
@iwienand:matrix.orghttps://zuul.opendev.org/t/zuul/build/1598cb6bf7cc41629a6c10d01d03f582/console#1/1/56/launcher is the failure21:49
@clarkb:matrix.orgah cool I can compare that on the site privew21:50
@clarkb:matrix.orgok with that I can confirm this fixes things. I +2'd but didn't approve as corvus has a suggestion for making the comment more specific. I'll let you decide if you want ot push a new patchset or a followup change21:53
@jim:acmegating.comseemed like a good thing to try to avoid having to rediscover again later :)21:53
@iwienand:matrix.orgyeah agree, i missed that too21:54
@iwienand:matrix.orgcorvus: hrm, that's interesting; i filed a bug on the lining up of the columns (from the datalist change) and it was padded out correctly for me21:58
@iwienand:matrix.orghttps://github.com/patternfly/patternfly/issues/505521:58
@iwienand:matrix.org(screenshot in second) ... but i just used the preview site (instead of my local server) and i agree it's now not quite lined up21:58
@iwienand:matrix.orgi wonder what's different21:58
@iwienand:matrix.orgit lines up with a 3em pad, not a 4em pad22:00
@jim:acmegating.comianw: oh, lemme see if i can get you a screenshot then22:08
@iwienand:matrix.orgcorvus: it's ok, i see it22:08
@jim:acmegating.comokay cool -- what i saw (in ff) was a similar to your bug but less severe22:09
@iwienand:matrix.orgthe zuul logo is perhaps related22:11
@fungicide:matrix.org> <@gerrit:opendev.org> Zuul merged on behalf of Ian Wienand: [zuul/zuul-jobs] 854335: ansible-lint: pin to < 6.5  https://review.opendev.org/c/zuul/zuul-jobs/+/85433522:13
just a heads up that in opendev we ended up blocking 6.5.0 and 6.5.1 since there is a pending pull request which is supposed to fix the regression (unfortunately they released 6.5.1 without merging it first)
@clarkb:matrix.orgfungi:  yup that change pins less than 6.5.0 to exclude both of those22:16
@fungicide:matrix.org> <@clarkb:matrix.org> fungi:  yup that change pins less than 6.5.0 to exclude both of those22:22
right, my point was that in opendev we're hoping 6.5.2 fixes it, i guess the zuul-jobs change went with a different approach, which is fine, but in the past we tried to keep them synchronized according to code comments i saw
@fungicide:matrix.orgso i can either push a change to adjust the zuul-jobs pin to match what we did in opendev, or push a change to adjust opendev to match zuul-jobs, or just not worry about it22:22
@fungicide:matrix.orgi don't really have a preference, but wanted to call out the divergence in case others do22:23
@clarkb:matrix.orgConsidering they made a 6.5.1 that did not fix 6.5.0 I think the zuul-jobs appraoch of assuming 6.5.x will be broken until proven otherwise is a good one :)22:24
@fungicide:matrix.orgcool, i'll pin similarly in opendev and someone who cares about ansible-lint can propose un-pinning it when they think it's safe to do so22:24
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 855096: WIP: Tracing: implement span save/restore https://review.opendev.org/c/zuul/zuul/+/85509623:07
@iwienand:matrix.orghttps://opendev.org/zuul/zuul/src/branch/master/web/src/containers/build/BuildList.jsx#L115 is sort of related to the datalist padding issue.  seems in this case we hide the icon ... an option bug not great23:59

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