| -@gerrit:opendev.org- Zuul merged on behalf of Tristan Cacqueray: [zuul/zuul] 835662: Add zuul_log_id to fully qualified command Ansible module args https://review.opendev.org/c/zuul/zuul/+/835662 | 00:23 | |
| @jim:acmegating.com | zuul-maint: how does this look for a zuul release? commit c0075f51bade23b75e000435e1fffa441e00e492 (HEAD -> master, tag: 5.2.1, origin/master) | 01:32 |
|---|---|---|
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 834198: Add spec for removing restricted Ansible https://review.opendev.org/c/zuul/zuul/+/834198 | 03:05 | |
| @mnaser:matrix.org | This change looks good to me, however, I think perhaps it might be good for tobiash to have a look at it since I think they use logs inside S3? | 03:41 |
| https://review.opendev.org/c/zuul/zuul-jobs/+/834043 | ||
| -@gerrit:opendev.org- Mohammed Naser proposed: | 04:22 | |
| - [zuul/zuul-registry] 835772: tests: Add regression test for manifests https://review.opendev.org/c/zuul/zuul-registry/+/835772 | ||
| - [zuul/zuul-registry] 835773: Fix "docker manifest create" issue https://review.opendev.org/c/zuul/zuul-registry/+/835773 | ||
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-registry] 835772: tests: Add regression test for manifests https://review.opendev.org/c/zuul/zuul-registry/+/835772 | 04:35 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-registry] 835772: tests: Add regression test for manifests https://review.opendev.org/c/zuul/zuul-registry/+/835772 | 04:41 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-registry] 835773: Fix "docker manifest create" issue https://review.opendev.org/c/zuul/zuul-registry/+/835773 | 04:42 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-registry] 835773: Fix "docker manifest create" issue https://review.opendev.org/c/zuul/zuul-registry/+/835773 | 05:04 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: | 05:07 | |
| - [zuul/zuul-registry] 835772: tests: Add regression test for manifests https://review.opendev.org/c/zuul/zuul-registry/+/835772 | ||
| - [zuul/zuul-registry] 835773: Fix "docker manifest create" issue https://review.opendev.org/c/zuul/zuul-registry/+/835773 | ||
| @mnaser:matrix.org | The changes above are ready to land for `zuul-registry`, they'd be really helpful for us to land since they fix breakages not allowing us to build multiarch images using `docker manifest create`. | 05:23 |
| @q:fricklercloud.de | is there a way for a job to know which username zuul is using on the nodes? I have a job that assumes "zuul" but fails when running on some cloud-image based node | 05:44 |
| @q:fricklercloud.de | ah, I guess ansible_user in hostvars should do | 05:48 |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 834857: Fix bug in getting changed files https://review.opendev.org/c/zuul/zuul/+/834857 | 10:20 | |
| @q:fricklercloud.de | anyone care to review this githubreporter patch? https://review.opendev.org/c/zuul/zuul/+/834671 | 10:39 |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 834857: Fix bug in getting changed files https://review.opendev.org/c/zuul/zuul/+/834857 | 11:05 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 834857: Fix bug in getting changed files https://review.opendev.org/c/zuul/zuul/+/834857 | 11:29 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 834857: Fix bug in getting changed files https://review.opendev.org/c/zuul/zuul/+/834857 | 12:30 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 834857: Fix bug in getting changed files https://review.opendev.org/c/zuul/zuul/+/834857 | 13:08 | |
| @jim:acmegating.com | mnaser: re https://review.opendev.org/834043 see https://review.opendev.org/776677 instead | 13:24 |
| @mnaser:matrix.org | > <@jim:acmegating.com> mnaser: re https://review.opendev.org/834043 see https://review.opendev.org/776677 instead | 13:39 |
| perfect. i dropped my +2. I remember something like this. | ||
| @mnaser:matrix.org | Appreciate eyes on https://review.opendev.org/c/zuul/zuul-registry/+/835772 and the follow up for it :) | 13:39 |
| @jim:acmegating.com | pushed 5.2.1 | 13:41 |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 834857: Fix bug in getting changed files https://review.opendev.org/c/zuul/zuul/+/834857 | 13:58 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-registry] 835773: Fix "docker manifest create" issue https://review.opendev.org/c/zuul/zuul-registry/+/835773 | 14:30 | |
| @clarkb:matrix.org | corvus: I don't see the tag? | 16:16 |
| @jim:acmegating.com | Clark: yes indeed the push failed... looking | 16:27 |
| @jim:acmegating.com | error: remote unpack failed: error Missing tree 3eb3a1eb041b611b4a4aef19876bdd6e94b60396 | 16:27 |
| fatal: Unpack error, check server log | ||
| @jim:acmegating.com | -> #opendev | 16:31 |
| @jim:acmegating.com | re-pushed 5.2.1 :) | 16:35 |
| @clarkb:matrix.org | Zuul's release jobs all indicate success too | 17:49 |
| @q:fricklercloud.de | do other deployments also only use a single flavor like opendev? I'm thinking of adding smaller node types to be more resource friendly for simple jobs like linters or docs builds | 18:01 |
| @clarkb:matrix.org | q: I think you should be able to do that. OpenDev hasn't really due to quotas having fixed numbers of instances. But if your providers aren't set up that way it should work | 18:03 |
| @mnaser:matrix.org | I don't see much traces in references to Ansible collections, so I assume Zuul doesn't support those natively (yet) ? | 18:05 |
| @clarkb:matrix.org | mnaser: it does not. Work/discussion has begun on updating the version of ansible that is supported to something that understands collections but getting there is the first step | 18:07 |
| @clarkb:matrix.org | I don't know that the step after to try and support collections has started | 18:07 |
| @fungicide:matrix.org | mnaser: also https://zuul-ci.org/docs/zuul/latest/developer/specs/unrestricted-ansible.html merged today as a sort of prerequisite to that, i think | 19:05 |
| @mnaser:matrix.org | Yeah, I’ve been following that. I think that will come in super handy once it’s made process since right now we’re relying on Molecule to run our tests because we can’t control the Ansible version inside Zuul | 19:06 |
| @mnaser:matrix.org | I’d imagine a ton of benefits we would get if we swapped to native Zuul jobs so the tasks for playbooks would all be as part of the final report | 19:06 |
| @fungicide:matrix.org | yeah, but more immediately, it'll make supporting newer ansible versions a lot easier | 19:07 |
| @mnaser:matrix.org | I think we can probably get away without needing collection support, at least not natively inside Zuul if we start to get a little bit more control over the executors bubblewrap | 19:08 |
| @fungicide:matrix.org | we no longer have to audit, fork and monkey-patch zuul's standard library (and adding collections support would have made that even more impossible anyway) | 19:08 |
| @fungicide:matrix.org | * we no longer have to audit, fork and monkey-patch ansible's standard library (and adding collections support would have made that even more impossible anyway) | 19:08 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 835910: Fix module loading for Windows jobs https://review.opendev.org/c/zuul/zuul/+/835910 | 20:30 | |
| @jim:acmegating.com | zuul-maint: ^ that's severe enough to warrant 5.2.2 asap i think. | 20:31 |
| @clarkb:matrix.org | corvus: tristanC left a thought but i went ahead and approved it. You should have plenty of time to update it if you choose to | 20:35 |
| @jim:acmegating.com | yeah, i'll go ahead and do that | 20:35 |
| @jim:acmegating.com | i'm just going to run it through my local test setup first :) | 20:35 |
| @pearcetyler:matrix.org | Has anyone run into the error `Pull request merge failed: Required status check "github-junipersquare/gate" is expected.` today? Haven't seen any issues like this before, then suddenly hitting a few of these today. No recent changes to our Zuul config | 22:29 |
| @clarkb:matrix.org | That status check is one set by zuul? I wonder if github is being a little less consistent in their backend after zuul completes and tries to merge it | 22:31 |
| @clarkb:matrix.org | I know they were talking about making database updates recently and that was the source of a number of their recent outages | 22:32 |
| @pearcetyler:matrix.org | Yeah that status check is set by Zuul. I was wondering the same thing, I figure it's changes on the GH side. Any ideas for workarounds? | 22:33 |
| @clarkb:matrix.org | I think the order of operations there is gate finishes, zuul reports status of the gate to github, zuul clicks the merge button. If those two steps are happening too close together for github then maybe that is the problem | 22:33 |
| @pearcetyler:matrix.org | Is there any way to increase the lag between those two steps? | 22:36 |
| @clarkb:matrix.org | I think the only thing between those two steps currently is setting a pull comment if configured to. Let me get a link | 22:40 |
| @clarkb:matrix.org | https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/github/githubreporter.py#L87-L100 | 22:42 |
| @clarkb:matrix.org | you might try adding a sleep there to see if it gets better | 22:42 |
| @pearcetyler:matrix.org | Thank you, I'll give it a look | 22:42 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 835910: Fix module loading for Windows jobs https://review.opendev.org/c/zuul/zuul/+/835910 | 23:11 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 835918: Create missing db entries and retry https://review.opendev.org/c/zuul/zuul/+/835918 | 23:43 | |
| @jim:acmegating.com | zuul-maint: how does this look for a zuul release? commit 08348143f5c438547b3b4b73f6af7ff7e753e013 (HEAD -> master, tag: 5.2.2, origin/master, gerrit/master, refs/changes/10/835910/2) | 23:45 |
| @jim:acmegating.com | okay, i'll push that now | 23:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!