Monday, 2020-03-09

« Sunday, 2020-03-08 Index Tuesday, 2020-03-10 »
*** threestrands has joined #zuul00:12
*** Defolos has quit IRC00:13
*** Defolos has joined #zuul00:18
*** Defolos has quit IRC00:30
*** jamesmcarthur has quit IRC00:39
*** swest has quit IRC02:17
*** swest has joined #zuul02:31
*** bhavikdbavishi has joined #zuul03:01
*** jamesmcarthur has joined #zuul03:11
*** igordc has joined #zuul03:19
*** jamesmcarthur has quit IRC03:31
*** jamesmcarthur has joined #zuul03:35
*** jamesmcarthur has quit IRC03:40
*** bhavikdbavishi1 has joined #zuul04:47
*** bhavikdbavishi has quit IRC04:48
*** bhavikdbavishi1 is now known as bhavikdbavishi04:48
*** bolg has joined #zuul05:07
*** evrardjp has quit IRC05:35
*** evrardjp has joined #zuul05:35
*** igordc has quit IRC06:31
bolgtobiash, corvus: I tried out the ideas for https://review.opendev.org/c/708812 including the https://review.opendev.org/c/708812 and came to a conclusion keeping just the execute method:06:41
bolg1) we have only one common helper method,06:41
bolg2) we can reference a scheduler itself in a call (e.g. self.config -> app.config),06:42
bolg3) we can reference the wrapping SchedulerTestApp (e.g. self.scheds.execute(lambda app: app.smartReconfigure(command_socket=command_socket))),06:42
bolg4) we can call methods on properties of a scheduler (e.g. self.scheds.execute(lambda app: app.sched.run_handler_lock.acquire()))06:42
bolgThe proxy approach is neat and we can use it for simple calls, but there aren't so many.06:42
*** threestrands has quit IRC06:42
bolgWhat do you think?06:43
bolg^^^ (edit: ... including the https://review.opendev.org/c/711289...)06:48
*** AJaeger has quit IRC07:12
openstackgerritJan Kubovy proposed zuul/zuul master: Scheduler test app manager  https://review.opendev.org/70881207:12
openstackgerritJan Kubovy proposed zuul/zuul master: Use scheduler manager consistently in tests  https://review.opendev.org/70954207:12
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor executor_client in tests  https://review.opendev.org/70967207:12
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor merge_client in tests  https://review.opendev.org/70967607:12
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor nodepool in tests  https://review.opendev.org/70970307:12
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor zookeeper in tests  https://review.opendev.org/70970907:12
openstackgerritJan Kubovy proposed zuul/zuul master: Consolidate scheduler pause/exit as hibernation  https://review.opendev.org/70972307:12
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor `self.event_queues` in tests  https://review.opendev.org/70999007:12
openstackgerritJan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality  https://review.opendev.org/70973507:12
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper  https://review.opendev.org/70571607:12
*** AJaeger has joined #zuul07:16
*** dpawlik has joined #zuul07:23
*** hashar has joined #zuul07:46
*** tosky has joined #zuul08:15
*** jcapitao has joined #zuul08:26
*** jpena|off is now known as jpena08:31
*** Defolos has joined #zuul08:36
openstackgerritJan Kubovy proposed zuul/zuul master: Use scheduler manager consistently in tests  https://review.opendev.org/70954208:43
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor executor_client in tests  https://review.opendev.org/70967208:43
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor merge_client in tests  https://review.opendev.org/70967608:54
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor nodepool in tests  https://review.opendev.org/70970309:08
*** yolanda has quit IRC09:11
*** yolanda has joined #zuul09:11
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor zookeeper in tests  https://review.opendev.org/70970910:03
*** zbr|pto is now known as zbr10:20
openstackgerritJan Kubovy proposed zuul/zuul master: Consolidate scheduler pause/exit as hibernation  https://review.opendev.org/70972310:22
openstackgerritJan Kubovy proposed zuul/zuul master: Refactor `self.event_queues` in tests  https://review.opendev.org/70999010:26
openstackgerritJan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality  https://review.opendev.org/70973510:26
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper  https://review.opendev.org/70571610:26
*** sshnaidm|afk is now known as sshnaidm10:27
*** bolg has quit IRC10:58
*** Defolos has quit IRC11:16
*** Defolos has joined #zuul11:17
*** AJaeger has quit IRC11:33
*** sshnaidm has quit IRC11:36
*** bolg has joined #zuul11:38
*** sshnaidm has joined #zuul11:40
*** AJaeger has joined #zuul11:47
*** jpena is now known as jpena|lunch11:48
*** jcapitao is now known as jcapitao_lunch11:51
*** sshnaidm has quit IRC11:53
*** sshnaidm has joined #zuul11:54
*** rlandy has joined #zuul11:58
*** jamesmcarthur has joined #zuul12:10
*** jamesmcarthur has quit IRC12:14
*** jamesmcarthur has joined #zuul12:20
*** jpena|lunch is now known as jpena12:31
mnaserhmm12:33
mnaserlooks like safari doesn't properly render our UI at the moment12:33
mnaserhttps://usercontent.irccloud-cdn.com/file/AbeuSINZ/image.png12:33
*** jamesmcarthur has quit IRC12:36
*** Goneri has joined #zuul12:40
*** jamesmcarthur has joined #zuul12:47
*** AJaeger has quit IRC12:50
*** jamesmcarthur has quit IRC12:56
*** jamesmcarthur has joined #zuul12:57
*** jamesmcarthur has quit IRC13:02
*** zxiiro has joined #zuul13:04
*** sshnaidm has quit IRC13:08
*** sshnaidm has joined #zuul13:09
*** sshnaidm has quit IRC13:10
*** jamesmcarthur has joined #zuul13:13
*** jcapitao_lunch is now known as jcapitao13:14
*** sshnaidm has joined #zuul13:18
*** sshnaidm has quit IRC13:18
*** sshnaidm has joined #zuul13:19
*** hashar has quit IRC13:29
*** jamesmcarthur has quit IRC13:32
*** jamesmcarthur has joined #zuul13:32
*** jamesmcarthur has quit IRC13:38
*** avass has joined #zuul13:40
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Make revoke-sudo more general.  https://review.opendev.org/70626213:43
openstackgerritBenjamin Schanzel proposed zuul/zuul-jobs master: Kubernetes Node Support for Mirroring Git Repos  https://review.opendev.org/71192013:44
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Adds variable to toggle whether to revoke sudo  https://review.opendev.org/70624813:45
*** flaper87 has quit IRC13:48
*** erbarr has joined #zuul13:49
*** AJaeger has joined #zuul13:49
*** flaper87 has joined #zuul13:50
*** harrymichal has joined #zuul13:52
*** flaper87 has quit IRC13:55
*** flaper87 has joined #zuul13:57
*** flaper87 has joined #zuul13:58
*** flaper87 has quit IRC13:59
*** flaper87 has joined #zuul14:01
*** flaper87 has quit IRC14:01
*** jamesmcarthur has joined #zuul14:03
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Control log archive and user preservation with vars  https://review.opendev.org/70138114:04
Shrewsmnaser: that's interesting. the notification bell doesn't even show for me on safari14:07
ShrewsSafari 13.0.514:08
openstackgerritAlbin Vass proposed zuul/zuul master: Fix minor spelling error  https://review.opendev.org/71192614:08
*** jamesmcarthur has quit IRC14:09
Shrewscorvus: tristanC: the nodepool zk-auth change lgtm (except the commit message still mentions the script that has been removed... and i assume will appear in the zuul side change)14:09
*** sshnaidm has quit IRC14:15
*** sshnaidm has joined #zuul14:28
corvusi'm going to be afk today; will be back tomorrow14:29
Shrewscorvus: enjoy your monday14:31
mnaserShrews: it should only be visible if you actually have alerts for the tenant (so mybe you opened a tenant that doesnt have any?)14:32
Shrewsmnaser: appears for me in the openstack tenant on FF14:32
mnaserah14:32
mnaseri dunno then :\14:32
* mnaser isn't in a "debug-js" morning14:33
Shrewsmnaser: oh, no, you're right.14:33
Shrewswas on a different tenant14:33
Shrewsyup, i see what you do14:33
mnaserok so at least that's a little more comforting14:33
mnaser:p14:33
*** plaurin has joined #zuul14:34
*** sshnaidm has quit IRC14:35
*** jamesmcarthur has joined #zuul14:39
*** jamesmcarthur has quit IRC14:44
*** flaper87 has joined #zuul14:46
*** flaper87 has quit IRC14:49
*** flaper87 has joined #zuul14:49
*** sgw has joined #zuul14:50
*** sshnaidm has joined #zuul14:50
*** flaper87 has quit IRC14:51
*** flaper87 has joined #zuul14:51
*** sshnaidm_ has joined #zuul14:52
*** sshnaidm has quit IRC14:54
*** sshnaidm_ is now known as sshnaidm14:56
tristanCShrews: corvus: ok, then i'll do the zuul zk-auth change today14:56
*** jamesmcarthur has joined #zuul15:18
*** jamesmcarthur has quit IRC15:19
*** jamesmcarthur_ has joined #zuul15:19
*** harrymichal has quit IRC15:27
*** mattw4 has joined #zuul15:34
*** jamesmcarthur_ has quit IRC15:47
*** jamesmcarthur has joined #zuul15:58
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Control log archive and user preservation with vars  https://review.opendev.org/70138116:29
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Make revoke-sudo more general.  https://review.opendev.org/70626216:31
*** avass is now known as Guest2641816:32
*** avass has joined #zuul16:32
*** jamesmcarthur has quit IRC16:35
*** jamesmcarthur has joined #zuul16:37
*** chandankumar is now known as raukadah17:00
*** jamesmcarthur has quit IRC17:00
*** AJaeger has quit IRC17:05
*** AJaeger has joined #zuul17:05
*** jamesmcarthur has joined #zuul17:11
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Tests bindep role on all-platforms  https://review.opendev.org/70870417:15
*** jpena is now known as jpena|off17:33
*** evrardjp has quit IRC17:35
*** evrardjp has joined #zuul17:35
*** Defolos has quit IRC17:38
*** jcapitao is now known as jcapitao_off17:41
*** jamesmcarthur has quit IRC18:00
*** y2kenny has joined #zuul18:06
*** jcapitao_off has quit IRC18:07
*** jamesmcarthur has joined #zuul18:08
*** jamesmcarthur has quit IRC18:16
*** jamesmcarthur has joined #zuul18:17
*** bhavikdbavishi has quit IRC18:25
y2kennyHi, I got a quick question about etc/zuul/main.yaml.  If I understand correctly, any projects mentioned in any zuul config must be added to main.yaml/tenant config first.  Is there any practice of dynamically loading tenant config (perhaps storing it in a git repository?)  I may be having some confusion about various project concept (config project18:40
y2kennyvs untrusted, etc.)18:40
y2kennyFor example, if I want to have zuul build a 3rd party project that is not "zuul native" (i.e., doesn't have .zuul.d along with the source) do I need to create a separate untrusted project to store the zuul config or can I place the config directly into the zuul-config repo and add the 3rd party project to the tenant config?18:42
clarkby2kenny: we run our zuul with a bit of configuration management that writes that list out of a git repo and then triggers a zuul config reload. That is one way of hosting it out of a git repo. Zuul also has a mechanism to replace the explicit config with a command to run that generates the config (I'm looking for docs on that now)18:42
mordredy2kenny: there is a mechanism where you can have zuul run a script which returns the main.yaml contents - I believe tristanC and mnaser both use this18:42
mordredy2kenny: you can just store the zuul jobs for that project in your config-project18:43
mordredthat's one of the things you can do with config projects - assign zuul jobs to other projects in the system18:43
mnasertenant_config_script instead of tenant_config afaik18:43
clarkbhttps://zuul-ci.org/docs/zuul/discussion/components.html#attr-scheduler.tenant_config_script18:43
y2kennyAh ok, thank you.18:43
clarkbmnaser: yup y2kenny ^ docs for that18:43
*** jamesmcarthur has quit IRC18:44
*** jamesmcarthur has joined #zuul18:56
*** y2kenny has quit IRC19:07
*** Defolos has joined #zuul19:11
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Tests bindep role on all-platforms  https://review.opendev.org/70870419:26
*** armstrongs has joined #zuul19:51
*** armstrongs has quit IRC20:00
*** jamesmcarthur has quit IRC20:06
*** jamesmcarthur has joined #zuul20:14
*** jamesmcarthur has quit IRC20:17
*** jamesmcarthur has joined #zuul20:18
*** AJaeger has quit IRC20:21
jamesmcarthurmordred: TristanC: I understand y'all have some thoughts on questions 18 and 19 on the proposed zuul survey20:28
jamesmcarthurWould you mind sending them my way via dm (IRC or email) and I'll get those ?s updated20:28
tristanCjamesmcarthur: you mean from https://www.surveymonkey.com/r/K2B2MWL ?20:30
jamesmcarthuroh sorry, yes tristanC :)20:32
tristanCjamesmcarthur: question 14, it seems odd that 'CI' and 'CD' are in the middle of equivalent options, perhaps these choices should be moved at the begining or the end, or even removed20:35
jamesmcarthurI'm open to whatever option you all think works best.20:35
tristanCnit on Q16 and 17: reverse the version list. it currently goes from 'Master' to version '0.0.1' :)20:37
jamesmcarthurtristanC: jeblair had suggested that "apply deployment" and "cd" are too similar. Perhaps we should get rid of CI and CD, then add "Apply Deployment"20:38
jamesmcarthurtristanC: would putting Master at the end of the list instead fix this problem?20:40
*** sshnaidm is now known as sshnaidm|afk20:40
fungi"ci" and "cd" are jargon, so might be worth avoiding on those grounds anyway when there are more specific terms which can be used in their place20:40
tristanCjamesmcarthur: mordred: it seems like Q18 is already answered by Q16, 17 and 19. Why not only keep Q19 and rename it: 'How do you deploy Zuul?'20:40
jamesmcarthurtristanC: fungi: done on question 16. Removed "CI" and replaced "CD" with "Apply deployment"20:42
fungicool, thanks!20:43
jamesmcarthurtristanC raises a good point on 18 being answered by 16 and 1720:43
jamesmcarthursorry - correction -- done on question 14. Removed "CI" and replaced "CD" with "Apply deployment"20:43
jamesmcarthur(i said 16 previously)20:43
jamesmcarthurI think if we can cut a question we should.  So I'd be in favor of ditching 18.20:44
clarkb think mordred's conscern with 18/19 is they conflate several things all at once20:44
clarkbwhat release train are you on, are you installing from source or packages if so what packages/source, and what deployment tooling do you use20:45
clarkbI kinda think that last question might be most important for zuul to know (as it affects design decisions in the software itself)20:45
jamesmcarthurIf it's appropriate to split them up, we definitely can.20:46
clarkbI've got to pop out now, but maybe mordred can provide more input20:47
tristanCclarkb: well, q16 and q17 already ask what versions is being used. i'm not sure what more information do you get with q18 by asking next if the user use master or releases only.20:50
clarkbtristanC: ya20:50
clarkbbut then q19 askes about packaging and deployment tool20:50
clarkbwhich are distinct but sort of mixed togethe rthere20:51
clarkb(and packaging questions will infer info about versions too)20:51
jamesmcarthurI don't have a backlogged IRC messages atm. If y'all can send me your final thoughts on the zuul thread, I'd be appreciative :)20:53
*** jamesmcarthur has quit IRC20:58
*** jamesmcarthur has joined #zuul20:58
*** zxiiro has quit IRC21:01
openstackgerritTristan Cacqueray proposed zuul/nodepool master: Add mypy to linter test  https://review.opendev.org/71175021:29
*** jamesmcarthur has quit IRC21:47
*** dpawlik has quit IRC22:00
*** plaurin has quit IRC22:18
openstackgerritTristan Cacqueray proposed zuul/zuul master: Implement zookeeper-auth  https://review.opendev.org/61915622:36
openstackgerritTristan Cacqueray proposed zuul/nodepool master: Implement zookeeper-auth  https://review.opendev.org/61915522:38
openstackgerritTristan Cacqueray proposed zuul/zuul master: Implement zookeeper-auth  https://review.opendev.org/61915623:00
*** mattw4 has quit IRC23:00
*** rlandy is now known as rlandy|bbl23:05
*** jamesmcarthur has joined #zuul23:22
*** jamesmcarthur has quit IRC23:23
*** jamesmcarthur has joined #zuul23:24
*** jamesmcarthur has joined #zuul23:25
*** tosky has quit IRC23:30
*** erbarr has quit IRC23:32
*** jamesmcarthur has quit IRC23:40
*** jamesmcarthur has joined #zuul23:40
*** jamesmcarthur has quit IRC23:46
*** Defolos has quit IRC23:52
« Sunday, 2020-03-08 Index Tuesday, 2020-03-10 »

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