| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 913914: Don't reset buildset when cycle dependency merged https://review.opendev.org/c/zuul/zuul/+/913914 | 09:03 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 912494: Use the triggering change as the zuul change https://review.opendev.org/c/zuul/zuul/+/912494 | 09:56 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 10:09 | |
| - [zuul/zuul] 912615: Attempt to preserve triggering event across re-enqueues https://review.opendev.org/c/zuul/zuul/+/912615 | ||
| - [zuul/zuul] 912817: Add a zuul.buildset_refs variable https://review.opendev.org/c/zuul/zuul/+/912817 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 913875: Correctly limit buildsets with multiple refs https://review.opendev.org/c/zuul/zuul/+/913875 | 16:20 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 912495: Use importlib instead of pkg_resources for ansible conf https://review.opendev.org/c/zuul/zuul/+/912495 | 17:11 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 913721: Convert pkg_resources usage to importlib https://review.opendev.org/c/zuul/zuul/+/913721 | 17:11 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 913722: Replace pkg_resource dep resolution with pip https://review.opendev.org/c/zuul/zuul/+/913722 | 17:14 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 905136: Add python 3.12 testing to Zuul https://review.opendev.org/c/zuul/zuul/+/905136 | 17:37 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 18:06 | |
| - [zuul/zuul] 913727: Record merger operations https://review.opendev.org/c/zuul/zuul/+/913727 | ||
| - [zuul/zuul] 913938: Store a repo state file in the log directory https://review.opendev.org/c/zuul/zuul/+/913938 | ||
| @jim:acmegating.com | Clark: https://c1a74d481d4107d30d2a-a7f43026294a53c04676a5990bff48a5.ssl.cf2.rackcdn.com/910550/3/gate/zuul-nox-py312/68decde/testr_results.html from https://zuul.opendev.org/t/zuul/build/68decde8bbef4363a2e446eefefce1a4 looks suspiciously like unexpected fallout from the 3.12 stack | 18:08 |
|---|---|---|
| @clarkb:matrix.org | yes, it is interesting that it didn't fail previously | 18:10 |
| @jim:acmegating.com | (i have no theory about it yet; and could just be a random fluke, but it's "packages" and "3.12" which smells funny) | 18:10 |
| @clarkb:matrix.org | yes it is specifically failing in the new check to see if there are additional packages to install | 18:10 |
| @clarkb:matrix.org | and that change doesn't update the package list or touch the package install code path so its odd that this managed to pass elsewhere | 18:11 |
| @clarkb:matrix.org | looks like no json was produced. In my local testing of pip it always produced json and the install key was either empty or had work to do | 18:13 |
| @clarkb:matrix.org | I wonder why this isn't producing any json output | 18:13 |
| @jim:acmegating.com | yeah i agree | 18:13 |
| @jim:acmegating.com | Clark: unrelated, in my local testing on an older venv since i haven't bothered to update yet, it's failing on extra output; i'll try to fix that and add more debug | 18:14 |
| @jim:acmegating.com | Clark: do you think `JSONDecodeError("Expecting value", s, err.value) from None` means that ret.stdout is None (ie, not empty string)? | 18:15 |
| @clarkb:matrix.org | corvus: yes that was my assumption reading the error | 18:15 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 911008: Continuously ensure the component registry is up to date https://review.opendev.org/c/zuul/nodepool/+/911008 | 18:17 | |
| @clarkb:matrix.org | corvus: as an alternative we could add setuptools to the ansible package install list and go back to the old pkg_resources check. A partial revert of the pkg_resources removals | 18:17 |
| @clarkb:matrix.org | just throwing that out there if debugging doesn't lead to anything useful | 18:18 |
| @jim:acmegating.com | Clark: https://paste.opendev.org/show/brArkuP0wfQmMAb1wLER/ | 18:21 |
| @jim:acmegating.com | Clark: that's in my local old venv after splitting stdout/stderr | 18:21 |
| @jim:acmegating.com | looks like we get that "None" even if stdout is the empty string | 18:21 |
| @clarkb:matrix.org | ah so its None valid json :) | 18:21 |
| @jim:acmegating.com | seems like a solution is to protect against that, but i still have no theory why it worked at all | 18:21 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 912791: Reset jobs behind non-mergeable cycle https://review.opendev.org/c/zuul/zuul/+/912791 | 18:21 | |
| @clarkb:matrix.org | corvus: the command adds extra packages to the end. I think this implies there are no extra packages? | 18:22 |
| @clarkb:matrix.org | so maybe the check is skip all this if extra pacakges is an empty list? | 18:22 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 910442: Add zuul_unreachable ansible host group https://review.opendev.org/c/zuul/zuul/+/910442 | 18:26 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 914145: Make ansible package check more robust https://review.opendev.org/c/zuul/zuul/+/914145 | 18:27 | |
| @jim:acmegating.com | Clark: ^ okay all of the above; i originally put in some debug log messages with the actual contents, but it can be a huge wall of text, so i say let's see if the above improves things and only add that if we see more problems | 18:27 |
| @clarkb:matrix.org | +2 | 18:29 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 910550: Emit per-branch queue stats separately https://review.opendev.org/c/zuul/zuul/+/910550 | 19:22 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 911386: gce: clear machine type cache on bad data https://review.opendev.org/c/zuul/nodepool/+/911386 | 19:35 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 19:53 | |
| - [zuul/zuul] 913199: Fix github docs for pull_request_review.state https://review.opendev.org/c/zuul/zuul/+/913199 | ||
| - [zuul/zuul] 914145: Make ansible package check more robust https://review.opendev.org/c/zuul/zuul/+/914145 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 913875: Correctly limit buildsets with multiple refs https://review.opendev.org/c/zuul/zuul/+/913875 | 20:31 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 914161: Add buildset event db table https://review.opendev.org/c/zuul/zuul/+/914161 | 21:31 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!