| *** abhishekk is now known as akekane|home | 13:42 | |
| *** akekane|home is now known as abhishekk | 13:43 | |
| abhishekk | #startmeeting glance | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Thu Aug 18 14:00:01 2022 UTC and is due to finish in 60 minutes. The chair is abhishekk. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'glance' | 14:00 |
| abhishekk | #topic roll call | 14:00 |
| abhishekk | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 14:00 |
| abhishekk | waiting for others to show | 14:00 |
| abhishekk | o/ | 14:00 |
| mrjoshi_ | o/ | 14:00 |
| dansmith | o/ | 14:00 |
| abhishekk | update, cyril is on leave for 3 weeks (including this) | 14:00 |
| abhishekk | mrjoshi_, there are some comments on your patch, if you have any doubts then get it cleared here | 14:01 |
| rosmaita | o/ | 14:02 |
| abhishekk | pdeore, will be late, lets start | 14:02 |
| abhishekk | #topic Update | 14:03 |
| rosmaita | (sorry, i am running a video meeting simultaneously) | 14:03 |
| jokke_ | o/ | 14:03 |
| abhishekk | np! | 14:03 |
| abhishekk | The PTG will be virtual now | 14:03 |
| abhishekk | We will follow the same time which we used to follow for previous PTG's | 14:04 |
| abhishekk | that will be between 1400 UTC to 1700 UTC | 14:04 |
| abhishekk | If anyone has any concerns related to timings then please let me know | 14:04 |
| abhishekk | moving forward; | 14:04 |
| abhishekk | #topic release/periodic jobs updates | 14:05 |
| abhishekk | Milestone 3 two weeks away | 14:05 |
| abhishekk | we have 2 features in waiting list | 14:05 |
| abhishekk | lets discuss this in next topic | 14:05 |
| abhishekk | glance-store final release will be next week | 14:05 |
| abhishekk | We don't have anything pending for store, so good to tag the final release by next Wednesday | 14:06 |
| abhishekk | I will suggest cores to go through list of patches of glance store and see it there is anything important we can get merged before next week | 14:07 |
| jokke_ | ack | 14:07 |
| abhishekk | cool, periodic jobs all green except time out for fips related jobs | 14:07 |
| abhishekk | moving to next topic | 14:08 |
| abhishekk | #topic Pending features | 14:08 |
| abhishekk | Store details API - mrjoshi | 14:08 |
| abhishekk | there are few suggestions from rosmaita, I think one is related to using actual parameter names | 14:09 |
| abhishekk | and other one is should we show computed values rather than in Bytes | 14:09 |
| rosmaita | actually, the patch is showing the computed values in bytes, but the operator can only configure in MB | 14:10 |
| rosmaita | so i guess the question is what is the primary goal of the discovery response? | 14:10 |
| * pdeore is back | 14:10 | |
| rosmaita | i was thinking it was for an operator to see how the store is configured | 14:11 |
| abhishekk | correct | 14:11 |
| rosmaita | ok, then we should probably not show the computed values ... or we could show them, but indicated separately somehow | 14:11 |
| rosmaita | the big problem though is that we already implemented store discovery for rbd store | 14:12 |
| abhishekk | rosmaita, I think if we decides to do it, do it separtely for both the stores | 14:12 |
| rosmaita | and that currently shows the computed value, not the configured value | 14:12 |
| abhishekk | at this moment I am good with computed values | 14:14 |
| abhishekk | we can explain about it in release notes | 14:14 |
| mrjoshi_ | I think initially we were going with the computed values only when we had it for the rbd store | 14:14 |
| abhishekk | right | 14:15 |
| abhishekk | and if you are going to change it then you need to file a bug fix it for rbd first and then we can merge this patch | 14:15 |
| abhishekk | that will be small change though | 14:15 |
| jokke_ | Will break the API | 14:16 |
| abhishekk | how? | 14:16 |
| jokke_ | To change it for rbd it's published and released already | 14:17 |
| abhishekk | in that case I think its better to explain it in docs and release note | 14:17 |
| jokke_ | so better option is to use another key name all together if we want to show it as it's configured rather than how the system sees it | 14:18 |
| dansmith | it will change only in the respect that it will be correct instead of incorrect right? | 14:18 |
| abhishekk | that will increase the confusion | 14:18 |
| dansmith | a user doesn't know that it's wrong right now and the units are no different once we show the right value yeah? | 14:18 |
| abhishekk | I think its better to show computed value and explain it in reno | 14:20 |
| jokke_ | it's not wrong, the code uses bytes, the config otoh is expecting Mi. So both are correct and it does not have unit with it on the field | 14:20 |
| dansmith | right, we expose bytes, but where they will only ever be a multiple of 1MiB today correct? | 14:21 |
| dansmith | there's no reasonable client behavior that would be broken by reading a byte value that is not MiB-aligned in the future, that I can think of :) | 14:21 |
| abhishekk | Ok, we need an agreement here otherwise this will get dropped from the cycle | 14:23 |
| jokke_ | So there is no usecase for that info in the first place. But what will be confusing AF, is that if we change it now, and you look the output from older system showing you bytes (without unit) and new version that shows you integer without units too but the value just happens to be MiB rather than bytes | 14:24 |
| abhishekk | I am still at opinion of implementing it same way as implemented in past | 14:24 |
| jokke_ | ++ | 14:25 |
| jokke_ | I'm sure we have something like API ref that explains what the value is and helptext in the config explaining what is expected | 14:25 |
| abhishekk | mrjoshi_, just change the parameter names of swift store at this moment | 14:25 |
| mrjoshi_ | abhishekk, ack | 14:26 |
| abhishekk | moving ahead | 14:26 |
| abhishekk | 2nd pending work is Glance download plugin | 14:26 |
| abhishekk | I am afraid both alistarl1 and Pierre-Samuel Le Stang are not around today | 14:27 |
| abhishekk | the patch is in good shape with few changes, should we go forward and work on that | 14:27 |
| abhishekk | or we should use FFE for the same? | 14:28 |
| dansmith | yeah I was going to ask..seems very close but not sure if they're working on those feedback items or not | 14:28 |
| abhishekk | not sure either, didn't heard from them since last meeting | 14:29 |
| abhishekk | and I also don't want to duplicate the efforts | 14:29 |
| abhishekk | I will wait till Monday to get update from them,else will start working on the patch | 14:30 |
| abhishekk | sounds good? | 14:30 |
| dansmith | sure | 14:30 |
| jokke_ | for the sake of keeping the patch moving, I'd say if there's no response from them by next week, feel free to propose a new rev ... I'm sure they'd appreciate having it in this cycle too | 14:30 |
| abhishekk | cool | 14:30 |
| abhishekk | ++ | 14:30 |
| abhishekk | moving forward | 14:31 |
| abhishekk | #topic Glance PTG planning etherpad | 14:31 |
| abhishekk | PTG planning etherpad is up | 14:31 |
| abhishekk | #link https://etherpad.opendev.org/p/antelope-ptg-glance-planning | 14:31 |
| abhishekk | Feel free to add topics which you want to discuss during this PTG | 14:32 |
| abhishekk | also this will be my last cycle as PTL | 14:32 |
| abhishekk | and hopefully pdeore is up for taking the responsibility | 14:32 |
| jokke_ | PTL is gone, long live PTL! | 14:32 |
| jokke_ | :) | 14:32 |
| abhishekk | :P | 14:32 |
| abhishekk | I will help pdeore to organize this PTG | 14:33 |
| abhishekk | that is all from me | 14:34 |
| abhishekk | any questions, suggestions? | 14:34 |
| abhishekk | Just to inform you, I will be going on leave from 10th of September (just before rc1 week) | 14:34 |
| abhishekk | if everything is alright and we manage to tag Zed release before that then it will be good | 14:35 |
| abhishekk | otherwise I will work part time during rc1 week | 14:35 |
| jokke_ | Working holiday? Aren' you getting old for that :P | 14:36 |
| abhishekk | :D if you have my back then I don't need to do that | 14:36 |
| abhishekk | I think we should wrap up now | 14:37 |
| abhishekk | thank you all | 14:37 |
| abhishekk | have a nice weekend ahead | 14:37 |
| jokke_ | thanks | 14:37 |
| abhishekk | #endmeeting | 14:38 |
| opendevmeet | Meeting ended Thu Aug 18 14:38:07 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2022/glance.2022-08-18-14.00.html | 14:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2022/glance.2022-08-18-14.00.txt | 14:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2022/glance.2022-08-18-14.00.log.html | 14:38 |
| *** dasm|off is now known as dasm | 15:09 | |
| *** dasm is now known as dasm|off | 22:08 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!