| @clarkb:matrix.org | corvus: hrm I've still got linter failures, but I think those can wait while we look at the inline commenting support. Also, I realize now that we test with python27 even if I assumed I didn't need python27 support. Arg my bad | 00:01 |
|---|---|---|
| @clarkb:matrix.org | Looks like siblings is working if I'm reading logs properly | 00:01 |
| @clarkb:matrix.org | corvus: the linter errors are due to me commenting out the inline comments stuff. Do you want me to squash the changes or should we keep them separate? | 00:03 |
| @clarkb:matrix.org | I'll just keep them separate | 00:04 |
| @jim:acmegating.com | either way wfm | 00:05 |
| @clarkb:matrix.org | I've got that all sorted and ready to push. I'm happy to wait for the buildset to return and check the inline comment behavior first though | 00:08 |
| @jim:acmegating.com | Clark: the zuul-client job failed.... | 00:09 |
| @clarkb:matrix.org | ya I was just looking at that. I think due to writing too much data into the subunit stream? THe actual siblings bits of this look correct | 00:09 |
| @clarkb:matrix.org | ya it looks like it may be extremely verbose and its making by browser sad to open the job output file | 00:11 |
| @clarkb:matrix.org | corvus: we probably need to make sure we're capturing logs properly by passing the env vars to do that? | 00:11 |
| @clarkb:matrix.org | and then zuul will only attach them for failed tests? | 00:12 |
| @clarkb:matrix.org | hrm it does look like tests are failing so maybe this is a side effect of the underlying failures | 00:13 |
| @clarkb:matrix.org | corvus: I think that target needs the manage-ansible command run to install ansible | 00:14 |
| @clarkb:matrix.org | `Exception: Requested ansible version '6' is not installed` is in the log. And double checking the siblings portion of the build it looks good to me | 00:14 |
| @clarkb:matrix.org | I'll go ahead and push my latest fixes and then update the zuul change noxfile.py on top of that | 00:16 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 868002: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 00:17 | |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 868134: Add inline comment support to nox https://review.opendev.org/c/zuul/zuul-jobs/+/868134 | 00:17 | |
| @jim:acmegating.com | Clark: ++ | 00:17 |
| @clarkb:matrix.org | corvus: will the remote session also need ansible installed? that actually runs ansible remotely so I think it does? | 00:18 |
| @clarkb:matrix.org | I'll go ahead and add it we can remove it later if it isn't needed | 00:18 |
| @jim:acmegating.com | yeah, they both should run z-m-a | 00:19 |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 867057: WIP: play with noxfile https://review.opendev.org/c/zuul/zuul/+/867057 | 00:19 | |
| @clarkb:matrix.org | arg one more linter failure. At this point I really will let this run since its just complaining about command vs shell. The old tox task needed shell for file redirection but this one shouldn't need a shell | 00:32 |
| @clarkb:matrix.org | zuul client job passed \o/ | 00:36 |
| @clarkb:matrix.org | And the inline comments worked. I'll get the linter fix up for zuul-jobs after dinner but maybe this is landable afterwards | 01:42 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 863440: Fix deduplication exceptions in pipeline processing https://review.opendev.org/c/zuul/zuul/+/863440 | 01:48 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 868002: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 02:09 | |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 868134: Add inline comment support to nox https://review.opendev.org/c/zuul/zuul-jobs/+/868134 | 02:09 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 868002: Add nox role and some simple jobs https://review.opendev.org/c/zuul/zuul-jobs/+/868002 | 02:41 | |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 868134: Add inline comment support to nox https://review.opendev.org/c/zuul/zuul-jobs/+/868134 | 02:41 | |
| @clarkb:matrix.org | Those both pass testing now. | 02:57 |
| @vonschultz:matrix.org | I've been having problems with "VcpuLimitExceeded" errors from AWS in Nodepool. I think it's a bug in Nodepool, and I think I know what the fix is. I uploaded my suggested fix to https://review.opendev.org/c/zuul/nodepool/+/867333. (It's the first time I try to contribute a fix, so I'm not entirely sure what the procedure is. Should I be announcing bugs and suggested fixes here?) | 13:00 |
| @fungicide:matrix.org | https://hachyderm.io/@paris/109546368606361349 | 15:13 |
| @jim:acmegating.com | Christian von Schultz: thanks -- i think we'll want to proceed carefully with that one and i suspect some of the folks we want to review it may have limited availability for the rest of the year. maybe look for progress in a few weeks. | 16:01 |
| @jamesharrisg:matrix.org | Hey: I posted https://storyboard.openstack.org/#!/story/2010483 recently on the storyboard and would like to consider fixing it but I wonder what the best approach would be to address this given the API doesn't have a built-in way to associate keys with repository strings. | 16:03 |
| @jim:acmegating.com | ^ related to `ensure-package-repository` role and keys (fyi) | 16:05 |
| @jamesharrisg:matrix.org | I'm not sure if other package managers in other repositories are able to set these up similarly. | 16:08 |
| @clarkb:matrix.org | OpenDev is able to use apt-key on Ubuntu Jammy | 16:11 |
| @clarkb:matrix.org | I agree this is something that should be improved, but I'm surprised it is breaking | 16:11 |
| @clarkb:matrix.org | (I know this because we ran into it with the opensuse apt repo for container tools having an expired key) | 16:11 |
| @jamesharrisg:matrix.org | is apt-key not installed by default? | 16:12 |
| @clarkb:matrix.org | it is part of the apt package on Jammy: https://packages.ubuntu.com/jammy/amd64/apt/filelist | 16:15 |
| @jamesharrisg:matrix.org | ok. then I think my attempts to use that role are more releated to us not finding the gnupg package. | 16:17 |
| @jamesharrisg:matrix.org | which seems bad. | 16:17 |
| @clarkb:matrix.org | we do explicitly install gnupg on our images | 16:17 |
| @jim:acmegating.com | fungi: https://fosstodon.org/@acmegating/109546910148509373 | 16:20 |
| @clarkb:matrix.org | James Harris: if gnupg is the issue, maybe we can make a short term fix to explicitly install it as part of the ensure repos role | 16:23 |
| @clarkb:matrix.org | and then plan a longer term solution for not using apt-key | 16:23 |
| @jamesharrisg:matrix.org | I think we already try to install it explicitly as part of `ensure-package-repository` | 16:27 |
| @jamesharrisg:matrix.org | * I think we (zuul-jobs) already try to install it explicitly as part of `ensure-package-repository` | 16:28 |
| @jamesharrisg:matrix.org | my issue was it wasn't in the cache but that might be because the cache hadn't been updated yet. | 16:28 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 867057: Add noxfile and switch to nox https://review.opendev.org/c/zuul/zuul/+/867057 | 16:46 | |
| @jim:acmegating.com | Clark: ^ mergeable version of that change | 16:46 |
| @clarkb:matrix.org | looks like you left zuul-tox-docs alone for now and we can sort that out in followups. That seems reasonable to me | 16:54 |
| @jim:acmegating.com | yeah, since we'll need the config project changes for that... | 16:55 |
| @clarkb:matrix.org | corvus: looks like that may have reverted the zuul-client and remote target manage-ansible additions | 16:56 |
| @jim:acmegating.com | oops | 16:56 |
| @clarkb:matrix.org | other than that I think the change looks great. And is a good way to ease into it | 16:57 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 867057: Add noxfile and switch to nox https://review.opendev.org/c/zuul/zuul/+/867057 | 16:58 | |
| @jim:acmegating.com | Clark: what's the story with fetch-tox-output and nox? | 17:05 |
| @clarkb:matrix.org | corvus: nox doesn't produce log files like tox so we just don't need it | 17:06 |
| @clarkb:matrix.org | the -v output is all the extra logging I could find | 17:06 |
| @jim:acmegating.com | gotcha | 17:07 |
| @clarkb:matrix.org | In a way this is a feature. Nox is not doing special things under the hood. It is running virtualenv and pip and then whatever else you explciitly tell it to do | 17:08 |
| @clarkb:matrix.org | with tox its behaviors are a bit more tox specific in places so careful logging becomes more important | 17:08 |
| @jim:acmegating.com | Clark: and i bet if we wanted to put anything in log files, we could, you know, just add code to that effect in noxfile.py | 17:09 |
| @jim:acmegating.com | Clark: remote: https://review.opendev.org/c/opendev/base-jobs/+/868226 Add nox-docs base jobs [NEW] | 17:09 |
| @clarkb:matrix.org | corvus: should ^ have a depends on to the zuul-jobs stack? | 17:11 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/project-config] 868228: Add nox-docs jobs https://review.opendev.org/c/zuul/project-config/+/868228 | 17:14 | |
| @clarkb:matrix.org | corvus: I also left a question on https://review.opendev.org/c/opendev/base-jobs/+/868226 I don't think its a major issue though | 17:16 |
| @jim:acmegating.com | Clark: replied | 17:21 |
| @clarkb:matrix.org | thanks. I see the secrets differ now | 17:21 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 868229: Switch to nox-docs https://review.opendev.org/c/zuul/zuul/+/868229 | 17:22 | |
| @jim:acmegating.com | Clark: okay i think that's the nox-docs stack for zuul | 17:22 |
| @jim:acmegating.com | i think once that's landed, we can do tox->nox for the rest of the zuul projects, then cleanup | 17:22 |
| @clarkb:matrix.org | yup and I think at that point we can add some self hosting testing of the nox role in zuul-jobs. I punted on that because most of the tox testing there seems to rely on the zuul-jobs uses tox itself | 17:24 |
| @clarkb:matrix.org | corvus: I think the zuul tests are also quite verbose due to deprecation warnings not being filtered? I Think we can followup onthat though and it isn't critical | 17:34 |
| @rgunasekaran:matrix.org | Hello All, | 20:59 |
| new to this group & the community so bear with me while i get my footing | ||
| I want to contribute, can you help me picking out something to work on? please suggest on accurate successful documentation for setting up the environment and start contributing! | ||
| @clarkb:matrix.org | rgunasekaran: I answered this question from an OpenDev perspective, but now I'll try to give you a Zuul perspective. The zuul/zuul-jobs repo is a great place to dig in as a zuul user. This is a central library of shared building blocks for zuul CI jobs. There is documentation that could be expanded on for the existing roles and jobs. You might also want to add roles or jobs that are useful to you as a zuul user | 21:41 |
| @rgunasekaran:matrix.org | Clark please share the Zulu/zuul-jobs repo | 21:44 |
| @rgunasekaran:matrix.org | I will read the documentation | 21:44 |
| @clarkb:matrix.org | https://opendev.org/zuul/zuul-jobs/ is the repo itself. Code review for it happens here: https://review.opendev.org/q/status:open+project:zuul/zuul-jobs | 21:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!