| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/zuul-jobs] 831326: ensure-podman: add containernetworking-plugins https://review.opendev.org/c/zuul/zuul-jobs/+/831326 | 03:36 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 833760: Fix ZKObject compressed data size estimate https://review.opendev.org/c/zuul/zuul/+/833760 | 09:10 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 833794: Log debug messages when remote ref is missing https://review.opendev.org/c/zuul/zuul/+/833794 | 09:30 | |
| @clarkb:matrix.org | If anyone is interested in helping with the Summit Forum schedule we've been asked if anyone from the Zuul community would like to help with that. Let me know if you are interested and I can pass that along | 15:23 |
|---|---|---|
| @mhuin:matrix.org | are there any suggested topics yet? | 15:24 |
| @clarkb:matrix.org | in general or from Zuul? I'm not aware of any but I'm sure they will start to come in | 15:25 |
| @mhuin:matrix.org | yeah I meant zuul | 15:25 |
| @clarkb:matrix.org | I think they may use the cfp tooling now to collect them to reduce the number of tools needed and I'm not sure how publci all of that is to start | 15:25 |
| @fungicide:matrix.org | not just zuul-specific topics, but also evaluating other proposed topics based on their applicability to or potential interest from zuul contributors/users | 15:32 |
| -@gerrit:opendev.org- Andy Ladjadj proposed: | 15:48 | |
| - [zuul/zuul-jobs] 834043: [upload-logs-base] add public url attribute https://review.opendev.org/c/zuul/zuul-jobs/+/834043 | ||
| - [zuul/zuul-jobs] 834044: [upload-logs-base] set ContentEncoding only if not 'None' https://review.opendev.org/c/zuul/zuul-jobs/+/834044 | ||
| - [zuul/zuul-jobs] 834045: [upload-logs-base] add ipa extension to known mime types https://review.opendev.org/c/zuul/zuul-jobs/+/834045 | ||
| - [zuul/zuul-jobs] 834046: [upload-logs-base] add android mime-type https://review.opendev.org/c/zuul/zuul-jobs/+/834046 | ||
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834043: [upload-logs-base] add public url attribute https://review.opendev.org/c/zuul/zuul-jobs/+/834043 | 15:56 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834044: [upload-logs-base] set ContentEncoding only if not 'None' https://review.opendev.org/c/zuul/zuul-jobs/+/834044 | 15:56 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834044: [upload-logs-base] set ContentEncoding only if not 'None' https://review.opendev.org/c/zuul/zuul-jobs/+/834044 | 15:57 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834045: [upload-logs-base] add ipa extension to known mime types https://review.opendev.org/c/zuul/zuul-jobs/+/834045 | 15:57 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834046: [upload-logs-base] add android mime-type https://review.opendev.org/c/zuul/zuul-jobs/+/834046 | 15:57 | |
| -@gerrit:opendev.org- Andy Ladjadj marked as active: | 15:57 | |
| - [zuul/zuul-jobs] 834043: [upload-logs-base] add public url attribute https://review.opendev.org/c/zuul/zuul-jobs/+/834043 | ||
| - [zuul/zuul-jobs] 834044: [upload-logs-base] set ContentEncoding only if not 'None' https://review.opendev.org/c/zuul/zuul-jobs/+/834044 | ||
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834045: [upload-logs-base] add ipa extension to known mime types https://review.opendev.org/c/zuul/zuul-jobs/+/834045 | 16:01 | |
| -@gerrit:opendev.org- Andy Ladjadj marked as active: [zuul/zuul-jobs] 834045: [upload-logs-base] add ipa extension to known mime types https://review.opendev.org/c/zuul/zuul-jobs/+/834045 | 16:02 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834046: [upload-logs-base] add android mime-type https://review.opendev.org/c/zuul/zuul-jobs/+/834046 | 16:02 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul] 829033: Pin elasticsearch to <8.0.0 https://review.opendev.org/c/zuul/zuul/+/829033 | 16:02 | |
| -@gerrit:opendev.org- Andy Ladjadj marked as active: [zuul/zuul-jobs] 834046: [upload-logs-base] add android mime-type https://review.opendev.org/c/zuul/zuul-jobs/+/834046 | 16:06 | |
| -@gerrit:opendev.org- Andy Ladjadj proposed: [zuul/zuul-jobs] 834044: [upload-logs-base] set ContentEncoding only if not 'None' https://review.opendev.org/c/zuul/zuul-jobs/+/834044 | 16:51 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 834057: Remove unused AWS implementation https://review.opendev.org/c/zuul/nodepool/+/834057 | 18:11 | |
| @q:fricklercloud.de | shouldn't there be releasenotes for nodepool 5.0.0? there's also no 5.0.0 tag in the version selector in the docs https://zuul-ci.org/docs/nodepool/releasenotes.html | 19:55 |
| @clarkb:matrix.org | q: https://zuul-ci.org/docs/nodepool/5.0.0/releasenotes.html has the 5.0.0 release notes. I think we need to land a change that updates release notes against master to regenerate it on the main doc pages? | 19:57 |
| @q:fricklercloud.de | hmm, the latest docs builds looks o.k. https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b2e/821711/10/gate/zuul-tox-docs/b2e299a/docs/releasenotes.html , not sure why it didn't seem to get published properly | 20:12 |
| @q:fricklercloud.de | ah, they get published to https://zuul-ci.org/docs/nodepool/latest/, so https://zuul-ci.org/docs/nodepool/ stays untouched | 20:20 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 830525: Update AWS driver to use statemachine framework https://review.opendev.org/c/zuul/nodepool/+/830525 | 20:20 | |
| @jim:acmegating.com | looks like we need some redirects for that | 20:24 |
| @clarkb:matrix.org | tristanC: https://review.opendev.org/c/zuul/nodepool/+/833683 may interest you. I tried to provide what I thought was useful feedback but we don't use that driver in opendev so I'm less familiar with why you might want it | 20:26 |
| @jim:acmegating.com | Clark: that change doesn't make sense to me | 20:30 |
| @clarkb:matrix.org | corvus: I think the idea is what whatever the container is built to do it should just do it rather than sitting idle and waiting for ansible tasks to run against it. This makes sense if you are doing a multinode job where maybe you deploy a service with a container and just want it to run then another node can do interactions against it? | 20:30 |
| @clarkb:matrix.org | At least that was how I interpreted it | 20:31 |
| @jim:acmegating.com | Clark: but that wasn't the idea of the driver at all | 20:31 |
| @clarkb:matrix.org | Ya the idea was that you'd run ansible against it and the command was a sleep loop. But I don't think it is necessarily wrong either? | 20:31 |
| @jim:acmegating.com | Clark: that's what the openshift pods driver is for | 20:31 |
| @jim:acmegating.com | i agree both behaviors are right; they are implemented by different drivers | 20:32 |
| @clarkb:matrix.org | well not everyone has an openshift? I'm not sure why that should be a requirement | 20:33 |
| @clarkb:matrix.org | (in fact I would argue that we should try and remove as much special functioanltiy from the openshift driver as possible because openshift is not testable anymore) | 20:33 |
| @jim:acmegating.com | well it's compatible? | 20:33 |
| @clarkb:matrix.org | oh is it? I'm not sure that was clear to me (or others) | 20:33 |
| @jim:acmegating.com | wait no... it's the type that i'm thinking of | 20:36 |
| @jim:acmegating.com | https://zuul-ci.org/docs/nodepool/kubernetes.html#attr-providers.[kubernetes].pools.labels.type | 20:36 |
| @jim:acmegating.com | namespace or pod | 20:36 |
| @jim:acmegating.com | so if you want your job to run a pod, then you get a namespace and have the job run the pod | 20:36 |
| @jim:acmegating.com | if you want to use a pod as an execution environment, you get a pod | 20:37 |
| @avass:vassast.org | corvus: not everyone is allowed to create namespaces/projects though. But it's still possible to start the process inside the container with Ansible so i dunno | 20:37 |
| @jim:acmegating.com | so the expected use for someone who wants a zuul job to, say, run an etcd for testing is to get a namespace and run the etcd pod in that namespace. | 20:37 |
| @clarkb:matrix.org | Albin Vass: ya I think spamaps ran into that too? | 20:38 |
| @clarkb:matrix.org | corvus: I agree that namespaces probably fit that need better though. Might be worth discussing with the chagne author if they considered that and if so what the issues were | 20:40 |
| @jim:acmegating.com | if you have nodepool just run some pod with its default command, then it's not actually testing anything | 20:40 |
| @clarkb:matrix.org | ya I guess the other issue is you don't get speculative build states in that case because the image is fixed | 20:44 |
| @jim:acmegating.com | right. but you can run a speculative build with a namespace | 20:44 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-website] 834072: Add redirects for nodepool https://review.opendev.org/c/zuul/zuul-website/+/834072 | 20:48 | |
| @jim:acmegating.com | q: Clark fungi ^ nodepool redirects | 20:49 |
| @clarkb:matrix.org | aha https://zuul-ci.org/docs/nodepool/latest/releasenotes.html has the 5.0.0 releasenotes. +2 on the redirects | 20:50 |
| @avass:vassast.org | It would be nice to allow nodesets to specify an image/tag, or somehow allow speculative containers though :) | 20:50 |
| @tristanc_:matrix.org | Clark: thanks, I agree with what @corvus said, the pod type is not designed to run arbitrary command. | 21:06 |
| @tristanc_:matrix.org | Albin Vass: yeah, it's currently tedious to have to register images in nodepool before being able to use them in zuul. It would be really useful if a job could define the required image directly in the nodeset configuration. | 21:07 |
| @tristanc_:matrix.org | we actually started to define new jobs variant to skip the ensure-* pre-run, but they requires specifics nodepool labels. | 21:09 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 830526: Refactor AWS driver tests https://review.opendev.org/c/zuul/nodepool/+/830526 | 21:27 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: | 21:56 | |
| - [zuul/zuul-registry] 831135: Update testing to Ubuntu Focal https://review.opendev.org/c/zuul/zuul-registry/+/831135 | ||
| - [zuul/zuul-registry] 831480: tox-py38 : don't run on Fedora https://review.opendev.org/c/zuul/zuul-registry/+/831480 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 23:13 | |
| - [zuul/nodepool] 830527: Add additional tests to the aws driver https://review.opendev.org/c/zuul/nodepool/+/830527 | ||
| - [zuul/nodepool] 834057: Remove unused AWS implementation https://review.opendev.org/c/zuul/nodepool/+/834057 | ||
| @clarkb:matrix.org | I was just going to ask about that. I assume there was a fix | 23:13 |
| @jim:acmegating.com | yeah... and it's super easy but it's buried in a rebase, let me pull out the diff | 23:13 |
| @clarkb:matrix.org | https://review.opendev.org/c/zuul/nodepool/+/830527/2..3/nodepool/tests/unit/test_driver_aws.py is what gerrit shows | 23:14 |
| @jim:acmegating.com | oh okay cool, then you're all set :) | 23:15 |
| @jim:acmegating.com | looks like gerrit has improved its inter patchset diffs? | 23:15 |
| @clarkb:matrix.org | yes, I think it has done a bit of work there. You'll see purple in diffs sometimes too where it is showing you changes between rebases that aren't related to the change in question | 23:16 |
| @jim:acmegating.com | cool | 23:16 |
| @jim:acmegating.com | Clark: i feel like the fix is trivial enough to go ahead and re-approve, what do you think? | 23:16 |
| @clarkb:matrix.org | ya I think so. You just had to use the mocks after setting the fixtures | 23:17 |
| @jim:acmegating.com | (also, it's amazing we managed to win that race 3 times before -- i'm guessing the tests from the other driver shifted the stestr run order to make it easier to lose the race now) | 23:17 |
| @jim:acmegating.com | reapproved | 23:18 |
| @clarkb:matrix.org | corvus: those changes probably deserve a nodepool release as well? opendev isn't in a position to test them though | 23:23 |
| @clarkb:matrix.org | those changes == aws and ibm driver updates | 23:23 |
| @jim:acmegating.com | Clark: yes i think so | 23:32 |
| @jim:acmegating.com | i'd like to get these merged then do an opendev upgrade and zuul release: https://review.opendev.org/q/hashtag:gitlab-timer | 23:37 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-website] 834072: Add redirects for nodepool https://review.opendev.org/c/zuul/zuul-website/+/834072 | 23:38 | |
| @clarkb:matrix.org | > <@jim:acmegating.com> i'd like to get these merged then do an opendev upgrade and zuul release: https://review.opendev.org/q/hashtag:gitlab-timer | 23:49 |
| I can review those first thing tomorrow | ||
| @clarkb:matrix.org | if someone else beats me too them that is fine too | 23:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!