Saturday, 2018-02-17

*** jtomasek has quit IRC07:08
*** jtomasek has joined #storyboard08:28
*** jtomasek has quit IRC08:57
*** jtomasek has joined #storyboard09:00
*** jtomasek has quit IRC09:36
openstackgerritMerged openstack-infra/python-storyboardclient master: build universal wheels  https://review.openstack.org/54544610:28
openstackgerritMerged openstack-infra/storyboard master: handle missing lanes arg when creating a board  https://review.openstack.org/54516810:31
*** jmccrory_ has joined #storyboard11:58
*** openstackgerrit has quit IRC12:00
*** jmccrory has quit IRC12:00
*** jmccrory_ is now known as jmccrory12:00
* SotK sends https://review.openstack.org/545594 in an attempt to fully fix the webclient publishing job12:40
persiaSotK: Doesn't that just run `git rev-parse --short HEAD` and use that value in the absence of a tag?12:53
persiaOh, nevermind.  The names confused me.12:54
persiaI now believe there to be a bug in "Get commits since tag" in version-from-git13:00
SotKI believe this to be the case too13:00
persiaI think that failed_when: needs to be not only on a nonzero exit code, but also on blank content.13:01
persiaBecause there is logic to function in the absence of a tag: it is just buggy.13:02
persiaSadly my ansible isn't sufficient to know the syntax, and looking at the docs isn't helping :(  I do think that fixing the bug is better than not using the role though.13:05
persia(but I don't want to complain in review, as the lack of publication is very annoying, and I don't know how to fix it the better way)13:05
SotKI'd agree if we actually used the output of the role for anything in the other parts of the job, but afaict we don't13:08
SotK(but yes, it would be good to fix it too)13:08
persiaDo you think "failed_when: false or none" would work?13:09
SotKI'm not sure13:11
SotKmy best guess based on my also somewhat lacking ansible would be `failed_when: false or commits_since_tag_output.stdout == ''`13:13
persiaIndeed.  Looking through the following jobs, it seems the version is entirely unused.13:15
* SotK should've probably checked that when creating the playbook in the first place rather than assuming based on others xD13:16
persiazuul is lovely at testing itself in many ways, but testing POST jobs is tricky to do before merging the results.13:21
*** openstackgerrit has joined #storyboard13:22
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Fix login when using python 3  https://review.openstack.org/54559613:22
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Fix login when using python 3  https://review.openstack.org/54559615:13
*** jtomasek has joined #storyboard17:12
*** jtomasek has quit IRC18:53

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