*** robcresswell_ has joined #zuul | 00:04 | |
*** qwc_ has joined #zuul | 00:08 | |
*** dims has joined #zuul | 00:10 | |
*** mgagne_ has joined #zuul | 00:10 | |
*** tflink_ has joined #zuul | 00:11 | |
*** dims_ has quit IRC | 00:12 | |
*** qwc has quit IRC | 00:12 | |
*** robcresswell has quit IRC | 00:12 | |
*** mgagne has quit IRC | 00:12 | |
*** tflink has quit IRC | 00:12 | |
*** elyezer has quit IRC | 00:12 | |
*** jlvillal has quit IRC | 00:12 | |
*** yolanda has quit IRC | 00:12 | |
*** nhicher has quit IRC | 00:12 | |
*** robled has quit IRC | 00:12 | |
*** logan- has quit IRC | 00:12 | |
*** clarkb has quit IRC | 00:12 | |
*** Diabelko has quit IRC | 00:12 | |
*** jesusaur has quit IRC | 00:12 | |
*** sc68cal- has quit IRC | 00:12 | |
*** gundalow has quit IRC | 00:12 | |
*** pabelanger has quit IRC | 00:12 | |
*** EmilienM has quit IRC | 00:12 | |
*** pbrobinson_PTO has quit IRC | 00:12 | |
*** ChanServ has quit IRC | 00:12 | |
*** robcresswell_ is now known as robcresswell | 00:12 | |
*** odyssey4me has quit IRC | 00:12 | |
*** odyssey4me has joined #zuul | 00:12 | |
*** EmilienM has joined #zuul | 00:13 | |
*** pabelanger has joined #zuul | 00:13 | |
*** jesusaur has joined #zuul | 00:13 | |
*** pbrobinson_PTO has joined #zuul | 00:13 | |
*** sc68cal- has joined #zuul | 00:13 | |
*** ChanServ has joined #zuul | 00:13 | |
*** gundalow has joined #zuul | 00:13 | |
*** barjavel.freenode.net sets mode: +o ChanServ | 00:13 | |
*** robcresswell is now known as Guest46098 | 00:16 | |
*** smyers has quit IRC | 00:37 | |
openstackgerrit | Bernd Bausch proposed openstack-infra/zuul master: Add clarity to *Zuul from Scratch* instructions https://review.openstack.org/556344 | 00:38 |
---|---|---|
*** smyers has joined #zuul | 00:41 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: mqtt: add basic reporter https://review.openstack.org/535543 | 01:04 |
*** gouthamr has joined #zuul | 01:06 | |
*** JasonCL has joined #zuul | 01:06 | |
*** JasonCL has quit IRC | 01:07 | |
*** JasonCL has joined #zuul | 01:07 | |
openstackgerrit | Clark Boylan proposed openstack-infra/zuul master: Report to all reporters even if one fails https://review.openstack.org/557859 | 01:08 |
*** JasonCL has quit IRC | 01:10 | |
openstackgerrit | Clark Boylan proposed openstack-infra/zuul master: Cleanup use of mock package in for python2 https://review.openstack.org/557860 | 01:11 |
*** JasonCL has joined #zuul | 01:18 | |
openstackgerrit | Clark Boylan proposed openstack-infra/zuul master: Report to all reporters even if one fails https://review.openstack.org/557859 | 01:20 |
*** tflink_ is now known as tflink | 01:21 | |
*** harlowja has quit IRC | 01:21 | |
*** JasonCL has quit IRC | 01:24 | |
*** elyezer has joined #zuul | 01:25 | |
*** JasonCL has joined #zuul | 01:27 | |
*** jlvillal has joined #zuul | 01:28 | |
*** yolanda has joined #zuul | 01:28 | |
*** nhicher has joined #zuul | 01:28 | |
*** robled has joined #zuul | 01:28 | |
*** logan- has joined #zuul | 01:28 | |
*** clarkb has joined #zuul | 01:28 | |
*** Diabelko has joined #zuul | 01:28 | |
*** weshay has quit IRC | 01:35 | |
*** weshay has joined #zuul | 01:36 | |
*** myoung has quit IRC | 01:38 | |
*** elyezer has quit IRC | 01:41 | |
*** myoung has joined #zuul | 01:43 | |
*** elyezer has joined #zuul | 01:43 | |
*** JasonCL has quit IRC | 01:51 | |
*** gouthamr has quit IRC | 02:17 | |
*** gouthamr has joined #zuul | 02:22 | |
dmsimard | I would very much like to have a .zuul.d folder in which I could have playbooks (and even roles) without blowing up the config. | 02:22 |
dmsimard | I understand the limitation but having a "roles" and "playbooks" dir for a project like ARA can be confusing for people looking at the root of the source :/ | 02:23 |
dmsimard | oh, hang on T_T | 02:25 |
*** JasonCL has joined #zuul | 02:31 | |
dmsimard | you know, I wonder what's stopping us from being to use jinja2 in zuul.yaml's | 02:32 |
dmsimard | I mean we're already importing things from ansible here and there | 02:32 |
dmsimard | import the jinja things and we can do for loops in zuul.yamls \o/ | 02:33 |
*** elyezer has quit IRC | 02:33 | |
clarkb | dmsimard: I think its been intentionally designed to avoid that | 02:37 |
clarkb | due to experience with jjb | 02:37 |
dmsimard | huh, interesting | 02:38 |
dmsimard | I sort of knew that but never realized it | 02:38 |
dmsimard | now that you mention it, it is similar to jjb | 02:38 |
*** elyezer has joined #zuul | 02:40 | |
tristanC | custom path for roles would be useful though | 02:43 |
*** gouthamr has quit IRC | 02:43 | |
dmsimard | constraint was the role preparation I think | 02:44 |
*** JasonCL has quit IRC | 03:02 | |
*** JasonCL has joined #zuul | 03:03 | |
*** gouthamr has joined #zuul | 03:06 | |
*** JasonCL has quit IRC | 03:12 | |
*** gouthamr has quit IRC | 03:13 | |
dmsimard | you know, the way job inheritance work it's almost fun to construct jobs in relation to playbooks and roles | 03:22 |
dmsimard | trying to find the one right way to do it (there's none) | 03:22 |
dmsimard | apologies for the verbosity, it's fun to be in creative mode vs troubleshooting/the world is on fire mode :D | 03:23 |
*** JasonCL has joined #zuul | 03:29 | |
*** JasonCL has quit IRC | 03:34 | |
*** harlowja has joined #zuul | 03:51 | |
*** elyezer has quit IRC | 04:58 | |
dmsimard | It looks like Ansible has a feature for displaying where modules/plugins have been loaded from -- maybe we could use that. | 05:01 |
dmsimard | https://paste.fedoraproject.org/paste/go6TYZOHJplQquSGMUAeZw | 05:03 |
dmsimard | ansible-doc -F | 05:03 |
*** elyezer has joined #zuul | 05:08 | |
*** harlowja has quit IRC | 05:21 | |
*** elyezer has quit IRC | 05:59 | |
*** elyezer has joined #zuul | 06:07 | |
*** flepied has joined #zuul | 06:31 | |
*** flepied has quit IRC | 06:35 | |
*** xinliang has quit IRC | 06:47 | |
*** JasonCL has joined #zuul | 06:53 | |
*** JasonCL has quit IRC | 06:58 | |
*** xinliang has joined #zuul | 07:00 | |
*** xinliang has quit IRC | 07:00 | |
*** xinliang has joined #zuul | 07:00 | |
*** electrofelix has joined #zuul | 07:42 | |
*** hashar has joined #zuul | 07:53 | |
*** xinliang has quit IRC | 08:40 | |
*** xinliang has joined #zuul | 08:53 | |
*** xinliang has quit IRC | 08:53 | |
*** xinliang has joined #zuul | 08:53 | |
*** xinliang has quit IRC | 09:11 | |
*** xinliang has joined #zuul | 09:23 | |
*** xinliang has quit IRC | 09:23 | |
*** xinliang has joined #zuul | 09:23 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: github: add missing application permission to the doc https://review.openstack.org/557915 | 09:25 |
*** maeca has quit IRC | 09:55 | |
tristanC | it seems like github doesn't remove approved review when new commits are addded to a PR... | 10:03 |
tristanC | is there a working github gate pipeline available somewhere? | 10:04 |
*** elyezer has quit IRC | 10:58 | |
*** elyezer has joined #zuul | 10:59 | |
*** elyezer has quit IRC | 11:09 | |
*** elyezer has joined #zuul | 11:13 | |
*** mrhillsman has quit IRC | 11:48 | |
*** odyssey4me has quit IRC | 12:19 | |
*** odyssey4me has joined #zuul | 12:19 | |
*** JasonCL has joined #zuul | 13:08 | |
*** elyezer has quit IRC | 13:10 | |
*** elyezer has joined #zuul | 13:10 | |
*** JasonCL has quit IRC | 13:12 | |
*** electrofelix has quit IRC | 13:21 | |
*** elyezer has quit IRC | 13:28 | |
*** elyezer has joined #zuul | 13:31 | |
mordred | tristanC: that's a known limitation in github reviews - I think jlk was working with folks at github (before he joined) on getting that behavior either updated or configurable - I don't remember the status of that | 13:56 |
mordred | tristanC: that said, I think tobiash has one - perhaps he's got insight | 13:57 |
tristanC | mordred: oh ok, that's good to know | 13:58 |
tristanC | mordred: the mitigation we are trying is to require a "zuulit" label to enter the gate pipeline, and unlabel it when the check pipeline starts | 14:00 |
mordred | tristanC: yes - I think this is likely the best bet - and is very similar to what ansible/ansible does via their ansibullbot | 14:00 |
tristanC | perhaps the github event filter could check if the approved review are more recent than last pr update timestamp? | 14:17 |
mordred | tristanC: maybe? that seems like it would have a potential to be confusing for users though - we'd be adding semantics to the review object that don't exist in the github ui | 14:20 |
mordred | tristanC: (although that's not a bad idea otherwise) | 14:20 |
tristanC | zuul could post a "please re-approve the last update" comment | 14:21 |
*** weshay is now known as weshay_mod | 14:37 | |
tobiash | tristanC: dismissal of reviews on pr update is configurable in the branch protection | 14:42 |
tobiash | Except negative reviews which always stay | 14:43 |
corvus | if anyone wants to write a 'howto setup gating' doc, that probably wouldn't be a bad idea | 14:48 |
tobiash | tristanC: this is my current gate pipeline: http://paste.openstack.org/show/718024/ | 14:48 |
tobiash | but it relies on https://review.openstack.org/535680 | 14:49 |
tobiash | without it you will need to add the check status to the requirements | 14:49 |
tobiash | pabelanger: ^ | 14:50 |
tristanC | tobiash: thanks! | 14:50 |
tristanC | tobiash: btw, for post and release pipeline, we found "ref: ^refs/heads/.*$" and "ref: ^refs/tags/[0-9]+(\.[0-9]+)*$" do the trick on push event | 14:51 |
tobiash | tristanC: regarding automatic review dismissal: https://help.github.com/articles/enabling-required-reviews-for-pull-requests/ | 14:52 |
tobiash | step 7 | 14:52 |
tobiash | tristanC: cool, thanks | 14:52 |
openstackgerrit | Merged openstack-infra/zuul master: Cleanup use of mock package in for python2 https://review.openstack.org/557860 | 15:01 |
tobiash | corvus: what do you think about eventually moving the build times db to zk? | 15:01 |
corvus | tobiash: i worry a bit about whether having that many znodes is a good idea. we also talked about making the sql db required, if we do that, we could use that | 15:03 |
*** JasonCL has joined #zuul | 15:05 | |
corvus | tobiash: we have 26,000 entries in ours. | 15:06 |
tobiash | Hrm, ok | 15:07 |
tobiash | That's not on the critical path regarding functionality right? | 15:08 |
fungi | i have no real frame of reference, but that certainly is a large-sounding number | 15:08 |
corvus | tobiash: no, but i think it's important for folks looking at the status page. i'd like to avoid declaring it optional. | 15:09 |
corvus | fungi: job*project*branch | 15:09 |
tobiash | I'd at least like to keep the sql db out of the critical path for the basic ci operations | 15:09 |
tobiash | Because of availability | 15:10 |
corvus | tobiash: ah, i see. | 15:10 |
corvus | that's a good point | 15:10 |
corvus | we could consider "requiring" it terms of assuming it's always there, but continue to have zuul function if it can't reach it. | 15:11 |
corvus | (we can also continue to have it optional -- i think the motivation for 'requiring' it is: a) move the timedb there, b) significant parts of the dashboard need it) | 15:12 |
openstackgerrit | Paul Belanger proposed openstack-infra/nodepool master: Add debian-stretch to nodepool-functional-py35-debian-src https://review.openstack.org/557942 | 15:13 |
pabelanger | corvus: tobiash: tristanC: yah, a howto setup gating doc is a great idea | 15:14 |
pabelanger | tobiash: thanks for info, looking | 15:15 |
tobiash | corvus: I think requiring it is fine (I use it too) but ci functionality should work without | 15:17 |
corvus | ++ | 15:18 |
*** JasonCL has quit IRC | 15:21 | |
*** JasonCL has joined #zuul | 15:23 | |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Add test-emit-job-header roles https://review.openstack.org/557946 | 15:43 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Add nodepool hostnames to test-emit-job-header https://review.openstack.org/557947 | 15:43 |
*** elyezer has quit IRC | 15:49 | |
*** elyezer has joined #zuul | 15:51 | |
mordred | corvus, tobiash: I had a thought a while back about making a zk reporter for the things the dashboard needs - not because zk is a good place to put that data for large systems, but to keep things simple for small AIO systems (where the cost of potentially iterating over a bunch of znodes to get the data wouldn't be crazy bad) - doing that would probably require a plugin interface for reading historical data | 16:25 |
mordred | back out that zuul-web could use ... | 16:25 |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config https://review.openstack.org/535511 | 16:25 |
mordred | if we did that, we could similarly have 'build-times-in-zk' vs 'build-times-in-sql' be a deployment option - since the suitability of zk for the task probably similarly is related to size of the system in question | 16:26 |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config https://review.openstack.org/535511 | 16:26 |
mordred | alternately though, to serve AIO installs simply if we required SQL for dashboard purposes we could just make sure sqlite works | 16:28 |
*** hashar is now known as hasharAway | 16:59 | |
*** EmilienM has quit IRC | 17:58 | |
*** elyezer has quit IRC | 17:59 | |
*** elyezer has joined #zuul | 18:00 | |
corvus | mordred: i lean toward sqlite -- it should be easy enough to support such that the benefits of storing in zk don't outweigh sqlite enough for us to maitain two interfaces to history data i think | 18:16 |
*** mrhillsman has joined #zuul | 18:34 | |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul master: Add reporter for Federated Message Bus (fedmsg) https://review.openstack.org/557987 | 19:15 |
pabelanger | tristanC: ^fedmsg reporter | 19:16 |
pabelanger | I still need to add some docs and nodepool-dsvm integration testing | 19:19 |
pabelanger | tflink: ^ | 19:22 |
*** elyezer has quit IRC | 19:22 | |
*** lids361 has joined #zuul | 19:23 | |
lids361 | .-. .-. | 19:23 |
lids361 | .-. .-. | 19:23 |
lids361 | / \ / \ | 19:23 |
lids361 | / \ / \ | 19:23 |
lids361 | | _ \ / _ | | 19:23 |
lids361 | | _ \ / _ | | 19:23 |
lids361 | ; | \ \ / / | ; | 19:23 |
lids361 | ; | \ \ / / | ; | 19:23 |
lids361 | \ \ \ \_.._/ / / / | 19:23 |
lids361 | \ \ \ \_.._/ / / / | 19:23 |
lids361 | '. '.;' ';,' .' | 19:23 |
lids361 | '. '.;' ';,' .' | 19:23 |
lids361 | './ _ _ \.' | 19:23 |
lids361 | './ _ _ \.' | 19:23 |
lids361 | .' a __ a '. | 19:23 |
lids361 | .' a __ a '. | 19:23 |
lids361 | '--./ _, \/ ,_ \.--' | 19:23 |
lids361 | '--./ _, \/ ,_ \.--' | 19:23 |
lids361 | ----| \ /\ / |---- | 19:24 |
lids361 | ----| \ /\ / |---- | 19:24 |
lids361 | .--'\ '-' '-' /'--. | 19:24 |
lids361 | .--'\ '-' '-' /'--. | 19:24 |
lids361 | _>.__ -- _.- `; | 19:24 |
lids361 | _>.__ -- _.- `; | 19:24 |
lids361 | .' _ __/ _/ | 19:24 |
lids361 | .' _ __/ _/ | 19:24 |
lids361 | / '.,:".-\ /:, | 19:24 |
lids361 | / '.,:".-\ /:, | 19:24 |
lids361 | | \.' `""`'.\\ | 19:24 |
lids361 | | \.' `""`'.\\ | 19:24 |
lids361 | '-,.__/ _ .-. ;|_ | 19:24 |
lids361 | '-,.__/ _ .-. ;|_ | 19:24 |
lids361 | /` `|| _/ `\/_ \_|| `\ | 19:24 |
lids361 | /` `|| _/ `\/_ \_|| `\ | 19:24 |
lids361 | | ||/ \-./` \ / || | | 19:24 |
lids361 | | ||/ \-./` \ / || | | 19:24 |
lids361 | \ ||__/__|___|__|| / | 19:24 |
lids361 | \ ||__/__|___|__|| / | 19:24 |
lids361 | \_ |_Happy Easter_| / | 19:24 |
lids361 | \_ |_Happy Easter_| / | 19:24 |
lids361 | jgs .' \ = _= _ = _= /`\ | 19:24 |
lids361 | jgs .' \ = _= _ = _= /`\ | 19:24 |
lids361 | / `-;----=--;--' \ | 19:24 |
lids361 | / `-;----=--;--' \ | 19:24 |
lids361 | \ _.-' '. / | 19:24 |
lids361 | \ _.-' '. / | 19:24 |
lids361 | `""` `""` | 19:24 |
lids361 | `""` `""` | 19:24 |
lids361 | L0DE AND CHRON FROM #LRH & L0DE RADIO HOUR (IRC.EFNET.ORG) WANTED TO SAY HAPPY EASTER!! | 19:25 |
lids361 | L0DE AND CHRON FROM #LRH & L0DE RADIO HOUR (IRC.EFNET.ORG) WANTED TO SAY HAPPY EASTER!! | 19:25 |
lids361 | mrhillsman JasonCL odyssey4me xinliang hasharAway myoung weshay_mod Diabelko clarkb logan- robled nhicher yolanda jlvillal smyers gundalow sc68cal- pbrobinson_PTO jesusaur pabelanger tflink mgagne_ dims qwc_ Guest46098 dvn sshnaidm mugsie patriciadomin jimi|ansible openstack jamielennox openstackgerrit tobiash kmalloc fbo_ mhu leifmadsen eventingmonkey adam_g toabctl swest ssbarnea _ari_ pwhalen timrc persia trishnag dmsimard andreaf dmellado fbouliane dtruo | 19:25 |
*** lids361 has quit IRC | 19:25 | |
*** elyezer has joined #zuul | 19:29 | |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul master: Add reporter for Federated Message Bus (fedmsg) https://review.openstack.org/557987 | 19:47 |
*** elyezer has quit IRC | 20:20 | |
*** elyezer has joined #zuul | 20:23 | |
*** mordred has quit IRC | 20:26 | |
*** mordred has joined #zuul | 20:27 | |
*** elyezer has quit IRC | 20:29 | |
*** elyezer has joined #zuul | 20:31 | |
*** elyezer has quit IRC | 20:39 | |
*** elyezer has joined #zuul | 20:40 | |
*** harlowja has joined #zuul | 21:28 | |
*** hasharAway has quit IRC | 22:21 | |
mrhillsman | should i expect to see a comment on this pr? https://github.com/kubernetes/cloud-provider-openstack/pull/76 | 22:36 |
mrhillsman | https://github.com/kubernetes/cloud-provider-openstack/commit/320fff3f33f227c2e009d647e4ad5390ea57ba65 | 22:36 |
clarkb | mrhillsman: I dont know, is there config in a zuul somewhere running jobs on that repo? | 22:38 |
mrhillsman | the app got installed | 22:38 |
mrhillsman | i updated main.yml | 22:38 |
mrhillsman | https://www.irccloud.com/pastebin/Pokw9dnd/ | 22:38 |
mrhillsman | 2018-03-30 21:34:27,713 DEBUG zuul.Scheduler: Change <Change 0x7f8dc1fa28d0 76,320fff3f33f227c2e009d647e4ad5390ea57ba65> has been updated, clearing dependent change caches | 22:39 |
mrhillsman | zuul yaml file is in place | 22:39 |
clarkb | you should see log lines like considering change foo in check | 22:40 |
clarkb | that us when zuul determines what (if any) jobs to run for changes | 22:40 |
mrhillsman | i don't need to restart anything after updating main.yml do i? | 22:42 |
clarkb | you have to do a config reload but not a restart | 22:42 |
mrhillsman | ah ok | 22:42 |
mrhillsman | i thought it was automagic | 22:42 |
mrhillsman | needed the reload, thx clarkb | 22:57 |
pabelanger | mrhillsman: app installed is good news | 23:09 |
mrhillsman | yep | 23:09 |
mrhillsman | definitely the easier route | 23:10 |
mrhillsman | i do have someone investigating the plugin for prow too | 23:10 |
pabelanger | yah | 23:11 |
mrhillsman | should there be a useful case to indeed have one | 23:11 |
mrhillsman | clarkb or pabelanger i need to change the job name in .zuul.yaml with a pr will that get picked up | 23:25 |
pabelanger | mrhillsman: yes, for untrusted projects right away | 23:27 |
mrhillsman | ty sir | 23:27 |
pabelanger | you'll also need to update your pipeline with the new job name too | 23:27 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!