| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 837117: Update change deps on refresh after cache miss https://review.opendev.org/c/zuul/zuul/+/837117 | 05:47 | |
| @iwienand:matrix.org | mnaser: lmn if you're trying to build manylinux wheels too; i have some experience with that from pyca/cryptography and would be interested in anything we can do to make any bits of it more generic and zuul-job-sy | 06:39 |
|---|---|---|
| @q:fricklercloud.de | these periodic jobs seem to be failing since forever because they lack zuul.change being defined. does anyone care to fix them or should they just be dropped? (affects all distros afaict, just selected on example) https://zuul.opendev.org/t/zuul/builds?job_name=zuul-jobs-test-base-roles-ubuntu-focal&project=zuul%2Fzuul-jobs&pipeline=periodic-weekly&skip=0 | 07:02 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 835918: Create missing db entries and retry https://review.opendev.org/c/zuul/zuul/+/835918 | 07:33 | |
| @mnaser:matrix.org | > <@iwienand:matrix.org> mnaser: lmn if you're trying to build manylinux wheels too; i have some experience with that from pyca/cryptography and would be interested in anything we can do to make any bits of it more generic and zuul-job-sy | 12:10 |
| okay cool, i am kinda cheating since the vms have extra-index-urls pointing at your work .. :p | ||
| @mnaser:matrix.org | btw, for users of Zuul that use vscode, i suggest adding the yaml extension which has native Zuul schema so it will have autocomplete and validation inside the editor thanks to ssbarnea -- i also just pushed up a few fixes here as well to make it avoid false warnings :) https://github.com/SchemaStore/schemastore/pull/2179 | 13:13 |
| @mhuin:matrix.org | nice! | 13:14 |
| @jpew:matrix.org | Ping on https://review.opendev.org/c/zuul/nodepool/+/833099 | 13:15 |
| @ssbarnea:matrix.org | mnaser: my original desire was to add schema definition in zuul itself but when i proposed this long time ago I did not get much interest. | 15:06 |
| @ssbarnea:matrix.org | mnaser: I reviewed it. it will take up to two days to get it merged. | 15:07 |
| @mnaser:matrix.org | has anyone used the {build,upload,promote}-docker-image jobs with registries other than dockerhub? | 16:14 |
| @mnaser:matrix.org | so far i think i am ok but now realized the promote jobs are fully hardcoded to dockerhub stuff | 16:14 |
| -@gerrit:opendev.org- Ron Izraeli proposed: [zuul/zuul] 837319: dependencies-by-topic changes should be updated when new patch-sets are created. https://review.opendev.org/c/zuul/zuul/+/837319 | 16:54 | |
| -@gerrit:opendev.org- Ron Izraeli proposed: [zuul/zuul] 837319: dependencies-by-topic changes should be updated when new patch-sets are created. https://review.opendev.org/c/zuul/zuul/+/837319 | 17:00 | |
| -@gerrit:opendev.org- Ron Izraeli marked as active: [zuul/zuul] 837319: dependencies-by-topic changes should be updated when new patch-sets are created. https://review.opendev.org/c/zuul/zuul/+/837319 | 17:01 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 837385: Add more debug info to AWS driver https://review.opendev.org/c/zuul/nodepool/+/837385 | 17:32 | |
| -@gerrit:opendev.org- Ron Izraeli proposed: [zuul/zuul] 837319: dependencies-by-topic changes should be updated when new patch-sets are created. https://review.opendev.org/c/zuul/zuul/+/837319 | 17:43 | |
| @mnaser:matrix.org | i'm almost done refactoring part of promote-docker-image to be using OCI standard APIs | 18:04 |
| @mnaser:matrix.org | but... man it's some of the grossest stuff ever :( | 18:04 |
| @clarkb:matrix.org | corvus: I didn't approve https://review.opendev.org/c/zuul/nodepool/+/837385 because it seemed like a good idea to run all that through the full set of testing first. But I think it is good to go if that comes back +1 | 18:07 |
| -@gerrit:opendev.org- Guillaume Chauvel proposed: [zuul/zuul] 762887: Improve and fix merger getFilesChanges https://review.opendev.org/c/zuul/zuul/+/762887 | 18:26 | |
| -@gerrit:opendev.org- Ron Izraeli proposed: [zuul/zuul] 837319: dependencies-by-topic changes should be updated when new patch-sets are created. https://review.opendev.org/c/zuul/zuul/+/837319 | 18:28 | |
| -@gerrit:opendev.org- Guillaume Chauvel proposed: [zuul/zuul] 762887: Improve and fix merger getFilesChanges https://review.opendev.org/c/zuul/zuul/+/762887 | 18:36 | |
| -@gerrit:opendev.org- Ron Izraeli proposed: [zuul/zuul] 837319: dependencies-by-topic changes should be updated when new patch-sets are created. https://review.opendev.org/c/zuul/zuul/+/837319 | 19:04 | |
| -@gerrit:opendev.org- Mohammed Naser proposed: [zuul/zuul-jobs] 837405: build-docker-image: Fix buildset registry pushes https://review.opendev.org/c/zuul/zuul-jobs/+/837405 | 19:23 | |
| @mnaser:matrix.org | ^ really trivial change, appreciate a quick eye on it :) | 19:24 |
| -@gerrit:opendev.org- Ron Izraeli proposed: [zuul/zuul] 837319: dependencies-by-topic changes should be updated when new patch-sets are created. https://review.opendev.org/c/zuul/zuul/+/837319 | 19:43 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Benjamin Schanzel: [zuul/zuul-jobs] 798802: Fix a bug in s3 log uploader with .gz files https://review.opendev.org/c/zuul/zuul-jobs/+/798802 | 20:04 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 837407: client: generate a token silently if possible and needed https://review.opendev.org/c/zuul/zuul/+/837407 | 20:09 | |
| @mhuin:matrix.org | ^ this should simplify the life of many admins | 20:09 |
| @clarkb:matrix.org | corvus: on https://review.opendev.org/c/zuul/zuul-jobs/+/826158 I didn't +A because tristanC has a comment that I'm not 100% of (though I did note what I thought the answer was) | 21:08 |
| @blaisep-sureify:matrix.org | mnaser: I confess I really want to feel ok about having Vexxhost manage my zuul-ci, but I'm having a really bad user experience. | 21:16 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 826158: Add upload-logs-ibm role https://review.opendev.org/c/zuul/zuul-jobs/+/826158 | 21:21 | |
| @jim:acmegating.com | Clark: tristanC thanks! fixed ^ | 21:21 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 837416: Add per-build WinRM cert generation https://review.opendev.org/c/zuul/zuul-jobs/+/837416 | 21:23 | |
| @mnaser:matrix.org | Blaise Pabon: can we take this to private messages so we can get you sorted out? :) | 21:28 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:29 | |
| - [zuul/zuul] 837418: Remove restricted Ansible documentation https://review.opendev.org/c/zuul/zuul/+/837418 | ||
| - [zuul/zuul] 837419: Remove ansible plugin tests https://review.opendev.org/c/zuul/zuul/+/837419 | ||
| - [zuul/zuul] 837420: Remove unused ansible modules https://review.opendev.org/c/zuul/zuul/+/837420 | ||
| - [zuul/zuul] 837421: Remove localhost restriction from command.py https://review.opendev.org/c/zuul/zuul/+/837421 | ||
| - [zuul/zuul] 837422: Remove workspace path check from zuul_return https://review.opendev.org/c/zuul/zuul/+/837422 | ||
| - [zuul/zuul] 837423: Remove unused functions from zuul.ansible.paths https://review.opendev.org/c/zuul/zuul/+/837423 | ||
| - [zuul/zuul] 837424: Remove actiongeneral / actiontrusted dirs https://review.opendev.org/c/zuul/zuul/+/837424 | ||
| - [zuul/zuul] 837425: Remove plugin tests https://review.opendev.org/c/zuul/zuul/+/837425 | ||
| - [zuul/zuul] 837426: Remove executor plugin checks https://review.opendev.org/c/zuul/zuul/+/837426 | ||
| - [zuul/zuul] 837427: Add docs about additional security considerations https://review.opendev.org/c/zuul/zuul/+/837427 | ||
| - [zuul/zuul] 837428: Remove unrestricted Ansible spec https://review.opendev.org/c/zuul/zuul/+/837428 | ||
| @clarkb:matrix.org | corvus: hrm it got a -1 with the latest ps | 22:03 |
| @jim:acmegating.com | Clark: doesn't appear related | 22:08 |
| @clarkb:matrix.org | `requests.exceptions.RequestException: Failed for a reason` might be my favorite exception message now | 22:09 |
| @clarkb:matrix.org | I've approved the change | 22:10 |
| @jim:acmegating.com | heh :) i ran the tests a few times locally under 3.9 successfully | 22:10 |
| @jim:acmegating.com | so i think it's probably just a rare race | 22:10 |
| @jim:acmegating.com | we should change that to "Failed for a really good reason" | 22:11 |
| @mnaser:matrix.org | > <@gerrit:opendev.org> James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:28 |
| > - [zuul/zuul] 837418: Remove restricted Ansible documentation https://review.opendev.org/c/zuul/zuul/+/837418 | ||
| > - [zuul/zuul] 837419: Remove ansible plugin tests https://review.opendev.org/c/zuul/zuul/+/837419 | ||
| > - [zuul/zuul] 837420: Remove unused ansible modules https://review.opendev.org/c/zuul/zuul/+/837420 | ||
| > - [zuul/zuul] 837421: Remove localhost restriction from command.py https://review.opendev.org/c/zuul/zuul/+/837421 | ||
| > - [zuul/zuul] 837422: Remove workspace path check from zuul_return https://review.opendev.org/c/zuul/zuul/+/837422 | ||
| > - [zuul/zuul] 837423: Remove unused functions from zuul.ansible.paths https://review.opendev.org/c/zuul/zuul/+/837423 | ||
| > - [zuul/zuul] 837424: Remove actiongeneral / actiontrusted dirs https://review.opendev.org/c/zuul/zuul/+/837424 | ||
| > - [zuul/zuul] 837425: Remove plugin tests https://review.opendev.org/c/zuul/zuul/+/837425 | ||
| > - [zuul/zuul] 837426: Remove executor plugin checks https://review.opendev.org/c/zuul/zuul/+/837426 | ||
| > - [zuul/zuul] 837427: Add docs about additional security considerations https://review.opendev.org/c/zuul/zuul/+/837427 | ||
| > - [zuul/zuul] 837428: Remove unrestricted Ansible spec https://review.opendev.org/c/zuul/zuul/+/837428 | ||
| super excited for this! | ||
| @mnaser:matrix.org | FYI, appreciate a quick review on a (trivial) change here: https://review.opendev.org/c/zuul/zuul-jobs/+/837405 | 22:30 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 826158: Add upload-logs-ibm role https://review.opendev.org/c/zuul/zuul-jobs/+/826158 | 22:30 | |
| @clarkb:matrix.org | mnaser: I'm not sure I understand that one since we push to the registry directly on line 9? | 22:33 |
| @clarkb:matrix.org | is the local name already prefixed with the buidlset registry? | 22:33 |
| @clarkb:matrix.org | I wonder why we don't see that in our jobs since they all use the registry | 22:33 |
| @mnaser:matrix.org | > <@clarkb:matrix.org> mnaser: I'm not sure I understand that one since we push to the registry directly on line 9? | 22:34 |
| The local name is not prefixed with buildset registry, its just bare and then the buildset registry gets added to it. | ||
| @mnaser:matrix.org | > <@clarkb:matrix.org> I wonder why we don't see that in our jobs since they all use the registry | 22:34 |
| I think because we're maybe one of the first ones who make use of the `docker_registry` variable (since we are trying to push to Google Artifact Registry and not Docker Hub) | ||
| @clarkb:matrix.org | I see | 22:34 |
| @mnaser:matrix.org | So usually since there is no prefix, the normal push goes to Docker Hub and that's done. Now, when using `docker_registry`, it builds the image with the actual `docker_registry` prefix tagged, which means that there is no image without the prefix | 22:35 |
| @clarkb:matrix.org | aha its the build the produces a local image with that prefix and we have to honor that when pushing the tag too | 22:35 |
| @clarkb:matrix.org | I think the comment in that file is what has confused me it says this is being done for the buildest registry which isn't entirely true | 22:36 |
| @clarkb:matrix.org | it is doing it for hte buildset registry and for the actual registry potentially | 22:36 |
| @mnaser:matrix.org | https://zuul.opendev.org/t/vexxhost/build/a492f4512ba54d31bf4dae088249d0ec | 22:36 |
| and we fail as such (notice in the buid logs, we build with tags): | ||
| ``` | ||
| Successfully tagged us-docker.pkg.dev/vexxhost-infra/openstack/tempest:change_837346_30.1.0-0-x86_64 | ||
| Successfully tagged us-docker.pkg.dev/vexxhost-infra/openstack/tempest:30.1.0-0-x86_64 | ||
| ``` | ||
| and then later it tries to run the `docker tag` and end up with: | ||
| ``` | ||
| Error response from daemon: No such image: tempest:30.1.0-0-x86_64 | ||
| ``` | ||
| @clarkb:matrix.org | no your commit message is fine | 22:36 |
| @clarkb:matrix.org | https://review.opendev.org/c/zuul/zuul-jobs/+/837405/1/roles/build-docker-image/tasks/push.yaml#1 that first line of that file says tag for the buildest registry | 22:37 |
| @mnaser:matrix.org | yeah "for the buildset registry" it's a little mixed up | 22:37 |
| @clarkb:matrix.org | I left a note about updating that but not worth respinning on its own I don't think | 22:38 |
| @jim:acmegating.com | Clark: it's subtle -- but i think the comment was trying to say "tag the $something with the buildset registry identifier so that we can push it to the buildset registry". the $something doesn't actually have anything to do with the buildset registry, and that's the thing that mnaser is fixing. | 23:04 |
| -@gerrit:opendev.org- Zuul merged on behalf of Mohammed Naser: [zuul/zuul-jobs] 837405: build-docker-image: Fix buildset registry pushes https://review.opendev.org/c/zuul/zuul-jobs/+/837405 | 23:23 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 836086: Fix implied branch matchers with regex chars https://review.opendev.org/c/zuul/zuul/+/836086 | 23:57 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!