Thursday, 2021-07-01

*** akekane_ is now known as abhishekk06:32
*** slaweq_ is now known as slaweq08:29
*** Steap_ is now known as croelandt13:40
abhishekk#startmeeting glance14:00
opendevmeetMeeting started Thu Jul  1 14:00:37 2021 UTC and is due to finish in 60 minutes.  The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'glance'14:00
croelandto/14:00
abhishekk#topic roll call14:00
abhishekk#link https://etherpad.openstack.org/p/glance-team-meeting-agenda14:00
dansmitho~14:00
abhishekko/14:00
rosmaitao/ (though i am not really paying attention)14:01
abhishekklets wait couple of minutes for others14:01
croelandtwait we're supposed to pay attention?14:01
croelandtthis job keeps getting harder14:01
jokke_o/14:01
abhishekkcool14:02
abhishekklets start14:02
abhishekk#topic Updates14:02
abhishekkVPTG is announced - October 18-22, 2021!14:02
abhishekk#link https://openinfra-ptg.eventbrite.com14:03
abhishekkRegistration is open as well14:03
abhishekk#topic release/periodic jobs update14:03
abhishekkM2 2 weeks from now14:03
abhishekkSame goes for spec freez14:03
abhishekkPolicy refactoring spec is waiting for reviews, kindly have a look (it already has 2 +2s and 1 -1)14:04
abhishekkWe need to get it merged in 2 weeks14:04
abhishekkPeriodic jobs, all green, so we haven't broke anything yet :D14:05
abhishekk#topic M2 Target progress check14:05
abhishekkUnified quota spec14:05
abhishekkGood progress and most of the patches are merged14:05
abhishekkwe have docs and releasenote up as well14:06
jokke_I got distracted from the cache api work, should have patch up today14:06
abhishekkack14:06
abhishekkFor cache-api glanceclient spec needs reviews so that it can be merged14:06
abhishekkI think we should move client support for cache API to M314:07
jokke_kk14:07
abhishekkand only target server side changes in this milestone 14:07
abhishekkmoving ahead14:07
abhishekk#topic Policy refactoring14:07
abhishekkMaster spec: https://review.opendev.org/c/openstack/glance-specs/+/79675314:08
abhishekkThere are some concerns raised by jokke_ and provided answers as well14:08
abhishekkjokke_, kindly have a look to get it moving14:08
abhishekkTests refactoring lite spec: https://review.opendev.org/c/openstack/glance-specs/+/797593/114:08
abhishekkPatch for this change is already in gate, I will move this dependency on the policy refactoring spec so that we can get this merged14:09
abhishekkAuthorization layer and its use?14:09
abhishekkI am seeing most of the concerns are about authorization layer14:09
abhishekkAFAIK we have similar kind of checks in db layer as well14:10
abhishekkI am going to write on lite spec which will highlight how we are going to move out of authorization layer14:10
dansmithare you thinking about the hard-coded "is admin or owner" checks on db update?14:10
abhishekkwe already have it on delete14:11
abhishekkso similar we can have on update as well, know?14:11
jokke_dansmith: yeah update and destroy are covered on the sqlalhemy api14:11
dansmithyeah, this: https://github.com/openstack/glance/blob/master/glance/db/sqlalchemy/api.py#L13514:12
abhishekkyeah14:12
dansmiththat's obviously going to have to change in order to do secure rbac14:12
abhishekkLets keep secure rbac out as of now14:13
abhishekkSo AFAIK we have these mutation checks in db for metadefs also14:14
abhishekkLets discuss this on the lite-spec, I will try to have it up for review by tomorrow EOD14:15
abhishekkanything else? 14:16
abhishekkI take that as know, moving ahead14:17
abhishekk#topic Bi-weekly Bug discussion (4th Meet)14:17
abhishekkWhile analyzing metadef code for policy refactor related changes I found that couple of APIs does not have full client support14:17
abhishekkglanceclient has no support to add/specify description  while creating metadef object using md-object-create command.14:17
abhishekk#link https://docs.openstack.org/api-ref/image/v2/metadefs-index.html?expanded=create-object-detail,create-property-detail,create-namespace-detail#create-object14:18
abhishekkand second one is more critical 14:18
abhishekkglanceclient has no support to add 'type' while creating md-property for namespace using md-property-create command14:18
croelandtis there a bug open for these 2 issues?14:19
abhishekkSo type is required property for creating metadef properties14:19
abhishekkcroelandt, no, need to report it14:19
abhishekkI found these issues around midnight14:19
abhishekk#link http://paste.openstack.org/show/807083/14:20
abhishekkSo both these issues needs to be reported and fixed on priority14:20
abhishekkAlso I am wondering now usage of metadef APIs 14:21
jokke_I think that just tells quite obvious story how widely those commands are used ;)14:21
abhishekkexactly14:21
abhishekkour devstack uses glance-manage command to load the metadefs from json file14:22
abhishekkI am not sure how others are using it (or really using it)14:22
abhishekkThere might be more issues around metadefs, I will have more close look at the code14:23
abhishekkThat's it from me today14:23
abhishekkmoving to Open discussion14:23
abhishekk#topic Open discussion14:23
abhishekkNothing from me14:24
jokke_abhishekk: I think those two you mentioned above are great bugs to include in your list of low hanging fruit and not necessarily something we need to spend our time urgently14:24
abhishekkcroelandt, I have mentioned these issues in our bug tracker as well14:24
rajiv_mucheli_Hi jokke_ any update on the swift bugs i raised ?14:25
croelandtabhishekk: thanks14:25
croelandtjokke_: yeah, they could go into the "for interns" pile :D14:25
jokke_rajiv_mucheli_: I have revert of that change that broke the trusts in review waiting. Thanks for figuring that out14:26
abhishekkjokke_, I also thought about that, lets see14:26
jokke_rajiv_mucheli_: can move on with the rest once we get over that bump14:26
rajiv_mucheli_oh ok, we tried few code changes to fix bulk deletes but had no luck, would increasing the delete_concurrency in swift help ?14:27
jokke_rajiv_mucheli_: I'm not sure. Either that or we need to start throttling the bulk deletes in Glance side to give swift room to get them done.14:28
jokke_rajiv_mucheli_: I think the delete_concurrency is good place to start as in general swift is very good with lot of parallel operations and performs better that way than serialized14:29
jokke_But I'm not sure how far that needs to be stretched if we're talking about 400GiB NFV image or something, so we might need to throttle it anyways from our side14:30
abhishekkanything else guys otherwise we can use remaining time in reviews14:32
rajiv_mucheli_okay, i will try this14:32
abhishekkcool, lets warp this up14:33
abhishekkthank you all!14:33
abhishekkhave a nice weekend ahead14:33
abhishekk#endmeeting14:34
opendevmeetMeeting ended Thu Jul  1 14:34:09 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:34
opendevmeetMinutes:        https://meetings.opendev.org/meetings/glance/2021/glance.2021-07-01-14.00.html14:34
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/glance/2021/glance.2021-07-01-14.00.txt14:34
opendevmeetLog:            https://meetings.opendev.org/meetings/glance/2021/glance.2021-07-01-14.00.log.html14:34
gagehugo#startmeeting security15:00
opendevmeetMeeting started Thu Jul  1 15:00:14 2021 UTC and is due to finish in 60 minutes.  The chair is gagehugo. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'security'15:00
gagehugo#link https://etherpad.opendev.org/p/security-agenda agenda15:00
gagehugoo/15:00
fungiohai15:01
gagehugohi15:02
fungii stuck something on the agenda in case we don't have anything else to talk about15:02
gagehugoI hope everything is going alright over there15:03
gagehugoI saw, it's an interesting read15:03
gagehugo#link https://github.com/irsl/gcp-dhcp-takeover-code-exec15:03
gagehugoThe only topic I had was related to the discussions about moving the irc meetings to their respective channels15:06
gagehugoIs that something we could do? Move the weekly meeting to #openstack-security?15:06
fungiwe can if you want, the channel is very low traffic already anyway15:07
fungii have no real preference15:07
gagehugoyeah15:07
fungii'm in both channels so it doesn't really matter to me which we use15:08
gagehugoPlus I figured that frees us up for more timeslots too if we want to move the meeting15:08
gagehugoI know this is the tc meeting time too15:08
gagehugoand I have a work meeting I'm always double booked in15:08
gagehugoso that may be the next step15:08
fungiyep, same slot as the tc meeting15:10
gagehugook, I'll make the modifications then to move the meeting channel, then we can talk about moving the meeting time15:11
gagehugoprobably in the ML15:11
fungicool15:15
fungithanks!15:15
gagehugoDo you have anything else you want to discuss?15:15
funginah, just wanted to bring up the gcp dhcp vulnerability as the sort of thing we should be keeping an eye out for, even though i don't think openstack deployments are specifically vulnerable to that exactly15:16
gagehugoOk15:20
gagehugoThanks fungi!15:20
gagehugo#endmeeting15:21
opendevmeetMeeting ended Thu Jul  1 15:21:05 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:21
opendevmeetMinutes:        https://meetings.opendev.org/meetings/security/2021/security.2021-07-01-15.00.html15:21
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/security/2021/security.2021-07-01-15.00.txt15:21
opendevmeetLog:            https://meetings.opendev.org/meetings/security/2021/security.2021-07-01-15.00.log.html15:21
fungithanks as always, gagehugo!15:21

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