Tuesday, 2023-05-02

*** dviroel__ is now known as dviroel11:36
opendevreviewDavid Wilde proposed openstack/keystone-tempest-plugin master: Add existing user logic  https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/87804711:58
*** sfinucan is now known as stephenfin13:04
opendevreviewJohn Garbutt proposed openstack/oslo.limit master: Two level limit enforcer  https://review.opendev.org/c/openstack/oslo.limit/+/69552713:21
opendevreviewSteven 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/+/88196914:03
knikollad34dh0r53: 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
d34dh0r53yeah, other meeting ran long15:03
d34dh0r53#startmeeting keystone15:03
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:03
opendevmeetThe meeting name has been set to 'keystone'15:03
dmendiza[m]🙋‍♂️15:03
d34dh0r53#topic roll call15:04
d34dh0r53admiyo, 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
d34dh0r53o/15:04
knikollao/ got pulled into a work meeting, sorry15:05
d34dh0r53no problem knikolla 15:06
d34dh0r53#topic review past meeting work items15:06
d34dh0r53#link https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-04-25-15.15.html15:06
d34dh0r53first up dmendiza[m], d34dh0r53 update the keystone groups to inherit from keystone-core15:07
d34dh0r53did 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
d34dh0r53sure15:08
d34dh0r53#action dmendiza[m], d34dh0r53 update the keystone groups to inherit from keystone-core15:08
d34dh0r53next up d34dh0r53 investigate https://bugs.launchpad.net/keystone/+bug/200975215:09
d34dh0r53I need to test this, my federation environment was donated to someone else so I need a new one15:09
d34dh0r53#action d34dh0r53 investigate https://bugs.launchpad.net/keystone/+bug/200975215:09
d34dh0r53d34dh0r53 Look into adding/restoring a known issues section to our documentation15:10
opendevreviewJohn Garbutt proposed openstack/oslo.limit master: Two level limit enforcer  https://review.opendev.org/c/openstack/oslo.limit/+/69552715:10
opendevreviewJohn Garbutt proposed openstack/oslo.limit master: Fix up tox -e cover to use coverage  https://review.opendev.org/c/openstack/oslo.limit/+/88207215:10
d34dh0r53also did not get to this last week15:10
d34dh0r53#action d34dh0r53 Look into adding/restoring a known issues section to our documentation15:10
d34dh0r53#action d34dh0r53 add https://bugs.launchpad.net/keystone/+bug/1305950 to the known issues section of our documentation15:10
d34dh0r53as well15:10
d34dh0r53d34dh0r53 to review LDAP bugs https://bugs.launchpad.net/keystone/+bugs?field.tag=ldap15:10
opendevreviewJohn Garbutt proposed openstack/oslo.limit master: Two level limit enforcer  https://review.opendev.org/c/openstack/oslo.limit/+/69552715:11
d34dh0r53I need to do this as well15:11
d34dh0r53#action https://bugs.launchpad.net/keystone/+bugs?field.tag=ldap15:11
d34dh0r53next up we have15:12
d34dh0r53#topic liaison updates15:12
d34dh0r53nothing from VMT15:12
d34dh0r53cool, as always help wanted15:13
d34dh0r53#help If you'd like to be a cross-project liaison please let d34dh0r53 know15:13
d34dh0r53moving on to specifications15:14
d34dh0r53#topic specification OAuth 2.0 (hiromu)15:14
d34dh0r53#link https://review.opendev.org/q/topic:bp%252Foauth2-client-credentials-ext15:14
zaitcevSo, what's left there? I was reviewing stuff on and off, but I lost the big picture.15:16
d34dh0r53hiromu isn't around, I think it's mostly reviewing the patches listed in the link above15:17
zaitcevI see.15:17
d34dh0r53primarily #link https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/87471615:17
d34dh0r53next up15: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
d34dh0r53hi hiromu 15:19
d34dh0r53do 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 atmark15:22
d34dh0r53guess it was just his client re-connecting, moving on.15:22
d34dh0r53dmendiza[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
d34dh0r53yeah, we need to get that moving again15:24
dmendiza[m]Not sure if gmann is around for any asks?15:24
gmannyeah, manager and service role is needed and good to have merged15:24
gmannbut not sure if review comment are fixed there. I will ping abhishek about those15:25
dmendiza[m]Thanks gmann .  I'll try to catch up on both of those efforts this week.15:25
gmannthanks dmendiza[m] really appreciate 15:25
d34dh0r53awesome, 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/keystone15:26
d34dh0r53we really need to review these, perhaps we can dedicate the reviewathon this week to these patches and backports15:27
d34dh0r53cool, I think we'll do that15:28
d34dh0r53#topic open discussion15:28
d34dh0r53lots of backports for drencrom15:28
opendevreviewJohn Garbutt proposed openstack/oslo.limit master: Fix up connection object  https://review.opendev.org/c/openstack/oslo.limit/+/88207715:29
d34dh0r53(drencrom) We need to merge these backports to fix pep8 tests15:29
d34dh0r53ussuri #link https://review.opendev.org/c/openstack/keystonemiddleware/+/87802815:29
d34dh0r53wallaby #link https://review.opendev.org/c/openstack/keystonemiddleware/+/87802615:29
d34dh0r53xena #link https://review.opendev.org/c/openstack/keystonemiddleware/+/87802515:29
d34dh0r53yoga #link https://review.opendev.org/c/openstack/keystonemiddleware/+/87802415:29
d34dh0r53zed #link https://review.opendev.org/c/openstack/keystonemiddleware/+/87802315:29
d34dh0r53cores, please take a look if you get a chance15:29
opendevreviewJohn Garbutt proposed openstack/oslo.limit master: Two level limit enforcer  https://review.opendev.org/c/openstack/oslo.limit/+/69552715:29
d34dh0r53once those are done this needs a WF #link https://review.opendev.org/c/openstack/keystonemiddleware/+/87392115:29
d34dh0r53next up we have the PooledLdapHandler message.clean() backports15:30
d34dh0r53(mustafakemalgilor) PooledLdapHandler message.clean() patch backports15:30
d34dh0r53review request15:30
d34dh0r53#link ussuri: https://review.opendev.org/c/openstack/keystone/+/87484615:30
d34dh0r53#link victoria: https://review.opendev.org/c/openstack/keystone/+/87484715:30
d34dh0r53#link wallaby: https://review.opendev.org/c/openstack/keystone/+/87484415:30
d34dh0r53#link xena: https://review.opendev.org/c/openstack/keystone/+/87484315:30
d34dh0r53again, these should be easy backports15:31
d34dh0r53finally15:31
d34dh0r53(reqa) OAuth 2.0 Device Authorization Grant bugfix15:31
d34dh0r53review request15:31
d34dh0r53#link https://review.opendev.org/c/openstack/keystoneauth/+/87689315:31
d34dh0r53that's a fixup https://review.opendev.org/q/topic:reqa/v3oidcdeviceauth - anything left to be done here?15:31
d34dh0r53moving on15:33
d34dh0r53#topic bug review15:33
d34dh0r53#link https://bugs.launchpad.net/keystone/?orderby=-id&start=015:33
d34dh0r53nothing new in keystone15:33
d34dh0r53#link https://bugs.launchpad.net/python-keystoneclient/?orderby=-id&start=015:33
d34dh0r53python-keystoneclient has no new bugs15:34
d34dh0r53#link https://bugs.launchpad.net/keystoneauth/+bugs?orderby=-id&start=015:34
d34dh0r53keystoneauth is clean15:34
d34dh0r53#link https://bugs.launchpad.net/keystonemiddleware/+bugs?orderby=-id&start=015:34
d34dh0r53no new ones in keystonemiddleware either15:35
d34dh0r53#link https://bugs.launchpad.net/pycadf/+bugs?orderby=-id&start=015:35
d34dh0r53pycadf is clean15:35
d34dh0r53#link https://bugs.launchpad.net/ldappool/+bugs?orderby=-id&start=015:35
d34dh0r53as is ldappool15:35
d34dh0r53#topic conclusion15:35
d34dh0r53anything else before we go?15:36
d34dh0r53Thanks for joining all! Hopefully I'll see you at the reviewathon 14:00 UTC Friday ;)15:36
d34dh0r53#endmeeting15:36
opendevmeetMeeting ended Tue May  2 15:36:59 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:37
opendevmeetMinutes:        https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-05-02-15.03.html15:37
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-05-02-15.03.txt15:37
opendevmeetLog:            https://meetings.opendev.org/meetings/keystone/2023/keystone.2023-05-02-15.03.log.html15:37
opendevreviewDavid Wilde proposed openstack/keystone-tempest-plugin master: Add existing user logic  https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/87804717:01
opendevreviewJohn Garbutt proposed openstack/oslo.limit master: Two level limit enforcer  https://review.opendev.org/c/openstack/oslo.limit/+/69552717:21
opendevreviewDavid Wilde proposed openstack/keystone-tempest-plugin master: Add existing user logic  https://review.opendev.org/c/openstack/keystone-tempest-plugin/+/87804719:11

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!