| *** dmellado has joined #zuul | 00:33 | |
| *** rlandy|rover|bbl is now known as rlandy|rover | 03:03 | |
| *** JasonCL has quit IRC | 03:25 | |
| *** JasonCL has joined #zuul | 03:32 | |
| *** JasonCL has quit IRC | 03:37 | |
| *** JasonCL has joined #zuul | 03:40 | |
| *** JasonCL has quit IRC | 03:44 | |
| *** rlandy|rover has quit IRC | 03:50 | |
| *** rlandy has joined #zuul | 03:59 | |
| *** rlandy is now known as rlandy|rover | 03:59 | |
| *** JasonCL has joined #zuul | 04:00 | |
| *** rlandy|rover has quit IRC | 04:01 | |
| *** JasonCL has quit IRC | 04:09 | |
| *** JasonCL has joined #zuul | 04:57 | |
| *** JasonCL has quit IRC | 05:01 | |
| *** JasonCL has joined #zuul | 05:07 | |
| *** JasonCL has quit IRC | 05:12 | |
| *** openstackgerrit has joined #zuul | 05:35 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Add /label-list to the webapp https://review.openstack.org/535563 | 05:35 |
|---|---|---|
| tristanC | tobiash: "As of Ansible 2.4, the parameter 'executable' is no longer supported with the 'command' module. Not using '/bin/bash'." which makes the bindep role now fails, hence the retry_limit on https://review.openstack.org/545603 | 05:47 |
| *** JasonCL has joined #zuul | 05:53 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: bindep: use shell instead of command with executable https://review.openstack.org/546869 | 05:56 |
| *** JasonCL has quit IRC | 05:57 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: bindep: use shell instead of command with executable https://review.openstack.org/546869 | 06:30 |
| *** JasonCL has joined #zuul | 06:33 | |
| tristanC | hum, "command_register|succeed" changed in 2.4, now if the command is running with "failed_when: false", then the "|succeed" returns true, even if the command failed... | 06:37 |
| tristanC | see https://github.com/ansible/ansible/issues/28455#issuecomment-323601653 | 06:38 |
| *** JasonCL has quit IRC | 06:38 | |
| tobiash | tristanC: that makes sense ;) | 06:38 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: bindep: use shell instead of command with executable https://review.openstack.org/546869 | 06:41 |
| *** JasonCL has joined #zuul | 06:48 | |
| tobiash | tristanC: you may want to grep through zuul-jobs, there might be more of these patterns in there | 06:49 |
| *** JasonCL has quit IRC | 06:54 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: ara: check for return code instead of succeed https://review.openstack.org/546892 | 06:57 |
| *** JasonCL has joined #zuul | 07:00 | |
| *** JasonCL has quit IRC | 07:05 | |
| AJaeger | tristanC: and openstack-zuul-jobs and project-config - best to use codesearch.openstack.org | 07:10 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Add abstract attribute to base jobs https://review.openstack.org/545603 | 07:16 |
| *** hashar has joined #zuul | 07:16 | |
| AJaeger | tristanC, tobiash, removed abstract from tox ^ | 07:16 |
| tobiash | +2 | 07:18 |
| AJaeger | thanks! | 07:22 |
| tristanC | tobiash: AJaeger: the only other affected |succeeded occurence seems to be the ara role ( https://review.openstack.org/546892 ) | 07:25 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Replace sphinx_check_warning_is_error.py with ini lookup https://review.openstack.org/528812 | 07:25 |
| AJaeger | tristanC: cool! | 07:26 |
| tristanC | tobiash: it seems like the TMPDIR variable doesn't get to the template module when used on a remote host (re: https://review.openstack.org/546698 ) | 07:28 |
| tristanC | i mean, it still fails with "Accessing files from outside the working dir /tmp/xxxx" but works when changing the hosts to 'localhost' | 07:29 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Replace sphinx_check_warning_is_error.py with ini lookup https://review.openstack.org/528812 | 07:33 |
| *** JasonCL has joined #zuul | 07:52 | |
| *** JasonCL has quit IRC | 07:57 | |
| *** threestrands_ has joined #zuul | 08:15 | |
| *** threestrands_ has quit IRC | 08:15 | |
| *** threestrands_ has joined #zuul | 08:15 | |
| *** JasonCL has joined #zuul | 08:16 | |
| *** threestrands has quit IRC | 08:18 | |
| *** JasonCL has quit IRC | 08:21 | |
| *** electrofelix has joined #zuul | 08:40 | |
| tobiash | tristanC: hm, that use case could be more difficult to solve | 08:45 |
| *** JasonCL has joined #zuul | 08:49 | |
| *** jpena|off is now known as jpena | 08:50 | |
| *** JasonCL has quit IRC | 08:54 | |
| *** JasonCL has joined #zuul | 09:02 | |
| *** JasonCL has quit IRC | 09:07 | |
| *** markush1 has joined #zuul | 09:14 | |
| *** markush1 is now known as maho | 09:15 | |
| *** threestrands_ has quit IRC | 09:37 | |
| *** JasonCL has joined #zuul | 09:39 | |
| tristanC | tobiash: this fixes the issues: https://github.com/ansible/ansible/commit/8a2bda0e067, however it's only merged in 2.5.0b1 | 09:40 |
| tobiash | tristanC: ah cool | 09:41 |
| tristanC | actually this one https://github.com/ansible/ansible/commit/b20df1b , the other was just a refactor | 09:42 |
| AJaeger | can we ask for a backport for 2.4? | 09:44 |
| *** JasonCL has quit IRC | 09:44 | |
| tristanC | AJaeger: not sure how it works on github/ansible, but https://github.com/ansible/ansible/pull/36572 | 09:51 |
| *** JasonCL has joined #zuul | 09:52 | |
| *** JasonCL has quit IRC | 10:01 | |
| *** tosky has joined #zuul | 10:22 | |
| *** JasonCL has joined #zuul | 10:36 | |
| *** JasonCL has quit IRC | 10:40 | |
| *** maho has quit IRC | 10:45 | |
| *** JasonCL has joined #zuul | 10:48 | |
| *** JasonCL has quit IRC | 10:52 | |
| *** JasonCL has joined #zuul | 10:54 | |
| *** EmilienM has quit IRC | 10:59 | |
| *** EmilienM has joined #zuul | 11:00 | |
| *** JasonCL has quit IRC | 11:04 | |
| *** markush1 has joined #zuul | 11:07 | |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Allow using remote refs to find commits for change https://review.openstack.org/544964 | 11:16 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Allow using remote refs to find commits for change https://review.openstack.org/544964 | 11:18 |
| swest | tobiash: addressed your comments ^ | 11:19 |
| *** bramwelt1 has joined #zuul | 11:27 | |
| tobiash | swest: added a comment | 11:33 |
| *** JasonCL has joined #zuul | 11:33 | |
| *** bramwelt has quit IRC | 11:34 | |
| *** rbergeron has quit IRC | 11:34 | |
| *** rbergeron has joined #zuul | 11:35 | |
| *** rbergero1 has joined #zuul | 11:41 | |
| *** JasonCL has quit IRC | 11:41 | |
| *** rbergeron has quit IRC | 11:46 | |
| *** jpena is now known as jpena|lunch | 11:57 | |
| *** markush1 has quit IRC | 11:59 | |
| openstackgerrit | Hoang Trung Hieu proposed openstack-infra/zuul-jobs master: Update and replace http with https for doc links https://review.openstack.org/546965 | 12:26 |
| openstackgerrit | Hoang Trung Hieu proposed openstack-infra/zuul master: Update and replace http with https for doc links https://review.openstack.org/546970 | 12:37 |
| *** maxamillion has quit IRC | 12:52 | |
| *** maxamillion has joined #zuul | 12:55 | |
| *** electrofelix has quit IRC | 13:14 | |
| *** zhuli has quit IRC | 13:16 | |
| *** zhuli has joined #zuul | 13:16 | |
| *** maxamillion has quit IRC | 13:17 | |
| *** maxamillion has joined #zuul | 13:20 | |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Allow using remote refs to find commits for change https://review.openstack.org/544964 | 13:22 |
| *** jpena|lunch is now known as jpena | 13:27 | |
| *** salbacutea has joined #zuul | 13:34 | |
| *** rlandy has joined #zuul | 13:36 | |
| *** rlandy is now known as rlandy|ruck | 13:36 | |
| salbacutea | ださい。 | 13:37 |
| *** sshnaidm|afk is now known as sshnaidm | 13:37 | |
| *** JasonCL has joined #zuul | 13:38 | |
| *** salbacutea has quit IRC | 13:41 | |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Allow using remote refs to find commits for change https://review.openstack.org/544964 | 13:42 |
| *** JasonCL has quit IRC | 13:42 | |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Allow using remote refs to find commits for change https://review.openstack.org/544964 | 13:43 |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Allow using remote refs to find commits for change https://review.openstack.org/544964 | 13:44 |
| *** JasonCL has joined #zuul | 13:47 | |
| *** JasonCL has quit IRC | 13:48 | |
| *** JasonCL has joined #zuul | 13:48 | |
| *** JasonCL_ has joined #zuul | 13:53 | |
| dmsimard | corvus: I remember you mentioned there was a patch in flight to allow zuul-scheduler exceptions bubble up to zuul-web but I can't find it, do you know which one it is ? | 13:54 |
| *** mwhahaha has quit IRC | 13:54 | |
| *** mwhahaha has joined #zuul | 13:56 | |
| *** JasonCL has quit IRC | 13:56 | |
| dmsimard | SpamapS: for https://review.openstack.org/#/c/536391/, I was wondering if that was really necessary ? Considering https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/notification/slack.py and https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/callback/slack.py | 13:56 |
| *** robcresswell has quit IRC | 13:57 | |
| dmsimard | added a comment in the patch :) | 13:59 |
| *** robcresswell has joined #zuul | 14:00 | |
| *** bstinson has quit IRC | 14:01 | |
| *** bstinson has joined #zuul | 14:03 | |
| *** myoung|afk is now known as myoung | 14:23 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: Add additional builder debug logging https://review.openstack.org/546303 | 14:28 |
| openstackgerrit | Merged openstack-infra/nodepool master: Add /label-list to the webapp https://review.openstack.org/535563 | 14:30 |
| openstackgerrit | Hoang Trung Hieu proposed openstack-infra/zuul master: WIP: Update and replace http with https for doc links https://review.openstack.org/546970 | 14:38 |
| *** cmurphy has quit IRC | 14:57 | |
| *** dkranz has joined #zuul | 15:18 | |
| *** maxamillion has quit IRC | 15:21 | |
| *** maxamillion has joined #zuul | 15:21 | |
| kklimonda | can I use do a cross-repo depends-on to load new zuul job configuration? It doesn't seem to be picking up, but perhaps there are some rules about that | 15:40 |
| pabelanger | should be fine if they are untrusted projects | 15:44 |
| *** eventingmonkey has quit IRC | 15:58 | |
| *** eventingmonkey has joined #zuul | 16:09 | |
| *** sshnaidm is now known as sshnaidm|off | 16:12 | |
| *** sshnaidm|off has quit IRC | 16:17 | |
| *** JasonCL_ has quit IRC | 16:22 | |
| *** yolanda has quit IRC | 16:30 | |
| *** JasonCL has joined #zuul | 16:30 | |
| *** yolanda has joined #zuul | 16:30 | |
| *** tosky has quit IRC | 16:31 | |
| *** JasonCL has quit IRC | 16:36 | |
| *** JasonCL has joined #zuul | 16:39 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 16:46 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 16:48 |
| *** cmurphy has joined #zuul | 16:48 | |
| tobiash | dmsimard: e.g. merge failure reporting is only possible via a real reporter as it's reported before the job even runs | 16:50 |
| dmsimard | tobiash: hmmm, I guess | 16:51 |
| tobiash | so I'd consider a slack/mattermost/hipchat reporter useful :) | 16:51 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Refactor status functions, add web endpoints, allow params https://review.openstack.org/536301 | 16:52 |
| dmsimard | something something when you have a hammer everything looks like nails | 16:52 |
| dmsimard | ansible does that for me | 16:52 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 16:53 |
| SpamapS | dmsimard: It's necessary when you want to trigger on Slack too. | 16:57 |
| SpamapS | dmsimard: that's the first step. ;) | 16:57 |
| dmsimard | Oh, you mean like get Slack to do something | 16:58 |
| dmsimard | ? | 16:58 |
| SpamapS | No | 16:58 |
| SpamapS | like, have slack tell zuul to do something | 16:58 |
| SpamapS | @ZuulBot Run kolla-ansible-deploy branch=prod-zone-4 | 16:58 |
| corvus | ya, the new driver interface is designed to support that kind of thing -- triggers from any trigger-supporting driver enqueing items from any other source-supporting driver | 16:59 |
| tobiash | chatops :) | 17:00 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Refactor status functions, add web endpoints, allow params https://review.openstack.org/536301 | 17:00 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Add separate modules for management commands https://review.openstack.org/536303 | 17:00 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: webapp: add optional admin endpoint https://review.openstack.org/536319 | 17:00 |
| dmsimard | SpamapS: neat. I'm not on the chatop bandwagon yet but I hear it's fashionable :p | 17:01 |
| SpamapS | TBH, I don't love the chatops pattern. | 17:01 |
| SpamapS | But we do it here at GoDaddy | 17:01 |
| SpamapS | I'd rather have a git repo that gets tagged for deploys. | 17:01 |
| SpamapS | or branches per zone and deploy on merge/tag | 17:01 |
| SpamapS | But there are times where like, you need to re-run a piece of the automation due to some external issue. | 17:02 |
| SpamapS | (I like to have a git repo with a bunch of null commit messages about that though... so I don't have to look at the slack backscroll and git when trying to figure out what changed when) | 17:02 |
| SpamapS | err.. a git repo with a bunch of null commits and good commit messages, is what I meant | 17:02 |
| Shrews | fwiw, i have recently rediscovered this week how nearly impossible it is to work on a computer when you can neither stand nor sit due to an injured lower back | 17:10 |
| clarkb | Shrews: :( but also maybe thats an excuse to try out vr office? | 17:11 |
| Shrews | so thankful i don't have to sit on an airplane this weekend. | 17:11 |
| Shrews | clarkb: is that a thing? | 17:11 |
| clarkb | Shrews: ya some companies are working out how to make VR a productivity tool | 17:12 |
| clarkb | basically instead of sitting in front of a monitor you put on headset and get a 360 degree computer window pane | 17:12 |
| clarkb | I have no idea if it works well or not | 17:12 |
| fungi | Shrews: not that it's something you likely want to hear, but christine has spent the last two years working half the day from a laptop while lying in bed due to only being able to stand or sit for limited periods of time | 17:27 |
| fungi | and yeah, it has made her give up a lot of travel opportunities due to an inability to sit on airplanes for more than a few hours | 17:27 |
| fungi | when we take road trips we can only go about 4 hours a day | 17:28 |
| fungi | back injuries are terrible | 17:28 |
| Shrews | fungi: ouch. that's one reason why i try to avoid flights more than a few hours too | 17:28 |
| Shrews | going to the US west coast is about my limit any more | 17:30 |
| clarkb | Shrews: the good news is we are pretty awesome out here | 17:31 |
| fungi | yeah, she wants to come to vancouver with me, but is unsure she'll feel up to even that long of a flight by then | 17:31 |
| fungi | hopefully you'll heal faster | 17:32 |
| Shrews | clarkb: tru dat | 17:32 |
| openstackgerrit | Merged openstack-infra/nodepool master: Refactor playbooks/nodepool-zuul-functional/pre.yaml https://review.openstack.org/546272 | 17:34 |
| *** jpena is now known as jpena|off | 17:52 | |
| *** bramwelt1 is now known as bramwelt | 18:13 | |
| *** bramwelt has joined #zuul | 18:14 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 18:26 |
| *** openstackgerrit has quit IRC | 18:33 | |
| pabelanger | just thinking out loud, I think it would be helpful to have zuul fail a config, if a label doesn't exist in nodepool. IIRC, it doesn't today | 18:34 |
| pabelanger | mostly because, as I am thinking about how we are going to remove fedora-26 images from nodepool, I am not sure how to validate all the zuul.yaml files have been properly updated to fedora-27 | 18:34 |
| pabelanger | I'm using codesearch.o.o today, but that doesn't help with stable branches | 18:35 |
| clarkb | grep! | 18:35 |
| clarkb | its scriptable but ya painful | 18:35 |
| fungi | lack of multi-branch indexing in hound is the main reason i still sync clones of all our gerrit repos to my local workstation | 18:36 |
| pabelanger | yah, it was just helpful to have zuul error when we tried to remove the tripleo-check pipeline, and say which zuul.yaml files still referenced it. Having that for nodes, would be great | 18:37 |
| clarkb | fungi: ya Ifind myself just using local grep for reasons likethat | 18:38 |
| pabelanger | fungi: I think dmsimard said he is running a patch to support multi-branch, maybe we should consider applying it. | 18:38 |
| fungi | i thought his patch just allowed changing the indexed branch for cases where projects are doing development in a branch other than master | 18:38 |
| fungi | rather than indexing multiple branches per repo | 18:39 |
| pabelanger | oh, maybe that was it | 18:39 |
| fungi | but i could have misunderstood | 18:39 |
| pabelanger | zuul does return a node_failure, if a job tries to run a label that doesn't exist. I guess maybe over communicate the removal and work with projects to switch | 18:42 |
| AJaeger | pabelanger: my heuristic: Use codesearch and check for those repos you found all branches... | 18:44 |
| clarkb | should just put everything on tumbleweed now to test new shinies >_> | 18:44 |
| corvus | pabelanger: label availability can't be a config-time check. a label can become unavailable on its own with no changes to zuul config. | 18:53 |
| corvus | this is probably a better match for the dashboard. have it be able to return jobs by nodeset | 18:54 |
| corvus | pabelanger: you also may want to consider doing things like defining a nodeset called "latest-fedora" and have jobs use that. then you just change the nodeset def. | 18:55 |
| corvus | (that's not something to do for all kinds of jobs since, for example, stable branch variants may need to be pinned to old versions of distros, but for ones where you know you only ever want to test on the latest that may be helpful) | 18:56 |
| clarkb | for stable it solves the problem of fedora releases being supported less time than our stable branches are supported | 18:57 |
| clarkb | so may not be the worst thing there, though jobs on stable unlikely to keep working with a fedora update | 18:57 |
| pabelanger | yah, we could try latest-fedora / fedora-latest | 18:58 |
| dmsimard | pabelanger, fungi: right -- hound only ever indexes one branch (hardcoded to master by default) | 19:32 |
| dmsimard | My fork just allows to configure that one branch per repo | 19:32 |
| dmsimard | We could in theory stand up more than one hound (one per branch?) with that strategy I guess | 19:33 |
| fungi | yeah, that would work, though kinda hacky | 19:34 |
| dmsimard | master.codesearch.o.o == codesearch.o.o and then have queens.codesearch, pike.codesearch.. I dunno, don't have any ideas :D | 19:35 |
| *** JasonCL has quit IRC | 19:59 | |
| *** openstackgerrit has joined #zuul | 20:31 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Refactor status functions, add web endpoints, allow params https://review.openstack.org/536301 | 20:31 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Add separate modules for management commands https://review.openstack.org/536303 | 20:31 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: webapp: add optional admin endpoint https://review.openstack.org/536319 | 20:32 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul-jobs master: role: Inject public keys in case of failure https://review.openstack.org/535803 | 20:42 |
| *** JasonCL has joined #zuul | 20:47 | |
| *** JasonCL has quit IRC | 20:49 | |
| *** JasonCL has joined #zuul | 20:53 | |
| *** JasonCL has quit IRC | 20:55 | |
| *** JasonCL has joined #zuul | 20:59 | |
| *** JasonCL has quit IRC | 21:01 | |
| *** JasonCL has joined #zuul | 21:01 | |
| *** JasonCL has quit IRC | 21:05 | |
| *** hashar has quit IRC | 21:07 | |
| *** JasonCL has joined #zuul | 21:20 | |
| *** JasonCL_ has joined #zuul | 21:27 | |
| *** JasonCL has quit IRC | 21:29 | |
| *** JasonCL_ has quit IRC | 21:32 | |
| *** JasonCL has joined #zuul | 21:32 | |
| *** JasonCL has quit IRC | 21:35 | |
| *** JasonCL has joined #zuul | 21:39 | |
| *** JasonCL has quit IRC | 21:46 | |
| *** JasonCL has joined #zuul | 21:47 | |
| *** JasonCL has quit IRC | 21:51 | |
| *** JasonCL has joined #zuul | 21:51 | |
| *** JasonCL has quit IRC | 22:04 | |
| *** JasonCL has joined #zuul | 22:06 | |
| *** JasonCL has quit IRC | 22:07 | |
| *** JasonCL has joined #zuul | 22:08 | |
| *** JasonCL has quit IRC | 22:14 | |
| *** rlandy|ruck is now known as rlandy|biab | 22:21 | |
| *** JasonCL has joined #zuul | 22:34 | |
| *** JasonCL has quit IRC | 22:38 | |
| *** sshnaidm has joined #zuul | 22:40 | |
| *** sshnaidm is now known as sshnaidm|off | 22:40 | |
| *** JasonCL has joined #zuul | 22:42 | |
| *** JasonCL has quit IRC | 22:43 | |
| *** JasonCL has joined #zuul | 22:59 | |
| *** JasonCL has quit IRC | 23:00 | |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul master: Add debug information when failing to autohold https://review.openstack.org/547160 | 23:00 |
| *** JasonCL has joined #zuul | 23:02 | |
| *** JasonCL has quit IRC | 23:04 | |
| *** rlandy|biab is now known as rlandy|ruck | 23:14 | |
| *** JasonCL has joined #zuul | 23:15 | |
| *** JasonCL has quit IRC | 23:22 | |
| *** rlandy|ruck is now known as rlandy|bbl | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!