Friday, 2018-03-30

*** robcresswell_ has joined #zuul00:04
*** qwc_ has joined #zuul00:08
*** dims has joined #zuul00:10
*** mgagne_ has joined #zuul00:10
*** tflink_ has joined #zuul00:11
*** dims_ has quit IRC00:12
*** qwc has quit IRC00:12
*** robcresswell has quit IRC00:12
*** mgagne has quit IRC00:12
*** tflink has quit IRC00:12
*** elyezer has quit IRC00:12
*** jlvillal has quit IRC00:12
*** yolanda has quit IRC00:12
*** nhicher has quit IRC00:12
*** robled has quit IRC00:12
*** logan- has quit IRC00:12
*** clarkb has quit IRC00:12
*** Diabelko has quit IRC00:12
*** jesusaur has quit IRC00:12
*** sc68cal- has quit IRC00:12
*** gundalow has quit IRC00:12
*** pabelanger has quit IRC00:12
*** EmilienM has quit IRC00:12
*** pbrobinson_PTO has quit IRC00:12
*** ChanServ has quit IRC00:12
*** robcresswell_ is now known as robcresswell00:12
*** odyssey4me has quit IRC00:12
*** odyssey4me has joined #zuul00:12
*** EmilienM has joined #zuul00:13
*** pabelanger has joined #zuul00:13
*** jesusaur has joined #zuul00:13
*** pbrobinson_PTO has joined #zuul00:13
*** sc68cal- has joined #zuul00:13
*** ChanServ has joined #zuul00:13
*** gundalow has joined #zuul00:13
*** barjavel.freenode.net sets mode: +o ChanServ00:13
*** robcresswell is now known as Guest4609800:16
*** smyers has quit IRC00:37
openstackgerritBernd Bausch proposed openstack-infra/zuul master: Add clarity to *Zuul from Scratch* instructions  https://review.openstack.org/55634400:38
*** smyers has joined #zuul00:41
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: mqtt: add basic reporter  https://review.openstack.org/53554301:04
*** gouthamr has joined #zuul01:06
*** JasonCL has joined #zuul01:06
*** JasonCL has quit IRC01:07
*** JasonCL has joined #zuul01:07
openstackgerritClark Boylan proposed openstack-infra/zuul master: Report to all reporters even if one fails  https://review.openstack.org/55785901:08
*** JasonCL has quit IRC01:10
openstackgerritClark Boylan proposed openstack-infra/zuul master: Cleanup use of mock package in for python2  https://review.openstack.org/55786001:11
*** JasonCL has joined #zuul01:18
openstackgerritClark Boylan proposed openstack-infra/zuul master: Report to all reporters even if one fails  https://review.openstack.org/55785901:20
*** tflink_ is now known as tflink01:21
*** harlowja has quit IRC01:21
*** JasonCL has quit IRC01:24
*** elyezer has joined #zuul01:25
*** JasonCL has joined #zuul01:27
*** jlvillal has joined #zuul01:28
*** yolanda has joined #zuul01:28
*** nhicher has joined #zuul01:28
*** robled has joined #zuul01:28
*** logan- has joined #zuul01:28
*** clarkb has joined #zuul01:28
*** Diabelko has joined #zuul01:28
*** weshay has quit IRC01:35
*** weshay has joined #zuul01:36
*** myoung has quit IRC01:38
*** elyezer has quit IRC01:41
*** myoung has joined #zuul01:43
*** elyezer has joined #zuul01:43
*** JasonCL has quit IRC01:51
*** gouthamr has quit IRC02:17
*** gouthamr has joined #zuul02:22
dmsimardI 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
dmsimardI 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
dmsimardoh, hang on T_T02:25
*** JasonCL has joined #zuul02:31
dmsimardyou know, I wonder what's stopping us from being to use jinja2 in zuul.yaml's02:32
dmsimardI mean we're already importing things from ansible here and there02:32
dmsimardimport the jinja things and we can do for loops in zuul.yamls \o/02:33
*** elyezer has quit IRC02:33
clarkbdmsimard: I think its been intentionally designed to avoid that02:37
clarkbdue to experience with jjb02:37
dmsimardhuh, interesting02:38
dmsimardI sort of knew that but never realized it02:38
dmsimardnow that you mention it, it is similar to jjb02:38
*** elyezer has joined #zuul02:40
tristanCcustom path for roles would be useful though02:43
*** gouthamr has quit IRC02:43
dmsimardconstraint was the role preparation I think02:44
*** JasonCL has quit IRC03:02
*** JasonCL has joined #zuul03:03
*** gouthamr has joined #zuul03:06
*** JasonCL has quit IRC03:12
*** gouthamr has quit IRC03:13
dmsimardyou know, the way job inheritance work it's almost fun to construct jobs in relation to playbooks and roles03:22
dmsimardtrying to find the one right way to do it (there's none)03:22
dmsimardapologies for the verbosity, it's fun to be in creative mode vs troubleshooting/the world is on fire mode :D03:23
*** JasonCL has joined #zuul03:29
*** JasonCL has quit IRC03:34
*** harlowja has joined #zuul03:51
*** elyezer has quit IRC04:58
dmsimardIt looks like Ansible has a feature for displaying where modules/plugins have been loaded from -- maybe we could use that.05:01
dmsimardhttps://paste.fedoraproject.org/paste/go6TYZOHJplQquSGMUAeZw05:03
dmsimardansible-doc -F05:03
*** elyezer has joined #zuul05:08
*** harlowja has quit IRC05:21
*** elyezer has quit IRC05:59
*** elyezer has joined #zuul06:07
*** flepied has joined #zuul06:31
*** flepied has quit IRC06:35
*** xinliang has quit IRC06:47
*** JasonCL has joined #zuul06:53
*** JasonCL has quit IRC06:58
*** xinliang has joined #zuul07:00
*** xinliang has quit IRC07:00
*** xinliang has joined #zuul07:00
*** electrofelix has joined #zuul07:42
*** hashar has joined #zuul07:53
*** xinliang has quit IRC08:40
*** xinliang has joined #zuul08:53
*** xinliang has quit IRC08:53
*** xinliang has joined #zuul08:53
*** xinliang has quit IRC09:11
*** xinliang has joined #zuul09:23
*** xinliang has quit IRC09:23
*** xinliang has joined #zuul09:23
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: github: add missing application permission to the doc  https://review.openstack.org/55791509:25
*** maeca has quit IRC09:55
tristanCit seems like github doesn't remove approved review when new commits are addded to a PR...10:03
tristanCis there a working github gate pipeline available somewhere?10:04
*** elyezer has quit IRC10:58
*** elyezer has joined #zuul10:59
*** elyezer has quit IRC11:09
*** elyezer has joined #zuul11:13
*** mrhillsman has quit IRC11:48
*** odyssey4me has quit IRC12:19
*** odyssey4me has joined #zuul12:19
*** JasonCL has joined #zuul13:08
*** elyezer has quit IRC13:10
*** elyezer has joined #zuul13:10
*** JasonCL has quit IRC13:12
*** electrofelix has quit IRC13:21
*** elyezer has quit IRC13:28
*** elyezer has joined #zuul13:31
mordredtristanC: 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 that13:56
mordredtristanC: that said, I think tobiash has one - perhaps he's got insight13:57
tristanCmordred: oh ok, that's good to know13:58
tristanCmordred: the mitigation we are trying is to require a "zuulit" label to enter the gate pipeline, and unlabel it when the check pipeline starts14:00
mordredtristanC: yes - I think this is likely the best bet - and is very similar to what ansible/ansible does via their ansibullbot14:00
tristanCperhaps the github event filter could check if the approved review are more recent than last pr update timestamp?14:17
mordredtristanC: 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 ui14:20
mordredtristanC: (although that's not a bad idea otherwise)14:20
tristanCzuul could post a "please re-approve the last update" comment14:21
*** weshay is now known as weshay_mod14:37
tobiashtristanC: dismissal of reviews on pr update is configurable in the branch protection14:42
tobiashExcept negative reviews which always stay14:43
corvusif anyone wants to write a 'howto setup gating' doc, that probably wouldn't be a bad idea14:48
tobiashtristanC: this is my current gate pipeline: http://paste.openstack.org/show/718024/14:48
tobiashbut it relies on https://review.openstack.org/53568014:49
tobiashwithout it you will need to add the check status to the requirements14:49
tobiashpabelanger: ^14:50
tristanCtobiash: thanks!14:50
tristanCtobiash: btw, for post and release pipeline, we found "ref: ^refs/heads/.*$" and "ref: ^refs/tags/[0-9]+(\.[0-9]+)*$" do the trick on push event14:51
tobiashtristanC: regarding automatic review dismissal: https://help.github.com/articles/enabling-required-reviews-for-pull-requests/14:52
tobiashstep 714:52
tobiashtristanC: cool, thanks14:52
openstackgerritMerged openstack-infra/zuul master: Cleanup use of mock package in for python2  https://review.openstack.org/55786015:01
tobiashcorvus: what do you think about eventually moving the build times db to zk?15:01
corvustobiash: 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 that15:03
*** JasonCL has joined #zuul15:05
corvustobiash: we have 26,000 entries in ours.15:06
tobiashHrm, ok15:07
tobiashThat's not on the critical path regarding functionality right?15:08
fungii have no real frame of reference, but that certainly is a large-sounding number15:08
corvustobiash: no, but i think it's important for folks looking at the status page.  i'd like to avoid declaring it optional.15:09
corvusfungi: job*project*branch15:09
tobiashI'd at least like to keep the sql db out of the critical path for the basic ci operations15:09
tobiashBecause of availability15:10
corvustobiash: ah, i see.15:10
corvusthat's a good point15:10
corvuswe 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
openstackgerritPaul Belanger proposed openstack-infra/nodepool master: Add debian-stretch to nodepool-functional-py35-debian-src  https://review.openstack.org/55794215:13
pabelangercorvus: tobiash: tristanC: yah, a howto setup gating doc is a great idea15:14
pabelangertobiash: thanks for info, looking15:15
tobiashcorvus: I think requiring it is fine (I use it too) but ci functionality should work without15:17
corvus++15:18
*** JasonCL has quit IRC15:21
*** JasonCL has joined #zuul15:23
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Add test-emit-job-header roles  https://review.openstack.org/55794615:43
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Add nodepool hostnames to test-emit-job-header  https://review.openstack.org/55794715:43
*** elyezer has quit IRC15:49
*** elyezer has joined #zuul15:51
mordredcorvus, 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 data16:25
mordredback out that zuul-web could use ...16:25
openstackgerritFabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config  https://review.openstack.org/53551116:25
mordredif 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 question16:26
openstackgerritFabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config  https://review.openstack.org/53551116:26
mordredalternately though, to serve AIO installs simply if we required SQL for dashboard purposes we could just make sure sqlite works16:28
*** hashar is now known as hasharAway16:59
*** EmilienM has quit IRC17:58
*** elyezer has quit IRC17:59
*** elyezer has joined #zuul18:00
corvusmordred: 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 think18:16
*** mrhillsman has joined #zuul18:34
openstackgerritPaul Belanger proposed openstack-infra/zuul master: Add reporter for Federated Message Bus (fedmsg)  https://review.openstack.org/55798719:15
pabelangertristanC: ^fedmsg reporter19:16
pabelangerI still need to add some docs and nodepool-dsvm integration testing19:19
pabelangertflink: ^19:22
*** elyezer has quit IRC19:22
*** lids361 has joined #zuul19: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
lids361jgs .'  \ =  _= _ = _= /`\19:24
lids361jgs .'  \ =  _= _ = _= /`\19:24
lids361/     `-;----=--;--'   \19:24
lids361/     `-;----=--;--'   \19:24
lids361\    _.-'        '.    /19:24
lids361\    _.-'        '.    /19:24
lids361`""`              `""`19:24
lids361`""`              `""`19:24
lids361L0DE AND CHRON FROM #LRH & L0DE RADIO HOUR (IRC.EFNET.ORG) WANTED TO SAY HAPPY EASTER!!19:25
lids361L0DE AND CHRON FROM #LRH & L0DE RADIO HOUR (IRC.EFNET.ORG) WANTED TO SAY HAPPY EASTER!!19:25
lids361mrhillsman 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 dtruo19:25
*** lids361 has quit IRC19:25
*** elyezer has joined #zuul19:29
openstackgerritPaul Belanger proposed openstack-infra/zuul master: Add reporter for Federated Message Bus (fedmsg)  https://review.openstack.org/55798719:47
*** elyezer has quit IRC20:20
*** elyezer has joined #zuul20:23
*** mordred has quit IRC20:26
*** mordred has joined #zuul20:27
*** elyezer has quit IRC20:29
*** elyezer has joined #zuul20:31
*** elyezer has quit IRC20:39
*** elyezer has joined #zuul20:40
*** harlowja has joined #zuul21:28
*** hasharAway has quit IRC22:21
mrhillsmanshould i expect to see a comment on this pr? https://github.com/kubernetes/cloud-provider-openstack/pull/7622:36
mrhillsmanhttps://github.com/kubernetes/cloud-provider-openstack/commit/320fff3f33f227c2e009d647e4ad5390ea57ba6522:36
clarkbmrhillsman: I dont know, is there config in a zuul somewhere running jobs on that repo?22:38
mrhillsmanthe app got installed22:38
mrhillsmani updated main.yml22:38
mrhillsmanhttps://www.irccloud.com/pastebin/Pokw9dnd/22:38
mrhillsman2018-03-30 21:34:27,713 DEBUG zuul.Scheduler: Change <Change 0x7f8dc1fa28d0 76,320fff3f33f227c2e009d647e4ad5390ea57ba65> has been updated, clearing dependent change caches22:39
mrhillsmanzuul yaml file is in place22:39
clarkbyou should see log lines like considering change foo in check22:40
clarkbthat us when zuul determines what (if any) jobs to run for changes22:40
mrhillsmani don't need to restart anything after updating main.yml do i?22:42
clarkbyou have to do a config reload but not a restart22:42
mrhillsmanah ok22:42
mrhillsmani thought it was automagic22:42
mrhillsmanneeded the reload, thx clarkb22:57
pabelangermrhillsman: app installed is good news23:09
mrhillsmanyep23:09
mrhillsmandefinitely the easier route23:10
mrhillsmani do have someone investigating the plugin for prow too23:10
pabelangeryah23:11
mrhillsmanshould there be a useful case to indeed have one23:11
mrhillsmanclarkb or pabelanger i need to change the job name in .zuul.yaml with a pr will that get picked up23:25
pabelangermrhillsman: yes, for untrusted projects right away23:27
mrhillsmanty sir23:27
pabelangeryou'll also need to update your pipeline with the new job name too23:27

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!