| *** jkulik has quit IRC | 00:02 | |
| *** jkulik has joined #openstack-meeting-alt | 00:02 | |
| *** eharney has quit IRC | 00:29 | |
| *** vishalmanchanda has quit IRC | 00:41 | |
| *** eharney has joined #openstack-meeting-alt | 00:43 | |
| *** rcernin has quit IRC | 01:43 | |
| *** rcernin has joined #openstack-meeting-alt | 02:20 | |
| *** rcernin has quit IRC | 02:24 | |
| *** rcernin has joined #openstack-meeting-alt | 02:28 | |
| *** rcernin has quit IRC | 02:31 | |
| *** rcernin has joined #openstack-meeting-alt | 02:32 | |
| *** yamamoto has quit IRC | 04:14 | |
| *** yamamoto has joined #openstack-meeting-alt | 04:14 | |
| *** yamamoto has quit IRC | 04:19 | |
| *** yamamoto has joined #openstack-meeting-alt | 04:19 | |
| *** zzzeek has quit IRC | 04:24 | |
| *** zzzeek has joined #openstack-meeting-alt | 04:25 | |
| *** yamamoto has quit IRC | 04:30 | |
| *** yamamoto has joined #openstack-meeting-alt | 04:43 | |
| *** zzzeek has quit IRC | 04:46 | |
| *** zzzeek has joined #openstack-meeting-alt | 04:49 | |
| *** yamamoto has quit IRC | 04:59 | |
| *** yamamoto has joined #openstack-meeting-alt | 05:03 | |
| *** yamamoto has quit IRC | 05:03 | |
| *** yamamoto has joined #openstack-meeting-alt | 05:03 | |
| *** yamamoto has quit IRC | 05:08 | |
| *** zzzeek has quit IRC | 05:47 | |
| *** zzzeek has joined #openstack-meeting-alt | 05:48 | |
| *** vishalmanchanda has joined #openstack-meeting-alt | 06:00 | |
| *** miniroy has quit IRC | 06:05 | |
| *** gyee has quit IRC | 06:08 | |
| *** yamamoto has joined #openstack-meeting-alt | 07:07 | |
| *** yamamoto has quit IRC | 07:18 | |
| *** yamamoto has joined #openstack-meeting-alt | 07:21 | |
| *** yamamoto has quit IRC | 07:22 | |
| *** ralonsoh has joined #openstack-meeting-alt | 07:31 | |
| *** slaweq has joined #openstack-meeting-alt | 07:43 | |
| *** yamamoto has joined #openstack-meeting-alt | 07:58 | |
| *** yamamoto has quit IRC | 08:12 | |
| *** lbragstad_ has joined #openstack-meeting-alt | 08:18 | |
| *** lbragstad has quit IRC | 08:22 | |
| *** tosky has joined #openstack-meeting-alt | 08:45 | |
| *** rcernin has quit IRC | 08:52 | |
| *** derekh has joined #openstack-meeting-alt | 09:10 | |
| *** rcernin has joined #openstack-meeting-alt | 09:15 | |
| *** slaweq has quit IRC | 09:17 | |
| *** slaweq has joined #openstack-meeting-alt | 09:19 | |
| *** rdopiera has joined #openstack-meeting-alt | 09:31 | |
| *** e0ne has joined #openstack-meeting-alt | 09:38 | |
| *** rcernin has quit IRC | 09:45 | |
| *** rcernin has joined #openstack-meeting-alt | 09:55 | |
| *** zzzeek has quit IRC | 09:56 | |
| *** zzzeek has joined #openstack-meeting-alt | 09:59 | |
| *** michael-mcaleer has joined #openstack-meeting-alt | 10:03 | |
| *** michael-mcaleer has quit IRC | 10:08 | |
| *** michael-mcaleer has joined #openstack-meeting-alt | 10:13 | |
| *** michael-mcaleer has quit IRC | 10:17 | |
| *** kevinz has quit IRC | 10:26 | |
| *** michael-mcaleer has joined #openstack-meeting-alt | 10:28 | |
| *** michael-mcaleer has quit IRC | 10:36 | |
| *** belmoreira has joined #openstack-meeting-alt | 10:46 | |
| *** michael-mcaleer has joined #openstack-meeting-alt | 11:02 | |
| *** lpetrut has joined #openstack-meeting-alt | 11:10 | |
| *** rcernin has quit IRC | 11:18 | |
| *** rcernin has joined #openstack-meeting-alt | 11:19 | |
| *** rcernin has quit IRC | 11:27 | |
| *** baojg has quit IRC | 11:30 | |
| *** baojg has joined #openstack-meeting-alt | 11:31 | |
| *** rcernin has joined #openstack-meeting-alt | 11:33 | |
| *** slaweq has quit IRC | 11:39 | |
| *** yamamoto has joined #openstack-meeting-alt | 11:39 | |
| *** e0ne has quit IRC | 11:44 | |
| *** slaweq has joined #openstack-meeting-alt | 11:45 | |
| *** rcernin has quit IRC | 11:46 | |
| *** rcernin has joined #openstack-meeting-alt | 11:52 | |
| *** rcernin has quit IRC | 11:55 | |
| *** zzzeek has quit IRC | 12:13 | |
| *** zzzeek has joined #openstack-meeting-alt | 12:16 | |
| *** e0ne has joined #openstack-meeting-alt | 12:18 | |
| *** zzzeek has quit IRC | 12:27 | |
| *** zzzeek has joined #openstack-meeting-alt | 12:28 | |
| *** sfernand has joined #openstack-meeting-alt | 12:37 | |
| *** yamamoto has quit IRC | 12:43 | |
| *** baojg has quit IRC | 12:48 | |
| *** baojg has joined #openstack-meeting-alt | 12:49 | |
| *** enriquetaso has joined #openstack-meeting-alt | 13:01 | |
| *** masahito has joined #openstack-meeting-alt | 13:03 | |
| *** yamamoto has joined #openstack-meeting-alt | 13:05 | |
| *** yamamoto has quit IRC | 13:05 | |
| *** yamamoto has joined #openstack-meeting-alt | 13:05 | |
| *** yamamoto has quit IRC | 13:10 | |
| *** yamamoto has joined #openstack-meeting-alt | 13:26 | |
| *** yamamoto has quit IRC | 13:26 | |
| *** yamamoto has joined #openstack-meeting-alt | 13:27 | |
| *** yamamoto has quit IRC | 13:31 | |
| *** rosmaita has joined #openstack-meeting-alt | 13:45 | |
| *** e0ne has quit IRC | 13:51 | |
| *** masahito has quit IRC | 13:52 | |
| *** sangeet has quit IRC | 13:52 | |
| *** rafaelweingartne has joined #openstack-meeting-alt | 13:53 | |
| *** sangeet has joined #openstack-meeting-alt | 13:55 | |
| *** zoharm has joined #openstack-meeting-alt | 13:57 | |
| *** abishop has joined #openstack-meeting-alt | 13:58 | |
| *** masahito has joined #openstack-meeting-alt | 13:58 | |
| *** ChuckPiercey has joined #openstack-meeting-alt | 13:58 | |
| *** ChuckPiercey has quit IRC | 13:59 | |
| *** ChuckPiercey has joined #openstack-meeting-alt | 13:59 | |
| *** geguileo has joined #openstack-meeting-alt | 14:00 | |
| rosmaita | #startmeeting cinder | 14:00 |
|---|---|---|
| openstack | Meeting started Wed Nov 18 14:00:45 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 |
| rosmaita | #topic roll call | 14:00 |
| *** openstack changes topic to "roll call (Meeting topic: cinder)" | 14:00 | |
| rafaelweingartne | \o | 14:00 |
| *** e0ne has joined #openstack-meeting-alt | 14:01 | |
| abishop | o/ | 14:01 |
| eharney | hi | 14:01 |
| michael-mcaleer | hi | 14:01 |
| sfernand | hi | 14:01 |
| whoami-rajat__ | Hi | 14:01 |
| e0ne | hi | 14:01 |
| jungleboyj | o/ | 14:01 |
| ChuckPiercey | Hi | 14:01 |
| zoharm | o/ | 14:02 |
| geguileo | hi! o/ | 14:02 |
| lseki | o/ | 14:02 |
| rosmaita | looks like a good turnout | 14:03 |
| rosmaita | #link https://etherpad.openstack.org/p/cinder-victoria-meetings | 14:03 |
| rosmaita | ok, let's get started | 14:03 |
| ChuckPiercey | Topic: a couple of KIOXIA dev team questions | 14:03 |
| rosmaita | #topic announcements | 14:03 |
| *** openstack changes topic to "announcements (Meeting topic: cinder)" | 14:03 | |
| enriquetaso | hi | 14:03 |
| rosmaita | ChuckPiercey: you are on the agenda for later | 14:03 |
| rosmaita | hi sofia! | 14:03 |
| rosmaita | hope your exams went well | 14:04 |
| enriquetaso | o/ :) | 14:04 |
| enriquetaso | thanks! | 14:04 |
| rosmaita | reminder: next week's meeting (last meeting of the month) is videoconference | 14:04 |
| rosmaita | it will be recorded in bluejeans as usual, unless there are any objections? | 14:04 |
| rosmaita | i will send out connection info to the ML (though it has not changed from last time) | 14:04 |
| *** thgcorrea has joined #openstack-meeting-alt | 14:04 | |
| rosmaita | #topic announcements - upcoming events | 14:05 |
| *** openstack changes topic to "announcements - upcoming events (Meeting topic: cinder)" | 14:05 | |
| rosmaita | this week is R-21 | 14:05 |
| rosmaita | Wallaby Milestone-1 is at R-19 (yes, in 2 weeks) | 14:05 |
| rosmaita | i want to set some priorities for the next 2 weeks | 14:05 |
| rosmaita | priority for M-1: adding mypy type-checking | 14:05 |
| rosmaita | here are the patches | 14:05 |
| rosmaita | #link https://review.opendev.org/#/q/topic:mypy+(status:open+OR+status:merged)+project:openstack/cinder | 14:06 |
| rosmaita | this is an important code quality initiative | 14:06 |
| rosmaita | and it has been languishing despite the best efforts of eharney | 14:06 |
| rosmaita | so, i propose that each cinder core will take over one of these patches | 14:06 |
| rosmaita | except eharney, of course, but he will be available to answer questions that come up | 14:07 |
| eharney | i'll bump those in merge conflict today | 14:07 |
| rosmaita | i was going to propose that whoever takes over the patch should fix the merge conflict | 14:07 |
| eharney | that also works | 14:07 |
| rosmaita | so this will be slightly weird, a reviewer will also be a committer | 14:08 |
| *** macz_ has joined #openstack-meeting-alt | 14:08 | |
| rosmaita | but i think that is the best way to handle minor changes | 14:08 |
| rosmaita | and if a reviewer decides to make a major change, then they can't +2 the patch, but they can still be "in charge" of getting it merged | 14:08 |
| rosmaita | so we will need to use common sense here | 14:08 |
| rosmaita | so, to save people the embarrassment of volunteering, here's my proposal: | 14:09 |
| rosmaita | mypy: driver.py https://review.opendev.org/736856 hemna | 14:09 |
| rosmaita | mypy: annotate volume manager https://review.opendev.org/733621 whoami-rajat | 14:09 |
| rosmaita | mypy: annotate volume_utils / utils / exc https://review.opendev.org/736855 rosmaita | 14:09 |
| rosmaita | mypy: annotate remotefs https://review.opendev.org/733623 lseki | 14:09 |
| rosmaita | mypy: annotate lvm https://review.opendev.org/733624 geguileo | 14:09 |
| rosmaita | mypy: annotate api.py https://review.opendev.org/733622 rosmaita | 14:09 |
| rosmaita | ok, e0ne don't feel left out, you are heading up the v2 removal effort | 14:09 |
| rosmaita | and you will likely be bugged for reviews | 14:10 |
| e0ne | rosmaita: I'm sorry, I was busy with our internal release last week | 14:10 |
| rosmaita | is everyone clear on what i am asking for here? | 14:10 |
| rosmaita | e0ne: no need to apologize | 14:10 |
| e0ne | so there were no community activity from my side | 14:10 |
| rosmaita | ummmm ... i could use an 'ack' from hemna whoami-rajat__ lseki geguileo ^^ | 14:11 |
| geguileo | rosmaita: yes, to do a massive downvote of eharney's patches, right? | 14:11 |
| whoami-rajat__ | ack | 14:11 |
| lseki | ack 👍 | 14:11 |
| eharney | geguileo: no need to frown, just turn that downvote upside-down | 14:11 |
| rosmaita | there you go | 14:12 |
| geguileo | eharney: rofl | 14:12 |
| jungleboyj | :-) | 14:12 |
| *** linshiyx has joined #openstack-meeting-alt | 14:12 | |
| rosmaita | jungleboyj and smcginnis, i suspect you will be bugged for reviews | 14:12 |
| rafaelweingartne | hello guys, are we going to have an open floor topic? I would like to ask your guidance in a patch that has been open for quite a while for Cinder | 14:12 |
| jungleboyj | rosmaita: ++ Can do. | 14:12 |
| *** simondodsley has joined #openstack-meeting-alt | 14:12 | |
| *** macz_ has quit IRC | 14:12 | |
| rosmaita | rafaelweingartne: hopefully later in the meeting | 14:13 |
| rosmaita | community goals are looking OK, details are on the agenda | 14:13 |
| rafaelweingartne | rosmaita: thanks :) | 14:13 |
| rosmaita | e0ne is heading up v2 removal, which we would like to have completed before M-2 | 14:14 |
| rosmaita | he can maybe talk about that next week? | 14:14 |
| e0ne | rosmaita: sounds good to me | 14:14 |
| e0ne | need to figure out what is going with grenade without v2 | 14:14 |
| rosmaita | great! | 14:14 |
| rosmaita | #topic announcements - midcycle | 14:14 |
| *** openstack changes topic to "announcements - midcycle (Meeting topic: cinder)" | 14:14 | |
| rosmaita | yeah, grenade will be ... interesting | 14:14 |
| rosmaita | ok the midcycle is supposed to be before specs freeze, which is coming up fast also | 14:15 |
| rosmaita | here is my proposal: | 14:15 |
| rosmaita | hold it on Wednesday 9 December, 2 hours either 1300-1500 or 1400-1600 | 14:15 |
| rosmaita | that way it overlaps with a time everyone has reserved on wednesday already | 14:15 |
| rosmaita | so i expect most people will at least be able to attend for an hour | 14:16 |
| e0ne | +1 | 14:16 |
| jungleboyj | I would vote for 14:00 to 16:00. | 14:16 |
| rosmaita | anyway, please check your calendars and we can verify/vote on a time next week | 14:16 |
| jungleboyj | ++ | 14:16 |
| lseki | ++ | 14:17 |
| rosmaita | ok and just to have it in the minutes: | 14:17 |
| rosmaita | "All Cinder Specs for features to be implemented in Wallaby must be approved by Friday 18 December 2020 (23:59 UTC)." | 14:17 |
| rosmaita | and finally | 14:17 |
| rosmaita | Wallaby Milestone-2 is at R-12 (week of 18 January) | 14:17 |
| rosmaita | ok, but the big effort is getting mypy merged before M-1 | 14:18 |
| rosmaita | and i forgot to mention the other part of my proposal about that | 14:18 |
| rosmaita | anyone who asks for reviews in #openstack-cinder and has not reviewed one of the mypy patches will be shunned | 14:18 |
| rosmaita | #link https://review.opendev.org/#/q/topic:mypy+(status:open+OR+status:merged)+project:openstack/cinder | 14:18 |
| rosmaita | the mypy patches for reference ^^ | 14:19 |
| rosmaita | ok, that's all for announcements | 14:19 |
| rosmaita | #topic bug review | 14:19 |
| *** openstack changes topic to "bug review (Meeting topic: cinder)" | 14:19 | |
| rosmaita | michael-mcaleer: that's you | 14:19 |
| michael-mcaleer | thanks rosmaita | 14:19 |
| michael-mcaleer | 6 bugs this week, all in cinder (3 x cinder, 1 x enhancement, 1 x doc, 1 x driver) | 14:19 |
| michael-mcaleer | cinder first | 14:19 |
| michael-mcaleer | Cinder backup error using Erasure Code Pool #link https://bugs.launchpad.net/cinder/+bug/1903713 | 14:19 |
| openstack | Launchpad bug 1903713 in Cinder "Cinder backup error using Erasure Code Pool" [Undecided,New] | 14:19 |
| michael-mcaleer | Not immediately obvious what is going on here, asked for more info from submitter | 14:20 |
| michael-mcaleer | importance also not set, anyone have a suggestion on that? | 14:20 |
| rosmaita | e0ne: can you look at this one? ceph + backup, kind of in your domain there | 14:21 |
| e0ne | rosmaita: sure, will take a look on it asap | 14:22 |
| rosmaita | ty! | 14:22 |
| michael-mcaleer | ok next bug | 14:22 |
| michael-mcaleer | cinder backup-import command fails, there is no way to repeat it #link https://bugs.launchpad.net/cinder/+bug/1903697 | 14:22 |
| openstack | Launchpad bug 1903697 in Cinder "cinder backup-import command fails, there is no way to repeat it" [Undecided,Incomplete] | 14:22 |
| michael-mcaleer | again not a lot of info provided here but its possible to discern what is going on in advance of logs coming back from submitter | 14:22 |
| michael-mcaleer | assume low importance because user can just change name of backup | 14:23 |
| rosmaita | sounds correct | 14:23 |
| michael-mcaleer | ok, last cinder bug | 14:23 |
| e0ne | I can check it with the latest master | 14:23 |
| michael-mcaleer | Cinder backup - Volume metadata inconsistency #link https://bugs.launchpad.net/cinder/+bug/1903839 | 14:23 |
| openstack | Launchpad bug 1903839 in Cinder "Cinder backup - Volume metadata inconsistency" [Medium,Confirmed] | 14:23 |
| michael-mcaleer | opened by powermax team, our QE team spotted this in April and we are only getting round to investigating now | 14:24 |
| eharney | this driver might be making invalid assumptions about how volume metadata works | 14:24 |
| michael-mcaleer | metadata is assumed to be unique to each volume here | 14:24 |
| e0ne | it's a Backup Bug Day | 14:24 |
| eharney | i don't think it's a bug that restoring a volume from backup restores its metadata | 14:25 |
| rosmaita | :) | 14:25 |
| eharney | also, this is end-user metadata and info used by the driver probably shouldn't be in that field | 14:25 |
| michael-mcaleer | ok, im going to stay on topic of bugs and we can discuss the ins and outs of this one in a bug review, save some time | 14:25 |
| eharney | yeah | 14:25 |
| michael-mcaleer | next up is an enhancement | 14:26 |
| michael-mcaleer | Cinder - volume show for GVG #link https://bugs.launchpad.net/cinder/+bug/1903845 | 14:26 |
| openstack | Launchpad bug 1903845 in Cinder "Cinder - volume show for GVG" [Wishlist,Triaged] | 14:26 |
| michael-mcaleer | Consider updating volume properties when its added to an Generic Volume Group. | 14:26 |
| eharney | makes sense to consider since we do it w/ CGs | 14:26 |
| michael-mcaleer | there is consistency group ID in there but no GVGs | 14:26 |
| michael-mcaleer | yeah | 14:26 |
| michael-mcaleer | set to enhancement/wish | 14:27 |
| rosmaita | sounds good | 14:27 |
| michael-mcaleer | last cinder related bug is docs... Rocky Series Release Notes in Cinder Release Notes #link https://bugs.launchpad.net/cinder/+bug/1904219 | 14:27 |
| openstack | Launchpad bug 1904219 in Cinder "Rocky Series Release Notes in Cinder Release Notes" [Low,Confirmed] | 14:27 |
| michael-mcaleer | simple fix here, incorrect designation of code snippet in the release notes | 14:27 |
| *** tmazur has joined #openstack-meeting-alt | 14:27 | |
| michael-mcaleer | and lastly the driver bug opened by ibm... os_type changed to 'linux' in python3 #link https://bugs.launchpad.net/cinder/+bug/1903648 | 14:28 |
| openstack | Launchpad bug 1903648 in Cinder "os_type changed to 'linux' in python3" [Low,Triaged] | 14:28 |
| michael-mcaleer | again a small fix here, ive set the tags for ibm so hopefully they see if if they didnt open it themselves | 14:28 |
| smcginnis | We can't update old release notes, otherwise they show up as new. There's also an outstanding bug in reno that may be causing that one. | 14:28 |
| rosmaita | smcginnis: thanks, the moral of the story is to make sure command line text is wrapped in `` in the releasenotes | 14:29 |
| rosmaita | (for future reviewers) | 14:30 |
| rosmaita | ok, thank you michael-mcaleer | 14:30 |
| michael-mcaleer | will I close that bug out and advise submitter on updating of release notes? | 14:30 |
| michael-mcaleer | no prob, thats all from me this week, thanks rosmaita | 14:30 |
| rosmaita | michael-mcaleer: yes, that sounds good | 14:30 |
| rosmaita | #topic stable releases | 14:30 |
| *** openstack changes topic to "stable releases (Meeting topic: cinder)" | 14:30 | |
| rosmaita | whoami-rajat__: that's you | 14:30 |
| whoami-rajat__ | Hi, just to provide an update on the patches to be merged before the stable release which we decided to be on 26th Nov | 14:31 |
| whoami-rajat__ | #link https://etherpad.opendev.org/p/stable-releases-review-tracker | 14:31 |
| rosmaita | oops, forgot to mention that in the updates | 14:31 |
| whoami-rajat__ | a lot of patches were merged in the past week, thanks to all the reviewers | 14:31 |
| whoami-rajat__ | victoria looks good to go, ussuri has 1 patch remaining, train has 4 patches in which there's a -1 on hemna 's patch from smcginnis | 14:32 |
| whoami-rajat__ | hopefully all patches will be merged before next cinder meeting to meet the release deadline on time | 14:33 |
| whoami-rajat__ | that's all from my side | 14:33 |
| rosmaita | ok, stable cores: ^^ | 14:33 |
| rosmaita | thanks, whoami-rajat__ | 14:33 |
| rosmaita | #topic marking config options as 'advanced' | 14:34 |
| *** openstack changes topic to "marking config options as 'advanced' (Meeting topic: cinder)" | 14:34 | |
| rosmaita | just want to bring to people's attention that this is possible | 14:34 |
| rosmaita | useful for config options that should only have their values changed for very specific reasons | 14:34 |
| rosmaita | anyway, take a look at this patch and read my comment | 14:34 |
| rosmaita | #link https://review.opendev.org/#/c/744746/ | 14:35 |
| rosmaita | #topic v2 support removal from cinderclient and last call for comments | 14:35 |
| *** openstack changes topic to "v2 support removal from cinderclient and last call for comments (Meeting topic: cinder)" | 14:35 | |
| rosmaita | i sent an announcement to the ML | 14:35 |
| rosmaita | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018697.html | 14:35 |
| rosmaita | and, there has been no response | 14:36 |
| rosmaita | so we have the green light to remove v2 support from the python-cinderclient this release cycle | 14:36 |
| rosmaita | hooray! | 14:36 |
| jungleboyj | Onward! | 14:36 |
| rosmaita | #topic restricting supported Ceph versions | 14:36 |
| *** openstack changes topic to "restricting supported Ceph versions (Meeting topic: cinder)" | 14:36 | |
| abishop | I know there are patches submitted to puppet-cinder and tripleo to prepare for v2 removal | 14:36 |
| rosmaita | ok, i also sent an email out about this to the ML asking for comments on the review | 14:37 |
| rosmaita | #link https://review.opendev.org/#/c/762592/ | 14:37 |
| rosmaita | but hemna raised an interesting question | 14:37 |
| rosmaita | btw, thanks for geguileo for also responding to a comment about this on the ML | 14:38 |
| rosmaita | so, i was looking at this like an announcement for how people should behave | 14:38 |
| rosmaita | that is, do not use unsupported ceph versions and expect it to work | 14:38 |
| rosmaita | but, hemna pointed out that if we don't enforce it in code, they could do it anyway | 14:38 |
| eharney | i think the main goal is in fact to just say "don't do that" | 14:39 |
| geguileo | eharney: +1 | 14:39 |
| rosmaita | that was my thought, too, but i wanted to check | 14:39 |
| eharney | which is the current state of affairs today -- and i don't recall us ever having issues that came up because someone was using an older version of ceph that we don't test | 14:39 |
| eharney | however | 14:39 |
| smcginnis | Then if someone comes along and says "hey, this doesn't work", we can point to the docs. | 14:39 |
| geguileo | but I'm ok adding a patch to check it | 14:39 |
| eharney | we are adding some advisory messages around the v2 clone API that says "since you are using this old version, your performance is worse" | 14:39 |
| jungleboyj | RTFM. ;-) | 14:40 |
| geguileo | lol | 14:40 |
| geguileo | eharney: once that patch merges we can have another patch that removes those and does a check on the setup, right? | 14:40 |
| rosmaita | ok, i think advisory is good for now, and if something needs code validation, then we can still add it at the appropriate time | 14:41 |
| eharney | geguileo: not sure, that one is less straightforward since it also requires backend configuration | 14:41 |
| geguileo | eharney: can't we just check the cluster version? | 14:41 |
| eharney | no | 14:42 |
| geguileo | eharney: isn't that enough? | 14:42 |
| eharney | because they still have lower perf if they don't manually set some options | 14:42 |
| eharney | and even if they set the options, it won't be applied retroactively to old images, etc | 14:42 |
| geguileo | eharney: ok, we cannot remove it, but we could barf if the ceph cluster version is not the right one | 14:43 |
| geguileo | (which is what hemna was asking for) | 14:43 |
| eharney | i don't buy that failing because they are using an old ceph version really accomplishes much for us | 14:44 |
| rosmaita | ok, let's continue discussion of this on the patch | 14:44 |
| rosmaita | #topic replication support for nvme connector - code structure | 14:44 |
| *** openstack changes topic to "replication support for nvme connector - code structure (Meeting topic: cinder)" | 14:44 | |
| rosmaita | zoharm: that's you | 14:44 |
| zoharm | hi all | 14:44 |
| zoharm | so we are working on upgrading the NVMe connector to also do replication | 14:44 |
| zoharm | the main question in the high level, how do we structure it in regards to the existing NVMe connector? | 14:45 |
| zoharm | for example, is it ok to tag alone the raid functionality for the connect/disconnect methods, if the connection properties specify replication? | 14:46 |
| zoharm | in a sense, this means expanding the connector to have two modes, and do one depending on whether the nvme volumes to connect to are replicated | 14:47 |
| zoharm | does this sound like the right way? | 14:47 |
| geguileo | zoharm: sorry, I'm not quite following | 14:47 |
| zoharm | are there any other suggestions or issues with this? | 14:47 |
| geguileo | zoharm: do you have an example of the connection_info dict? | 14:48 |
| geguileo | it's probably easier for me to understand what you are actually proposing like that | 14:48 |
| zoharm | yes, so basically, if the connection_info dict has a field "volume_replicas" with connection info for each nvme device in it | 14:48 |
| zoharm | then, put them into raid | 14:48 |
| zoharm | for bare nvme its: | 14:49 |
| zoharm | connection_info: | 14:49 |
| zoharm | nqn | 14:49 |
| zoharm | portal | 14:49 |
| rafaelweingartne | It might be easier to write down a spec and propose it there. Then, the discussion can happen within a context, and maybe with some examples on what you want to accomplish | 14:49 |
| geguileo | zoharm: That idea does sound good to me | 14:49 |
| zoharm | ok, just to finish this part | 14:49 |
| zoharm | if connection info looks like this: | 14:49 |
| zoharm | conection_info: | 14:49 |
| geguileo | zoharm: and it would not conflict with adding multipathing to the nvme connector later on | 14:49 |
| zoharm | volume_repliacs: | 14:49 |
| zoharm | nqn, etc... | 14:50 |
| zoharm | yep geguileo | 14:50 |
| zoharm | so multipathing will be handled under a "portals" field for each nvme device | 14:50 |
| rosmaita | is the question whether it's better to complicate the connect_volume function in current connector, or subclass it with a new connector that does replication? | 14:50 |
| zoharm | which can have multiple portals | 14:50 |
| geguileo | yeah, volume_replicas would be a list of connection_info that just doesn't have the volume_replicas in it | 14:50 |
| zoharm | rosmaita, in a way yes, and if subclassing is better, how to do it right? | 14:51 |
| geguileo | I think sublcassing is probably better for readability | 14:52 |
| geguileo | the base nvme-of class can just do simle connections | 14:52 |
| zoharm | true, though biggest benefit for readability is breaking up into re-usable functions (which are usable by both bare nvme and replicated) | 14:52 |
| geguileo | and the raid version inherits from it and handles the replicas with specific code, but calls the base nvme-of class for individual connections | 14:52 |
| zoharm | right, however, if a single volume backends supports both simple and replicated volumes, this complicates things | 14:53 |
| zoharm | additionally, a new connector and libvirt volume driver needs to be named and registered | 14:53 |
| *** cliffparsons has quit IRC | 14:53 | |
| geguileo | zoharm: why a new libvirt volume driver? | 14:53 |
| geguileo | zoharm: can't we just create a new os-brick connector? | 14:53 |
| zoharm | yes, and then we will need to tell nova to use it somehow | 14:54 |
| *** cliffparsons has joined #openstack-meeting-alt | 14:54 | |
| rosmaita | zoharm: best thing would be to write the way you think is correct in the spec, and say in the alternatives section why you don't like the other option | 14:54 |
| rosmaita | then we'll have a better basis for discussion | 14:54 |
| zoharm | as far as i saw, this means adding it to libvirt/drivers | 14:54 |
| geguileo | zoharm: you are correct | 14:55 |
| zoharm | because current nvme connector is referenced in the same way in its own libvirt driver | 14:55 |
| zoharm | so to make this work we would need to just duplicate this whole wrapping, and just change one name | 14:55 |
| zoharm | this is another tradeoff for this approach | 14:55 |
| zoharm | also it requires a code submission to nova :) | 14:55 |
| geguileo | zoharm: you don't need to replicate the wrapping, you can just reference the same class in nova/virt/libvirt/driver.py | 14:56 |
| geguileo | but with the name of the new connector | 14:56 |
| geguileo | in libvirt_volume_drivers | 14:56 |
| geguileo | 'nvmeof-raid=nova.virt.libvirt.volume.nvme.LibvirtNVMEVolumeDriver' | 14:57 |
| rosmaita | zoharm: you also had another question? | 14:57 |
| zoharm | yes, the problem is in that libvirtnvmevolumedriver | 14:57 |
| rosmaita | abishop: i think we will not get to your topic this week, but i will put you first for next week | 14:58 |
| zoharm | it hardcodes which connector to use it seems | 14:58 |
| abishop | story of my life :-/ | 14:58 |
| zoharm | rosmaita my other question is very quick | 14:58 |
| abishop | folks can read etherpad and check out review in meantime | 14:58 |
| zoharm | but anyway i will take this offline in favor of time | 14:58 |
| rosmaita | ok, thanks! | 14:58 |
| zoharm | other question is simple | 14:58 |
| rosmaita | #topic open discussion | 14:58 |
| *** openstack changes topic to "open discussion (Meeting topic: cinder)" | 14:58 | |
| *** lbragstad_ is now known as lbragstad | 14:59 | |
| rafaelweingartne | rosmaita: I just have a question regarding https://review.opendev.org/#/c/666886/. It is a patch that has been on the making for quite a while. | 14:59 |
| geguileo | zoharm: on second thought, I think it's probably best to use the existing connector | 14:59 |
| rafaelweingartne | Before solving the conflicts, we would like to understand what we are missing there. for instance, a formal specification | 14:59 |
| rosmaita | rafaelweingartne: i completely lost track of that one | 15:00 |
| rosmaita | let me look it over and leave some comments | 15:00 |
| rosmaita | check back in a few days? | 15:00 |
| rafaelweingartne | ok | 15:00 |
| rafaelweingartne | thanks | 15:00 |
| rosmaita | and we are out of time ... | 15:00 |
| rosmaita | thanks, everyone | 15:00 |
| rafaelweingartne | no problem, take your time | 15:00 |
| rosmaita | #endmeeting | 15:00 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 15:00 | |
| openstack | Meeting ended Wed Nov 18 15:00:42 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-11-18-14.00.html | 15:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-11-18-14.00.txt | 15:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-11-18-14.00.log.html | 15:00 |
| zoharm | can we continue a bit in #openstack-cinder? | 15:01 |
| rosmaita | sure | 15:01 |
| e0ne | #startmeeting horizon | 15:01 |
| openstack | Meeting started Wed Nov 18 15:01:37 2020 UTC and is due to finish in 60 minutes. The chair is e0ne. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| *** openstack changes topic to " (Meeting topic: horizon)" | 15:01 | |
| openstack | The meeting name has been set to 'horizon' | 15:01 |
| e0ne | hi everybody | 15:02 |
| rdopiera | hey | 15:02 |
| vishalmanchanda | hi all | 15:02 |
| *** abishop has left #openstack-meeting-alt | 15:02 | |
| amotoki | hi | 15:03 |
| e0ne | let's start | 15:05 |
| e0ne | #topic Notices | 15:05 |
| *** openstack changes topic to "Notices (Meeting topic: horizon)" | 15:05 | |
| e0ne | I created a poll to select a new time for our weekly meeting | 15:05 |
| e0ne | #link https://doodle.com/poll/a5eeh6h3v356w8ss | 15:05 |
| e0ne | it should contain 13.00 utc, 14.00 utc and 15.00 Tue-Thu | 15:06 |
| e0ne | I removed 14.00 utc on Wednesday because there is a conflict with cinder meeting | 15:07 |
| e0ne | I hope, we'll find a suitable time for everybody | 15:07 |
| e0ne | tmazur, rdopiera: thanks for your votes! | 15:08 |
| e0ne | I checked release goals for Wallaby | 15:08 |
| e0ne | #link https://governance.openstack.org/tc/goals/selected/wallaby/index.html | 15:09 |
| e0ne | there aren't anything we should do in horizon | 15:09 |
| *** rosmaita has quit IRC | 15:11 | |
| e0ne | we'll reach Wallaby-1 in two weeks | 15:11 |
| amotoki | migriation to YAML policy format may be related to us indirectly. | 15:11 |
| e0ne | #link https://releases.openstack.org/wallaby/schedule.html | 15:11 |
| vishalmanchanda | yes | 15:11 |
| e0ne | amotoki: I expect it will go smoothly for us | 15:13 |
| e0ne | json => yaml migration in python is pretty easy | 15:13 |
| amotoki | e0ne: yes. my policy-in-code support patch will cover it. | 15:13 |
| amotoki | the point is to still try to look up policy.json. | 15:13 |
| amotoki | it is suggested in the community goal (while cinder does not do so) | 15:14 |
| amotoki | perhaps it is better to have such fallback logic in horizon too. | 15:15 |
| e0ne | good point. it'll help us with upgrades | 15:16 |
| amotoki | anyway https://review.opendev.org/#/c/750134/ will cover the migration path. | 15:16 |
| *** rafaelweingartne has quit IRC | 15:18 | |
| e0ne | amotoki: great | 15:18 |
| e0ne | according to wallaby-1 milestone | 15:20 |
| *** lpetrut has quit IRC | 15:21 | |
| e0ne | it would be good to drop deprecated features until it and release the first version of horizon in Wallaby cycle | 15:21 |
| amotoki | +1 | 15:21 |
| amotoki | according to L.13 in https://etherpad.opendev.org/p/horizon-release-priorities, do we drop openstack_dashboard/wsgi/django.wsgi in this release? | 15:21 |
| e0ne | I'm ok with it | 15:22 |
| e0ne | we've got openstack_dashboard/wsgi.py for a while | 15:23 |
| amotoki | yes, it was added in rocky and enough time has passed. | 15:24 |
| e0ne | +1 | 15:24 |
| *** rosmaita has joined #openstack-meeting-alt | 15:24 | |
| e0ne | I +2'ed patches with OPENSTACK_NOVA_EXTENSIONS_BLACKLIST removal today | 15:24 |
| e0ne | the same for legacy neutron API usage | 15:25 |
| amotoki | thanks. it is a long way :) | 15:25 |
| e0ne | #link https://review.opendev.org/#/c/759652/ | 15:25 |
| e0ne | amotoki: I hope, contrail/tsungen fabric plugin will be fixed in Wallaby to use a new APIs | 15:26 |
| e0ne | we waited too long to not remove it from horizon | 15:26 |
| amotoki | totally agree | 15:27 |
| e0ne | I thinks, that's all announcements this week | 15:27 |
| e0ne | #topic Bug deputy report | 15:28 |
| *** openstack changes topic to "Bug deputy report (Meeting topic: horizon)" | 15:28 | |
| e0ne | amotoki: did you add that bugs into the etherpad? | 15:28 |
| amotoki | e0ne: yes. I added this section. | 15:29 |
| e0ne | thanks! | 15:29 |
| e0ne | #link https://bugs.launchpad.net/horizon/+bug/1902821 | 15:29 |
| openstack | Launchpad bug 1902821 in OpenStack Dashboard (Horizon) "When open a link for the detal of image as 'Open Link in New Tab', the new tab doesn't show the details of image." [Medium,Confirmed] | 15:29 |
| amotoki | it looks like AngularJS is not initialized properly but I didn't checked it in detail. | 15:30 |
| e0ne | it's angularjs-related issue :( | 15:30 |
| amotoki | it would be nice if someone has time. | 15:30 |
| e0ne | tmazur: are you around? | 15:30 |
| tmazur | e0ne: yes | 15:30 |
| e0ne | tmazur: you're our angularjs expert :) | 15:30 |
| tmazur | give me a sec :) | 15:31 |
| e0ne | I can take a look on this bug tomorrow morning, but I won't assign it to me before I'll be sure that I can fix it with a reasonable time | 15:31 |
| vishalmanchanda | I have assigned it to myself but if anyone else have quick solution for it. | 15:32 |
| tmazur | I will look into it later today | 15:32 |
| vishalmanchanda | feel free to push a patch. | 15:32 |
| e0ne | tmazur: thanks a lot! | 15:33 |
| tmazur | vishalmanchanda: ok! | 15:33 |
| e0ne | #link https://etherpad.opendev.org/p/horizon-release-priorities | 15:33 |
| e0ne | oops | 15:33 |
| e0ne | #link https://bugs.launchpad.net/horizon/+bug/1903320 | 15:33 |
| openstack | Launchpad bug 1903320 in OpenStack Dashboard (Horizon) "Port in OPENSTACK_KEYSTONE_URL" [Undecided,New] | 15:33 |
| e0ne | I would like to mark it as 'won't fix' | 15:34 |
| e0ne | we can't provide defaults that fits everybody | 15:34 |
| vishalmanchanda | It's a duplicate bug I think. | 15:34 |
| vishalmanchanda | I have seen a similar bug before. | 15:35 |
| amotoki | if it is valid for all CentOS/RHEL/Fedora packages, it is a valid bug | 15:35 |
| amotoki | otherwise, we cannot handle it :p | 15:35 |
| e0ne | rdopiera: do you have any comments from your side? | 15:37 |
| rdopiera | e0ne: no problem with won't fix for me | 15:38 |
| amotoki | another question is whether we would like to maintain the installation guide. we did not verify it for long. | 15:39 |
| rdopiera | e use the 5000 explicit port in all our configs, as far as I know, and if not, we can update it | 15:40 |
| e0ne | amotoki: it's something we definitely should do | 15:40 |
| vishalmanchanda | JFYI similar bug https://bugs.launchpad.net/horizon/+bug/1892279 | 15:42 |
| openstack | Launchpad bug 1892279 in OpenStack Dashboard (Horizon) "Install and configure for Red Hat Enterprise Linux and CentOS in horizon" [Undecided,New] | 15:42 |
| e0ne | vishalmanchanda: thank | 15:42 |
| vishalmanchanda | e0ne: agreed with ivan. | 15:42 |
| e0ne | I changed my mind | 15:45 |
| e0ne | I agree now that we need to keep this bug and verify setup guide | 15:45 |
| amotoki | looking at vishalmanchanda's comment in https://bugs.launchpad.net/horizon/+bug/1892279 you assumed devstack configuration but it is about the defaults of Fedora/CentOS/REHEL package | 15:45 |
| openstack | Launchpad bug 1892279 in OpenStack Dashboard (Horizon) "Install and configure for Red Hat Enterprise Linux and CentOS in horizon" [Undecided,New] | 15:45 |
| amotoki | so you cannot assume devstack configuration. it is a different thing. | 15:46 |
| vishalmanchanda | yeah my bad:( | 15:46 |
| e0ne | amotoki: +1 | 15:46 |
| amotoki | vishalmanchanda: no problem. thanks for checking it. | 15:46 |
| vishalmanchanda | I have never used rhel & centos.. | 15:47 |
| amotoki | it just marked the new one as duplicate of the previous bug vishalmanchanda pasted. | 15:47 |
| rdopiera | you are missing out on a lot of fun ;-) | 15:47 |
| vishalmanchanda | really. | 15:48 |
| rdopiera | no, it's actually all pretty much the same | 15:48 |
| amotoki | I logged into RHEL6/CentOS6 yesterday :p | 15:48 |
| rdopiera | did you have to blow the dust from it first? | 15:49 |
| tmazur | :D | 15:49 |
| amotoki | :) | 15:49 |
| vishalmanchanda | 🙂 | 15:49 |
| e0ne | rdopiera: :) | 15:49 |
| amotoki | VMs cannot have the dust ;p | 15:49 |
| e0ne | friendly reminder: we've to 10 minutes left | 15:50 |
| amotoki | https://bugs.launchpad.net/horizon/+bug/1900851 is interesting. we still have a code incompatible with py3. | 15:51 |
| openstack | Launchpad bug 1900851 in OpenStack Dashboard (Horizon) "Cannot Create Port with Fixed IP Address" [High,In progress] - Assigned to Akihiro Motoki (amotoki) | 15:51 |
| amotoki | it is just FYI | 15:52 |
| amotoki | i think we covered all bugs we need to discuss today. | 15:52 |
| e0ne | amotoki: thanks for bringing up these bugs! | 15:53 |
| vishalmanchanda | amotoki: nice. | 15:53 |
| e0ne | I would like to continue these effort next week | 15:54 |
| e0ne | is any volunteer to do the same for the next meeting? | 15:55 |
| vishalmanchanda | I can do that after next week | 15:55 |
| vishalmanchanda | as I was on holiday for some days in next week. | 15:56 |
| amotoki | I can cover one more week. I said I will push a note on a bug triaging during the PTG but I haven't pushed it yet. | 15:57 |
| amotoki | perhaps some doc would help us to do it, so let me push it first. | 15:57 |
| vishalmanchanda | amotoki: thanks. | 15:57 |
| e0ne | amotoki, vishalmanchandaL thanks! | 15:57 |
| e0ne | we're almost out of time! thanks a lot for a productive discussion | 15:58 |
| e0ne | see you next week | 15:59 |
| vishalmanchanda | bye | 15:59 |
| amotoki | thanks all! | 15:59 |
| tmazur | thanks everyone! | 15:59 |
| rdopiera | thanks | 16:00 |
| e0ne | #endmeeting | 16:01 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 16:01 | |
| openstack | Meeting ended Wed Nov 18 16:01:02 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-11-18-15.01.html | 16:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-11-18-15.01.txt | 16:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-11-18-15.01.log.html | 16:01 |
| *** linshiyx has quit IRC | 16:05 | |
| *** macz_ has joined #openstack-meeting-alt | 16:13 | |
| *** rosmaita has left #openstack-meeting-alt | 16:22 | |
| *** belmoreira has quit IRC | 16:27 | |
| *** masahito has quit IRC | 16:33 | |
| *** e0ne has quit IRC | 17:21 | |
| *** e0ne has joined #openstack-meeting-alt | 17:22 | |
| *** e0ne has quit IRC | 17:33 | |
| *** yamamoto has joined #openstack-meeting-alt | 17:33 | |
| *** yamamoto has quit IRC | 17:37 | |
| *** baojg has quit IRC | 17:44 | |
| *** baojg has joined #openstack-meeting-alt | 17:45 | |
| *** e0ne has joined #openstack-meeting-alt | 17:57 | |
| *** derekh has quit IRC | 18:14 | |
| *** sarmienm has joined #openstack-meeting-alt | 18:16 | |
| *** sarmienm has quit IRC | 18:19 | |
| *** ralonsoh has quit IRC | 18:31 | |
| *** e0ne has quit IRC | 18:36 | |
| *** rdopiera has quit IRC | 18:38 | |
| *** ChuckPiercey has quit IRC | 19:03 | |
| *** rosmaita has joined #openstack-meeting-alt | 19:42 | |
| *** rosmaita has left #openstack-meeting-alt | 19:45 | |
| *** baojg has quit IRC | 19:46 | |
| *** baojg has joined #openstack-meeting-alt | 19:47 | |
| *** vishalmanchanda has quit IRC | 19:49 | |
| *** gyee has joined #openstack-meeting-alt | 20:10 | |
| *** ricolin has quit IRC | 20:40 | |
| *** slaweq has quit IRC | 20:58 | |
| *** rcernin has joined #openstack-meeting-alt | 21:13 | |
| *** thgcorrea has quit IRC | 21:19 | |
| *** slaweq has joined #openstack-meeting-alt | 21:31 | |
| *** tmazur has quit IRC | 21:48 | |
| *** zoharm has quit IRC | 21:50 | |
| *** yamamoto has joined #openstack-meeting-alt | 22:01 | |
| *** baojg has quit IRC | 22:03 | |
| *** baojg has joined #openstack-meeting-alt | 22:03 | |
| *** yamamoto has quit IRC | 22:10 | |
| *** yamamoto has joined #openstack-meeting-alt | 22:11 | |
| *** slaweq has quit IRC | 22:16 | |
| *** hrybacki has quit IRC | 22:26 | |
| *** vkmc has quit IRC | 22:26 | |
| *** hrybacki has joined #openstack-meeting-alt | 22:26 | |
| *** vkmc has joined #openstack-meeting-alt | 22:26 | |
| *** johnsom has quit IRC | 22:27 | |
| *** johnsom has joined #openstack-meeting-alt | 22:29 | |
| *** baojg has quit IRC | 23:08 | |
| *** baojg has joined #openstack-meeting-alt | 23:09 | |
| *** number80 has quit IRC | 23:13 | |
| *** number80 has joined #openstack-meeting-alt | 23:26 | |
| *** yamamoto has quit IRC | 23:32 | |
| *** raildo_ has quit IRC | 23:42 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!