tristanC | gundalow: rcarrillocruz another solution is to start the ansible-network with a new config repository name, for example a-n/ci-config. | 01:24 |
---|---|---|
tristanC | rcarrillocruz: can do today for sure, i'll prepare a PR on a-n/z-c | 01:25 |
tristanC | gundalow: iiuc, awx, core and runner are the same org so they could share a single tenant | 01:31 |
*** matburt has quit IRC | 01:49 | |
tristanC | rcarrillocruz: gundalow: pabelanger: all the change has been proposed for https://tree.taiga.io/project/morucci-software-factory/us/1647 , please make sure they are merged in order. let me know when you want to proceed | 02:23 |
tristanC | https://ansible-network.softwarefactory-project.io/ is now correctly setup, the white label tenant returns 404 as expected | 02:35 |
*** openstack has joined #softwarefactory | 02:53 | |
*** mattclay has joined #softwarefactory | 04:23 | |
*** nchakrab has joined #softwarefactory | 06:11 | |
sfbender | Tristan de Cacqueray created software-factory/sf-config master: config-update: remove exec from local post hook https://softwarefactory-project.io/r/13525 | 06:31 |
sfbender | Tristan de Cacqueray created software-factory/sf-config master: grafana: support missing providers https://softwarefactory-project.io/r/13526 | 06:34 |
sfbender | Tristan de Cacqueray created software-factory/sf-config master: runC: add missing rsync and iproute package https://softwarefactory-project.io/r/13527 | 06:48 |
*** fbo has joined #softwarefactory | 07:13 | |
*** jpena|off is now known as jpena | 07:51 | |
*** sshnaidm|afk is now known as sshnaidm | 08:59 | |
*** gundalow has joined #softwarefactory | 09:38 | |
gundalow | tristanC: looks like I got disconnected ~7 hours ago, was there any more discussion? | 09:39 |
tristanC | gundalow: all the change has been proposed for https://tree.taiga.io/project/morucci-software-factory/us/1647 , please make sure they are merged in order. let me know when you want to proceed | 09:41 |
tristanC | gundalow: https://ansible-network.softwarefactory-project.io/ is now correctly setup, the white label tenant returns 404 as expected | 09:41 |
gundalow | rcarrillocruz: pabelanger You both happy with this? | 09:44 |
tristanC | gundalow: also, another solution is to start the ansible-network with a new config repository name, for example a-n/ci-config. | 09:44 |
tristanC | like that we can prepare the new config without interuption with the current ci setup | 09:45 |
*** nchakrab has quit IRC | 09:53 | |
rcarrillocruz | i don't think it's a big deal, we don't have such a big CI traffic | 10:05 |
rcarrillocruz | i'm swamped at the moment | 10:06 |
rcarrillocruz | would like to get pabelanger around , it's too early for him still | 10:06 |
tristanC | rcarrillocruz: it shouldn't take more than 30 minutes, i was hoping we can do that today, i have personal duty tomorrow evening | 10:09 |
tristanC | to be clear, we need to merge in that order: | 10:10 |
tristanC | - https://softwarefactory-project.io/r/13522 Add a tenant for ansible-network | 10:10 |
tristanC | - https://softwarefactory-project.io/r/13523 Remove ansible-network projects from ansible tenant | 10:11 |
tristanC | - https://github.com/ansible-network/zuul-config/pull/43 Import shared config from ansible/zuul-config | 10:11 |
tristanC | - https://softwarefactory-project.io/r/13524 Add the rest of ansible-network projects to the ansible-network tenant | 10:11 |
tristanC | at that point, CI is working again, then we can proceed with: | 10:11 |
tristanC | - https://github.com/ansible/zuul-config/pull/15 Remove ansible-network jobs | 10:11 |
tristanC | - https://github.com/ansible/community/pull/345 Update ansible-network zuul usage instruction | 10:12 |
rcarrillocruz | i'm getting too many redirects when logging to github on SF Gerrit | 10:13 |
rcarrillocruz | you can go ahead with the first four | 10:13 |
rcarrillocruz | i will liaise with pabelanger with the rest | 10:13 |
rcarrillocruz | thanks | 10:13 |
tristanC | rcarrillocruz: would you mind notifying ansible-network chan about ci maintaince/interruption? | 10:13 |
tristanC | first change is getting merged | 10:15 |
tristanC | well we could also check webhook and issue recheck if needed | 10:20 |
tristanC | ok, tenant is created: https://ansible-network.softwarefactory-project.io/zuul/projects.html | 10:22 |
tristanC | merging the project tenant re-assignment now | 10:24 |
gundalow | tristanC: I'll notify people that CI will be going down | 10:24 |
gundalow | tristanC: they have been notified | 10:25 |
tristanC | gundalow: once config-update ran https://softwarefactory-project.io/r/13523, could you merge https://github.com/ansible-network/zuul-config/pull/43 ? | 10:26 |
tristanC | well, after recheck succeed | 10:26 |
tristanC | gundalow: ansible-network CI will be offline in a couple of minutes | 10:28 |
gundalow | tristanC: https://github.com/ansible-network/zuul-config/pull/43#issuecomment-416905062 Issue with secret | 10:30 |
tristanC | ok i made a mistake in the second change, it's fixed now, zuul scheduler is reloading now | 10:38 |
tristanC | i forgot about https://softwarefactory-project.io/r/13528 Remove ansible-network shadow object from ansible tenant | 10:39 |
gundalow | no problem, that's why we have tests and give notice of downtime :) | 10:40 |
tristanC | btw, you can use the new React web interface i have been working on for zuul at this url: http://38.145.32.112/react-zuul/t/ansible-network/status | 10:41 |
tristanC | it has a notification bell at the top-right to show config errors | 10:41 |
tristanC | oh nevermind, it's missing a rewrite rule | 10:42 |
tristanC | anyway, tenant is now loaded: https://ansible-network.softwarefactory-project.io/zuul/projects.html | 10:43 |
tristanC | and ansible is cleaned: https://ansible.softwarefactory-project.io/zuul/projects.html | 10:43 |
tristanC | gundalow: zuul doesn't complain about https://github.com/ansible-network/zuul-config/pull/43, can you merge it please? | 10:44 |
gundalow | tristanC: I'm seeing `ansible/check — check status: failure` on #43 | 10:45 |
gundalow | and no comment from `recheck` | 10:45 |
tristanC | yes, that's the ansible tenant, it will be ansible-network/check for the new pipeline, in scheduler log there was no config error so i think that can be merged to re-enable pipelines and jobs | 10:46 |
gundalow | Cool, just wanted to be sure | 10:46 |
gundalow | tristanC: merged #43 | 10:47 |
tristanC | gundalow: and pipelines are configured! https://ansible-network.softwarefactory-project.io/zuul/status.html | 10:47 |
gundalow | \o/ | 10:49 |
tristanC | rechecking https://github.com/ansible-network/cloud_vpn/pull/35 seems correct | 10:50 |
tristanC | gundalow: similarly for https://github.com/ansible/zuul-config/pull/15 , can you approve it please when ansible/check is ok | 10:51 |
rcarrillocruz | Thx folks | 10:56 |
tristanC | ouch, Out of memory: Kill process 1244 (zuul-scheduler) score 730 or sacrifice child | 10:56 |
gundalow | tristanC: #15 merged | 10:58 |
tristanC | gundalow: thanks! then the last one is https://github.com/ansible/community/pull/345 | 10:59 |
tristanC | rcarrillocruz: gundalow: pabelanger: the migration seems completed on my side | 11:03 |
tristanC | rcarrillocruz: gundalow: any concern if we mention ansible-network in https://softwarefactory-project.io/etherpad/p/zuul-case-study ? L34 | 11:04 |
gundalow | tristanC: merged #345 | 11:05 |
tristanC | rcarrillocruz: gundalow: it's an interview for zuul-ci.org/users ^ | 11:08 |
tristanC | gundalow: thanks! | 11:08 |
gundalow | https://github.com/ansible-network/cloud_vpn/pull/35#issuecomment-416911060 `ERROR Project github.com/ansible-network/cisco_ios does not have the default branch master` | 11:09 |
gundalow | tristanC: I like the idea of adding Ansible on http://zuul-ci.org/users not sure if we need to tidy anything else up first | 11:11 |
tristanC | gundalow: rcarrillocruz: didn't the branch of ansible-network/cisco_ios got renamed from master to devel? | 11:12 |
gundalow | oh, yes https://github.com/ansible-network/cisco_ios/branches/all shows only `devel` | 11:14 |
tristanC | gundalow: https://github.com/ansible-network/cloud_vpn/pull/35#issuecomment-414717272 got run with master branch, so i think that error is normal | 11:16 |
gundalow | looks like rcarrillocruz renamed `master` to `devel` ~day ago | 11:17 |
gundalow | Going to grab some lunch, back shortly | 11:17 |
rcarrillocruz | tristanC: I don't mind, but providing we say ansible-network. Namely because we are testing ansible-network namespace, if we say generically Ansible we may give the impression that the whole Ansible project is tested with Zuul which is not (yet, I hope it will be some day) | 11:25 |
gundalow | +1 (too all points) | 11:37 |
*** jpena is now known as jpena|lunch | 11:37 | |
gundalow | Gating is working as well | 11:41 |
gundalow | tristanC: Thanks :) | 11:41 |
tristanC | gundalow: you're welcome :) | 11:42 |
tristanC | btw, the react zuul interface is fixed, here is what it looks like: http://38.145.32.112/react-zuul/t/ansible-network/job/ansible-role-tests-2.6-py3 | 11:42 |
gundalow | I believe https://tree.taiga.io/project/morucci-software-factory/task/1654 can be marked as complete as the corresponding PR has been merged | 11:42 |
tristanC | pabelanger: demo of config-errors, there is a 'bell' on the top-right of: http://38.145.32.112/react-zuul/t/rdoproject.org/job/devstack | 11:44 |
pabelanger | tristanC: config-errors look nice | 12:04 |
pabelanger | will get coffee and catch up on backscroll | 12:05 |
gundalow | pabelanger: Morning :) | 12:06 |
sfbender | Fabien Boucher created software-factory/sf-docs master: Add sentence to lower dep to vbox in the sandbox howto https://softwarefactory-project.io/r/13529 | 12:14 |
*** sshnaidm is now known as sshnaidm|afk | 12:28 | |
*** jpena|lunch is now known as jpena | 12:46 | |
*** matburt has joined #softwarefactory | 12:52 | |
*** sshnaidm|afk is now known as sshnaidm | 13:02 | |
sfbender | Merged www.softwarefactory-project.io master: 2018-32 summary https://softwarefactory-project.io/r/13402 | 13:08 |
*** sfbender has quit IRC | 13:39 | |
*** sfbender has joined #softwarefactory | 14:14 | |
sfbender | Merged scl/nodepool-distgit master: Add openshiftpods driver https://softwarefactory-project.io/r/13363 | 14:14 |
sfbender | Fabien Boucher created software-factory/sf-docs master: Fix a typo introduced during last correction fix https://softwarefactory-project.io/r/13530 | 15:23 |
*** trishnag has quit IRC | 16:23 | |
*** jpena is now known as jpena|off | 16:31 | |
*** trishnag has joined #softwarefactory | 16:56 | |
*** nhicher has joined #softwarefactory | 17:40 | |
matburt | tristanC: rcarrillocruz : about ready to submit the runner/awx initial config repo changes (after rebasing, I see yall split out the network tenant) I'm hoping yall can help me a bit on the nodepool though. there are quite a lot to choose from ... I see ansible-centos-7 there in the labels but glancing through the network project configuration I see this: | 18:56 |
matburt | https://github.com/ansible-network/ansible-zuul-jobs/blob/master/zuul.d/jobs.yaml#L15-L17 which seems super nice | 18:56 |
matburt | can you break down the types of nodes there are here and a basic rundown on that container-based fc27? | 18:57 |
matburt | (sorry for being dense, this is pretty new to me) | 18:57 |
pabelanger | matburt: I'd use ansible-centos-7 to start, that is a generic centos-7 node. | 19:44 |
pabelanger | the container jobs, is a little misleading, as IIRC, only used for simple jobs | 19:44 |
matburt | np, I did notice that the comment there mentioned it had more python versions | 19:44 |
pabelanger | we should create ansible-fedora-28, which is newer and likely want you need for dependencies | 19:44 |
matburt | sounds reasonable | 19:44 |
matburt | I signed up for the service in order to submit the change review, I did it using github and it pulled my personal email as my preferred email... I added my redhat email and have been waiting for the confirmation email for a little while | 19:45 |
matburt | is gerrit hooked up to an email service? | 19:48 |
matburt | also... I reckon I'm going to need to add a github app to my repos so zuul can interact with them | 19:50 |
matburt | I can see that we already have the zuul github app configured... just need to sort through this email address confirmation thing :/ | 19:59 |
rcarrillocruz | heya | 20:24 |
rcarrillocruz | yeah, ansible centos should work | 20:25 |
sfbender | Merged software-factory/sf-docs master: Fix a typo introduced during last correction fix https://softwarefactory-project.io/r/13530 | 22:31 |
sfbender | Merged software-factory/sf-config master: sfconfig: default config template add the webhook_token in the right section https://softwarefactory-project.io/r/12983 | 22:37 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!