| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 866855: Correctly set the repo remote URL https://review.opendev.org/c/zuul/zuul/+/866855 | 13:46 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 866855: Correctly set the repo remote URL https://review.opendev.org/c/zuul/zuul/+/866855 | 13:51 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 866855: Correctly set the repo remote URL https://review.opendev.org/c/zuul/zuul/+/866855 | 13:53 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 866863: Store paused events on the build and report them https://review.opendev.org/c/zuul/zuul/+/866863 | 14:32 | |
| @noonedeadpunk:matrix.org | Hey folks! Question - I've jsut found that secrets rsa key and ssh project key look like quite different RSAs and even with different lenght. I assume, that executor loads ssh key, not secrets key? But also /api/tenant/{tenant}/key/{project}.pub seems to return public key for secrets, not for ssh? | 14:42 |
|---|---|---|
| @noonedeadpunk:matrix.org | As while I can encrypt/use secrets with public key provided with api - it seems not being accepted by hosts when zuul tries to log in. In executor log I do see though that it's added to the ssh agent | 14:44 |
| @noonedeadpunk:matrix.org | (after conversion to appropriate format with `ssh-keygen -i -m PKCS8` | 14:44 |
| @jim:acmegating.com | Dmitriy Rabotyagov: for ssh you want something like https://zuul.opendev.org/api/tenant/zuul/project-ssh-key/zuul/zuul-jobs.pub | 14:46 |
| @noonedeadpunk:matrix.org | aha project-ssh-key | 14:47 |
| @noonedeadpunk:matrix.org | I wonder how this never was mentioned in API docs.... | 14:48 |
| @noonedeadpunk:matrix.org | I will try to check this and patch! | 14:48 |
| @noonedeadpunk:matrix.org | Thanks @corvus | 14:48 |
| @noonedeadpunk:matrix.org | yes, sweet, works like a charm | 14:52 |
| -@gerrit:opendev.org- Dmitriy Rabotyagov proposed: [zuul/zuul] 866874: Add project-ssh-key to API docs https://review.opendev.org/c/zuul/zuul/+/866874 | 15:25 | |
| -@gerrit:opendev.org- Ade Lee proposed: [zuul/zuul-jobs] 866881: Add ubuntu to enable-fips role https://review.opendev.org/c/zuul/zuul-jobs/+/866881 | 17:11 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866920: Specify crio repo key ids for apt_key https://review.opendev.org/c/zuul/zuul-jobs/+/866920 | 18:35 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866926: Set tox NO_COLOR=1 flag https://review.opendev.org/c/zuul/zuul-jobs/+/866926 | 19:12 | |
| @clarkb:matrix.org | Heads up tox 4.0 just released and is breaking things. 866926 is an attempt at maybe addressing some of the worst bits | 19:13 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866928: Pin tox to <4 in ensure-tox installation https://review.opendev.org/c/zuul/zuul-jobs/+/866928 | 19:19 | |
| @jim:acmegating.com | Clark: some comments on 926 | 19:19 |
| @jim:acmegating.com | Clark: are you thinking 928 instead of 926? | 19:20 |
| @clarkb:matrix.org | corvus: yes I think so for now since basically everything is broken and we still don't have a good idea why. | 19:20 |
| @clarkb:matrix.org | The reason I didn't use environment: in those ansible tasks is we seem to want to have the calling tasks provide the env. But maybe I can do a merge there or something. | 19:21 |
| @jim:acmegating.com | yeah, i think something like a merge would be appropriate if that's the way we want to go | 19:21 |
| @jim:acmegating.com | (but also, maybe we just do 928 and then fix whatever parsing we're doing?) | 19:22 |
| @clarkb:matrix.org | corvus: yes I think that is what we should do | 19:22 |
| @clarkb:matrix.org | since removing the tox pin will be self testing on that change which we can use to confirm the broken has gone away | 19:22 |
| @clarkb:matrix.org | corvus: another thing I noticed running tox4 --showconfig locally is that it doesn't emit config for things like docs or py310 | 19:22 |
| @clarkb:matrix.org | and I'm not sure if that will break siblings from a behavioral standpoint yet | 19:23 |
| @clarkb:matrix.org | all the more reason to pin and understand this better | 19:23 |
| @clarkb:matrix.org | arg it is still failing. What is going on | 19:27 |
| @jim:acmegating.com | https://zuul.opendev.org/t/zuul/build/cefa18ff96404262a6c76eee15469a90/log/job-output.txt#442 | 19:27 |
| @clarkb:matrix.org | the docs job seemed to use an unpinned tox install. Is that not speculative? | 19:28 |
| @clarkb:matrix.org | the other tox jobs do seem to be passing which implies this isn't a global issue | 19:28 |
| @jim:acmegating.com | it may be special... | 19:28 |
| @jim:acmegating.com | yeah, it's in opendev/base-jobs | 19:29 |
| @jim:acmegating.com | Clark: i think we can make it non-voting in gate in your change then flip it back on | 19:29 |
| @clarkb:matrix.org | Ok | 19:30 |
| @clarkb:matrix.org | Firefox just crashed on me so give me a sec | 19:30 |
| @clarkb:matrix.org | I was trying to paste the playbook path that showed base-jobs and it exploded | 19:30 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866928: Pin tox to <4 in ensure-tox installation https://review.opendev.org/c/zuul/zuul-jobs/+/866928 | 19:32 | |
| @clarkb:matrix.org | That adds nonvoting flags to the zuul-tox-docs job | 19:32 |
| @jim:acmegating.com | Clark: want to push up the revert for nonvoting too? | 19:34 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 866929: Make zuul-tox-docs voting again https://review.opendev.org/c/zuul/zuul-jobs/+/866929 | 19:36 | |
| @jim:acmegating.com | nm | 19:36 |
| @clarkb:matrix.org | oh sorry I got distracted. But yes good call | 19:37 |
| @clarkb:matrix.org | I think the next step is to run the tox siblings tox config stuff locally and try to inspect the actual showconfig content to see how it differs as zuul+ansible process it | 19:39 |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul-jobs] 866928: Pin tox to <4 in ensure-tox installation https://review.opendev.org/c/zuul/zuul-jobs/+/866928 | 19:51 | |
| @fungicide:matrix.org | zuul-promote-docs failed on 866928, presumably for the same reason we pinned the check/gate job | 19:59 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 866929: Make zuul-tox-docs voting again https://review.opendev.org/c/zuul/zuul-jobs/+/866929 | 20:03 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866935: DNM Cat the tox --showconfig output under tox v4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | 20:06 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866935: DNM Cat the tox --showconfig output under tox v4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | 20:18 | |
| @jim:acmegating.com | fungi: and is working again after the merge, so i think we're good | 20:28 |
| @fungicide:matrix.org | yep! observed the same | 20:33 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866935: DNM Cat the tox --showconfig output under tox v4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | 20:41 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866935: DNM Cat the tox --showconfig output under tox v4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | 21:22 | |
| -@gerrit:opendev.org- Clark Boylan proposed: | 22:05 | |
| - [zuul/zuul-jobs] 866935: Update zuul-jobs to handle tox3 and tox4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | ||
| - [zuul/zuul-jobs] 866939: Move tox logs into env specific dirs https://review.opendev.org/c/zuul/zuul-jobs/+/866939 | ||
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866935: Update zuul-jobs to handle tox3 and tox4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | 22:21 | |
| -@gerrit:opendev.org- Clark Boylan proposed: | 22:35 | |
| - [zuul/zuul-jobs] 866935: Update zuul-jobs to handle tox3 and tox4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | ||
| - [zuul/zuul-jobs] 866943: Flip ensure-tox to tox v4 by default https://review.opendev.org/c/zuul/zuul-jobs/+/866943 | ||
| -@gerrit:opendev.org- Clark Boylan proposed: | 23:14 | |
| - [zuul/zuul-jobs] 866935: Update zuul-jobs to handle tox3 and tox4 https://review.opendev.org/c/zuul/zuul-jobs/+/866935 | ||
| - [zuul/zuul-jobs] 866943: Flip ensure-tox to tox v4 by default https://review.opendev.org/c/zuul/zuul-jobs/+/866943 | ||
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866943: Flip ensure-tox to tox v4 by default https://review.opendev.org/c/zuul/zuul-jobs/+/866943 | 23:16 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 866945: Avoid tox-bindep in test-tox job https://review.opendev.org/c/zuul/zuul-jobs/+/866945 | 23:24 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866943: Flip ensure-tox to tox v4 by default https://review.opendev.org/c/zuul/zuul-jobs/+/866943 | 23:30 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 866945: Avoid tox-bindep in test-tox job https://review.opendev.org/c/zuul/zuul-jobs/+/866945 | 23:36 | |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 866943: Flip ensure-tox to tox v4 by default https://review.opendev.org/c/zuul/zuul-jobs/+/866943 | 23:48 | |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 866945: Avoid tox-bindep in test-tox job https://review.opendev.org/c/zuul/zuul-jobs/+/866945 | 23:48 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!