| *** mhen_ is now known as mhen | 01:20 | |
| mhen | ftr, I won't be able to attend this week's meeting but I've documented my progress on the domain manager in the agenda points under open discussion, feel free to have a look | 09:28 |
|---|---|---|
| kevko | Hi folks, can anybody fix keystone false-positive logging >> https://paste.openstack.org/show/btGUAr04Df5WmzqNh39K/ | 10:39 |
| kevko | it's producing a lot of logs ....but actually it's everything OK and working | 10:40 |
| gtema | kevko: https://review.opendev.org/c/openstack/keystone/+/924153 was recently proposed | 10:54 |
| kevko | gtema: but is this real fix ? because I dig into the code lastly and found that this is about argument passed into some function/s where there is a id passed but it's logging name | 11:19 |
| kevko | gtema: so I am just wondering if it is a real fix to just hide this ...shouldn't it be fixed better, deeper in a code ? | 11:20 |
| gtema | 404 is not an error in itself, so no need to fix anything else | 11:21 |
| kevko | gtema: so this is completly ok ? | 11:24 |
| kevko | gtema: https://paste.openstack.org/show/bBeJDmftPJqUecMPavt4/ | 11:24 |
| gtema | It is absolutely ok when user asks for unexisting resource. And it should not be treated/logged as an error in keystone | 11:27 |
| kevko | gtema: but user is asking for *existing* resource, but not with id, but with name | 11:28 |
| gtema | Name is not a valid id of a resource and keystone rightfully returns 404 to the SDK/cli which try to find it differently (through listing) | 11:30 |
| kevko | gtema: yeah, you are right, checking keystone api reference where it's defined that for example for role it's role_id not the name | 11:31 |
| gtema | It is true for all open stack services | 11:32 |
| kevko | gtema: thank you ! | 11:46 |
| gtema | wlcm | 11:46 |
| matusjenca | Hello. Sorry to bother, does anyone know where i can contact the devs of [keystonemiddleware](https://opendev.org/openstack/keystonemiddleware)? | 12:03 |
| gtema | here in this room. Whether anybody is now able to respond is a different question | 12:06 |
| matusjenca | alright, thanks | 12:15 |
| matusjenca | I would like someone to review my patch, it's been up since April with no comments or votes https://review.opendev.org/c/openstack/keystonemiddleware/+/915872 | 12:16 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.policy master: Enable enforce_scope and enforce_new_defaults by default https://review.opendev.org/c/openstack/oslo.policy/+/924283 | 16:00 |
| opendevreview | Ade Lee proposed openstack/keystone master: last_active_at should always be set on login https://review.opendev.org/c/openstack/keystone/+/924360 | 19:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!