| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: zk: remove leaked FAILED or FULFILLED request. https://review.openstack.org/576287 | 00:18 |
|---|---|---|
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: zk: skip node already being deleted in cleanup leaked instance task https://review.openstack.org/576288 | 00:38 |
| *** sshnaidm_ has joined #zuul | 00:53 | |
| *** sshnaidm|afk has quit IRC | 00:55 | |
| *** sshnaidm__ has joined #zuul | 00:56 | |
| *** sshnaidm_ has quit IRC | 00:57 | |
| *** sshnaidm__ has quit IRC | 01:02 | |
| *** sshnaidm__ has joined #zuul | 01:16 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add PATH to `ip` command execution https://review.openstack.org/576274 | 02:30 |
| *** rlandy has quit IRC | 02:49 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Log output and exception when traceroute fails https://review.openstack.org/576351 | 05:00 |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Log output and exception when traceroute fails https://review.openstack.org/576351 | 05:30 |
| tobiash | jhesketh: do you want someone else looking on 575617? | 05:48 |
| *** gtema has joined #zuul | 06:47 | |
| *** hashar has joined #zuul | 07:14 | |
| *** sshnaidm__ is now known as sshnaidm | 07:25 | |
| *** Rohaan has joined #zuul | 07:30 | |
| *** jpena|off is now known as jpena | 07:33 | |
| *** logan- has quit IRC | 07:35 | |
| *** logan- has joined #zuul | 07:39 | |
| *** pcaruana has joined #zuul | 07:50 | |
| *** AJaeger has quit IRC | 07:58 | |
| openstackgerrit | Merged openstack-infra/nodepool master: Move QuotaInformation to driver utils https://review.openstack.org/572860 | 09:11 |
| openstackgerrit | Merged openstack-infra/nodepool master: fix tox python3 overrides https://review.openstack.org/574344 | 09:11 |
| *** Rohaan has quit IRC | 09:17 | |
| openstackgerrit | Merged openstack-infra/zuul master: Rename action-general to actiongeneral https://review.openstack.org/575617 | 09:23 |
| openstackgerrit | Merged openstack-infra/zuul master: Add missing __init__.py files https://review.openstack.org/575591 | 09:23 |
| openstackgerrit | Merged openstack-infra/zuul master: Remove failed_when when creating /tmp/console-None.log https://review.openstack.org/574672 | 09:23 |
| openstackgerrit | Merged openstack-infra/zuul master: Treat ::1 as localhost in addition to 127.0.0.1 https://review.openstack.org/572847 | 09:23 |
| *** flawi has quit IRC | 09:24 | |
| openstackgerrit | Merged openstack-infra/zuul master: Handle multiple stats in assertReportedStats https://review.openstack.org/572230 | 09:25 |
| openstackgerrit | Merged openstack-infra/zuul master: zuul-migrate: run / post-run playbooks need .yaml extensions https://review.openstack.org/572160 | 09:25 |
| *** Rohaan has joined #zuul | 09:27 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Add tenant yaml validation option to scheduler https://review.openstack.org/574265 | 10:37 |
| *** electrofelix has joined #zuul | 10:42 | |
| *** jpena is now known as jpena|lunch | 11:07 | |
| *** sshnaidm is now known as sshnaidm|lnch | 11:31 | |
| *** jpena|lunch is now known as jpena | 12:00 | |
| *** sshnaidm|lnch is now known as sshnaidm | 12:11 | |
| *** _ari_ has quit IRC | 12:13 | |
| *** _ari_ has joined #zuul | 12:13 | |
| *** rlandy has joined #zuul | 12:17 | |
| mordred | tobiash, corvus, dmsimard: https://review.openstack.org/#/c/551989/ is green again (yay for symlinks and ansible 2.5) | 12:20 |
| tobiash | mordred: then let me check the autoscroll checkbox... | 12:21 |
| mordred | also, https://review.openstack.org/#/c/572588 should be ready after it - I just gave it a recheck | 12:22 |
| tobiash | mordred: http://logs.openstack.org/89/551989/43/check/zuul-build-dashboard/f3ee16a/npm/html/stream.html?uuid=51377e6b55b9443fb5e7082da759e191&logfile=console.log | 12:23 |
| tobiash | mordred: ugly but functional now :) | 12:23 |
| *** goern has joined #zuul | 12:23 | |
| mordred | tobiash: yah- I'm hoping that once we land this giant thing more people like mnaser can come in and make improvements | 12:24 |
| tobiash | mordred: yah so that's no blocker for landing that stack | 12:24 |
| tobiash | mordred: gave +1 for now as I'm not an expert in that domain | 12:27 |
| mordred | tobiash: kk. cool - thanks! | 12:28 |
| tobiash | mordred: in case my +2 is needed I can raise it once more people looked at it ;) | 12:28 |
| *** sshnaidm is now known as sshnaidm|afk | 12:29 | |
| mordred | tobiash: :) ... everybody loves javascript :) | 12:29 |
| *** sshnaidm__ has joined #zuul | 12:29 | |
| goern | hey all, has someone taken a look how to integrate kubernetes/openshift into zuul? I would like to run containers on an OpenShift cluster rather than doing stuff on nodes... | 12:36 |
| tristanC | goern: i have a few patches for openshift: https://review.openstack.org/#/q/topic:openshift-zuul-build-resource | 12:38 |
| tristanC | goern: and a more simple nodepool driver for k8s: https://review.openstack.org/535557 | 12:39 |
| goern | tristanC, the 2nd patch set will introduce a kubernetes namespace as a node available in nodepool? | 12:40 |
| tristanC | goern: that's the openshift-zuul-build-resource topic. the "simple" k8s driver creates a pod with a sshd process to re-use regular zuul-jobs | 12:42 |
| goern | I see, will you follow both approaches or will just one make it into a release (some day)? | 12:43 |
| goern | tristanC, ^^^ | 12:43 |
| tristanC | goern: please note those are experimental prototype waiting for the container spec to be finished: https://review.openstack.org/#/c/560136/ | 12:43 |
| tristanC | once this specs is approved, i'd like to follow both approaches yes | 12:46 |
| goern | tristanC, nice! that sounds like what I am looking for | 12:48 |
| goern | tristanC, what do I need to do to deploy such an enhanced zuul? :) | 12:48 |
| Shrews | tristanC: interesting find on 576287. Rather than cleaning up after the bug, let's try to find what's causing it | 12:50 |
| tristanC | Shrews: yes, thanks for having a look, i mostly pushed the review to keep it under my radar. it's a big wip that needs better investigation indeed | 12:50 |
| Shrews | tristanC: sure. if you can provide me test case to reproduce it, i can help | 12:51 |
| tristanC | Shrews: i suspect hard reboot of the nodepool-launcher may be the cause | 12:51 |
| tristanC | goern: applying the patches on a vanilla zuul should enable this new feature | 12:52 |
| *** elyezer has quit IRC | 12:53 | |
| goern | tristanC, so I'll start with deploying zuul from source... | 12:53 |
| *** elyezer has joined #zuul | 12:54 | |
| *** dkranz has quit IRC | 12:59 | |
| *** dkranz has joined #zuul | 13:00 | |
| *** myoung|off is now known as myoung | 13:06 | |
| tristanC | goern: i also wrote a prototype guide to experiment with this new feature using software factory: https://softwarefactory-project.io/draft/zuul-openshift/ | 13:08 |
| tristanC | goern: but again this is all very experimental, and we need to agree on the container spec (560136) first | 13:08 |
| *** sshnaidm__ is now known as sshnaidm | 13:08 | |
| goern | tristanC, I am good with Experiment quality of things, right now I am trying to figure if there is a better way to build software than having Jenkins on OpenShift, zuul/project-gating feels more natural... | 13:12 |
| mordred | goern: ++ | 13:17 |
| mordred | goern: we're excited for the possibilities | 13:17 |
| *** sshnaidm is now known as sshnaidm|afk | 13:21 | |
| rcarrillocruz | yas indeed! | 13:25 |
| rcarrillocruz | \o/ | 13:25 |
| dmsimard | mordred: awesome work on the dashboard, it's a shame that it's more or less no-op in terms of frontend :) | 13:58 |
| mordred | dmsimard: well - hopefully we'll be set up to make improvements | 14:01 |
| dmsimard | mordred: yup, let's call it an investment :) | 14:01 |
| mordred | ++ | 14:01 |
| dmsimard | it'll pay off | 14:01 |
| tristanC | dmsimard: it's a bit improvement for the frontend, each page now doesn't reload the whole bundle. it feels much more snappy | 14:02 |
| tristanC | s/bit/noticable/ | 14:03 |
| dmsimard | tristanC: ah, you mean it won't hang up my entire firefox every few seconds? :) | 14:03 |
| mordred | dmsimard: it still might do that :) | 14:03 |
| tristanC | dmsimard: i meant when you click navigation buttons | 14:03 |
| dmsimard | mordred: to be fair it's only when the queues are really backed up | 14:03 |
| dmsimard | for some reason I don't remember that issue from back when I used chrome | 14:04 |
| mordred | dmsimard: yah | 14:04 |
| mordred | dmsimard: although even with that - hopefully soon we can get the status page to be reactive so that there's not hangs when it's super backed up | 14:05 |
| dmsimard | nice | 14:05 |
| dmsimard | I have a lot of of respect for people who can write javascript :) | 14:05 |
| *** Rohaan has quit IRC | 14:07 | |
| mnaser | mordred, dmsimard: yeah if i can find some spare time (or a flight soon) i'll probably work on something like that | 14:10 |
| mnaser | also one thing that i wanted to work on too was getting rid of the 'STREAM END' if the streamer hasnt started yet | 14:10 |
| mnaser | i'd love for it to just wait until the streamer starts, avoiding constant f5's | 14:10 |
| dmsimard | mnaser: I took a stab at that but apparently it didn't fully resolve it | 14:10 |
| * mnaser has learned to push js patches in doses | 14:11 | |
| mnaser | http://logs.openstack.org/88/572588/3/check/zuul-build-dashboard/43f8e48/npm/html/status.html looks so cleeeean | 14:11 |
| dmsimard | mnaser: http://git.zuul-ci.org/cgit/zuul/commit/?id=57965e33f337e74fb648300d07eabe0d0d75b369 | 14:12 |
| mnaser | dmsimard: i see that sometimes, i think this has to be a client side thing | 14:12 |
| mnaser | because it's before that | 14:12 |
| dmsimard | mnaser: the link should probably not be made clickable until the console is /really/ ready | 14:13 |
| dmsimard | unless we want a feature when the job console is "clickable" even before it starts and then it automatically attaches or something | 14:13 |
| mnaser | yeah a lot of ci tools do that | 14:13 |
| mnaser | where they just kinda spin waiting for job to start | 14:13 |
| gtema | mordred, anybody: any chance to finally review https://review.openstack.org/#/c/572829/? | 14:25 |
| dmsimard | btw, might be worth keeping an eye on if you're interested -- multiple role in a single repository support for galaxy: https://github.com/ansible/galaxy/issues/244 | 14:36 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Improve test case test_unprotected_branches https://review.openstack.org/576536 | 14:37 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Move exclude unprotected branches check into tenant https://review.openstack.org/576537 | 14:37 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Don't reconfigure tenant on unprotected branch pushes https://review.openstack.org/576538 | 14:37 |
| *** acozine1 has joined #zuul | 14:37 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Don't reconfigure tenant on unprotected branch pushes https://review.openstack.org/576538 | 14:53 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul master: Add a CVE-2018-12557 release note https://review.openstack.org/576545 | 14:59 |
| *** pcaruana has quit IRC | 15:29 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul master: Add a CVE-2018-12557 release note https://review.openstack.org/576545 | 15:36 |
| *** sshnaidm|afk is now known as sshnaidm | 15:41 | |
| *** myoung is now known as myoung|lunch | 16:05 | |
| *** sshnaidm is now known as sshnaidm|afk | 16:08 | |
| *** gtema has quit IRC | 16:39 | |
| tobiash | fungi: thanks | 16:56 |
| fungi | thanks for reporting! and this has given me some practical insights to incorporate into the vulnerability management process i'm drafting | 16:59 |
| *** pcaruana has joined #zuul | 17:05 | |
| mwhahaha | ahoy zuul folks, question about how overrides work. context: https://review.openstack.org/#/c/576587/2/.zuul.yaml | 17:14 |
| mwhahaha | i'm attempting to limit a job to a specific set of branches but i am also trying to override a child job to have a different branches config | 17:15 |
| mwhahaha | it looks like the child job override isn't taking so the parent one is winning and the child job isn't run | 17:15 |
| ssbarnea | is any zuul/gerrit client that i could use to get quick notification regarding test results? i don't follow email due to its spammy nature, but i do want to watch some specific changes, across multiple gerrit servers. | 17:16 |
| tobiash | ssbarnea: there is an android app which makes it easy to look at changes | 17:20 |
| pabelanger | ssbarnea: mqtt publisher might help with that, we recently added support. But I don't think we have a out of the box client yet. Best today is using the status page | 17:20 |
| tobiash | ssbarnea: but I don't know it it can notify | 17:21 |
| pabelanger | tobiash: woah, really? | 17:21 |
| tobiash | yeah :) | 17:21 |
| tobiash | it's named Rview | 17:21 |
| ssbarnea | who, knows, i may endup creating something that exposes results in cc.xml format | 17:21 |
| pabelanger | that's is also why I wanted to add fedmsg support into zuul, to pick up all its client | 17:21 |
| tobiash | and it's even usable to do real reviews on smaller changes | 17:21 |
| ssbarnea | well, i am not on the Android userbase myself, more on the apple Apple, so not much luck here. but it would have being cool to also have a mobile app, for doing reviews while being on... remote areas :) | 17:24 |
| tobiash | ssbarnea: unfortunately I didn't find an appropriate ios app for my ipad | 17:30 |
| ssbarnea | tobiash: i think that the trick is to find something generic that can deal with this kind of notifications and to build middleware that does watch multiple gerrit servers and exposes the results for the generic client to consume. it may even work for watching bugs. | 17:31 |
| tobiash | ssbarnea: zuul now has a mqtt reporter | 17:32 |
| tobiash | so you could hook into that | 17:32 |
| ssbarnea | taskwarrior / bugwarrior was something I was considering for that. | 17:32 |
| ssbarnea | i am still in exploration phase | 17:32 |
| *** myoung|lunch is now known as myoung | 17:32 | |
| tobiash | mwhahaha: the branches attribute isn't inherited in the normal manner | 17:37 |
| mwhahaha | tobiash: so you can't override it? like that | 17:37 |
| tobiash | mwhahaha: that selects if that variant matches | 17:37 |
| tobiash | mwhahaha: https://zuul-ci.org/docs/zuul/user/config.html#attr-job.branches | 17:37 |
| tobiash | mwhahaha: I think it should work if you could remove the matchers in the base job | 17:38 |
| mwhahaha | k i'll try that | 17:38 |
| mwhahaha | not ideal but workable | 17:38 |
| tobiash | but I'm not that deep into the branching topic | 17:38 |
| fungi | pabelanger: ssbarnea: regarding mqtt publishing, if it's test results you care about then we've had the gerrit event stream replicated into the firehose for over a year so could filter on that | 17:41 |
| tobiash | mwhahaha: you want to expand the list of branches in the child job? | 17:41 |
| fungi | though also not a solution for any gerrits besides ours | 17:41 |
| mwhahaha | tobiash: so the child job needs to run on master, while all the other jobs that inherit from the parent shouldn't. so i'm just in this case i was trying to define the limit on the parent job to limit all the other job definitions except specific jobs | 17:42 |
| mwhahaha | tobiash: so i can just apply the branches to all the child job definitions since the inheritance doesn't work, i was just trying to leverage inheritance to achieve my specific desired config | 17:43 |
| clarkb | mwhahaha: tobiash aiui the zuulv3 way would be to not use explicit branches matchers in this case but to define the job in the appropriate pipeline on the branch you want it to run in | 17:43 |
| clarkb | then each branch explicitly manages its own config | 17:43 |
| tobiash | mwhahaha: so the case why your job isn't run is that the child matches but doesn't find a non-filtered parent | 17:43 |
| mwhahaha | clarkb: i'm trying to not have to backport a bunch of stuff, it gets unwieldy with so many repos but i can look into that as well. | 17:44 |
| mordred | mwhahaha: you could make a base job, marked abstract and protected, without any matchers, then make your current child job a child job of it, and make another child job of the parent job that has the matchers you're wanting to put onto the other versions | 17:44 |
| mwhahaha | mordred: that's what it is currently structured as, except it's not currently marked abstract | 17:45 |
| mwhahaha | because these job defs predate some of that | 17:45 |
| mwhahaha | i'll try a few things then | 17:45 |
| *** electrofelix has quit IRC | 18:27 | |
| mwhahaha | is there anyway to see the merged zuul config, trying to figure out why a job is getting triggered even though i moved the branches designator to exclude master | 18:48 |
| mwhahaha | hmm looks like the running config doesn't like it when you remove a branches in the change and move it. it's still processing the original definition it seems | 19:09 |
| *** pcaruana has quit IRC | 20:29 | |
| *** hashar has quit IRC | 20:36 | |
| *** acozine1 has quit IRC | 21:03 | |
| *** myoung is now known as myoung|off | 21:59 | |
| *** yolanda has quit IRC | 22:43 | |
| *** yolanda has joined #zuul | 22:45 | |
| *** rlandy has quit IRC | 23:34 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!