*** diablo_rojo has joined #openstack-meeting-4 | 00:03 | |
*** anastzhyr has quit IRC | 00:07 | |
*** spiette has quit IRC | 00:25 | |
*** spiette has joined #openstack-meeting-4 | 00:29 | |
*** k_mouza has quit IRC | 00:51 | |
*** yamamoto has joined #openstack-meeting-4 | 00:54 | |
*** yamamoto has quit IRC | 01:12 | |
*** yamamoto has joined #openstack-meeting-4 | 01:48 | |
*** yamamoto has quit IRC | 01:58 | |
*** yamamoto has joined #openstack-meeting-4 | 02:00 | |
*** yamamoto has quit IRC | 02:31 | |
*** yamamoto has joined #openstack-meeting-4 | 02:31 | |
*** roman_g has quit IRC | 02:34 | |
*** enriquetaso has quit IRC | 02:38 | |
*** yamamoto has quit IRC | 02:40 | |
*** psachin has joined #openstack-meeting-4 | 03:36 | |
*** rihabb252 has quit IRC | 03:51 | |
*** rihabb252 has joined #openstack-meeting-4 | 03:52 | |
*** macz has joined #openstack-meeting-4 | 05:00 | |
*** macz has quit IRC | 05:05 | |
*** stevthedev has quit IRC | 05:28 | |
*** stevthedev has joined #openstack-meeting-4 | 05:28 | |
*** evrardjp has quit IRC | 05:34 | |
*** evrardjp has joined #openstack-meeting-4 | 05:34 | |
*** links has joined #openstack-meeting-4 | 05:56 | |
*** k_mouza has joined #openstack-meeting-4 | 06:53 | |
*** k_mouza has quit IRC | 06:57 | |
*** roman_g has joined #openstack-meeting-4 | 07:06 | |
*** Luzi has joined #openstack-meeting-4 | 07:21 | |
*** e0ne has joined #openstack-meeting-4 | 07:53 | |
*** Adri2000 has quit IRC | 07:59 | |
*** fungi has quit IRC | 07:59 | |
*** fnordahl has quit IRC | 07:59 | |
*** Roamer` has quit IRC | 07:59 | |
*** toanster has quit IRC | 07:59 | |
*** Jeffrey4l has quit IRC | 07:59 | |
*** cmurphy has quit IRC | 07:59 | |
*** zhipeng[m] has quit IRC | 08:01 | |
*** e0ne has quit IRC | 08:02 | |
*** Jeffrey4l has joined #openstack-meeting-4 | 08:02 | |
*** cmurphy has joined #openstack-meeting-4 | 08:02 | |
*** irclogbot_1 has quit IRC | 08:04 | |
*** irclogbot_3 has joined #openstack-meeting-4 | 08:05 | |
*** bnemec has joined #openstack-meeting-4 | 08:05 | |
*** dansmith has quit IRC | 08:06 | |
*** dansmith has joined #openstack-meeting-4 | 08:08 | |
*** bnemec has quit IRC | 08:41 | |
*** Adri2000 has joined #openstack-meeting-4 | 08:45 | |
*** fungi has joined #openstack-meeting-4 | 08:45 | |
*** fnordahl has joined #openstack-meeting-4 | 08:45 | |
*** Roamer` has joined #openstack-meeting-4 | 08:45 | |
*** toanster has joined #openstack-meeting-4 | 08:45 | |
*** bnemec has joined #openstack-meeting-4 | 08:50 | |
*** bnemec has quit IRC | 09:08 | |
*** pcaruana has joined #openstack-meeting-4 | 09:30 | |
*** bnemec has joined #openstack-meeting-4 | 09:33 | |
*** Liang__ has joined #openstack-meeting-4 | 09:43 | |
*** k_mouza has joined #openstack-meeting-4 | 09:44 | |
*** k_mouza has quit IRC | 09:48 | |
*** e0ne has joined #openstack-meeting-4 | 09:50 | |
*** k_mouza has joined #openstack-meeting-4 | 09:54 | |
*** k_mouza has quit IRC | 10:18 | |
*** k_mouza has joined #openstack-meeting-4 | 10:19 | |
*** k_mouza has quit IRC | 10:34 | |
*** dviroel has joined #openstack-meeting-4 | 10:41 | |
*** k_mouza has joined #openstack-meeting-4 | 10:44 | |
*** k_mouza has quit IRC | 10:48 | |
*** k_mouza has joined #openstack-meeting-4 | 10:49 | |
*** bnemec has quit IRC | 11:02 | |
*** k_mouza has quit IRC | 11:11 | |
*** k_mouza has joined #openstack-meeting-4 | 11:13 | |
*** k_mouza has quit IRC | 11:27 | |
*** k_mouza has joined #openstack-meeting-4 | 11:27 | |
*** Liang__ has quit IRC | 11:28 | |
*** k_mouza has quit IRC | 11:29 | |
*** k_mouza has joined #openstack-meeting-4 | 11:33 | |
*** zhipeng[m] has joined #openstack-meeting-4 | 12:20 | |
*** bnemec has joined #openstack-meeting-4 | 12:29 | |
*** bobmel has quit IRC | 12:57 | |
*** bobmel has joined #openstack-meeting-4 | 12:59 | |
*** yamamoto has joined #openstack-meeting-4 | 13:12 | |
*** bobmel has quit IRC | 13:13 | |
*** bobmel has joined #openstack-meeting-4 | 13:13 | |
*** k_mouza has quit IRC | 13:17 | |
*** yamamoto has quit IRC | 13:19 | |
*** yamamoto has joined #openstack-meeting-4 | 13:21 | |
*** belmoreira has joined #openstack-meeting-4 | 13:22 | |
*** enriquetaso has joined #openstack-meeting-4 | 13:31 | |
*** abhishekk has joined #openstack-meeting-4 | 13:50 | |
*** rosmaita has joined #openstack-meeting-4 | 13:54 | |
*** yebinama_ has joined #openstack-meeting-4 | 13:56 | |
*** tosky has joined #openstack-meeting-4 | 13:58 | |
abhishekk | #startmeeting glance | 14:00 |
---|---|---|
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
*** openstack changes topic to " (Meeting topic: glance)" | 14:00 | |
abhishekk | #topic roll call | 14:00 |
openstack | The 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-agenda | 14:00 |
rosmaita | o/ | 14:00 |
abhishekk | o/ | 14:00 |
yebinama_ | o/ | 14:00 |
abhishekk | Cool | 14:00 |
abhishekk | lets start | 14:00 |
abhishekk | #topic release/periodic jobs update | 14:01 |
*** openstack changes topic to "release/periodic jobs update (Meeting topic: glance)" | 14:01 | |
abhishekk | We still have 1 failure in periodic job which is causing because of oslo_config new version | 14:01 |
abhishekk | #link https://review.opendev.org/701580 | 14:02 |
abhishekk | I have added workaround on this patch but there was no response from the owner | 14:02 |
abhishekk | So workaround sounds good to you people I will go ahead and submit this patch | 14:03 |
*** k_mouza has joined #openstack-meeting-4 | 14:03 | |
abhishekk | Kindly comment on the patch about the same, moving ahead | 14:03 |
abhishekk | Ussuri milestone 2 - 4 weeks to go | 14:03 |
abhishekk | We have 3 specs targeted to milestone 2 | 14:04 |
abhishekk | So kindly focus on reviews, I don't want to miss this deadline | 14:04 |
abhishekk | This Monday we have released stable train and stein version | 14:05 |
abhishekk | any comments so far? | 14:05 |
abhishekk | cool | 14:06 |
abhishekk | #topic Multiple store import plugins | 14:06 |
*** openstack changes topic to "Multiple store import plugins (Meeting topic: glance)" | 14:06 | |
abhishekk | multiple import impl | 14:06 |
abhishekk | yebinama_, you want to talk about this? | 14:06 |
yebinama_ | I've added the notification part | 14:06 |
abhishekk | I guess except documentation and releasenote everything is covered | 14:07 |
abhishekk | yebinama_, yes I saw that, looks good to me | 14:07 |
jokke_ | Are the specs finally green? | 14:07 |
yebinama_ | Yep only these are missing | 14:07 |
abhishekk | #link https://review.opendev.org/667132 | 14:07 |
abhishekk | jokke_, yes | 14:07 |
abhishekk | jokke_, rosmaita kindly review the patch ^^^ | 14:08 |
yebinama_ | Should I create a new doc.rst for multi store import? | 14:08 |
abhishekk | yebinama_, I guess you could add in existing file only | 14:08 |
jokke_ | ++ | 14:09 |
abhishekk | I have added info about copying stuff it in existing file | 14:09 |
yebinama_ | Ok fine | 14:09 |
jokke_ | for adding the details to the existing import docs | 14:09 |
abhishekk | refer my patch for the same | 14:09 |
yebinama_ | Ok thanks | 14:09 |
abhishekk | yebinama_, you also need to modify api-ref guide as well | 14:09 |
abhishekk | Copying existing image impl | 14:09 |
abhishekk | #link https://review.opendev.org/696457 | 14:10 |
abhishekk | So regarding this patch only release note is remaining | 14:10 |
abhishekk | I have added functional test to cover the revert logic as well | 14:10 |
yebinama_ | abhishekk: I'll work on addind this | 14:10 |
abhishekk | with 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 stores | 14:11 |
abhishekk | yebinama_, ack | 14:11 |
rosmaita | that sounds like a good test! | 14:11 |
abhishekk | Hopefully | 14:11 |
yebinama_ | Great I'll take a look | 14:12 |
abhishekk | I have written two test for these scenario with considering 'all_stores_must_succeed' parameter | 14:12 |
abhishekk | As copying patch is dependent on yebinama_'s import patch, we need to first put priority on reviewing and merging his patch | 14:13 |
abhishekk | I have tested his patch functionally with 3 file and one ceph backend | 14:13 |
yebinama_ | And just to let you know I'll be on holidays starting 5 of february | 14:14 |
rosmaita | please remove "WIP" and "PoC' from the commit messages on those patches, that's why i have been ignoring them | 14:14 |
abhishekk | rosmaita, ack | 14:14 |
abhishekk | yebinama_, so we need to make sure we can complete all the work before that and get this merged | 14:14 |
yebinama_ | rosmaita: ok i'll update the commit | 14:14 |
rosmaita | ty | 14:14 |
abhishekk | In 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 it | 14:15 |
abhishekk | yebinama_, cool | 14:15 |
abhishekk | jokke_, rosmaita need your expert opinions on the patches | 14:16 |
abhishekk | Moving ahead | 14:16 |
abhishekk | #topic Delete image from single store | 14:16 |
*** openstack changes topic to "Delete image from single store (Meeting topic: glance)" | 14:16 | |
jokke_ | We can skip this topic and move on | 14:17 |
abhishekk | I saw jokke_ has abandoned the patch, any particular reason | 14:17 |
abhishekk | jokke_, at some point we need to work on this | 14: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 API | 14:18 |
rosmaita | all i asked for was to fix some typos! | 14:19 |
abhishekk | so what if instead of using location's pop method we use store delete method to delete the location? | 14:19 |
abhishekk | this will avoid race condition and all the fuss | 14:20 |
*** yamamoto has quit IRC | 14:20 | |
abhishekk | as we have store identifier in location metadata and we have store instance in memory, we can retrieve the same and call store delete method directly | 14:20 |
rosmaita | somehow the location needs to be deleted, and i believe location.remove() calls location.pop() | 14:20 |
*** alistarle has joined #openstack-meeting-4 | 14:21 | |
jokke_ | mhm | 14:21 |
rosmaita | there is some really weird coupling in the code | 14:21 |
jokke_ | We got the race condition solved that wasn't the problem left | 14:21 |
abhishekk | jokke_, I really don't want you to throw away your efforts | 14:21 |
rosmaita | let me say something here about the consistency issue and how we talk about stuff in docs and error messages | 14:22 |
rosmaita | before multistore, the locations stuff was esoteric | 14:22 |
rosmaita | now, with the edge stuff we need to be very clear | 14:22 |
abhishekk | We can fix the typos and if required later revisit all multi-store related work to reword the help text | 14:22 |
abhishekk | even I have made mess at some places | 14:22 |
*** yamamoto has joined #openstack-meeting-4 | 14:23 | |
rosmaita | that "Image" and "image data" are different | 14:23 |
rosmaita | before, you could talk about them as pretty much the same | 14:23 |
rosmaita | but now we have the Image record and various stores where the image data is stored | 14:23 |
rosmaita | so we need to be really clear in talking about what's being removed/deleted | 14:23 |
*** heso has joined #openstack-meeting-4 | 14:23 | |
rosmaita | that's why i'm insisting on the message changes about image data is being removed from store | 14:24 |
rosmaita | i think the locations are an implementation detail | 14:24 |
rosmaita | so we don't need to discuss them | 14:25 |
abhishekk | jokke_, please reconsider and restore the patch, its required otherwise we need to be depend on location API to remove the location | 14:25 |
abhishekk | Moving ahead | 14:26 |
abhishekk | #topic Open Discussion | 14:26 |
*** openstack changes topic to "Open Discussion (Meeting topic: glance)" | 14:26 | |
abhishekk | So we need to focus on these three patches | 14:26 |
abhishekk | jokke_, rosmaita, smcginnis kindly review these patches | 14:26 |
alistarle | Hi guy's, just a question about implementation of limits/quota in glance | 14:27 |
abhishekk | alistarle, shoot | 14:27 |
smcginnis | abhishekk: Which three patches need attention? The above topics? | 14:28 |
alistarle | We 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 that | 14:28 |
abhishekk | smcginnis, yes topic 2 and 3 from the agenda | 14:28 |
rosmaita | alistarle: yes, our "quotas" are really just hard limits | 14:28 |
alistarle | And I see that Keystone implement the unified limit framework accross all openstack project, did you look at that ? | 14:29 |
smcginnis | Will do. | 14:29 |
alistarle | Seems that only nova are implementing it now, but maybe it is the time to implement it in glance no ? | 14:29 |
rosmaita | alistarle: we would be happy to review your code if you want to implement that | 14:29 |
abhishekk | smcginnis, thank you | 14:29 |
alistarle | Yes I would like to work on that, seems the library oslo.limits must be used, but I don't understand some point of their framework | 14:30 |
abhishekk | alistarle, also it will be great if you submit the glance-specs for the same | 14:30 |
alistarle | Sure | 14:30 |
smcginnis | alistarle: 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 |
abhishekk | it will help us to understand the design and providing suggestions | 14: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 |
alistarle | Yes 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 usage | 14:31 |
*** Luzi has quit IRC | 14:32 | |
alistarle | The point is we needed quotas, so I think working on unified limits is better than trying to do something on our side | 14:32 |
smcginnis | Definitely agree it's about time we get to a common solution for this. | 14:34 |
abhishekk | +1 | 14: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 efforts | 14:35 |
*** links has quit IRC | 14:35 | |
abhishekk | ++ | 14:35 |
alistarle | Good, I will try to make by best so ;) | 14:37 |
jokke_ | :) | 14:37 |
abhishekk | smcginnis, regarding oslo_config related failure, I will report a bug and submit a patch for the same | 14:38 |
smcginnis | Great, thanks abhishekk! | 14:39 |
abhishekk | or will update the same patch | 14:39 |
abhishekk | anything else to discuss? | 14:39 |
abhishekk | jokke_, 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 us | 14:40 |
smcginnis | Nothing from me. | 14:40 |
jokke_ | nothing from me | 14:40 |
abhishekk | jokke_, I will add him as co-author | 14:40 |
rosmaita | i may have missed something | 14:40 |
rosmaita | did jokke_ say whether he will un-abandon that patch? | 14:41 |
abhishekk | he has not, but he will definitely consider as we need that patch | 14:42 |
rosmaita | ok | 14:42 |
jokke_ | No I did not ... will see how much more time I have to keep fighting with that | 14:42 |
abhishekk | Ok, lets windup to utilize remaining 15 minutes | 14:44 |
abhishekk | Thank you all | 14:44 |
yebinama_ | Bye | 14:45 |
rosmaita | bye! | 14:45 |
smcginnis | Thanks! | 14:45 |
jokke_ | thanks | 14:45 |
abhishekk | #endmeeting | 14:45 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 14:45 | |
openstack | Meeting ended Thu Jan 23 14:45:54 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:45 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-01-23-14.00.html | 14:45 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-01-23-14.00.txt | 14:45 |
openstack | Log: http://eavesdrop.openstack.org/meetings/glance/2020/glance.2020-01-23-14.00.log.html | 14:45 |
*** abhishekk has left #openstack-meeting-4 | 14:46 | |
*** alistarle has quit IRC | 14:47 | |
*** tosky has left #openstack-meeting-4 | 14:49 | |
*** bnemec has quit IRC | 14:51 | |
*** andrebeltrami has joined #openstack-meeting-4 | 15:02 | |
*** pcaruana has quit IRC | 15:04 | |
*** carloss has quit IRC | 15:10 | |
*** carloss has joined #openstack-meeting-4 | 15:11 | |
*** lpetrut has quit IRC | 15:12 | |
*** psachin has quit IRC | 15:13 | |
*** yamamoto has quit IRC | 15:22 | |
*** yamamoto has joined #openstack-meeting-4 | 15:52 | |
*** yamamoto has quit IRC | 16:03 | |
*** k_mouza has quit IRC | 16:54 | |
*** k_mouza has joined #openstack-meeting-4 | 17:06 | |
*** k_mouza has quit IRC | 17:11 | |
*** diablo_rojo has quit IRC | 17:15 | |
*** bobmel has quit IRC | 17:19 | |
*** bobmel has joined #openstack-meeting-4 | 17:20 | |
*** diablo_rojo has joined #openstack-meeting-4 | 17:22 | |
*** bobmel has quit IRC | 17:26 | |
*** k_mouza has joined #openstack-meeting-4 | 17:27 | |
*** evrardjp has quit IRC | 17:34 | |
*** evrardjp has joined #openstack-meeting-4 | 17:34 | |
*** k_mouza has quit IRC | 17:36 | |
*** e0ne has quit IRC | 17:46 | |
*** bobmel has joined #openstack-meeting-4 | 17:56 | |
*** psachin has joined #openstack-meeting-4 | 17:58 | |
*** k_mouza has joined #openstack-meeting-4 | 18:25 | |
*** k_mouza has quit IRC | 18:30 | |
*** e0ne has joined #openstack-meeting-4 | 18:35 | |
*** psachin has quit IRC | 18:43 | |
*** k_mouza has joined #openstack-meeting-4 | 18:48 | |
*** k_mouza has quit IRC | 19:02 | |
*** k_mouza has joined #openstack-meeting-4 | 19:06 | |
*** andrebeltrami has quit IRC | 19:11 | |
*** heso has quit IRC | 19:13 | |
*** gmann is now known as gmann_lunch | 19:26 | |
*** gmann_lunch is now known as gmann | 19:52 | |
*** e0ne has quit IRC | 20:05 | |
*** e0ne has joined #openstack-meeting-4 | 20:45 | |
*** diablo_rojo has quit IRC | 21:16 | |
*** dansmith has quit IRC | 21:18 | |
*** irclogbot_3 has quit IRC | 21:18 | |
*** ricolin has quit IRC | 21:19 | |
*** stephenfin has quit IRC | 21:19 | |
*** tonyb has quit IRC | 21:19 | |
*** irclogbot_1 has joined #openstack-meeting-4 | 21:25 | |
*** tonyb has joined #openstack-meeting-4 | 21:26 | |
*** diablo_rojo has joined #openstack-meeting-4 | 21:34 | |
*** diablo_rojo has quit IRC | 21:37 | |
*** diablo_rojo has joined #openstack-meeting-4 | 21:39 | |
*** dansmith has joined #openstack-meeting-4 | 22:01 | |
*** stephenfin has joined #openstack-meeting-4 | 22:01 | |
*** diablo_rojo__ has joined #openstack-meeting-4 | 22:09 | |
*** diablo_rojo has quit IRC | 22:11 | |
*** e0ne has quit IRC | 22:41 | |
*** e0ne has joined #openstack-meeting-4 | 22:41 | |
*** slaweq_ has joined #openstack-meeting-4 | 22:45 | |
*** diablo_rojo__ is now known as diablo_rojo | 22:46 | |
*** enriquetaso has quit IRC | 22:46 | |
*** slaweq_ has quit IRC | 23:01 | |
*** e0ne has quit IRC | 23:26 | |
*** dviroel has quit IRC | 23:39 | |
*** diablo_rojo has quit IRC | 23:42 | |
*** macz has joined #openstack-meeting-4 | 23:50 | |
*** k_mouza has quit IRC | 23:51 | |
*** macz has quit IRC | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!