| * SpamapS catches mnaser's table adds an 'a', and flips it back on its feet | 01:00 | |
| *** elyezer has quit IRC | 01:50 | |
| *** elyezer has joined #zuul | 01:52 | |
| *** neilsun has joined #zuul | 02:58 | |
| *** elyezer has quit IRC | 03:06 | |
| *** elyezer has joined #zuul | 03:07 | |
| *** elyezer has quit IRC | 03:13 | |
| *** elyezer has joined #zuul | 03:16 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-sphinx master: Add config option to find other roles https://review.openstack.org/593477 | 03:22 |
|---|---|---|
| *** elyezer has quit IRC | 03:31 | |
| *** elyezer has joined #zuul | 03:37 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-sphinx master: Open role readme files in utf-8 mode https://review.openstack.org/579474 | 04:33 |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-sphinx master: Add config option to find other roles https://review.openstack.org/593477 | 04:33 |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-sphinx master: Add config option to find other roles https://review.openstack.org/593477 | 04:42 |
| *** elyezer has quit IRC | 05:45 | |
| *** elyezer has joined #zuul | 05:53 | |
| *** aluria has joined #zuul | 06:18 | |
| *** elyezer has quit IRC | 06:21 | |
| *** elyezer has joined #zuul | 06:25 | |
| *** pcaruana has joined #zuul | 06:46 | |
| *** elyezer has quit IRC | 07:00 | |
| *** elyezer has joined #zuul | 07:01 | |
| *** jpena|off is now known as jpena | 07:27 | |
| *** elyezer has quit IRC | 07:55 | |
| *** elyezer has joined #zuul | 07:57 | |
| *** snapiri has joined #zuul | 08:02 | |
| *** electrofelix has joined #zuul | 08:28 | |
| *** hashar has joined #zuul | 08:30 | |
| *** rcarrillocruz has joined #zuul | 09:20 | |
| *** sshnaidm is now known as sshnaidm|afk | 09:29 | |
| *** jpena is now known as jpena|lunch | 11:03 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: WIP web: rewrite interface in react https://review.openstack.org/591604 | 11:13 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: WIP web: rewrite interface in react https://review.openstack.org/591604 | 11:28 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: fix multiple builds query parameters for a single column https://review.openstack.org/593605 | 12:00 |
| *** panda is now known as panda|lunch | 12:03 | |
| *** sshnaidm|afk is now known as sshnaidm | 12:04 | |
| *** jpena|lunch is now known as jpena | 12:19 | |
| *** needssleep is now known as TheJulia | 12:23 | |
| *** rlandy has joined #zuul | 12:36 | |
| *** gtema has joined #zuul | 13:05 | |
| *** elyezer has quit IRC | 13:08 | |
| *** elyezer has joined #zuul | 13:09 | |
| *** elyezer has quit IRC | 13:09 | |
| *** elyezer has joined #zuul | 13:16 | |
| SpamapS | https://storyboard.openstack.org/#!/story/2003492 ... hit this yesterday... took a bit to debug | 13:16 |
| tristanC | SpamapS: oh, https://review.openstack.org/583841 | 13:21 |
| SpamapS | It also happens when the github app hasn't been installed btw. | 13:51 |
| SpamapS | Found that out after creating the config repo and it kept spewing | 13:51 |
| * SpamapS has a zuul running in his k8s! | 13:53 | |
| SpamapS | http://paste.openstack.org/show/728425/ | 13:54 |
| SpamapS | Now to figure out a) nodepool, b) why gearman doesn't like my certs | 13:54 |
| *** jlv-sick is now known as jlvillal | 14:00 | |
| mnaser | so | 14:04 |
| mnaser | something i found over the past few days when trying zuul was there is a few assumptions | 14:05 |
| mnaser | ensure-tox does pip install --user tox | 14:05 |
| mnaser | however the tox role tries to look for it in the /usr/bin/tox or whatever, so it fails | 14:05 |
| mnaser | so ensure-tox is not really doing what it's supposed to do, and because in openstack world we ship images with tox, it just skips through it | 14:05 |
| mnaser | i changed our nodepool images to add tox.. but it'd be nice to avoid that, though not sure what the best way would be (maybe a set_fact in ensure-tox ?) | 14:06 |
| pabelanger | mnaser: yah, ran into that issue too for summit training. I baked tox into the image also. Couldn't figure out the best way to check for both ~/.local and /usr/local tox | 14:08 |
| SpamapS | mnaser: I've had a lot of problems with the assumptions around the images too. | 14:09 |
| SpamapS | mnaser: but I think I submitted fixes for most of them. Maybe not tox. | 14:09 |
| mnaser | yeah so if the node doesnt have tox, the job will pretty much fail, but i wasnt sure what the cleanest way to do it was | 14:09 |
| SpamapS | I think fixing the tox roles to not assume /usr/bin/tox would be ideal | 14:10 |
| SpamapS | they *should* just be running 'tox' | 14:10 |
| SpamapS | and let pathing do its job. | 14:10 |
| SpamapS | whoops, forgot to upload my custom nodepool | 14:11 |
| SpamapS | RuntimeError: aws: unknown driver | 14:11 |
| mnaser | gross | 14:11 |
| SpamapS | tristanC: ^^ I'll be testing your driver soon. :) | 14:11 |
| mnaser | :p | 14:11 |
| SpamapS | mnaser: SO gross | 14:11 |
| mnaser | (totally cool tho) | 14:12 |
| SpamapS | not to worry, you're on my list for demonstration-of-portability :-D | 14:12 |
| mnaser | i think using openstack is cool bc you just point your nodepool to another endpoint and call it a day if you have to move | 14:12 |
| mnaser | yeah i guess we'll just have to add ~/.local/ to $PATH for tox | 14:12 |
| mnaser | and that'll be a relatively simple fix | 14:12 |
| SpamapS | since so far, the only aws-specific thing in this entire k8s deploy is the loadbalancer object I created for zuul-web. | 14:12 |
| SpamapS | mnaser: Vexx has magnum right? Can the k8s's that Magnum spins up use Octavia loadbalancers? | 14:12 |
| mnaser | SpamapS: yes i can | 14:13 |
| mnaser | er | 14:13 |
| mnaser | it can* | 14:13 |
| SpamapS | yes we can! | 14:13 |
| SpamapS | :-D | 14:13 |
| mnaser | it spins up k8s that's fully integrated with octavia/cinder/etc | 14:13 |
| SpamapS | nice | 14:13 |
| mnaser | so you can even create things like pvc's (i.e. for git storage, etc) | 14:13 |
| SpamapS | yeah I want to show that by using k8s we get portability without resorting to gross lowest-common-denominator tactics | 14:14 |
| mnaser | yeah "get me a k8s cluster" works pretty much most places | 14:16 |
| tristanC | SpamapS: cool, though it needs to be adapted for the "simplified driver api", please check https://review.openstack.org/#/c/535557/7..8 for the changes needed | 14:25 |
| *** panda|lunch is now known as panda | 14:25 | |
| *** myoung is now known as myoung|training | 14:31 | |
| *** shachar has joined #zuul | 14:40 | |
| *** snapiri has quit IRC | 14:40 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Implement an Amazon EC2 driver https://review.openstack.org/535558 | 14:42 |
| *** shachar is now known as snapiri | 14:42 | |
| tristanC | SpamapS: there, this version should work with https://review.openstack.org/#/c/568704/ | 14:42 |
| *** jiapei has joined #zuul | 14:48 | |
| *** pcaruana has quit IRC | 15:11 | |
| rcarrillocruz | hmm | 15:34 |
| rcarrillocruz | i just realized i could use metadata on cloud-images to derive logic later in jobs | 15:35 |
| rcarrillocruz | but i don't see meta section in cloud-images | 15:35 |
| rcarrillocruz | /sad trombone | 15:35 |
| pabelanger | +1 | 15:35 |
| rcarrillocruz | pabelanger: e.g. we upload vqfx image, and we put as metadata 'ansible_network_os: junos', so we can then derive from it what module to use, rather than tying that value as nodeset host-vars | 15:36 |
| rcarrillocruz | it's not terribly bad, but... would be nice to use that pattern the same as nodepool 'user' or nodepool 'connection-type' | 15:37 |
| pabelanger | rcarrillocruz: what's the downside of using host-vars in the nodeset? | 15:38 |
| rcarrillocruz | none, just consistency... nodepool image thing vs zuul nodeset attr | 15:39 |
| pabelanger | speaking of images, was trying to think of a way to detect if a playbook was running inside of zuul-job. And so far, best idea I had was looking to zuul_console process. | 16:01 |
| *** jpena is now known as jpena|off | 16:01 | |
| pabelanger | was also thinking, maybe have ever zuul drop down a /.zuulenv file, which maybe contains version info too | 16:03 |
| pabelanger | shrugs | 16:03 |
| *** hashar has quit IRC | 16:03 | |
| mordred | pabelanger: I wouldn' tlook for zuul_console - its existence is a thing we want to make go away | 16:09 |
| mordred | pabelanger: however, the 'zuul' variable will be set, you could look for that | 16:09 |
| mordred | pabelanger: like - when: zuul.executor.hostname is defined - for instance | 16:10 |
| pabelanger | mordred: at this point in time, I'm actually in testinfra, so was trying to avoid passing in an ansible variable into it, mostly to keep tests simple | 16:11 |
| *** gtema has quit IRC | 16:11 | |
| mordred | pabelanger: oh - gotcha | 16:12 |
| pabelanger | for now, I'm looking at /etc/ci/mirror_info.sh, but not all zuuls use that file | 16:12 |
| mordred | pabelanger: I misunderstood the context | 16:12 |
| pabelanger | np, I didn't really explain it well to start :) | 16:12 |
| *** hashar has joined #zuul | 16:15 | |
| pabelanger | mordred: corvus: tobiash: do you mind adding https://review.openstack.org/591457/ to review queue, would like to see if we could promote the change up for emit-job-header and node info | 16:18 |
| *** sshnaidm is now known as sshnaidm|off | 16:58 | |
| *** neilsun has quit IRC | 17:05 | |
| *** trishnag has quit IRC | 17:27 | |
| *** hashar is now known as hasharDinner | 17:32 | |
| *** trishnag has joined #zuul | 17:51 | |
| *** electrofelix has quit IRC | 17:59 | |
| *** jiapei has quit IRC | 18:08 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul master: Add container push job https://review.openstack.org/593125 | 18:24 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul master: Add container push job https://review.openstack.org/593125 | 18:25 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul master: Add container push job https://review.openstack.org/593125 | 18:27 |
| Shrews | 3rd time's the charm (maybe) | 18:27 |
| pabelanger | exciting! | 18:27 |
| Shrews | or the 10th (if you count last week's pushes) | 18:27 |
| pabelanger | Shrews: left comment on parent change | 18:34 |
| Shrews | pabelanger: yes! great comments. will fix it up | 18:34 |
| Shrews | pabelanger: mordred suggested protected=true, too (which i forgot) | 18:36 |
| *** hasharDinner is now known as hashar | 18:58 | |
| *** elyezer has quit IRC | 19:10 | |
| *** elyezer has joined #zuul | 19:23 | |
| *** EvilienM is now known as EmilienM | 19:31 | |
| pabelanger | could I get a +3 on new zuul jobs: https://review.openstack.org/593150/ tox-with-sudo | 19:37 |
| *** timburke has quit IRC | 19:58 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: htmlify: don't log https://review.openstack.org/593901 | 20:00 |
| *** trishnag has quit IRC | 20:08 | |
| *** timburke has joined #zuul | 20:09 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: htmlify: don't log https://review.openstack.org/593901 | 20:31 |
| openstackgerrit | Merged openstack-infra/zuul master: Map file comment line numbers https://review.openstack.org/590442 | 21:24 |
| corvus | jhesketh: when you have a moment, can you look at https://review.openstack.org/589762 and confirm my assumption is correct (and if so, +3)? | 21:28 |
| *** hashar has quit IRC | 21:48 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!