| -@gerrit:opendev.org- Simon Westphahl proposed: | 07:41 | |
| - [zuul/zuul] 859066: Link span of queue item to trigger event span https://review.opendev.org/c/zuul/zuul/+/859066 | ||
| - [zuul/zuul] 859067: Trace received Github events https://review.opendev.org/c/zuul/zuul/+/859067 | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 859218: Fix instance check when refreshing dependencies https://review.opendev.org/c/zuul/zuul/+/859218 | 08:02 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: | 12:10 | |
| - [zuul/zuul] 859067: Trace received Github events https://review.opendev.org/c/zuul/zuul/+/859067 | ||
| - [zuul/zuul] 859280: Trace Gerrit connection events https://review.opendev.org/c/zuul/zuul/+/859280 | ||
| - [zuul/zuul] 859281: Trace Gitlab connection events https://review.opendev.org/c/zuul/zuul/+/859281 | ||
| - [zuul/zuul] 859282: Trace Pagure connection events https://review.opendev.org/c/zuul/zuul/+/859282 | ||
| - [zuul/zuul] 859283: Trace Git connection events https://review.opendev.org/c/zuul/zuul/+/859283 | ||
| - [zuul/zuul] 859284: Trace timer trigger events https://review.opendev.org/c/zuul/zuul/+/859284 | ||
| - [zuul/zuul] 859285: Trace Zuul trigger events https://review.opendev.org/c/zuul/zuul/+/859285 | ||
| @jpew:matrix.org | Are the minutes from the last tech meeting somewhere? | 14:54 |
|---|---|---|
| @jim:acmegating.com | jpew: do you have the right channel? | 14:56 |
| @jpew:matrix.org | Nope! | 14:56 |
| @jim:acmegating.com | whew! :) | 14:56 |
| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/zuul-jobs] 851334: test-requirements: bump to Ansible 2.8 https://review.opendev.org/c/zuul/zuul-jobs/+/851334 | 16:16 | |
| -@gerrit:opendev.org- Clark Boylan proposed: | 16:27 | |
| - [zuul/nodepool] 859172: Collect libvirt logs in openstack functional testing https://review.opendev.org/c/zuul/nodepool/+/859172 | ||
| - [zuul/nodepool] 859311: Use stage-output to collect openstack functional logs https://review.opendev.org/c/zuul/nodepool/+/859311 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Ian Wienand: [zuul/zuul-jobs] 852932: zuul_azure_storage_upload: rename https://review.opendev.org/c/zuul/zuul-jobs/+/852932 | 16:29 | |
| @clarkb:matrix.org | That is somethign I noticed when trying to collect libvirt logs in the openstack functional job. I think we should either drop zuul_copy_output or include stage-output and use zuul_copy_output for more stuff. I went with the use zuul_copy_output for more stuff option to dog food more | 16:29 |
| -@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/zuul-jobs] 852932: zuul_azure_storage_upload: rename https://review.opendev.org/c/zuul/zuul-jobs/+/852932 | 16:36 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 858726: Fix CORS and endpoint in AWS log upload https://review.opendev.org/c/zuul/zuul-jobs/+/858726 | 16:51 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 859331: Add prefix to azure log upload https://review.opendev.org/c/zuul/zuul-jobs/+/859331 | 17:55 | |
| @clarkb:matrix.org | corvus: do you know why stage-output seems to stage things to a directory that is different than zuul_output_dir by default? | 18:08 |
| @clarkb:matrix.org | I think this means that you cannot use stage-output and fetch-output together without modifying vars for stage-output or fetch-output | 18:08 |
| @clarkb:matrix.org | https://opendev.org/opendev/system-config/src/branch/master/zuul.d/system-config-roles.yaml#L20 is what system-config appears to do to address this | 18:09 |
| -@gerrit:opendev.org- Clark Boylan proposed: | 18:10 | |
| - [zuul/nodepool] 859311: Use stage-output to collect openstack functional logs https://review.opendev.org/c/zuul/nodepool/+/859311 | ||
| - [zuul/nodepool] 859172: Collect libvirt logs in openstack functional testing https://review.opendev.org/c/zuul/nodepool/+/859172 | ||
| @jim:acmegating.com | Clark: i can think of no reason for that to have been done intentionally. probably an accident of the development history (i think stage-output may actually predate fetch-output) | 18:11 |
| @clarkb:matrix.org | I suspect that we can fairly safely update stage-output's default since its efforts are largely temporary anyway | 18:11 |
| @jim:acmegating.com | seems reasonable (with notification period etc) | 18:12 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 859333: Clarify that stage-output doesn't actually write to zuul_output_dir https://review.opendev.org/c/zuul/zuul-jobs/+/859333 | 18:24 | |
| @clarkb:matrix.org | This small update to docs should help people understand they aren't currently aligned at least (would've helped me) | 18:24 |
| @jim:acmegating.com | Clark: oh wow the docs said it did the thing you thought it did but it didn't do that thing? | 18:25 |
| @clarkb:matrix.org | correct | 18:25 |
| @clarkb:matrix.org | it wasn't until I started digging into the job console log and seeing the path mismatches that I realized there was something unexpceted there | 18:26 |
| @clarkb:matrix.org | at first I thought maybe the jobs weren't generating the files I wanted to copy or something like that | 18:26 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-jobs] 859333: Clarify that stage-output doesn't actually write to zuul_output_dir https://review.opendev.org/c/zuul/zuul-jobs/+/859333 | 18:29 | |
| @clarkb:matrix.org | Sorry I caught a typo | 18:29 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 859331: Add prefix to azure log upload https://review.opendev.org/c/zuul/zuul-jobs/+/859331 | 18:33 | |
| @y2kenny:matrix.org | corvus: I sent you a DM. | 19:28 |
| @clarkb:matrix.org | corvus: for the latest zuul result on https://review.opendev.org/c/zuul/nodepool/+/859172/3 the image build job timed out which caused jobs that depend on that to be skipped. Instead of listing the jobs as SKIPPED the comment indicates two jobs are skipped which is less info. | 20:01 |
| @clarkb:matrix.org | Would it make sense to list them with a status of SKIPPED? | 20:02 |
| @jim:acmegating.com | Clark: we used to, but there's a pattern where people intentionally skip hundreds of jobs using a dispatch job, and that change was to make their lives better. i guess we could make it a heuristic? (more than 5 jobs skipped? abbreviate, otherwise, list) but also, is the info important enough to include in the report text? | 20:08 |
| maybe the issue is that these are skipped due to a hard dependency, not a zuul.child_jobs return value. maybe we should distinguish between those, but at this point in the reporter, i don't think we have that information. | ||
| that's probably the actual fix. | ||
| @jim:acmegating.com | (i think we can probably get that information to the reporter, then we should be able to make that fix) | 20:09 |
| @clarkb:matrix.org | well I just spent a bit of time trying to understand why the job I wanted to run didn't run | 20:10 |
| @clarkb:matrix.org | I agree distinguishing between dynamic job skips and hard dependencies is probably a good way to address it | 20:15 |
| @clarkb:matrix.org | Zuulians care to weigh in on https://review.opendev.org/c/zuul/zuul-jobs/+/858961 ? I think testing shows this should be a safe update and one that saves time for certain job setups | 20:39 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 23:55 | |
| - [zuul/zuul] 858987: WIP: Rename admin-rule to authorization-rule https://review.opendev.org/c/zuul/zuul/+/858987 | ||
| - [zuul/zuul] 859353: Remove unused /api/user/authorizations REST endpoint https://review.opendev.org/c/zuul/zuul/+/859353 | ||
| - [zuul/zuul] 859354: Simplify tenant_authorizatons check https://review.opendev.org/c/zuul/zuul/+/859354 | ||
| - [zuul/zuul] 859355: Add check_auth tool to zuul-web https://review.opendev.org/c/zuul/zuul/+/859355 | ||
| - [zuul/zuul] 859356: Hide components in whitelabel tenants https://review.opendev.org/c/zuul/zuul/+/859356 | ||
| - [zuul/zuul] 859357: Add api-root tenant config object https://review.opendev.org/c/zuul/zuul/+/859357 | ||
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!