Friday, 2018-02-09

*** threestrands has joined #zuul00:27
corvusokay the v2 config does work... i got an error earlier which i can't reproduce00:34
*** elyezer has quit IRC00:42
*** elyezer has joined #zuul00:47
*** xinliang has joined #zuul00:48
*** xinliang has quit IRC00:48
*** xinliang has joined #zuul00:48
openstackgerritJames E. Blair proposed openstack-infra/zuul master: WIP Add zuul from scratch doc  https://review.openstack.org/54251800:52
corvuspabelanger, mordred, leifmadsen: i'm going through that and updating it and rstifying it.  i've gotten through the service startup.  next is setting up jobs and repos.00:53
*** harlowja has joined #zuul00:57
tristanCcorvus: mordred: i vote for keeping the nodepool webapp as-is then merge nodepool into zuul and use the zuul-web route registration interface00:59
*** elyezer has quit IRC01:00
corvusthat's an idea too... i like the idea of nodepool being usable without zuul, so if we do that, we'll want to be careful about how we do it, so we don't impact that.01:01
tristanCcorvus: couldn't it just be documented that the zuul-nodepool command doesn't need a scheduler or executor to be usable?01:04
tristanCor, zuul-launcher*01:04
tristanCsimilarly to installing gcc when you just need as01:05
*** elyezer has joined #zuul01:05
corvustristanC: that's one part of it -- we'd also want to think about whether the extra dependencies are a burden01:06
openstackgerritJames E. Blair proposed openstack-infra/nodepool master: Default min-ready to 0  https://review.openstack.org/54241001:06
openstackgerritJames E. Blair proposed openstack-infra/nodepool master: Store build logs automatically  https://review.openstack.org/54238601:06
*** xinliang has quit IRC01:07
*** xinliang has joined #zuul01:20
*** xinliang has quit IRC01:20
*** xinliang has joined #zuul01:20
*** xinliang has quit IRC01:26
*** threestrands has quit IRC01:29
*** xinliang has joined #zuul01:43
*** xinliang has quit IRC01:43
*** xinliang has joined #zuul01:43
*** gothicmindfood has quit IRC02:10
*** gothicmindfood has joined #zuul02:10
*** Wei_Liu has joined #zuul03:07
*** harlowja has quit IRC05:10
*** elyezer has quit IRC05:57
*** elyezer has joined #zuul06:05
openstackgerritMerged openstack-infra/nodepool master: Add default logging configuration  https://review.openstack.org/54192106:43
openstackgerritMerged openstack-infra/nodepool master: Exit launchers and builders immediately  https://review.openstack.org/54193006:53
openstackgerritMerged openstack-infra/nodepool master: Exercise pidfile before daemonizing  https://review.openstack.org/54231506:53
openstackgerritMerged openstack-infra/nodepool master: Resolve paths before demonization  https://review.openstack.org/54235306:53
openstackgerritMerged openstack-infra/nodepool master: Default min-ready to 0  https://review.openstack.org/54241006:53
*** elyezer has quit IRC07:23
*** elyezer has joined #zuul07:23
openstackgerritMerged openstack-infra/zuul-jobs master: Change the list of extensions to a dict  https://review.openstack.org/54048507:47
*** jpena|off is now known as jpena08:10
*** elyezer has quit IRC08:15
*** elyezer has joined #zuul08:18
*** hashar has joined #zuul08:43
*** elyezer has quit IRC10:44
*** elyezer has joined #zuul10:46
*** chrnils has joined #zuul11:24
*** elyezer has quit IRC12:07
*** elyezer has joined #zuul12:12
*** fdegir has quit IRC13:14
*** jpena is now known as jpena|lunch13:15
*** fdegir has joined #zuul13:15
*** kklimonda has quit IRC13:15
*** kklimonda has joined #zuul13:16
*** TheJulia has quit IRC14:02
*** TheJulia has joined #zuul14:03
*** jpena|lunch is now known as jpena14:18
*** weshay is now known as weshay|ruck14:24
*** sshnaidm has quit IRC14:24
*** kmalloc has quit IRC14:40
*** kmalloc has joined #zuul14:41
*** hogepodge has quit IRC14:51
*** hogepodge has joined #zuul14:52
tobiashmordred: trying out ansible 2.4 in reality14:56
tobiashlooks almost good so far14:56
tobiashwhat I noticed is that the inventory_file variable seems to be missing: http://paste.openstack.org/show/666928/14:57
tobiashhr, during normal playbooks it's there15:18
tobiashit seems that during plays targeting localhost the inventory_file variable is now missing :/15:19
*** jimi|ansible has quit IRC15:21
*** jimi|ansible has joined #zuul15:22
*** jimi|ansible has joined #zuul15:22
corvustobiash, mordred: can you take a look at https://review.openstack.org/542386 ?15:29
tobiashcorvus: wanted to review that this morning and then saw the -1 from ianw so I waited so see your feedback ;)15:30
corvusyeah, i'd be happy to add it to the logs, but the new job doesn't have any :(15:30
tobiashyeah, I also saw no nodepool logs at all15:30
corvusno devstack logs at all even15:31
tobiashlgtm15:31
corvusoh wait, i think i may have made a mistake15:31
corvusthe logs are there, i just didn't see them earlier15:32
tobiashhttp://logs.openstack.org/86/542386/3/check/nodepool-functional-py35/ac1e308/controller/15:32
tobiashthese?15:32
corvusyeah, and here's a builder log: http://logs.openstack.org/86/542386/3/check/nodepool-functional-py35/ac1e308/controller/logs/screen-nodepool-builder.txt.gz15:32
corvusokay, that's a totally valid -1 then.  let me fix it.15:33
tobiashah that is the reason I didn;t find them15:33
tobiashscreen-...15:33
tobiash:)15:33
tobiashk15:33
tobiashmordred: with delegate (as zuul-jobs is using for copying the inventory) it's there, so that should be a non-issue after I fixed my base job ;)15:37
*** elyezer has quit IRC15:39
tobiashmordred, corvus: just ran the first successful job with ansible 2.4 in my testenv :):):)15:47
mordredtobiash: woot!15:48
*** elyezer has joined #zuul15:48
openstackgerritJames E. Blair proposed openstack-infra/nodepool master: Store build logs automatically  https://review.openstack.org/54238615:50
tobiashBut I guess it's a bad idea to deploy it into production and then run straight away into vacation ;)15:53
*** hashar is now known as hasharAway15:54
*** hasharAway has quit IRC16:00
*** chrnils has quit IRC16:11
jtannermordred: who's the right person to talk to about https://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/ansible/library/command.py?16:51
dmsimardjtanner: I would venture to volounteer mordred :)16:52
fungijtanner: yeah, mordred accounts for ~42% of the commits to that file, followed by corvus with ~26%16:55
jtanneranyone know how logfile is consumed?16:55
jtanneri presume something in zuul reads it16:56
corvusjtanner: i have to run to an appointment now; i can answer questions when i get back (maybe an hour or two).  in the mean time, you may want to look at https://review.openstack.org/541434  to see how that's expected to change soon.16:56
tobiashjtanner: yes, it;s currently the zuul_stream module16:57
tobiashand what corvus said, it's being reworked currently by mordred16:57
tobiashs/zuul_stream/zuul_console/16:58
tobiashhttps://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/ansible/library/zuul_console.py16:58
tobiashthat's the sending side on the target ndoe16:59
pabelangerI always love peoples reaction when they seen ansible streaming in realtime :)16:59
*** elyezer has quit IRC17:03
*** myoung is now known as myoung|food17:04
openstackgerritMerged openstack-infra/nodepool master: Store build logs automatically  https://review.openstack.org/54238617:04
*** elyezer has joined #zuul17:04
jtannercorvus: tobiash dmsimard thanks, i think i'm good for now17:16
jtanneri may send some internal folks your way for further discussion if they want it17:16
dmsimardjtanner: like Tobiash said, it's bound to change soon (https://review.openstack.org/#/c/541434/) so be wary of forming opinions on the current implementation17:17
jtannerthis is ansible ... everything is always changing =)17:18
dmsimardindeed17:19
dmsimardjtanner: thanks for sending help our way :D17:23
*** harlowja has joined #zuul17:37
*** myoung|food is now known as myoung18:14
tobiashkklimonda: I've commented on https://review.openstack.org/#/c/54003518:22
*** hashar has joined #zuul18:48
*** jpena is now known as jpena|off19:05
*** elyezer has quit IRC19:11
*** elyezer has joined #zuul19:14
corvusjtanner: oh, yeah, sorry i didn't have full context earlier.  mordred's changes in progres are specifically intended to start working toward an upstreamable version of this19:15
*** dkranz has joined #zuul19:19
*** dkranz has quit IRC20:10
*** elyezer has quit IRC20:24
*** elyezer has joined #zuul20:24
*** yolanda has quit IRC20:28
*** hashar has quit IRC20:28
*** xinliang has quit IRC21:02
*** xinliang has joined #zuul21:04
*** xinliang has quit IRC21:04
*** xinliang has joined #zuul21:04
*** myoung is now known as myoung|off22:06
*** patrickeast has quit IRC22:26
*** patrickeast has joined #zuul22:26
pabelangerHmm, I am seeing a syntax error in nodepool: http://logs.openstack.org/18/543018/2/check/ansible-role-nodepool-fedora-27/96e06b1/job-output.txt.gz#_2018-02-09_21_05_38_48681422:39
pabelangerlet me see why that is22:39
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Add zuul from scratch doc  https://review.openstack.org/54251822:39
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Reclassify some debug log lines as info  https://review.openstack.org/54304422:39
corvuspabelanger, mordred, leifmadsen: ^ there's the zuul-from-scratch doc as a hopefully correct skeleton.  i think we can expand on it from there, but even as it is, if you run through it, you should end up with a working zuul talking to github22:40
pabelangerodd, using python2 for some reason22:40
pabelangercorvus: ack, will review shortly22:41
persiacorvus: This might be from my naiveté, but most of the doc seems to talk about github, which makes me wonder why "baseurl=https://git.openstack.org/" appears in /etc/zuul/zuul.conf on the base system.22:45
corvuspersia: it's to establish a connection to get a copy of the zuul-jobs and base-zuul-jobs repos.  the tutorial doesn't really use them yet (actually creating a job would likely use them).22:48
corvuspersia: zuul can have lots of connections to remote systems22:48
persiaAh, that makes perfect sense.  Apologies for the noise then.22:49
corvuspersia: all of this is just to answer your curiosity now -- of coures the doc needs to answer that as well.  but it's missing a lot of *words* right now :)22:49
corvus(it's more of a shell script than actual documentation at this point) but it's in a form we can collaborate on now :)22:49
persiaI was just very excited to see "Add zuul from scratch doc" in my IRC feed, as I've failed the few times I've tried to set it up.  Yes it needs words, but at least all the necessary bits are there.22:50
corvuspersia: cool, it believe it's workable at this point, if you felt like exercising it22:51
corvusthat culminated in https://github.com/gtest-org/jeblair-test/pull/1 for me22:52
persiaIt looks like I have some yaks to shave first, but I'll try it (although I strongly recommend nothing waiting on my feedback)22:54
*** zaro_ has quit IRC23:11
*** zaro_ has joined #zuul23:11
*** elyezer has quit IRC23:18
*** elyezer has joined #zuul23:25

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