Tuesday, 2019-05-14

*** dklyle has quit IRC00:11
*** ttsiouts has joined #openstack-placement01:22
*** bhagyashris has joined #openstack-placement01:25
*** dklyle has joined #openstack-placement01:37
*** ttsiouts has quit IRC01:54
*** dklyle has quit IRC01:54
*** ttsiouts has joined #openstack-placement03:03
*** sean-k-mooney has quit IRC03:09
*** sean-k-mooney has joined #openstack-placement03:11
*** ttsiouts has quit IRC03:35
*** ttsiouts has joined #openstack-placement05:33
*** ttsiouts has quit IRC06:06
*** amodi has quit IRC06:31
*** belmoreira has joined #openstack-placement06:41
*** tssurya has joined #openstack-placement07:25
*** helenafm has joined #openstack-placement07:27
*** ttsiouts has joined #openstack-placement07:31
*** openstackgerrit has joined #openstack-placement07:47
openstackgerritTetsuro Nakamura proposed openstack/placement master: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65897707:47
*** belmoreira has quit IRC07:59
*** ttsiouts has quit IRC08:34
*** belmoreira has joined #openstack-placement08:39
*** ttsiouts has joined #openstack-placement09:10
*** e0ne has joined #openstack-placement09:12
openstackgerritTetsuro Nakamura proposed openstack/placement master: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65897709:33
*** bhagyashris has quit IRC09:35
*** tssurya has quit IRC09:41
*** tssurya has joined #openstack-placement09:41
*** ttsiouts has quit IRC09:44
*** belmoreira has quit IRC09:50
*** gibi is now known as gibi_off10:02
*** belmoreira has joined #openstack-placement10:56
*** e0ne has quit IRC11:31
*** e0ne has joined #openstack-placement11:31
*** e0ne has quit IRC12:02
*** tetsuro has joined #openstack-placement12:09
openstackgerritTetsuro Nakamura proposed openstack/placement master: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65897712:09
*** tetsuro has quit IRC12:11
*** mriedem has joined #openstack-placement12:49
*** e0ne has joined #openstack-placement12:50
*** ttsiouts has joined #openstack-placement13:02
*** belmoreira has quit IRC13:02
*** ttsiouts has quit IRC13:07
*** ttsiouts has joined #openstack-placement13:26
*** ttsiouts has quit IRC13:30
*** belmoreira has joined #openstack-placement13:58
*** dklyle has joined #openstack-placement14:13
*** ttsiouts has joined #openstack-placement15:27
*** helenafm has quit IRC15:30
*** belmoreira has quit IRC15:48
*** dklyle has quit IRC15:50
*** dklyle has joined #openstack-placement15:52
*** openstackgerrit has quit IRC15:54
*** e0ne has quit IRC15:57
*** ttsiouts has quit IRC16:01
*** dklyle has quit IRC16:11
*** dklyle has joined #openstack-placement16:16
*** dklyle has quit IRC16:21
*** dklyle has joined #openstack-placement16:23
*** dklyle has quit IRC16:59
*** dklyle has joined #openstack-placement17:01
*** tssurya has quit IRC17:06
*** dklyle has quit IRC17:52
*** e0ne has joined #openstack-placement17:57
*** ttsiouts has joined #openstack-placement17:58
*** e0ne has quit IRC18:05
*** ttsiouts has quit IRC18:31
*** e0ne has joined #openstack-placement18:38
efriededleafe: Is it legal and/or acceptable to have a queryparam that's just a key with no value?19:08
efriedPresumably ?foo= is legal - would translate to {'foo': ''} - but what about just ?foo19:09
edleafeefried: dunno - I've never tried that. Usually it's foo=119:10
edleafeLooking through the spec, I don't see any requirement for '='19:12
efriededleafe: how would that show up in python on the other side of my wsgi?19:15
edleafeefried: https://tools.ietf.org/html/rfc3986#section-3.419:15
edleafeefried: like I said, I haven't tried it.19:15
*** dklyle has joined #openstack-placement19:16
efriedight19:16
edleafelemme try something quick19:16
*** dklyle has quit IRC19:19
*** david-lyle has joined #openstack-placement19:19
edleafeefried: running flask with uwsgi, you just get back `request.query_string`, so sending ?foo,bar,baz=7 gives "foo,bar,baz=7" as request.query_string19:21
edleafeIOW, you have to do your own parsing19:22
efriedmm, that makes sense.19:22
efriededleafe: I guess the methods we're using in the placement handlers, like req.GET.getall('foo'), are doing that parsing19:24
gryfefried, delimiter for the query params should be & isn;t it?19:24
efriedyes19:24
gryfsorry, was referring to ed19:24
edleafegryf: yeah, I was just doing a quick and dirty test19:24
edleafesame results with &19:24
gryfack19:24
efriedso a "normal" parser (whatever that means) would return {'foo,bar,baz': '7'} for that one. But I took ed's point.19:25
edleafeefried: we pass resource requests like this: ?resources=VCPU:2,MEMORY_MB:1024, so /query19:28
edleafedoh!19:29
edleafeso I think that we have some stuff that assumes key=val19:29
efriedwell, yes, we do, but we could recode if we wanted19:32
efriedepic comment coming in the nested magic spec that might explain why I'm asking.19:32
* edleafe can hardly wait19:33
efriededleafe: https://review.opendev.org/#/c/658510/2/doc/source/specs/train/approved/2005575-nested-magic.rst@13419:38
edleafeefried: got it in an open tab. Will look when I can refocus my brain19:40
*** david-lyle has quit IRC19:42
efriededleafe: Not sure that's the criterion you should use19:58
edleafeefried: s/refocus/unfocus19:59
*** openstackgerrit has joined #openstack-placement20:04
openstackgerritMerged openstack/placement master: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65897720:04
*** e0ne has quit IRC20:20
*** ttsiouts has joined #openstack-placement20:28
*** cdent has joined #openstack-placement20:36
*** cdent has quit IRC20:58
*** ttsiouts has quit IRC21:01
*** dklyle has joined #openstack-placement21:27
*** dklyle has quit IRC21:44
*** amodi has joined #openstack-placement21:44
*** dklyle has joined #openstack-placement21:44
*** dklyle has quit IRC21:50
*** ttsiouts has joined #openstack-placement22:58
*** ttsiouts has quit IRC23:32
*** irclogbot_2 has quit IRC23:45
*** irclogbot_2 has joined #openstack-placement23:48

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