*** andrebeltrami has quit IRC | 00:38 | |
*** yamamoto has joined #openstack-meeting-alt | 00:46 | |
*** yamamoto has quit IRC | 00:49 | |
*** yamamoto has joined #openstack-meeting-alt | 00:58 | |
*** Liang__ has joined #openstack-meeting-alt | 01:17 | |
*** markvoelker has joined #openstack-meeting-alt | 01:20 | |
*** markvoelker has quit IRC | 01:24 | |
*** gyee has quit IRC | 01:35 | |
*** yaawang has quit IRC | 01:42 | |
*** ricolin_ has joined #openstack-meeting-alt | 01:43 | |
*** yaawang has joined #openstack-meeting-alt | 01:43 | |
*** Liang__ has quit IRC | 01:46 | |
*** yaawang has quit IRC | 02:45 | |
*** yaawang has joined #openstack-meeting-alt | 02:46 | |
*** rfolco has quit IRC | 03:20 | |
*** ricolin_ is now known as ricolin | 03:30 | |
*** Liang__ has joined #openstack-meeting-alt | 03:58 | |
*** markvoelker has joined #openstack-meeting-alt | 04:03 | |
*** markvoelker has quit IRC | 04:08 | |
*** apetrich has quit IRC | 04:43 | |
*** mugsie has quit IRC | 04:53 | |
*** mugsie has joined #openstack-meeting-alt | 04:57 | |
*** markvoelker has joined #openstack-meeting-alt | 05:03 | |
*** strigazi has quit IRC | 05:04 | |
*** strigazi has joined #openstack-meeting-alt | 05:06 | |
*** markvoelker has quit IRC | 05:07 | |
*** vishalmanchanda has joined #openstack-meeting-alt | 05:23 | |
*** links has joined #openstack-meeting-alt | 05:34 | |
*** yaawang has quit IRC | 05:45 | |
*** yaawang has joined #openstack-meeting-alt | 05:45 | |
*** strigazi has quit IRC | 05:46 | |
*** ccamacho has quit IRC | 05:58 | |
*** ccamacho has joined #openstack-meeting-alt | 06:31 | |
*** tetsuro has joined #openstack-meeting-alt | 06:33 | |
*** tetsuro has quit IRC | 06:43 | |
*** tetsuro has joined #openstack-meeting-alt | 06:43 | |
*** tetsuro has quit IRC | 06:43 | |
*** yaawang has quit IRC | 06:43 | |
*** yaawang has joined #openstack-meeting-alt | 06:51 | |
*** tetsuro has joined #openstack-meeting-alt | 06:55 | |
*** yaawang has quit IRC | 06:55 | |
*** yaawang has joined #openstack-meeting-alt | 06:55 | |
*** tetsuro has quit IRC | 06:56 | |
*** rcernin has quit IRC | 07:02 | |
*** markvoelker has joined #openstack-meeting-alt | 07:04 | |
*** rcernin has joined #openstack-meeting-alt | 07:05 | |
*** markvoelker has quit IRC | 07:09 | |
*** ivanvtimofeev has quit IRC | 07:09 | |
*** slaweq has quit IRC | 07:12 | |
*** slaweq has joined #openstack-meeting-alt | 07:13 | |
*** markvoelker has joined #openstack-meeting-alt | 07:18 | |
*** markvoelker has quit IRC | 07:23 | |
*** rcernin has quit IRC | 07:30 | |
*** Liang__ has quit IRC | 07:30 | |
*** Liang__ has joined #openstack-meeting-alt | 07:31 | |
*** ralonsoh has joined #openstack-meeting-alt | 07:33 | |
*** jtomasek has quit IRC | 07:36 | |
*** jtomasek has joined #openstack-meeting-alt | 07:40 | |
*** rcernin has joined #openstack-meeting-alt | 07:50 | |
*** ttsiouts has joined #openstack-meeting-alt | 07:53 | |
*** ttsiouts has quit IRC | 07:53 | |
*** ttsiouts has joined #openstack-meeting-alt | 07:54 | |
*** e0ne has joined #openstack-meeting-alt | 07:58 | |
*** rcernin has quit IRC | 08:06 | |
*** rdopiera has joined #openstack-meeting-alt | 08:09 | |
*** ttsiouts has quit IRC | 08:37 | |
*** ttsiouts has joined #openstack-meeting-alt | 08:38 | |
*** ttsiouts has quit IRC | 08:42 | |
*** derekh has joined #openstack-meeting-alt | 08:45 | |
*** yaawang has quit IRC | 08:46 | |
*** yaawang has joined #openstack-meeting-alt | 08:46 | |
*** apetrich has joined #openstack-meeting-alt | 08:48 | |
*** jtomasek_ has joined #openstack-meeting-alt | 09:05 | |
*** jtomasek has quit IRC | 09:07 | |
*** ttsiouts has joined #openstack-meeting-alt | 09:27 | |
*** yaawang has quit IRC | 09:30 | |
*** yaawang has joined #openstack-meeting-alt | 09:31 | |
*** yamamoto has quit IRC | 09:37 | |
*** ricolin has quit IRC | 09:52 | |
*** yamamoto has joined #openstack-meeting-alt | 09:55 | |
*** yamamoto has quit IRC | 10:06 | |
*** yamamoto has joined #openstack-meeting-alt | 10:07 | |
*** apetrich has quit IRC | 10:12 | |
*** Liang__ has quit IRC | 10:40 | |
*** priteau has joined #openstack-meeting-alt | 11:02 | |
*** markvoelker has joined #openstack-meeting-alt | 11:15 | |
*** markvoelker has quit IRC | 11:19 | |
*** raildo has joined #openstack-meeting-alt | 11:28 | |
*** apetrich has joined #openstack-meeting-alt | 11:45 | |
*** rnoriega_ has quit IRC | 11:58 | |
*** rnoriega- has joined #openstack-meeting-alt | 11:58 | |
*** andrebeltrami has joined #openstack-meeting-alt | 12:06 | |
*** irclogbot_2 has quit IRC | 12:16 | |
*** irclogbot_1 has joined #openstack-meeting-alt | 12:17 | |
*** derekh has quit IRC | 12:28 | |
*** trident has quit IRC | 12:46 | |
*** derekh has joined #openstack-meeting-alt | 12:48 | |
*** trident has joined #openstack-meeting-alt | 12:49 | |
*** ricolin has joined #openstack-meeting-alt | 12:54 | |
*** rfolco has joined #openstack-meeting-alt | 13:00 | |
*** jhesketh has quit IRC | 13:22 | |
*** jhesketh has joined #openstack-meeting-alt | 13:24 | |
*** liuyulong has joined #openstack-meeting-alt | 13:50 | |
*** yamamoto has quit IRC | 13:50 | |
*** rosmaita has joined #openstack-meeting-alt | 13:52 | |
*** tosky has joined #openstack-meeting-alt | 13:57 | |
rosmaita | #startmeeting cinder | 14:00 |
---|---|---|
rosmaita | #link https://etherpad.openstack.org/p/cinder-victoria-meetings | 14:00 |
rosmaita | #topic roll call | 14:00 |
openstack | Meeting started Wed Jul 1 14:00:03 2020 UTC and is due to finish in 60 minutes. The chair is rosmaita. 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: cinder)" | 14:00 | |
openstack | The meeting name has been set to 'cinder' | 14:00 |
*** openstack changes topic to "roll call (Meeting topic: cinder)" | 14:00 | |
tosky | o/ | 14:00 |
walshh_ | hi | 14:00 |
jungleboyj | o/ | 14:00 |
whoami-rajat | Hi | 14:00 |
e0ne | hi | 14:00 |
rosmaita | hello everyone | 14:01 |
rosmaita | #link https://etherpad.openstack.org/p/cinder-victoria-meetings | 14:01 |
geguileo | hi! o/ | 14:01 |
jungleboyj | Happy Wednesday! | 14:02 |
rosmaita | specs freeze in a few hours, so that's what we'll be discussing mostly today | 14:02 |
rosmaita | but first some announcements | 14:02 |
rajinir | o/ | 14:02 |
rosmaita | #topic updates - mid-cycle | 14:02 |
*** openstack changes topic to "updates - mid-cycle (Meeting topic: cinder)" | 14:02 | |
*** sfernand has joined #openstack-meeting-alt | 14:02 | |
rosmaita | thanks to everyone who participated last week | 14:02 |
rosmaita | if you missed it, you can catch up on youtube | 14:02 |
*** Liang__ has joined #openstack-meeting-alt | 14:02 | |
*** Liang__ is now known as LiangFang | 14:02 | |
rosmaita | though abishop said he was going to wait until after part 2 and then binge watch | 14:03 |
rosmaita | so, whatever works for you | 14:03 |
LiangFang | o/ | 14:03 |
rosmaita | #link https://wiki.openstack.org/wiki/CinderVictoriaMidCycleSummary | 14:03 |
lseki | o/ | 14:03 |
rosmaita | you can find a link to the recording ^^ | 14:03 |
jungleboyj | :-) | 14:03 |
rosmaita | #topic updates - monthly video meeting poll | 14:03 |
smcginnis | o/ | 14:03 |
*** openstack changes topic to "updates - monthly video meeting poll (Meeting topic: cinder)" | 14:03 | |
rosmaita | #link https://rosmaita.wufoo.com/forms/monthly-video-meeting-proposal/ | 14:04 |
rosmaita | i noticed that there were some communication difficulties at the mid-cycle, so the poll includes an option about whether we should really do one meeting a month in video or not | 14:04 |
rosmaita | but if we commit to taking good notes in irc, i think it's worth a try | 14:05 |
rosmaita | anyway, the proposed first video meeting would be 29 July | 14:05 |
rosmaita | so we have a bit of time to think about this | 14:05 |
rosmaita | anyway, the poll closes 1200 UTC on Thursday 9 July | 14:05 |
rosmaita | #updates - brocade FCZM driver | 14:06 |
rosmaita | oops | 14:06 |
rosmaita | #topic updates - brocade FCZM driver | 14:06 |
*** openstack changes topic to "updates - brocade FCZM driver (Meeting topic: cinder)" | 14:06 | |
rosmaita | i sent out an email summarizing what we discussed at the mid-cycle | 14:07 |
rosmaita | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015692.html | 14:07 |
rosmaita | so, we'll see if anyone cares | 14:07 |
rosmaita | #topic updates - need to keep an eye on the cinder-plugin-ceph-tempest job | 14:07 |
*** openstack changes topic to "updates - need to keep an eye on the cinder-plugin-ceph-tempest job (Meeting topic: cinder)" | 14:07 | |
rosmaita | this was really bugging me yesterday, but maybe it's a fluke | 14:07 |
tosky | rosmaita: master or previous branches? | 14:07 |
rosmaita | we made the job voting on 11 march this year | 14:07 |
rosmaita | tosky: both | 14:07 |
tosky | there is an open review that should fix the issues | 14:07 |
rosmaita | #link https://zuul.opendev.org/t/openstack/builds?job_name=cinder-plugin-ceph-tempest | 14:08 |
rosmaita | tosky: that's good news | 14:08 |
rosmaita | #topic updates - code coverage job | 14:09 |
*** openstack changes topic to "updates - code coverage job (Meeting topic: cinder)" | 14:09 | |
rosmaita | i finally followed up on my PTG action to put up a job so you can have the code coverage report in groovy html on each patch | 14:09 |
rosmaita | #link https://review.opendev.org/#/c/738687/ | 14:09 |
rosmaita | (it still needs reviews) | 14:10 |
rosmaita | right now, all it does is post the coverage output for you to look at | 14:10 |
rosmaita | we could add something that would pass/fail based on coverage falling past a threshhold | 14:10 |
rosmaita | e0ne is in favor of that | 14:11 |
whoami-rajat | #link https://6d4b1c17bd5712b1da1e-24da3718548989a700aa54b9db0ff79c.ssl.cf2.rackcdn.com/738687/8/check/cinder-code-coverage/7bb5c35/cover/ | 14:11 |
whoami-rajat | looks great | 14:11 |
*** yamamoto has joined #openstack-meeting-alt | 14:11 | |
*** yamamoto has quit IRC | 14:11 | |
e0ne | I know that not everybody agrees with me, so I put +1 on that patch | 14:11 |
*** yamamoto has joined #openstack-meeting-alt | 14:12 | |
smcginnis | Putting a pass/fail on coverage is a hard thing to do right. | 14:12 |
smcginnis | I've been in endless debates about what the right amount of coverage should be. | 14:12 |
jungleboyj | This is cool. Good info at least. | 14:12 |
rosmaita | yeah, it's a start | 14:12 |
rosmaita | i'll think about how we could track the coverage to address e0ne's concern | 14:12 |
rosmaita | my worry is that i don't want it to be non-voting and turn into the pylint job | 14:13 |
rosmaita | that's always red and so you don't notice anymore | 14:13 |
e0ne | rosmaita: +1 | 14:13 |
*** eharney has joined #openstack-meeting-alt | 14:13 | |
smcginnis | Alternatively, we could just add coverage as part of the base commands that are run so it's always available for the normal UT runs. | 14:14 |
e0ne | rosmaita, smcginnis: we can use --fault-unnder (https://coverage.readthedocs.io/en/coverage-5.1/cmd.html) option | 14:14 |
rosmaita | let's discuss on the patch, or in a few meetings | 14:15 |
rosmaita | i think for now just having the info available is good | 14:15 |
e0ne | Agree. it's good for the beginning | 14:16 |
rosmaita | that's it for updates, unless anyone else has an announcement | 14:16 |
jungleboyj | ++ Starting by having it out there is a good step. | 14:16 |
*** pots has joined #openstack-meeting-alt | 14:16 | |
*** yamamoto has quit IRC | 14:16 | |
jungleboyj | Don't want the voting question to stop that. | 14:16 |
rosmaita | yeah, let's make it a separate discussion | 14:16 |
rosmaita | #topic minor driver refactoring | 14:17 |
*** openstack changes topic to "minor driver refactoring (Meeting topic: cinder)" | 14:17 | |
e0ne | rosmaita: thanks for raising this up! | 14:17 |
rosmaita | #link https://review.opendev.org/#/c/656888/ | 14:17 |
rosmaita | e0ne: thanks for doing it! | 14:17 |
tosky | e0ne: --fault-under is not good IMHO, it should be a delta | 14:17 |
e0ne | a minor change with a huge diff | 14:17 |
rosmaita | ok, what's going on is that e0ne did some refactoring | 14:17 |
rosmaita | and, it turns out that he didn't have to modify a lot of unit tests | 14:18 |
rosmaita | because they weren't there | 14:18 |
*** TusharTgite has joined #openstack-meeting-alt | 14:18 | |
rosmaita | so my point in bringing this up is ... | 14:18 |
rosmaita | if you are a driver maintainer, look at that patch ^^ and check your driver | 14:18 |
rosmaita | and you might want to think about a follow up patch adding a test or two | 14:18 |
rosmaita | there are some sample tests on that patch | 14:19 |
e0ne | just for the note: I'm not going to add unit-tests for drivers for this patch or as a follow up patch | 14:19 |
rosmaita | the main thing is, we will review the patch by hand, but it's easy to miss something | 14:19 |
rosmaita | e0ne: good note | 14:19 |
rosmaita | yeah, this is a driver maintainer's responsibility | 14:19 |
rosmaita | so please take a look | 14:20 |
smcginnis | ++ | 14:20 |
rosmaita | but do it soon, like within the next few hours | 14:20 |
rosmaita | because it looks good, and we will merge it soon so e0ne isn't in merge conflict hell | 14:20 |
e0ne | :) | 14:21 |
rosmaita | #topic victoria spec review | 14:21 |
*** openstack changes topic to "victoria spec review (Meeting topic: cinder)" | 14:21 | |
rosmaita | ok, i got sidetracked on some stuff, so my knowledge of the state of specs is from last night | 14:22 |
rosmaita | but here we go | 14:22 |
*** dave-mccowan has quit IRC | 14:22 | |
rosmaita | #topic spec - Remove quota usage cache | 14:22 |
*** openstack changes topic to "spec - Remove quota usage cache (Meeting topic: cinder)" | 14:22 | |
rosmaita | #link https://review.opendev.org/#/c/730701/ | 14:22 |
rosmaita | Sam left some performance info on the spec, i think | 14:22 |
rosmaita | looks like Lucio's objection is formatting | 14:23 |
rosmaita | (which is fine, we want to be able to read the thing) | 14:23 |
rosmaita | so, at a glance, it looks like getting the usage dynamically isn't too big a hit | 14:24 |
jungleboyj | Looks like the performance impact is sufficiently minimal though. | 14:24 |
rosmaita | i suspect there will be some weird things that we won't know about until someone actually tries to implement this | 14:25 |
rosmaita | so probably no point trying to work them out in advance | 14:25 |
eharney | IMO we should look at our db schema and see if we have indexes etc. there that would make this perform as well as possible | 14:25 |
rosmaita | eharney: that is a good point, please add it as a comment to the spec | 14:26 |
*** dave-mccowan has joined #openstack-meeting-alt | 14:26 | |
rosmaita | yeah, good indices are key | 14:26 |
rosmaita | so, the author is in australia, and i think today is almost over there | 14:27 |
rosmaita | i think it would be ok to stretch the deadline for him to attend to some changes | 14:27 |
rosmaita | i think the mood here is mostly positive toward this spec? | 14:28 |
e0ne | rosmaita: +1 | 14:28 |
jungleboyj | +1 | 14:28 |
rosmaita | i guess this spec-freeze is a bit early relative to the ptg, but we can at least weed out the no-chance specs | 14:29 |
rosmaita | ok, i'll put a note that he can revise and it will still be acceptable until the next cinder meeting | 14:29 |
rosmaita | #topic specs - Support modern compression algorithms in cinder backup | 14:30 |
*** openstack changes topic to "specs - Support modern compression algorithms in cinder backup (Meeting topic: cinder)" | 14:30 | |
rosmaita | #link https://review.opendev.org/#/c/726307/ | 14:30 |
rosmaita | this one is kind of in the same boat, revision wise | 14:30 |
rosmaita | my feeling is that we should update our compression algo options | 14:31 |
rosmaita | but to what ... i think i agree with eric that we should just pick 1 of the 2 options | 14:31 |
rosmaita | as to which one ... | 14:31 |
eharney | yeah | 14:31 |
rosmaita | we're going to have to get this past the requirements team | 14:31 |
eharney | i haven't done much looking into which would be a better choice | 14:31 |
rosmaita | so i think in preparing the info for the patch for global-requirements, we can decide which is best | 14:32 |
eharney | i assume any would pass the requirements concerns | 14:32 |
e0ne | what about make these things plugable? | 14:32 |
eharney | they are | 14:32 |
eharney | but we don't want to give people a bunch of extra options just because we can, and have to maintain that forever | 14:33 |
rosmaita | i like zstandard better, but i'm not sure it's packaged as widely as the other option | 14:33 |
rosmaita | i think packaging is an issue for inclusion in global-requirements | 14:34 |
rosmaita | anyway, the author already has a patch up adding the algos to cinder | 14:34 |
rosmaita | so either one will be an easy code change | 14:34 |
rosmaita | it's really a matter of working out which is best | 14:34 |
rosmaita | so i think give him another week to get an argument together, and onto the spec? | 14:35 |
rosmaita | any objection? | 14:35 |
e0ne | eharney: actually, all algorithms and compressors are hard-coded | 14:36 |
e0ne | we can make it extendable like we did for db provider | 14:36 |
eharney | why? | 14:36 |
e0ne | so end-users will be able to add out-of-tree compressing algos to use it with cinder | 14:37 |
eharney | and then end up with backups that can only be consumed by certain versions of cinder on certain clouds on certain platforms.. | 14:37 |
smcginnis | ^ | 14:37 |
eharney | i don't see a benefit to justify that mess | 14:37 |
e0ne | goor point | 14:37 |
rosmaita | yeah, let's keep these as 2 separate issues | 14:38 |
jungleboyj | ++ | 14:38 |
rosmaita | so for now, we can say let's add a modern compression algo | 14:38 |
rosmaita | and we can discuss later whether the plugin approach makes sense | 14:38 |
geguileo | would it make sense to update the default compressor? | 14:38 |
geguileo | if we are going to add it to the requirements already and it won't be optional | 14:39 |
rosmaita | no, i think gzip is a venerable compression algo and we can honor it by keeping it the default | 14:39 |
geguileo | lol | 14:39 |
rosmaita | also, it's a standard library algo | 14:40 |
e0ne | geguileo: we should not break current deployments with rolling upgrades | 14:40 |
jungleboyj | It seems like this is a new feature that people can use if they wish. | 14:40 |
rosmaita | #topic specs - Reset state robustification | 14:40 |
*** openstack changes topic to "specs - Reset state robustification (Meeting topic: cinder)" | 14:40 | |
geguileo | e0ne: that's a very good reason | 14:41 |
rosmaita | #link https://review.opendev.org/#/c/682456/ | 14:41 |
jungleboyj | Mmmm robustification | 14:41 |
e0ne | geguileo: that's why we still have tgt as a default option :( | 14:41 |
rosmaita | i think this is good, the open question is how to handle the --force option | 14:41 |
geguileo | e0ne: ouch | 14:41 |
eharney | i prefer just making the API safe and leaving the --force cases to cinder-manage | 14:41 |
rosmaita | i am ok with that | 14:42 |
rosmaita | eharney: why don't you revise the spec and people can comment, you can have a 1 week extension too | 14:42 |
eharney | ok | 14:43 |
rosmaita | #topic specs - Default volume type overrides | 14:43 |
*** openstack changes topic to "specs - Default volume type overrides (Meeting topic: cinder)" | 14:43 | |
rosmaita | #link https://review.opendev.org/#/c/733555/ | 14:43 |
rosmaita | this one had 2 +2s, i just left it open so people could still comment | 14:43 |
whoami-rajat | it looks perfect now | 14:43 |
whoami-rajat | thanks geguileo for the quick update | 14:44 |
geguileo | np | 14:44 |
rosmaita | ok, i will re-read and we can get this one approved before the deadline | 14:44 |
rosmaita | yay! | 14:44 |
rosmaita | topic specs - Backup Backends Configuration | 14:45 |
rosmaita | #link https://review.opendev.org/#/c/712301/ | 14:45 |
rosmaita | this one looks good too | 14:45 |
e0ne | thanks everybody for your feedback on this! | 14:45 |
rosmaita | looks like e0ne has revised, i think we can get this approved in the next hour or so | 14:46 |
rosmaita | #topic specs - Support revert any snapshot to the volume | 14:46 |
*** openstack changes topic to "specs - Support revert any snapshot to the volume (Meeting topic: cinder)" | 14:46 | |
e0ne | :) | 14:46 |
rosmaita | #link https://review.opendev.org/#/c/736111/ | 14:46 |
rosmaita | smcginnis: thanks for your comment on there, i think you explained clearly why we want this to have a generic implementation | 14:47 |
rosmaita | that can be overriden by backends that can do better | 14:47 |
eharney | i think this spec needs more thought on the implications of doing this... it seems like a minefield of usability and perf problems | 14:47 |
smcginnis | I hate to be negative about it, because I do think it would be very useful. But there are many issues with trying to do this for all backends. | 14:47 |
eharney | it also doesn't explain how it actually works | 14:48 |
smcginnis | Very light on details. | 14:48 |
*** LiangFang has quit IRC | 14:48 | |
smcginnis | The work item list of "change restriction" is a little vague. :) | 14:48 |
rosmaita | ok, i think this one we ask them to continue working on for W | 14:48 |
rosmaita | smcginnis: i think W has a name? | 14:48 |
smcginnis | Wombat IIRC. | 14:48 |
rosmaita | glad i asked, i was thinking Wallaby | 14:49 |
rosmaita | ok, i will leave an encouraging comment on the spec | 14:49 |
smcginnis | Oh, maybe you're right. | 14:49 |
jungleboyj | I thought it was Wallaby. | 14:49 |
smcginnis | I really should know since I ran it, but now I forget. | 14:49 |
jungleboyj | I am pretty sure it is. | 14:49 |
smcginnis | Yeah, you're all right. Don't listen to me. :) | 14:49 |
eharney | this needs some details on what actually happens on a typical backend that can do this in the "ideal" way | 14:49 |
jungleboyj | Because that was the one I liked. | 14:49 |
smcginnis | eharney: ++ | 14:49 |
rosmaita | i was so devastated by my V name defeat, that i can't remember anything anymore | 14:49 |
eharney | because it's very unclear to the newer snapshots if you revert to an old snap | 14:50 |
eharney | unclear what happens to* | 14:50 |
jungleboyj | You weren't Victoriaous ? | 14:50 |
rosmaita | jungleboyj: :P | 14:50 |
smcginnis | eharney: I know at least for some it's an internal update of a pointer, but that's definitely not how all storage works. | 14:51 |
jungleboyj | This feels like one of those proposals that is a good idea but ends up being a dangerous minefield upon implementation. | 14:51 |
rosmaita | eharney: keep your thoughts coming, i will pull them out of the meeting log and put them on the spec | 14:51 |
rosmaita | ok, one final spec | 14:51 |
rosmaita | #topic specs - volume list query optimization | 14:51 |
*** openstack changes topic to "specs - volume list query optimization (Meeting topic: cinder)" | 14:51 | |
*** Liang__ has joined #openstack-meeting-alt | 14:51 | |
rosmaita | #link https://review.opendev.org/#/c/726070/ | 14:51 |
rosmaita | i think you may have seen my note to the ML | 14:52 |
rosmaita | i think this is really a bug | 14:52 |
rosmaita | or actually a set of 3 or so bugs | 14:52 |
rosmaita | but, it may still be worth a spec | 14:52 |
eharney | i think rosmaita teased out some good details on this that i wasn't able to quite grasp when we last discussed this | 14:52 |
rosmaita | becuase the fix will have to be in 2 places | 14:53 |
eharney | but i haven't followed up enough on picking through this yetr | 14:53 |
eharney | yet* | 14:53 |
rosmaita | rest api layer and also the db layer | 14:53 |
rosmaita | but i don't think it's going to require a mv change | 14:53 |
eharney | i would suggest not calling this an "optimization" | 14:54 |
rosmaita | yes, the title needs to change | 14:55 |
rosmaita | it's basically "correct the volume-list filtering" | 14:55 |
rosmaita | and maybe "correct the volume list" | 14:55 |
*** links has quit IRC | 14:55 | |
rosmaita | and maybe "never, ever do this kind of change again where we hijack the volume status to have internal states that aren't displayable in the rest api" | 14:56 |
eharney | lol | 14:56 |
rosmaita | i think maybe i will write this up as a bug, and tell the author to feel free to pick up the bug? | 14:57 |
jungleboyj | Two Minute warning. | 14:58 |
eharney | sensible enough | 14:58 |
rosmaita | ok, works for me | 14:58 |
jungleboyj | ++ | 14:58 |
rosmaita | ok, cool, we got through all the non-WIP specs | 14:58 |
rosmaita | #topic open discussion for 60 seconds | 14:59 |
*** openstack changes topic to "open discussion for 60 seconds (Meeting topic: cinder)" | 14:59 | |
jungleboyj | *celebration* | 14:59 |
tosky | regarding the cinder-plugin-ceph-tempest test, this patch may help on train and older branches, but it needs to go through master and ussuri first: https://review.opendev.org/#/c/738273/ | 14:59 |
* tosky was ready | 14:59 | |
rosmaita | ty tosky | 14:59 |
rosmaita | already has a +2! | 15:00 |
rosmaita | ok, gotta go | 15:00 |
rosmaita | #endmeeting | 15:00 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 15:00 | |
openstack | Meeting ended Wed Jul 1 15:00:19 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-07-01-14.00.html | 15:00 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-07-01-14.00.txt | 15:00 |
openstack | Log: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-07-01-14.00.log.html | 15:00 |
whoami-rajat | thanks | 15:00 |
jungleboyj | Thanks! | 15:00 |
rosmaita | apologies to horizon team for running over | 15:00 |
e0ne | rosmaita: np | 15:00 |
e0ne | #startmeeting horizon | 15:00 |
openstack | Meeting started Wed Jul 1 15:00:54 2020 UTC and is due to finish in 60 minutes. The chair is e0ne. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
*** openstack changes topic to " (Meeting topic: horizon)" | 15:00 | |
openstack | The meeting name has been set to 'horizon' | 15:00 |
vishalmanchanda | hi | 15:01 |
*** tosky has left #openstack-meeting-alt | 15:01 | |
e0ne | vishalmanchanda: hi | 15:01 |
e0ne | let's wait until more people will join us | 15:01 |
-amotoki- lurks here :) | 15:02 | |
rdopiera | o/ | 15:02 |
e0ne | vishalmanchanda: before we start. I was AFK whilte you were discussing cinder messages with amotoki | 15:02 |
e0ne | #topic Notices | 15:03 |
*** openstack changes topic to "Notices (Meeting topic: horizon)" | 15:03 | |
e0ne | actually, I don't have any updates this week | 15:03 |
e0ne | I'm going to setup a poll for our virtual mid-cycle later tonight to clarify what date and time work for us | 15:04 |
vishalmanchanda | +1. | 15:05 |
e0ne | in cinder we did it instead of a regular weekly meeting but it could be pretty late for vishalmanchanda and amotoki | 15:05 |
e0ne | it's supposed to be later this month around Victoria-2 milestone | 15:06 |
vishalmanchanda | e0ne: hmm.. I can manage if it work for everyone. | 15:06 |
amotoki | I missed meeting last two weeks as I was sleepy. I am trying to move my life rhythm earlier, but I got sleepy too early :-( | 15:07 |
amotoki | no intention to skip the meeting | 15:07 |
*** dave-mccowan has quit IRC | 15:08 | |
e0ne | amotokiL that's why I asked at PTG if a current time works good for everybody | 15:09 |
e0ne | #topic Stable maintenance and EOL | 15:10 |
*** openstack changes topic to "Stable maintenance and EOL (Meeting topic: horizon)" | 15:10 | |
e0ne | amotoki: you added this topic for PTG but we had no time to discuss it | 15:10 |
e0ne | #linke https://etherpad.opendev.org/p/horizon-v-ptg | 15:10 |
e0ne | I was waiting for all stable cores to discuss it | 15:11 |
*** dave-mccowan has joined #openstack-meeting-alt | 15:11 | |
e0ne | #link https://etherpad.opendev.org/p/horizon-v-ptg (line #105) | 15:12 |
amotoki | I just wonder who are stakeholders for older branches and whether we have bandwidth to check which stable branches including gate healthiness | 15:12 |
e0ne | personally, I'm not interested in ocata anymore | 15:12 |
e0ne | but I still have customers with pike and queens | 15:13 |
rdopiera | we use queens, stein and train | 15:13 |
e0ne | so I'm OK to mark ocata as EOL (end of life) | 15:13 |
amotoki | ocata is already under discussion for EOL | 15:13 |
amotoki | as it is not easy to maintain its gate related to py27 drop | 15:14 |
e0ne | that's true | 15:14 |
amotoki | as you know ocata will be EOL soon as a whole | 15:14 |
amotoki | it is a different story outside of horizon | 15:15 |
e0ne | +1 | 15:15 |
e0ne | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-May/015112.html | 15:16 |
amotoki | rocky and older are now in EM phase. which branches are you interested in? | 15:16 |
amotoki | according to the above comment from e0ne and rdopiera, pike is oldest | 15:17 |
amotoki | and queens has common interest at least from RH and mirantis | 15:17 |
e0ne | unfortunately, I have to support pike( | 15:17 |
amotoki | hehe, it is not so surprising | 15:18 |
amotoki | I think what we need to do are (1) to check gate healthiness in stable branches and then (2) fix gate failures if any | 15:19 |
e0ne | amotoki: I can do it | 15:19 |
amotoki | I believe periodic jobs help us for (1) as it is not easy to check it regularly | 15:20 |
amotoki | (2) is totally up to who can work on it. | 15:20 |
*** TusharTgite has quit IRC | 15:21 | |
e0ne | #action e0ne to check health of gates fro EM branches | 15:21 |
e0ne | at least, I'll check the current situation and will do the best to fix if gages are broken | 15:22 |
amotoki | I plan to backport https://review.opendev.org/#/c/737686/ to stable branches and then prepare the grafana dashboard at grafana.o.o. | 15:23 |
amotoki | we can check the healiness (for example) in the team meeting. | 15:23 |
e0ne | amotoki: awesome! | 15:23 |
vishalmanchanda | cool. | 15:23 |
e0ne | let's start with this and see how it works during the next meeting | 15:26 |
e0ne | #topic stable review policy | 15:27 |
*** openstack changes topic to "stable review policy (Meeting topic: horizon)" | 15:27 | |
e0ne | here is general guideline: https://docs.openstack.org/project-team-guide/stable-branches.html#review-guidelines | 15:29 |
e0ne | we follow it | 15:29 |
e0ne | but it seam reasonable to fit them a bit with a current situation in horizon team | 15:29 |
e0ne | from the etherpad: we previously had a policy to allow a single +2 if a backport is proposed by a stable core to handle the situation with only a couple of stable cores. | 15:30 |
e0ne | I didn't know about this rule before, but lt sounds absolutely reasonable for me | 15:30 |
e0ne | at least for a simple fixes | 15:31 |
amotoki | the comment is from me. It was true when Rob was the PTL | 15:31 |
e0ne | it seams it was introduced before I started contribution to horizon | 15:32 |
amotoki | this rule assumes that stable reviewers are familiar with the stable policy, so stable reviewers are assumed to consider the stable policy when they propose backports. | 15:33 |
*** Liang__ has quit IRC | 15:33 | |
amotoki | I beleve the current stable reviewers are familiar with the policy | 15:34 |
e0ne | amotoki: sure. in any case, stable core and put -1 or -2 for any proposed patch | 15:34 |
amotoki | yes, and if a stable core is not sure we can ping another stable core :) | 15:35 |
e0ne | +2 | 15:35 |
e0ne | or we can ping openstack stable team | 15:35 |
amotoki | yeah | 15:36 |
e0ne | rdopiera: are you agree with this proposal? | 15:36 |
amotoki | in such case we discuss it in the mailing list. it works in most cases I believe. | 15:36 |
rdopiera | e0ne: I don't like it, but I guess we don't have much choice | 15:39 |
e0ne | rdopiera: It's not ideal solution | 15:40 |
amotoki | or we can always ping second core. we have three active stable cores now, so it can work too. | 15:41 |
amotoki | (as we do now) | 15:42 |
amotoki | I am just afraid what happens if my activity slows down. | 15:43 |
amotoki | I want not be a bottleneck :) | 15:44 |
rdopiera | let's try it, and we can always get back to the old way | 15:47 |
e0ne | we can try to do stable reviews on a weekly basis | 15:47 |
e0ne | what is our agreement on this? | 15:49 |
* rdopiera agrees | 15:50 | |
amotoki | the original proposal is from me, so I have no objection. If you are okay, we can try a single +2 from bakcports from stable cores. | 15:51 |
amotoki | looks like we agree | 15:51 |
e0ne | #agreed to allow a single +2 if a backport is proposed by a stable core to handle the situation with only a couple of stable cores | 15:51 |
e0ne | we can re-visit our decision in the future if any objection will be | 15:52 |
e0ne | #topic Open Discussion | 15:53 |
*** openstack changes topic to "Open Discussion (Meeting topic: horizon)" | 15:53 | |
amotoki | let's raise if you see any question or issue in IRC or meeting :) | 15:53 |
vishalmanchanda | I want to discuss this cinder messages bp. | 15:53 |
e0ne | we've got seven minutes left | 15:53 |
vishalmanchanda | I have already discuss some points with amotoki and I am almost agrees with him. | 15:53 |
vishalmanchanda | e0ne: I also need your opinion on this as you draft this bp for horizon. | 15:54 |
e0ne | two cores agreed, I don't want to argue :) | 15:54 |
vishalmanchanda | #link https://specs.openstack.org/openstack/cinder-specs/specs/newton/summarymessage.html | 15:54 |
e0ne | vishalmanchanda: FYI, I'm going to add some backup-related messages this cycle | 15:55 |
vishalmanchanda | e0ne: nice. | 15:55 |
vishalmanchanda | e0ne: What's you think how Can we show it in horizon? | 15:56 |
amotoki | as summary, I pointed that messages related to snapshots need to be covered in the volume detail page and whether it fits normal user scenarios. | 15:56 |
amotoki | and if a user have some problem(s) on a snapshot they need to check the volume detail instead of the snapshot detail in the current proposal. | 15:57 |
amotoki | honestly I am not sure we need to cover all messages related to a specific volume including snapshots (and potentially backups) in the volume detail. | 15:58 |
amotoki | so it leads to my question on user scenarios. | 15:59 |
e0ne | it's a good question | 16:00 |
e0ne | I need to think a bit | 16:00 |
e0ne | we're out of time | 16:00 |
e0ne | let's continue discussion in the #openstack-horizon channel | 16:01 |
e0ne | #endmeeting | 16:01 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 16:01 | |
openstack | Meeting ended Wed Jul 1 16:01:20 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-07-01-15.00.html | 16:01 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-07-01-15.00.txt | 16:01 |
openstack | Log: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-07-01-15.00.log.html | 16:01 |
amotoki | o/ | 16:01 |
*** ricolin has quit IRC | 16:01 | |
*** liuyulong has quit IRC | 16:05 | |
*** ttsiouts has quit IRC | 16:06 | |
*** ttsiouts has joined #openstack-meeting-alt | 16:07 | |
*** ttsiouts has quit IRC | 16:11 | |
*** derekh has quit IRC | 16:15 | |
*** sfernand has quit IRC | 17:02 | |
*** ralonsoh has quit IRC | 17:10 | |
*** ralonsoh has joined #openstack-meeting-alt | 17:10 | |
*** bnemec has quit IRC | 17:15 | |
*** ganso has quit IRC | 17:15 | |
*** ganso has joined #openstack-meeting-alt | 17:18 | |
*** bnemec has joined #openstack-meeting-alt | 17:21 | |
*** rnoriega- has quit IRC | 17:26 | |
*** jtomasek_ has quit IRC | 17:26 | |
*** megheisler has quit IRC | 17:26 | |
*** tsmith2 has quit IRC | 17:26 | |
*** zzzeek has quit IRC | 17:26 | |
*** freerunner has quit IRC | 17:26 | |
*** hongbin has joined #openstack-meeting-alt | 17:28 | |
*** irclogbot_1 has quit IRC | 17:28 | |
*** gyee has joined #openstack-meeting-alt | 17:29 | |
*** rnoriega- has joined #openstack-meeting-alt | 17:29 | |
*** jtomasek_ has joined #openstack-meeting-alt | 17:29 | |
*** megheisler has joined #openstack-meeting-alt | 17:29 | |
*** tsmith2 has joined #openstack-meeting-alt | 17:29 | |
*** zzzeek has joined #openstack-meeting-alt | 17:29 | |
*** freerunner has joined #openstack-meeting-alt | 17:29 | |
*** irclogbot_3 has joined #openstack-meeting-alt | 17:30 | |
*** yamamoto has joined #openstack-meeting-alt | 17:44 | |
*** yamamoto has quit IRC | 17:53 | |
*** e0ne has quit IRC | 17:59 | |
*** eharney has quit IRC | 18:16 | |
*** slaweq has quit IRC | 18:25 | |
*** hongbin has quit IRC | 18:44 | |
*** rosmaita has left #openstack-meeting-alt | 18:46 | |
*** hongbin has joined #openstack-meeting-alt | 19:18 | |
*** rdopiera has quit IRC | 19:41 | |
*** vishalmanchanda has quit IRC | 19:47 | |
*** ralonsoh has quit IRC | 19:50 | |
*** eharney has joined #openstack-meeting-alt | 19:59 | |
*** hongbin has quit IRC | 20:06 | |
*** e0ne has joined #openstack-meeting-alt | 20:20 | |
*** slaweq has joined #openstack-meeting-alt | 20:25 | |
*** hongbin has joined #openstack-meeting-alt | 20:54 | |
*** ircuser-1 has quit IRC | 21:03 | |
*** e0ne has quit IRC | 21:07 | |
*** slaweq has quit IRC | 21:19 | |
*** priteau has quit IRC | 21:21 | |
*** rcernin has joined #openstack-meeting-alt | 22:37 | |
*** rcernin has quit IRC | 22:47 | |
*** rcernin has joined #openstack-meeting-alt | 22:47 | |
*** hongbin has quit IRC | 23:11 | |
*** hongbin has joined #openstack-meeting-alt | 23:16 | |
*** rfolco has quit IRC | 23:48 | |
*** yamamoto has joined #openstack-meeting-alt | 23:52 | |
*** yamamoto has quit IRC | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!