| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-jobs] 847111: Update ensure-ghc to use ghcup https://review.opendev.org/c/zuul/zuul-jobs/+/847111 | 00:32 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-jobs] 847111: Update ensure-ghc to use ghcup https://review.opendev.org/c/zuul/zuul-jobs/+/847111 | 00:45 | |
| @tobias.henkel:matrix.org | corvus: yeah, that makes sense | 05:19 |
|---|---|---|
| @chgans:matrix.org | Is James in this room? He wrote the deduplicate stuff, and i found some odd clues I would like to share. | 06:37 |
| @chgans:matrix.org | https://opendev.org/zuul/zuul/src/branch/master/zuul/model.py#L4715 In my case all the items have no jobs (other_item.getJobs() return an empty list/set) | 06:38 |
| @chgans:matrix.org | I think i've found the root cause: https://storyboard.openstack.org/#!/story/2010098 | 07:18 |
| @chgans:matrix.org | Basically i bind mounted my local zuul repo to overlay the container's /usr/local/lib/python3.8/site-packages/zuul/ path. And was able to do print-based troubleshooting. Hope this can help to investigate and fix the issue | 07:20 |
| @tristanc_:matrix.org | Christian Gagneraud: James is corvus here | 11:42 |
| @tristanc_:matrix.org | corvus: I may have a send a deprecation notice for zuul-jobs to the wrong mailing list, feel free to discard my mail and i'll send it to zuul-discuss | 11:43 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 846801: Avoid collisions after ZK image data import https://review.opendev.org/c/zuul/nodepool/+/846801 | 13:44 | |
| @jim:acmegating.com | tristanC: i think we usually send those to zuul-announce, i'll moderate it through. | 13:47 |
| @jim:acmegating.com | tristanC: oh, it looks like that's just a proposal for now? maybe once it's passing tests and has a +2 you could resend that message to zuul-announce and include a date for the change? | 13:49 |
| @jim:acmegating.com | i'll discard it for now | 13:49 |
| @jim:acmegating.com | ianw: let's get some +2s on https://review.opendev.org/846248 before sending that msg through | 13:52 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com marked as active: [zuul/nodepool] 846442: WIP Update ZooKeeper class connection methods https://review.opendev.org/c/zuul/nodepool/+/846442 | 14:05 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 846442: Update ZooKeeper class connection methods https://review.opendev.org/c/zuul/nodepool/+/846442 | 14:05 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-jobs] 847111: Update ensure-ghc to use ghcup https://review.opendev.org/c/zuul/zuul-jobs/+/847111 | 14:10 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed wip: [zuul/zuul] 846445: Add ssh_server option to Gerrit driver https://review.opendev.org/c/zuul/zuul/+/846445 | 14:10 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com marked as active: [zuul/zuul] 846445: Add ssh_server option to Gerrit driver https://review.opendev.org/c/zuul/zuul/+/846445 | 14:10 | |
| -@gerrit:opendev.org- Tristan Cacqueray proposed: [zuul/zuul-jobs] 847111: Update ensure-ghc to use ghcup https://review.opendev.org/c/zuul/zuul-jobs/+/847111 | 15:17 | |
| @fzzfh:matrix.org | hello folks. I want let zuul return actual log address on comment. but it return build url like http://47.111.113.21:8081/zuul/t/local/build/19e74214767543b298c4e6c94c7788ed. | 16:10 |
| I wonder if it can be able to do by modify zuul code. | ||
| I found zuul scheduler may report by | ||
| https://opendev.org/zuul/zuul/src/tag/4.6.0/zuul/scheduler.py#L459 | ||
| https://opendev.org/zuul/zuul/src/tag/4.6.0/zuul/driver/gerrit/gerritreporter.py#L38 | ||
| but I don't find zuul return build-url code. any help would be appreciated. | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 846432: AWS driver create/delete improvements https://review.opendev.org/c/zuul/nodepool/+/846432 | 20:29 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 846442: Update ZooKeeper class connection methods https://review.opendev.org/c/zuul/nodepool/+/846442 | 20:31 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 846469: Skip RSA key validation on load https://review.opendev.org/c/zuul/zuul/+/846469 | 21:37 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 21:54 | |
| - [zuul/zuul] 842010: Add global semaphore support https://review.opendev.org/c/zuul/zuul/+/842010 | ||
| - [zuul/zuul] 841336: Add always-dynamic-branches option https://review.opendev.org/c/zuul/zuul/+/841336 | ||
| @iwienand:matrix.org | corvus: we've discussed https://review.opendev.org/846248 pretty thoroughly but ok. Clark do you mind adding a +2 now? | 22:12 |
| @clarkb:matrix.org | ianw: I think the suggestion for an email is that it is a potentially breaking change for some users. Not to discuss if it should happen | 22:14 |
| @iwienand:matrix.org | Clark: sorry, lack of context -- i've sent the email but it's in the moderation queue. corvus just requested +2's on it before releasing the mail | 22:15 |
| @jim:acmegating.com | ianw: er, when i looked at it this morning, it had no reviews... so that's great if it was discussed, but i had no indication there was consensus on it. | 23:12 |
| @clarkb:matrix.org | corvus: ya sorry I think the conversation happened elsewhere bceause there ended up being like 7 related changes | 23:13 |
| @clarkb:matrix.org | as we tried to sort out how to get a job to use the extra python version's venv to make a virtualenv | 23:13 |
| @clarkb:matrix.org | and ianw's change came out of that as "this is confusing and should just go away as it doens't work in many situations" | 23:14 |
| @jim:acmegating.com | okay, and that's fine. i think my expectation is that before we send out a zuul-announce email, we should have a change positively reviewed and able to merge. | 23:14 |
| @jim:acmegating.com | ianw: Clark anyone else involved in that convo that can throw another +2 on that now? | 23:20 |
| @iwienand:matrix.org | fungi: ^ has also been in the discussion | 23:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!