| mnasiadka | Grzegorz Grasza, Dave Wilde (d34dh0r53) : Sorry to be chasing again - but also need reviews on https://review.opendev.org/c/openstack/pycadf/+/930971 :) | 07:15 |
|---|---|---|
| opendevreview | Merged openstack/keystone master: Add JSON schema to `service provider` and validation decorators to service provider resource. https://review.opendev.org/c/openstack/keystone/+/930610 | 14:00 |
| opendevreview | Merged openstack/keystone master: Add doc of OAuth 2.0 Mutual-TLS Authenticate https://review.opendev.org/c/openstack/keystone/+/860928 | 14:00 |
| opendevreview | Artem Goncharov proposed openstack/keystone master: Add JSON Schema to `application credentials` and validation decorators to application credential resource. https://review.opendev.org/c/openstack/keystone/+/925517 | 14:16 |
| opendevreview | Merged openstack/keystone master: Add jsonschemas to the User resource https://review.opendev.org/c/openstack/keystone/+/938788 | 14:36 |
| opendevreview | Merged openstack/keystone master: Add jsonschema for Group resource https://review.opendev.org/c/openstack/keystone/+/938789 | 14:36 |
| opendevreview | Merged openstack/keystone master: Implement pagination for users and groups https://review.opendev.org/c/openstack/keystone/+/938814 | 14:36 |
| opendevreview | Merged openstack/pycadf master: Remove audit map for retired ceilometer-api and panko https://review.opendev.org/c/openstack/pycadf/+/930971 | 15:46 |
| opendevreview | Merged openstack/pycadf master: Add missing maps into setup.cfg https://review.opendev.org/c/openstack/pycadf/+/936741 | 15:46 |
| opendevreview | Merged openstack/keystone master: Remove templated catalog driver https://review.opendev.org/c/openstack/keystone/+/917923 | 16:06 |
| opendevreview | Merged openstack/keystone master: Add JSON Schema to `endpoints` and validation decorators to endpoints resource. https://review.opendev.org/c/openstack/keystone/+/927856 | 18:52 |
| opendevreview | Artem Goncharov proposed openstack/keystone master: Add JSON schema to `identity provider` and validation decorators to identity provider resource. https://review.opendev.org/c/openstack/keystone/+/930633 | 18:53 |
| opendevreview | Merged openstack/keystone master: Add JSON schema to `trust` and validation decorators to trust resource. https://review.opendev.org/c/openstack/keystone/+/930361 | 19:13 |
| frickler | gtema: seems this job started failing this afternoon, the only delta between passing and failing jobs that I found were the keystone changes that merged earlier, but I have no idea yet how things broke https://zuul.opendev.org/t/openstack/builds?job_name=openstacksdk-functional-devstack&skip=0 | 20:36 |
| frickler | might be a good idea though to run that job for keystone changes, too? | 20:36 |
| gtema | Flicker, that's weird. Wondering most that it is compute test failing and not identity | 20:39 |
| frickler | yes, maybe next step is to bisect to find out which of the changes is causing the failure. I can try that tomorrow | 20:44 |
| gtema | Me as well. Short look from mobile into logs (nova) shows weird mismatch of the user_id between positive entries and negatives | 20:45 |
| opendevreview | Pierre Riteau proposed openstack/keystone master: [DNM] Revert "Implement pagination for users and groups" https://review.opendev.org/c/openstack/keystone/+/939559 | 20:48 |
| priteau | I am just trying this revert in Nova CI | 20:49 |
| priteau | Not saying it will need to be reverted, maybe the issue will need to be fixed elsewhere | 20:50 |
| gtema | That might indeed make sense. Order of returned users is potentially altered so if test picks entry[0] is might get the wrong one | 20:53 |
| gtema | Yeah, seems precisely the case: https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/tests/functional/compute/v2/test_keypair.py#L57 so the ask test should be adapted since it tries to delete the key pair of a wrong user | 21:04 |
| gtema | If nobody does this till my tomorrow I will fix that | 21:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!