*** nickx-intel has quit IRC | 01:38 | |
SpamapS | pabelanger:https://review.opendev.org/#/c/656541/ adds support for Zuul to submit reviews. I believe you were looking to make use of something like that. I plan to use it to allow self-approve with "requires review" | 01:45 |
---|---|---|
*** bhavikdbavishi has joined #zuul | 01:53 | |
*** sshnaidm|ptg has joined #zuul | 02:11 | |
*** bhavikdbavishi1 has joined #zuul | 02:18 | |
*** bhavikdbavishi has quit IRC | 02:20 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 02:20 | |
openstackgerrit | Paul Belanger proposed zuul/nodepool master: DNM: Switch to fedora-30 for nodepool tests https://review.opendev.org/656608 | 03:01 |
*** rfolco|ruck is now known as rfolco|pto | 03:15 | |
*** bhavikdbavishi has quit IRC | 04:16 | |
*** bhavikdbavishi has joined #zuul | 04:17 | |
*** sshnaidm|ptg has quit IRC | 04:38 | |
*** bjackman__ has joined #zuul | 04:45 | |
*** bhavikdbavishi has quit IRC | 04:54 | |
*** bhavikdbavishi has joined #zuul | 04:54 | |
bjackman__ | I have a a job defined in project A, its required-projects is [B, C]. The job passes in the pipeline for A. In project B I add the job to a pipeline the job starts OK, but when I lookup zuul.projects[A.canonical_name] it fails | 05:02 |
bjackman__ | Reading the docs in Lawyer Mode, I think this behaviour is actually correct; I don't see any statement that the project where a job is defined is automatically included | 05:03 |
bjackman__ | But it seems that it should be! | 05:03 |
bjackman__ | Am I missing something? | 05:03 |
bjackman__ | (Not a big deal because presumably I can just add A to the job's required-projects and it will fix it - will find out if this works later today) | 05:04 |
tristanC | bjackman__: for project like zuul/zuul-jobs, it makes sense to not auto require the project | 05:45 |
*** bhavikdbavishi has quit IRC | 06:25 | |
*** bhavikdbavishi has joined #zuul | 07:08 | |
*** pcaruana has joined #zuul | 07:14 | |
*** quiquell|off is now known as quiquell|ptg | 07:56 | |
*** themroc has joined #zuul | 08:12 | |
*** themroc has quit IRC | 08:15 | |
*** bhavikdbavishi has quit IRC | 09:23 | |
*** bhavikdbavishi has joined #zuul | 09:35 | |
bjackman__ | tristanC, why is that? Because it's assumed to be included via job.roles anyway? | 10:05 |
tristanC | bjackman__: because the job playbooks doesn't need the zuul-jobs project copied to the workspace | 10:09 |
bjackman__ | Ahh right yeah it's needed on the executor host but not on the test node | 10:13 |
bjackman__ | tristanC, I see, thanks | 10:13 |
*** themroc has joined #zuul | 11:11 | |
*** bhavikdbavishi has quit IRC | 11:27 | |
*** bhavikdbavishi has joined #zuul | 11:38 | |
*** themroc has quit IRC | 12:01 | |
*** bhavikdbavishi has quit IRC | 12:06 | |
*** bhavikdbavishi has joined #zuul | 12:28 | |
*** bjackman__ has quit IRC | 12:31 | |
*** bjackman has joined #zuul | 12:33 | |
*** ianychoi_ has quit IRC | 12:35 | |
*** rlandy has joined #zuul | 12:35 | |
*** ianychoi_ has joined #zuul | 12:35 | |
*** bjackman has quit IRC | 12:44 | |
*** pcaruana has quit IRC | 12:49 | |
*** pcaruana has joined #zuul | 12:54 | |
*** ianychoi_ has quit IRC | 13:00 | |
*** bhavikdbavishi has quit IRC | 13:14 | |
*** ianychoi has joined #zuul | 13:42 | |
*** zbr|rover has quit IRC | 14:27 | |
*** zbr has joined #zuul | 14:30 | |
*** zbr is now known as zbr|rover | 14:35 | |
*** mattw4 has joined #zuul | 15:01 | |
clarkb | corvus: memory use remains stable. I think we should make a release (likely 3.8.1 as I think only the bugfix and test fixes have really gotten in) | 15:21 |
pabelanger | clarkb: we likely want to finish off reviews on https://review.opendev.org/655805/ (depends-on) then also land https://review.opendev.org/655804 (with timeout bump to 3600). So far that has been 10 rechecks (100 tox jobs) without a failure | 15:28 |
clarkb | pabelanger: that only affects tests so I d ont think we need it in the release (which should fix the memory leak) | 15:33 |
pabelanger | true | 15:33 |
pabelanger | but, unsure how stable this current are without those. | 15:33 |
pabelanger | we have made improvements since last week however | 15:33 |
pabelanger | that just lets us unblock merging new features I believe | 15:34 |
clarkb | d7b3d43557d3658a5d57f28a7b82121380b637ed "Recover cached repos from corrupt object files" would also be in the next release and a13d92209daf1c6fae1dc6447cd13202e267db22 'Revert "Prepend path with bin dir of ansible virtualenv"' | 15:50 |
clarkb | not sure if that means we need a 3.9.0 | 15:51 |
pabelanger | maybe, the virtuelenv revert could break people that were using ara, from a virutalenv | 15:51 |
pabelanger | but that was how it worked in 3.7.0 | 15:52 |
clarkb | pabelanger: but not in 3.8 | 15:53 |
clarkb | not sure if a big enough change to matter | 15:53 |
pabelanger | yah, agree | 15:55 |
*** tjgresha has joined #zuul | 16:21 | |
*** sshnaidm has joined #zuul | 16:28 | |
*** rfolco has joined #zuul | 16:29 | |
*** rfolco|pto has quit IRC | 16:29 | |
*** sshnaidm has quit IRC | 16:40 | |
openstackgerrit | Merged zuul/nodepool master: Fix leaked loggers https://review.opendev.org/656575 | 17:09 |
*** mattw4 has quit IRC | 17:11 | |
Shrews | tobiash: i've restarted the openstack nodepool launchers with your mem leak fix. Will monitor the memory usage | 17:36 |
*** mattw4 has joined #zuul | 17:49 | |
*** sshnaidm has joined #zuul | 17:53 | |
pabelanger | heh, tox-nodejs-npm is hard coded to use ubuntu | 17:55 |
pabelanger | we should fix that | 17:55 |
*** jamesmcarthur has joined #zuul | 18:10 | |
*** jamesmcarthur has quit IRC | 18:44 | |
pabelanger | Shrews: tobiash: if that goes well over next few days, I take it we are going to tag a release of nodepool? | 18:50 |
*** jamesmcarthur has joined #zuul | 18:52 | |
*** jamesmcarthur has quit IRC | 19:02 | |
Shrews | pabelanger: I would expect we could | 19:04 |
*** jamesmcarthur has joined #zuul | 19:04 | |
*** sshnaidm has quit IRC | 19:05 | |
tobiash | ++ | 19:08 |
*** jamesmcarthur has quit IRC | 19:09 | |
pabelanger | great | 19:11 |
*** jamesmcarthur has joined #zuul | 19:14 | |
*** jamesmcarthur has quit IRC | 19:17 | |
*** jamesmcarthur has joined #zuul | 19:19 | |
* SpamapS has an idea to make a role which enables devs to ssh in to their test nodes | 20:00 | |
pabelanger | SpamapS: I think sf folks might already have something like that | 20:26 |
corvus | can haz in zuul jobs pls? | 20:27 |
pabelanger | tristanC: dmsimard: ^or did I mis-remember that | 20:29 |
*** pcaruana has quit IRC | 20:42 | |
*** jamesmcarthur has quit IRC | 20:57 | |
*** jamesmcarthur has joined #zuul | 21:14 | |
SpamapS | Yeah srrsly | 21:24 |
*** jamesmcarthur has quit IRC | 21:51 | |
*** jamesmcarthur has joined #zuul | 21:52 | |
corvus | https://www.theregister.co.uk/2019/04/30/kata_containers_zuul/ | 21:56 |
*** jamesmcarthur has quit IRC | 21:57 | |
*** jamesmcarthur has joined #zuul | 21:59 | |
paladox | that's awesome corvus! | 22:04 |
* paladox passes that news onto the wmf releng team! | 22:05 | |
*** rlandy is now known as rlandy|bbl | 22:18 | |
*** tjgresha_nope has joined #zuul | 22:22 | |
*** tjgresha has quit IRC | 22:25 | |
*** jamesmcarthur has quit IRC | 22:30 | |
dmsimard | pabelanger, SpamapS: https://review.opendev.org/#/c/562321/ ? | 22:34 |
dmsimard | corvus: I'm cheering live from Canada, good job :) | 22:35 |
SpamapS | dmsimard:nope, that's a much more complete solution. I just want to let devs drop an SSH key in the repo and have this role pick it up in post, install it as an auth key, print out connection details, and then wait for the timeout of the job. | 22:57 |
SpamapS | that way they can SSH in and attach debuggers, read logs, etc. etc. | 22:57 |
fungi | zuul project update at the denver open infra summit is starting in ~5 minutes | 23:02 |
*** cosss_ has joined #zuul | 23:43 | |
tristanC | SpamapS: https://review.opendev.org/535803 ? | 23:46 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!