| *** wuchunyang has joined #zuul | 00:01 | |
| *** wuchunyang has quit IRC | 00:05 | |
| *** rfolco|ruck has quit IRC | 00:07 | |
| *** tosky has quit IRC | 00:17 | |
| *** Goneri has quit IRC | 00:37 | |
| *** wuchunyang has joined #zuul | 00:57 | |
| *** openstackgerrit has joined #zuul | 01:01 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Bump diskimage-builder requirement to 3.4.0 https://review.opendev.org/c/zuul/nodepool/+/763894 | 01:01 |
|---|---|---|
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Pin keystoneauth1 and cachetools on older Python https://review.opendev.org/c/zuul/zuul-jobs/+/763866 | 01:29 |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Use Python 3.x with launchpadlib https://review.opendev.org/c/zuul/zuul-jobs/+/763834 | 01:29 |
| fungi | https://review.opendev.org/763866 now fixes the py27 unit test jobs which 763834 was previously failing | 01:54 |
| *** holser has quit IRC | 02:05 | |
| *** iurygregory has quit IRC | 02:25 | |
| *** bhavikdbavishi has joined #zuul | 02:57 | |
| *** bhavikdbavishi1 has joined #zuul | 03:04 | |
| *** bhavikdbavishi has quit IRC | 03:06 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:06 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Bump diskimage-builder requirement to 3.4.0 https://review.opendev.org/c/zuul/nodepool/+/763894 | 03:31 |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: nodepool-functional-k8s: don't run on Xenial https://review.opendev.org/c/zuul/nodepool/+/763921 | 03:31 |
| *** wuchunyang has quit IRC | 04:04 | |
| *** bhavikdbavishi has quit IRC | 04:33 | |
| *** bhavikdbavishi has joined #zuul | 04:33 | |
| *** sanjayu_ has joined #zuul | 04:38 | |
| *** saneax has quit IRC | 04:38 | |
| *** y2kenny has quit IRC | 04:55 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:33 | |
| ianw | fungi: ^ that's somewhat related to related to your change to pin keystoneauth. the dib bump is a follow-on requirement to get centos building again | 06:01 |
| *** bhavikdbavishi1 has joined #zuul | 06:28 | |
| *** bhavikdbavishi has quit IRC | 06:30 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 06:30 | |
| *** Phoenikzz has quit IRC | 06:33 | |
| *** Phoenikzz has joined #zuul | 06:35 | |
| *** Phoenikzz has quit IRC | 06:36 | |
| *** Phoenikzz has joined #zuul | 06:36 | |
| *** Phoenikzz has quit IRC | 06:43 | |
| *** Phoenikzz has joined #zuul | 06:43 | |
| *** Phoenikzz has quit IRC | 06:49 | |
| *** Phoenikzz has joined #zuul | 06:50 | |
| *** Phoenikzz has joined #zuul | 06:51 | |
| *** bhavikdbavishi has quit IRC | 06:59 | |
| *** iurygregory has joined #zuul | 07:02 | |
| *** mach1na has joined #zuul | 07:11 | |
| *** Phoenikzz has quit IRC | 07:13 | |
| *** Phoenikzz has joined #zuul | 07:14 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Use logical timestamp to detect outdated changes https://review.opendev.org/c/zuul/zuul/+/763755 | 07:27 |
| *** mach1na has quit IRC | 07:30 | |
| *** rpittau|afk is now known as rpittau | 07:39 | |
| *** bhavikdbavishi has joined #zuul | 07:42 | |
| *** bhavikdbavishi1 has joined #zuul | 07:45 | |
| *** bhavikdbavishi has quit IRC | 07:46 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 07:46 | |
| *** zenkuro has quit IRC | 08:06 | |
| *** zenkuro has joined #zuul | 08:06 | |
| *** jcapitao has joined #zuul | 08:09 | |
| *** jpena|off is now known as jpena | 08:12 | |
| *** mach1na has joined #zuul | 08:15 | |
| *** tosky has joined #zuul | 08:48 | |
| *** zenkuro has quit IRC | 08:57 | |
| *** zenkuro has joined #zuul | 08:57 | |
| openstackgerrit | Sorin Sbârnea proposed zuul/zuul-jobs master: Add ensure-ansible role https://review.opendev.org/c/zuul/zuul-jobs/+/749706 | 09:04 |
| *** iurygregory has quit IRC | 09:37 | |
| *** iurygregory has joined #zuul | 09:43 | |
| *** piotrowskim has joined #zuul | 09:54 | |
| *** holser has joined #zuul | 10:02 | |
| *** CraigR has joined #zuul | 10:03 | |
| *** mach1na has quit IRC | 10:06 | |
| *** bhavikdbavishi has quit IRC | 10:06 | |
| zbr | which zuul-maint can help me ditch the tox-py27 from zuul-jobs? | 10:09 |
| *** nils has joined #zuul | 10:21 | |
| zbr | avass: https://review.opendev.org/c/zuul/zuul-jobs/+/749706 -- no brainer: ensure-ansible role. thanks | 10:32 |
| *** wuchunyang has joined #zuul | 10:43 | |
| avass | Do we need a role for that? | 10:49 |
| avass | all it does is run the pip module | 10:50 |
| avass | zbr: I think the tox-py27 is supposed to be kept | 10:51 |
| zbr | avass: yes we need as we will want to add extra features to it, also this is a building block for testing ansible collections, with various ansible versions. | 10:53 |
| zbr | avass: i agree with you that by itself it may look weird, but only because is not a goal by itself. | 10:53 |
| avass | So why not add it when it does more? | 10:54 |
| zbr | avass: sagi started working on this 3 months ago, see https://review.opendev.org/c/zuul/zuul-jobs/+/730360 -- i asked him to to the split. | 10:55 |
| zbr | there are a bunch of ansible related roles that depend on it. build/install/publish. | 10:56 |
| zbr | i personally hope that soon we will be able to publish zuul-jobs as a collection | 10:56 |
| avass | doesn't the role names cause problems with ansible collections? | 10:57 |
| *** wuchunyang has quit IRC | 10:57 | |
| zbr | yes it does but only for publish, and there are workarounds that can be taken even for this. | 10:58 |
| avass | I mean, the ansible-collection roles makes more sense to me but the ensure-ansible only runs the pip module, so why create a role for that. | 10:58 |
| avass | it's just having a role as a wrapper for a module so why not put the pip module directly in the pre-run? | 10:59 |
| *** sanjayu_ has quit IRC | 10:59 | |
| zbr | we will likely add ability to install ansible from distro packages after that, "pip" is not the only way to install it. | 10:59 |
| *** sanjayu_ has joined #zuul | 10:59 | |
| zbr | like we have with ensure-docker for example. | 11:00 |
| zbr | we also have to specify which ansible we want: the base or the full-blown,... | 11:00 |
| avass | so why not add that role when it does more than just the pip module, when it makes sense to have it as a role. | 11:01 |
| zbr | add a comment to the review asking for this. | 11:01 |
| zbr | i guess for main distros this can be quite easy to implement, and for others it can use the pip as fallback, i guess. | 11:02 |
| zbr | the only downside is that IMHO, when installing from distro, you likely lose the option to pick the version. | 11:02 |
| avass | I think I did, but maybe it got lost since the changes was split up | 11:02 |
| zbr | sshnaidm: ^ how hard it would be add ability to install ansible from distro in addition to pip for ensure-ansible? | 11:05 |
| sshnaidm | should be easy | 11:05 |
| zbr | in fact it would be cool to add a "system" option to ensure_ansible_version argument. | 11:06 |
| zbr | when system is mentioned, the role will install ansible using package instead of pip, for platforms supporting it, for others will keep using pip. | 11:07 |
| zbr | this will make this role very useful for testing other ansible related issues, as ansible version may vary across platforms. | 11:07 |
| sshnaidm | avass, the problem is in ansible versions, with pip you can install whatever you need, with package - only those provided by distro | 11:12 |
| sshnaidm | when ansible move fast and change its names every week, it might be very useful to have this choice | 11:12 |
| *** bhavikdbavishi has joined #zuul | 11:12 | |
| *** bhavikdbavishi1 has joined #zuul | 11:15 | |
| *** bhavikdbavishi has quit IRC | 11:16 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 11:16 | |
| avass | sshnaidm: yeah sure, I don't have anything against that. But I don't think it makes sense to create a role that only wraps a module | 11:24 |
| sshnaidm | avass, I've been looking at all ensure-* that we have | 11:25 |
| sshnaidm | thought it will be logical to have the same for ansible, because of PIPs multiple config options that not a lot of users are familiar with | 11:26 |
| zbr | sshnaidm: if i understood correctly the only remark avass had was that unless ensure-ansible also allows user to mention that he wants system ansible, the use case for having a different role is weak, as it can be replaced by a single ansible task. | 11:40 |
| zbr | based on this approach we could easily endup with ensure-<any-cli-from-pypa> role. | 11:41 |
| avass | zbr: yes :) | 11:41 |
| zbr | but I do happen to know that ansible is different beast, well cow. | 11:41 |
| zbr | afaik no distro allows user to specify version of ansible they ship, it means that we can have another special value for version, in addition to latest, one named "system". | 11:42 |
| zbr | when system is mentioned, the role would using system packager to install it instead of pip. | 11:43 |
| zbr | imho this would really make the role useful for many usecases. | 11:43 |
| *** bhavikdbavishi has quit IRC | 11:43 | |
| *** bhavikdbavishi has joined #zuul | 11:44 | |
| *** CraigR has quit IRC | 11:55 | |
| *** hashar has joined #zuul | 12:00 | |
| *** rfolco|ruck has joined #zuul | 12:11 | |
| *** jcapitao is now known as jcapitao_lunch | 12:12 | |
| *** wuchunyang has joined #zuul | 12:19 | |
| *** wuchunyang has quit IRC | 12:19 | |
| *** wuchunyang has joined #zuul | 12:21 | |
| *** wuchunyang has quit IRC | 12:26 | |
| tristanC | zbr: why would you want to remove tox-py27? | 12:26 |
| *** nils has quit IRC | 12:32 | |
| *** jpena is now known as jpena|lunch | 12:34 | |
| *** rlandy has joined #zuul | 12:41 | |
| *** mach1na has joined #zuul | 12:55 | |
| *** bhavikdbavishi has quit IRC | 13:02 | |
| *** jcapitao_lunch is now known as jcapitao | 13:09 | |
| zbr | tristanC: ignore it, i wanted to say from gerrit* related project not zuul (gerritlib) | 13:11 |
| *** nils has joined #zuul | 13:12 | |
| *** jpena|lunch is now known as jpena | 13:31 | |
| *** Goneri has joined #zuul | 13:36 | |
| *** bhavikdbavishi has joined #zuul | 14:15 | |
| *** bhavikdbavishi1 has joined #zuul | 14:18 | |
| *** bhavikdbavishi has quit IRC | 14:20 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 14:20 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add option to install kubernetes with kind https://review.opendev.org/c/zuul/zuul-jobs/+/740935 | 14:22 |
| fungi | anyone else interested in getting the zuul-jobs py27 job passing again? https://review.opendev.org/763866 | 14:25 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add option to install kubernetes with kind https://review.opendev.org/c/zuul/zuul-jobs/+/740935 | 14:25 |
| fungi | ianw: if you're still around, ttx commented on 763834 pointing out why it's not a problem (script explicitly invokes python3 anyway) | 14:27 |
| fungi | trying to finish transitioning those release jobs to python3, so if we do run into more corner cases where python 2.7 is being called we'll fix them anyway | 14:28 |
| avass | fungi: ++ | 14:32 |
| fungi | it's a bit of a game of whack-a-mole because those jobs only run in post-merge pipelines and interact with external resources (e.g. launchpad bugs in that particular case) | 14:36 |
| *** gouthamr_ has quit IRC | 14:36 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Pin keystoneauth1 and cachetools on older Python https://review.opendev.org/c/zuul/zuul-jobs/+/763866 | 14:57 |
| zbr | fungi: https://review.opendev.org/c/zuul/zuul-jobs/+/735923 part of py3 migration, slipped. | 15:06 |
| zbr | do we need to send emails about these? anything related to py2 seems very unstable these days. | 15:06 |
| *** mach1na has quit IRC | 15:27 | |
| fungi | it's worth discussing whether a blanket announcement that we're going to start changing any python defaults to python3 in coming weeks is sufficient | 15:30 |
| fungi | i still feel like giving people a heads up is useful, but i agree it's likely noise to do a separate announcement for every single case given there are likely to be at least a few | 15:31 |
| zbr | go for it! i have no authority to send such announcement :D | 15:33 |
| avass | I agree, a single announcement with clear intention that the jobs will default to python3 should be enough. | 15:33 |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: DNM - testing registry jobs https://review.opendev.org/c/zuul/zuul-jobs/+/764006 | 16:09 |
| *** hashar is now known as hasharAway | 16:11 | |
| pabelanger | mnaser: just confirming, on your lgtm pipeline, you have post-review: true set? | 16:27 |
| pabelanger | this has got me thinking, I wonder if we can limit secrets to specific pipelines | 16:27 |
| mnaser | pabelanger: I don’t but this job is defined and lives in a config project with final: true | 16:28 |
| pabelanger | mnaser: okay, that is what I was thinking | 16:29 |
| openstackgerrit | Tobias Henkel proposed zuul/nodepool master: Check for images to upload single threaded https://review.opendev.org/c/zuul/nodepool/+/743790 | 16:29 |
| pabelanger | I think our zuul-jobs-test-registry-* jobs are broken | 16:34 |
| pabelanger | it is complaining about an expired SSL cert | 16:34 |
| pabelanger | https://c5696350317ad68c03d1-7c6e098ee109e2cfc4ca8fa68868cc2d.ssl.cf5.rackcdn.com/764006/1/check/zuul-jobs-test-registry-podman/22905db/job-output.txt | 16:34 |
| pabelanger | x509: certificate has expired or is not yet valid | 16:35 |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Refresh intermediate TLS certs for testing https://review.opendev.org/c/zuul/zuul-jobs/+/764023 | 16:40 |
| avass | oh, uh, why don't we generate that in the job? | 16:41 |
| pabelanger | not sure, legacy reasons? | 16:41 |
| avass | could be | 16:41 |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Switch to container_images for push-to-intermediate-registry https://review.opendev.org/c/zuul/zuul-jobs/+/763836 | 16:42 |
| fungi | maybe somebody set the cert expiration as a reminder to look into generating it in the job later ;) | 16:42 |
| pabelanger | if it unblocks my testing, I can convert it to be more dynamic | 16:42 |
| -openstackstatus- NOTICE: The Gerrit service on review.opendev.org is being restarted quickly to troubleshoot an SMTP queuing backlog, downtime should be less than 5 minutes | 16:43 | |
| avass | corvus doesn't seem to have left an explanation: https://review.opendev.org/c/zuul/zuul-jobs/+/661327 | 16:45 |
| avass | maybe he just didn't think about generating it and caching it | 16:45 |
| *** hasharAway is now known as hashar | 16:46 | |
| avass | or maybe caching facts wasn't available in 2019 :) | 16:46 |
| *** timburke has quit IRC | 17:00 | |
| *** rpittau is now known as rpittau|afk | 17:01 | |
| *** jpena is now known as jpena|off | 17:02 | |
| *** rlandy is now known as rlandy|biab | 17:11 | |
| pabelanger | mnaser: neat! https://github.com/ansible-network/sandbox/pull/60#issuecomment-733120373 | 17:19 |
| pabelanger | mnaser: I forked locally and hacked in config file support, over env | 17:29 |
| pabelanger | mnaser: okay, I think I understand lgtm role now. Great work, I totally think we can start to add new features like having bot leave 'review comment' and even label changes. | 17:44 |
| pabelanger | but I won't work on that right now | 17:44 |
| *** iurygregory has quit IRC | 17:59 | |
| openstackgerrit | Tobias Henkel proposed zuul/nodepool master: Check for images to upload single threaded https://review.opendev.org/c/zuul/nodepool/+/743790 | 18:08 |
| *** sanjayu_ has quit IRC | 18:13 | |
| *** iurygregory has joined #zuul | 18:27 | |
| *** jcapitao has quit IRC | 18:38 | |
| fungi | ianw: when you're around again, can you see if the followup comments on 763834 are satisfactory? i'm waiting for that to land to retry broken openstack release jobs | 18:49 |
| *** nils has quit IRC | 18:53 | |
| *** rlandy|biab is now known as rlandy | 18:58 | |
| ianw | fungi: hrm, ok, so it's already called by /usr/bin/python3 <script> directly -- and it was working, which must mean that launchpadlib is actually already installed for python3 some other way? | 19:03 |
| *** bhavikdbavishi has quit IRC | 19:04 | |
| fungi | ianw: yeah, i expect so. we may wind up having to deduplicate it elsewhere in the consuming job later | 19:04 |
| fungi | it's unfortunately a big mess of shell scripts dating back to the jenkins job days so tough to follow in some places | 19:04 |
| ianw | fungi: ok, approved it; at least if there's version conflicts or something we have a place to start | 19:12 |
| fungi | thanks! and yep, we're slowly iterating on retrying it to get it working on focal (so that the distro package of git-review will work with opendev's new gerrit) | 19:15 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Update tests/base.py to use proper git data https://review.opendev.org/c/zuul/zuul/+/742746 | 19:16 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix gerrit merge commit change with zuul configuration https://review.opendev.org/c/zuul/zuul/+/762886 | 19:16 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix gerrit amended merge commit change with zuul configuration https://review.opendev.org/c/zuul/zuul/+/762887 | 19:16 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/c/zuul/zuul/+/742747 | 19:16 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/c/zuul/zuul/+/742748 | 19:16 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/c/zuul/zuul/+/739198 | 19:16 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/c/zuul/zuul/+/742749 | 19:16 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/c/zuul/zuul/+/739078 | 19:16 |
| openstackgerrit | Merged zuul/zuul-jobs master: Use Python 3.x with launchpadlib https://review.opendev.org/c/zuul/zuul-jobs/+/763834 | 19:23 |
| *** zenkuro has quit IRC | 20:03 | |
| *** y2kenny has joined #zuul | 20:16 | |
| *** hamalq has joined #zuul | 20:16 | |
| y2kenny | quick question about yaml formatting. For job definition Is there a difference between saying | 20:18 |
| y2kenny | pre-run: <path> | 20:18 |
| y2kenny | vs | 20:18 |
| y2kenny | pre-run: | 20:18 |
| y2kenny | - <path> | 20:18 |
| y2kenny | ? | 20:18 |
| tobiash | y2kenny: no | 20:19 |
| tristanC | y2kenny: iirc that attribute is converted as_list | 20:19 |
| y2kenny | I ran into a situation where the parent pre-run didn't get executed | 20:19 |
| y2kenny | an I am wondering if one get replaced | 20:19 |
| tobiash | it's just syntactic sugar that you don't have to specify a list | 20:19 |
| y2kenny | and* | 20:19 |
| y2kenny | so all the pre-run playbooks should get executed right? Only run playbook would be replaced by the child? | 20:20 |
| tobiash | yes | 20:20 |
| fungi | y2correct | 20:20 |
| y2kenny | ok thanks | 20:21 |
| fungi | looks like i somehow missed my tab key | 20:22 |
| *** wuchunyang has joined #zuul | 20:25 | |
| *** wuchunyang has quit IRC | 20:29 | |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Switch to container_images for push-to-intermediate-registry https://review.opendev.org/c/zuul/zuul-jobs/+/763836 | 20:44 |
| openstackgerrit | Daniel Blixt proposed zuul/zuul-jobs master: Use script to populate test file tree fixtures https://review.opendev.org/c/zuul/zuul-jobs/+/764062 | 20:48 |
| *** hamalq has quit IRC | 20:55 | |
| *** hashar has quit IRC | 20:59 | |
| *** hamalq has joined #zuul | 21:10 | |
| *** hamalq has quit IRC | 21:15 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: gerrit: restore change filter when querying https://review.opendev.org/c/zuul/zuul/+/764069 | 21:30 |
| *** rfolco|ruck has quit IRC | 21:32 | |
| *** Goneri has quit IRC | 22:08 | |
| openstackgerrit | Paul Belanger proposed zuul/zuul-jobs master: Switch to container_images for push-to-intermediate-registry https://review.opendev.org/c/zuul/zuul-jobs/+/763836 | 22:14 |
| *** ikhan has joined #zuul | 22:18 | |
| y2kenny | about the pause job (zuul_return, pause= true), the parent child relationship is between dependent job and not the other parent child relationship of pre and post run, is the correct? | 22:52 |
| y2kenny | (btw, how do you folks separate the two parent-child relationships usually? is there a formal name for each of the relationship?) | 22:52 |
| *** rfolco|ruck has joined #zuul | 22:57 | |
| clarkb | y2kenny: correct on the first thing, So you'd need to pause at the end of pre-run I think then let the child run its pre-run to nest | 22:58 |
| clarkb | y2kenny: what we've been doing a lot of us the artifact dependencies which is similar but different | 22:58 |
| clarkb | y2kenny: a job can declare it produces some named thing and others can depend on that | 22:58 |
| clarkb | then zuul figures it out | 22:58 |
| clarkb | (the actual storage of that thing is up to the job implementation) | 22:59 |
| y2kenny | ok | 22:59 |
| y2kenny | my use case is a bit different. Right now I am trying to have a parallel "thing" that monitor a baremetal serial console using IPMI serial over lan while tests are being done on the baremetal machine | 23:00 |
| y2kenny | I have been trying different "thing" but kind of stuck right now. | 23:00 |
| clarkb | y2kenny: would a multinode job solve that problem better? | 23:01 |
| y2kenny | yes about multinode and that's what I was thinking of doing initially | 23:01 |
| clarkb | you could have the job specify it needs the target and the monitor, configure them both, then start testing on the target and at the end the job collects from the monitor | 23:01 |
| y2kenny | but my understanding is that I can't get a different node from different providers? | 23:01 |
| y2kenny | or may be I misunderstood something earlier. | 23:02 |
| y2kenny | the baremetal node is currently provided by the cobbler thing that I wrote. I was actually thinking of launching a separate pod in the container cluster to do the monitoring | 23:03 |
| clarkb | oh ya I think that may still be the case | 23:04 |
| clarkb | though I think there has been talk of making it more flexible | 23:04 |
| y2kenny | I didn't quite figure out how to get that to work so I ended up using the executor/localhost and ansible async to do the monitoring. | 23:04 |
| clarkb | (we run into similar issues where we want to test arm64 but also have x86 for throughput) | 23:04 |
| y2kenny | so essentially I have a delegate to localhost task that launches the ipmitool to do the monitoring async, run the test and then close the monitoring | 23:05 |
| y2kenny | but then apparently I was exploiting a security hole :) | 23:05 |
| y2kenny | (since I was running the localhost call in an untrusted project pre-submit) | 23:06 |
| y2kenny | so then I refactored the monitoring to a parent job in a trusted project | 23:06 |
| y2kenny | but then I think the monitoring just die as the pre-run finish (because the execution is done inside the bubblewrap/container thing? I am speculating.) | 23:07 |
| y2kenny | this is how I was thinking of using the pause job thing but sounds like the pause job is for a different parent-child relationship. | 23:08 |
| y2kenny | (dependency relationship ship instead of the inheritance relationship... I am going to call them this until you guys correct me with an official vocabular :) ) | 23:09 |
| fungi | if someone works out clear terminology, please make sure it gets added to https://zuul-ci.org/docs/zuul/reference/glossary.html (assuming it's not there already) | 23:12 |
| fungi | we have "job dependency" and "provided/required artifact" documented there already | 23:13 |
| fungi | see which of those fits your mental model | 23:13 |
| y2kenny | fungi: ah yes, thanks. I am getting my words mix up because both job dependency and job inheritance have parent-child relationship) | 23:16 |
| fungi | yep, we've got "child/parent job" terms in there too | 23:17 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: use opendev-buildset-registry, buil tutorial image one time https://review.opendev.org/c/zuul/zuul/+/733005 | 23:17 |
| fungi | i tried to avoid saying "depend" when talking about inheritance so as to avoid people getting the concepts confused | 23:18 |
| *** Goneri has joined #zuul | 23:18 | |
| y2kenny | the glossary seems to use child/parent mostly for the job inheritance. But then the pause job documentation uses child job term for the dependent job relationship | 23:19 |
| y2kenny | https://zuul-ci.org/docs/zuul/reference/jobs.html#pausing-the-job | 23:19 |
| fungi | yeah, that may be worth some cleanup | 23:19 |
| fungi | i also have a long-standing to do list entry to go through and cross-link glossary terms from wherever they're used throughout the documentation | 23:20 |
| y2kenny | Does the doc on the website derive from here: https://review.opendev.org/plugins/gitiles/zuul/zuul/+/refs/heads/master ? | 23:22 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: use opendev-buildset-registry, buil tutorial image one time https://review.opendev.org/c/zuul/zuul/+/733005 | 23:22 |
| y2kenny | I can probably post some clean up after I figure out this baremetal monitoring thing | 23:23 |
| *** rlandy has quit IRC | 23:34 | |
| fungi | y2kenny: yes, https://opendev.org/zuul/zuul/raw/branch/master/doc/source/reference/jobs.rst | 23:34 |
| *** hamalq has joined #zuul | 23:37 | |
| *** holser has quit IRC | 23:38 | |
| *** holser has joined #zuul | 23:39 | |
| *** hamalq has quit IRC | 23:41 | |
| *** rfolco|ruck has quit IRC | 23:50 | |
| *** rfolco|ruck has joined #zuul | 23:50 | |
| *** hamalq has joined #zuul | 23:52 | |
| *** rfolco|ruck has quit IRC | 23:55 | |
| *** tosky has quit IRC | 23:57 | |
| *** hamalq has quit IRC | 23:57 | |
| *** openstackgerrit has quit IRC | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!