Thursday, 2020-01-23

*** diablo_rojo has joined #openstack-meeting-400:03
*** anastzhyr has quit IRC00:07
*** spiette has quit IRC00:25
*** spiette has joined #openstack-meeting-400:29
*** k_mouza has quit IRC00:51
*** yamamoto has joined #openstack-meeting-400:54
*** yamamoto has quit IRC01:12
*** yamamoto has joined #openstack-meeting-401:48
*** yamamoto has quit IRC01:58
*** yamamoto has joined #openstack-meeting-402:00
*** yamamoto has quit IRC02:31
*** yamamoto has joined #openstack-meeting-402:31
*** roman_g has quit IRC02:34
*** enriquetaso has quit IRC02:38
*** yamamoto has quit IRC02:40
*** psachin has joined #openstack-meeting-403:36
*** rihabb252 has quit IRC03:51
*** rihabb252 has joined #openstack-meeting-403:52
*** macz has joined #openstack-meeting-405:00
*** macz has quit IRC05:05
*** stevthedev has quit IRC05:28
*** stevthedev has joined #openstack-meeting-405:28
*** evrardjp has quit IRC05:34
*** evrardjp has joined #openstack-meeting-405:34
*** links has joined #openstack-meeting-405:56
*** k_mouza has joined #openstack-meeting-406:53
*** k_mouza has quit IRC06:57
*** roman_g has joined #openstack-meeting-407:06
*** Luzi has joined #openstack-meeting-407:21
*** e0ne has joined #openstack-meeting-407:53
*** Adri2000 has quit IRC07:59
*** fungi has quit IRC07:59
*** fnordahl has quit IRC07:59
*** Roamer` has quit IRC07:59
*** toanster has quit IRC07:59
*** Jeffrey4l has quit IRC07:59
*** cmurphy has quit IRC07:59
*** zhipeng[m] has quit IRC08:01
*** e0ne has quit IRC08:02
*** Jeffrey4l has joined #openstack-meeting-408:02
*** cmurphy has joined #openstack-meeting-408:02
*** irclogbot_1 has quit IRC08:04
*** irclogbot_3 has joined #openstack-meeting-408:05
*** bnemec has joined #openstack-meeting-408:05
*** dansmith has quit IRC08:06
*** dansmith has joined #openstack-meeting-408:08
*** bnemec has quit IRC08:41
*** Adri2000 has joined #openstack-meeting-408:45
*** fungi has joined #openstack-meeting-408:45
*** fnordahl has joined #openstack-meeting-408:45
*** Roamer` has joined #openstack-meeting-408:45
*** toanster has joined #openstack-meeting-408:45
*** bnemec has joined #openstack-meeting-408:50
*** bnemec has quit IRC09:08
*** pcaruana has joined #openstack-meeting-409:30
*** bnemec has joined #openstack-meeting-409:33
*** Liang__ has joined #openstack-meeting-409:43
*** k_mouza has joined #openstack-meeting-409:44
*** k_mouza has quit IRC09:48
*** e0ne has joined #openstack-meeting-409:50
*** k_mouza has joined #openstack-meeting-409:54
*** k_mouza has quit IRC10:18
*** k_mouza has joined #openstack-meeting-410:19
*** k_mouza has quit IRC10:34
*** dviroel has joined #openstack-meeting-410:41
*** k_mouza has joined #openstack-meeting-410:44
*** k_mouza has quit IRC10:48
*** k_mouza has joined #openstack-meeting-410:49
*** bnemec has quit IRC11:02
*** k_mouza has quit IRC11:11
*** k_mouza has joined #openstack-meeting-411:13
*** k_mouza has quit IRC11:27
*** k_mouza has joined #openstack-meeting-411:27
*** Liang__ has quit IRC11:28
*** k_mouza has quit IRC11:29
*** k_mouza has joined #openstack-meeting-411:33
*** zhipeng[m] has joined #openstack-meeting-412:20
*** bnemec has joined #openstack-meeting-412:29
*** bobmel has quit IRC12:57
*** bobmel has joined #openstack-meeting-412:59
*** yamamoto has joined #openstack-meeting-413:12
*** bobmel has quit IRC13:13
*** bobmel has joined #openstack-meeting-413:13
*** k_mouza has quit IRC13:17
*** yamamoto has quit IRC13:19
*** yamamoto has joined #openstack-meeting-413:21
*** belmoreira has joined #openstack-meeting-413:22
*** enriquetaso has joined #openstack-meeting-413:31
*** abhishekk has joined #openstack-meeting-413:50
*** rosmaita has joined #openstack-meeting-413:54
*** yebinama_ has joined #openstack-meeting-413:56
*** tosky has joined #openstack-meeting-413:58
abhishekk#startmeeting glance14:00
openstackMeeting started Thu Jan 23 14:00:15 2020 UTC and is due to finish in 60 minutes.  The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: glance)"14:00
abhishekk#topic roll call14:00
openstackThe meeting name has been set to 'glance'14:00
*** openstack changes topic to "roll call (Meeting topic: glance)"14:00
jokke_o/14:00
abhishekk#link https://etherpad.openstack.org/p/glance-team-meeting-agenda14:00
rosmaitao/14:00
abhishekko/14:00
yebinama_o/14:00
abhishekkCool14:00
abhishekklets start14:00
abhishekk#topic release/periodic jobs update14:01
*** openstack changes topic to "release/periodic jobs update (Meeting topic: glance)"14:01
abhishekkWe still have 1 failure in periodic job which is causing because of oslo_config new version14:01
abhishekk#link https://review.opendev.org/70158014:02
abhishekkI have added workaround on this patch but there was no response from the owner14:02
abhishekkSo workaround sounds good to you people I will go ahead and submit this patch14:03
*** k_mouza has joined #openstack-meeting-414:03
abhishekkKindly comment on the patch about the same, moving ahead14:03
abhishekkUssuri milestone 2 - 4 weeks to go14:03
abhishekkWe have 3 specs targeted to milestone 214:04
abhishekkSo kindly focus on reviews, I don't want to miss this deadline14:04
abhishekkThis Monday we have released stable train and stein version14:05
abhishekkany comments so far?14:05
abhishekkcool14:06
abhishekk#topic Multiple store import plugins14:06
*** openstack changes topic to "Multiple store import plugins (Meeting topic: glance)"14:06
abhishekkmultiple import impl14:06
abhishekkyebinama_, you want to talk about this?14:06
yebinama_I've added the notification part14:06
abhishekkI guess except documentation and releasenote everything is covered14:07
abhishekkyebinama_, yes I saw that, looks good to me14:07
jokke_Are the specs finally green?14:07
yebinama_Yep only these are missing14:07
abhishekk#link https://review.opendev.org/66713214:07
abhishekkjokke_, yes14:07
abhishekkjokke_, rosmaita kindly review the patch ^^^14:08
yebinama_Should I create a new doc.rst for multi store import?14:08
abhishekkyebinama_, I guess you could add in existing file only14:08
jokke_++14:09
abhishekkI have added info about copying stuff it in existing file14:09
yebinama_Ok fine14:09
jokke_for adding the details to the existing import docs14:09
abhishekkrefer my patch for the same14:09
yebinama_Ok thanks14:09
abhishekkyebinama_, you also need to modify api-ref guide as well14:09
abhishekkCopying existing image impl14:09
abhishekk#link https://review.opendev.org/69645714:10
abhishekkSo regarding this patch only release note is remaining14:10
abhishekkI have added functional test to cover the revert logic as well14:10
yebinama_abhishekk: I'll work on addind this14:10
abhishekkwith this test we are able to verify that on failure we are deleting image data from only those stores where it was copied and not from the existing stores14:11
abhishekkyebinama_, ack14:11
rosmaitathat sounds like a good test!14:11
abhishekkHopefully14:11
yebinama_Great I'll take a look14:12
abhishekkI have written two test for these scenario with considering 'all_stores_must_succeed' parameter14:12
abhishekkAs copying patch is dependent on yebinama_'s import patch, we need to first put priority on reviewing and merging his patch14:13
abhishekkI have tested his patch functionally with 3 file and one ceph backend14:13
yebinama_And just to let you know I'll be on holidays starting 5 of february14:14
rosmaitaplease remove "WIP" and "PoC' from the commit messages on those patches, that's why i have been ignoring them14:14
abhishekkrosmaita, ack14:14
abhishekkyebinama_, so we need to make sure we can complete all the work before that and get this merged14:14
yebinama_rosmaita: ok i'll update the commit14:14
rosmaitaty14:14
abhishekkIn case if some review comments come then should I fix those on your behalf?14:15
yebinama_abhishekk: I'll try to bring my laptop so I should b3 able to work on it14:15
abhishekkyebinama_, cool14:15
abhishekkjokke_, rosmaita need your expert opinions on the patches14:16
abhishekkMoving ahead14:16
abhishekk#topic Delete image from single store14:16
*** openstack changes topic to "Delete image from single store (Meeting topic: glance)"14:16
jokke_We can skip this topic and move on14:17
abhishekkI saw jokke_ has abandoned the patch, any particular reason14:17
abhishekkjokke_, at some point we need to work on this14:18
jokke_It was clear to me that we won't be getting into agreement between what rosmaita wanted nd how far I was willing to push the consistencies with the rest of our API14:18
rosmaitaall i asked for was to fix some typos!14:19
abhishekkso what if instead of using location's pop method we use store delete method to delete the location?14:19
abhishekkthis will avoid race condition and all the fuss14:20
*** yamamoto has quit IRC14:20
abhishekkas we have store identifier in location metadata and we have store instance in memory, we can retrieve the same and call store delete method directly14:20
rosmaitasomehow the location needs to be deleted, and i believe location.remove() calls location.pop()14:20
*** alistarle has joined #openstack-meeting-414:21
jokke_mhm14:21
rosmaitathere is some really weird coupling in the code14:21
jokke_We got the race condition solved that wasn't the problem left14:21
abhishekkjokke_, I really don't want you to throw away your efforts14:21
rosmaitalet me say something here about the consistency issue and how we talk about stuff in docs and error messages14:22
rosmaitabefore multistore, the locations stuff was esoteric14:22
rosmaitanow, with the edge stuff we need to be very clear14:22
abhishekkWe can fix the typos and if required later revisit all multi-store related work to reword the help text14:22
abhishekkeven I have made mess at some places14:22
*** yamamoto has joined #openstack-meeting-414:23
rosmaitathat "Image" and "image data" are different14:23
rosmaitabefore, you could talk about them as pretty much the same14:23
rosmaitabut now we have the Image record and various stores where the image data is stored14:23
rosmaitaso we need to be really clear in talking about what's being removed/deleted14:23
*** heso has joined #openstack-meeting-414:23
rosmaitathat's why i'm insisting on the message changes about image data is being removed from store14:24
rosmaitai think the locations are an implementation detail14:24
rosmaitaso we don't need to discuss them14:25
abhishekkjokke_, please reconsider and restore the patch, its required otherwise we need to be depend on location API to remove the location14:25
abhishekkMoving ahead14:26
abhishekk#topic Open Discussion14:26
*** openstack changes topic to "Open Discussion (Meeting topic: glance)"14:26
abhishekkSo we need to focus on these three patches14:26
abhishekkjokke_, rosmaita, smcginnis kindly review these patches14:26
alistarleHi guy's, just a question about implementation of limits/quota in glance14:27
abhishekkalistarle, shoot14:27
smcginnisabhishekk: Which three patches need attention? The above topics?14:28
alistarleWe experience some capacity planning issue for image, so we wanted to implement some quota out there, we see that there is only a conf option about that14:28
abhishekksmcginnis, yes topic 2 and 3 from the agenda14:28
rosmaitaalistarle: yes, our "quotas" are really just hard limits14:28
alistarleAnd I see that Keystone implement the unified limit framework accross all openstack project, did you look at that ?14:29
smcginnisWill do.14:29
alistarleSeems that only nova are implementing it now, but maybe it is the time to implement it in glance no ?14:29
rosmaitaalistarle: we would be happy to review your code if you want to implement that14:29
abhishekksmcginnis, thank you14:29
alistarleYes I would like to work on that, seems the library oslo.limits must be used, but I don't understand some point of their framework14:30
abhishekkalistarle, also it will be great if you submit the glance-specs for the same14:30
alistarleSure14:30
smcginnisalistarle: I'm not sure if the unified limits work is complete. The last I read on it, nova was a first test case to help flush out the API. If it's ready now, it would be great to get some details about how to implement its use.14:30
abhishekkit will help us to understand the design and providing suggestions14:30
jokke_alistarle: I have not looked into it. Is that now something that the majority of OpenStack community finally got behind or is that something implemented in Keystone when the consensus was not reached? ;)14:31
alistarleYes that's what I was thinking, because it respond very well for enforcing /consulting limits, but not a all about how a user can consult it's own usage14:31
*** Luzi has quit IRC14:32
alistarleThe point is we needed quotas, so I think working on unified limits is better than trying to do something on our side14:32
smcginnisDefinitely agree it's about time we get to a common solution for this.14:34
abhishekk+114:34
jokke_alistarle: happy to hear that there is someone like you actually needing it and willing to work to make it happen. This has been topic that comes up couple of times per year for quite a few years and no-one has been willing to step up and tell that they need it enough to start driving the efforts14:35
*** links has quit IRC14:35
abhishekk++14:35
alistarleGood, I will try to make by best so ;)14:37
jokke_:)14:37
abhishekksmcginnis, regarding oslo_config related failure, I will report a bug and submit a patch for the same14:38
smcginnisGreat, thanks abhishekk!14:39
abhishekkor will update the same patch14:39
abhishekkanything else to discuss?14:39
abhishekkjokke_, rosmaita, smcginnis, yebinama_, alistarle ?14:39
jokke_abhishekk: I'm pretty sure Stephen doesn't mind if you take that patch over with better approach for us14:40
smcginnisNothing from me.14:40
jokke_nothing from me14:40
abhishekkjokke_, I will add him as co-author14:40
rosmaitai may have missed something14:40
rosmaitadid jokke_ say whether he will un-abandon that patch?14:41
abhishekkhe has not, but he will definitely consider as we need that patch14:42
rosmaitaok14:42
jokke_No I did not ... will see how much more time I have to keep fighting with that14:42
abhishekkOk, lets windup to utilize remaining 15 minutes14:44
abhishekkThank you all14:44
yebinama_Bye14:45
rosmaitabye!14:45
smcginnisThanks!14:45
jokke_thanks14:45
abhishekk#endmeeting14:45
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"14:45
openstackMeeting ended Thu Jan 23 14:45:54 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:45
openstackMinutes:        http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-01-23-14.00.html14:45
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-01-23-14.00.txt14:45
openstackLog:            http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-01-23-14.00.log.html14:45
*** abhishekk has left #openstack-meeting-414:46
*** alistarle has quit IRC14:47
*** tosky has left #openstack-meeting-414:49
*** bnemec has quit IRC14:51
*** andrebeltrami has joined #openstack-meeting-415:02
*** pcaruana has quit IRC15:04
*** carloss has quit IRC15:10
*** carloss has joined #openstack-meeting-415:11
*** lpetrut has quit IRC15:12
*** psachin has quit IRC15:13
*** yamamoto has quit IRC15:22
*** yamamoto has joined #openstack-meeting-415:52
*** yamamoto has quit IRC16:03
*** k_mouza has quit IRC16:54
*** k_mouza has joined #openstack-meeting-417:06
*** k_mouza has quit IRC17:11
*** diablo_rojo has quit IRC17:15
*** bobmel has quit IRC17:19
*** bobmel has joined #openstack-meeting-417:20
*** diablo_rojo has joined #openstack-meeting-417:22
*** bobmel has quit IRC17:26
*** k_mouza has joined #openstack-meeting-417:27
*** evrardjp has quit IRC17:34
*** evrardjp has joined #openstack-meeting-417:34
*** k_mouza has quit IRC17:36
*** e0ne has quit IRC17:46
*** bobmel has joined #openstack-meeting-417:56
*** psachin has joined #openstack-meeting-417:58
*** k_mouza has joined #openstack-meeting-418:25
*** k_mouza has quit IRC18:30
*** e0ne has joined #openstack-meeting-418:35
*** psachin has quit IRC18:43
*** k_mouza has joined #openstack-meeting-418:48
*** k_mouza has quit IRC19:02
*** k_mouza has joined #openstack-meeting-419:06
*** andrebeltrami has quit IRC19:11
*** heso has quit IRC19:13
*** gmann is now known as gmann_lunch19:26
*** gmann_lunch is now known as gmann19:52
*** e0ne has quit IRC20:05
*** e0ne has joined #openstack-meeting-420:45
*** diablo_rojo has quit IRC21:16
*** dansmith has quit IRC21:18
*** irclogbot_3 has quit IRC21:18
*** ricolin has quit IRC21:19
*** stephenfin has quit IRC21:19
*** tonyb has quit IRC21:19
*** irclogbot_1 has joined #openstack-meeting-421:25
*** tonyb has joined #openstack-meeting-421:26
*** diablo_rojo has joined #openstack-meeting-421:34
*** diablo_rojo has quit IRC21:37
*** diablo_rojo has joined #openstack-meeting-421:39
*** dansmith has joined #openstack-meeting-422:01
*** stephenfin has joined #openstack-meeting-422:01
*** diablo_rojo__ has joined #openstack-meeting-422:09
*** diablo_rojo has quit IRC22:11
*** e0ne has quit IRC22:41
*** e0ne has joined #openstack-meeting-422:41
*** slaweq_ has joined #openstack-meeting-422:45
*** diablo_rojo__ is now known as diablo_rojo22:46
*** enriquetaso has quit IRC22:46
*** slaweq_ has quit IRC23:01
*** e0ne has quit IRC23:26
*** dviroel has quit IRC23:39
*** diablo_rojo has quit IRC23:42
*** macz has joined #openstack-meeting-423:50
*** k_mouza has quit IRC23:51
*** macz has quit IRC23:54

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!