Friday, 2016-10-28

*** krotscheck has quit IRC02:32
*** pleia2 has quit IRC02:34
*** pleia2 has joined #storyboard02:34
*** krotscheck has joined #storyboard02:35
*** jtomasek has joined #storyboard05:38
*** jtomasek has quit IRC05:49
*** jtomasek has joined #storyboard07:02
Zaramorning, storyboard!08:00
openstackgerritMichael Still proposed openstack-infra/python-storyboardclient: Remove o/c/local.py and o/c/log.py  https://review.openstack.org/39136408:04
Zara\o/08:12
*** bethwhite has joined #storyboard08:14
openstackgerritMerged openstack-infra/storyboard: Fix modification of private stories  https://review.openstack.org/39131008:19
Zarawe need another +1 on the 'remove local.py' patch above before we can merge it08:20
* persia keeps getting server unavaiable messages this morning :(08:21
Zara:(08:23
ZaraI'm not great with faces, but I'm fairly sure the author of that patch is the original author of the oslo local.py code, and these days regrets it and wants to remove all trace of it from the internet08:23
Zaraso even if it weren't a TC goal, I'd want to help there :)08:24
openstackgerritMichael Still proposed openstack-infra/storyboard: Remove o/c/local.py and o/c/log.py  https://review.openstack.org/39136908:24
openstackgerritMichael Still proposed openstack-infra/storyboard: Remove o/c/local.py and o/c/log.py  https://review.openstack.org/39136908:25
*** jtomasek has quit IRC08:38
persiaWas the intent of 371620 to have examples for common review, or to land them for consumers of the library?09:18
Zarathe latter, I think; they're meant to just serve as examples but I thought they'd be more useful as docs in the client itself than at a gerrit link09:20
Zarathough I've waved my arms around about them a few times, so maybe people aren't fussed09:21
Zara(sorry was afk and about to be afk again for a bit, will then look at second local.py patch)09:22
persiaIt is to allow us to step away from keyboards and/or disconnect devices that IRC bouncers exist :)09:26
Zara:) am connected once more09:59
Zarathanks for reviewing the local.py patches10:00
openstackgerritMerged openstack-infra/python-storyboardclient: Remove o/c/local.py and o/c/log.py  https://review.openstack.org/39136410:04
persiaSimple, obvious, self-contained patches are my favourite to review :)10:14
Zara:)10:14
* Zara is relocating to lunch/awards place10:14
Zaraoh, turns out it's on the same floor and I can't go in the room yet anyway10:19
Zarabest relocation ever10:20
Zaraplease can some kind soul(s) review https://review.openstack.org/#/c/373212/10:24
Zarathe button alignment upsets me every time I see it10:24
openstackgerritMerged openstack-infra/storyboard: Remove o/c/local.py and o/c/log.py  https://review.openstack.org/39136910:27
openstackgerritMerged openstack-infra/storyboard: Allow permissions to be set for teams on stories  https://review.openstack.org/36891710:35
openstackgerritMichael Still proposed openstack-infra/python-storyboardclient: Remove incubated uuidutils.  https://review.openstack.org/39144812:30
openstackgerritMichael Still proposed openstack-infra/storyboard: Remove unused oslo incubated fixtures, and deps  https://review.openstack.org/39145712:48
persiaI remember there being some storyboard-specific hacks to openstack-common at some point.  Did all those get cleaned away?12:51
SotKI think they are only in python-storyboardclient12:51
* persia may also have a faulty memory, in which case recall correction would be alternately appreciated12:51
Zarathe tests currently expect to import from openstack-common13:29
Zarahave left a note in review13:29
Zaraoh yeah, I want this logged for posterity:13:32
Zara00:02  * SotK is basicaly propping his laptop up on his hands, typgin is hard13:32
Zara^ SotK after half a bottle of wine last night.13:32
SotK:D13:32
SotKmy desk was really uncomfortable tbf13:32
openstackgerritMichael Still proposed openstack-infra/storyboard: Remove unused oslo incubated fixtures, and deps  https://review.openstack.org/39145713:34
openstackgerritAdam Coldrick proposed openstack-infra/storyboard: Add filters to the /v1/tasks/search endpoint  https://review.openstack.org/39098314:27
openstackgerritAdam Coldrick proposed openstack-infra/storyboard: Add filters to the /v1/stories/search endpoint  https://review.openstack.org/38999714:27
openstackgerritMerged openstack-infra/storyboard: Remove unused oslo incubated fixtures, and deps  https://review.openstack.org/39145715:15
SotKplease can someone review https://review.openstack.org/#/c/373467/2 ?15:40
SotKI need this patch in my life15:40
Zarahahahaha15:40
Zarabelieve it or not, he has consumed no wine today.15:41
SotKpedroalvarez: \o/ thanks15:45
SotKany chance of review on https://review.openstack.org/#/c/373212/ also please?15:48
pedroalvarezI wasn't sure about that one, but happy with it if you are  :)15:50
Zarayeah, I don't think it's perfect but I think it's nicer than what came before it15:51
ZaraI agree with adam's concern about it15:51
openstackgerritMerged openstack-infra/storyboard-webclient: Give certain lists a border  https://review.openstack.org/37346715:52
SotK\o/15:52
SotK~o~15:52
SotK\o/15:52
Zarahe's dancing b/c of the aircon in this room15:53
pedroalvarez:)16:05
pedroalvarezready for zuul to digest16:06
*** zigo has quit IRC16:40
*** fungi has quit IRC16:41
*** zigo has joined #storyboard16:45
*** fungi has joined #storyboard16:46
*** bethwhite has quit IRC16:48
*** alexismonville has joined #storyboard17:28
*** alexismonville has quit IRC17:32
* SotK feeds zuul19:37
openstackgerritMerged openstack-infra/storyboard-webclient: Put task notes in a box  https://review.openstack.org/37321219:38

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