openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement a static driver for Nodepool https://review.openstack.org/468624 | 07:22 |
---|---|---|
openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Refactor provider config to driver module https://review.openstack.org/488384 | 07:22 |
*** amoralej|off is now known as amoralej | 07:32 | |
*** dmellado has quit IRC | 08:06 | |
*** amoralej has quit IRC | 08:06 | |
*** amoralej has joined #zuul | 08:46 | |
*** openstackgerrit has quit IRC | 08:48 | |
*** electrofelix has joined #zuul | 09:11 | |
*** jkilpatr has quit IRC | 10:49 | |
*** jkilpatr has joined #zuul | 10:49 | |
*** isaacb has joined #zuul | 10:52 | |
*** xinliang has quit IRC | 12:14 | |
*** isaacb has quit IRC | 12:28 | |
*** isaacb has joined #zuul | 12:31 | |
*** isaacb has quit IRC | 12:36 | |
*** isaacb has joined #zuul | 12:41 | |
*** dkranz_ has quit IRC | 12:48 | |
*** dkranz has joined #zuul | 12:48 | |
*** dkranz has quit IRC | 12:48 | |
*** dkranz has joined #zuul | 12:49 | |
electrofelix | Has there been any thoughts on displaying single changes with zuulv3? There is one suggestion at https://storyboard.openstack.org/#!/story/2000873 that I wouldn't mind doing, but will that fit with the plans for v3? | 13:09 |
electrofelix | Reading through the code it appears someone has added such support for zuulv3 | 13:32 |
*** isaacb has quit IRC | 13:59 | |
*** isaacb has joined #zuul | 14:11 | |
jeblair | electrofelix: yes, i think the github driver makes us of that now | 14:21 |
*** isaacb has quit IRC | 15:10 | |
jeblair | mordred, Shrews, pabelanger, jlk: will you all be able to attend the meeting today? i'd like to sync up on what we need to do before the ptg. | 15:47 |
pabelanger | yes, I will be attending | 15:47 |
Shrews | jeblair: yep | 15:47 |
*** smyers has quit IRC | 15:55 | |
*** smyers has joined #zuul | 15:56 | |
*** smyers has quit IRC | 16:10 | |
*** smyers has joined #zuul | 16:16 | |
mordred | jeblair: yes. I will be there | 16:19 |
jeblair | anyone else interested in reviewing zuul-sphinx changes? https://review.openstack.org/493248 could use a +3. it's a new repo, and i don't really know who's interested in it yet :) | 16:25 |
jeblair | it's to support this change: https://review.openstack.org/493250 | 16:26 |
*** electrofelix has quit IRC | 16:36 | |
Shrews | jeblair: done | 17:03 |
*** openstackgerrit has joined #zuul | 17:03 | |
openstackgerrit | Merged openstack-infra/zuul-sphinx master: Add jobvar and rolevar directives https://review.openstack.org/493248 | 17:03 |
Shrews | in the spirit of asking for reviews, a simplish nodepool-launcher change: https://review.openstack.org/492219 | 17:04 |
jeblair | i like that one | 17:07 |
Shrews | jeblair: can do the same for the builder once we move to py3 only | 17:07 |
Shrews | (builder needs a barrier for thread sync) | 17:07 |
jeblair | i tagged zuul-sphinx 0.1.5 | 17:11 |
* SpamapS sits back down at IRC for the first time in a bit and shakes off the cobwebs | 17:21 | |
jeblair | SpamapS: welcome back! i'm hoping to re-sync on pre-ptg tasks at meeting today if you can attend | 17:31 |
SpamapS | jeblair: yes I will definitely be in attendance. | 17:34 |
SpamapS | for the meeting | 17:34 |
SpamapS | PTG is still up in the air | 17:34 |
SpamapS | jeblair: would you say Zuulv3 has this already? I think it does: https://storyboard.openstack.org/#!/story/2000873 | 17:37 |
SpamapS | we can probably mark that as merged and tag it zuulv3 | 17:37 |
jeblair | SpamapS, fungi: grr; i'm getting ocsp errors and can't seem to access storyboard (or review or several other servers) via firefox atm. that's been happening for about an hour | 17:59 |
jeblair | seems to work in chromium | 18:00 |
fungi | strange | 18:01 |
fungi | wasn't an issue for me in ff at least as of a few minutes ago | 18:01 |
jeblair | SpamapS: yes. i think we may have a teeny bit of work needod to expose that so that it can be left in comments (right now, i think only github driver uses it and does so via its special status url thingy) | 18:01 |
jeblair | SpamapS: but it's mostly there. | 18:02 |
fungi | jeblair: oh! i wonder if the fact that i replaced the certs instead of renewing them is the cause | 18:02 |
jeblair | fungi: was that recent? | 18:03 |
fungi | i think the old certs for review and storyboard were due to expire nowish | 18:03 |
jeblair | oh hrm | 18:03 |
fungi | i replaced them a month or so ago though | 18:03 |
fungi | maybe your browser has hung onto and cached the originals? | 18:03 |
jeblair | i can't really convince firefox to give me any further useful info | 18:03 |
jeblair | fungi: let's move to infra | 18:03 |
*** amoralej is now known as amoralej|off | 18:43 | |
SpamapS | https://storyboard.openstack.org/#!/board/41 <-- looking entirely managable maybe for the first time ever :) | 18:52 |
jeblair | omg omg omg omg | 18:52 |
SpamapS | gmo gmo gmo gmo (sorry, I have a palindrome reflex when I'm confused) | 18:55 |
SpamapS | omg gmo is my new favorite palindrome btw | 18:55 |
jeblair | wow i can see that in a mayonnaise advertisement | 18:56 |
SpamapS | OMG no on GMO | 18:57 |
openstackgerrit | Merged openstack-infra/nodepool feature/zuulv3: Allow launcher to stop quicker when asked https://review.openstack.org/492219 | 19:23 |
*** smyers has quit IRC | 19:42 | |
*** smyers has joined #zuul | 19:43 | |
* SpamapS decides to re-insert self into zuul matrix by enabling test_delayed_repo_init | 19:49 | |
Shrews | jeblair: would it make sense to move the majority of the "pipeline" portion of the user guide to the admin guide? A user isn't going to be defining pipelines in their project. | 19:59 |
SpamapS | There should definitely be a clear delineation between "consuming zuul" and "operating zuul" | 19:59 |
Shrews | A user should definitely know what a pipeline is, but I'm not sure if they need the specifics on defining one | 20:01 |
jeblair | Shrews: that's something i've wrestled with a bit, but here's why it's landed on that side of the line for me: 1) the line i've been trying to draw is the one where "admin" requires root access on the server. (ignore that you can change our tenant config without that access -- that's just the way we do it, it's not built into zuul) | 20:01 |
jeblair | with that admin/user divide, it is the case that a user can define a pipeline. i get the contrary argument -- *most* users don't define pipelines. | 20:02 |
SpamapS | I think as long as it's clear "Pipelines can only be defined in [whatever we call trusted configs]" | 20:02 |
jeblair | but yes, they definitely need to know what they are and be able to see how they work. but we also have had a lot of folks i wouldn't consider zuul admins define or alter pipelines in openstack. | 20:03 |
Shrews | ok. i had my mental line of delineation drawn elsewhere, but moving it to align with yours makes it more logical now | 20:04 |
jeblair | Shrews: there's a really tricky part of this that i haven't figured out yet, which is that currently, the driver-specific trigger and reporter sections of pipeline config are documented in the admin guide (because that's where drivers are documented), even though they fall on the user side of the divide. that's solely due to not having spent enough time to figure out how to reorganize docs to accommodate that. for now, there's just a lot of ... | 20:07 |
jeblair | ... cross-referencing between them. | 20:07 |
Shrews | hrm yeah. that does muddy the waters a bit | 20:08 |
jeblair | (but also, i wouldn't want to make the problem you percieved *worse* by 'fixing' that -- it's clearly 'power user' material) | 20:08 |
jeblair | so for now, i've just accepted that's a low-priority bug. | 20:08 |
SpamapS | Lots of things have admin-only driver-specific bits that usually get some kind of "dangerous" or "expert" tag. | 20:11 |
jeblair | yeah -- indicating that folks can skip ahead for some of this might be useful. :) | 20:11 |
SpamapS | "So You Think You Can GitHub" | 20:16 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable test_delayed_repo_init https://review.openstack.org/493659 | 20:21 |
SpamapS | I thought I had gotten pretty good at these test re-enables | 20:21 |
SpamapS | but the one above is failing because it can't find the project that I'm adding to the zuul.yaml by saying "You must add all projects to zuul.yaml" .. so.. | 20:22 |
* SpamapS starts debugging deeply | 20:22 | |
* SpamapS may have a bug in that error reporting code | 20:22 | |
jeblair | SpamapS: oh does it say 'zuul.yaml' in that error? that sounds like it should say the tenant config file. | 20:22 |
SpamapS | The project "org/new-project" was not found. All projects referenced | 20:23 |
SpamapS | within a Zuul configuration must first be added to the main | 20:23 |
SpamapS | configuration file by the Zuul administrator. | 20:23 |
SpamapS | I thought I was updating the "main configuration file" | 20:23 |
SpamapS | with commitConfigUpdate | 20:23 |
jeblair | ah yeah, that means the tenant config file (aka main.yaml by convention but it's configurable). maybe we should have that say 'tenant config file' instead. | 20:24 |
jeblair | i think the error message may predate the docs settling on phrasing. :) | 20:24 |
jeblair | also, i'd love to have errors link to docs, but all the doc urls are going to change... :/ | 20:24 |
SpamapS | Yeah that would make more sense. | 20:25 |
jeblair | maybe once our webserving is more robust, zuul should serve it's own docs (we talked about this for jobs, but now i'm thinking maybe more than just jobs -- *all* the docs) then it can reference its own doc webserver. | 20:25 |
jeblair | then our errors could just call a function which generates a link to a sphinx target in the self-hosted docs | 20:26 |
SpamapS | I like the way Rust has organized errors in the compiler. They just spit out error codes and you can plug that into any search engine and it finds the right one. | 20:27 |
jeblair | i updated the meeting agenda and started an etherpad: https://wiki.openstack.org/wiki/Meetings/Zuul | 20:27 |
SpamapS | so E0239 has a specific known description and a list of version changes. | 20:27 |
jeblair | SpamapS: that's nice too, though we know we're running a webserver, so we can do better i think :) | 20:27 |
jeblair | SpamapS: that's right. i said we can do *better than rust*. ;) | 20:28 |
jeblair | pabelanger: have we restarted with all the secrets changes? | 20:31 |
SpamapS | jeblair: you cut me deep | 20:33 |
SpamapS | ;) | 20:33 |
pabelanger | jeblair: I don't think so | 20:34 |
jeblair | pabelanger: hows about i do that now | 20:35 |
pabelanger | jeblair: ++ | 20:35 |
jeblair | restarting now | 20:39 |
SpamapS | hmmmmmmm | 20:45 |
SpamapS | looks like we don't really have facilities to update the tenant config in tests just yet | 20:46 |
SpamapS | or I haven't found them | 20:46 |
jeblair | SpamapS: we may not i think mostly we add a new config scenario (new directory in tests/fixtures/config/) | 20:47 |
jeblair | SpamapS: however, it should reload the tenant config on a reconfigure event | 20:48 |
SpamapS | Right I think this test is intended to test the path of "adding a new repo" | 20:48 |
SpamapS | so worth it to go ahead and make that happen | 20:49 |
jeblair | SpamapS: yeah, so i think to keep the spirit of the test, that's probably what we'll have to do | 20:49 |
SpamapS | Will probably just cargo commitConfigUpdate sort of | 20:49 |
jeblair | zuulv3 is inop until this config syntax change lands: https://review.openstack.org/493667 | 20:56 |
jeblair | zuulv2 estimates it in 6 minutes | 20:56 |
mordred | cool | 20:56 |
*** dmsimard|off has quit IRC | 21:03 | |
SpamapS | argh.. we just load main.yaml directly out of the source repo right now | 21:06 |
* SpamapS has a plan though | 21:07 | |
jeblair | restarting zuulv3 agin | 21:08 |
jeblair | okay it's running now | 21:09 |
pabelanger | cool | 21:24 |
*** dmsimard|off has joined #zuul | 21:30 | |
*** smyers has quit IRC | 21:57 | |
*** smyers has joined #zuul | 21:59 | |
jeblair | zuul meeting time in #openstack-meeting-alt | 22:00 |
Shrews | So, of all the things that need to be done before PTG, I'm not comfortable driving myself. But if someone else needs a co-pilot on things, ping me if you think I can chip in on something. | 22:59 |
mordred | jeblair: how come https://review.openstack.org/#/c/492715 isn't in the gate pipeline or merged? I feel like I'm nmissing something obvious | 23:03 |
mordred | Shrews: ++ ... I betcha the d-g+ansible stuff will be somewhat approachable - our first test case is shade's jobs, so you've at least got a good background in those | 23:05 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Re-enable test_delayed_repo_init https://review.openstack.org/493659 | 23:13 |
* SpamapS EOD's with an almost-working test re-enablement | 23:13 | |
*** amoralej|off has quit IRC | 23:51 | |
*** robled has quit IRC | 23:51 | |
*** Shrews has quit IRC | 23:51 | |
*** jamielennox has quit IRC | 23:51 | |
*** mmedvede has quit IRC | 23:51 | |
*** olaph has quit IRC | 23:51 | |
*** EmilienM has quit IRC | 23:51 | |
*** zigo has quit IRC | 23:51 | |
*** dmsimard|off has quit IRC | 23:51 | |
*** jkilpatr has quit IRC | 23:51 | |
*** openstackgerrit has quit IRC | 23:51 | |
*** yolanda has quit IRC | 23:51 | |
*** Diabelko has quit IRC | 23:51 | |
*** tristanC has quit IRC | 23:51 | |
*** toabctl has quit IRC | 23:51 | |
*** jeblair has quit IRC | 23:51 | |
*** eventingmonkey has quit IRC | 23:51 | |
*** jlk has quit IRC | 23:51 | |
*** SotK has quit IRC | 23:51 | |
*** rbergeron has quit IRC | 23:51 | |
*** fbouliane has quit IRC | 23:51 | |
*** adam_g has quit IRC | 23:51 | |
*** jtanner has quit IRC | 23:51 | |
*** mattclay has quit IRC | 23:51 | |
*** maxamillion has quit IRC | 23:51 | |
*** ianw has quit IRC | 23:51 | |
*** mnaser has quit IRC | 23:51 | |
*** pbrobinson has quit IRC | 23:51 | |
*** _ari_ has quit IRC | 23:51 | |
*** TheJulia has quit IRC | 23:51 | |
*** robcresswell has quit IRC | 23:51 | |
*** patrickeast has quit IRC | 23:51 | |
*** zaro has quit IRC | 23:51 | |
*** mgagne has quit IRC | 23:51 | |
*** fungi has quit IRC | 23:51 | |
*** leifmadsen has quit IRC | 23:51 | |
*** kklimonda has quit IRC | 23:51 | |
*** tflink has quit IRC | 23:51 | |
*** bstinson has quit IRC | 23:51 | |
*** clarkb has quit IRC | 23:51 | |
*** cinerama` has quit IRC | 23:51 | |
*** pabelanger has quit IRC | 23:51 | |
*** jesusaur has quit IRC | 23:51 | |
*** rcarrillocruz has quit IRC | 23:51 | |
*** ajafo has quit IRC | 23:51 | |
*** SpamapS has quit IRC | 23:51 | |
*** ChanServ has quit IRC | 23:51 | |
*** rfolco has quit IRC | 23:51 | |
*** gothicmindfood has quit IRC | 23:51 | |
*** smyers has quit IRC | 23:51 | |
*** dkranz has quit IRC | 23:51 | |
*** lennyb has quit IRC | 23:51 | |
*** mordred has quit IRC | 23:51 | |
*** kmalloc has quit IRC | 23:51 | |
*** kmalloc has joined #zuul | 23:55 | |
*** mordred has joined #zuul | 23:55 | |
*** jkilpatr has joined #zuul | 23:55 | |
*** lennyb has joined #zuul | 23:55 | |
*** dkranz has joined #zuul | 23:55 | |
*** amoralej|off has joined #zuul | 23:55 | |
*** tflink has joined #zuul | 23:55 | |
*** TheJulia has joined #zuul | 23:55 | |
*** maxamillion has joined #zuul | 23:55 | |
*** jtanner has joined #zuul | 23:55 | |
*** Diabelko has joined #zuul | 23:55 | |
*** ianw has joined #zuul | 23:55 | |
*** rbergeron has joined #zuul | 23:55 | |
*** fbouliane has joined #zuul | 23:55 | |
*** mnaser has joined #zuul | 23:55 | |
*** olaph has joined #zuul | 23:55 | |
*** robled has joined #zuul | 23:55 | |
*** bstinson has joined #zuul | 23:55 | |
*** adam_g has joined #zuul | 23:55 | |
*** robcresswell has joined #zuul | 23:55 | |
*** pbrobinson has joined #zuul | 23:55 | |
*** clarkb has joined #zuul | 23:55 | |
*** kklimonda has joined #zuul | 23:55 | |
*** jamielennox has joined #zuul | 23:55 | |
*** tristanC has joined #zuul | 23:55 | |
*** Shrews has joined #zuul | 23:55 | |
*** toabctl has joined #zuul | 23:55 | |
*** mattclay has joined #zuul | 23:55 | |
*** patrickeast has joined #zuul | 23:55 | |
*** SotK has joined #zuul | 23:55 | |
*** zaro has joined #zuul | 23:55 | |
*** jeblair has joined #zuul | 23:55 | |
*** rfolco has joined #zuul | 23:55 | |
*** eventingmonkey has joined #zuul | 23:55 | |
*** jlk has joined #zuul | 23:55 | |
*** mmedvede has joined #zuul | 23:55 | |
*** mgagne has joined #zuul | 23:55 | |
*** EmilienM has joined #zuul | 23:55 | |
*** _ari_ has joined #zuul | 23:55 | |
*** gothicmindfood has joined #zuul | 23:55 | |
*** zigo has joined #zuul | 23:55 | |
*** fungi has joined #zuul | 23:55 | |
*** leifmadsen has joined #zuul | 23:55 | |
*** ChanServ has joined #zuul | 23:56 | |
*** tepper.freenode.net sets mode: +o ChanServ | 23:56 | |
*** dmsimard|off has joined #zuul | 23:56 | |
*** openstackgerrit has joined #zuul | 23:56 | |
*** yolanda has joined #zuul | 23:56 | |
*** cinerama` has joined #zuul | 23:56 | |
*** pabelanger has joined #zuul | 23:56 | |
*** jesusaur has joined #zuul | 23:56 | |
*** rcarrillocruz has joined #zuul | 23:56 | |
*** ajafo has joined #zuul | 23:56 | |
*** SpamapS has joined #zuul | 23:56 | |
*** ChanServ has quit IRC | 23:56 | |
*** maxamillion has quit IRC | 23:57 | |
*** ianw has quit IRC | 23:57 | |
*** mnaser has quit IRC | 23:57 | |
*** pbrobinson has quit IRC | 23:57 | |
*** _ari_ has quit IRC | 23:57 | |
*** cinerama` has quit IRC | 23:57 | |
*** pabelanger has quit IRC | 23:57 | |
*** jesusaur has quit IRC | 23:57 | |
*** rcarrillocruz has quit IRC | 23:57 | |
*** ajafo has quit IRC | 23:57 | |
*** SpamapS has quit IRC | 23:57 | |
*** dmsimard|off has quit IRC | 23:57 | |
*** openstackgerrit has quit IRC | 23:57 | |
*** yolanda has quit IRC | 23:57 | |
*** kmalloc has quit IRC | 23:57 | |
*** jkilpatr has quit IRC | 23:57 | |
*** amoralej|off has quit IRC | 23:57 | |
*** robled has quit IRC | 23:57 | |
*** Shrews has quit IRC | 23:57 | |
*** jamielennox has quit IRC | 23:57 | |
*** mmedvede has quit IRC | 23:57 | |
*** olaph has quit IRC | 23:57 | |
*** EmilienM has quit IRC | 23:57 | |
*** zigo has quit IRC | 23:57 | |
*** Diabelko has quit IRC | 23:57 | |
*** tristanC has quit IRC | 23:57 | |
*** toabctl has quit IRC | 23:57 | |
*** jeblair has quit IRC | 23:57 | |
*** eventingmonkey has quit IRC | 23:57 | |
*** jlk has quit IRC | 23:57 | |
*** SotK has quit IRC | 23:57 | |
*** mordred has quit IRC | 23:57 | |
*** dkranz has quit IRC | 23:57 | |
*** lennyb has quit IRC | 23:57 | |
*** rbergeron has quit IRC | 23:57 | |
*** fbouliane has quit IRC | 23:57 | |
*** adam_g has quit IRC | 23:57 | |
*** jtanner has quit IRC | 23:57 | |
*** mattclay has quit IRC | 23:57 | |
*** TheJulia has quit IRC | 23:57 | |
*** robcresswell has quit IRC | 23:57 | |
*** patrickeast has quit IRC | 23:57 | |
*** zaro has quit IRC | 23:57 | |
*** mgagne has quit IRC | 23:57 | |
*** fungi has quit IRC | 23:57 | |
*** leifmadsen has quit IRC | 23:57 | |
*** kklimonda has quit IRC | 23:57 | |
*** tflink has quit IRC | 23:57 | |
*** bstinson has quit IRC | 23:57 | |
*** clarkb has quit IRC | 23:57 | |
*** rfolco has quit IRC | 23:57 | |
*** gothicmindfood has quit IRC | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!