| opendevreview | Jake Yip proposed openstack/openstacksdk master: Cast all header values to string https://review.opendev.org/c/openstack/openstacksdk/+/924301 | 07:49 |
|---|---|---|
| ganso | hi stephenfin ! thanks for the improvements in https://review.opendev.org/c/openstack/openstacksdk/+/922965 ! I left you a question regarding the changes. If you have a minute could you please take a look? I am sure the question will be trivial to you as you are more familiar with it | 14:02 |
| stephenfin | ganso: It's a fair point. I don't have a good answer for how to resolve that, unfortunately. Personally, I tend to ignore the `location` field entirely but that's just me | 15:49 |
| stephenfin | ganso: As I understand it, that the location attribute is either inferred by the auth information or provided explicitly from a `project_id` attribute of a given resource. Unfortunately cinder's '/limits do not expose this attribute | 15:56 |
| ganso | stephenfin: yea I had the same impression as well, that the location attribute is completely unrelated to the resource but more to who is performing the request, as it is in attribute that is included by the framework. I would be fine with that if you're fine with it, I just wanted to raise awareness just to confirm that's the behavior we want | 16:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!