| sfbender | Tristan de Cacqueray created software-factory/sf-config master: openshift: switch oc rsync to quiet https://softwarefactory-project.io/r/13748 | 03:06 |
|---|---|---|
| sfbender | Tristan de Cacqueray created software-factory/sf-config master: zuul: add experimental pipeline https://softwarefactory-project.io/r/13750 | 03:12 |
| *** chkumar|off is now known as chkumar|ruck | 05:42 | |
| *** jpena|off is now known as jpena | 07:53 | |
| *** chmouel_ has joined #softwarefactory | 08:23 | |
| *** chmouel_ is now known as chmouel | 08:37 | |
| *** sshnaidm has joined #softwarefactory | 09:07 | |
| *** sshnaidm is now known as sshnaidm|afk | 09:29 | |
| *** sshnaidm|afk is now known as sshnaidm|off | 09:37 | |
| *** sshnaidm|off has quit IRC | 09:42 | |
| *** zoli is now known as zoli|lunch | 10:11 | |
| *** zoli|lunch is now known as zoli | 10:11 | |
| *** jpena is now known as jpena|lunch | 11:38 | |
| *** jpena|lunch is now known as jpena | 12:40 | |
| pabelanger | nhicher: https://softwarefactory-project.io/r/13746/ starts some clean up with glean testing yesterday | 13:27 |
| nhicher | pabelanger: done | 13:35 |
| sfbender | Merged software-factory/sf-ci master: Re-enable test pages https://softwarefactory-project.io/r/13589 | 13:53 |
| sfbender | Tristan de Cacqueray created software-factory/sfinfo master: Add experimental job to sf-config https://softwarefactory-project.io/r/13755 | 14:08 |
| *** matburt has joined #softwarefactory | 14:09 | |
| matburt | tristanC did you want some of those PRs merged? It looks like the one on runner is just a test PR | 14:10 |
| sfbender | Merged software-factory/sfinfo master: Add experimental job to sf-config https://softwarefactory-project.io/r/13755 | 14:12 |
| tristanC | matburt: not really, they are just example use case. The actual job remains to be written | 14:20 |
| matburt | gotcha | 14:20 |
| tristanC | matburt: https://github.com/ansible/zuul-jobs/pull/8 should be safe to land | 14:20 |
| matburt | I'm reworking some of our job/project configuration today after talking to spredzy | 14:20 |
| matburt | tristanC reviewed and tagged | 14:22 |
| spredzy | tristanC: is the template.yml file loaded automatically or it needs something from your side ? | 14:24 |
| spredzy | in zuul-config/zuul.d/ | 14:24 |
| spredzy | ? | 14:24 |
| spredzy | We have it in DCI, but it isn't currently present on ansible/zuul-config | 14:24 |
| tristanC | spredzy: zuul-config is a config project, so change needs to be merged to be effective | 14:24 |
| tristanC | spredzy: what do you mean by "template.yml" ? | 14:25 |
| tristanC | spredzy: "project-template" zuul configuration doesn't needs to be in the config project, you can have them in ansible/zuul-jobs if you prefer, (e.g. https://softwarefactory-project.io/cgit/software-factory/sfinfo/tree/zuul.d/templates.yaml ) | 14:26 |
| spredzy | tristanC: https://softwarefactory-project.io/r/gitweb?p=dci-config.git;a=tree;f=zuul.d;h=c428b5845dfc1b7782d4c6afe387b1255cb4b69f;hb=HEAD | 14:26 |
| spredzy | See the templates.yaml file | 14:26 |
| spredzy | Is it automatically loaded/parsed ? | 14:27 |
| tristanC | spredzy: however, to use "project" zuul configuration for foreign project (e.g. assign template to awx from zuul-config), then that needs to be defined in a config project | 14:27 |
| tristanC | that is to preven one project from changing the ci configuration of another project | 14:27 |
| spredzy | Because it isn't their https://github.com/ansible/zuul-config/tree/master/zuul.d | 14:27 |
| tristanC | spredzy: zuul configuration is automtically loaded/parsed, nothing to do from our side | 14:28 |
| tristanC | spredzy: however, config-project changes needs to be merged first to be effective | 14:28 |
| spredzy | yep, this is fine | 14:28 |
| tristanC | spredzy: only regular (untrusted-project) are loaded/parsed speculatively | 14:28 |
| spredzy | I was just wondering about the templates.yaml file | 14:28 |
| spredzy | if it was loaded/parsed automatically | 14:29 |
| pabelanger | spredzy: filenames in zuul.d don't mean anything, just a way to group data over 1 large file. | 14:29 |
| pabelanger | zuul will merge them when loading config | 14:29 |
| spredzy | ack, so any *.yaml file will be loaded | 14:29 |
| spredzy | ack | 14:29 |
| pabelanger | any zuul config yaml file, yah | 14:29 |
| tristanC | spredzy: to keep things simple, puts the templates.yaml and projects.yaml (like the one you have in dci-config) in ansible/zuul-config | 14:29 |
| spredzy | thanks guys - I wasn't sure about this last point | 14:29 |
| spredzy | yes, this is where it is landing | 14:29 |
| tristanC | spredzy: we could move the template later in ansible/zuul-jobs if needed | 14:29 |
| tristanC | spredzy: note that files needs to have .yaml extension, .yml are silently ignored | 14:30 |
| spredzy | noted | 14:30 |
| pabelanger | Yah, that is a good point | 14:31 |
| *** dmsimard has quit IRC | 14:31 | |
| matburt | so anything in templates will be merged with jobs defined on the pipelines in the repo itself? | 14:33 |
| *** dmsimard has joined #softwarefactory | 14:33 | |
| tristanC | matburt: what do you mean? | 14:33 |
| tristanC | matburt: zuul-config can define template (which are just meta project definition) on behalf of another project | 14:34 |
| matburt | say tox is applied to a particular repo in a template and in the pipeline configuration another job is defined in .zuul.yaml they will both be run... | 14:34 |
| tristanC | matburt: yes, project configuration are merged | 14:35 |
| tristanC | matburt: i actually tried to document that in https://review.openstack.org/571420 | 14:36 |
| tristanC | e.g. if you look at org-config/zuul.d/pti.yaml and org-config/zuul.d/projects.yaml of http://logs.openstack.org/20/571420/3/check/tox-docs/79564ab/html/user/howtos/pti.html#projects-content | 14:36 |
| tristanC | you can see org-client has the org-pti template that is defining org-codestyle and org-integration-test for that project check pipeline | 14:37 |
| tristanC | then org-client can still define its own jobs, as shown in http://logs.openstack.org/20/571420/3/check/tox-docs/79564ab/html/user/howtos/pti.html#project-tests | 14:37 |
| tristanC | resulting in org-client check pipeline to run both the org-codestyle, org-integration-test and org-client-ui-validation job to run | 14:38 |
| tristanC | matburt: does that answer your question? | 14:38 |
| tristanC | note that job are uniques accross a tenant, there can only be one tox job defined in one place | 14:39 |
| pabelanger | yah, it is usually best to prefix your repo to the start of a name, if you want to create a new / variant job. So for tox, awx-tox. | 14:40 |
| pabelanger | but ideally, as more projects come into zuul, there is a PTI that is created so projects can share the same jobs | 14:41 |
| pabelanger | but understand in some cases, there are project specific things that need to be done | 14:41 |
| tristanC | pabelanger: i keep on linking that open review (https://review.openstack.org/571420) about all of that, any chance you give it a review please? | 14:43 |
| pabelanger | tristanC: yah, just seen that for first time, bookmarked. Will take a look for sure | 14:44 |
| pabelanger | remote: https://softwarefactory-project.io/r/13757 Bring online vexxhost-ansible-network-mtl1 | 14:51 |
| pabelanger | tristanC: mhu: fbo: nhicher: ^ do you mind reviewing, starting the process to rotate out the original provider (name too long) for mtl1 rename. Also starts to lay the ground work to bring online sjc1 too, which will have nested virt support | 14:52 |
| matburt | tristanC super helpful, thanks! | 15:01 |
| spredzy | tristanC: For Github, how does the Depends-On work ? | 15:06 |
| spredzy | Should we point to the URL of the PR ? | 15:06 |
| * spredzy not familiar with the Github/Zuul workflow yet | 15:07 | |
| pabelanger | Depends-On: https://github.com/ansible-network/ansible-zuul-jobs/pull/54 for example | 15:08 |
| pabelanger | https://github.com/ansible-network/yang/pull/25 | 15:08 |
| pabelanger | however, there is a difference in github vs gerrit | 15:08 |
| pabelanger | it github, you need to edit the first comment of the PR to update depends-on headers | 15:08 |
| pabelanger | with gerrit, you can do it via commit messages | 15:08 |
| pabelanger | not really a fan of github for that, but does work | 15:09 |
| spredzy | ack, so the Depends-On goes to the PR comment | 15:09 |
| matburt | gotcha, that's good to know | 15:09 |
| spredzy | Not really the commit message (if there are N commit in a single PR) | 15:09 |
| pabelanger | right, --ammend doesnt' even work there | 15:09 |
| spredzy | Gotcha | 15:10 |
| pabelanger | I haven't looked, but wonder if there would be a way to have zuul iterate over the comments of a PR and look for latest commit message | 15:10 |
| pabelanger | then use that | 15:10 |
| *** chkumar|ruck is now known as chkumar|off | 15:17 | |
| *** chmouel has left #softwarefactory | 15:25 | |
| *** sshnaidm|off has joined #softwarefactory | 16:02 | |
| *** sshnaidm|off has quit IRC | 16:06 | |
| *** sshnaidm has joined #softwarefactory | 16:06 | |
| *** zoli is now known as zoli|gone | 16:10 | |
| *** zoli|gone is now known as zoli | 16:10 | |
| pabelanger | remote: https://softwarefactory-project.io/r/13759 Refactory ansible-network pools for nodepool | 16:35 |
| pabelanger | nhicher: ^ if you don't mind reviewing | 16:36 |
| sfbender | Fabien Boucher created software-factory/sf-config master: Fix config/sf-jobs location path gen for external gerrit https://softwarefactory-project.io/r/13760 | 16:37 |
| *** jpena is now known as jpena|off | 17:29 | |
| *** sfbender has quit IRC | 18:53 | |
| pabelanger | nhicher: seeing timeout on ansible-network.sf.io | 18:58 |
| nhicher | pabelanger: we can't reach sf.io and review.rdo, rhos-ops are checking what is the issue | 18:59 |
| pabelanger | nhicher: k, thanks for update | 19:00 |
| matburt | oof | 19:45 |
| *** sfbender has joined #softwarefactory | 21:02 | |
| sfbender | Ken Dreyer created rdopkg master: doc: fix spelling of rdopkg https://softwarefactory-project.io/r/13767 | 21:02 |
| nhicher | pabelanger: sf is back | 21:20 |
| pabelanger | nhicher: thanks, any idea the issue? | 21:21 |
| nhicher | pabelanger: probably ovs | 21:22 |
| pabelanger | nhicher: yah | 21:22 |
| pabelanger | nhicher: is 0000191220 ready / locked by chance in nodepool? | 22:31 |
| nhicher | pabelanger: ready | 00:00:23:01 | unlocked | 22:48 |
| pabelanger | nhicher: I don't know why that is ready, since min-ready is 0 | 22:51 |
| pabelanger | let me try to cycle it | 22:51 |
| pabelanger | I am guessing there is a race some place | 22:53 |
| pabelanger | in nodepool | 22:53 |
| pabelanger | and know how to fix | 22:54 |
| pabelanger | will push up a patch in the morning | 22:54 |
| nhicher | pabelanger: ok, good evening =) | 22:58 |
| pabelanger | remote: https://softwarefactory-project.io/r/13768 Set max-ready-age for vexxhost-ansible-network | 22:59 |
| pabelanger | nhicher: for the morning^ | 22:59 |
| sfbender | Ken Dreyer created rdopkg master: tests: use pytest.parametrize in spec tests https://softwarefactory-project.io/r/13769 | 23:06 |
| sfbender | Ken Dreyer created rdopkg master: specfile: document release_parts() return value https://softwarefactory-project.io/r/13770 | 23:11 |
| sfbender | Ken Dreyer created rdopkg master: specfile: add comments to recognized_release() https://softwarefactory-project.io/r/13771 | 23:11 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!