Monday, 2018-01-08

*** JasonCL has quit IRC00:43
*** JasonCL has joined #zuul00:44
*** JasonCL has quit IRC00:51
*** JasonCL has joined #zuul00:52
*** JasonCL has joined #zuul00:53
*** JasonCL has quit IRC00:54
*** sshnaidm has quit IRC01:11
*** sshnaidm has joined #zuul01:12
*** JasonCL has joined #zuul01:32
*** JasonCL has quit IRC01:38
*** sshnaidm has quit IRC01:39
*** sshnaidm has joined #zuul01:40
*** dkranz has quit IRC01:57
*** JasonCL has joined #zuul02:01
*** JasonCL has quit IRC02:47
*** JasonCL has joined #zuul02:49
*** JasonCL has quit IRC02:54
*** JasonCL has joined #zuul03:29
*** JasonCL has quit IRC03:33
*** JasonCL has joined #zuul03:41
*** JasonCL has quit IRC03:46
*** JasonCL has joined #zuul04:03
*** JasonCL has quit IRC04:07
*** bhavik1 has joined #zuul04:33
*** bhavik1 has quit IRC04:50
*** JasonCL has joined #zuul04:56
*** JasonCL has quit IRC05:01
*** JasonCL has joined #zuul05:51
*** JasonCL has quit IRC05:56
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Don't treat finger client disconnect as exception  https://review.openstack.org/53140806:24
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Share a fake pull request database across connections  https://review.openstack.org/53150606:41
*** JasonCL has joined #zuul06:45
*** JasonCL has quit IRC06:50
*** JasonCL has joined #zuul06:56
openstackgerritMerged openstack-infra/nodepool feature/zuulv3: Remove the default argument to secure in nodepoolcmd  https://review.openstack.org/48031006:58
*** JasonCL has quit IRC07:01
*** Shrews has quit IRC07:13
*** Shrews has joined #zuul07:14
*** JasonCL has joined #zuul07:37
*** JasonCL has quit IRC07:42
*** pabelanger has quit IRC08:19
*** weshay has quit IRC08:19
*** pabelanger has joined #zuul08:20
*** weshay has joined #zuul08:20
*** mhu has quit IRC08:20
*** mhu has joined #zuul08:24
*** JasonCL has joined #zuul08:32
*** hashar has joined #zuul08:36
*** JasonCL has quit IRC08:36
*** jpena has joined #zuul08:38
*** sshnaidm is now known as sshnaidm|afk09:07
*** sshnaidm|afk has quit IRC09:08
*** sshnaidm has joined #zuul09:08
*** JasonCL has joined #zuul09:19
openstackgerritMatthieu Huin proposed openstack-infra/nodepool feature/zuulv3: Add a separate module for node management commands  https://review.openstack.org/53170309:58
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Set remote url on every getRepo in merger  https://review.openstack.org/52929310:29
openstackgerritliusheng proposed openstack-infra/zuul feature/zuulv3: Use hotlink instead log url in github job report  https://review.openstack.org/53154510:34
*** electrofelix has joined #zuul10:56
openstackgerritMatthieu Huin proposed openstack-infra/nodepool feature/zuulv3: [WIP] webapp: add optional admin endpoint  https://review.openstack.org/53171811:00
openstackgerritMatthieu Huin proposed openstack-infra/nodepool feature/zuulv3: Refactor status functions, add web endpoints, allow params  https://review.openstack.org/53139011:02
*** rcarrill1 has joined #zuul11:32
*** rcarrillocruz has quit IRC11:35
*** jkilpatr has quit IRC11:50
openstackgerritFabien Boucher proposed openstack-infra/zuul-jobs master: Add roles for static web content build and publication  https://review.openstack.org/53140612:11
openstackgerritFabien Boucher proposed openstack-infra/zuul-jobs master: Add roles for static web content build and publication  https://review.openstack.org/53140612:14
*** jkilpatr has joined #zuul12:22
*** rcarrill1 is now known as rcarrillocruz12:24
openstackgerritFabien Boucher proposed openstack-infra/zuul-jobs master: Add roles for static web content build and publication  https://review.openstack.org/53140612:26
*** jkt_ is now known as jkt12:30
*** jpena is now known as jpena|lunch12:33
openstackgerritTobias Henkel proposed openstack-infra/zuul feature/zuulv3: Also prefix the indexes if needed  https://review.openstack.org/53174212:55
*** sshnaidm has quit IRC13:14
*** sshnaidm has joined #zuul13:15
*** dkranz has joined #zuul13:21
*** jpena|lunch is now known as jpena13:31
*** jkilpatr has quit IRC13:31
*** rlandy has joined #zuul13:38
*** jkilpatr has joined #zuul13:44
openstackgerritFabien Boucher proposed openstack-infra/zuul-jobs master: Add roles for static web content build and publication  https://review.openstack.org/53140613:48
*** JasonCL has quit IRC13:53
*** JasonCL has joined #zuul13:55
mrhillsmandoes NODE_FAILURE mean there are no VMs to run a job?14:16
mrhillsmanor could mean a number of different things?14:17
dmsimardmrhillsman: perhaps nodepool logs would have hints14:20
dmsimardor zuul executor14:20
mrhillsmanreally i just want to be sure that this would not be for something rudimentary as i only have 2 VMs available and 3 jobs kick off so job 3 returns NODE_FAILURE rather than waiting for a new VM14:21
mrhillsmanmy understanding is job 3 would queue and when a new VM is built that it can run on it would then run14:22
mordredmrhillsman: yes - job 3 *should* queue and wait14:23
Shrewsmrhillsman: check nodepool logs for the failed request number. should be more info there as to why the request couldn't be fulfilled14:25
mrhillsmanthx, need to go look at some logs now :)14:25
openstackgerritFabien Boucher proposed openstack-infra/zuul feature/zuulv3: Make Zuul able to start with a broken config  https://review.openstack.org/52906014:56
*** rlandy_ has joined #zuul15:09
openstackgerritMatthieu Huin proposed openstack-infra/nodepool feature/zuulv3: [WIP] webapp: add optional admin endpoint  https://review.openstack.org/53171815:33
rcarrillocruzhey folks, it seems the gitdriver merged, does it mean i can consume openstack infra upstream jobs/roles?15:56
rcarrillocruzif so, any pointers of anyone doing that? config files, examples, etc15:56
*** JasonCL has quit IRC16:05
*** JasonCL has joined #zuul16:12
pabelangerrcarrillocruz: yah, leifmadsen_ and I are going to try using it this week for quickstart docs16:13
rcarrillocruzyou don't have a branch for it yet , do you?16:14
pabelangerhttps://gist.github.com/leifmadsen/93b9283d10dfddba096e32fb172cf569 is the latest version, we need to import into RST this week16:14
leifmadsen_before that, need to figure out the actual sample configuration, so probably a bunch of changes to the gtest-org16:15
leifmadsen_or another repo16:15
pabelangerrcarrillocruz: however, we haven't tried using git driver just yet, just github16:15
leifmadsen_yea, just github so far16:15
leifmadsen_git didn't exist at the time :)16:15
rcarrillocruzright, same here16:16
leifmadsen_would be easy enough to test though at the end of that set of docs though16:16
leifmadsen_I suspect I'll find at least 3 bugs16:16
leifmadsen_(not because I mean that the implementation is terrible, just that I'm a magnet for bugs :))16:30
leifmadsen_pabelanger: when do you want to get together again? I'm out on Wednesday, and I'm out all of next week16:30
leifmadsen_honestly, today might really be my best day this week16:31
pabelangerleifmadsen_: tomorrow? I am on dadops today, so in and out16:31
leifmadsen_tomorrow no good, too many meetings16:31
leifmadsen_Maybe Thursday afternoon16:31
leifmadsen_possibly tomorrow afternoon16:31
pabelangerYah, today is my worst day this week16:31
leifmadsen_all good, it's Monday16:32
leifmadsen_actually, lots of meetings, but mostly community stuff I could skip16:32
leifmadsen_so we could likely do tomorrow16:32
leifmadsen_don't be at that coffee shop again :)16:32
leifmadsen_it doesn't have Mr. Robot coffee shop bandwidth16:32
pabelangerleifmadsen_: Yah, I'll have the bandwidth :)16:34
*** JasonCL has quit IRC16:34
*** kmalloc has joined #zuul16:38
dmsimardDo we really want to point to the develop branch of github3 ? There's a lot of commits there on a regular basis.16:40
corvusdmsimard: nope, there's a story about fixing that.  jlk is on it.16:40
dmsimardok16:40
*** jkilpatr has quit IRC16:44
*** jpena is now known as jpena|brb16:44
*** JasonCL has joined #zuul16:50
*** jkilpatr has joined #zuul16:59
*** bhavik1 has joined #zuul17:10
*** bhavik1 has quit IRC17:11
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: Support cross-source dependencies  https://review.openstack.org/53080617:23
*** jpena|brb is now known as jpena17:24
fdegir leifmadsen_: regarding zfs documentation - it looks pretty good and I setup zuul prototype running against opnfv by following it17:38
leifmadsen_zfs docs?17:38
fdegirleifmadsen_: so, thanks for that and i’ll continue watching it and trying it behind the scenes17:38
fdegirleifmadsen_: yes, zfs gist17:38
leifmadsen_I didn't write a zfs gist...17:39
leifmadsen_oh wait... you created an acronym from Zuul From Scratch17:39
fdegiryep17:39
leifmadsen_I thought you were talking about the zfs filesystem17:39
leifmadsen_I was so confused17:39
fdegirnope17:39
fdegirsorry, laziness17:39
leifmadsen_:)17:39
fdegiri also setup on ubuntu as well17:39
leifmadsen_coolio17:39
leifmadsen_this was Fedora only in my testing17:40
fdegirso both fedora and ubuntu worked for me17:40
leifmadsen_awesome17:40
leifmadsen_yea, other than package names, not sure much different17:40
fdegirthere were few things i had to fix but they mght have been fixed on the latest version17:40
leifmadsen_ideally I can get pabelanger to fix the GitHub package conflicts :)17:40
leifmadsen_latest version is whatever is shown17:40
fdegirzookeeper setup is different on ub7ntu17:40
leifmadsen_if you find issues, best to note them otherwise I won't find it17:40
leifmadsen_ah ok17:41
leifmadsen_yea, in that case, not really a high level priority for me17:41
fdegirand on fedora, zookeeper logging doesn’t work17:41
fdegirneed log4j in CLASSPATH17:41
leifmadsen_would be ideal to note that in the docs for me17:41
fdegirat least it was like that for me17:41
leifmadsen_I will forget all of this in about 15 seconds17:41
fdegiri’m recording them on my side17:41
fdegironce you start moving things to official docs, i can perhaps read it, those pieces and send for review17:42
fdegirand also I can add ubuntu instructions as well if anyone is interested17:42
*** JasonCL has quit IRC17:44
*** JasonCL has joined #zuul17:45
*** JasonCL has quit IRC17:55
*** JasonCL has joined #zuul17:58
*** jpena is now known as jpena|off18:18
*** hashar is now known as hasharDinner18:19
*** jkilpatr has quit IRC18:19
*** jkilpatr has joined #zuul18:33
*** JasonCL has quit IRC18:41
*** JasonCL has joined #zuul18:49
*** JasonCL has quit IRC18:53
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: Add support for a secure ZooKeeper configuration  https://review.openstack.org/53188218:56
Shrews^^^ made me realize that we (maybe just me?) have woefully neglected the secure stuff.18:56
corvusShrews: i think we deferred it at the time.  now is a good time to address it.  :)18:59
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: Rename _useBuilder method to useBuilder  https://review.openstack.org/53188318:59
*** JasonCL has joined #zuul19:00
*** harlowja has joined #zuul19:19
clarkbleifmadsen_: fdegir re zookeeper stuff like that is why I have suggested in the past it might be easiest to just use the official zookeeper docs for installing and configuring that19:24
clarkbrather than rely on per distro packaging and docing all those differences ourselves19:24
*** jkilpatr has quit IRC19:33
SpamapSleifmadsen_: where can I watch/review your getting started work? I've now brought up zuulv3 3 or 4 times so I'd like to help (but have found little time to "get started" on the "get started" docs myself. ;)19:34
leifmadsen_https://gist.github.com/leifmadsen/93b9283d10dfddba096e32fb172cf56919:34
leifmadsen_WIP19:34
leifmadsen_at this point, everything "works" on Fedora, but there isn't really much in the way of configuration for zuul itself yet19:35
leifmadsen_that's what pabelanger and I are going to do tomorrow19:35
leifmadsen_SpamapS: that's the issue :) everyone says they want to write docs, then real work gets in the way19:36
SpamapSleifmadsen_: well also I could sit down and hammer something out but it would likely duplicate what you've done. Can't birth a baby with more mommies. ;)19:37
leifmadsen_:)19:37
leifmadsen_yea, at this point, the docs get Zuul installed19:37
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: Support cross-source dependencies  https://review.openstack.org/53080619:37
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Move CRD tests to test_gerrit_legacy_crd  https://review.openstack.org/53188619:37
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add skipped CRD tests  https://review.openstack.org/53188719:37
leifmadsen_but there is almost no configuration19:37
leifmadsen_so it's still a thing19:37
leifmadsen_just doesn't get you very far19:38
SpamapSYeah that actually turns out to be a huge challenge.19:38
SpamapSEspecially getting a good working base job.19:38
leifmadsen_yep. in theory fungi is working on a new base job that can be easily consumed, and pabelanger has this gtest-org that we're probably going to make work as an example configuration19:38
leifmadsen_the idea being that you can use the git module and consume a repo that will give you something "working"19:39
SpamapSI still kind of wish there was a way to make Zuul automatically run every single job that parents to a job you're changing. like 'check: { jobs: $children }' or something like that.19:39
SpamapSI have a pretty stripped down base job that works for me in single-server mode. I'm happy with it. But one thing I had to do was mount in my log target to bwrap so I didn't have to scp things back to the same server.19:40
SpamapSThat is nice for single server but not when I make my second executor. :-P19:41
fungileifmadsen_: SpamapS: the new base job exists. it's in the git.openstack.org/openstack-infra/zuul-base-jobs repo19:41
leifmadsen_rock19:41
fungii just need to update the zuul user guide to mention where to find it19:41
fungihttps://docs.openstack.org/infra/zuul-base-jobs/19:41
*** JasonCL has quit IRC19:51
*** JasonCL has joined #zuul19:55
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: Add support for a secure ZooKeeper configuration  https://review.openstack.org/53188219:57
dmsimardShrews, gundalow, rcarrillocruz: how bad of an idea would it be to override/overload the ssh connection plugin ? I'd like to have a more verbose error here (with the exception stack traces) https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/connection/ssh.py#L520-L53520:00
dmsimardWe're getting a lot of UNREACHABLE tasks and "Make sure this host can be reached over ssh" is not super helpful20:00
dmsimardWorking our way up the stack traces may prove helpful in seeing what kind of bottleneck or problem we're hitting20:01
*** jkilpatr has joined #zuul20:11
Shrewsno idea20:11
dmsimardI suppose we could also be hitting https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/connection/ssh.py#L867 actually20:13
dmsimardexit code 255 in ssh is fairly catch all20:15
rcarrillocruzyup, that except is actually hiding the issues20:22
rcarrillocruzyou could play with traceback20:22
rcarrillocruzor with q20:22
rcarrillocruzto debug what it actually is the exception20:22
rcarrillocruzin other news20:22
rcarrillocruzhttp://zuul-web-9000-ansible-networking-zuul-ci.38.145.34.172.nip.io/20:22
rcarrillocruzmucho bonito20:22
rcarrillocruzi like the new dashboard20:22
hasharDinnerhello! If there is any Gerrit review, https://review.openstack.org/#/c/162315/ "Migrate zuul timer to apscheduler version 3.0" can be abandonned.  That has been done later via  https://review.openstack.org/#/c/260637/20:26
dmsimardrcarrillocruz: yeah the zuul dashboard is neat20:28
*** openstack has quit IRC20:38
*** openstack has joined #zuul20:42
*** ChanServ sets mode: +o openstack20:42
hasharmy last comment is about making that change to bump Paramiko minimum version  and move a few imports back at the top of the file20:43
openstackgerritAntoine Musso proposed openstack-infra/zuul master: zuul-cloner: recognizes bare repos in cache dir  https://review.openstack.org/24920720:45
hashar^^ is another example :]20:46
fungithat's the one i was talking about above20:50
corvusfungi, hashar: the only reason we haven't merged v3 onto master is that we haven't landed the puppet-openstackci changes to make it safe for people using that20:52
corvusother than that delay, we would have done it already20:52
hasharcorvus: so I guess master / 2.x is under embargo utnil the merge happens isn't it ?20:52
hashar"until"20:53
corvushashar: i feel that way -- i made what i thought would be the last 2.x release; i'm focused tightly on making a v3.0 release and don't have the bandwidth to do any more on 2.x20:53
corvus(we're hoping to do the 3.0 release no later than the end of feb)20:54
hasharand I guess most of your third parties are on (or are migrating to) Zuul v320:54
corvushashar: more than i would have expected, actually :|20:54
hasharWikimedia is probably going to stick to v2 + a few patches for most of 201820:55
hasharso that is like a v2 fork :]20:56
hasharfungi: corvus: I will rebase all the changes I got cherry picked against the tip of master. But if 2.x is frozen, dont bother reviewing / merging them I guess20:57
hasharand maybe one day I will catch up with v3!20:57
corvushashar: i hope so!20:58
fungicool, so just to confirm, we should consider master basically frozen until we merge the feature branch into it20:59
corvusfungi: let's go with 'yes'21:01
openstackgerritAntoine Musso proposed openstack-infra/zuul master: Add __repr__ to a few classes  https://review.openstack.org/36106421:01
hasharshould I mark my patches as WIP ?21:01
corvusand review https://review.openstack.org/523951 if folks have time21:01
corvushashar: i wouldn't worry about it21:02
hasharOK :]21:03
*** openstackgerrit has quit IRC21:03
*** openstackgerrit has joined #zuul21:18
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: Support cross-source dependencies  https://review.openstack.org/53080621:18
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove updateChange history from github driver  https://review.openstack.org/53190421:18
*** JasonCL has quit IRC21:18
pabelangerany suggestions on how to fake some memory values?21:20
openstackgerritPaul Belanger proposed openstack-infra/zuul feature/zuulv3: Add memory awareness to system load governor  https://review.openstack.org/50896021:25
pabelangerSpamapS: okay, rebase ^. going to see how we can come up with a test21:25
hasharpabelanger: "just"  mock psutil.virtual_memory() to yields whatever you need ? :D21:30
pabelangerhashar: yah, that's my first guess. Wasn't sure if there was another suggestion21:33
clarkbpabelanger: you need to ballon your memory usage?21:34
clarkbyou are probably better off mocking or faking whatever it is that is checking memory usage21:34
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Correctly use project name in getGitUrl  https://review.openstack.org/52830921:35
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Initialize github auth in getGitUrl if needed  https://review.openstack.org/52831021:35
pabelangerclarkb: okay, I can look into doing that first21:36
clarkbor set the check threshold so low that you'd always trip it just from the zuul test suite running21:36
clarkbeg 1 bte21:36
clarkb*1 byte21:36
corvusyep, this is a good use of mocks :)21:37
hasharpabelanger: and probably you can refactor the check of load and memory limits to a couple standalone functions. Might end up nicer than all those if elif elif :D21:38
openstackgerritJeremy Stanley proposed openstack-infra/zuul feature/zuulv3: Link to zuul-base-jobs docs from User's Guide  https://review.openstack.org/53191221:39
fungicorvus: i've got a conflicting engagement with the zuul meeting today, but ^ is hopefully the next bit to filling the stdlib base job gap21:39
fungileifmadsen_: SpamapS: as mentioned in our earlier discussion ^21:39
pabelangerhashar: agree, I was thinking of doing that after I get some tests working21:39
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Use configured github server in app mode  https://review.openstack.org/52831121:39
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Fix github app authentication in _getNeededByFromPR  https://review.openstack.org/52831221:39
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Add the project to GithubUser  https://review.openstack.org/52831321:39
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Reprime the installation map in getGitUrl if needed  https://review.openstack.org/52950621:39
hasharpabelanger: also you could have manageLoad be a wrapper that retrive the memory and load then invoke something like IsWithInLimits(load, mem)  which would then be trivial to unit test21:39
hasharoh and Zuul supports Github nowadays nice21:40
*** dkranz has quit IRC21:43
corvushashar: yeah, we're up to 3 source drivers now: github, gerrit, git (bare git is limited and is basically a read-only driver for dependencies at the moment).  but the api is definitely getting fleshed out.  :)21:44
corvusfungi: thanks!21:44
mrhillsmanis it too farfetched of an idea for zuul to access an openstack environment via vpn?21:45
fungicorvus: there were several other places in the user's guide we _could_ conceivably also link to it, but wasn't sure whether they made sense, or whether we should centralize that link somewhat in the user guide and then refer back to wherever that central section is21:46
fungiso suggestions welcome, of course21:46
fungimrhillsman: transport mode or tunnel mode?21:47
fungitunnel mode is "easy" (as long as your gateways have tunnels preestablished for you)21:47
clarkbfungi: mrhillsman but also zuulv3 has been designed to better accomodate corporate network designs by making everything outbound connections21:48
clarkbso depending on why the vpn is there you may not need it?21:48
mrhillsmanah ok21:48
mrhillsmanthe VMs do not have public IP addresses21:49
mrhillsmanno 4/6 address space available and zuul lives elsewhere21:49
fungimrhillsman: yeah, then your zuul would need a route to those (which a vpn could provide)21:49
clarkbanother option may be to run zuul-executor in the same network space21:50
clarkbthen only manage that connection21:50
fungitrue, could even multi-home the executor on internal and globally-addressed networks21:50
clarkbas for bouncing through a vpn it should just work though as long as the executor can route from itself to the vpn'd network21:50
SpamapSthat then makes it so you have to have git access from the executor ;)21:50
fungiwell, assuming egress is viable via nat/pat21:51
mrhillsmanawesome, i need to get more VMs available and i have an openstack environment but it is behind a vpn21:51
mrhillsmanthanks for the details21:51
fungieven a privately-addressed executor could still hit globally-routed git sources through egress nat21:51
mrhillsmanwill explore a few options21:51
mrhillsmanthe VMs can get out but all ingress is "blocked"21:52
clarkbmrhillsman: in that case running executor in that env may be easiest21:52
mrhillsmanand nodepool?21:52
dmsimardzuul meeting in 7 minutes21:53
clarkbbut I'm not sure if you can tell zuul that specific executors must be used to talk to specific test nodes21:53
clarkbmrhillsman: nodepool only talks to cloud apis not the test nodes21:53
clarkb(this assumes zuulv3)21:53
mrhillsmanah ok21:53
mrhillsmanyes, using v321:53
corvusclarkb, mrhillsman: there's a story about executor affinity in storyboard.  it's open for implementation if someone wants to work on it.21:54
Shrewsclarkb: mrhillsman: nodepool will do a keyscan on the nodes21:55
mrhillsmanmaybe i can bug one of you next week to go over the architecture and get some guidanc21:55
mordredmrhillsman: ++21:55
clarkbShrews: hrm that does break the assumption that nodepool won't connect at all to test nodes21:55
mrhillsmani have a small openstack environment and there is a larger one being stood up but i think i will have limited root access but leeway to do as i please with quota21:56
clarkbso you may have to put the nodepool launcher in the same env21:56
clarkbor run both nodepool launcher and zuul executor on nodes that can route (via vpn or whatever) to the test nodes21:56
corvusmrhillsman, clarkb: https://storyboard.openstack.org/#!/story/200112521:56
mordredmrhillsman: I think it's a pattern that'll come up again ... and also once it's implemented it might also be useful in scale-out multi-cloud environments - so definitely useful to connect on architecture/guidance21:56
mrhillsmanbut i am sure i could have things re-architected if necessary21:57
mrhillsmancool21:57
mrhillsmanagreed mordred - thinking blog mini-whitepaper for others to use21:57
clarkbcorvus: idea is have executor register execute-job:$zone if it is configured with a zone?21:57
corvusclarkb: yep21:57
corvusclarkb: pretty sure storyboard ate some markup on bullet point #221:58
clarkbcorvus: then any nodepool node sets with a zone set must uze execute-job:$zone rather than the default execute-job job21:58
clarkbmakes sense21:58
corvusclarkb: yep21:58
corvus* Register the zuul:execute function as zuul:execute:<zone>21:59
corvusyep ^21:59
corvusclarkb: i changed it to your metasyntactic markup :)21:59
clarkbthe <zone> was killed by markdown21:59
corvusit's zuul meeting time in #openstack-meeting-alt22:00
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: Add support for a secure ZooKeeper configuration  https://review.openstack.org/53188222:05
corvusjlk: are you around?  can you join the zuul meeting ^ if you have a moment?22:08
openstackgerritClark Boylan proposed openstack-infra/nodepool feature/zuulv3: Clarify provider manager vs provider config  https://review.openstack.org/53161822:11
*** JasonCL has joined #zuul22:16
*** rlandy is now known as rlandy|bbl22:47
*** hashar has quit IRC22:58
clarkbseriously though please take a few minutes to look over https://etherpad.openstack.org/p/zuulv3-issues and update anything you can and/or file bugs23:01
clarkbI've been interspersing that with other work and it does take a bit of time if you have to context switch in an entire problem to then go figure out what to do with it so if you already have some of that context it will likely go quicker for you23:01
*** JasonCL has quit IRC23:04
*** JasonCL has joined #zuul23:05
openstackgerritMatthieu Huin proposed openstack-infra/nodepool feature/zuulv3: [WIP] webapp: add optional admin endpoint  https://review.openstack.org/53171823:06
*** JasonCL has quit IRC23:09
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: Support cross-source dependencies  https://review.openstack.org/53080623:36
mordredgundalow: heads up - we're eversoslowly starting to turn on some zuul onto ansible/ansible - landing the patch that lets me update the shade job config to reference ansible/ansible now - will make sure that gets us the shade ansible-devel job set up properly, then will land the patch to start running shade's ansible functional tests on PRs to openstack modules23:36
mordredgundalow: so you still are currently unlikley to notice unless you just enjoy lurking on openstack patches - but thought I'd give you a heads-up anyway23:36
clarkbcorvus: https://review.openstack.org/#/c/531618/ is refactor that I mentioned briefly when we had problems with the quota changes before we all took time off for holidays23:37
corvusclarkb: cool, thanks.  i will look at that when my brain is not full of cross-repo-deps.  :)23:40
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add job for testing against devel of ansible  https://review.openstack.org/53193423:43
*** JasonCL has joined #zuul23:49

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