| -@gerrit:opendev.org- Tudor-Stefan Tabacel-Manea proposed: [zuul/zuul] 963201: SQLReporter: Log error of artifact name too long https://review.opendev.org/c/zuul/zuul/+/963201 | 06:23 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 962760: Use global S3 endpoint to get bucket location https://review.opendev.org/c/zuul/zuul/+/962760 | 09:17 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 963399: Further fix to node assignment upgrade https://review.opendev.org/c/zuul/zuul/+/963399 | 14:23 | |
| @jangutter:matrix.org | While fixing https://review.opendev.org/c/zuul/zuul/+/963276 I uncovered an old proposal where folks were thinking of capturing stdout/stderr in the registered var (but suppressing the logging). Has there been any follow-up demand for this? | 15:53 |
|---|---|---|
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 963399: Further fix to node assignment upgrade https://review.opendev.org/c/zuul/zuul/+/963399 | 16:25 | |
| @jim:acmegating.com | jangutter: Clark described the notable problems that would need to be addressed, since it's very easy for something registered in a var to end up incidentally logged. it's a can of worms and would probably need quite a lot of work to get done right, especially in the area of identifying all the ways secret data could leak through. | 16:43 |
| @jangutter:matrix.org | Yeah, agreed. I don't have a need for this myself. I can see how it would be useful, but clearly no_log for commands aren't very loadbearing or some sucker would have run into this before. If anyone else wants this, my cache is hot for the next month or so, and I might be able to assist. | 16:50 |
| @jim:acmegating.com | yeah, i just wouldn't do it out of a sense of completeness since there's a big "notabug" conversation as part of it. :) | 16:56 |
| @jangutter:matrix.org | I just took a look at win_command and, oh man, I know dangerously little C# code. I'm... kinda wondering how it's going to behave in the `no_log: true` situation. From what I can tell, the C# portion might be streaming logs even if no_log is set. | 17:49 |
| I don't have a test environment anywhere, so assistance here would be greatly appreciated. I'll send up a review soon updating the ansible testcase (no code changes), asserting the "expected behavior". | ||
| -@gerrit:opendev.org- Jan Gutter proposed: [zuul/zuul] 963276: Fix ansible command plugin when using no_log https://review.opendev.org/c/zuul/zuul/+/963276 | 18:18 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:40 | |
| - [zuul/zuul] 963340: Add static driver https://review.opendev.org/c/zuul/zuul/+/963340 | ||
| - [zuul/zuul] 963466: Add tenant ssh keys https://review.opendev.org/c/zuul/zuul/+/963466 | ||
| - [zuul/zuul] 963467: Don't redact ssh key info in freeze job https://review.opendev.org/c/zuul/zuul/+/963467 | ||
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!