| *** bhavikdbavishi has joined #zuul | 01:47 | |
| *** bhavikdbavishi has quit IRC | 02:37 | |
| *** bhavikdbavishi has joined #zuul | 03:55 | |
| *** bhavikdbavishi has quit IRC | 04:02 | |
| *** bhavikdbavishi has joined #zuul | 04:02 | |
| *** bjackman has joined #zuul | 05:48 | |
| *** bhavikdbavishi has quit IRC | 06:05 | |
| *** bjackman has quit IRC | 06:10 | |
| *** bjackman has joined #zuul | 06:26 | |
| *** AJaeger has quit IRC | 07:11 | |
| *** AJaeger has joined #zuul | 07:20 | |
| *** bhavikdbavishi has joined #zuul | 07:24 | |
| *** pcaruana has joined #zuul | 08:14 | |
| *** pcaruana has quit IRC | 08:24 | |
| *** rcarrillocruz has joined #zuul | 08:45 | |
| *** pcaruana has joined #zuul | 08:46 | |
| *** hashar has joined #zuul | 09:08 | |
| *** gtema has joined #zuul | 09:12 | |
| *** ssbarnea|bkp2 has joined #zuul | 09:37 | |
| *** ssbarnea has quit IRC | 09:39 | |
| tristanC | jhesketh: would you mind if we rebase and continue the work on topic:freeze_job ? | 09:44 |
|---|---|---|
| jhesketh | tristanC: not at all. I have a partial rebase, but it's probably outdated enough that I would just begin rebasing again | 09:46 |
| jhesketh | tristanC: was there a part in particular you wanted to look at ? | 09:47 |
| tristanC | jhesketh: getting to the point where there is a documentation about how to run a job locally :) | 09:49 |
| tristanC | jhesketh: i mean, using the zuul-runner utility you proposed in that freeze_job topic | 09:51 |
| tristanC | if it's ok for you, we may allocate some time and work on this starting next week | 09:52 |
| jhesketh | tristanC: sure, that'd be awesome. Hopefully the general direction and framework is there enough. | 09:56 |
| jhesketh | It should be pretty close to a kind of PoC for getting playbooks that can then be ran with some manual work around inventories and images | 09:57 |
| tristanC | jhesketh: yes, the stack looks great already, i'd like to give it a try and fix ci on the way | 09:58 |
| jhesketh | Cool :-) | 09:59 |
| *** hashar has quit IRC | 10:02 | |
| *** hashar has joined #zuul | 10:06 | |
| *** bhavikdbavishi has quit IRC | 10:21 | |
| openstackgerrit | Merged openstack-infra/zuul master: dict_object.keys() is not required for *in* operator https://review.openstack.org/621482 | 10:27 |
| *** sshnaidm|off is now known as sshnaidm | 10:30 | |
| *** arxcruz|brb is now known as arxcruz | 10:35 | |
| *** pcaruana has quit IRC | 11:11 | |
| *** pcaruana has joined #zuul | 11:16 | |
| *** hashar has quit IRC | 11:41 | |
| tobiash | tristanC: is the swagger.json hand written or somehow generated? | 11:47 |
| jhesketh | tobiash: there are tools that will help generate it, but afaik we aren't using any of those. So unless tristanC hacked something together to generate it my guess is that it would be hand written | 11:53 |
| tobiash | jhesketh: thanks | 11:53 |
| jhesketh | we should consider our options for deeper integration with it though. swagger could go in either direction. That is, either generated from our code, or we could have our routing defined by swagger.json | 11:53 |
| jhesketh | I'd like to see something like that as it'll help ensure that our documentation reflects reality. However, all options (including manually writing obviously) have drawbacks | 11:54 |
| *** bhavikdbavishi has joined #zuul | 11:54 | |
| tobiash | hrm, ok, the current version of 535541 doesn't really seem to test the schema against the api | 11:55 |
| tristanC | tobiash: i wrote the initial swagger file in earlier patchset in yaml, and converted in json in later patchset so that it's part of the public file | 11:56 |
| tobiash | I fear that if we add such a detailed documentation without any coupling or testing it will become outdated pretty quickly | 11:57 |
| * SpamapS would love to see that doc in-code somehow. | 12:00 | |
| tristanC | tobiash: agreed, perhaps we could use something like https://github.com/apigee-127/sway to validate the reference | 12:01 |
| tristanC | SpamapS: that swagger file could probably be constructed from class method docstring | 12:01 |
| tristanC | note that the initial goal was just to document what we have... :) | 12:02 |
| SpamapS | tristanC: yeah that's what I was hoping. | 12:03 |
| SpamapS | but, honestly, hand-written and mostly up to date is great | 12:03 |
| jhesketh | I think there may have been a reason that doing it in the docstring wasn't straight forward. It might be due to the way we construct the routes. However, if we can move it into docstrings that would be awesome | 12:05 |
| *** dkehn has quit IRC | 12:21 | |
| *** zigo has quit IRC | 12:31 | |
| *** bhavikdbavishi has quit IRC | 12:32 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool master: Extract common config parsing for ProviderConfig https://review.openstack.org/625094 | 12:34 |
| *** rlandy has joined #zuul | 12:42 | |
| *** gtema has quit IRC | 12:43 | |
| SpamapS | FYI: https://www.vectorlogo.zone/logos/zuul-ci/index.html we exist! :-D | 12:46 |
| *** gtema has joined #zuul | 12:47 | |
| *** evrardjp has joined #zuul | 12:54 | |
| *** bjackman has quit IRC | 12:55 | |
| *** evrardjp has quit IRC | 12:59 | |
| *** evrardjp has joined #zuul | 12:59 | |
| *** jhesketh has quit IRC | 13:05 | |
| *** bjackman has joined #zuul | 13:05 | |
| *** jhesketh has joined #zuul | 13:06 | |
| *** hashar has joined #zuul | 13:24 | |
| *** gtema has quit IRC | 13:34 | |
| *** gtema has joined #zuul | 13:44 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-base-jobs master: Add fetch-output to base jobs https://review.openstack.org/628975 | 14:07 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-base-jobs master: Ignore errors on ssh key removal https://review.openstack.org/628976 | 14:07 |
| *** irclogbot_3 has quit IRC | 14:14 | |
| *** gtema has quit IRC | 14:25 | |
| *** needsleep is now known as TheJulia | 14:36 | |
| *** irclogbot_3 has joined #zuul | 14:41 | |
| mordred | tristanC: btw - I absolutely love the 'show the errors' bit in the build page | 14:53 |
| *** rlandy is now known as rlandy|rover | 14:56 | |
| tristanC | mordred: yeah, i'm really looking forward to return the build page as job result now ;) | 14:57 |
| AJaeger | tristanC: cool - didn't notice that one before | 14:59 |
| tristanC | it's just a small processing, we could extend it to display more things, such as the list of roles and how long they last each | 15:01 |
| pabelanger | so, I noticed we are leaking command sockets for zuul. Any objections on cleaning those up? I'm considering using them for an ansible playbook check to confirm a zuul service is stopped | 15:02 |
| mordred | tristanC: me too | 15:02 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul master: Clean up command sockets on stop https://review.openstack.org/628990 | 15:03 |
| pabelanger | for example: ^ | 15:03 |
| mordred | pabelanger: I do not have any objections | 15:03 |
| *** irclogbot_3 has quit IRC | 15:06 | |
| *** irclogbot_3 has joined #zuul | 15:15 | |
| *** openstackgerrit has quit IRC | 15:22 | |
| *** openstackgerrit has joined #zuul | 15:23 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul master: Ensure command_socket is last thing to close https://review.openstack.org/628995 | 15:23 |
| pabelanger | ^ is a follow up to command socket cleanup. All services should now stop it as the last step, that is how the scheduler was working | 15:24 |
| tobiash | pabelanger: do you think it makes sense to validate a cleaned up command socket in ZuulTestCase.shutdown? | 15:41 |
| pabelanger | tobiash: great idea, can add a test to confirm | 15:42 |
| tobiash | cool | 15:42 |
| tobiash | anyone want to look at https://review.openstack.org/621391 ? This fixes an annoying warning in my ide ;) | 15:43 |
| tobiash | it's just a tiny gitignore fix | 15:44 |
| *** pcaruana has quit IRC | 16:21 | |
| *** bjackman has quit IRC | 16:40 | |
| mordred | tobiash: +A | 16:41 |
| *** hashar has quit IRC | 16:51 | |
| *** rfolco has quit IRC | 16:58 | |
| *** rfolco has joined #zuul | 16:59 | |
| openstackgerrit | Merged openstack-infra/zuul master: Fix ignored but tracked .keep file https://review.openstack.org/621391 | 17:06 |
| *** hashar has joined #zuul | 17:07 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul-jobs master: [WIP] upload-pypi: add option to register packages https://review.openstack.org/629018 | 17:19 |
| *** rf0lc0 has joined #zuul | 17:30 | |
| *** rf0lc0 has quit IRC | 17:31 | |
| *** rfolco has quit IRC | 17:33 | |
| *** rfolco has joined #zuul | 17:42 | |
| *** dkehn has joined #zuul | 18:29 | |
| dkehn | clarkb: when running the zuul-migrate is there an easy apporach i.e. zuul-migrate layout.yam output-dir? | 18:29 |
| clarkb | dkehn: Its been about a year since I had to think about that stuff. I'm not sure. Basically you get a bunch of output broken up into some logical files | 18:31 |
| dkehn | clarkb: ok, is the run-magrate script around anywhere https://git.zuul-ci.org/cgit/zuul/tree/playbooks/zuul-migrate/run.yaml | 18:41 |
| clarkb | dkehn: https://git.zuul-ci.org/cgit/zuul/commit/tools?id=9e20f12ca84a0853d79090aebddd6fe3e9e0d1bc it was removed in that commit so if you checkout the commit previous you should have it | 18:42 |
| dkehn | clarkb: thanks, was wondering of the calling, because when I just do the layout and outdir I get the job-config error, it helps | 18:45 |
| clarkb | dkehn: I think the mapping file there is a hints file taht says rename foo to bar | 18:46 |
| clarkb | dkehn: but I don't remember all the details | 18:46 |
| clarkb | if you go back in the history of the git repo you'll likely find those details though | 18:46 |
| pabelanger | dkehn: keep in mind, zuul-migrate was very openstack specific. I used it to migrate rdoproject.org to zuulv3, but think i pushed up all my changes upstream | 19:11 |
| pabelanger | basically, multi node jobs, you are better off re-writing from scratch IMO | 19:11 |
| *** ianw_pto is now known as ianw | 22:26 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well https://review.openstack.org/511853 | 22:32 |
| *** hashar has quit IRC | 22:47 | |
| openstackgerrit | Merged openstack-infra/zuul master: Add timer for starting_builds https://review.openstack.org/623468 | 22:57 |
| *** ianychoi has quit IRC | 23:21 | |
| clarkb | tobiash: corvus: I left some comments on the distributed scheduler spec | 23:35 |
| clarkb | for some of that I don't actually know if one appraoch will turn out better than another, but tried to think about it in terms of avoiding the known drawbacks with the listed appraoches | 23:36 |
| corvus | clarkb: replied, something is unclear to me; maybe you can clarify | 23:43 |
| clarkb | corvus: given the costs of git merges that makes sense (I wasn't really advocating for that method as much as sharing a high level of what other application have done) I also responded in gerrit so that this comment isn't lost :) | 23:48 |
| corvus | clarkb: cool, thx | 23:49 |
| -openstackstatus- NOTICE: The Etherpad service at https://etherpad.openstack.org/ has been offline since 23:22 UTC due to a hypervisor issue in our service provider, but should hopefully return to service shortly. | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!