| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 824938: Update build set states after scheduling actions https://review.opendev.org/c/zuul/zuul/+/824938 | 00:31 | |
| @fzzfh:matrix.org | hello folks, According to https://github.com/openstack/manila-tempest-plugin/blob/master/zuul.d/manila-tempest-jobs.yaml , I define a child job https://paste.opendev.org/show/812149/ , but I don't know how to configure its devstack_localrc and devstack_local_conf variables. How to know which variables should to use. does anyone know this. any help would be appreciated. | 06:15 |
|---|---|---|
| @felixedel:matrix.org | corvus: Thanks for the comment on 824864. I haven't thought about the list being that large. But if you say so, it makes sense to make it a JobData attribute. I will update the change. | 06:19 |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 825042: Use new layout UUID when resetting pipeline state https://review.opendev.org/c/zuul/zuul/+/825042 | 08:03 | |
| @avass:vassast.org | tristanC: gerritbot doesn't seem to send a message to matrix when a change is marked ready for review. Is that intentional? Otherwise that could be a nice addition :) | 08:16 |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 825079: Protect database migration by a lock in Zookeeper https://review.opendev.org/c/zuul/zuul/+/825079 | 09:56 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 825087: Log duration of local tenant layout update https://review.opendev.org/c/zuul/zuul/+/825087 | 11:18 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 824864: Serialize FrozenJob.affected_projects to ZooKeeper https://review.opendev.org/c/zuul/zuul/+/824864 | 12:38 | |
| @tristanc_:matrix.org | Albin Vass: that is not intentional, the bot currently interpret change and patchset created event. It would need to decode https://gerrit-review.googlesource.com/Documentation/cmd-stream-events.html#_work_in_progress_state_changed for such message, I can have a look. Thanks for the suggestion! | 12:59 |
| @tristanc_:matrix.org | Albin Vass: should the bot send a message when a change is marked as WIP too? | 13:42 |
| @avass:vassast.org | tristanC: i don't think that's as important as an unwip message since that's to signal that the change is now ready to review. At least i don't see a usecase for it | 13:44 |
| @tristanc_:matrix.org | Albin Vass: here is the change to add change ready message: https://softwarefactory-project.io/r/c/software-factory/gerritbot-matrix/+/23646 . Thank you for the suggestion | 14:50 |
| @avass:vassast.org | tristanC: you got a type in there :) | 14:55 |
| @avass:vassast.org | * tristanC: you got a typo in there :) | 14:55 |
| @jim:acmegating.com | aside from typo lgtm :) | 14:56 |
| @tristanc_:matrix.org | Albin Vass: good catch! | 14:55 |
| -@gerrit:opendev.org- Szymon Datko proposed: [zuul/zuul-jobs] 825118: Add support for RPM packages https://review.opendev.org/c/zuul/zuul-jobs/+/825118 | 15:00 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/nodepool] 825129: Fix tenant-quota for providers without label quota infos https://review.opendev.org/c/zuul/nodepool/+/825129 | 15:41 | |
| @clarkb:matrix.org | > <@fzzfh:matrix.org> hello folks, According to https://github.com/openstack/manila-tempest-plugin/blob/master/zuul.d/manila-tempest-jobs.yaml , I define a child job https://paste.opendev.org/show/812149/ , but I don't know how to configure its devstack_localrc and devstack_local_conf variables. How to know which variables should to use. does anyone know this. any help would be appreciated. | 16:42 |
| You are probably best off asking the OpenStack QA team about this as those jobs and their inputs are things they help out with | ||
| @clarkb:matrix.org | corvus: do we need another restart on 824864 before 4.12.0? I'm guessing that is a good idea. | 16:48 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 825140: Bump test nodes up to Focal https://review.opendev.org/c/zuul/zuul/+/825140 | 16:54 | |
| @jim:acmegating.com | Clark: not a bad idea. we can make it a rolling restart. | 17:13 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 825152: DNM Testing the functional zuul stream jobs https://review.opendev.org/c/zuul/zuul/+/825152 | 17:45 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Felix Edel: [zuul/zuul] 824864: Serialize FrozenJob.affected_projects to ZooKeeper https://review.opendev.org/c/zuul/zuul/+/824864 | 17:57 | |
| @clarkb:matrix.org | Any idea why bumping the test nodes to focal causes ansible to not find the zuul module in the streaming functional jobs: https://zuul.opendev.org/t/zuul/build/9966bf5fa426458591c56f9b3b1f0791/console ? I guess the next step is to add -vvv to that invocation | 18:10 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 825152: DNM Testing the functional zuul stream jobs https://review.opendev.org/c/zuul/zuul/+/825152 | 18:14 | |
| @jim:acmegating.com | Clark: do you have a bionic run to compare against? | 18:21 |
| @clarkb:matrix.org | corvus: ya https://zuul.opendev.org/t/zuul/build/63b8426ab242431fa6fc26f8cc68783e/console | 18:21 |
| @clarkb:matrix.org | I'm going to see if the -vvv results are helpful | 18:21 |
| @jim:acmegating.com | that's super weird; zuul-manage-ansible is running out of the system-wide zuul install.... :/ | 18:22 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 825152: DNM Testing the functional zuul stream jobs https://review.opendev.org/c/zuul/zuul/+/825152 | 18:30 | |
| @clarkb:matrix.org | it is failing when loading callbacks, but we aren't getting the ansible.cfg with the previous failures because we short circuit in the post.yaml. That latest update shuold get us the ansible.cfg and we can compare to a successful job | 18:30 |
| @clarkb:matrix.org | `File "/home/zuul/src/opendev.org/zuul/zuul/zuul/ansible/2.8/callback/zuul_stream.py", line 46, in <module> | 18:34 |
| from zuul.ansible import paths` | ||
| @clarkb:matrix.org | corvus: we need to install zuul on the remote for some reason? | 18:35 |
| @clarkb:matrix.org | oh I think I see it | 18:37 |
| @clarkb:matrix.org | hrm no that is an ara problem not a zuul stream problem. I'll fix it anyway though | 18:37 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 825152: DNM Testing the functional zuul stream jobs https://review.opendev.org/c/zuul/zuul/+/825152 | 18:43 | |
| @clarkb:matrix.org | ya ^ confirms the ara fix isn't a general fix. Might need to hold a node. But I have to go run a meeting now. | 18:57 |
| -@gerrit:opendev.org- Tristan Cacqueray marked as active: [zuul/zuul-website-media] 824055: Add gource visualization render script https://review.opendev.org/c/zuul/zuul-website-media/+/824055 | 19:36 | |
| @iwienand:matrix.org | is anyone aware of issues holding failed jobs ATM? | 21:01 |
| @iwienand:matrix.org | i have this hold setup | 21:02 |
| @iwienand:matrix.org | https://paste.opendev.org/show/812199/ | 21:02 |
| @iwienand:matrix.org | and this job that i expected to match it -> https://zuul.opendev.org/t/openstack/build/0aac47452d76495da916054389404ba9 | 21:03 |
| @iwienand:matrix.org | i guess i want to look at the executor for logs | 21:09 |
| @iwienand:matrix.org | 2022-01-18 20:34:54,477 INFO zuul.ExecutorServer: [e: 90a751cb188e46cca05883cce7d9a2cc] [build: 0aac47452d76495da916054389404ba9] Held status set to False | 21:14 |
| @iwienand:matrix.org | that seems to confirm what we already know, that 0aac4745 was not held | 21:14 |
| @iwienand:matrix.org | 2022-01-18 20:34:54,474 DEBUG zuul.ExecutorServer: Checking build autohold key ('openstack', 'opendev.org/opendev/system-config', 'system-config-run-review-3.4') | 21:25 |
| @iwienand:matrix.org | 0000000000 | openstack | opendev/system-config | system-config-run-review-3.4 | .* | 1 | ianw: gerrit-3.4 hold | 21:25 |
| @jim:acmegating.com | ianw: i think it may need the canonical repo name | 21:26 |
| @iwienand:matrix.org | yeah, that seems to be the only difference in the match | 21:27 |
| @jim:acmegating.com | that might be a slight difference between zuul-client and zuul | 21:27 |
| @jim:acmegating.com | could probably be fixed in zuul though | 21:28 |
| @jim:acmegating.com | * could probably be fixed in the executor though | 21:28 |
| @iwienand:matrix.org | 0000000001 | openstack | opendev.org/opendev/system-config | system-config-run-review-3.4 | .* | 1 | ianw: gerrit-3.4 testing | 21:33 |
| @iwienand:matrix.org | i'll try again with that | 21:33 |
| @y2kenny:matrix.org | Hi, does the zuul-client release on Docker Hub get regularly updated? | 21:42 |
| @y2kenny:matrix.org | (I am trying to follow the upgrade note and do delete-state but I am not sure if I am doing it properly.) | 21:44 |
| @iwienand:matrix.org | Kenny Ho: it should be pushed automatically, so yes | 21:44 |
| @y2kenny:matrix.org | If I understand the notes properly, I should not have anything running that is zuul except zookeeper and run zuul-client delete-state (or is it zuul delete-state?) | 21:45 |
| @y2kenny:matrix.org | from the zuul-client image, I don't see delete-state option | 21:45 |
| @jim:acmegating.com | Kenny Ho: `zuul delete-state` | 21:45 |
| @y2kenny:matrix.org | but docker hub zuul/zuul image has delete-state | 21:45 |
| @y2kenny:matrix.org | Ok. | 21:45 |
| @y2kenny:matrix.org | so I also tried that but zuul delete-state seems to have hung | 21:46 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:47 | |
| - [zuul/zuul] 825022: Add a model API version https://review.opendev.org/c/zuul/zuul/+/825022 | ||
| - [zuul/zuul] 825023: DNM: Example model upgrade change https://review.opendev.org/c/zuul/zuul/+/825023 | ||
| @y2kenny:matrix.org | or take a very long time to return (I did not wait for it to finish.) | 21:47 |
| @jim:acmegating.com | Kenny Ho: it's interactive, it will ask you to confirm with 'yes'; if you don't see that you may be running it in a container without a stdin/out? | 21:47 |
| @y2kenny:matrix.org | the hung happens after I said yes. | 21:48 |
| @jim:acmegating.com | give it a few minutes then :) | 21:48 |
| @y2kenny:matrix.org | Ok... I wasn't sure how long the process should take. | 21:49 |
| @y2kenny:matrix.org | I will try again | 21:49 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 825079: Protect database migration by a lock in Zookeeper https://review.opendev.org/c/zuul/zuul/+/825079 | 21:52 | |
| @y2kenny:matrix.org | An unrelated question. I understand the job definitions have some kind of class hierarchy with jobs being able to inherit another jobs. Are there any mechanism in the job definitions that would allow for something more composible like interfaces? | 21:53 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 825079: Protect database migration by a lock in Zookeeper https://review.opendev.org/c/zuul/zuul/+/825079 | 21:53 | |
| @y2kenny:matrix.org | the use case I am thinking of is having a set of components for a set of distros and I need a build for all combinations. At this point we have been using yaml anchors but I am wondering if that's the best way to do it. | 21:54 |
| @y2kenny:matrix.org | (basically I have components A, B, C each needs to be released for distro X, Y, Z so I would need build job for AX, AY, AZ, BX, BY, BZ, CX, CY, XZ) | 21:55 |
| @jim:acmegating.com | there's a way to do multiple inheritance, but that doesn't sound like what you need -- that gets you one job with lots of input combinations you need lots of jobs with one input combination. | 21:56 |
| @jim:acmegating.com | for that, i'd recommend an external script to generate that for you. we do something similar (lots of os's to test) in zuul-jobs. i'll get a link | 21:56 |
| @jim:acmegating.com | Kenny Ho: https://opendev.org/zuul/zuul-jobs/src/branch/master/tools/update-test-platforms.py | 21:57 |
| @y2kenny:matrix.org | corvus: thanks! As to the multiple inheritance you mentioned, which zuul concept were you referring to? | 21:57 |
| @jim:acmegating.com | note that uses 'tags' to decide which jobs to build. that might be akin to your 'interface' idea. use tags to say this job exports that interface, then use the script to build all the jobs you need based on the interfaces the tags indicate. | 21:58 |
| @jim:acmegating.com | Kenny Ho: the input and output of that script is in https://opendev.org/zuul/zuul-jobs/src/branch/master/zuul-tests.d (same files input and output -- the script rewrites them) | 21:58 |
| @jim:acmegating.com | > <@y2kenny:matrix.org> corvus: thanks! As to the multiple inheritance you mentioned, which zuul concept were you referring to? | 22:00 |
| you can define multiple variants of a job with different parents which match the same conditions; they'll all be applied, so therefore they inherit from multiple parents. it's basically an emergent behavior of having both single-inheritance and variants. | ||
| @jim:acmegating.com | we have occasionally used in in zuul's own jobs, but i don't think it's in use right now... | 22:01 |
| @y2kenny:matrix.org | ok I will take a look at that also. | 22:01 |
| @jim:acmegating.com | it's *almost* never necessary, and it's particularly brain-bending, so use with caution. :) | 22:02 |
| @y2kenny:matrix.org | yea... I am just reading the docs right now and the script generation thing seems more straightforward and readable. | 22:02 |
| @y2kenny:matrix.org | (vs variants.) | 22:03 |
| @y2kenny:matrix.org | Oh and another unrelated observation/comment. With the new documentation template, docs don't seems to rank very high or even shows up in some search engine any more. (I wonder if it's related to some SEO thing.) Searching for 'zuul ci documentation job content' in ddg/bing, nothing from zuul-ci.org/docs shows up. Google seems to do slightly better, returning zuul-jobs docs in the new format but not the core zuul docs (there are results that points to zuul docs for older releases.) There's probably not much you guys can do about it. I just want to leave a comment and see if other folks observed similar things. | 22:16 |
| @jim:acmegating.com | it's possible adding redirects from the old urls could improve that | 22:28 |
| @clarkb:matrix.org | corvus: tobiash swest can you check my comments on https://review.opendev.org/c/zuul/zuul/+/825079 ? | 22:34 |
| @jim:acmegating.com | Clark: yeah i think you're right | 22:36 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 825079: Protect database migration by a lock in Zookeeper https://review.opendev.org/c/zuul/zuul/+/825079 | 22:41 | |
| @jim:acmegating.com | Clark: tobiash ^ i went ahead and updated it; i think that's simple enough we can save a day if we want to go ahead and approve that :) | 22:42 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 825042: Use new layout UUID when resetting pipeline state https://review.opendev.org/c/zuul/zuul/+/825042 | 22:45 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 825190: autohold: use canonical project name in REST interface https://review.opendev.org/c/zuul/zuul/+/825190 | 23:43 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Tobias Henkel: [zuul/zuul] 824706: Enable reprime by default https://review.opendev.org/c/zuul/zuul/+/824706 | 23:47 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 825079: Protect database migration by a lock in Zookeeper https://review.opendev.org/c/zuul/zuul/+/825079 | 23:52 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!