17:00:36 <cmurphy> #startmeeting keystone
17:00:37 <openstack> Meeting started Tue Apr  7 17:00:36 2020 UTC and is due to finish in 60 minutes.  The chair is cmurphy. Information about MeetBot at http://wiki.debian.org/MeetBot.
17:00:38 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
17:00:41 <openstack> The meeting name has been set to 'keystone'
17:00:49 <cmurphy> #link https://etherpad.openstack.org/p/keystone-weekly-meeting agenda
17:01:01 <gagehugo> o/
17:01:16 <vishakha> o/
17:01:20 <knikolla> o/
17:03:20 <cmurphy> how's everyone doing?
17:03:25 <cmurphy> the weather's gotten really nice here
17:03:41 <gagehugo> no cabin fever yet
17:03:46 <gagehugo> it is also nice here
17:04:04 <knikolla> i'm slowly going insane
17:04:39 <vishakha> good here too
17:04:41 <cmurphy> knikolla: oof :(
17:06:48 <cmurphy> make sure to go outside every once in a while, helps with the sanity-keeping
17:07:11 <knikolla> i have, there's a park close by and it doesn't get too populated
17:07:47 <cmurphy> nice :)
17:08:02 <cmurphy> #topic announcements
17:08:44 <cmurphy> the nomination period is over and there were no contentions so no elections, so glad to say knikolla is going to be taking over :)
17:08:52 <cmurphy> i think the actual cutoff is sometime in the future
17:09:04 <cmurphy> but we can start transitioning ;)
17:09:25 <knikolla> :)
17:10:35 <knikolla> i'll do my best
17:10:54 <knikolla> victoria is my mother's name, so next cycle is a tiny bit more special
17:11:11 <cmurphy> cool! that means it's good luck
17:11:13 <vishakha> Ohh what a nice coincidence
17:11:40 <cmurphy> we have feature freeze and client freeze this week, so please help review the two specs knikolla is working on and python-keystoneclient changes
17:12:08 <vishakha> +1
17:13:40 <cmurphy> #topic l1 duty rotation
17:14:02 <cmurphy> I was on duty last week, only thing to report is I closed out https://bugs.launchpad.net/keystone/+bug/1866817 it was user error
17:14:03 <openstack> Launchpad bug 1866817 in OpenStack Identity (keystone) "Invalid input for field 'roles/0/id': 'role_admin' does not match '^[a-zA-Z0-9-]+$'" [Undecided,Invalid]
17:15:08 <cmurphy> up next is knikolla but since i know knikolla is busy finishing up spec work i wonder if vishakha or gagehugo would be interested in covering this week?
17:15:26 <vishakha> I can take for the next week
17:16:29 <gagehugo> I can probably do this week, might not be 100% available
17:17:37 <cmurphy> okay will write in gagehugo for this week and vishakha for next if that works with knikolla
17:17:50 <knikolla> sure, i appreciate it
17:18:12 <cmurphy> done
17:18:32 <cmurphy> #topic review requests
17:19:00 <cmurphy> knikolla: i commented on the tests for federated attrs
17:20:06 <cmurphy> vishakha: has some requests it looks like
17:20:15 <knikolla> cmurphy: thanks, i can do that
17:20:27 <vishakha> I started updating the contributor doc https://review.opendev.org/#/c/718145/
17:22:03 <vishakha> Next is https://review.opendev.org/#/c/715269/ for immutable option in openstackclient
17:22:32 <vishakha> and trivial change https://review.opendev.org/#/c/714477/
17:22:58 <vishakha> I am planning to update the contributor docments in some parts
17:23:17 <cmurphy> thanks so much for working on that vishakha
17:24:05 <cmurphy> any other review requests?
17:24:26 <knikolla> https://review.opendev.org/#/c/718156/
17:24:35 <knikolla> this should fix a race condition in federation tests
17:24:54 <knikolla> (provided i haven't messed it up, waiting on CI)
17:25:01 <cmurphy> awesome
17:26:31 <vishakha> The same  race condition related to https://review.opendev.org/#/c/704271/?
17:26:51 <knikolla> no, i don't think that one can be solved
17:26:58 <knikolla> because the identity provider must be named keystone
17:27:11 <knikolla> because that is what the saml issuer is
17:27:30 <knikolla> for that i suggest you just use the k2k test that is already there
17:27:37 <knikolla> and add the groups stuff on that
17:27:47 <vishakha> Yes i tried changing the keystoone name but it is failing anyhow
17:29:18 <cmurphy> vishakha: does knikolla's suggestion make sense? do you have a way forward?
17:30:15 <vishakha> Not yet. So instead of adding new class , I should update the group stuff in the existing class TestK2KFederatedAuthentication ?
17:31:15 <knikolla> vishakha: yes. they're both testing k2k, testing it with additional groups will not diminish the objective of the test
17:31:42 <vishakha> knikolla: thanks. I understood
17:32:41 <cmurphy> great
17:32:43 <vishakha> I just need to update theoverride setup_mapping onl
17:32:47 <vishakha> *only
17:33:55 <vishakha> I meant I need to override the setup_mapping in k2k class
17:33:58 <knikolla> and check that the user has the groups
17:34:15 <vishakha> yes
17:35:19 <vishakha> Thanks knikolla cmurphy
17:35:26 <cmurphy> thanks vishakha
17:35:31 <cmurphy> #topic open floor
17:35:50 <cmurphy> anything else going on?
17:35:53 <vishakha> We already discussed about the testcase
17:36:02 <knikolla> cmurphy: you think we're on track for the 2 specs by end of week?
17:36:22 <knikolla> i am starting to get mildly anxious, haha.
17:36:34 <cmurphy> knikolla: i think so, they both look good there's just a small amount of feedback to address
17:36:43 <cmurphy> i will look at the latest changes today
17:36:57 <cmurphy> hopefully lbragstad and gagehugo can help review
17:37:07 <knikolla> and i will update the tests based on your latest feedback by end of day.
17:37:14 <cmurphy> sweet
17:39:05 <knikolla> on a different notes, there's going to be TC voting this week. make sure to read the discussion threads on the mailing list and exercise your democratic right.
17:39:15 <cmurphy> ++
17:42:06 <cmurphy> alright thanks everyone
17:42:11 <cmurphy> #endmeeting