*** dviroel__ is now known as dviroel | 11:36 | |
opendevreview | David Wilde proposed openstack/keystone-tempest-plugin master: Add existing user logic https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/878047 | 11:58 |
---|---|---|
*** sfinucan is now known as stephenfin | 13:04 | |
opendevreview | John Garbutt proposed openstack/oslo.limit master: Two level limit enforcer https://review.opendev.org/c/openstack/oslo.limit/+/695527 | 13:21 |
opendevreview | Steven Relf proposed openstack/keystoneauth master: Updates the v3oidcpassword function to also send the client_id as part of the request body, this seems to be a requirement for services like onelogin. https://review.opendev.org/c/openstack/keystoneauth/+/881969 | 14:03 |
knikolla | d34dh0r53: i'm sorry for not getting to the code reviews I said I would last week. Got pulled into some work-related priorities. I blocked some time for this Thursday, but I hopefully can get to them sooner. | 14:36 |
dmendiza[m] | d34dh0r53: weekly meeting? 🤔 | 15:02 |
d34dh0r53 | yeah, other meeting ran long | 15:03 |
d34dh0r53 | #startmeeting keystone | 15:03 |
opendevmeet | Meeting started Tue May 2 15:03:37 2023 UTC and is due to finish in 60 minutes. The chair is d34dh0r53. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:03 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:03 |
opendevmeet | The meeting name has been set to 'keystone' | 15:03 |
dmendiza[m] | 🙋♂️ | 15:03 |
d34dh0r53 | #topic roll call | 15:04 |
d34dh0r53 | admiyo, bbobrov, crisloma, d34dh0r53, dpar, dstanek, hrybacki, knikolla[m], lbragstad, lwanderley, kmalloc, rodrigods, samueldmq, ruan_he, wxy, sonuk, vishakha, Ajay, rafaelwe, xek, gmann, zaitcev, reqa, dmendiza[m] | 15:04 |
d34dh0r53 | o/ | 15:04 |
knikolla | o/ got pulled into a work meeting, sorry | 15:05 |
d34dh0r53 | no problem knikolla | 15:06 |
d34dh0r53 | #topic review past meeting work items | 15:06 |
d34dh0r53 | #link https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-04-25-15.15.html | 15:06 |
d34dh0r53 | first up dmendiza[m], d34dh0r53 update the keystone groups to inherit from keystone-core | 15:07 |
d34dh0r53 | did you ever get this inheriting properly dmendiza[m] ? | 15:07 |
dmendiza[m] | d34dh0r53: negative ... but we can look at it after this meeting if we end early? | 15:08 |
d34dh0r53 | sure | 15:08 |
d34dh0r53 | #action dmendiza[m], d34dh0r53 update the keystone groups to inherit from keystone-core | 15:08 |
d34dh0r53 | next up d34dh0r53 investigate https://bugs.launchpad.net/keystone/+bug/2009752 | 15:09 |
d34dh0r53 | I need to test this, my federation environment was donated to someone else so I need a new one | 15:09 |
d34dh0r53 | #action d34dh0r53 investigate https://bugs.launchpad.net/keystone/+bug/2009752 | 15:09 |
d34dh0r53 | d34dh0r53 Look into adding/restoring a known issues section to our documentation | 15:10 |
opendevreview | John Garbutt proposed openstack/oslo.limit master: Two level limit enforcer https://review.opendev.org/c/openstack/oslo.limit/+/695527 | 15:10 |
opendevreview | John Garbutt proposed openstack/oslo.limit master: Fix up tox -e cover to use coverage https://review.opendev.org/c/openstack/oslo.limit/+/882072 | 15:10 |
d34dh0r53 | also did not get to this last week | 15:10 |
d34dh0r53 | #action d34dh0r53 Look into adding/restoring a known issues section to our documentation | 15:10 |
d34dh0r53 | #action d34dh0r53 add https://bugs.launchpad.net/keystone/+bug/1305950 to the known issues section of our documentation | 15:10 |
d34dh0r53 | as well | 15:10 |
d34dh0r53 | d34dh0r53 to review LDAP bugs https://bugs.launchpad.net/keystone/+bugs?field.tag=ldap | 15:10 |
opendevreview | John Garbutt proposed openstack/oslo.limit master: Two level limit enforcer https://review.opendev.org/c/openstack/oslo.limit/+/695527 | 15:11 |
d34dh0r53 | I need to do this as well | 15:11 |
d34dh0r53 | #action https://bugs.launchpad.net/keystone/+bugs?field.tag=ldap | 15:11 |
d34dh0r53 | next up we have | 15:12 |
d34dh0r53 | #topic liaison updates | 15:12 |
d34dh0r53 | nothing from VMT | 15:12 |
d34dh0r53 | cool, as always help wanted | 15:13 |
d34dh0r53 | #help If you'd like to be a cross-project liaison please let d34dh0r53 know | 15:13 |
d34dh0r53 | moving on to specifications | 15:14 |
d34dh0r53 | #topic specification OAuth 2.0 (hiromu) | 15:14 |
d34dh0r53 | #link https://review.opendev.org/q/topic:bp%252Foauth2-client-credentials-ext | 15:14 |
zaitcev | So, what's left there? I was reviewing stuff on and off, but I lost the big picture. | 15:16 |
d34dh0r53 | hiromu isn't around, I think it's mostly reviewing the patches listed in the link above | 15:17 |
zaitcev | I see. | 15:17 |
d34dh0r53 | primarily #link https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/874716 | 15:17 |
d34dh0r53 | next up | 15:18 |
d34dh0r53 | #topic specification Secure RBAC (dmendiza[m]) | 15:18 |
d34dh0r53 | #link https://governance.openstack.org/tc/goals/selected/consistent-and-secure-rbac.html#z-release-timeline_ | 15:18 |
d34dh0r53 | hi hiromu | 15:19 |
d34dh0r53 | do you have any updates on the OAuth 2.0 BZ? Anything in particular you need reviews or input on? | 15:20 |
*** Guest74 is now known as atmark | 15:22 | |
d34dh0r53 | guess it was just his client re-connecting, moving on. | 15:22 |
d34dh0r53 | dmendiza[m]: any s-rbac updates/asks? | 15:22 |
dmendiza[m] | Not anything off the top of my head. | 15:23 |
dmendiza[m] | I think we need to move the "manager" role spec forward? | 15:23 |
dmendiza[m] | I'll have to check ... I've been mostly looking at downstream stuff for SRBAC. | 15:24 |
d34dh0r53 | yeah, we need to get that moving again | 15:24 |
dmendiza[m] | Not sure if gmann is around for any asks? | 15:24 |
gmann | yeah, manager and service role is needed and good to have merged | 15:24 |
gmann | but not sure if review comment are fixed there. I will ping abhishek about those | 15:25 |
dmendiza[m] | Thanks gmann . I'll try to catch up on both of those efforts this week. | 15:25 |
gmann | thanks dmendiza[m] really appreciate | 15:25 |
d34dh0r53 | awesome, thanks gmann and dmendiza[m] | 15:26 |
d34dh0r53 | #topic SQLAlchemy 2.0 (stephenfin) | 15:26 |
d34dh0r53 | #link https://review.opendev.org/q/topic:sqlalchemy-20+is:open+project:openstack/keystone | 15:26 |
d34dh0r53 | we really need to review these, perhaps we can dedicate the reviewathon this week to these patches and backports | 15:27 |
d34dh0r53 | cool, I think we'll do that | 15:28 |
d34dh0r53 | #topic open discussion | 15:28 |
d34dh0r53 | lots of backports for drencrom | 15:28 |
opendevreview | John Garbutt proposed openstack/oslo.limit master: Fix up connection object https://review.opendev.org/c/openstack/oslo.limit/+/882077 | 15:29 |
d34dh0r53 | (drencrom) We need to merge these backports to fix pep8 tests | 15:29 |
d34dh0r53 | ussuri #link https://review.opendev.org/c/openstack/keystonemiddleware/+/878028 | 15:29 |
d34dh0r53 | wallaby #link https://review.opendev.org/c/openstack/keystonemiddleware/+/878026 | 15:29 |
d34dh0r53 | xena #link https://review.opendev.org/c/openstack/keystonemiddleware/+/878025 | 15:29 |
d34dh0r53 | yoga #link https://review.opendev.org/c/openstack/keystonemiddleware/+/878024 | 15:29 |
d34dh0r53 | zed #link https://review.opendev.org/c/openstack/keystonemiddleware/+/878023 | 15:29 |
d34dh0r53 | cores, please take a look if you get a chance | 15:29 |
opendevreview | John Garbutt proposed openstack/oslo.limit master: Two level limit enforcer https://review.opendev.org/c/openstack/oslo.limit/+/695527 | 15:29 |
d34dh0r53 | once those are done this needs a WF #link https://review.opendev.org/c/openstack/keystonemiddleware/+/873921 | 15:29 |
d34dh0r53 | next up we have the PooledLdapHandler message.clean() backports | 15:30 |
d34dh0r53 | (mustafakemalgilor) PooledLdapHandler message.clean() patch backports | 15:30 |
d34dh0r53 | review request | 15:30 |
d34dh0r53 | #link ussuri: https://review.opendev.org/c/openstack/keystone/+/874846 | 15:30 |
d34dh0r53 | #link victoria: https://review.opendev.org/c/openstack/keystone/+/874847 | 15:30 |
d34dh0r53 | #link wallaby: https://review.opendev.org/c/openstack/keystone/+/874844 | 15:30 |
d34dh0r53 | #link xena: https://review.opendev.org/c/openstack/keystone/+/874843 | 15:30 |
d34dh0r53 | again, these should be easy backports | 15:31 |
d34dh0r53 | finally | 15:31 |
d34dh0r53 | (reqa) OAuth 2.0 Device Authorization Grant bugfix | 15:31 |
d34dh0r53 | review request | 15:31 |
d34dh0r53 | #link https://review.opendev.org/c/openstack/keystoneauth/+/876893 | 15:31 |
d34dh0r53 | that's a fixup https://review.opendev.org/q/topic:reqa/v3oidcdeviceauth - anything left to be done here? | 15:31 |
d34dh0r53 | moving on | 15:33 |
d34dh0r53 | #topic bug review | 15:33 |
d34dh0r53 | #link https://bugs.launchpad.net/keystone/?orderby=-id&start=0 | 15:33 |
d34dh0r53 | nothing new in keystone | 15:33 |
d34dh0r53 | #link https://bugs.launchpad.net/python-keystoneclient/?orderby=-id&start=0 | 15:33 |
d34dh0r53 | python-keystoneclient has no new bugs | 15:34 |
d34dh0r53 | #link https://bugs.launchpad.net/keystoneauth/+bugs?orderby=-id&start=0 | 15:34 |
d34dh0r53 | keystoneauth is clean | 15:34 |
d34dh0r53 | #link https://bugs.launchpad.net/keystonemiddleware/+bugs?orderby=-id&start=0 | 15:34 |
d34dh0r53 | no new ones in keystonemiddleware either | 15:35 |
d34dh0r53 | #link https://bugs.launchpad.net/pycadf/+bugs?orderby=-id&start=0 | 15:35 |
d34dh0r53 | pycadf is clean | 15:35 |
d34dh0r53 | #link https://bugs.launchpad.net/ldappool/+bugs?orderby=-id&start=0 | 15:35 |
d34dh0r53 | as is ldappool | 15:35 |
d34dh0r53 | #topic conclusion | 15:35 |
d34dh0r53 | anything else before we go? | 15:36 |
d34dh0r53 | Thanks for joining all! Hopefully I'll see you at the reviewathon 14:00 UTC Friday ;) | 15:36 |
d34dh0r53 | #endmeeting | 15:36 |
opendevmeet | Meeting ended Tue May 2 15:36:59 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:37 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-05-02-15.03.html | 15:37 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-05-02-15.03.txt | 15:37 |
opendevmeet | Log: https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-05-02-15.03.log.html | 15:37 |
opendevreview | David Wilde proposed openstack/keystone-tempest-plugin master: Add existing user logic https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/878047 | 17:01 |
opendevreview | John Garbutt proposed openstack/oslo.limit master: Two level limit enforcer https://review.opendev.org/c/openstack/oslo.limit/+/695527 | 17:21 |
opendevreview | David Wilde proposed openstack/keystone-tempest-plugin master: Add existing user logic https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/878047 | 19:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!