Thursday, 2024-05-30

@fungicide:matrix.orgwe've also now discovered that having repos owned by zuul:zuul has raised the same problem in the other direction for opendev's deployment jobs that run as root, so we're probably stuck applying the safe directories config somewhere12:22
-@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 920841: Add not and tox py312 jobs https://review.opendev.org/c/zuul/zuul-jobs/+/92084115:55
-@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed: [zuul/zuul-jobs] 920842: Add nox-py312 job https://review.opendev.org/c/zuul/zuul-jobs/+/92084215:59
-@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 920841: Add nox and tox py312 jobs https://review.opendev.org/c/zuul/zuul-jobs/+/92084118:57
@mnaser:matrix.orgAny thoughts on why a job would run _some_ promote job but not all run (even though their corresponding upload jobs ran just fine?)19:21
@mnaser:matrix.orgThis gated job https://ci.atmosphere.dev/t/atmosphere/buildset/5714170a461045298fa79898b2737d27 eventually hit promote https://ci.atmosphere.dev/t/atmosphere/buildset/2bcf4a13c4214191952777bd970f12aa but not all of them got promoted, even if I have promote jobs defined for all of them.. it seems kinda strange but I can't find a reason19:21
@clarkb:matrix.orgOne reason we've seen that in opendev is if you update the job directly that triggers the job to run in check/gate but when it gets to promote that job was updated in the current change context is gone and simply updating the job is insufficient in the post merge pipelines and it falls back to checking file matchers19:28
@clarkb:matrix.orgNot sure if that happened here as you seem to be modifying dockerfiles too but maybe check the jobs that didn't run have appropriate file matchers for those dockerfiles?19:30
@mnaser:matrix.orgClark:  ah!  to make a big "rebuild" I changed the job def'n and I was assuming that since the variables were changed, it would trigger the job19:36
@clarkb:matrix.orgYa it works for premerge stuff but not post merge aiui19:39
@mnaser:matrix.orgSo the way to go would be to edit the files themselves, got it got it19:52
@clarkb:matrix.orgmnaser: ya either that or don't use file matchers at all which implies the job should always run when the event is triggered20:12
@clarkb:matrix.orgfungi: (or anyone else really) any idea why ansible's six moves module wouldn't be found during python3.12 testing but seems to be fine during python3.11 testing? https://zuul.opendev.org/t/zuul/build/302e83cfc07441a4a0c7c6e7355cc425/log/job-output.txt#1189-120520:19
@clarkb:matrix.orgits almost like when ansible installs itself under python3.12 it doesn't include that code20:19
@clarkb:matrix.orghttps://github.com/ansible/ansible/blob/v2.15.12/lib/ansible/module_utils/six/__init__.py the code seems to still be there20:21
@clarkb:matrix.orgoh wait its installing ansible 2.8?20:21
@clarkb:matrix.orgok that gives me something to work with20:22
@fungicide:matrix.orgmaybe pip is backtracking to the most recent package that didn't declare supported python versions, because it can't find one for 3.12?20:22
-@gerrit:opendev.org- Clark Boylan proposed:20:26
- [zuul/zuul-jobs] 920841: Add not and tox py312 jobs https://review.opendev.org/c/zuul/zuul-jobs/+/920841
- [zuul/zuul-jobs] 920857: Update ansible versions used in unittesting https://review.opendev.org/c/zuul/zuul-jobs/+/920857
@clarkb:matrix.orgfungi:  no we were pinning it to be conservative for support in zuul-jobs20:26
@clarkb:matrix.orgcorvus: ^ I wonder if we need a small checklist for zuul reelases now that remind us to do updates like this20:26
@clarkb:matrix.orgarg I forgot I updated the commit mesasge via gerrit and needed to repull so I just pushed a revert of the commit message20:35
@clarkb:matrix.orgit seems like we may be to a point where supporting python3.12 and python2.7/python3.8 is not a straightforward thing any longer20:35
@clarkb:matrix.orgI'm going to go ahead and update the change to drop testing for those platforms and we can discuss in review if that is a sane change to make at this point in time20:36
-@gerrit:opendev.org- Clark Boylan proposed:20:39
- [zuul/zuul-jobs] 920857: Update ansible versions used in unittesting https://review.opendev.org/c/zuul/zuul-jobs/+/920857
- [zuul/zuul-jobs] 920841: Add nox and tox py312 jobs https://review.opendev.org/c/zuul/zuul-jobs/+/920841
@jim:acmegating.com> <@clarkb:matrix.org> corvus: ^ I wonder if we need a small checklist for zuul reelases now that remind us to do updates like this20:40
or at least the ansible update changes
@clarkb:matrix.org++20:40

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