| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-base-jobs master: Initial boilerplate, packaging and testing https://review.openstack.org/526139 | 01:07 |
|---|---|---|
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-base-jobs master: Add generic base and base-test jobs/playbooks https://review.openstack.org/526140 | 01:07 |
| tristanC | mordred: regarding zuul-web api documentation, how do you feel about using os-api-ref? | 01:27 |
| mordred | tristanC: it's not a bad idea ... although I'd love to get this: https://review.openstack.org/#/c/464255/ fixed and landed (similar to how the zuul sphinx extension handles sub-elements and whatnot) | 02:01 |
| mordred | tristanC: I have two additional quesitons though- I haven't looked at the codebase in a little while- is it set up to be able to use it without also using openstackdoctheme? | 02:02 |
| mordred | tristanC: yah- http://git.openstack.org/cgit/openstack/os-api-ref/tree/requirements.txt#n9 <-- we might want to eventually figure out a mechanism to not strongly tie the two together if we're going to use it (although we use openstackdocstheme currently, so it's not an immediate problem) | 02:03 |
| mordred | tristanC: I thnk that's my main question. the other one would be is there any benefit in using swagger/openapi instead (even if it's just some hand-written swagger entries rather than using swagger for codegen) | 02:07 |
| mordred | tristanC: like with http://aiohttp-swagger.readthedocs.io/en/latest/ - for instance. I believe openstack was interested in swagger at some point but mapping the /action endpoints ends up being hard | 02:07 |
| mordred | tristanC: but - I also like the os-api-ref work, so I don't have a *strong* opinion (other than that having API docs is a good idea) | 02:09 |
| mordred | tristanC: did you see the live draft copies in the new stack? | 02:15 |
| mordred | tristanC: http://logs.openstack.org/98/528298/10/check/build-javascript-content/2e5b6ac/npm/html/builds.html?source_url=http:%2F%2Fzuulv3.openstack.org - for instance | 02:16 |
| tristanC | mordred: aiohttp-swagger sounds better indeed | 02:19 |
| tristanC | mordred: i mentioned os-api-ref because it's already there and the doc is rst so easy to add to the current zuul doc | 02:20 |
| tristanC | mordred: yes, the job result and the npm stuff looks all good to me | 02:21 |
| mordred | tristanC: yah - os-api-ref is not a bad idea for sure | 02:23 |
| tristanC | mordred: as clarkb said, let's just hope that all those javascript goodies doesn't get deprecated too soon | 02:24 |
| mordred | tristanC: :) | 02:27 |
| *** flepied has joined #zuul | 02:44 | |
| *** rlandy|rover has quit IRC | 03:10 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul-jobs master: Support autodoc_tree_index_modules in sphinx role https://review.openstack.org/528946 | 03:25 |
| *** flepied has quit IRC | 03:45 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/zuul-jobs master: Support autodoc_tree_index_modules in sphinx role https://review.openstack.org/528946 | 03:51 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: zk: automatically retry command when connection is lost https://review.openstack.org/523640 | 04:14 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Support autodoc_tree_index_modules in sphinx role https://review.openstack.org/528946 | 06:06 |
| *** flepied has joined #zuul | 06:08 | |
| * tobiash wanted to attend the zuul meeting but failed and fell asleep five minutes before start | 06:14 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix docs building https://review.openstack.org/528822 | 06:21 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: config: add statsd-server config parameter https://review.openstack.org/528969 | 06:44 |
| *** yolanda has joined #zuul | 06:47 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Git driver https://review.openstack.org/525614 | 06:48 |
| SpamapS | Hm | 07:38 |
| SpamapS | Today I had a MERGER_FAILURE because my zuul vm had a momentary network blip that made it unable to reach my github service... | 07:38 |
| SpamapS | Looking through the executor log.. it only failed once. That seems retriable. | 07:39 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement an OpenContainer driver https://review.openstack.org/468753 | 07:46 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement a Kubernetes driver https://review.openstack.org/521356 | 07:46 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement an Amazon EC2 driver https://review.openstack.org/528982 | 07:46 |
| *** xinliang has quit IRC | 08:22 | |
| *** hashar has joined #zuul | 08:32 | |
| *** yolanda__ has joined #zuul | 08:34 | |
| *** xinliang has joined #zuul | 08:34 | |
| *** yolanda has quit IRC | 08:36 | |
| *** jpena|off is now known as jpena | 08:51 | |
| *** hashar has quit IRC | 08:54 | |
| *** hashar has joined #zuul | 08:59 | |
| *** jaianshu has joined #zuul | 09:27 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add swagger api documentation https://review.openstack.org/529013 | 09:58 |
| *** Shrews has quit IRC | 10:00 | |
| tristanC | mordred: jeblair: the zuul-web api swagger documentation rendered with aiohttp_swagger: https://fedorapeople.org/~tdecacqu/zuul-swagger.png | 10:03 |
| *** Shrews has joined #zuul | 10:10 | |
| *** bhavik has joined #zuul | 10:10 | |
| *** bhavik has quit IRC | 10:26 | |
| *** electrofelix has joined #zuul | 10:46 | |
| *** bhavik has joined #zuul | 11:19 | |
| jaianshu | Hello all..i'm facing issue at the time zuul executes ansible job from playbooks. [http://paste.openstack.org/show/629297/] | 11:21 |
| jaianshu | i get SSH error and my node gets deleted. Can anyone suggest how to triage this, as i do not have any backtrace of this error? | 11:23 |
| tobiash | jaianshu: execute 'zuul-executor verbose' and rerun the job | 11:32 |
| tobiash | that will give more debug output | 11:32 |
| tobiash | but that looks like some sort of connection or ssh pubkey auth problem | 11:33 |
| jaianshu | tobiash: one question, do we need to specify ssh keys anywhere in zuul for it to contact the node that is created by nodepool or it gets that info automatically? | 11:46 |
| *** yolanda__ is now known as yolanda | 11:50 | |
| *** bhavik has quit IRC | 12:02 | |
| *** flepied has quit IRC | 12:07 | |
| *** flepied has joined #zuul | 12:10 | |
| tobiash | jaianshu: yes you need | 12:25 |
| tobiash | then that's probably your problem ;) | 12:25 |
| tobiash | jaianshu: https://docs.openstack.org/infra/zuul/feature/zuulv3/admin/drivers/gerrit.html#connection-configuration | 12:26 |
| tobiash | the sshkey option for the gerrit driver | 12:26 |
| tobiash | jaianshu: forget the last comment, the sshkey option for the gerrit driver is not used for accessing the slaves ;) | 12:28 |
| tobiash | jaianshu: the correct setting to access the slaves is in the executor section: https://docs.openstack.org/infra/zuul/feature/zuulv3/admin/components.html#attr-executor.private_key_file | 12:28 |
| jaianshu | tobiash: yes..got it..trying it now | 12:29 |
| *** jpena is now known as jpena|lunch | 12:48 | |
| *** weshay_pto is now known as weshay | 12:48 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Do not exit early tenant configuration in case of config issue https://review.openstack.org/529060 | 13:04 |
| fbo_ | Hi, not sure at all we can safely do that ^ but it seems a wrong config in a zuul.yaml will broke the whole tenant config. Do you think that can be improved, is there any plan about it ? | 13:14 |
| *** hashar has quit IRC | 13:25 | |
| *** hashar has joined #zuul | 13:26 | |
| jaianshu | tobiash: thanks,...it worked :) | 13:27 |
| *** jaianshu has quit IRC | 13:28 | |
| *** rlandy has joined #zuul | 13:29 | |
| *** rlandy is now known as rlandy|rover | 13:29 | |
| *** jpena|lunch is now known as jpena | 13:49 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Dereference symlinks when staging artefacts https://review.openstack.org/528748 | 13:58 |
| *** dkranz has joined #zuul | 14:22 | |
| *** sshnaidm has quit IRC | 14:46 | |
| *** sshnaidm has joined #zuul | 14:48 | |
| *** flepied has quit IRC | 14:51 | |
| *** flepied has joined #zuul | 14:59 | |
| jeblair | mordred: we do not use openstackdocstheme. | 15:30 |
| jeblair | mordred, tristanC: if it's possible to use os-api-ref without openstackdocs, i expect it should be fine. is it possible to incorporate swagger into the rst docs build? i'd like it to be part of the same documentation and match -- in particular, i'd like to be able to write narrative documentation about the api in rst. | 15:33 |
| *** hashar is now known as hasharAway | 16:18 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant configuration in case of config issue https://review.openstack.org/529060 | 17:06 |
| rbergeron | greetings, lovely humans: someone pinged on the ansible-container mailing list about whether or not ansible-container can work with zuul v3, and stated their use case -- it's not an address i recognize offhand, either. anyway: | 17:12 |
| rbergeron | https://groups.google.com/forum/#!topic/ansible-container/MX6g4PRMjQ0 -- if anyone happens to know the answer or wants to take a guess, that would be lovely, since i got pinged by someone who was like "ummm.... clues. none." | 17:13 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant configuration in case of config issue https://review.openstack.org/529060 | 17:14 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul feature/zuulv3: Attempt to improve tenant config loading in case of config issue https://review.openstack.org/529060 | 17:17 |
| mordred | rbergeron: interesting - I think we should ping rcarrillocruz as he's been doin ansible-container things recently | 17:19 |
| rcarrillocruz | sup | 17:19 |
| * rcarrillocruz reads | 17:19 | |
| mordred | jeblair: Im not sure about swagger incorporation with sphinx - looking at the patch from tristanC, I think that's next on his todo list to figure out perhaps | 17:20 |
| rcarrillocruz | rbergeron: replied | 17:28 |
| *** flepied_ has joined #zuul | 17:44 | |
| *** flepied has quit IRC | 17:46 | |
| rbergeron | rcarrillocruz: i just saw that, ty! :) | 17:59 |
| *** harlowja has joined #zuul | 18:15 | |
| *** jpena is now known as jpena|off | 18:33 | |
| pabelanger | mordred: clarkb: jeblair: working on a downstream nodepool issue with paramiko, any objections if we uncap paramiko in nodepool / zuul? https://review.openstack.org/528719/ we only capped before to address deadlock issue and seems to have been since fixed | 18:36 |
| clarkb | uncapped paramiko brings in a bunch of new deps that require you to potentially have system header packages installed | 18:38 |
| clarkb | (typically upstream puts wheels up for that but not always) | 18:38 |
| clarkb | that would be my only concern I think | 18:38 |
| SpamapS | that is probably due to the crypto | 18:38 |
| SpamapS | and thus probably worthwhile | 18:38 |
| mordred | yah - other than what clarkb says I don't have any issues with it | 18:39 |
| SpamapS | better now than 3.1 | 18:40 |
| pabelanger | okay cool | 18:42 |
| pabelanger | https://review.openstack.org/528729/ is for zuul too | 18:43 |
| clarkb | pabelanger: the change should be > whatever version was broken | 18:46 |
| clarkb | and exclude 1.x entirely if we are saying that is no longer good | 18:47 |
| pabelanger | clarkb: ah, yes. we could do that too | 18:48 |
| pabelanger | >2.0.1 has the deadlock fixes | 18:49 |
| openstackgerrit | Paul Belanger proposed openstack-infra/nodepool feature/zuulv3: requirements: remove paramiko <2.0 cap https://review.openstack.org/528719 | 18:54 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: requirements: remove paramiko <2.0 cap https://review.openstack.org/528729 | 18:56 |
| pabelanger | clarkb: ^ | 18:56 |
| clarkb | pabelanger: is it >2.0.1 or >=2.0.1 (we have one above and the other in gerrit) | 18:57 |
| pabelanger | 2.0.1 has the fix | 18:58 |
| clarkb | on the nodepool side integration testing will catch any straightforward issues, I'm not sure about the zuul side | 18:58 |
| pabelanger | 2.0.0 was the bad release | 18:58 |
| pabelanger | what in zuul are we using paramiko for anyways? | 18:58 |
| pabelanger | I haven't actually looked | 18:58 |
| clarkb | oh right this is the ansible dep that we don't actually use? | 18:59 |
| pabelanger | no, looks like gerritdriver | 18:59 |
| clarkb | oh right | 18:59 |
| clarkb | for stream events | 18:59 |
| pabelanger | yah | 18:59 |
| tobiash | not only stream events but also the gerrit api (query, etc) | 19:24 |
| mordred | oh. right. we still don't use gerritlib in zuul | 19:30 |
| *** openstack has joined #zuul | 19:44 | |
| *** ChanServ sets mode: +o openstack | 19:44 | |
| *** hasharAway is now known as hashar | 19:47 | |
| *** electrofelix has quit IRC | 19:48 | |
| *** rlandy|rover is now known as rlandy|rover|bia | 19:49 | |
| pabelanger | with nodepool, did we not have a patch up to delete ready nodes > 8 hours? | 20:10 |
| pabelanger | I know in feature/zuulv3 that changed some | 20:11 |
| SpamapS | I've seen nodes sit ready on mine for days | 20:31 |
| SpamapS | When I removed all the jobs that used a label the min ready sat there for a long time. | 20:31 |
| pabelanger | k, maybe we haven't changed it | 20:32 |
| mordred | jeblair, tobiash: DUH. I'm actually quite stupid wrt the unittest failure | 20:34 |
| mordred | oh - wait - no, nevermind | 20:34 |
| mordred | I thought I had a really simple fix | 20:34 |
| *** rlandy|rover|bia is now known as rlandy|rover | 20:38 | |
| *** myoung is now known as myoung|bbl | 21:35 | |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Fail on quota-exceeded (partial revert) https://review.openstack.org/529173 | 22:56 |
| *** myoung|bbl is now known as myoung | 22:56 | |
| *** lennyb has quit IRC | 22:58 | |
| *** lennyb has joined #zuul | 23:00 | |
| *** hashar has quit IRC | 23:15 | |
| pabelanger | +2^ | 23:15 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Log provider names with quota https://review.openstack.org/529178 | 23:21 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool feature/zuulv3: Assume a quota limit of -1 means unlimited https://review.openstack.org/529180 | 23:27 |
| jeblair | tobiash: when you're awake, see the 3 changes above, based on our experience using the new quota system in openstack-infra | 23:32 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
