Wednesday, 2018-08-29

tristanCgundalow: rcarrillocruz another solution is to start the ansible-network with a new config repository name, for example a-n/ci-config.01:24
tristanCrcarrillocruz: can do today for sure, i'll prepare a PR on a-n/z-c01:25
tristanCgundalow: iiuc, awx, core and runner are the same org so they could share a single tenant01:31
*** matburt has quit IRC01:49
tristanCrcarrillocruz: 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 proceed02:23
tristanChttps://ansible-network.softwarefactory-project.io/ is now correctly setup, the white label tenant returns 404 as expected02:35
*** openstack has joined #softwarefactory02:53
*** mattclay has joined #softwarefactory04:23
*** nchakrab has joined #softwarefactory06:11
sfbenderTristan de Cacqueray created software-factory/sf-config master: config-update: remove exec from local post hook  https://softwarefactory-project.io/r/1352506:31
sfbenderTristan de Cacqueray created software-factory/sf-config master: grafana: support missing providers  https://softwarefactory-project.io/r/1352606:34
sfbenderTristan de Cacqueray created software-factory/sf-config master: runC: add missing rsync and iproute package  https://softwarefactory-project.io/r/1352706:48
*** fbo has joined #softwarefactory07:13
*** jpena|off is now known as jpena07:51
*** sshnaidm|afk is now known as sshnaidm08:59
*** gundalow has joined #softwarefactory09:38
gundalowtristanC: looks like I got disconnected ~7 hours ago, was there any more discussion?09:39
tristanCgundalow: 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 proceed09:41
tristanCgundalow: https://ansible-network.softwarefactory-project.io/ is now correctly setup, the white label tenant returns 404 as expected09:41
gundalowrcarrillocruz: pabelanger You both happy with this?09:44
tristanCgundalow: also, another solution is to start the ansible-network with a new config repository name, for example a-n/ci-config.09:44
tristanClike that we can prepare the new config without interuption with the current ci setup09:45
*** nchakrab has quit IRC09:53
rcarrillocruzi don't think it's a big deal, we don't have such a big CI traffic10:05
rcarrillocruzi'm swamped at the moment10:06
rcarrillocruzwould like to get pabelanger around , it's too early for him still10:06
tristanCrcarrillocruz: it shouldn't take more than 30 minutes, i was hoping we can do that today, i have personal duty tomorrow evening10:09
tristanCto be clear, we need to merge in that order:10:10
tristanC- https://softwarefactory-project.io/r/13522 Add a tenant for ansible-network10:10
tristanC- https://softwarefactory-project.io/r/13523 Remove ansible-network projects from ansible tenant10:11
tristanC- https://github.com/ansible-network/zuul-config/pull/43 Import shared config from ansible/zuul-config10:11
tristanC- https://softwarefactory-project.io/r/13524 Add the rest of ansible-network projects to the ansible-network tenant10:11
tristanCat 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 jobs10:11
tristanC- https://github.com/ansible/community/pull/345 Update ansible-network zuul usage instruction10:12
rcarrillocruzi'm getting too many redirects when logging to github on SF Gerrit10:13
rcarrillocruzyou can go ahead with the first four10:13
rcarrillocruzi will liaise with pabelanger with the rest10:13
rcarrillocruzthanks10:13
tristanCrcarrillocruz: would you mind notifying ansible-network chan about ci maintaince/interruption?10:13
tristanCfirst change is getting merged10:15
tristanCwell we could also check webhook and issue recheck if needed10:20
tristanCok, tenant is created: https://ansible-network.softwarefactory-project.io/zuul/projects.html10:22
tristanCmerging the project tenant re-assignment now10:24
gundalowtristanC: I'll notify people that CI will be going down10:24
gundalowtristanC: they have been notified10:25
tristanCgundalow: once config-update ran https://softwarefactory-project.io/r/13523, could you merge https://github.com/ansible-network/zuul-config/pull/43 ?10:26
tristanCwell, after recheck succeed10:26
tristanCgundalow: ansible-network CI will be offline in a couple of minutes10:28
gundalowtristanC: https://github.com/ansible-network/zuul-config/pull/43#issuecomment-416905062 Issue with secret10:30
tristanCok i made a mistake in the second change, it's fixed now, zuul scheduler is reloading now10:38
tristanCi forgot about https://softwarefactory-project.io/r/13528 Remove ansible-network shadow object from ansible tenant10:39
gundalowno problem, that's why we have tests and give notice of downtime :)10:40
tristanCbtw, 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/status10:41
tristanCit has a notification bell at the top-right to show config errors10:41
tristanCoh nevermind, it's missing a rewrite rule10:42
tristanCanyway, tenant is now loaded: https://ansible-network.softwarefactory-project.io/zuul/projects.html10:43
tristanCand ansible is cleaned: https://ansible.softwarefactory-project.io/zuul/projects.html10:43
tristanCgundalow: zuul doesn't complain about https://github.com/ansible-network/zuul-config/pull/43, can you merge it please?10:44
gundalowtristanC: I'm seeing `ansible/check — check status: failure` on #4310:45
gundalowand no comment from `recheck`10:45
tristanCyes, 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 jobs10:46
gundalowCool, just wanted to be sure10:46
gundalowtristanC: merged #4310:47
tristanCgundalow: and pipelines are configured! https://ansible-network.softwarefactory-project.io/zuul/status.html10:47
gundalow\o/10:49
tristanCrechecking https://github.com/ansible-network/cloud_vpn/pull/35 seems correct10:50
tristanCgundalow: similarly for https://github.com/ansible/zuul-config/pull/15 , can you approve it please when ansible/check is ok10:51
rcarrillocruzThx folks10:56
tristanCouch, Out of memory: Kill process 1244 (zuul-scheduler) score 730 or sacrifice child10:56
gundalowtristanC: #15 merged10:58
tristanCgundalow: thanks! then the last one is https://github.com/ansible/community/pull/34510:59
tristanCrcarrillocruz: gundalow: pabelanger: the migration seems completed on my side11:03
tristanCrcarrillocruz: gundalow: any concern if we mention ansible-network in https://softwarefactory-project.io/etherpad/p/zuul-case-study ? L3411:04
gundalowtristanC: merged #34511:05
tristanCrcarrillocruz: gundalow: it's an interview for zuul-ci.org/users ^11:08
tristanCgundalow: thanks!11:08
gundalowhttps://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
gundalowtristanC: I like the idea of adding Ansible on http://zuul-ci.org/users not sure if we need to tidy anything else up first11:11
tristanCgundalow: rcarrillocruz: didn't the branch of ansible-network/cisco_ios got renamed from master to devel?11:12
gundalowoh, yes https://github.com/ansible-network/cisco_ios/branches/all shows only `devel`11:14
tristanCgundalow: https://github.com/ansible-network/cloud_vpn/pull/35#issuecomment-414717272 got run with master branch, so i think that error is normal11:16
gundalowlooks like rcarrillocruz renamed `master` to `devel` ~day ago11:17
gundalowGoing to grab some lunch, back shortly11:17
rcarrillocruztristanC: 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|lunch11:37
gundalowGating is working as well11:41
gundalowtristanC: Thanks :)11:41
tristanCgundalow: you're welcome :)11:42
tristanCbtw, 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-py311:42
gundalowI believe https://tree.taiga.io/project/morucci-software-factory/task/1654 can be marked as complete as the corresponding PR has been merged11:42
tristanCpabelanger: demo of config-errors, there is a 'bell' on the top-right of: http://38.145.32.112/react-zuul/t/rdoproject.org/job/devstack11:44
pabelangertristanC: config-errors look nice12:04
pabelangerwill get coffee and catch up on backscroll12:05
gundalowpabelanger: Morning :)12:06
sfbenderFabien Boucher created software-factory/sf-docs master: Add sentence to lower dep to vbox in the sandbox howto  https://softwarefactory-project.io/r/1352912:14
*** sshnaidm is now known as sshnaidm|afk12:28
*** jpena|lunch is now known as jpena12:46
*** matburt has joined #softwarefactory12:52
*** sshnaidm|afk is now known as sshnaidm13:02
sfbenderMerged www.softwarefactory-project.io master: 2018-32 summary  https://softwarefactory-project.io/r/1340213:08
*** sfbender has quit IRC13:39
*** sfbender has joined #softwarefactory14:14
sfbenderMerged scl/nodepool-distgit master: Add openshiftpods driver  https://softwarefactory-project.io/r/1336314:14
sfbenderFabien Boucher created software-factory/sf-docs master: Fix a typo introduced during last correction fix  https://softwarefactory-project.io/r/1353015:23
*** trishnag has quit IRC16:23
*** jpena is now known as jpena|off16:31
*** trishnag has joined #softwarefactory16:56
*** nhicher has joined #softwarefactory17:40
matburttristanC: 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
matburthttps://github.com/ansible-network/ansible-zuul-jobs/blob/master/zuul.d/jobs.yaml#L15-L17 which seems super nice18:56
matburtcan 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
pabelangermatburt: I'd use ansible-centos-7 to start, that is a generic centos-7 node.19:44
pabelangerthe container jobs, is a little misleading, as IIRC, only used for simple jobs19:44
matburtnp, I did notice that the comment there mentioned it had more python versions19:44
pabelangerwe should create ansible-fedora-28, which is newer and likely want you need for dependencies19:44
matburtsounds reasonable19:44
matburtI 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 while19:45
matburtis gerrit hooked up to an email service?19:48
matburtalso... I reckon I'm going to need to add a github app to my repos so zuul can interact with them19:50
matburtI can see that we already have the zuul github app configured... just need to sort through this email address confirmation thing :/19:59
rcarrillocruzheya20:24
rcarrillocruzyeah, ansible centos should work20:25
sfbenderMerged software-factory/sf-docs master: Fix a typo introduced during last correction fix  https://softwarefactory-project.io/r/1353022:31
sfbenderMerged software-factory/sf-config master: sfconfig: default config template add the webhook_token in the right section  https://softwarefactory-project.io/r/1298322:37

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!