| matburt | tristanC: what do you mean by installing it on those repos? | 00:18 |
|---|---|---|
| matburt | ahh yes I get it | 00:18 |
| matburt | so right now it looks like someone on the ansible team already has an app, should we ditch that and use the software projects one? | 00:19 |
| *** sfbender has joined #softwarefactory | 00:21 | |
| sfbender | Merged software-factory/sf-web-assets-distgit master: Add patternfly xstatic https://softwarefactory-project.io/r/13540 | 00:21 |
| tristanC | matburt: we could setup the existing app on our zuul, but that's not ideal | 00:22 |
| matburt | absolutely... I'm not sure what this existing one is, maybe gundalow or rcarrillocruz knows | 00:23 |
| tristanC | matburt: well we would need to update the app setting to use our web hook and create a new github connection in sf's zuul with the private key | 00:23 |
| matburt | I think it's fine to use yalls app | 00:23 |
| sfbender | Tristan de Cacqueray created logreduce master: Add api skeleton https://softwarefactory-project.io/r/13541 | 01:20 |
| sfbender | Tristan de Cacqueray created logreduce master: Add model skeleton and report import method https://softwarefactory-project.io/r/13542 | 01:20 |
| sfbender | Tristan de Cacqueray created logreduce master: Add anomaly update method https://softwarefactory-project.io/r/13543 | 01:20 |
| sfbender | Tristan de Cacqueray created logreduce master: wip: Add GraphQL interface https://softwarefactory-project.io/r/13544 | 01:20 |
| sfbender | Merged logreduce master: Add api skeleton https://softwarefactory-project.io/r/13541 | 01:24 |
| sfbender | Merged logreduce master: Add model skeleton and report import method https://softwarefactory-project.io/r/13542 | 01:24 |
| sfbender | Merged logreduce master: Add anomaly update method https://softwarefactory-project.io/r/13543 | 01:27 |
| sfbender | Merged logreduce master: Add worker service to process user report https://softwarefactory-project.io/r/12931 | 01:29 |
| sfbender | Merged logreduce master: Add React web interface https://softwarefactory-project.io/r/13469 | 02:13 |
| sfbender | Fabien Boucher created www.softwarefactory-project.io master: Add Hands-on: Use Zuul hob library https://softwarefactory-project.io/r/13545 | 06:26 |
| gundalow | tristanC: matburt So for OpenStack we have "Zuul GitHub integration" for network and AWX we are using "Software Facor Zuul GitHub integration" | 07:12 |
| gundalow | `softwarefactory-project-zuul` | 07:13 |
| gundalow | matburt: so on https://github.com/apps/softwarefactory-project-zuul/installations/234854 add the GitHub repo you want to be Zuul managed, I don't believe you need to create a new GitHub connection | 07:15 |
| *** jpena|off is now known as jpena | 07:31 | |
| sfbender | Tristan de Cacqueray created www.softwarefactory-project.io master: Add blog React for python-dev https://softwarefactory-project.io/r/13547 | 08:40 |
| tristanC | gundalow: matburt: so, we would like to merge https://github.com/ansible/zuul-config/pull/16 so that we can remove ansible tenant projects from https://softwarefactory-project.io/r/13548 | 10:31 |
| tristanC | err, pull/17 | 10:35 |
| sfbender | Merged software-factory/sf-config master: hypervisor-openshift: use a service account token instead of password https://softwarefactory-project.io/r/13335 | 11:17 |
| gundalow | tristanC: hum, I would have expected some of #17 not be in there. ie `playbooks/config/check.yaml` I thought we'd want to just use the upstream (sf) job, rather than forking it in ansible/zuul-config | 11:25 |
| gundalow | I assume what you've done is right, just want to understand a bit more :) | 11:25 |
| *** jpena is now known as jpena|lunch | 11:28 | |
| tristanC | gundalow: sure, we don't have to do that, but, if we want to store projects information, e.g.: https://github.com/ansible/zuul-config/blob/9bb2c8751b6d571575401aecb2fe3d0fb702c5dc/resources/ansible.yaml | 11:29 |
| tristanC | gundalow: then we need a config-check/config-update job to apply them to the sf-project.io/zuul | 11:29 |
| gundalow | ah, OK | 11:29 |
| gundalow | I knew there was a reason :) | 11:30 |
| tristanC | it could be in another project though, but i think it's better to keep it like that | 11:30 |
| gundalow | tristanC: you have a -1 on the the PR still | 11:30 |
| tristanC | the whole PR content was automated, it's generated by the sfconfig command | 11:30 |
| tristanC | gundalow: it's https://github.com/ansible/zuul-config/pull/17 , 16 was a mistake | 11:31 |
| tristanC | gundalow: if we merge it, then we would merge https://softwarefactory-project.io/r/#/c/13548/ | 11:31 |
| tristanC | gundalow: but no hurry, this can wait | 11:32 |
| gundalow | tristanC: https://github.com/ansible/zuul-config/pull/17 has-1 from you | 11:32 |
| gundalow | Just want to check that your comments regarding `site_ansiblelogs` have been addressed | 11:32 |
| tristanC | gundalow: oh, my bad, didn't realize review stay when pr is updated. | 11:33 |
| gundalow | Ah, yup, slightly different in GH | 11:33 |
| gundalow | tristanC: #17 has been merged | 11:34 |
| tristanC | gundalow: alright, seems are looking correct, we may wait next week to merge https://softwarefactory-project.io/r/#/c/13548/ | 11:37 |
| tristanC | perfect, https://softwarefactory-project.io/logs/48/13548/3/check/config-check/31bf935/logs/build/main.yaml has been generated using https://github.com/ansible/zuul-config/blob/master/resources/ansible.yaml | 11:40 |
| matburt | tristanC: what were these changes for? Did we want to remove the ansible tenant? | 11:45 |
| tristanC | matburt: with these changes, you won't have to open a review on sf-project.io to add a project into zuul | 11:47 |
| matburt | ahh that is super nice | 11:47 |
| tristanC | gundalow: matburt: could you install the application on https://github.com/ansible/zuul-jobs please? | 11:50 |
| gundalow | matburt: Which repo does it need adding to? | 11:50 |
| matburt | probably zuul-jobs and ansible-runner | 11:51 |
| matburt | I, unfortunately, do not have github admin access to do this so I have to wait until someone who does comes along | 11:54 |
| gundalow | I've requested access, though will need to wait for someone with powers to approve it | 11:57 |
| tristanC | gundalow: here is the last step of ansible tenant setup, the config-check and config-update job: https://github.com/ansible/zuul-config/pull/18 | 12:18 |
| gundalow | Merged | 12:22 |
| *** jpena|lunch is now known as jpena | 12:23 | |
| tristanC | matburt: gundalow: demo: https://github.com/ansible/zuul-config/pull/19/files generated: https://ansible.softwarefactory-project.io/logs/19/19/7ded7524b01a4f212e8acbceb3b68081909627e3/check/config-check/2ada8f1/logs/build/main.yaml | 12:24 |
| matburt | nice! | 12:25 |
| gundalow | Woot, thanks tristanC | 12:27 |
| jruzicka | tristanC, great job on these guides! They're so good I feel like I should also add something useful eventually :) Especially as I can do so with files without web GUI, woohoo. | 12:27 |
| tristanC | matburt: gundalow: please ask for sf-ops review before merging resources change, we haven't validated the config-update job yet and we need to check zuul behaves as expected | 12:27 |
| tristanC | jruzicka: you're welcome, hope they help you :-) | 12:28 |
| jruzicka | tristanC, gonna try gertty as soon as the blog posts renders and react as soon as I have a good excuse to ;) | 12:29 |
| tristanC | gundalow: well, the job has a post failure, but it worked! https://ansible.softwarefactory-project.io/zuul/projects.html has awx | 12:34 |
| gundalow | Cool | 12:34 |
| *** EmilienM is now known as EvilienM | 12:57 | |
| tristanC | note that scheduler is a bit overloaded, lots of change happening from openstack doign release stuff | 13:25 |
| matburt | looks like we've got the github app added to our repos... is there some checkin process from the zuul side that needs to happen? it doesn't seem to think it's linked | 13:26 |
| matburt | or maybe that's related to the scheduler? | 13:26 |
| tristanC | matburt: it has a ansible/zuul-jobs change queued in status, see https://ansible.softwarefactory-project.io/zuul/status.html | 13:31 |
| matburt | ahh I see it now | 13:31 |
| matburt | I didn't at t he time, I need to be more patient ;) | 13:31 |
| tristanC | matburt: ftr, on the status page, you can see Queue lengths: 246 events is high | 13:36 |
| tristanC | matburt: and when click a change, if it doesn't expand into a job list, that means zuul is still waiting for a merge job | 13:37 |
| matburt | gotcha | 13:37 |
| matburt | I didn't notice that queue length, | 13:37 |
| tristanC | matburt: once it complete the merge job, it will load the zuul config, set the status and start the queue job execution | 13:37 |
| matburt | dig it... am I just seeing the status for the ansible tenant? | 13:37 |
| tristanC | matburt: the queue is global for all tenants | 13:38 |
| matburt | yeah, I see the tenant subdomain that I'm looking at | 13:38 |
| tristanC | matburt: you should be able to merge https://github.com/ansible/zuul-jobs/pull/4 by adding a approved review and setting a "mergeit" label | 13:55 |
| tristanC | (and let zuul do the gate and merge). otherwise feel free to manually merge it | 13:56 |
| tristanC | i'm leaving now, have a great week ends folks, talk to you next week! | 13:56 |
| matburt | tristanC: thanks for all your help this week! | 13:56 |
| jruzicka | jpena, what do you think about the driver conditionals? https://softwarefactory-project.io/r/#/c/13330/ | 14:33 |
| *** jpena is now known as jpena|off | 14:42 | |
| *** jpena|off is now known as jpena | 14:43 | |
| *** jpena is now known as jpena|off | 16:03 | |
| *** sshnaidm|off has quit IRC | 17:37 | |
| *** sshnaidm|off has joined #softwarefactory | 17:38 | |
| *** sshnaidm|off has quit IRC | 17:41 | |
| *** sshnaidm|off has joined #softwarefactory | 17:41 | |
| *** EvilienM is now known as EmilienM | 22:04 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!