Tuesday, 2022-09-13

-@gerrit:opendev.org- Ian Wienand proposed:00:11
- [zuul/zuul] 857244: web: fix logo CSS https://review.opendev.org/c/zuul/zuul/+/857244
- [zuul/zuul] 857245: web: move navbar to tertiary https://review.opendev.org/c/zuul/zuul/+/857245
@iwienand:matrix.orgcorvus: ^ to me, after looking these two changes seem to stand-alone as better anyway; even if what has made them appear may be a backwards incompat issue with patternfly.00:12
@iwienand:matrix.orgi guess i should probably restack it all under the upgrade00:12
-@gerrit:opendev.org- Ian Wienand proposed:00:14
- [zuul/zuul] 857244: web: fix logo CSS https://review.opendev.org/c/zuul/zuul/+/857244
- [zuul/zuul] 857245: web: move navbar to tertiary https://review.opendev.org/c/zuul/zuul/+/857245
- [zuul/zuul] 855577: web: update to patternfly 4.235.7 https://review.opendev.org/c/zuul/zuul/+/855577
- [zuul/zuul] 854556: web: console: convert to PF4 DataList https://review.opendev.org/c/zuul/zuul/+/854556
- [zuul/zuul] 855297: web: console: compress datalist https://review.opendev.org/c/zuul/zuul/+/855297
- [zuul/zuul] 855574: web: task summary: make more consistent with other tabs https://review.opendev.org/c/zuul/zuul/+/855574
- [zuul/zuul] 855575: web: console: better task modal titles https://review.opendev.org/c/zuul/zuul/+/855575
- [zuul/zuul] 855578: web: build/autohold: fix props input https://review.opendev.org/c/zuul/zuul/+/855578
@jim:acmegating.comianw: based on a quick look on my local build, i agree the navbar looks better (and seems to function the same); thanks!00:21
@iwienand:matrix.orgi guess i'm learning about what App.test.jsx is doing today :)00:42
-@gerrit:opendev.org- Ian Wienand proposed:01:25
- [zuul/zuul] 857245: web: move navbar to tertiary https://review.opendev.org/c/zuul/zuul/+/857245
- [zuul/zuul] 855577: web: update to patternfly 4.235.7 https://review.opendev.org/c/zuul/zuul/+/855577
- [zuul/zuul] 854556: web: console: convert to PF4 DataList https://review.opendev.org/c/zuul/zuul/+/854556
- [zuul/zuul] 855297: web: console: compress datalist https://review.opendev.org/c/zuul/zuul/+/855297
- [zuul/zuul] 855574: web: task summary: make more consistent with other tabs https://review.opendev.org/c/zuul/zuul/+/855574
- [zuul/zuul] 855575: web: console: better task modal titles https://review.opendev.org/c/zuul/zuul/+/855575
- [zuul/zuul] 855578: web: build/autohold: fix props input https://review.opendev.org/c/zuul/zuul/+/855578
@vlotorev:matrix.orgianw: could you please have a look on clarifying docs for zuul-results-summary https://gerrit-review.googlesource.com/#/c/345235 ?06:32
-@gerrit:opendev.org- Zuul merged on behalf of Tristan Cacqueray: [zuul/zuul] 854774: elasticsearch: Remove the deprecated document type https://review.opendev.org/c/zuul/zuul/+/85477406:50
-@gerrit:opendev.org- Zuul merged on behalf of daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org: [zuul/zuul] 857001: Change custom type field to default one https://review.opendev.org/c/zuul/zuul/+/85700106:50
@avass:vassast.orgWe got another deadlock from `zuul_s3_upload.py` module, so it looks like zuul-jobs modules that uses threads can cause deadlocks too.06:54
-@gerrit:opendev.org- Marvin Becker proposed: [zuul/nodepool] 853993: Add hold command to disable nodes https://review.opendev.org/c/zuul/nodepool/+/85399309:44
@mbecker12:matrix.orgcorvus: ^ I reworked the tests a bit and I stumbled across the 'blocking' argument in the zk.lockNode function. Is there a reason why it has to be blocking=True in the delete command? Is it okay if it's not blocking for the hold command? What's the underlying reasoning here? I also left some comments in the patch again if you prefer the discussion to be held there :)09:51
@g_gobi:matrix.orgHi,10:34
I have 3 jobs.
job1, job2 and job3.
Job3 depends on job1 and job2.
So, zuul will start the job when job1 and job2 pass.
1. Is there any option to allow job3 even though job2 failed?
2. Is there any option to skip the jobs in the pipeline? In `openstack/tripleo-heat-templates pipeline` I can see some jobs are missing in other pipelines. How to skip some jobs in same pipeline ?
3. Here https://zuul.opendev.org/t/openstack/status some jobs are in paused state? What is paused state and how to pause the jobs?
@g_gobi:matrix.org * Hi,10:35
I have 3 jobs.
job1, job2 and job3.
Job3 depends on job1 and job2.
So, zuul will start the job when job1 and job2 passed.
1. Is there any option to allow job3 even though job2 failed?
2. Is there any option to skip the jobs in the pipeline? In `openstack/tripleo-heat-templates pipeline` I can see some jobs are missing in other pipelines. How to skip some jobs in same pipeline ?
3. Here https://zuul.opendev.org/t/openstack/status some jobs are in paused state? What is paused state and how to pause the jobs?
@vlotorev:matrix.orgGobi G: regarding #3, pausing the jobs described here https://zuul-ci.org/docs/zuul/latest/job-content.html#pausing-the-job11:38
@g_gobi:matrix.org * Hi,12:13
I have 3 jobs.
job1, job2 and job3.
Job3 depends on job1 and job2.
So, zuul will start the job3 when job1 and job2 pass.
1. Is there any option to allow job3 even though job2 failed?
2. Is there any option to skip the jobs in the pipeline? In `openstack/tripleo-heat-templates pipeline` I can see some jobs are missing in other pipelines. How to skip some jobs in same pipeline ?
3. Here https://zuul.opendev.org/t/openstack/status some jobs are in paused state? What is paused state and how to pause the jobs?
-@gerrit:opendev.org- Simon Westphahl proposed:12:39
- [zuul/zuul] 856523: Add span for builds and propagate via request https://review.opendev.org/c/zuul/zuul/+/856523
- [zuul/zuul] 857421: Trace merge requests and merger operations https://review.opendev.org/c/zuul/zuul/+/857421
-@gerrit:opendev.org- Simon Westphahl proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 856568: Use implicit trace context in build requests https://review.opendev.org/c/zuul/zuul/+/85656812:39
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 857421: Trace merge requests and merger operations https://review.opendev.org/c/zuul/zuul/+/85742113:12
@g_gobi:matrix.org> <@g_gobi:matrix.org> Hi,13:56
> I have 3 jobs.
> job1, job2 and job3.
>
> Job3 depends on job1 and job2.
> So, zuul will start the job3 when job1 and job2 pass.
>
> 1. Is there any option to allow job3 even though job2 failed?
> 2. Is there any option to skip the jobs in the pipeline? In `openstack/tripleo-heat-templates pipeline` I can see some jobs are missing in other pipelines. How to skip some jobs in same pipeline ?
> 3. Here https://zuul.opendev.org/t/openstack/status some jobs are in paused state? What is paused state and how to pause the jobs?
Any idea about this?
@fungicide:matrix.org> <@g_gobi:matrix.org> Hi,14:40
> I have 3 jobs.
> job1, job2 and job3.
>
> Job3 depends on job1 and job2.
> So, zuul will start the job3 when job1 and job2 pass.
>
> 1. Is there any option to allow job3 even though job2 failed?
> 2. Is there any option to skip the jobs in the pipeline? In `openstack/tripleo-heat-templates pipeline` I can see some jobs are missing in other pipelines. How to skip some jobs in same pipeline ?
> 3. Here https://zuul.opendev.org/t/openstack/status some jobs are in paused state? What is paused state and how to pause the jobs?
it's hard to know what the answers are without understanding a bit more about your expectations and what you mean by "skip in th epipeline" and "missing in other pipelines." if you're talking about conditionally skipping a dependent job based on decisions made in the job it's depending on, then that can be done by altering the value of zuul.child_jobs in a zuul_return: https://zuul-ci.org/docs/zuul/latest/job-content.html#skipping-dependent-jobs
@fungicide:matrix.org * it's hard to know what the answers are without understanding a bit more about your expectations and what you mean by "skip in the pipeline" and "missing in other pipelines." if you're talking about conditionally skipping a dependent job based on decisions made in the job it's depending on, then that can be done by altering the value of zuul.child\_jobs in a zuul\_return: https://zuul-ci.org/docs/zuul/latest/job-content.html#skipping-dependent-jobs14:40
@fungicide:matrix.orgbut it would be good to know exactly what it is you're trying to achieve (with examples if possible), since it's not easy to determine whether you're trying to use zuul in ways it wasn't designed for14:41
@fungicide:matrix.orgfor example, why would you want to run a job if another job it depends on failed? why just make that job not fail instead, or not be a dependency?14:42
@fungicide:matrix.org * for example, why would you want to run a job if another job it depends on failed? why not just make that job not fail under the relevant circumstances instead, or make it not be a dependency?14:43
@fungicide:matrix.orgif you're talking about not running some jobs under certain circumstances, that can be done with a variety of filters like which files changed, what branch is targeted, and so on14:44
@fungicide:matrix.orgif you want a job to be able to run even if its dependency didn't run, then you have the option of making them "soft" dependencies: https://zuul-ci.org/docs/zuul/latest/config/job.html#attr-job.dependencies.soft14:47
@fungicide:matrix.orgthat's specifically when the dependency didn't run though, not when it ran but failed14:47
@fungicide:matrix.org * that's specifically when the dependency didn't get selected to run though (perhaps because of a filter), not when it ran but failed14:48
@fungicide:matrix.orgit's possible you could use provides/requires relationships to make a "run job3 if either of job1 or job2 succeeded" but i'm not entirely sure, that's usually used when you have dependencies on artifacts which can be supplied by other buildsets earlier in the queue14:50
@fungicide:matrix.orgalso, what's the reporting expectation for these scenarios? like if job1 succeeds and job2 fails and then job3 runs and succeeds, is that a successful result or a failed result for their buildset?14:52
-@gerrit:opendev.org- Thomas Cardonne proposed: [zuul/zuul] 856294: feat(elasticsearch): support datastreams and rollover aliases https://review.opendev.org/c/zuul/zuul/+/85629415:32
@g_gobi:matrix.orgfungi: My expectation is like is there any option in zuul like allow_failure in gitlab. https://docs.gitlab.cn/14.0/ee/ci/yaml/#allow_failure16:32
It should fail the job but not the pipeline.
So, if I have one pipeline with 3 jobs.
1-> testing code in linux environment
2-> testing code in windows environment (this was new implementation it may break)
3-> release the code
We know windows will fail sometimes . bcz of that I don’t want to block the release.
@jim:acmegating.comGobi G: https://zuul-ci.org/docs/zuul/latest/config/job.html#attr-job.voting16:33
@g_gobi:matrix.orgcorvus: thanks. Currently, if the windows job failed, the release job not getting triggered. that also, solved if I use `vote: false` for windows job?16:36
@jim:acmegating.comif the release job is a dependency on the test job and the test job fails, the release job will not run.16:37
@jim:acmegating.comand i think that's true even if voting is false16:38
@g_gobi:matrix.orgIs there a way to solve this? 16:42
job3 depends on job1 and job2.
* if job2 fails, job3 should be triggered
* build should be marked as success and job2 marked as failure (It should be solved by `vote: false`)
@g_gobi:matrix.org * Is there a way to solve this?16:42
job3 depends on job1 and job2.
- even job2 fails, job3 should be triggered
- build should be marked as success and job2 marked as failure (It should be solved by `vote: false`)
@avass:vassast.orgGobi G: why does the release job need a depency towards the window test job in that case? Isn't it enough to be dependent on the Linux test?16:48
@avass:vassast.org* Gobi G: why does the release job need a depency towards the windows test job in that case? Isn't it enough to be dependent on the Linux test?16:48
@g_gobi:matrix.orgAlbin Vass: before releasing we will do some work. like updating the supported environments info like that. So, it depends on both linux and windows.16:53
@jim:acmegating.comGobi G: using the facilities available in zuul, i think your choices are to expand your workflow into more jobs so that you can express the relationships between steps better, or just have the intermediate job always succeed.16:59
@g_gobi:matrix.org**`even job2 fails, job3 should be triggered`**17:06
Can you explain how we can support this case in detail?
@jim:acmegating.comthere isn't a way to do that directly in zuul.  i think your options that i outlined above: construct the job graph a different way so that steps that you don't have any dependencies on jobs that are expected to fail, or make that job always succeed.17:12
@g_gobi:matrix.orgOh Okay. Thanks for the details.17:15
@jim:acmegating.com(maybe something like build->linux-test->release + build->windows-test)17:18
@g_gobi:matrix.orgcorvus: My existing setup looks like this only.17:25
build->linux-test->if-pass->update-report->release-only-for-linux
build->windows-test->if-pass->update-report->release-only-for-windows
just wanted
build->test-linux-and-windows-parallel->update-report-for-passed-env->release-passed-env
So, that my Resource utilization will be minimized.
Anyways `voting: false` I'll use that in my Pipeline so, that even windows job fail my buildset will pass.
Thanks
@jim:acmegating.comgood luck! :)17:29
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 855309: zuul-stream: Add variable to disable writing streaming files https://review.opendev.org/c/zuul/zuul/+/85530919:10
@iwienand:matrix.org> <@vlotorev:matrix.org> ianw: could you please have a look on clarifying docs for zuul-results-summary https://gerrit-review.googlesource.com/#/c/345235 ?19:36
I'll comment in the review; but we should also match "bare" comments (```const statusRe = /^Build (?<status>\w+)\./gm;``` ... to be exact :). What output did you have that didn't match?
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 854939: Revert "Use debian bookworm version of libc-bin" https://review.opendev.org/c/zuul/zuul/+/85493919:41
@iwienand:matrix.orgcorvus: just to follow up on https://review.opendev.org/q/topic:web-results-strings -- the css override bug should be fixed in an upcoming patternfly release.  but the pf4 update is now stacked ontop of two cleanups suggested that are better than what we had anyway, imo (upstream also clarified they're good).  so in short, that stack should be ready for review, again :)20:20
@jim:acmegating.comianw: thanks!20:30
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 857240: Cause providers to continue to decline requests when at quota https://review.opendev.org/c/zuul/nodepool/+/85724020:55
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed:21:38
- [zuul/zuul] 856568: Use implicit trace context in build requests https://review.opendev.org/c/zuul/zuul/+/856568
- [zuul/zuul] 857515: Return real (invalid) spans from null saved state https://review.opendev.org/c/zuul/zuul/+/857515
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 856523: Add span for builds and propagate via request https://review.opendev.org/c/zuul/zuul/+/85652321:38
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 856338: Add semaphore support to web UI https://review.opendev.org/c/zuul/zuul/+/85633821:44
@jim:acmegating.comianw: teensy -1 on 857244 otherwise stack is +2 from me21:54
@iwienand:matrix.orgcorvus: ahh, yep.  how about i add a change to prevent it running all that ci again21:55
@jim:acmegating.comwfm21:55
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 857517: web: remove unused class from brand tag https://review.opendev.org/c/zuul/zuul/+/85751722:00
@iwienand:matrix.orgcorvus: ^22:01
@iwienand:matrix.orgi also think with the svg; it's got a lot of white space around it.  i can pull it into inkscape and trim that, it will probably make the height make more sense22:01
@jim:acmegating.commight work better at smaller screen sizes too22:12
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 857522: web: update whitespace around logo https://review.opendev.org/c/zuul/zuul/+/85752223:38

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!