| openstackgerrit | Merged openstack-infra/zuul master: docs: Add compile profile to bindep https://review.openstack.org/601459 | 01:21 |
|---|---|---|
| *** jiapei has quit IRC | 01:58 | |
| *** gouthamr has quit IRC | 02:18 | |
| *** dmellado has quit IRC | 02:18 | |
| *** jiapei has joined #zuul | 02:52 | |
| *** yolanda has joined #zuul | 03:07 | |
| *** threestrands has joined #zuul | 04:34 | |
| *** pcaruana has joined #zuul | 05:00 | |
| *** pcaruana has quit IRC | 05:09 | |
| *** yolanda has quit IRC | 05:11 | |
| *** nilashishc has joined #zuul | 06:15 | |
| openstackgerrit | Merged openstack-infra/zuul master: Need SSH client for Ansible https://review.openstack.org/596875 | 06:16 |
| *** smyers has quit IRC | 06:19 | |
| *** smyers has joined #zuul | 06:21 | |
| *** smyers has quit IRC | 06:51 | |
| *** hashar has joined #zuul | 06:57 | |
| *** pcaruana has joined #zuul | 07:13 | |
| *** gouthamr has joined #zuul | 07:13 | |
| *** dmellado has joined #zuul | 07:16 | |
| *** smyers has joined #zuul | 07:38 | |
| *** jpena|off is now known as jpena | 07:39 | |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: Cleanup of leaked resource for static driver https://review.openstack.org/600084 | 07:41 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: wip: Implement liveness probe for static nodes https://review.openstack.org/601513 | 07:41 |
| swest | Shrews: thanks for the comments. How about something like https://review.openstack.org/#/c/601513/ ? | 07:42 |
| *** eumel8 has joined #zuul | 07:47 | |
| *** jiapei has quit IRC | 08:08 | |
| *** threestrands has quit IRC | 08:30 | |
| openstackgerrit | Simon Westphahl proposed openstack-infra/nodepool master: wip: Implement liveness probe for static nodes https://review.openstack.org/601513 | 08:37 |
| tobiash | Shrews: the thing is that most of the static nodes are not under our control but are managed by projects. So we need a way to handle static nodes that disappear and return (e.g. for maintenance) | 08:45 |
| *** electrofelix has joined #zuul | 08:55 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/nodepool master: AWS driver fix to not explode on common images https://review.openstack.org/601361 | 10:39 |
| *** jpena is now known as jpena|lunch | 11:25 | |
| *** dkranz has quit IRC | 12:18 | |
| *** jpena|lunch is now known as jpena | 12:28 | |
| *** yolanda has joined #zuul | 12:30 | |
| *** elyezer has joined #zuul | 12:38 | |
| *** samccann has joined #zuul | 13:06 | |
| tobiash | pabelanger: we don't manage github labels but just require the label 'merge' in the gate pipeline to mimic the workflow label you use | 13:22 |
| pabelanger | tobiash: ack, same. Right now it is up to user to add right labels, but was thinking about either updating manage-project to support github labels, or another project to manage them | 13:23 |
| pabelanger | would like to store that into in yaml and do code review on it | 13:23 |
| *** yolanda has quit IRC | 13:29 | |
| pabelanger | pull request authors can't approve their own pull request :( | 13:30 |
| pabelanger | SpamapS: I think you ran into that before, but cannot remember how you solved it^ | 13:31 |
| *** yolanda has joined #zuul | 13:45 | |
| *** leifmadsen has quit IRC | 13:48 | |
| *** leifmadsen has joined #zuul | 13:50 | |
| clarkb | is it bad I immediately thought: everyone use two github accounts | 13:57 |
| pabelanger | yah, I'm not even sure if a github admin can toggle it | 13:58 |
| *** xbend has joined #zuul | 14:01 | |
| dmsimard | pabelanger: maybe jlk knows or knows someone that knows :p | 14:06 |
| *** yolanda has quit IRC | 14:11 | |
| *** luizbag has joined #zuul | 14:17 | |
| luizbag | hi, I have a CI setup with zuul that I want it to skip the job for a certain project if some files aren't changed | 14:18 |
| luizbag | is that even possible? | 14:18 |
| *** annabelleB has joined #zuul | 14:18 | |
| AJaeger | luizbag: yes, see https://zuul-ci.org/docs/zuul/user/config.html#attr-job.irrelevant-files | 14:19 |
| luizbag | I'm using version 2 | 14:19 |
| AJaeger | luizbag: my answer was for version 3;) | 14:19 |
| luizbag | I just can't find any documentation regarding this on zuul v2 | 14:20 |
| AJaeger | luizbag: yes, version 2 had a similar concept but I swapped out all of that ;( | 14:20 |
| AJaeger | luizbag: https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml?h=before-jenkins-config-removed&id=8643da6c1c246249d694fb7f7c24d51a7449df94 is our old v2 config | 14:21 |
| AJaeger | skip-if was the option there, see https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml?h=before-jenkins-config-removed&id=8643da6c1c246249d694fb7f7c24d51a7449df94#n1197 | 14:21 |
| luizbag | thanks, maybe I can use the all-files-match-any and use a ! on the regex | 14:24 |
| AJaeger | luizbag: or all-files-match-any on its own - check the lines further down | 14:25 |
| *** nilashis_ has joined #zuul | 14:25 | |
| *** dkranz has joined #zuul | 14:25 | |
| AJaeger | luizbag: ignore me, I misread - let me dig further | 14:26 |
| AJaeger | luizbag: there'S files keyword as well, see https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml?h=before-jenkins-config-removed&id=8643da6c1c246249d694fb7f7c24d51a7449df94#n1763 | 14:26 |
| AJaeger | so, you can say: Run job only if files A, B or C are changed. | 14:27 |
| luizbag | yes, I've seen it, but I can't filter it by project. or can I? | 14:27 |
| tobiash | pabelanger: you cannot solve it if you enforce a pull request review | 14:28 |
| *** nilashishc has quit IRC | 14:28 | |
| tobiash | github just doesn't support self approval | 14:28 |
| tobiash | but admins can be granted to ignore that and merge manually (which would bypass zuul) | 14:29 |
| pabelanger | right | 14:29 |
| pabelanger | don't want to bypass zuul | 14:29 |
| pabelanger | just, some projects only have a single user right now | 14:29 |
| clarkb | wow so two accounts is a solutuin | 14:29 |
| pabelanger | that, or drop code review :( | 14:29 |
| tobiash | clarkb: yes, our solution is my tobiash-evil-twin account ;) | 14:30 |
| Shrews | tobiash: re: the liveness check, i think you could override the checkReusableNode() api call to validate if it is still reachable. No need for a new thread. | 14:33 |
| Shrews | swest: ^^^ | 14:33 |
| Shrews | that should eliminate *any* attempts to use an unreachable node | 14:34 |
| tobiash | Shrews: thanks, we will check that | 14:41 |
| *** annabelleB has quit IRC | 14:43 | |
| *** pcaruana has quit IRC | 14:47 | |
| *** yolanda has joined #zuul | 14:49 | |
| openstackgerrit | Steve Kowalik proposed openstack-infra/zuul master: Correct show remaining time as unknown https://review.openstack.org/601462 | 15:12 |
| *** hashar is now known as hasharAway | 15:16 | |
| *** jiapei has joined #zuul | 15:22 | |
| *** nilashis_ has quit IRC | 15:30 | |
| *** nilashishc has joined #zuul | 15:30 | |
| *** nilashishc has quit IRC | 15:34 | |
| jlk | pabelanger: dmsimard: yeah, the not being able to do a pull request review on your own PR is kind of a lame thing. It makes using those for a very small project not very fun. | 15:40 |
| jlk | pabelanger: dmsimard: it's not something an admin can toggle, it's just built into the platform at this point. | 15:40 |
| dmsimard | sounds like an arbitrary limitation | 15:40 |
| dmsimard | but thanks for confirming at least | 15:41 |
| jlk | yeah, it's just a workflow that wasn't imagined w/ the pull request review setup. | 15:41 |
| jlk | You can admin toggle whether or not they're required for merge. | 15:42 |
| jlk | It just means that with Zuul you have to use a different signal to indicate approval. | 15:42 |
| openstackgerrit | Steve Kowalik proposed openstack-infra/zuul master: Correct show remaining time as unknown https://review.openstack.org/601462 | 15:44 |
| *** annabelleB has joined #zuul | 15:52 | |
| SpamapS | pabelanger: we solved it by making the gate requirement a label: "approved". Only admins can label things. Then we made a special set of pipelines that label/unlabel things that are approved by people with write access to the repo. | 15:54 |
| SpamapS | I think perhaps a better thing would be if zuul could express "or" in requirements. | 15:54 |
| * SpamapS has begun work on making the aws driver support nodepool-builder | 15:56 | |
| SpamapS | tired of waiting for docker to install on my stock images. Not wanting to build one-offs. | 15:56 |
| jlk | I need to get back to working on the checks API stuff | 15:58 |
| jlk | I got a bit... sidetracked by some PC gaming. | 15:59 |
| SpamapS | zomg yes | 16:00 |
| SpamapS | It would fit so nicely with zuul | 16:00 |
| pabelanger | SpamapS: thanks | 16:03 |
| jlk | I've got most of the github3.py work done, just have to write some tests to cover it. Then I can move on to the zuul work | 16:05 |
| dmsimard | oh yeah I saw the checks API, it sounds like a good fit for zuul | 16:05 |
| jlk | it's something I was trying to allude to in Dublin. | 16:06 |
| *** yolanda has quit IRC | 16:08 | |
| *** yolanda has joined #zuul | 16:10 | |
| corvus | fdegir: are you at the ptg? if so, can you find me in the project help room? | 16:15 |
| openstackgerrit | Steve Kowalik proposed openstack-infra/zuul master: Correct show remaining time as unknown https://review.openstack.org/601462 | 16:21 |
| fdegir | corvus: I am - walking there now | 16:24 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: ZFS: add ubuntu documentation https://review.openstack.org/601631 | 16:24 |
| *** nilashishc has joined #zuul | 16:33 | |
| *** sshnaidm has joined #zuul | 16:38 | |
| *** nilashishc has quit IRC | 16:55 | |
| odyssey4me | corvus: got a few mins to chat through some zuul api things? | 16:57 |
| corvus | odyssey4me: yep | 16:58 |
| *** nilashishc has joined #zuul | 17:00 | |
| *** nilashis_ has joined #zuul | 17:01 | |
| *** nilashishc has quit IRC | 17:01 | |
| *** nilashis_ has quit IRC | 17:05 | |
| *** nilashishc has joined #zuul | 17:05 | |
| *** jpena is now known as jpena|off | 17:09 | |
| *** nilashishc has quit IRC | 17:09 | |
| *** nilashishc has joined #zuul | 17:09 | |
| *** nilashis_ has joined #zuul | 17:12 | |
| *** sshnaidm has quit IRC | 17:14 | |
| *** yolanda has quit IRC | 17:14 | |
| *** nilashishc has quit IRC | 17:15 | |
| *** pcaruana has joined #zuul | 17:32 | |
| *** nilashis_ has quit IRC | 17:42 | |
| pabelanger | corvus: is it appropriate to have a discussion this week about moving zuul git repos into its own tenant? Or do is there still some work to do | 17:44 |
| *** annabelleB has quit IRC | 17:53 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: WIP Support building pdfs of sphinx docs https://review.openstack.org/570957 | 17:54 |
| *** annabelleB has joined #zuul | 17:57 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: WIP Support building pdfs of sphinx docs https://review.openstack.org/570957 | 18:00 |
| *** annabelleB has quit IRC | 18:01 | |
| corvus | pabelanger: yeah, i think that depends on what we decide for winterscale which we should discuss this week | 18:07 |
| fungi | https://superuser.openstack.org/articles/zuul-case-study-software-factory/ | 18:13 |
| fungi | that just got uploaded, i'm told | 18:14 |
| SpamapS | neaty | 18:17 |
| openstackgerrit | Steve Kowalik proposed openstack-infra/zuul master: Correct show remaining time as unknown https://review.openstack.org/601462 | 18:25 |
| dmsimard | corvus: what kind of approach did you have in mind for ara containers? using pbrx ? | 18:32 |
| pabelanger | corvus: ack | 18:40 |
| *** electrofelix has quit IRC | 18:42 | |
| *** mestery has joined #zuul | 19:14 | |
| *** hasharAway is now known as hashar | 19:40 | |
| *** pcaruana has quit IRC | 19:55 | |
| *** annabelleB has joined #zuul | 20:08 | |
| *** dkranz has quit IRC | 20:34 | |
| *** annabelleB has quit IRC | 20:34 | |
| *** annabelleB has joined #zuul | 20:40 | |
| *** luizbag has quit IRC | 20:41 | |
| *** samccann has quit IRC | 20:46 | |
| *** hashar has quit IRC | 20:50 | |
| openstackgerrit | Steve Kowalik proposed openstack-infra/zuul master: Correct show remaining time as unknown https://review.openstack.org/601462 | 21:01 |
| *** sshnaidm has joined #zuul | 21:34 | |
| *** annabelleB has quit IRC | 21:34 | |
| openstackgerrit | Steve Kowalik proposed openstack-infra/zuul master: Correct Last-Modified header for status API https://review.openstack.org/601832 | 21:55 |
| openstackgerrit | Merged openstack-infra/zuul master: Add 'build' method to API https://review.openstack.org/592225 | 22:04 |
| *** annabelleB has joined #zuul | 22:16 | |
| *** annabelleB has quit IRC | 22:46 | |
| *** sshnaidm has quit IRC | 23:31 | |
| *** sshnaidm has joined #zuul | 23:40 | |
| *** sshnaidm is now known as sshnaidm|afk | 23:40 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!