Tuesday, 2020-07-28

*** jamesmcarthur has joined #openstack-meeting-alt00:12
*** diurnalist has quit IRC00:16
*** knomura has joined #openstack-meeting-alt00:45
*** diurnalist has joined #openstack-meeting-alt00:46
*** portdirect has quit IRC00:51
*** gmann has quit IRC00:51
*** gmann has joined #openstack-meeting-alt00:51
*** portdirect has joined #openstack-meeting-alt00:51
*** amito has quit IRC00:51
*** amito has joined #openstack-meeting-alt00:51
*** rfolco has quit IRC00:58
*** jamesmcarthur has quit IRC01:04
*** knomura has quit IRC01:07
*** tetsuro has joined #openstack-meeting-alt01:13
*** ricolin has joined #openstack-meeting-alt01:21
*** jamesmcarthur has joined #openstack-meeting-alt01:37
*** baojg has quit IRC01:38
*** knomura has joined #openstack-meeting-alt01:38
*** baojg has joined #openstack-meeting-alt01:41
*** yaawang has quit IRC01:45
*** yaawang has joined #openstack-meeting-alt01:46
*** jamesmcarthur has quit IRC01:48
*** diurnalist has quit IRC02:24
*** tetsuro has quit IRC02:45
*** ircuser-1 has joined #openstack-meeting-alt02:46
*** jamesmcarthur has joined #openstack-meeting-alt02:51
*** rcernin has quit IRC02:59
*** hongbin has joined #openstack-meeting-alt03:00
*** rcernin has joined #openstack-meeting-alt03:04
*** jamesmcarthur has quit IRC03:15
*** tetsuro has joined #openstack-meeting-alt03:51
*** tetsuro has quit IRC04:01
*** tetsuro has joined #openstack-meeting-alt04:02
*** tetsuro has quit IRC04:06
*** gyee has quit IRC04:10
*** hongbin has quit IRC04:20
*** vishalmanchanda has joined #openstack-meeting-alt04:22
*** diurnalist has joined #openstack-meeting-alt04:31
*** diurnalist has quit IRC04:35
*** geguileo has quit IRC05:54
*** geguileo has joined #openstack-meeting-alt05:55
*** lpetrut has joined #openstack-meeting-alt06:04
*** tetsuro has joined #openstack-meeting-alt06:06
*** tetsuro has quit IRC06:11
*** baojg has quit IRC06:15
*** diurnalist has joined #openstack-meeting-alt06:43
*** slaweq has joined #openstack-meeting-alt06:48
*** diurnalist has quit IRC06:48
*** e0ne has joined #openstack-meeting-alt06:52
*** baojg has joined #openstack-meeting-alt07:21
*** rcernin has quit IRC07:32
*** diurnalist has joined #openstack-meeting-alt07:41
*** tosky has joined #openstack-meeting-alt07:42
*** diurnalist has quit IRC07:46
*** ralonsoh has joined #openstack-meeting-alt07:53
*** derekh has joined #openstack-meeting-alt08:41
*** rcernin has joined #openstack-meeting-alt08:49
*** priteau has joined #openstack-meeting-alt08:50
*** rcernin has quit IRC08:54
*** diurnalist has joined #openstack-meeting-alt08:56
*** tetsuro has joined #openstack-meeting-alt08:58
priteau#startmeeting blazar09:00
openstackMeeting started Tue Jul 28 09:00:19 2020 UTC and is due to finish in 60 minutes.  The chair is priteau. Information about MeetBot at http://wiki.debian.org/MeetBot.09:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.09:00
*** openstack changes topic to " (Meeting topic: blazar)"09:00
openstackThe meeting name has been set to 'blazar'09:00
priteau#topic Roll call09:00
*** openstack changes topic to "Roll call (Meeting topic: blazar)"09:00
*** diurnalist has quit IRC09:00
tetsuroo/09:01
toskyo/09:01
priteauHi tetsuro and tosky09:01
priteauI guess since tosky is here, we should discuss:09:02
priteau#topic Victoria OpenStack-wide Goals09:02
*** openstack changes topic to "Victoria OpenStack-wide Goals (Meeting topic: blazar)"09:02
toskyyep09:02
toskyI managed to understand the reason for the failure of https://review.opendev.org/#/c/732694/: I totally missed the statement which reduces the concurrency of the tests to 109:03
priteauAh, that totally explains it09:03
toskyit seems those tests need to be run in serial mode, but anyway the statement was there in the original job and I missed it >.<09:03
toskyit seems to work, but please double check09:03
priteauThe logs looked like multiple tests were running in parallel09:03
*** masahito has joined #openstack-meeting-alt09:04
priteauYes, our tests are meant to run sequentially because we have only one compute node that we can reserve09:04
priteauHi masahito09:04
masahitohello09:04
toskyif that works as expected and it's merged, then the next step is https://review.opendev.org/#/c/732697/, whose only error is openstack-tox-py3809:04
toskyand I would say that failure is totally unrelated to the zuul patch09:04
priteauopenstack-tox-py38 was passing when you first submitted the patch09:04
toskyI can try to recheck but it may be something else and I don't know that code too much09:05
toskybut the error, "Working outside of request context." looks familiar09:06
toskyI think I have seen a similar error in another project09:06
priteauMaybe our tests are not using Flask as intended09:06
priteauThere are docs about it: https://flask.palletsprojects.com/en/1.1.x/reqcontext/#manually-push-a-context09:06
toskyoh, right, in sahara: https://storyboard.openstack.org/#!/story/200759609:07
toskywe had to rewrite the way the tests were written, maybe this may help: https://review.opendev.org/#/c/724675/09:07
priteauhttps://review.opendev.org/#/c/724675/09:07
*** ricolin has quit IRC09:07
toskyso something that should be fixed regardless09:08
priteauSounds like the right approach to follow09:08
toskyanyway, that's it from my side - the zuul part seems to be working, and if it passes your review, that should be it09:08
priteauThanks a lot for your perseverance tosky09:10
priteauThe other Victoria priority is migrate-to-focal which is underway, we've already merged a few patches09:11
priteauSo, please review tosky's patch at https://review.opendev.org/#/c/732694/09:11
toskywith the native jobs you won't need any other change on the zuul side to switch to focal - only fix the job failures in case of packaging changes, for example09:12
priteauGreat09:13
priteauAnyone would like to look into the py38 issues we just discussed?09:14
priteauLet's record it as an action09:16
priteau#action Fix py38 issues in https://review.opendev.org/#/c/732697/ following what was done for Sahara in https://review.opendev.org/#/c/724675/09:17
priteau#topic Other Victoria priorities09:17
*** openstack changes topic to "Other Victoria priorities (Meeting topic: blazar)"09:17
priteauFirst, specs09:18
priteauI had an action item to create a new draft spec for PCI passthrough09:18
priteauSorry, I haven't been able to do this yet, I'll aim for next week09:18
priteauAnother code review priority is this patch: https://review.opendev.org/#/c/731586/09:19
priteauAnything else to highlight?09:21
priteauBy the way we are at Victoria-2 milestone this week09:25
priteauLet's focus on the highest priorities during August09:25
priteau#topic AOB09:25
*** openstack changes topic to "AOB (Meeting topic: blazar)"09:25
priteauAnything else?09:25
priteauHoliday schedules, etc?09:26
priteauI was off earlier this month but should be working every week for the rest of the summer09:27
tetsuroI'm going to be off on Aug. 1409:27
tetsurooh no I meant Aug. 11 Tue09:28
priteauThanks for letting us know tetsuro09:28
priteauLet's wrap up09:29
priteauThanks everyone for joining today09:29
priteauTalk to you next Tuesday09:29
priteau#endmeeting09:29
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"09:29
openstackMeeting ended Tue Jul 28 09:29:58 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)09:30
openstackMinutes:        http://eavesdrop.openstack.org/meetings/blazar/2020/blazar.2020-07-28-09.00.html09:30
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/blazar/2020/blazar.2020-07-28-09.00.txt09:30
openstackLog:            http://eavesdrop.openstack.org/meetings/blazar/2020/blazar.2020-07-28-09.00.log.html09:30
*** tetsuro has quit IRC09:32
*** tetsuro has joined #openstack-meeting-alt09:40
*** rdopiera has joined #openstack-meeting-alt09:46
*** tetsuro has quit IRC09:48
*** tetsuro has joined #openstack-meeting-alt09:51
*** tetsuro has quit IRC09:53
*** rcernin has joined #openstack-meeting-alt10:20
*** masahito has quit IRC10:23
*** vishakha has joined #openstack-meeting-alt10:26
*** rcernin has quit IRC10:43
*** diurnalist has joined #openstack-meeting-alt11:38
*** diurnalist has quit IRC11:42
*** lbragstad has quit IRC11:45
*** ricolin has joined #openstack-meeting-alt11:47
*** priteau has quit IRC11:55
*** raildo has joined #openstack-meeting-alt11:56
*** priteau has joined #openstack-meeting-alt11:57
*** rfolco has joined #openstack-meeting-alt12:02
*** jamesmcarthur has joined #openstack-meeting-alt12:16
*** diurnalist has joined #openstack-meeting-alt12:19
*** diurnalist has quit IRC12:24
*** ralonsoh has quit IRC12:34
*** ralonsoh has joined #openstack-meeting-alt12:34
*** lbragstad has joined #openstack-meeting-alt12:50
*** diurnalist has joined #openstack-meeting-alt13:01
*** enriquetaso has joined #openstack-meeting-alt13:03
*** diurnalist has quit IRC13:05
*** knomura has quit IRC13:17
*** jamesmcarthur has quit IRC13:18
*** baojg has quit IRC13:20
*** baojg has joined #openstack-meeting-alt13:21
*** geguileo has quit IRC14:15
*** geguileo has joined #openstack-meeting-alt14:16
*** diurnalist has joined #openstack-meeting-alt14:21
*** diurnalist has quit IRC14:26
*** jamesmcarthur has joined #openstack-meeting-alt14:30
*** lpetrut has quit IRC14:40
*** diurnalist has joined #openstack-meeting-alt14:49
*** sangeet has joined #openstack-meeting-alt15:00
gagehugo#startmeeting openstack-helm15:01
openstackMeeting started Tue Jul 28 15:01:04 2020 UTC and is due to finish in 60 minutes.  The chair is gagehugo. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
*** openstack changes topic to " (Meeting topic: openstack-helm)"15:01
openstackThe meeting name has been set to 'openstack_helm'15:01
gagehugo#link https://etherpad.opendev.org/p/openstack-helm-weekly-meeting agenda15:01
lamto/15:01
*** jamesmcarthur has quit IRC15:02
stevthedevo/15:03
gagehugonothing on the agenda for this week so15:04
gagehugo#topic open discussion15:04
*** openstack changes topic to "open discussion (Meeting topic: openstack-helm)"15:04
gagehugoDoes anyone have anything they want to discuss or reviews?15:04
*** andrii_ostapenko has joined #openstack-meeting-alt15:06
megheislerThe only thing worth mentioning right now is that the call for presentations for the virtual Open Infra summit in October is open15:08
gagehugoThat ends in a week right?15:08
megheisleryes15:09
gagehugogood to know15:10
gagehugothanks everyone15:13
gagehugo#endmeeting15:13
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"15:13
openstackMeeting ended Tue Jul 28 15:13:38 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:13
openstackMinutes:        http://eavesdrop.openstack.org/meetings/openstack_helm/2020/openstack_helm.2020-07-28-15.01.html15:13
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/openstack_helm/2020/openstack_helm.2020-07-28-15.01.txt15:13
openstackLog:            http://eavesdrop.openstack.org/meetings/openstack_helm/2020/openstack_helm.2020-07-28-15.01.log.html15:13
*** baojg has quit IRC15:13
*** sangeet has left #openstack-meeting-alt15:13
*** vishakha has quit IRC15:18
*** baojg has joined #openstack-meeting-alt15:25
*** baojg has quit IRC15:26
*** baojg has joined #openstack-meeting-alt15:27
*** knomura has joined #openstack-meeting-alt15:29
*** baojg has quit IRC15:33
*** knomura has quit IRC15:34
*** priteau has quit IRC16:08
*** knomura has joined #openstack-meeting-alt16:17
*** vishakha has joined #openstack-meeting-alt16:41
knikolla#startmeeting keystone16:59
openstackMeeting started Tue Jul 28 16:59:40 2020 UTC and is due to finish in 60 minutes.  The chair is knikolla. Information about MeetBot at http://wiki.debian.org/MeetBot.16:59
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:59
*** openstack changes topic to " (Meeting topic: keystone)"16:59
openstackThe meeting name has been set to 'keystone'16:59
cmurphyo/16:59
knikollao/16:59
*** derekh has quit IRC17:00
lbragstado/17:00
vishakhao/17:01
gagehugoo/17:01
*** knomura has quit IRC17:02
knikollacool, we can get started.17:02
knikolla#topic Review Requests17:02
*** openstack changes topic to "Review Requests (Meeting topic: keystone)"17:02
knikollaI'm way behind on reviews, I apologize.17:02
toskyif you want to remove keystone from the list of projects which still need to clean the remaining zuul legacy jobs, please take a look at https://review.opendev.org/#/c/733801/ :)17:08
knikollacopying from vishakha from the #openstack-keystone room17:08
knikolla1:03 PM <vishakha> Seems like keystone gate is broken , I pushed a fix for that #link #link https://review.opendev.org/#/c/742956/ (Keystone gate fix)17:08
knikolla1:03 PM <vishakha> #link https://review.opendev.org/#/c/737225/ (Targeted for Victoria-Milestone-2),17:08
knikolla1:05 PM <vishakha> and some other #link https://review.opendev.org/#/c/743489/  #link https://review.opendev.org/#/c/739784/17:08
knikolla1:05 PM <vishakha>  #link https://review.opendev.org/#/c/742233/ (NIT)17:08
knikolla1:05 PM <vishakha> #link https://review.opendev.org/#/c/731087/17:08
vishakhaSorry I didnt realize I pasted on the wrong chaneel17:09
knikollathanks tosky, i pushed that through17:10
toskythank you! I will propose a backport to cover at least ussuri, but that's out of goal scope, so less priority17:11
knikollaok, cool17:11
knikollagrazie17:11
knikolla#topic Bugs17:15
*** openstack changes topic to "Bugs (Meeting topic: keystone)"17:15
knikollawith regards to https://bugs.launchpad.net/keystone/+bug/169349817:18
openstackLaunchpad bug 1693498 in OpenStack Identity (keystone) "Credential list API returns list of available credentials when user passes invalid name as query parameter" [Undecided,In progress] - Assigned to Vishakha Agarwal (vishakha.agarwal)17:18
vishakha#link https://bugs.launchpad.net/keystone/+bug/1693498 As per this bug, the bug reportee expects the output to be empty list when passing wrong filters in the CURL request, but in keystone the functionality is to ignore the invalid filters in def build_driver_hints https://github.com/openstack/keystone/blob/dc68ee48160e8ad9b637f3d91b18a42dbfdf012b/keystone/server/flask/common.py#L82017:18
vishakhawhich results giving the whole list.17:19
knikollais the report about the wrong filter query key, or the wrong value?17:19
cmurphyi think it's about the key17:20
vishakhaYes its about the keys "In my opinion to maintain the consistency, Credential list API should return empty list when invalid query parameter is passed"17:21
knikollai have concerns about suddenly switching our API behavior to return 400 on wrong key.17:21
cmurphyknikolla: ++17:21
vishakhaWe test the behavior too https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_v3_filters.py#L18717:21
knikollain that case i would say this is a wontfix, unless we have microversions.17:22
lbragstadhttps://bugs.launchpad.net/keystone/+bug/165408417:22
openstackLaunchpad bug 1654084 in OpenStack Identity (keystone) "Listing resources with invalid filters should result in a 400" [Wishlist,Triaged]17:22
knikollathanks lbragstad, that provides more context.17:23
cmurphyhttps://bugs.launchpad.net/keystone/+bug/1654084/comments/717:23
lbragstadsounds like we can close both?17:23
cmurphyi would add priority "wishlist" and tag "needs microversion" or whatever17:24
knikollawe can keep it around as wishlist and tagged as need microversion17:24
cmurphywe have a few others like that17:24
knikollacmurphy: ++17:24
vishakhaSHould I update these that we need microversion?17:25
cmurphydone17:25
lbragstadcould we consolidate them?17:26
lbragstador do we have a reason to keep both open17:26
knikollawe can mark as duplicate17:26
knikollathe bug that lbragstad link provides a lot more context17:26
cmurphy++17:26
lbragstadi think if we were to do this, we should do it consistently across the entire api17:26
lbragstadand not just the the credential api17:27
knikollayes, and across other openstack apis17:27
knikolla(if there is such a consistency in the first place)17:27
cmurphymarked as duplicate17:27
knikollathanks cmurphy17:27
knikolla#topic Open Floor17:29
*** openstack changes topic to "Open Floor (Meeting topic: keystone)"17:29
vishakha#link https://review.opendev.org/#/c/738190/17:29
vishakhaI wanted to discuss about this change.17:30
vishakhaAs per the current behavior in keystone while listing system role assignments with filter role_id, it doesn't display the implied role assignments17:31
vishakhaand I not sure that originally if it was displaying because after the change #link https://review.opendev.org/#/c/700826, it only displays the role assignments that matches the role_id17:34
cmurphymaybe https://bugs.launchpad.net/keystone/+bug/1858012 is a duplicate of https://bugs.launchpad.net/keystone/+bug/1846817 ? i had completely forgotten about 185801217:36
openstackLaunchpad bug 1858012 in OpenStack Identity (keystone) "List role assignments by role ID may leak extra system assignments outside of filter" [Undecided,Fix released] - Assigned to Colleen Murphy (krinkle)17:36
openstackLaunchpad bug 1846817 in OpenStack Identity (keystone) "v3/role_assignments filtering exposes unnecessary role assignments" [Medium,In progress] - Assigned to Vishakha Agarwal (vishakha.agarwal)17:36
knikollaah, i see.17:37
cmurphylbragstad: any thoughts on whether https://bugs.launchpad.net/keystone/+bug/1846817 is still applicable? i wasn't able to reproduce the bug as i understood it from the report17:38
openstackLaunchpad bug 1846817 in OpenStack Identity (keystone) "v3/role_assignments filtering exposes unnecessary role assignments" [Medium,In progress] - Assigned to Vishakha Agarwal (vishakha.agarwal)17:38
knikollaso one piece of the bug was the leaking of more role assignment, and the other piece is the implied roles.17:39
knikollai think the first one was fixed by your patch to the other bug.17:39
cmurphyno, i don't think implied roles was ever a part of it17:39
lbragstadchecking17:39
knikollaper bug description "If I ask keystone to filter a list of role assignments by --system all and --role member, I should only see a list with the "system-support" user and the "operator" user."17:39
knikollathe operator user would be included becase admin implies member.17:40
cmurphyyes but in that description operator is included, the problem is that reader is also included17:41
knikollawhich feels like the extraneous entries which your patch fixed.17:41
cmurphyright17:41
*** redrobot has quit IRC17:41
vishakhaMy only concern is do we want to display the implied roles as per the bug description?17:42
vishakhaBecause till now we weren't17:42
cmurphyvishakha: is https://review.opendev.org/700826 what changed that behavior of the implied roles?17:43
cmurphyit is concerning that that changed somehow17:43
knikollawe already do implied roles with the "effective" query parameter, IIRC.17:43
knikollai don't think it's automatic.17:43
knikollacan you try with "&effective" and see if you get the implied roles as well?17:44
cmurphyit seemed to have been automatic in the output given in that bug report but the behavior since seemed to have changed to exclude them17:44
* lbragstad is recreating the bug now 17:44
cmurphyregardless i think behavior of implied roles is a different bug17:45
knikollaas per our api reference "If the query parameter effective is specified, rather than simply returning a list of role assignments that have been made, the API returns a list of effective assignments at the user, project and domain level, having allowed for the effects of group membership, role inference rules as well as inheritance from the parent domain or project."17:45
knikollathe API says that it shouldn't have been automatic.17:46
lbragstadi don't think i can recreate https://bugs.launchpad.net/keystone/+bug/184681717:46
openstackLaunchpad bug 1846817 in OpenStack Identity (keystone) "v3/role_assignments filtering exposes unnecessary role assignments" [Medium,In progress] - Assigned to Vishakha Agarwal (vishakha.agarwal)17:46
cmurphyknikolla: so whatever changed changed it to be correct i guess?17:46
knikollai guess so17:47
knikollaand if it works correctly with "effective", we're where we want to be.17:47
lbragstadhttp://paste.openstack.org/raw/796387/17:47
knikollalbragstad: that looks the correct default to me.17:49
lbragstadso - something must have fixed that17:49
*** ralonsoh has quit IRC17:49
vishakhaSo this is the right behavior and we dont need to display the implied roles until "effective" is passed?(I still need to check the "effective" filter)17:52
knikollavishakha: correct.17:52
vishakhaBug is duplicate in such case. And I can update in the comment section about the implied roles?17:53
knikollayes17:54
cmurphythanks for tracking down the other bug vishakha17:54
knikollayes, thank you :)17:54
vishakhayw :)17:54
knikollamy lunch is here, so if there's nothing else, we can have a few minutes back.17:56
lbragstadthanks all17:56
knikolla#endmeeting keystone17:57
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"17:57
openstackMeeting ended Tue Jul 28 17:57:14 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:57
openstackMinutes:        http://eavesdrop.openstack.org/meetings/keystone/2020/keystone.2020-07-28-16.59.html17:57
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/keystone/2020/keystone.2020-07-28-16.59.txt17:57
openstackLog:            http://eavesdrop.openstack.org/meetings/keystone/2020/keystone.2020-07-28-16.59.log.html17:57
*** e0ne has quit IRC18:27
*** knomura has joined #openstack-meeting-alt19:14
*** knomura has quit IRC19:19
*** e0ne has joined #openstack-meeting-alt19:23
*** tosky has quit IRC19:25
*** vishakha has quit IRC20:16
*** vishalmanchanda has quit IRC20:41
*** e0ne has quit IRC20:42
*** rdopiera has quit IRC20:43
*** gyee has joined #openstack-meeting-alt21:02
*** e0ne has joined #openstack-meeting-alt21:20
*** raildo has quit IRC21:23
*** slaweq has quit IRC21:34
*** redrobot has joined #openstack-meeting-alt21:43
*** e0ne has quit IRC22:08
*** rcernin has joined #openstack-meeting-alt22:10
*** knomura has joined #openstack-meeting-alt22:14
*** knomura has quit IRC22:18
*** rcernin has quit IRC22:22
*** geguileo has quit IRC22:36
*** geguileo has joined #openstack-meeting-alt22:37
*** rcernin has joined #openstack-meeting-alt22:37
*** e0ne has joined #openstack-meeting-alt22:48
*** ircuser-1 has quit IRC22:48
*** rcernin has quit IRC22:50
*** rcernin has joined #openstack-meeting-alt22:50
*** e0ne has quit IRC22:59
*** ircuser-1 has joined #openstack-meeting-alt23:05
*** diurnalist has quit IRC23:52

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