| *** patriciadomin has quit IRC | 00:06 | |
| *** patriciadomin has joined #zuul | 00:09 | |
| *** JasonCL has quit IRC | 00:17 | |
| *** JasonCL has joined #zuul | 00:18 | |
| *** JasonCL has quit IRC | 00:33 | |
| *** JasonCL has joined #zuul | 00:34 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 00:34 |
|---|---|---|
| *** JasonCL has quit IRC | 00:44 | |
| *** patriciadomin has quit IRC | 00:51 | |
| *** patriciadomin has joined #zuul | 00:52 | |
| *** JasonCL has joined #zuul | 00:57 | |
| *** JasonCL has quit IRC | 01:02 | |
| *** patriciadomin has quit IRC | 01:03 | |
| *** patriciadomin has joined #zuul | 01:04 | |
| *** patriciadomin has quit IRC | 01:09 | |
| *** patriciadomin has joined #zuul | 01:09 | |
| *** patriciadomin has quit IRC | 01:17 | |
| *** patriciadomin has joined #zuul | 01:18 | |
| *** patriciadomin has quit IRC | 01:56 | |
| *** patriciadomin has joined #zuul | 01:56 | |
| *** JasonCL has joined #zuul | 02:00 | |
| *** JasonCL has quit IRC | 02:05 | |
| *** JasonCL has joined #zuul | 02:45 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 02:49 |
| *** JasonCL has quit IRC | 02:52 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 02:58 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Fix legacy dsvm jobs https://review.openstack.org/543327 | 02:58 |
| *** patriciadomin has quit IRC | 03:11 | |
| *** patriciadomin has joined #zuul | 03:11 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 03:14 |
| *** elyezer has quit IRC | 03:15 | |
| *** elyezer has joined #zuul | 03:16 | |
| ianw | corvus / Shrews : apropos the switch to native jobs for nodepool & trying to debug resulting log output ... https://review.openstack.org/543266 which means we can click on .log.gz files; reviews appreciated | 03:19 |
| ianw | the builder .log files get caught by this | 03:19 |
| *** patriciadomin has quit IRC | 03:24 | |
| ianw | #status log received rax notification that host of nl02.o.o and zm06.o.o has some sort of issue; currently can't log into either. updates from rax pending | 03:25 |
| ianw | (sorry wrong channel) | 03:25 |
| *** patriciadomin has joined #zuul | 03:25 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Fix legacy dsvm jobs https://review.openstack.org/543327 | 03:47 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 03:47 |
| *** patriciadomin has quit IRC | 03:48 | |
| *** patriciadomin has joined #zuul | 03:50 | |
| *** patriciadomin has quit IRC | 04:05 | |
| *** patriciadomin has joined #zuul | 04:05 | |
| *** JasonCL has joined #zuul | 04:26 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Fix legacy dsvm jobs https://review.openstack.org/543327 | 04:27 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 04:27 |
| *** Wei_Liu has quit IRC | 04:30 | |
| *** JasonCL has quit IRC | 04:31 | |
| *** xinliang has joined #zuul | 04:36 | |
| *** patriciadomin has quit IRC | 04:42 | |
| *** patriciadomin has joined #zuul | 04:43 | |
| *** elyezer has quit IRC | 05:15 | |
| *** patriciadomin has quit IRC | 05:19 | |
| *** elyezer has joined #zuul | 05:20 | |
| *** patriciadomin has joined #zuul | 05:20 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 05:39 |
| *** patriciadomin has quit IRC | 05:46 | |
| *** patriciadomin has joined #zuul | 05:47 | |
| *** patriciadomin has quit IRC | 05:59 | |
| *** patriciadomin has joined #zuul | 06:00 | |
| *** Wei_Liu has joined #zuul | 06:11 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 06:41 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Emit a warning when using unsafe public key url https://review.openstack.org/543206 | 06:47 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 06:48 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Revert fixes for legacy boot jobs https://review.openstack.org/543350 | 06:48 |
| *** elyezer has quit IRC | 06:51 | |
| *** elyezer has joined #zuul | 06:52 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Remove unused method checkoutLocalBranch https://review.openstack.org/543359 | 07:09 |
| *** swest has quit IRC | 07:10 | |
| *** swest has joined #zuul | 07:10 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 07:33 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Revert fixes for legacy boot jobs https://review.openstack.org/543350 | 07:33 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Don't decode str in LogStreamingHandler https://review.openstack.org/543365 | 07:42 |
| *** hashar has joined #zuul | 07:44 | |
| *** elyezer has quit IRC | 07:47 | |
| *** elyezer has joined #zuul | 07:48 | |
| *** elyezer has quit IRC | 08:01 | |
| *** elyezer has joined #zuul | 08:03 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix broken fail_json in zuul_console https://review.openstack.org/543370 | 08:04 |
| *** sc68cal has quit IRC | 08:34 | |
| *** jpena|off is now known as jpena | 08:45 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 08:53 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Revert fixes for legacy boot jobs https://review.openstack.org/543350 | 08:53 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 10:05 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Revert fixes for legacy boot jobs https://review.openstack.org/543350 | 10:05 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul autohold: allow operator to specify nodes TTL https://review.openstack.org/543403 | 10:08 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Clean held nodes automatically after configurable timeout https://review.openstack.org/536295 | 10:17 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Add /label-list to the webapp https://review.openstack.org/535563 | 10:19 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Refactor status functions, add web endpoints, allow params https://review.openstack.org/536301 | 10:19 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Add separate modules for management commands https://review.openstack.org/536303 | 10:20 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: webapp: add optional admin endpoint https://review.openstack.org/536319 | 10:21 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul autohold: allow operator to specify nodes TTL https://review.openstack.org/543403 | 10:23 |
| *** elyezer has quit IRC | 10:25 | |
| *** elyezer has joined #zuul | 10:30 | |
| *** electrofelix has joined #zuul | 10:36 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul autohold: allow operator to specify nodes TTL https://review.openstack.org/543403 | 10:40 |
| *** mrhillsman has quit IRC | 10:57 | |
| *** mrhillsman has joined #zuul | 10:58 | |
| *** JasonCL has joined #zuul | 11:03 | |
| *** bhavik1 has joined #zuul | 11:13 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Clean held nodes automatically after configurable timeout https://review.openstack.org/536295 | 11:23 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: webapp: add optional admin endpoint https://review.openstack.org/536319 | 11:30 |
| *** bhavik1 has quit IRC | 12:01 | |
| *** sshnaidm is now known as sshnaidm|ruck | 12:09 | |
| *** jpena is now known as jpena|lunch | 12:26 | |
| *** weshay|ruck is now known as weshay | 13:10 | |
| *** jpena|lunch is now known as jpena | 13:26 | |
| *** rlandy has joined #zuul | 13:32 | |
| *** myoung|off is now known as myoung | 13:53 | |
| *** dkranz has joined #zuul | 14:14 | |
| *** patriciadomin_ has joined #zuul | 14:16 | |
| *** patriciadomin has quit IRC | 14:19 | |
| pabelanger | with zuulv3, jobs that share the same name no longer imply a share queue for dependent pipeline, is that correct? | 15:25 |
| AJaeger | pabelanger: correct | 15:25 |
| AJaeger | pabelanger: that's why we have "queue: X" in the gate pipeline | 15:25 |
| pabelanger | Yah, knew we had to add queue: foobar now to be more specific, was there a specific reason for no longer making that implied? | 15:27 |
| corvus | pabelanger: because jobs no longer have their project names in them. almost every project in openstack runs a job called 'tox-py27', for example. | 15:30 |
| corvus | we might be able to bring some of the automation back, however, by analyzing required-projects entries. i haven't thought that through completely yet though. | 15:31 |
| pabelanger | kk, that make sense. Was mostly trying to see why it changed. | 15:32 |
| *** gregdek has quit IRC | 15:43 | |
| *** gregdek has joined #zuul | 15:43 | |
| corvus | SpamapS: any update on the pyre2 release? | 15:48 |
| *** elyezer has quit IRC | 15:58 | |
| *** elyezer has joined #zuul | 16:02 | |
| *** mattclay has quit IRC | 16:03 | |
| *** mattclay has joined #zuul | 16:03 | |
| SpamapS | corvus: the author said they'd try to cut a release, but gave no time frame. I took that to be "in the next few days" | 16:35 |
| SpamapS | corvus: and he did tag.. | 16:37 |
| SpamapS | https://github.com/facebook/pyre2/tree/v1.0.6 | 16:37 |
| SpamapS | so.. I assume he forgot his pypi password or something ;) | 16:37 |
| SpamapS | actually https://pypi.python.org/pypi/fb-re2 shows sid0 as the owner in pypi, so maybe sid0 was out last week or something. | 16:37 |
| corvus | it's so weird for tag and release to not be the same thing :) | 16:42 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: webapp: add optional admin endpoint https://review.openstack.org/536319 | 16:42 |
| pabelanger | hmm | 16:48 |
| pabelanger | this doesn't look right | 16:48 |
| pabelanger | 2018-02-12 16:40:10.718800 | ze01 | :stderr: aiohttp requires Python '>=3.5.3' but the running Python is 3.5.2 | 16:48 |
| SpamapS | corvus: spoken like a true robot wrangelr. | 16:48 |
| pabelanger | getting that from aiohttp now | 16:48 |
| pabelanger | seems to be a 3.0.0 release today | 16:49 |
| corvus | greeeaaat | 16:49 |
| SpamapS | all hail the .0 release gods | 16:49 |
| SpamapS | for they shall smite all thy automation and burn thy untested code to the ground. | 16:49 |
| pabelanger | Minimal supported Python version is 3.5.3. | 16:49 |
| pabelanger | ouch | 16:49 |
| corvus | there's also a 3.0.1 | 16:50 |
| SpamapS | We might want to ping doko. He should be updating Xenial to latest patch release sometime soon. | 16:50 |
| SpamapS | Oh wow | 16:50 |
| SpamapS | 3.5.3 is like, < 30 days old | 16:50 |
| corvus | pabelanger: want to throw in a < pin? | 16:51 |
| SpamapS | https://www.python.org/downloads/release/python-353/ | 16:51 |
| pabelanger | corvus: sure | 16:51 |
| SpamapS | I love these commit messages. https://github.com/aio-libs/aiohttp/commit/8e9c6b5079f6acbbdfc352250e21ae5bf56c25e8 | 16:53 |
| SpamapS | "I did some stuff. Deal with it." | 16:53 |
| SpamapS | [paraphrased] | 16:53 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul master: Cap aiohttp <3.0.0 https://review.openstack.org/543546 | 16:53 |
| corvus | the code says what you did. the commit message doesn't need to repeat it. neither one says why. :( | 16:53 |
| SpamapS | no like, "because this bug was fixed" or "because we hate 3.5.2's face" | 16:54 |
| corvus | pabelanger: where does that error show up? | 16:55 |
| SpamapS | not even a reference to a bug | 16:55 |
| SpamapS | nothing | 16:55 |
| SpamapS | wow | 16:55 |
| SpamapS | aiohttp devs.. you disappoint me. :( | 16:55 |
| pabelanger | corvus: I noticed it at: http://logs.openstack.org/84/543484/1/gate/windmill-ubuntu-xenial/0c8b364/job-output.txt.gz#_2018-02-12_16_40_10_718500 | 16:55 |
| corvus | yeah... i don't think aiohttp has been an unqualified success. | 16:56 |
| corvus | pabelanger: ok. zuul01.o.o hasn't upgraded it... yet... | 16:57 |
| pabelanger | kk | 16:57 |
| SpamapS | Shame devs on Twitter [X] https://twitter.com/spamaps/status/963094549735211008 | 16:57 |
| corvus | SpamapS: you want to +3 https://review.openstack.org/543546 ? | 17:00 |
| *** maxamillion has quit IRC | 17:04 | |
| *** maxamillion has joined #zuul | 17:04 | |
| *** maxamillion has quit IRC | 17:13 | |
| *** maxamillion has joined #zuul | 17:13 | |
| SpamapS | looks like tobiash beat me to it :) | 17:16 |
| electrofelix | Does zuul have support for HA around the scheduling part? I recall that it can be gracefully restarted but can't seem to put my hands on details around upgrading/updating | 17:43 |
| *** myoung is now known as myoung|food | 17:44 | |
| corvus | electrofelix: not yet. that's what we call zuul v4 | 17:44 |
| electrofelix | corvus: so all updates are currently lossy (requires retriggering) | 17:46 |
| SpamapS | Zuul v4 - The Wrath of Zookeeper | 17:46 |
| SpamapS | electrofelix: you can save the queue and re-load it. So scheduled restarts are not lossy. | 17:47 |
| SpamapS | But if you lose the process unexpectedly, all must be retriggered. | 17:47 |
| electrofelix | SpamapS: good enough, I'm only concerned about the scheduled restart/update | 17:48 |
| SpamapS | corvus: part of me wonders if we can get a 3.x that more or less abuses zookeeper to store state in it without having the actual multi-scheduler HA model. | 17:48 |
| corvus | SpamapS: that's 99% of the work | 17:49 |
| SpamapS | like just something coarse grained and dumb, like wrapping trigger handling in queue.save() calls | 17:49 |
| SpamapS | and making the queue save happen in zk, instead of on disk. | 17:49 |
| SpamapS | corvus: no, think dumber. :) | 17:50 |
| corvus | SpamapS: i am :) you throw a lock around that and you've got HA | 17:50 |
| SpamapS | like, the same thing we have now, but to zk, and more often. | 17:50 |
| corvus | SpamapS: we don't save the pipeline contents anywhere | 17:50 |
| SpamapS | ooh | 17:50 |
| SpamapS | just the queue then. | 17:51 |
| SpamapS | I see now, we are already as dumb as we wanna be. :) | 17:51 |
| SpamapS | btw... latest updates to Zuul are nice. Little things getting better and better. | 17:51 |
| corvus | so the main task is creating the zk<->model mapping. the rest falls out from there. | 17:51 |
| SpamapS | Sorry I won't be in Dublin. Too insane at work to leave. | 17:52 |
| corvus | i was polishing last week | 17:52 |
| corvus | SpamapS: drat. maybe we can irc/phone you in on some things if you have time | 17:52 |
| SpamapS | Yeah I'm sure I can get in on some of those discussions. | 17:53 |
| SpamapS | Just can't be gone. one of our senior engs just left for Paternity leave.. so we're already skeleton crew. :-P | 17:53 |
| SpamapS | corvus: I'm especially interested in working out how we turn Zuul into a first class deployer. | 17:54 |
| SpamapS | We have it CI'ing our stuff now, but we still hand-off to a deployment system for reasons. | 17:54 |
| corvus | SpamapS: great! that's the dream :) | 17:55 |
| SpamapS | Mainly that people want to pass --limit and --tags to the Ansible we run, and while we already let zuul run kolla-ansible on a runner VM... the whole thing of "now run it with these args. Now these args. Now these." .. not super smooth in zuul. | 17:55 |
| corvus | SpamapS: i responded on https://review.openstack.org/542518 - i think most of the things are straightforward, but i think there was some secure.yaml/clouds.yaml confusion -- can you make sure i've sussed that out correctly and everything is straight? | 17:56 |
| SpamapS | Hoping we can work out manual triggering / parameterization. | 17:56 |
| corvus | SpamapS: i don't see any reason we can't support tags, that seems straightforward. the limit things sounds like a high-bandwidth conversation we should have to make sure we all get on the same page. | 17:57 |
| SpamapS | There's this thing bubbling in my brain about having ansible-orchestration be a thing. | 17:58 |
| SpamapS | Which sounds weird | 17:58 |
| SpamapS | but it's like, sometimes I want to express a pattern for running a playbook with certain arguments. | 17:58 |
| SpamapS | The specific example is doing AZ-aware canary-sensitive rolling updates on a schedule. | 17:58 |
| SpamapS | So like, "You can update any of these, 10% at a time, start with 1, then do 2, then 5. Stop if any monitors goes off." | 17:59 |
| SpamapS | serial: sort of wants to be that, but it's not quite good enough | 18:00 |
| *** bhavik1 has joined #zuul | 18:00 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul master: Make timeout value apply to entire job https://review.openstack.org/541485 | 18:02 |
| clarkb | SpamapS: ^ the timeout is being passed to runAnsiblePlaybook so I called it ansible_timeout. Hopefully that is more clear | 18:02 |
| *** bhavik1 has quit IRC | 18:04 | |
| *** harlowja has joined #zuul | 18:14 | |
| *** jpena is now known as jpena|off | 18:18 | |
| *** elyezer has quit IRC | 18:32 | |
| *** elyezer has joined #zuul | 18:33 | |
| *** electrofelix has quit IRC | 18:34 | |
| *** electrofelix has joined #zuul | 18:37 | |
| *** rlandy is now known as rlandy|brb | 18:46 | |
| corvus | SpamapS: sudo adduser --system nodepool --home-dir /var/lib/nodepool --create-home | 18:47 |
| corvus | SpamapS: how's that look? | 18:47 |
| *** elyezer has quit IRC | 18:49 | |
| *** elyezer has joined #zuul | 18:50 | |
| pabelanger | IIRC using --system means you'd no longer be able to do sudo su nodepool | 18:50 |
| *** electrofelix has quit IRC | 18:51 | |
| *** dkranz has quit IRC | 18:52 | |
| corvus | pabelanger: seems to work | 18:54 |
| pabelanger | cool! | 18:54 |
| pabelanger | wfm | 18:55 |
| *** myoung|food is now known as myoung | 18:57 | |
| corvus | SpamapS: your suggestion to use the openstackclient to create a keypair means we need to install 81 more dependencies | 19:01 |
| corvus | SpamapS: i'm not sure that's desirable... | 19:02 |
| corvus | maybe that's why the todo note suggested shade | 19:02 |
| corvus | then again, maybe we should go ahead and recommend the installation, that way it's there for nodepool debugging, etc... (eg openstack server list) | 19:03 |
| clarkb | doesn't nova do keypair generation on the server side? | 19:04 |
| corvus | clarkb: i don't understand | 19:04 |
| clarkb | oh you are talking to use teh create command nevermind I misunderstood (I thought you wanted a replacement for paramiko) | 19:05 |
| *** rlandy|brb is now known as rlandy | 19:08 | |
| *** sshnaidm|ruck is now known as sshnaidm|off | 19:11 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Add zuul from scratch doc https://review.openstack.org/542518 | 19:34 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Reclassify some debug log lines as info https://review.openstack.org/543044 | 19:34 |
| corvus | tobiash, SpamapS: ^ okay, I think that takes care of all your suggestions | 19:35 |
| clarkb | corvus: fwiw some clouds publish an openrc and a clouds.yaml via horizon now. But I am not sure how common that is at this point (probably still more generic to make clouds.yaml from openrc) | 19:37 |
| clarkb | there is also no mention of the firewall which surprises me. I thought fedora (and other red hat distros) had very restrictive firewall rules by default? maybe not on the cloud images? | 19:42 |
| *** elyezer has quit IRC | 19:49 | |
| ianw | could i get a couple of reviews on https://review.openstack.org/#/c/543327 to fix the dib gate ... this is the minimal fix for the legacy nodepool boot jobs to keep working | 19:58 |
| *** elyezer has joined #zuul | 19:59 | |
| corvus | clarkb: the cloud i'm working with (rdo) doesn't have clouds.yaml. and yeah, apparently firewall is not an issue using the fedora 27 cloud image. | 20:02 |
| corvus | clarkb: i'm certain it's stock, because i had to download and upload the image myself. | 20:02 |
| ianw | corvus: re the copy of logs via zuul_copy_output @ https://git.openstack.org/cgit/openstack-infra/nodepool/tree/.zuul.yaml#n30 ... does that not copy in failure case? see http://logs.openstack.org/28/543328/2/check/legacy-dsvm-nodepool-opensuse-src/6e9d662/logs/ | 20:06 |
| ianw | oh, don't mind me, that's the legacy job | 20:15 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Add native distro test jobs https://review.openstack.org/543268 | 20:22 |
| openstackgerrit | Ian Wienand proposed openstack-infra/nodepool master: Revert fixes for legacy boot jobs https://review.openstack.org/543350 | 20:22 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul web: add admin endpoint, enqueue commands https://review.openstack.org/539004 | 20:27 |
| * rbergeron appears and babbles about yay fedora cloud images | 20:44 | |
| openstackgerrit | Merged openstack-infra/nodepool master: Fix legacy dsvm jobs https://review.openstack.org/543327 | 20:45 |
| openstackgerrit | Merged openstack-infra/zuul master: Cap aiohttp <3.0.0 https://review.openstack.org/543546 | 21:01 |
| mnaser | clarkb: late but the clouds.yaml change shipped thanks to mordred in pike | 21:01 |
| *** ChanServ has quit IRC | 21:02 | |
| *** ChanServ has joined #zuul | 21:11 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 21:11 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: add abstract job attribute https://review.openstack.org/543619 | 21:15 |
| dmsimard | corvus: I'll send a patch for it but do you have any suggestions on how to improve the headers/anchors in the docs such as these ? https://docs.openstack.org/infra/zuul/admin/drivers/gerrit.html | 21:46 |
| dmsimard | An anchor can end up being displayed/linked as "https://docs.openstack.org/infra/zuul/admin/drivers/gerrit.html#attr-pipeline.trigger.%3Cgerrit%20source%3E.approval" which is not very parseable when not encoded | 21:47 |
| dmsimard | pabelanger linked me "https://docs.openstack.org/infra/zuul/admin/drivers/gerrit.html?highlight=verified#attr-pipeline.trigger.<gerrit source>.require-approval" earlier :) | 21:47 |
| dmsimard | I guess it depends on how each browser ends up displaying/copying the link | 21:48 |
| corvus | dmsimard: i don't have an immediate suggestion for that -- tbh, i was *thrilled* just to get that working at all :) | 21:50 |
| corvus | dmsimard: i agree about the desirability of it though :) | 21:51 |
| jhesketh | Morning | 21:52 |
| dmsimard | FWIW Firefox properly encodes that for me | 21:52 |
| pabelanger | jhesketh: morning! | 21:52 |
| dmsimard | I mean, when I click on the header -- clicking on pabelanger's link is hopeless and my IRC client doesn't understand it :p | 21:52 |
| corvus | dmsimard: maybe we can translate <> to something else in add_target_and_index ? | 21:53 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Clean held nodes automatically after configurable timeout https://review.openstack.org/536295 | 21:53 |
| corvus | dmsimard: that's where i'd start poking | 21:53 |
| dmsimard | corvus: worth a look | 21:53 |
| corvus | dmsimard: if we change it, we should do a matching change in zuul-sphinx. and/or see if we can drop that from zuul and switch zuul itself over to depend on zuul-sphinx for doc builds. | 21:54 |
| * jhesketh has to run to the airport so will only be intermittently at the zuul meeting | 21:54 | |
| corvus | jhesketh: heading off to the ptg? ;) | 21:55 |
| jhesketh | corvus: I will be :-), but not this early. Just dropping my wife off so not travelling anywhere myself this week | 21:55 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul master: Delete IncludeRole object from result object for include_role tasks https://review.openstack.org/543633 | 21:56 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul master: Delete IncludeRole object from result object for include_role tasks https://review.openstack.org/543633 | 21:58 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul autohold: allow operator to specify nodes TTL https://review.openstack.org/543403 | 21:58 |
| dmsimard | Zuul meeting imminent in #openstack-meeting-alt | 21:59 |
| pabelanger | will only be around for first 1/2 today | 22:00 |
| pabelanger | then off to local linux meetup | 22:00 |
| pabelanger | year of the LUG | 22:00 |
| pabelanger | :) | 22:00 |
| *** threestrands has joined #zuul | 22:12 | |
| openstackgerrit | Merged openstack-infra/zuul master: Emit a warning when using unsafe public key url https://review.openstack.org/543206 | 22:18 |
| openstackgerrit | Merged openstack-infra/zuul master: Remove unused method checkoutLocalBranch https://review.openstack.org/543359 | 22:18 |
| *** hashar has quit IRC | 22:28 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Add abstract job attribute https://review.openstack.org/543619 | 23:36 |
| ianw | has there been any disucssion of the webapp endpoints of nodepool (dib-image-list) etc? | 23:54 |
| ianw | i'm just fixing up the puppet for the build logs and wondering if it's worth going to effort to expose them | 23:54 |
| clarkb | ianw: I want to say it came up last last week? | 23:55 |
| clarkb | but I have been a zombie the last 4 days or so so may misremember | 23:55 |
| ianw | ok. the complication comes if you want a builder on the same host as the launcher. i don't want to over-engineer a solution that nobody ever uses | 23:58 |
| dmsimard | ianw: there are already some patches in that direction I think | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!