Friday, 2018-08-31

*** jamesmcarthur has joined #storyboard02:23
*** jamesmcarthur has quit IRC02:27
*** jamesmcarthur has joined #storyboard02:32
*** jamesmcarthur has quit IRC02:36
*** udesale has joined #storyboard03:59
*** jamesmcarthur has joined #storyboard04:46
*** jamesmcarthur has quit IRC04:50
*** jamesmcarthur has joined #storyboard07:34
*** jamesmcarthur has quit IRC07:38
*** jpich has joined #storyboard07:41
*** dtantsur|afk is now known as dtantsur09:07
-openstackstatus- NOTICE: Jobs using devstack-gate (legacy devstack jobs) have been failing due to an ara update. We use now a newer ansible version, it's safe to recheck if you see "ImportError: No module named manager" in the logs.09:58
*** udesale has quit IRC11:12
*** EmilienM is now known as EvilienM12:57
*** tosky has joined #storyboard13:02
*** jamesmcarthur has joined #storyboard14:08
*** jaosorior has quit IRC14:43
*** jpich has quit IRC15:59
*** openstackgerrit has quit IRC16:06
*** jamesmcarthur has quit IRC17:30
*** jamesmcarthur has joined #storyboard17:34
*** jamesmcarthur has quit IRC17:45
*** dtantsur is now known as dtantsur|afk17:59
*** openstackgerrit has joined #storyboard18:18
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Refactor Project Group editing to match other objects  https://review.openstack.org/59840718:18
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Support linking to Project Groups by name  https://review.openstack.org/59840818:18
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Link to Project Groups by name  https://review.openstack.org/59840918:18
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Refactor Project Group editing to match other objects  https://review.openstack.org/59840718:21
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Support linking to Project Groups by name  https://review.openstack.org/59840818:21
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Link to Project Groups by name  https://review.openstack.org/59840918:21
SotKanyone feeling bored and like they fancy being a second reviewer on https://review.openstack.org/#/c/590047/ ?18:22
clarkbSotK: left a couple comments, I'm not super familiar with pecan so unsure if that is expected use but it feels like the wrong thing to do in python18:35
clarkb(basically overriding _route is probably not a great idea if pecan changes internals later, could we instead have pecan do route dispatch then we branch after based on the type of the input18:36
clarkbbut I'm unsure of common practice around that so didn'1 -118:36
clarkb*didn't18:36
*** tosky has quit IRC18:52
*** openstackstatus has quit IRC18:58
SotKclarkb: thanks!19:13
SotKoverriding _route is the documented way to do custom routing in Pecan (https://pecan.readthedocs.io/en/latest/routing.html#pecan-s-routing-algorithm) and is also consistent with what we currently do elsewhere that we need custom routing19:15
SotKthough I agree thinking about it that branching after Pecan has handled the routing would probably be neater19:15
SotKalthough actually I'm not sure how easy it is to neatly make wsme allow either a string or int on the same method19:20
clarkbSotK: is it an error to treat 12345 as a string in wsme? (I sort of assumed that was allowed) then you'd be able to is_int after and convert to int if necesary19:21
clarkbbut its possible wsme treats that as an error19:22
SotKyeah that would probably work19:23
* SotK tries it19:27
*** openstackstatus has joined #storyboard19:39
*** ChanServ sets mode: +v openstackstatus19:39
SotKit works fine for the projects subcontroller, so I'll make the change there19:51
SotKit'll work fine for the project controller which is the other place we override that function, I'll update that in a different patch because I prefer this approach19:52
SotKhowever it doesn't entirely work for the project groups controller (and I found a bug with the existing patch in the process) since the default RestController routing can't handle doing something like `http://localhost:8080/v1/project_groups/sotk/programming/projects`19:54
SotK(it gives a 404 rather than routing to the projects subcontroller as it should do)19:54
*** openstackstatus has quit IRC19:55
*** openstackstatus has joined #storyboard19:59
*** ChanServ sets mode: +v openstackstatus19:59
*** openstackstatus has quit IRC20:11
*** openstackstatus has joined #storyboard20:13
*** ChanServ sets mode: +v openstackstatus20:13
SotKI'm too sleepy to reason about this logic, I'll update my patch tomorrow20:27
*** openstackstatus has quit IRC20:36
*** openstackstatus has joined #storyboard20:39
*** ChanServ sets mode: +v openstackstatus20:39
*** jamesmcarthur has joined #storyboard21:44
*** EvilienM is now known as EmilienM22:04
*** tosky has joined #storyboard22:09
*** jamesmcarthur has quit IRC23:01
*** jamesmcarthur has joined #storyboard23:10
*** tosky has quit IRC23:14
*** jamesmcarthur has quit IRC23:15
fungiSotK: clarkb: on the rare occasions i've had pecan/wsme questions, i've sought input from dhellmann. he seems to have some familiarity there23:23
*** jamesmcarthur has joined #storyboard23:31
*** jamesmcarthur has quit IRC23:40
*** jamesmcarthur has joined #storyboard23:53
*** jamesmcarthur has quit IRC23:57
*** jamesmcarthur has joined #storyboard23:58
*** jamesmcarthur has quit IRC23:58
*** jamesmcarthur has joined #storyboard23:58

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