| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938011: sqlreporter: expand some columns, enforce length check in config https://review.opendev.org/c/zuul/zuul/+/938011 | 00:21 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938011: Enforce required length check in config https://review.opendev.org/c/zuul/zuul/+/938011 | 00:30 | |
| @mhuin:matrix.org | ^ I removed the columns changes and only kept the length check in the config loader | 00:31 |
|---|---|---|
| -@gerrit:opendev.org- Tony Breeds proposed on behalf of Joel Capitao: [zuul/zuul-jobs] 936023: DNM Switch to KVM https://review.opendev.org/c/zuul/zuul-jobs/+/936023 | 05:17 | |
| -@gerrit:opendev.org- Tony Breeds proposed on behalf of Joel Capitao: [zuul/zuul-jobs] 936023: DNM Switch to KVM https://review.opendev.org/c/zuul/zuul-jobs/+/936023 | 06:22 | |
| -@gerrit:opendev.org- Tony Breeds proposed: [zuul/nodepool] 938047: DNM: Quick test to install podman-compose on ubuntu-noble https://review.opendev.org/c/zuul/nodepool/+/938047 | 10:15 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938011: Enforce required length check in config https://review.opendev.org/c/zuul/zuul/+/938011 | 15:15 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938067: QueueItem: Assign buildset's event id for traceability https://review.opendev.org/c/zuul/zuul/+/938067 | 16:49 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 937983: Use FilterIcon for filter dropdown on QueueItem https://review.opendev.org/c/zuul/zuul/+/937983 | 17:03 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938067: QueueItem: Assign buildset's event id for traceability https://review.opendev.org/c/zuul/zuul/+/938067 | 17:17 | |
| @mnasiadka:matrix.org | Hello, trying to play around with github driver - I keep hitting GitHub.com API rate limit like every hour - am I doing something wrong? | 17:50 |
| @jim:acmegating.com | mnasiadka: one way to double check your configuration would be to step through the configurator here: https://acmegating.com/acme-enterprise-zuul/ to double check that you set up everything correctly with github and the resulting config file looks like what you have. | 17:58 |
| @mnasiadka:matrix.org | Thanks corvus - will give it a go :) | 17:59 |
| @clarkb:matrix.org | it helps tremendously to use a github application too (their rate limits are much higher iirc) | 18:01 |
| -@gerrit:opendev.org- Jan Horstmann proposed on behalf of Nils Gondermann: [zuul/nodepool] 930205: Add option to specify volume_type in openstack driver https://review.opendev.org/c/zuul/nodepool/+/930205 | 18:08 | |
| @jim:acmegating.com | yeah the configurator walks through that | 18:17 |
| @mhuin:matrix.org | it looks like an issue was introduced in Zuul's code base that prevents some unit testing from passing, specifically the py312 test. I've tested rechecking an open change that doesn't touch the python code base (https://review.opendev.org/c/zuul/zuul/+/937983) and it failed. Looks like the problematic test is `test_supercedent_github_circular_deps_closed` - any idea what's going on? | 18:50 |
| @clarkb:matrix.org | mhu: can you link to the failing test logs? | 18:52 |
| @mhuin:matrix.org | this one for example https://8a28d37c8a21ba47ad86-2bdb8be3dd1329f8a48d0e165eec17e9.ssl.cf1.rackcdn.com/938067/2/check/zuul-nox-py311/8f6dfc1/testr_results.html but pretty much every build in the check pipeline is failing since yesterday | 18:53 |
| @fungicide:matrix.org | er, link to the failing build in the zuul dashboard is what he meant | 18:53 |
| @mhuin:matrix.org | this one if you want to check felix's change that doesn't touch the python code base https://zuul.opendev.org/t/zuul/build/ea808eaae4c94d988d98fcd6491bb50b | 18:53 |
| @mhuin:matrix.org | I'll have a look at the git history | 18:55 |
| @clarkb:matrix.org | right this the behavior issue that I was describing yseterday | 18:55 |
| @clarkb:matrix.org | people constantly link the raw log url and its difficult to navigate from there to other useful locations | 18:56 |
| @clarkb:matrix.org | https://zuul.opendev.org/t/zuul/build/ea808eaae4c94d988d98fcd6491bb50b/log/job-output.txt#4563-4576 is the useful bit from the logs | 18:57 |
| @mhuin:matrix.org | well to be honest, you asked for "the failing test logs" and I pointed you to the log of the unit test that was failing. But as long as you got the info you needed, that's what matters | 18:58 |
| @clarkb:matrix.org | I guess the html files are only available in raw form baceuse they are already html | 18:59 |
| @clarkb:matrix.org | But still it is difficult to navigate from there to the change or other log file assocaited with the job | 18:59 |
| @mhuin:matrix.org | right, fair enough | 19:00 |
| @clarkb:matrix.org | which is what I was trying to express that we not make any worse with the url length problem solutions | 19:00 |
| @mhuin:matrix.org | as someone who has to investigate CI issues for others, I get that pain too | 19:01 |
| @clarkb:matrix.org | Looking at that test case we emit two different github pull requests closed evenst to the zuul schedule and then assert that zuul can process them without logging any errors | 19:02 |
| @clarkb:matrix.org | we fail because an error was logged | 19:02 |
| @fungicide:matrix.org | and people follow our own behaviors as an example, for better or worse ;) | 19:02 |
| @mhuin:matrix.org | lead by example, be the patchset, er change you want to see, yadda yadda 🙂 | 19:03 |
| @clarkb:matrix.org | https://zuul.opendev.org/t/zuul/build/ea808eaae4c94d988d98fcd6491bb50b/log/job-output.txt#4810-4816 I'm wondering if that error is expected when the pipeline doesn't exist yet | 19:03 |
| @fungicide:matrix.org | indeed! | 19:03 |
| @clarkb:matrix.org | and maybe this is a new race and the pipeline was being created prior to the assertion of no error logs but now we're not? | 19:03 |
| @clarkb:matrix.org | I think a fix is probably to emit a PR created event for each of those PRs and wait for things to settle. Then make the assertion about closing them without errors? | 19:04 |
| @clarkb:matrix.org | that should load the configs I think | 19:04 |
| @clarkb:matrix.org | or maybe there is a more explicit way to wait for zuul to load and apply all configs first | 19:05 |
| @mhuin:matrix.org | what's odd is that I started seeing this error like less than 24h ago in my own tests (that were also failing for other reasons) but nothing seems to have been merged at that time, or correct me if I'm wrong? | 19:05 |
| @clarkb:matrix.org | I don't see any merges either, but it could be a dependency that chagned timing not zuul itself | 19:05 |
| @mhuin:matrix.org | and Felix's change did pass the check pipeline yesterday. All I did was a rebase to test it out | 19:06 |
| @mhuin:matrix.org | yeah makes sense | 19:06 |
| @fungicide:matrix.org | looks like the captured logging from the service doesn't include the underlying exception that the test assert is finding? or i'm just not spotting it | 19:06 |
| @mhuin:matrix.org | I haven't checked requirements.txt but maybe something's unpinned there | 19:06 |
| @clarkb:matrix.org | fungi: I linked to it in my second log link | 19:07 |
| @fungicide:matrix.org | most things are unpinned there so that we'll spot regressions and other behavior changes in dependencies as early as possible] | 19:07 |
| @fungicide:matrix.org | clarkb: unless i missed it, you linked to the test exception, which complains about a logged error that mentions an exception i can't locate | 19:08 |
| @mhuin:matrix.org | do you do a pip freeze to compare dependencies between builds? | 19:09 |
| @fungicide:matrix.org | AssertionError: Unexpected logs found: ['ERROR:zuul.Scheduler-0:Exception loading ZKObject at /zuul/tenant/tenant-one/pipeline/post/status'] | 19:09 |
| @fungicide:matrix.org | wondering where it found the Exception loading ZKObject ... in the subunit stream | 19:09 |
| @clarkb:matrix.org | fungi: https://zuul.opendev.org/t/zuul/build/ea808eaae4c94d988d98fcd6491bb50b/log/job-output.txt#4810-4816 this link | 19:10 |
| @clarkb:matrix.org | mhu: the job logs capture every package installed but I don't think it is in a neat format like freeze emits | 19:11 |
| @fungicide:matrix.org | aha, thanks, i was searching for the object path with the /status included, so missed that | 19:11 |
| @clarkb:matrix.org | in any case I suspect that a self.waitForSettled() before the with block will cause things to sync up first | 19:11 |
| @clarkb:matrix.org | fungi: ya I don't know about the status suffix. Maybe a logging artifact or its possible I have misdiagnosed it | 19:12 |
| @clarkb:matrix.org | but the block I linked to are the only errors that vaguely match what it is complaining about | 19:13 |
| @fungicide:matrix.org | agreed | 19:15 |
| @clarkb:matrix.org | the closed events are emitted not too long after the block I linked though so I think it could just be a schedulign thing | 19:15 |
| @clarkb:matrix.org | ~200ms which seems like an eternity except python and cpu contention | 19:16 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 938077: Fix test_supercedent_github_circular_deps_closed race https://review.opendev.org/c/zuul/zuul/+/938077 | 19:16 | |
| @fungicide:matrix.org | hah | 19:17 |
| @jim:acmegating.com | hopefully commit msg answers outstanding questions | 19:17 |
| @fungicide:matrix.org | thanks! | 19:17 |
| @clarkb:matrix.org | corvus: would the schedulers are primed check in waitUntilSettled be sufficient to ensure pipelines are created? | 19:18 |
| @clarkb:matrix.org | just wondering if we can simplify that a bit more | 19:18 |
| @jim:acmegating.com | Clark: unclear. | 19:19 |
| @clarkb:matrix.org | ack +2 from me | 19:20 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 938078: Fix test_supercedent_github_circular_deps_closed race (2) https://review.opendev.org/c/zuul/zuul/+/938078 | 19:20 | |
| @jim:acmegating.com | Clark: ^ we could recheck that a bit. | 19:21 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:21 | |
| - [zuul/zuul] 937210: Add an option to skip workspace checkouts https://review.opendev.org/c/zuul/zuul/+/937210 | ||
| - [zuul/zuul] 937211: Further reduce workspace checkouts https://review.opendev.org/c/zuul/zuul/+/937211 | ||
| - [zuul/zuul] 937212: Use empty sparse checkouts in mergers https://review.opendev.org/c/zuul/zuul/+/937212 | ||
| - [zuul/zuul] 937571: Parallelize workspace repo restoration https://review.opendev.org/c/zuul/zuul/+/937571 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:22 | |
| - [zuul/zuul] 937384: Use a TreeCache for job request queues https://review.opendev.org/c/zuul/zuul/+/937384 | ||
| - [zuul/zuul] 937385: Reduce ZK lock contention in executor https://review.opendev.org/c/zuul/zuul/+/937385 | ||
| - [zuul/zuul] 937386: Disable cache event log https://review.opendev.org/c/zuul/zuul/+/937386 | ||
| - [zuul/zuul] 937387: Make executor sensor messages more useful https://review.opendev.org/c/zuul/zuul/+/937387 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 937475: Add ability to mark individual job attributes as final https://review.opendev.org/c/zuul/zuul/+/937475 | 19:23 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:23 | |
| - [zuul/zuul] 937661: Add provider list web API endpoint https://review.opendev.org/c/zuul/zuul/+/937661 | ||
| - [zuul/zuul] 937662: Add images web endpoint https://review.opendev.org/c/zuul/zuul/+/937662 | ||
| - [zuul/zuul] 937817: Add provider image web ui https://review.opendev.org/c/zuul/zuul/+/937817 | ||
| - [zuul/zuul] 937818: Add images web ui https://review.opendev.org/c/zuul/zuul/+/937818 | ||
| - [zuul/zuul] 937824: Add buttons to delete image upload https://review.opendev.org/c/zuul/zuul/+/937824 | ||
| - [zuul/zuul] 937946: Add image/upload delete lifecycle https://review.opendev.org/c/zuul/zuul/+/937946 | ||
| - [zuul/zuul] 937947: Add web API image delete endpoints https://review.opendev.org/c/zuul/zuul/+/937947 | ||
| - [zuul/zuul] 938022: Allow deleting images through web UI https://review.opendev.org/c/zuul/zuul/+/938022 | ||
| - [zuul/zuul] 938023: Add REST API method to trigger image build https://review.opendev.org/c/zuul/zuul/+/938023 | ||
| - [zuul/zuul] 938024: Add a web UI button to build an image https://review.opendev.org/c/zuul/zuul/+/938024 | ||
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938011: Enforce required length check in config https://review.opendev.org/c/zuul/zuul/+/938011 | 20:35 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 937895: Add "autohold_if_failure" var in job inventory https://review.opendev.org/c/zuul/zuul/+/937895 | 20:35 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938067: QueueItem: Assign buildset's event id for traceability https://review.opendev.org/c/zuul/zuul/+/938067 | 20:35 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:27 | |
| - [zuul/zuul] 938087: Add labels and flavors to web https://review.opendev.org/c/zuul/zuul/+/938087 | ||
| - [zuul/zuul] 938088: Add niz nodes to rest api nodes list endpoint https://review.opendev.org/c/zuul/zuul/+/938088 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 938092: Add disambiguator to k8s/openshift resources https://review.opendev.org/c/zuul/nodepool/+/938092 | 22:30 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!