| *** rlandy|rover is now known as rlandy|rover|bbl | 00:04 | |
| *** jamesmcarthur has joined #zuul | 00:53 | |
| *** jamesmcarthur has quit IRC | 01:30 | |
| *** jamesmcarthur has joined #zuul | 01:38 | |
| *** jamesmcarthur has quit IRC | 02:13 | |
| *** rlandy|rover|bbl is now known as rlandy|rover | 02:31 | |
| *** rlandy|rover has quit IRC | 02:31 | |
| *** jkt has quit IRC | 02:37 | |
| *** jkt has joined #zuul | 02:38 | |
| *** jamesmcarthur has joined #zuul | 02:48 | |
| *** jamesmcarthur has quit IRC | 02:51 | |
| *** jamesmcarthur has joined #zuul | 02:52 | |
| *** jamesmcarthur has quit IRC | 03:14 | |
| *** jamesmcarthur has joined #zuul | 03:44 | |
| *** jamesmcarthur has quit IRC | 03:48 | |
| *** adam_g has quit IRC | 03:51 | |
| *** adam_g has joined #zuul | 03:52 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Get executor job params https://review.openstack.org/607078 | 04:47 |
|---|---|---|
| openstackgerrit | Merged openstack-infra/zuul master: web: check if project has variant https://review.openstack.org/647260 | 05:20 |
| *** saneax has joined #zuul | 05:30 | |
| *** quiquell|off is now known as quiquell | 06:43 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Separate out executor server from runner https://review.openstack.org/607079 | 06:45 |
| tristanC | jhesketh: i'll propose a refactor on top of 607079 to make the interface between AnsibleBaseJob and the executor cleaner | 06:49 |
| jhesketh | tristanC: +1, sounds good thanks | 06:50 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Organize documentation by subject area https://review.openstack.org/645955 | 06:51 |
| *** pcaruana has joined #zuul | 07:10 | |
| *** themroc has joined #zuul | 08:12 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Elasticsearch Zuul reporter https://review.openstack.org/644927 | 08:17 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Separate out executor server from runner https://review.openstack.org/607079 | 08:18 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: runner: implement prep-workspace https://review.openstack.org/607082 | 08:18 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: runner: add configuration schema https://review.openstack.org/640672 | 08:18 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: runner: add execute sub-command https://review.openstack.org/630944 | 08:18 |
| tristanC | jhesketh: not that i'm super found of long contrusctor, but here are the arguments needed by AnsibleJobBase: https://review.openstack.org/#/c/607082/21/zuul/executor/common.py@550 | 08:20 |
| *** jpena|off is now known as jpena | 08:21 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Elasticsearch Zuul reporter https://review.openstack.org/644927 | 08:23 |
| jhesketh | tristanC: hmm, I think it's okay | 08:23 |
| jhesketh | tristanC: Are you just rebasing, or looking to address James' other feedback? | 08:24 |
| jhesketh | I'm happy to help out, but don't want to step on your toes/duplicate efforts etc | 08:24 |
| tristanC | i think i addressed James' comment on the first couple of patches. Then for the config schema and cli interface i didn't look into them | 08:27 |
| jhesketh | cool (I haven't had a chance to look at the shuffling yet sorry) | 08:29 |
| tristanC | for the config schema, perhaps we should drop runtime (e.g. project name, job name) information and only keep runner dirs, user provided nodes, connections informations, (and latter nodepool provider) | 08:29 |
| tristanC | and for the cli, perhaps we should drop the sub-command, zuul-runner should execute the job by default, unless we use a "--dry" flag to just prep-workspace | 08:30 |
| tristanC | corvus: jhesketh: btw, the current stack head is https://review.openstack.org/630944 | 08:32 |
| *** electrofelix has joined #zuul | 08:40 | |
| *** quiquell has quit IRC | 08:47 | |
| *** quiquell has joined #zuul | 08:47 | |
| *** sshnaidm|pto is now known as sshnaidm|rover | 09:00 | |
| *** hashar has joined #zuul | 09:07 | |
| *** gtema has joined #zuul | 09:56 | |
| *** fakir has joined #zuul | 10:03 | |
| fakir | Hey guys, I got a problem with connecting second openstack project to the zuul. I ve add new provider with correct pools and labels to the nodepool.yaml and 2nd. project to the clouds.yaml, but still i cannot spawn nodes on that machine :/ Did anyone had same problems? | 10:04 |
| arxcruz | jhesketh: hey, around? | 10:44 |
| arxcruz | jhesketh: thanks for the patch, https://review.openstack.org/#/c/607077 :) | 10:45 |
| arxcruz | jhesketh: can you help me? how can i discover the tenant to consume the api ? | 10:45 |
| jhesketh | arxcruz: there's the /api/tenants endpoint if that's what you are after? | 10:54 |
| jhesketh | otherwise take a look at https://review.openstack.org/#/c/630944/ and its parents commits for how the api is intended to be used | 10:54 |
| arxcruz | jhesketh: thanks, i'm not able to reach api/tenants however, the code isn't in production yet anyway | 10:55 |
| arxcruz | http://zuul.openstack.org/api/tenants isn't returning anything | 10:55 |
| jhesketh | arxcruz: that's because it's deployed in a single-tenant mode | 10:59 |
| jhesketh | so /api/info will give you some info, but the tenant is always openstack | 10:59 |
| jhesketh | so you can drop the "tenant/openstack" part of the path | 11:00 |
| jhesketh | but as for the freeze-jobs endpoint, I don't think zuul has been restarted to pick that up yet anyway | 11:01 |
| arxcruz | jhesketh: so, when zuul restart, it will be zuul.openstack.org/api/tenant/openstack/pipeline/check/project/openstack/tripleo-quickstart/branch/master/freeze-jobs for example right ? | 11:02 |
| jhesketh | I think zuul.openstack.org/api/pipeline/check/project/openstack/tripleo-quickstart/branch/master/freeze-jobs should be correct (no tenant, as that's rewritten) | 11:03 |
| arxcruz | jhesketh: cool, thanks! | 11:04 |
| jhesketh | (also my internet provider is currently dropping ~40% packets, so I may be a little flaky on IRC atm) | 11:05 |
| arxcruz | :) | 11:07 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add OpenAPI documentation https://review.openstack.org/535541 | 11:46 |
| quiquell | Hi we have a question about using job.override-checkout | 11:52 |
| quiquell | In chase the branch we put the does not exists it will checkout master branch ? | 11:52 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: runner: implement prep-workspace https://review.openstack.org/607082 | 11:53 |
| *** dmsimard has joined #zuul | 12:10 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: runner: add configuration schema https://review.openstack.org/640672 | 12:25 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: runner: add execute sub-command https://review.openstack.org/630944 | 12:26 |
| *** gtema has quit IRC | 12:33 | |
| *** jpena is now known as jpena|lunch | 12:35 | |
| *** rlandy has joined #zuul | 12:37 | |
| *** jamesmcarthur has joined #zuul | 12:46 | |
| *** fakir has quit IRC | 13:20 | |
| pabelanger | for a config-project, I learned last night that is seems to only support a single branch (master) to load its configuration from: https://zuul-ci.org/docs/zuul/user/config.html#configuration-loading | 13:28 |
| pabelanger | Is there a way that could be changed to selected which branch contains its configuration? The use case I am thinking of is migrating a config-project between 2 different zuul. | 13:29 |
| pabelanger | eg: zuul A -> branch A and zuul B -> branch B, cut over projects from zuul A to zuul B, then delete branch A in config-project | 13:29 |
| pabelanger | otherwise, it seems I have to create 2 seperate config-projects and only attach a single zuul to each | 13:30 |
| *** jpena|lunch is now known as jpena | 13:35 | |
| *** gtema has joined #zuul | 13:38 | |
| *** jamesmcarthur has quit IRC | 14:11 | |
| *** jamesmcarthur has joined #zuul | 14:13 | |
| *** jamesmcarthur has quit IRC | 14:14 | |
| *** quiquell is now known as quiquell|off | 14:18 | |
| Shrews | fyi to all: out this afternoon for doctor things | 14:26 |
| *** jamesmcarthur has joined #zuul | 14:36 | |
| *** zbr has quit IRC | 14:47 | |
| *** zbr has joined #zuul | 14:50 | |
| *** saneax has quit IRC | 14:56 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Elasticsearch Zuul reporter https://review.openstack.org/644927 | 15:15 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Elasticsearch Zuul reporter https://review.openstack.org/644927 | 15:15 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: WIP: Fix for image build leaks https://review.openstack.org/647599 | 15:17 |
| fungi | trying to put together a brief blurb for the osf bi-weekly newsletter... is the main thrust of the zuul-runner work to make it easier to users to locally reproduce zuul jobs? not finding any related spec (approved nor proposed) | 15:18 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Elasticsearch Zuul reporter https://review.openstack.org/644927 | 15:24 |
| corvus | fungi: that's the idea, and yeah, no spec. we'd have a spec if we started something like that today. | 15:27 |
| fungi | no problem, just didn't want to ask questions where the answer was already clearly documented in a spec. thanks! | 15:28 |
| fungi | the project update was a little light on the goal of that feature | 15:29 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: WIP: Fix for image build leaks https://review.openstack.org/647599 | 15:32 |
| *** themroc has quit IRC | 15:35 | |
| corvus | tobiash: what do you think of this change? is it the right behavior? https://review.openstack.org/644392 | 15:44 |
| *** hashar has quit IRC | 16:10 | |
| corvus | tobiash: comment on https://review.openstack.org/635166 | 16:14 |
| corvus | pabelanger: can you see if your concerns on https://review.openstack.org/632620 are addresse? | 16:16 |
| corvus | addressed | 16:16 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add the tox_install_bindep option to the tox job https://review.openstack.org/635877 | 16:23 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Add a role to mirror a git repository to a remote git server https://review.openstack.org/643394 | 16:24 |
| corvus | dmsimard: ^ i updated your change because my docs reorg would have broken it | 16:25 |
| dmsimard | corvus: np, it removed your +2 though | 16:25 |
| corvus | dmsimard: yep, readding now | 16:25 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Pin nodejs package to upstream repo https://review.openstack.org/627823 | 16:31 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: run-test-command: support list in test_command variable https://review.openstack.org/610888 | 16:34 |
| tobiash | corvus: re 644392: yes, I think that should be correct. It's like specifying dependencies | 16:43 |
| tobiash | corvus: responded on 635166 | 16:45 |
| corvus | tobiash: ok, i +3d it, we can flip the default later | 16:46 |
| tobiash | :) | 16:46 |
| *** gtema has quit IRC | 16:54 | |
| dmsimard | The fact that Zuul adds comments in gerrit to tell me on which line exactly I screwed up is pretty cool | 17:00 |
| dmsimard | ++ | 17:00 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Optionally silence git in mirror-workspace-git-repos https://review.openstack.org/635166 | 17:00 |
| corvus | there's a change out there to do that for pep8 jobs too, but no one is pushing on that right now | 17:00 |
| corvus | https://review.openstack.org/611549 is the latest on that | 17:01 |
| dmsimard | corvus: is there a generic way to get Zuul to write different things when returning the results of a patchset's jobs ? | 17:06 |
| dmsimard | oh wait | 17:07 |
| corvus | dmsimard: write things where? | 17:07 |
| dmsimard | corvus: in the gerrit/github comment | 17:07 |
| dmsimard | but reading that pep8 code gave me part of the answer | 17:07 |
| corvus | dmsimard: well, that pep8 code only does inline file comments | 17:09 |
| corvus | dmsimard: https://zuul-ci.org/docs/zuul/user/config.html#attr-job.success-message is relevant | 17:09 |
| corvus | dmsimard: as is https://zuul-ci.org/docs/zuul/user/config.html#attr-pipeline.success-message | 17:09 |
| dmsimard | yeah I knew about the pipelien success message, TIL about the job equivalent | 17:09 |
| corvus | here's the docs for file comments https://zuul-ci.org/docs/zuul/user/jobs.html#leaving-file-comments | 17:10 |
| dmsimard | That is super cool. | 17:11 |
| *** chandankumar is now known as raukadah | 17:14 | |
| dmsimard | corvus: I can work with file_comments but there's probably a gap between file_comments being dynamic and success-message being static (to a job or pipeline) | 17:16 |
| corvus | dmsimard: that's true | 17:16 |
| dmsimard | i.e, Something inbetween the two, a dynamic message not necessarily targetted at a file :p | 17:17 |
| *** jamesmcarthur has quit IRC | 17:25 | |
| *** zbr|ssbarnea has joined #zuul | 17:29 | |
| *** jpena is now known as jpena|off | 17:31 | |
| *** zbr has quit IRC | 17:31 | |
| *** rlandy is now known as rlandy|biab | 17:35 | |
| *** hashar has joined #zuul | 17:41 | |
| *** jamesmcarthur has joined #zuul | 17:56 | |
| *** manjeets_ has joined #zuul | 18:01 | |
| *** rlandy|biab is now known as rlandy | 18:23 | |
| *** hashar is now known as hasharDinner | 19:15 | |
| 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 | 19:19 |
| 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 | 19:20 |
| *** yolanda has quit IRC | 19:33 | |
| *** hasharDinner has quit IRC | 19:35 | |
| *** hasharDinner has joined #zuul | 19:36 | |
| *** electrofelix has quit IRC | 19:37 | |
| *** jamesmcarthur has quit IRC | 19:45 | |
| *** panda|ko is now known as panda|off | 19:55 | |
| *** yolanda has joined #zuul | 20:01 | |
| *** hasharDinner has quit IRC | 21:22 | |
| *** pcaruana has quit IRC | 21:33 | |
| *** hasharDinner has joined #zuul | 21:53 | |
| *** panda|off has quit IRC | 22:24 | |
| *** hasharDinner is now known as hashar | 22:45 | |
| *** rlandy has quit IRC | 23:27 | |
| openstackgerrit | William Huang proposed openstack-infra/zuul-jobs master: Add variable `npm_mirror` for changing default npm package registry https://review.openstack.org/647305 | 23:48 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!