| *** hashar has quit IRC | 00:33 | |
| *** jamesmcarthur has joined #zuul | 00:49 | |
| *** swest has quit IRC | 01:05 | |
| *** swest has joined #zuul | 01:20 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add OpenAPI documentation https://review.openstack.org/535541 | 01:24 |
|---|---|---|
| openstackgerrit | James E. Blair proposed openstack-infra/zone-zuul-ci.org master: Add temporary record for cert renewal https://review.openstack.org/647970 | 01:34 |
| openstackgerrit | James E. Blair proposed openstack-infra/zone-zuul-ci.org master: Revert "Add temporary record for cert renewal" https://review.openstack.org/647971 | 01:34 |
| openstackgerrit | Merged openstack-infra/zone-zuul-ci.org master: Add temporary record for cert renewal https://review.openstack.org/647970 | 01:37 |
| openstackgerrit | Merged openstack-infra/zone-zuul-ci.org master: Revert "Add temporary record for cert renewal" https://review.openstack.org/647971 | 01:46 |
| *** ianychoi has quit IRC | 02:03 | |
| *** ianychoi has joined #zuul | 02:03 | |
| *** SotK has quit IRC | 02:15 | |
| *** mgagne has quit IRC | 02:15 | |
| *** mgagne has joined #zuul | 02:15 | |
| *** SotK has joined #zuul | 02:16 | |
| *** jamesmcarthur has quit IRC | 02:52 | |
| *** jamesmcarthur has joined #zuul | 03:36 | |
| *** jamesmcarthur has quit IRC | 03:44 | |
| *** jamesmcarthur has joined #zuul | 03:45 | |
| *** jamesmcarthur has quit IRC | 03:47 | |
| *** jamesmcarthur has joined #zuul | 03:47 | |
| *** jamesmcarthur has quit IRC | 03:52 | |
| *** jamesmcarthur has joined #zuul | 04:00 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add OpenAPI documentation https://review.openstack.org/535541 | 04:01 |
| *** raukadah is now known as chandankumar | 04:02 | |
| *** tflink has quit IRC | 04:43 | |
| *** tflink has joined #zuul | 04:44 | |
| *** tflink has quit IRC | 04:44 | |
| *** tflink has joined #zuul | 04:46 | |
| *** saneax has joined #zuul | 04:49 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add OpenAPI documentation https://review.openstack.org/535541 | 05:01 |
| *** quiquell|off is now known as quiquell | 06:31 | |
| *** lennyb has joined #zuul | 07:04 | |
| *** quiquell is now known as quiquell|brb | 07:33 | |
| *** gtema has joined #zuul | 07:48 | |
| *** quiquell|brb is now known as quiquell | 08:08 | |
| openstackgerrit | William Huang proposed openstack-infra/zuul-jobs master: Change the npm registry if the mirror url is defined https://review.openstack.org/647305 | 08:09 |
| openstackgerrit | William Huang proposed openstack-infra/zuul-jobs master: Change the npm registry if the mirror url is defined https://review.openstack.org/647305 | 08:11 |
| *** themroc has joined #zuul | 08:37 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: zuul-runner: add command line interface https://review.openstack.org/644770 | 08:50 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: runner: add job parameters listing https://review.openstack.org/644795 | 08:50 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Add depends-on support to frozen jobs API https://review.openstack.org/639022 | 08:50 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: zuul-runner: add support for depends-on https://review.openstack.org/632064 | 08:50 |
| *** jpena|off is now known as jpena | 08:56 | |
| *** pcaruana has joined #zuul | 08:57 | |
| badboy | there's a certificate mismatch on https://zuul-ci.org/ | 09:31 |
| AJaeger | badboy: thanks for info, already noted on other channel. The certificate was updated yesterday and has wrong hostname. | 09:37 |
| badboy | AJaeger: np | 09:37 |
| *** gtema has quit IRC | 10:27 | |
| *** gtema has joined #zuul | 10:27 | |
| *** electrofelix has joined #zuul | 10:41 | |
| *** hughsaunders has quit IRC | 11:03 | |
| *** zbr|ssbarnea has quit IRC | 11:06 | |
| *** zbr has joined #zuul | 11:07 | |
| *** sdatko has joined #zuul | 11:13 | |
| sdatko | hello, just spotted an expired ssl certificate on https://zuul-ci.org/docs/zuul/ | 11:13 |
| sdatko | ach, sorry, not expired; just common name mismatch | 11:14 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: zuul-runner: add support for depends-on https://review.openstack.org/632064 | 11:25 |
| AJaeger | sdatko: thanks for telling - hope an admin can fix this later. | 11:33 |
| *** quiquell is now known as quiquell|lunch | 12:01 | |
| *** rlandy has joined #zuul | 12:13 | |
| *** jamesmcarthur has joined #zuul | 12:23 | |
| *** jamesmcarthur has quit IRC | 12:32 | |
| *** pcaruana has quit IRC | 12:39 | |
| *** hashar has joined #zuul | 12:40 | |
| *** pcaruana has joined #zuul | 12:42 | |
| *** pcaruana has quit IRC | 12:42 | |
| *** pcaruana has joined #zuul | 12:43 | |
| *** jpena is now known as jpena|lunch | 12:46 | |
| *** jamesmcarthur has joined #zuul | 12:51 | |
| *** jamesmcarthur has quit IRC | 12:52 | |
| *** jamesmcarthur has joined #zuul | 12:52 | |
| *** quiquell|lunch is now known as quiquell | 13:02 | |
| *** gtema has quit IRC | 13:09 | |
| *** jhesketh has quit IRC | 13:10 | |
| fungi | badboy: sdatko: AJaeger: it should be fixed now. not expired, just was updated to serve the wrong cert | 13:28 |
| fungi | but it has the correct one back in place again | 13:28 |
| *** jamesmcarthur has quit IRC | 13:28 | |
| fungi | thanks for letting us know! | 13:29 |
| badboy | fungi: :) | 13:29 |
| *** jpena|lunch is now known as jpena | 13:32 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add OpenAPI documentation https://review.openstack.org/535541 | 13:37 |
| *** jamesmcarthur has joined #zuul | 13:44 | |
| *** gtema has joined #zuul | 13:52 | |
| *** jamesmcarthur_ has joined #zuul | 14:02 | |
| *** jamesmcarthur has quit IRC | 14:05 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: A reporter for Elasticsearch with the capability to index build and buildset results in an index. https://review.openstack.org/644927 | 14:10 |
| *** sshnaidm_ has joined #zuul | 14:19 | |
| *** sshnaidm_ is now known as sshnaidm|bbl | 14:19 | |
| *** sshnaidm|rover has quit IRC | 14:21 | |
| *** sshnaidm|bbl has quit IRC | 14:22 | |
| *** gtema has quit IRC | 14:35 | |
| *** sshnaidm|bbl has joined #zuul | 14:48 | |
| *** sshnaidm|bbl has quit IRC | 15:00 | |
| *** smyers has quit IRC | 15:07 | |
| *** smyers has joined #zuul | 15:08 | |
| *** sshnaidm|bbl has joined #zuul | 15:15 | |
| *** paladox has quit IRC | 15:26 | |
| *** paladox has joined #zuul | 15:28 | |
| *** swest has quit IRC | 15:28 | |
| *** quiquell is now known as quiquell|off | 15:38 | |
| *** saneax has quit IRC | 15:43 | |
| *** saneax has joined #zuul | 15:44 | |
| openstackgerrit | Merged openstack-infra/zuul master: Ensure valid Ansible variable names in config https://review.openstack.org/644574 | 15:45 |
| *** sdatko has quit IRC | 15:47 | |
| *** pwhalen has quit IRC | 15:47 | |
| *** pwhalen has joined #zuul | 15:49 | |
| *** sshnaidm|bbl has quit IRC | 15:50 | |
| *** hashar has quit IRC | 15:53 | |
| *** saneax has quit IRC | 16:06 | |
| *** chandankumar is now known as raukadah | 16:28 | |
| *** jamesmcarthur_ has quit IRC | 16:42 | |
| *** themroc has quit IRC | 16:42 | |
| *** sshnaidm|bbl has joined #zuul | 16:46 | |
| *** bhavikdbavishi has joined #zuul | 16:49 | |
| pabelanger | corvus: tobiash: yesterday, I asked the following: http://eavesdrop.openstack.org/irclogs/%23zuul/%23zuul.2019-03-26.log.html#t2019-03-26T13:28:04 wanted to see if you had anything thoughts | 16:52 |
| corvus | pabelanger: i could imagine supporting a branch specifier in the tenant config | 17:05 |
| tobiash | pabelanger, corvus: yes, a branch specifier should work. It should also simplify downstream consuming of zuul-jobs. | 17:48 |
| corvus | yes, it could be a sha or tag too | 17:49 |
| *** jpena is now known as jpena|off | 18:03 | |
| *** jamesmcarthur has joined #zuul | 18:05 | |
| corvus | i've performed some local testing and i believe that zuul will follow the pending opendev redirects | 18:15 |
| corvus | fungi has set up our static fileserver, 23.253.125.17, with a vhost for git.openstack.org with the opendev redirects in place (so if you add that ip to your /etc/hosts entry as 'git.openstack.org' you can try them out) | 18:17 |
| corvus | i did that, and set up a zuul configured to use zuul-jobs as 'git.openstack.org/openstack-infra/zuul-jobs' | 18:17 |
| corvus | there were no errors, and tcpdump suggests that it's talking to 23.253.125.17 (the server handling the redirects) and 38.108.68.124 (the opendev load balancer) | 18:18 |
| corvus | so i believe that zuul operators won't need to take any immediate action when we redirect git.zuul-ci.org to opendev.org, and once that's in place, we can simply suggest that for clarity folks update their configs to use 'opendev.org/zuul/zuul-jobs' | 18:19 |
| corvus | (i think this all "just works" since we're outsourcing all the remote operations to the git binary) | 18:20 |
| fungi | that's awesome news! | 18:20 |
| fungi | also glad the adapted copy of your htaccess rules seems to be working after adjusting the apache allowances | 18:21 |
| fungi | i'm working on the git.zuul-ci.org redirects still | 18:21 |
| *** electrofelix has quit IRC | 18:27 | |
| *** bhavikdbavishi has quit IRC | 18:40 | |
| *** themroc has joined #zuul | 18:52 | |
| *** jamesmcarthur has quit IRC | 18:53 | |
| *** dkehn has joined #zuul | 19:00 | |
| *** jamesmcarthur has joined #zuul | 19:16 | |
| pabelanger | corvus: tobiash: thanks! I'll look into how I can write a patch | 19:38 |
| *** pcaruana has quit IRC | 19:53 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Update cached repo during job startup only if needed https://review.openstack.org/648229 | 20:08 |
| tobiash | corvus: what do you think about ^? | 20:08 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Add a job that runs the upload-git-mirror role https://review.openstack.org/648230 | 20:10 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Update cached repo during job startup only if needed https://review.openstack.org/648229 | 20:10 |
| *** themroc has quit IRC | 21:15 | |
| corvus | tobiash: i'll have to think about that a bit; i'll put in on my list | 21:31 |
| tobiash | thanks | 21:31 |
| corvus | tristanC: have you finished revising the zuul-runner patches? i notice that some of my comments haven't been addressed, and i was hoping for a lot more documentation (on the developer side i still don't understand the refactor, and the user side needs narrative documentation explaining what zuul-runner is and how to use it) | 21:34 |
| corvus | tristanC: if you're still working on it, i'll leave you to it, but otherwise, if i can help clarify what i think we need, let me know. | 21:35 |
| *** rlandy is now known as rlandy|bbl | 22:08 | |
| *** jamesmcarthur has quit IRC | 22:33 | |
| *** jamesmcarthur has joined #zuul | 22:33 | |
| *** jamesmcarthur has quit IRC | 22:44 | |
| *** jamesmcarthur has joined #zuul | 22:48 | |
| *** jamesmcarthur has quit IRC | 22:51 | |
| *** jamesmcarthur has joined #zuul | 22:52 | |
| *** jamesmcarthur has quit IRC | 22:55 | |
| tristanC | corvus: please help clarify what is missing | 23:15 |
| tristanC | perhaps the refactor could use better names, for example s/AnsibleJob/ThreadedAnsibleJob/ and s/AnsibleBaseJob/AnsibleJob/ | 23:17 |
| corvus | tristanC: there's 2 main things i don't understand. | 23:31 |
| corvus | tristanC: first, as a developer/maintainer, i don't understand the refactor. at the end of the stack, here's the documentation about AnsibleJobBase: http://logs.openstack.org/64/632064/12/check/tox-docs/3cdbe6c/html/developer/jobmodel.html | 23:31 |
| corvus | tristanC: somehow that's there to enable the runner command, but i don't understand how. | 23:33 |
| corvus | tristanC: i need more information about what the different responsibilities of the two classes are | 23:34 |
| corvus | tristanC: but the biggest thing is user documentation: http://logs.openstack.org/64/632064/12/check/tox-docs/3cdbe6c/html/user/runner.html | 23:36 |
| corvus | tristanC: we need a lot more than just the command line reference. how does a person use it? this is for end-users of zuul, which, up to this point, have never even needed to install zuul | 23:37 |
| corvus | so we need to take people through the process starting from installation through configuration and setup to finally running the command | 23:37 |
| corvus | there should be a lot of text explaining what will happen, how to run it securely, the risks of running it, etc. | 23:38 |
| corvus | tristanC: i suggested that you could do this as followup documentation in order to try to get it merged with thi minimum of extra work, but i'm starting to think we should consider writing a spec | 23:39 |
| tristanC | corvus: please note that only https://review.openstack.org/607078 is needed on server side | 23:40 |
| corvus | tristanC: i understand | 23:40 |
| tristanC | corvus: i'll rename the class and try to make it for clear for a developer point of view, though the common.AnsibleJob may needs more work to better isolate the responsability | 23:41 |
| corvus | tristanC: yeah, and once the purpose of the different classes is clearer, it may be easier to see what should be in which class | 23:42 |
| tristanC | corvus: though I'm not entirely sure what was joshua's intent with the different classes. | 23:43 |
| tristanC | I guess the idea was to put in a common module what can be re-used by the runner, so that the runner doesn't have to import executor.server | 23:44 |
| corvus | tristanC: yeah, i'm starting to get the idea that there are some things that the executor will need to do that the runner does not (and perhaps vice versa). so the idea makes sense to me in general. | 23:45 |
| tristanC | corvus: i can write more user documentation, but i was hoping for some feedback first. I mean the code work as it is now, but i don't mind changing the interface | 23:45 |
| corvus | tristanC: the biggest thing i'm worried about though is the user docs. zuul has, up to this point, been a server-side application. asking end-users to install and configure (even part of) it is a big step. i think there are a lot of things we need to discuss (such as whether it's okay to have a config file, what the 'nodes' argument means, etc). | 23:46 |
| corvus | tristanC: ok, so let's figure out the best way to do that | 23:47 |
| tristanC | corvus: i think it would help if the server side patch was merged so that anyone can test the cli and provide feedback | 23:47 |
| corvus | tristanC: i don't think we should merge the server side without the client | 23:47 |
| corvus | we don't have a spec for this, so it's being designed in a series of patches, and we don't know where they will end up. there is a risk that the system could change significantly, or perhaps not merge at all (i hope that's not the case and will work to avoid that) | 23:49 |
| corvus | tristanC: i can think of 2 ways to work on the user story side: 1) expand the user documentation you've already started with a draft of how to use the software. it doesn't have to be perfect or have all the details, but if there's enough for us to see the workflow and discuss things like what the config file looks like, or if there should even be a config file, then that will help. | 23:51 |
| corvus | then when we achive consensus on that, the docs can be expanded with more detail | 23:52 |
| corvus | tristanC: 2) or write a spec. it doesn't have to be too long, but it would similarly give us a place to discuss the workflow and talk about what the system should look like at the end when it's fully implemented. | 23:52 |
| corvus | tristanC: #2 is the safe bet. it's the best way to discuss a large change like this. maybe #1 will be sufficient, but if it isn't we might end up doing #2 anyway. | 23:53 |
| tristanC | since we'll have to do #1 anyway, i'll expand the user documentation of what we have today | 23:54 |
| corvus | tristanC: sounds good, thanks | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!