| stephenfin | frickler: gtema: TIL that keystone is case-insensitive. The difference in behavior appears to be because Resource.find is case sensitive https://paste.opendev.org/show/bkHaAZHo7xaUKu2I5WwW/ | 10:10 |
|---|---|---|
| stephenfin | Changing that globally is likely to be a large change that could have knock-on effects for other services that may not (a) allow filtering by name query string parameters and/or (b) are case sensitive. idk if we want to do that | 10:11 |
| gtema | neah, we must stick to case sensitivity | 10:11 |
| gtema | services themselves are free to be case sensitive or insensitive. SDK must respect case | 10:12 |
| stephenfin | This is also presumably a change we've seen in the other identity resources that have switched to using SDK in recent releases. I'm guessing this is just the first one we've noticied | 10:12 |
| stephenfin | *noticed | 10:12 |
| stephenfin | If this is important enough as a regression, we probably need to temporarily reimplement Resource.find in OSC and issue warnings on case-insensitive matches | 10:13 |
| stephenfin | s/as a/to be classified as a/ | 10:14 |
| gtema | wrt keystone I see additional undocumented "case_sensitive" parameters support in the code | 10:14 |
| frickler | stephenfin: gtema: how about defining this to be a feature instead of a regression, but being nice to consumers and add a reno with a big warning? | 11:57 |
| gtema | I would also prefer feature rather than regression | 11:58 |
| opendevreview | yatin proposed openstack/openstacksdk stable/2024.1: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943177 | 13:09 |
| opendevreview | yatin proposed openstack/openstacksdk stable/2024.1: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943177 | 14:39 |
| opendevreview | yatin proposed openstack/openstacksdk stable/2024.2: [Stable Only] Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/943182 | 14:39 |
| *** priteau is now known as Guest10549 | 14:46 | |
| *** priteau2 is now known as priteau | 14:46 | |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: Bootstrap magnum OpenAPI build https://review.opendev.org/c/openstack/codegenerator/+/943058 | 19:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!