| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 945665: Fix finger gateway testing https://review.opendev.org/c/zuul/zuul/+/945665 | 01:08 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 945665: Fix finger gateway testing https://review.opendev.org/c/zuul/zuul/+/945665 | 02:01 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 945679: Ignore job requests not found for lost requests https://review.opendev.org/c/zuul/zuul/+/945679 | 06:11 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 945686: Add queue name to MQTT payload https://review.opendev.org/c/zuul/zuul/+/945686 | 09:52 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 945719: metastatic: Pass tenant_name to backing node request https://review.opendev.org/c/zuul/nodepool/+/945719 | 15:21 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 945665: Fix finger gateway testing https://review.opendev.org/c/zuul/zuul/+/945665 | 17:07 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 944303: Launcher: implement resource-limits https://review.opendev.org/c/zuul/zuul/+/944303 | 17:11 | |
| @clarkb:matrix.org | For re2 I have discovered the code is developed on gerrit but wheels are built using their mirror on github with github actions. https://code-review.googlesource.com/c/re2/+/63490 this change is a proposal to add 3.13 wheel builds to the github actions | 17:17 |
|---|---|---|
| @clarkb:matrix.org | Unfortunately it has been ignored for quite some time. However, that gives us a template for how to build our own wheels and publish them should we wish to do so I guess. I don't see where pybuild11 and libabsl are coming from in the github actions but maybe the many linux container includes tools like that? | 17:18 |
| @jim:acmegating.com | fb-re2, google-re2, zuul-re2 ? :) | 17:31 |
| @fungicide:matrix.org | re-re-re-re2 | 17:32 |
| @clarkb:matrix.org | fb-re2 is no longer maintained as far I can tell. google-re2 is the one we use and is maintained (there are still new commits) just no python3.13. But ya maybe we end up with a zuul-re2... | 17:33 |
| @clarkb:matrix.org | looks like two googlers have to approve teh change using google special methods in gerrit to get that to land | 17:34 |
| @clarkb:matrix.org | I like that gerrit made submit requirements easily discoverable on the change pages | 17:34 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 19:49 | |
| - [zuul/zuul] 945743: Remove tenant references from project parsing https://review.opendev.org/c/zuul/zuul/+/945743 | ||
| - [zuul/zuul] 945744: Stop copying ProjectConfig objects when adding to layout https://review.opendev.org/c/zuul/zuul/+/945744 | ||
| - [zuul/zuul] 945745: Check for nodeset label perms in layout https://review.opendev.org/c/zuul/zuul/+/945745 | ||
| - [zuul/zuul] 945746: Check for job timeouts in layout https://review.opendev.org/c/zuul/zuul/+/945746 | ||
| - [zuul/zuul] 945747: Resolve required-projects in layout https://review.opendev.org/c/zuul/zuul/+/945747 | ||
| - [zuul/zuul] 945748: Resolve include-vars projects in layout https://review.opendev.org/c/zuul/zuul/+/945748 | ||
| - [zuul/zuul] 945749: Check allowed-projects in layout https://review.opendev.org/c/zuul/zuul/+/945749 | ||
| - [zuul/zuul] 945750: Move ignore_allowed_projects to job freeze https://review.opendev.org/c/zuul/zuul/+/945750 | ||
| - [zuul/zuul] 945751: Move post_review setting to job freezing https://review.opendev.org/c/zuul/zuul/+/945751 | ||
| - [zuul/zuul] 945752: Move base job check to layout https://review.opendev.org/c/zuul/zuul/+/945752 | ||
| - [zuul/zuul] 945753: Resolve role references when freezing https://review.opendev.org/c/zuul/zuul/+/945753 | ||
| - [zuul/zuul] 945754: Set playbook trusted flag when freezing https://review.opendev.org/c/zuul/zuul/+/945754 | ||
| - [zuul/zuul] 945755: Determine job branch matchers when freezing https://review.opendev.org/c/zuul/zuul/+/945755 | ||
| - [zuul/zuul] 945756: Remove trusted flag from source context https://review.opendev.org/c/zuul/zuul/+/945756 | ||
| - [zuul/zuul] 945757: Move some pipeline attributes to manager https://review.opendev.org/c/zuul/zuul/+/945757 | ||
| - [zuul/zuul] 945758: Move some manager attributes to pipeline https://review.opendev.org/c/zuul/zuul/+/945758 | ||
| - [zuul/zuul] 945759: Move queue-related methods to PipelineState https://review.opendev.org/c/zuul/zuul/+/945759 | ||
| - [zuul/zuul] 945760: Remove the manager attribute from pipelines https://review.opendev.org/c/zuul/zuul/+/945760 | ||
| - [zuul/zuul] 945761: Remove layout.pipelines https://review.opendev.org/c/zuul/zuul/+/945761 | ||
| - [zuul/zuul] 945762: Remove tenant attribute from pipeline https://review.opendev.org/c/zuul/zuul/+/945762 | ||
| - [zuul/zuul] 945763: Move formatStatusJSON to manager https://review.opendev.org/c/zuul/zuul/+/945763 | ||
| - [zuul/zuul] 945764: Move allowed triggers/reporter checks to layout https://review.opendev.org/c/zuul/zuul/+/945764 | ||
| - [zuul/zuul] 945765: Remove tenant from configloader parse context https://review.opendev.org/c/zuul/zuul/+/945765 | ||
| - [zuul/zuul] 945766: Refactor unprotected branch tests https://review.opendev.org/c/zuul/zuul/+/945766 | ||
| - [zuul/zuul] 945767: Add error list to error accumulator https://review.opendev.org/c/zuul/zuul/+/945767 | ||
| - [zuul/zuul] 945768: Store object errors separately https://review.opendev.org/c/zuul/zuul/+/945768 | ||
| - [zuul/zuul] 945769: Add ConfigObjectCache https://review.opendev.org/c/zuul/zuul/+/945769 | ||
| - [zuul/zuul] 945770: Reuse configuration objects https://review.opendev.org/c/zuul/zuul/+/945770 | ||
| @clarkb:matrix.org | I just updated the logging bot and want to make sure it is working. Test | 21:27 |
| @fungicide:matrix.org | lgtm, thanks! | 21:29 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 945780: Fix flaky test_cache_use https://review.opendev.org/c/zuul/zuul/+/945780 | 21:56 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:07 | |
| - [zuul/zuul] 945767: Add error list to error accumulator https://review.opendev.org/c/zuul/zuul/+/945767 | ||
| - [zuul/zuul] 945768: Store object errors separately https://review.opendev.org/c/zuul/zuul/+/945768 | ||
| - [zuul/zuul] 945769: Add ConfigObjectCache https://review.opendev.org/c/zuul/zuul/+/945769 | ||
| - [zuul/zuul] 945770: Reuse configuration objects https://review.opendev.org/c/zuul/zuul/+/945770 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 945784: Fix race in queue tests https://review.opendev.org/c/zuul/zuul/+/945784 | 22:24 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 945784: Fix race in queue tests https://review.opendev.org/c/zuul/zuul/+/945784 | 22:29 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:37 | |
| - [zuul/zuul] 945761: Remove layout.pipelines https://review.opendev.org/c/zuul/zuul/+/945761 | ||
| - [zuul/zuul] 945762: Remove tenant attribute from pipeline https://review.opendev.org/c/zuul/zuul/+/945762 | ||
| - [zuul/zuul] 945763: Move formatStatusJSON to manager https://review.opendev.org/c/zuul/zuul/+/945763 | ||
| - [zuul/zuul] 945764: Move allowed triggers/reporter checks to layout https://review.opendev.org/c/zuul/zuul/+/945764 | ||
| - [zuul/zuul] 945765: Remove tenant from configloader parse context https://review.opendev.org/c/zuul/zuul/+/945765 | ||
| - [zuul/zuul] 945766: Refactor unprotected branch tests https://review.opendev.org/c/zuul/zuul/+/945766 | ||
| - [zuul/zuul] 945767: Add error list to error accumulator https://review.opendev.org/c/zuul/zuul/+/945767 | ||
| - [zuul/zuul] 945768: Store object errors separately https://review.opendev.org/c/zuul/zuul/+/945768 | ||
| - [zuul/zuul] 945769: Add ConfigObjectCache https://review.opendev.org/c/zuul/zuul/+/945769 | ||
| - [zuul/zuul] 945770: Reuse configuration objects https://review.opendev.org/c/zuul/zuul/+/945770 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 942158: Add a test for Gerrit merge event order https://review.opendev.org/c/zuul/zuul/+/942158 | 22:51 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:52 | |
| - [zuul/zuul] 945769: Add ConfigObjectCache https://review.opendev.org/c/zuul/zuul/+/945769 | ||
| - [zuul/zuul] 945770: Reuse configuration objects https://review.opendev.org/c/zuul/zuul/+/945770 | ||
| @jim:acmegating.com | `Unregistering due to low memory 4.3% < 5.0` is popping up a bit in the middle of that stack; that's causing timeouts. i guess the added swap isn't sufficient for that. now -- that stack increases memory and cpu use for a little bit, and then at the end it should reduce both. perhaps that is a temporary issue and the only question is how to get the middle of that stack through. or, perhaps we should go ahead and start using larger nodes via https://review.opendev.org/940824 | 23:07 |
| @jim:acmegating.com | i *think* things are far enough along we could actually start to have zuul depend on zuul-launcher. it still may not be perfect, but i think i expect it to work more than not at this point | 23:09 |
| @clarkb:matrix.org | I haven't looked at the changes yet. But if the end is expected to get under the memory limit then maybe we try to tough it out first? Also I bet swap doesn't factor into that governor so it may not be a help for that specific issue | 23:09 |
| @clarkb:matrix.org | We might be able to increase swappiness to force it to evict a little more aggressively? | 23:10 |
| @jim:acmegating.com | yeah. i'd probably be solidly in the camp of toughing it out -- if we weren't already looking at increasing memory before that stack | 23:10 |
| @clarkb:matrix.org | I think the swap role does set a swappiness value we could modify and experiment with if we want to go that route | 23:10 |
| @jim:acmegating.com | (basically, even before this stack we were saying "we could use more ram" and "we should dogfood niz more"; so maybe hit 3 birds with one stone) | 23:11 |
| @clarkb:matrix.org | Ya | 23:12 |
| @clarkb:matrix.org | I'm just brainstorming options not really implying a preference for anything | 23:12 |
| @jim:acmegating.com | i feel like tweaking swappiness might be too much work for too little gain? especially when we are dealing with so many cloud variables (like different cpu values) | 23:13 |
| @clarkb:matrix.org | It may depend a lot on the overlapping timing of specific test cases | 23:13 |
| @jim:acmegating.com | let's see what happens to nodeset request 4aeac8dff75d4e25b2166a684a2cd819 before we commit :) | 23:16 |
| @jim:acmegating.com | (if we go the tough-it-out route, we can temporarily lower the concurrency and have 2 hour test runs for a bit) | 23:18 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 945780: Fix flaky test_cache_use https://review.opendev.org/c/zuul/zuul/+/945780 | 23:22 | |
| @jim:acmegating.com | looks like even the end of that stack is having memory pressures. | 23:42 |
| @jim:acmegating.com | ftr, that nodeset request did fail, but due to what looks like a cloud error, and it was retried on a different cloud, so, yay | 23:55 |
| @jim:acmegating.com | * ftr, that nodeset request did fail to launch a server on its first attempt, but due to what looks like a cloud error, and it was retried on a different cloud, so, yay | 23:55 |
| @jim:acmegating.com | * ftr, that nodeset request did fail to launch a server on its first attempt, due to what looks like a cloud error, and it was retried on a different cloud, so, yay | 23:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!