| *** mhen_ is now known as mhen | 02:30 | |
| *** ykarel_ is now known as ykarel | 09:05 | |
| opendevreview | Quentin GROLLEAU proposed openstack/keystone master: Index ON revocation_event table to optimize token checks. https://review.opendev.org/c/openstack/keystone/+/929736 | 09:35 |
|---|---|---|
| *** ykarel__ is now known as ykarel | 12:44 | |
| dmsimard[m] | hi, just putting a bit of visibility on this bug that bit us recently: https://bugs.launchpad.net/keystone/+bug/2081082 | 13:44 |
| gtema | Yes dmsimard - noticed respinned change. Linter gods demands another sacrifice | 13:45 |
| dmsimard[m] | we know about ('idx_revocation_event_composite', 'revocation_event', ['issued_before,user_id,project_id,audit_id']) but not sure about the other one | 13:46 |
| gtema | sadly me neither | 13:47 |
| gtema | I can't really explain the potential usecase for myself. But on the other side it should not harm either | 13:47 |
| gtema | on prtactice, however, the table should be relatively small | 13:48 |
| dmsimard[m] | for us it was small until it wasn't 😅 | 13:49 |
| dmsimard[m] | there was a bug in glance that prevented it from creating trusts, fixing it lead to the growth of the revocation_event table which increased load and slowed down stuff | 13:50 |
| gtema | yeah, due to the pretty unfortunate keystone design there is no dedicated cleanup process for the table - this is definitely different with the KeystoneNG | 13:50 |
| opendevreview | Artem Goncharov proposed openstack/keystone master: Index ON revocation_event table to optimize token checks. https://review.opendev.org/c/openstack/keystone/+/929736 | 13:52 |
| opendevreview | Arnaud Morin proposed openstack/keystone master: Add a new index on revocation_event table https://review.opendev.org/c/openstack/keystone/+/929736 | 13:53 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.policy master: Remove support for JSON format policy file https://review.opendev.org/c/openstack/oslo.policy/+/929715 | 15:55 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!