*** zxiiro has quit IRC | 00:22 | |
*** Goneri has quit IRC | 01:05 | |
cloudnull | mordred clarkb ++ | 01:15 |
---|---|---|
cloudnull | I actually had that in the original spec, though it was called out for being out of scope: https://review.opendev.org/#/c/723665/1/specs/victoria/simple-container-generation.rst@309 | 01:15 |
cloudnull | that said, its something we really want to look at / do | 01:15 |
cloudnull | -cc EmilienM | 01:15 |
cloudnull | once (if?) we get further along expect lots of pestering questions :D | 01:16 |
*** zxiiro has joined #zuul | 01:45 | |
*** swest has quit IRC | 01:57 | |
*** swest has joined #zuul | 02:12 | |
*** sanjayu__ has quit IRC | 02:33 | |
*** bhavikdbavishi has joined #zuul | 02:39 | |
*** sanjayu__ has joined #zuul | 02:46 | |
*** bhavikdbavishi1 has joined #zuul | 02:54 | |
logan- | o/ earlier today AJaeger noticed in #opendev that "Limestone Networks CI" had commented on https://review.opendev.org/#/c/717375/ ...it looks like it periodically comments (a couple times a month) on opendev zuul-jobs changes regarding patches failing to merge. | 02:55 |
logan- | https://review.opendev.org/#/q/reviewer:%22Limestone+Networks+CI+%253Copenstack-ci%2540limestonenetworks.com%253E%22 | 02:55 |
logan- | the relevant main.yaml snippet and logs from zuul-scheduler are here https://pasted.tech/pastes/50eca681f6de7b4a46ed114ba083040c9506dfbd.raw | 02:55 |
*** sanjayu__ has quit IRC | 02:55 | |
*** bhavikdbavishi has quit IRC | 02:56 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 02:56 | |
logan- | is this expected behavior in zuul installations that only load jobs from zuul-jobs? | 02:56 |
*** bhavikdbavishi has quit IRC | 03:17 | |
*** bhavikdbavishi has joined #zuul | 04:06 | |
*** evrardjp has quit IRC | 04:35 | |
*** evrardjp has joined #zuul | 04:36 | |
openstackgerrit | Merged zuul/zuul-jobs master: Only run tests for ensure-bazel when it is updated https://review.opendev.org/724933 | 05:48 |
*** bhavikdbavishi has quit IRC | 05:55 | |
*** bhavikdbavishi has joined #zuul | 07:21 | |
*** sgw has quit IRC | 07:22 | |
avass | logan: interesting, and no I don't think that's expected behaviour | 08:17 |
*** bhavikdbavishi has quit IRC | 08:28 | |
*** bhavikdbavishi has joined #zuul | 08:32 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ansible-lint: use matchplay instead of matchtask https://review.opendev.org/724910 | 08:36 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ansible-lint: use matchplay instead of matchtask https://review.opendev.org/724910 | 08:43 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ansible-lint: use matchplay instead of matchtask https://review.opendev.org/724910 | 08:48 |
*** tosky has joined #zuul | 08:48 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ansible-lint: use matchplay instead of matchtask https://review.opendev.org/724910 | 08:54 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ansible-lint: use matchplay instead of matchtask https://review.opendev.org/724910 | 08:55 |
avass | AJaeger: you were right, we were missing quite a lot of cases where loop_var wasn't set, unless that ^ breaks anything it should be ready | 08:59 |
avass | There are still cases where it looks like the role itself is designed to be included in a loop though (since it uses {{ item }} in tasks/main.yaml), I don't like that so I'll see what I can do about that. | 09:00 |
avass | like: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/build-container-image/tasks/main.yaml#L10 | 09:02 |
avass | that looks broken | 09:02 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: ansible-lint: use matchplay instead of matchtask https://review.opendev.org/724910 | 09:17 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 09:30 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 09:47 |
avass | This got increasingly complicated :( | 09:47 |
*** guillaumec has joined #zuul | 09:53 | |
*** iurygregory has quit IRC | 09:56 | |
*** bhavikdbavishi has quit IRC | 10:42 | |
*** bhavikdbavishi has joined #zuul | 10:55 | |
*** bhavikdbavishi has quit IRC | 11:17 | |
*** bhavikdbavishi has joined #zuul | 11:49 | |
*** bhavikdbavishi has quit IRC | 11:53 | |
*** bhavikdbavishi has joined #zuul | 12:00 | |
*** bhavikdbavishi1 has joined #zuul | 12:03 | |
*** bhavikdbavishi has quit IRC | 12:05 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 12:05 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 12:06 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 12:14 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 12:19 |
mordred | avass: you found a lovely rabbit hole to journey down didn't you/ | 12:27 |
mordred | corvus: I'm going to need your help again on zuul-registry | 12:30 |
mordred | corvus: two differen things - the podman job is now _consistently_ failing with "Error uploading layer" | 12:31 |
AJaeger | avass: what fun! One advise: Better make small changes and merge tests at the end. | 12:31 |
mordred | corvus: and the multiarch test now has: | 12:31 |
mordred | #4 ERROR: failed commit on ref "manifest-sha256:ea0c56e835247500a01f122c7b622bb237b51c82f8fa5fec3f9a4d35f6a6419e": unexpected commit digest sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855, expected sha256:ea0c56e835247500a01f122c7b622bb237b51c82f8fa5fec3f9a4d35f6a6419e: failed precondition | 12:32 |
mordred | https://zuul.opendev.org/t/zuul/build/c7c8206a66d54363835abb074d17b11b | 12:32 |
mordred | corvus: which seems non-awesome | 12:32 |
AJaeger | avass: even a testsuite - wow! | 12:37 |
avass | mordred: yeah :) | 12:38 |
avass | AJaeger: yeah, I thought it might be good to be able to extend that easily | 12:39 |
AJaeger | logan-, corvus, I think we had the same situation some days ago where our tenant vexxhost was reporting on changes on an repo that was just included. | 12:42 |
AJaeger | Example: https://review.opendev.org/#/c/716449/ - see the two Zuul reports one after each other | 12:46 |
avass | AJaeger: I'm adding the tests while developing to be able to iterate faster ;) | 12:51 |
AJaeger | ;) | 12:53 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 13:01 |
avass | that's annoying, there's no way to do -wholename in the ansible find module | 13:20 |
*** bhavikdbavishi has quit IRC | 13:34 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 13:35 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 13:40 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 13:48 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: use zj_image instead of image as loopvar https://review.opendev.org/725012 | 13:48 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: use zj_log_file instead of item as loop_var https://review.opendev.org/725013 | 13:48 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Check blocks recursively for loops https://review.opendev.org/724967 | 13:53 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Update ansible-lint-rules testsuite to only test with the relevant rule https://review.opendev.org/725014 | 13:58 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Update ansible-lint-rules testsuite to only test with the relevant rule https://review.opendev.org/725014 | 14:07 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Update ansible-lint-rules testsuite to only test with the relevant rule https://review.opendev.org/725014 | 14:13 |
corvus | mordred: that's really weird; i don't see it ever "GET"ting that sha here; only heading it: https://zuul.opendev.org/t/zuul/build/c7c8206a66d54363835abb074d17b11b/log/builder/docker/buildset_registry.txt | 14:17 |
avass | oooh, nice looks like that works | 14:17 |
corvus | mordred: might be worth holding a node for that job | 14:23 |
mordred | corvus: which job? or both? | 14:23 |
corvus | mordred: oh the multiarch; i didn't look at podman; i'll do that now | 14:56 |
corvus | mordred: wow, yes, also that one | 14:58 |
*** zbr has quit IRC | 15:05 | |
*** zbr has joined #zuul | 15:05 | |
corvus | mordred: for the podman thing, it looks a lot like two different upload sessions are uploading the same layer sha; but even if that's the case, i don't understand why it would fail at that particular line. that isn't a collision on the target sha, that's trying to delete a temporary upload segment which should have been used only a few lines above. | 15:12 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: dhall-diff: add new job https://review.opendev.org/718694 | 15:18 |
mordred | corvus: ok - I've put in autholds for both jobs and have rechecked | 15:38 |
*** toabctl has quit IRC | 15:42 | |
*** toabctl has joined #zuul | 15:43 | |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: Add regex for matching ansible-lint output https://review.opendev.org/725027 | 16:04 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: Add regex for matching ansible-lint output https://review.opendev.org/725027 | 16:06 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: DNM: Test ansible-lint commenting https://review.opendev.org/725028 | 16:06 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: Add regex for matching ansible-lint output https://review.opendev.org/725027 | 16:12 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: DNM: Test ansible-lint commenting https://review.opendev.org/725028 | 16:12 |
zbr | avass: parsing ansible-lint output is a very bad idea!, instead force it to use parsable output (aka pep8 compliant) | 16:13 |
zbr | otherwise, your logic will fail | 16:13 |
zbr | the verbose output has zero guarantees about what it produces | 16:13 |
avass | zbr: oh, I'll take a look at that instead :) | 16:25 |
zbr | most linters are able to use the pep8 format, it become some kind of standard. | 16:25 |
zbr | while it is less verbose, it is clearly machine parsable. | 16:26 |
avass | zbr: it's already in a parseable format, I think our pep8 regex needs updating then | 16:26 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: update pep8 regex to not require column https://review.opendev.org/725030 | 16:29 |
zbr | avass: i got myself busy fixing a bug in yamllint, https://github.com/adrienverge/yamllint/pull/262 | 16:29 |
zbr | one that I found while trying to run it on zuul-jobs | 16:29 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: DNM: Test ansible-lint commenting https://review.opendev.org/725028 | 16:29 |
avass | zbr: I should have seen that to be honest, since I just copied the pep8 regex and modified it | 16:30 |
avass | zbr: thanks | 16:31 |
*** evrardjp has quit IRC | 16:35 | |
*** evrardjp has joined #zuul | 16:36 | |
mordred | corvus: the podman job didn't fail this time, but the multiarch one did | 16:41 |
mordred | corvus: and in the same way | 16:41 |
avass | oooh, looks like we're trying to avoid warnings, but that makes it so it doesn't conform to the pep8 standard | 16:53 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: tox: update pep8 regex to not require column https://review.opendev.org/725030 | 17:34 |
openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: DNM: Test ansible-lint commenting https://review.opendev.org/725028 | 17:34 |
*** timburke has quit IRC | 18:14 | |
*** iurygregory has joined #zuul | 18:49 | |
openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add git name and email for quickstart executor https://review.opendev.org/724096 | 19:02 |
*** zxiiro has quit IRC | 19:12 | |
*** Goneri has joined #zuul | 20:13 | |
openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix quickstart gating, Add git name and email to executor https://review.opendev.org/724096 | 22:19 |
*** tosky has quit IRC | 23:28 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!