| *** ttsiouts has joined #openstack-meeting-alt | 00:52 | |
| *** gyee has quit IRC | 01:07 | |
| *** Liang__ has joined #openstack-meeting-alt | 01:13 | |
| *** ysandeep|away is now known as ysandeep|rover | 01:17 | |
| *** ttsiouts has quit IRC | 01:25 | |
| *** ricolin has joined #openstack-meeting-alt | 01:58 | |
| *** ysandeep|rover is now known as ysandeep|brb | 02:10 | |
| *** apetrich has quit IRC | 02:11 | |
| *** yamamoto has joined #openstack-meeting-alt | 02:14 | |
| *** ttsiouts has joined #openstack-meeting-alt | 03:22 | |
| *** ysandeep|brb is now known as ysandeep|rover | 03:42 | |
| *** yamamoto has quit IRC | 03:42 | |
| *** yamamoto has joined #openstack-meeting-alt | 03:44 | |
| *** yaawang__ has quit IRC | 03:53 | |
| *** vishalmanchanda has joined #openstack-meeting-alt | 03:53 | |
| *** ttsiouts has quit IRC | 03:56 | |
| *** andrebeltrami has quit IRC | 04:26 | |
| *** yamamoto has quit IRC | 04:42 | |
| *** yamamoto has joined #openstack-meeting-alt | 04:47 | |
| *** igordc has quit IRC | 05:32 | |
| *** ttsiouts has joined #openstack-meeting-alt | 05:53 | |
| *** yaawang has joined #openstack-meeting-alt | 05:58 | |
| *** masahito has joined #openstack-meeting-alt | 06:05 | |
| *** links has joined #openstack-meeting-alt | 06:20 | |
| *** ttsiouts has quit IRC | 06:27 | |
| *** ttsiouts has joined #openstack-meeting-alt | 06:45 | |
| *** yamamoto has quit IRC | 06:52 | |
| *** slaweq_ has joined #openstack-meeting-alt | 06:57 | |
| *** yaawang has quit IRC | 07:04 | |
| *** yaawang has joined #openstack-meeting-alt | 07:05 | |
| *** yamamoto has joined #openstack-meeting-alt | 07:05 | |
| *** lpetrut has joined #openstack-meeting-alt | 07:09 | |
| *** belmoreira has joined #openstack-meeting-alt | 07:16 | |
| *** ysandeep|rover is now known as ysandeep|lunch | 07:24 | |
| *** e0ne has joined #openstack-meeting-alt | 07:38 | |
| *** ccamacho has joined #openstack-meeting-alt | 07:39 | |
| *** chenke has joined #openstack-meeting-alt | 08:00 | |
| licanwei | #startmeeting Watcher | 08:00 |
|---|---|---|
| openstack | Meeting started Wed Apr 8 08:00:43 2020 UTC and is due to finish in 60 minutes. The chair is licanwei. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:00 |
| chenke | Hi all | 08:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:00 |
| *** openstack changes topic to " (Meeting topic: Watcher)" | 08:00 | |
| openstack | The meeting name has been set to 'watcher' | 08:00 |
| licanwei | hi~ | 08:00 |
| chenke | hi canwei | 08:01 |
| Dantalion | hello | 08:01 |
| chenke | hello dantalion | 08:01 |
| licanwei | Dantalion: hello | 08:01 |
| chenke | long time no watcher meeting. | 08:02 |
| chenke | seems | 08:02 |
| licanwei | This week is Ussuri-3 milestone | 08:02 |
| licanwei | and release for watcherclient | 08:02 |
| licanwei | https://review.opendev.org/#/c/718272/ | 08:03 |
| licanwei | This is the commit for release for watcherclient | 08:03 |
| Dantalion | I think watcherclient is ready for release if we merge cleanup for python 2.7 | 08:05 |
| licanwei | https://review.opendev.org/#/c/714357/ | 08:05 |
| licanwei | This is highlights for Watcher | 08:05 |
| Dantalion | that looks good to me | 08:06 |
| Dantalion | It seems I can't +1 in openstack/releases | 08:07 |
| licanwei | Dantalion: I can +1 | 08:08 |
| *** rdopiera has joined #openstack-meeting-alt | 08:09 | |
| licanwei | #topic Review | 08:09 |
| *** openstack changes topic to "Review (Meeting topic: Watcher)" | 08:09 | |
| licanwei | https://review.opendev.org/#/c/717445/ Cleanup py27 support for watcherclient | 08:10 |
| chenke | this patch lgtm. | 08:11 |
| licanwei | Dantalion: we need merge this before watcherclient release | 08:11 |
| Dantalion | I agree, this patch looks good to me. I can review today | 08:11 |
| licanwei | https://review.opendev.org/#/c/717541 | 08:12 |
| licanwei | https://review.opendev.org/#/c/718075 | 08:12 |
| licanwei | https://review.opendev.org/#/c/717540 | 08:12 |
| licanwei | These all are about cleanup py27 | 08:13 |
| licanwei | Dantalion: thanks | 08:13 |
| licanwei | https://review.opendev.org/#/c/717962/ Block Sphinx 3.0.0 needs review | 08:14 |
| Dantalion | Is this still needed now it is masked in global requirements? | 08:16 |
| licanwei | Dantalion: is it masked in global? | 08:18 |
| Dantalion | https://review.opendev.org/#/c/717949/ | 08:18 |
| Dantalion | Yes this patch has merged | 08:18 |
| licanwei | then maybe no need | 08:20 |
| licanwei | Dantalion: how about helm charts for Watcher? | 08:20 |
| Dantalion | I have to submit the patch, I intend to do this soon. I will write it down hope to do it this week. | 08:21 |
| licanwei | Dantalion: great | 08:22 |
| chenke | cool | 08:23 |
| licanwei | any questions need to discuss? | 08:24 |
| chenke | no. thanks. | 08:24 |
| Dantalion | No questions for me | 08:24 |
| licanwei | if no questions, i'll end the meeting | 08:26 |
| licanwei | thank you all~ | 08:26 |
| Dantalion | Yhank you and until next time | 08:26 |
| chenke | thank you all,bye | 08:26 |
| licanwei | Dantalion: see you next time | 08:26 |
| licanwei | #endmeeting | 08:26 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 08:26 | |
| openstack | Meeting ended Wed Apr 8 08:26:58 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 08:27 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/watcher/2020/watcher.2020-04-08-08.00.html | 08:27 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/watcher/2020/watcher.2020-04-08-08.00.txt | 08:27 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/watcher/2020/watcher.2020-04-08-08.00.log.html | 08:27 |
| *** rcernin has quit IRC | 08:27 | |
| *** ysandeep|lunch is now known as ysandeep|rover | 08:29 | |
| *** derekh has joined #openstack-meeting-alt | 08:33 | |
| *** slaweq_ is now known as slaweq | 08:36 | |
| *** yaawang has quit IRC | 08:48 | |
| *** yaawang has joined #openstack-meeting-alt | 08:50 | |
| *** baiziyu-inspur_ has joined #openstack-meeting-alt | 08:55 | |
| *** yaawang has quit IRC | 09:14 | |
| *** yaawang has joined #openstack-meeting-alt | 09:16 | |
| *** ricolin_ has quit IRC | 09:29 | |
| *** ricolin_ has joined #openstack-meeting-alt | 09:30 | |
| *** ricolin has quit IRC | 09:30 | |
| *** yaawang has quit IRC | 09:35 | |
| *** yaawang has joined #openstack-meeting-alt | 09:36 | |
| *** Liang__ has quit IRC | 09:38 | |
| *** chenke has quit IRC | 09:47 | |
| *** ricolin_ has quit IRC | 09:57 | |
| *** ricolin_ has joined #openstack-meeting-alt | 09:57 | |
| *** ricolin has joined #openstack-meeting-alt | 10:00 | |
| *** ricolin_ has quit IRC | 10:02 | |
| *** masahito has quit IRC | 10:04 | |
| *** links has quit IRC | 10:08 | |
| *** yamamoto has quit IRC | 10:13 | |
| *** slaweq has quit IRC | 10:23 | |
| *** slaweq has joined #openstack-meeting-alt | 10:25 | |
| *** ysandeep|rover is now known as ysandeep|afk | 10:30 | |
| *** links has joined #openstack-meeting-alt | 10:31 | |
| *** yamamoto has joined #openstack-meeting-alt | 10:33 | |
| *** yamamoto has quit IRC | 10:34 | |
| *** masahito has joined #openstack-meeting-alt | 10:37 | |
| *** ricolin has quit IRC | 10:39 | |
| *** masahito has quit IRC | 10:41 | |
| *** baiziyu-inspur_ has quit IRC | 10:43 | |
| *** yamamoto has joined #openstack-meeting-alt | 10:53 | |
| *** jtomasek has joined #openstack-meeting-alt | 10:55 | |
| *** e0ne has quit IRC | 10:58 | |
| *** e0ne has joined #openstack-meeting-alt | 10:58 | |
| *** apetrich has joined #openstack-meeting-alt | 11:16 | |
| *** ysandeep|afk is now known as ysandeep|rover | 11:31 | |
| *** links has quit IRC | 11:44 | |
| *** links has joined #openstack-meeting-alt | 11:45 | |
| *** raildo has joined #openstack-meeting-alt | 11:53 | |
| *** andrebeltrami has joined #openstack-meeting-alt | 12:40 | |
| *** ttsiouts has quit IRC | 12:53 | |
| *** ttsiouts has joined #openstack-meeting-alt | 12:57 | |
| *** ricolin has joined #openstack-meeting-alt | 12:57 | |
| *** gshippey has joined #openstack-meeting-alt | 13:02 | |
| *** ysandeep|rover is now known as ysandeep|away | 13:25 | |
| *** eharney has joined #openstack-meeting-alt | 13:28 | |
| *** enriquetaso has joined #openstack-meeting-alt | 13:39 | |
| *** yamamoto has quit IRC | 13:50 | |
| *** jraju__ has joined #openstack-meeting-alt | 13:50 | |
| *** Liang__ has joined #openstack-meeting-alt | 13:50 | |
| *** links has quit IRC | 13:52 | |
| *** rosmaita has joined #openstack-meeting-alt | 13:53 | |
| *** linshiyx has joined #openstack-meeting-alt | 13:56 | |
| *** liuyulong has joined #openstack-meeting-alt | 13:59 | |
| *** tosky has joined #openstack-meeting-alt | 13:59 | |
| rosmaita | #startmeeting cinder | 14:00 |
| rosmaita | #topic roll call | 14:00 |
| rosmaita | #link https://etherpad.openstack.org/p/cinder-ussuri-meetings | 14:00 |
| openstack | Meeting started Wed Apr 8 14:00:20 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 | |
| sfernand | hi | 14:00 |
| eharney | hi | 14:00 |
| lseki | o/ | 14:00 |
| enriquetaso | hi | 14:00 |
| Liang__ | hi | 14:00 |
| e0ne | hi | 14:00 |
| *** Liang__ is now known as LiangFang | 14:00 | |
| m5z | hi =] | 14:00 |
| smcginnis | o/ | 14:00 |
| LiangFang | o/ | 14:00 |
| rosmaita | wow, nice turnout | 14:01 |
| whoami-rajat | Hi | 14:01 |
| rosmaita | #topic announcements | 14:01 |
| *** openstack changes topic to "announcements (Meeting topic: cinder)" | 14:01 | |
| jungleboyj | o/ | 14:01 |
| tosky | o/ | 14:01 |
| rosmaita | Friday and/or Monday are holidays in a lot of countries | 14:01 |
| rosmaita | just be aware when requesting reviews, will be a bit slow this weekend | 14:01 |
| *** belmoreira has quit IRC | 14:02 | |
| rosmaita | TC election (look in your email for a ballot) ... voting ends 2020-04-14 23:45 UTC | 14:02 |
| walshh_ | hi | 14:02 |
| jungleboyj | Please vote! | 14:02 |
| rosmaita | i guess this is the stuff-happening-around-openstack edition of the announcements | 14:02 |
| rosmaita | ironic declaration of independence: http://lists.openstack.org/pipermail/openstack-discuss/2020-April/013757.html | 14:02 |
| smcginnis | :) | 14:02 |
| rosmaita | really long thread | 14:02 |
| *** throne82 has joined #openstack-meeting-alt | 14:02 | |
| rosmaita | just in case you are interested in that kind of thing | 14:03 |
| rosmaita | ok, on to cinder project news | 14:03 |
| rosmaita | os-brick 3.0.1 "ussuri-official" released last week | 14:03 |
| e0ne | Ironic Independence Day - sounds good | 14:03 |
| rosmaita | :) | 14:03 |
| rosmaita | python-cinderclient 7.0.0 almost ready to go | 14:03 |
| rosmaita | just need the release note approved: https://review.opendev.org/#/c/718234/ | 14:04 |
| * jungleboyj clicks | 14:04 | |
| rosmaita | reminder: the stable/ussuri branch for both of those are cut upon release | 14:04 |
| rosmaita | note to self: did we cut a branch for the brick-cinderclient-ext ? | 14:05 |
| rosmaita | so from now on, bugfixes in master and backport to stable/ussuri | 14:05 |
| rosmaita | tomorrow is milestone-3 release and FEATURE FREEZE for cinder | 14:05 |
| rosmaita | so let's take a quick look at the feature situation | 14:05 |
| rosmaita | #link https://blueprints.launchpad.net/cinder/ussuri | 14:06 |
| rosmaita | these are the ussuri features that i'm aware of | 14:06 |
| *** ttsiouts has quit IRC | 14:06 | |
| *** ttsiouts has joined #openstack-meeting-alt | 14:06 | |
| rosmaita | quick question for driver maintainers: are there driver features you have patches up for that aren't represented here? | 14:07 |
| rosmaita | because review priority for the rest of the week is features | 14:07 |
| lseki | I think sfernand has one | 14:07 |
| rosmaita | is that for netapp active-active? | 14:08 |
| lseki | yes! | 14:08 |
| sfernand | rosmaita: We added support for active/active to the SolidFire recently | 14:08 |
| smcginnis | I think some of the Dell teams have some patches up for new driver features. | 14:08 |
| sfernand | we are getting reviews this week | 14:08 |
| rosmaita | great | 14:08 |
| sfernand | Do you think this change would require a spec or something? https://review.opendev.org/#/c/712799/5 | 14:08 |
| rosmaita | sfernand: no, a bp in launchpad will be fine | 14:09 |
| rosmaita | i will create one and put it into ussuri after the meeting so we can track | 14:09 |
| sfernand | ok! Thanks | 14:09 |
| rosmaita | as far as the Dell/EMC drivers go, does someone have a list of feature patches? | 14:10 |
| jungleboyj | I was getting pinged by walshh_ about their features. We merged a couple. | 14:11 |
| rosmaita | ok, i will take a quick look at the open reviews after the meeting and make bps for any open ones | 14:11 |
| walshh_ | all our features are merged. Thanks to all who reviewed | 14:11 |
| rosmaita | well, that makes it easy! | 14:12 |
| jungleboyj | Yay! Go team! | 14:12 |
| rosmaita | ok, so reviewers: please concentrate on the feature patches for the rest of the week | 14:12 |
| rosmaita | which is basically today and tomorrow for all intents and purposed | 14:12 |
| rosmaita | *purposes | 14:12 |
| rajinir | We have some VXFlexOS patches and several other Dell EMC driver patches pending | 14:13 |
| rosmaita | rajinir: can you get a list together of the feature-oriented patches? | 14:13 |
| jungleboyj | ++ | 14:13 |
| rosmaita | because as you know, features are not backportable | 14:13 |
| rajinir | <rosmaita> will do | 14:13 |
| rosmaita | ok, great | 14:14 |
| *** ganso has joined #openstack-meeting-alt | 14:15 | |
| rosmaita | smcginnis: jungleboyj: do we need to do an M-3 release? i believe it's optional these days | 14:15 |
| *** kaisers_ has joined #openstack-meeting-alt | 14:15 | |
| jungleboyj | rosmaita: I believe it is optional. | 14:15 |
| smcginnis | rosmaita: Correct, it's optional. | 14:15 |
| smcginnis | We just need to be ready for the RC deadline. | 14:16 |
| rosmaita | ok, i don't see a need to do one | 14:16 |
| jungleboyj | ++ | 14:16 |
| rosmaita | so since smcginnis mentioned it, RC-1 is the week of 20 April | 14:16 |
| ganso | o/ | 14:16 |
| rosmaita | so two weeks away | 14:16 |
| rosmaita | i'll be working on the third-party CI compliance check early next week | 14:17 |
| *** yamamoto has joined #openstack-meeting-alt | 14:17 | |
| rosmaita | checking to see the they are responding, keeping logs, etc | 14:17 |
| rosmaita | if any problems come up, they'll need to be addressed before RC-1 | 14:17 |
| jungleboyj | rosmaita: ++ | 14:17 |
| rosmaita | and of course, any driver maintainers can check their own CI before i do it if you want to get a head start | 14:18 |
| jungleboyj | :-) | 14:18 |
| rosmaita | otherwise, drivers may be subject to being marked as not supported | 14:18 |
| rosmaita | ok, i think that's everything | 14:18 |
| whoami-rajat | rosmaita, is the feature freeze also the deadline to mark drivers as supported? | 14:19 |
| rosmaita | rajinir: don't forget to get me that list of patches | 14:19 |
| rosmaita | whoami-rajat: no, they have until RC-1 | 14:19 |
| rajinir | rosmaita> compiling now will share soon | 14:19 |
| rosmaita | but new drivers must be merged before FF | 14:19 |
| rosmaita | or request an FFE | 14:19 |
| rosmaita | rajinir: ty | 14:20 |
| whoami-rajat | rosmaita, okay. i also see macroSAN encapsulated some features with marking it as supported https://review.opendev.org/#/c/711388/ | 14:20 |
| whoami-rajat | they've a functional CI with one issue that we discussed the other day on cinder channel | 14:21 |
| rosmaita | ok, thanks for bringing that up | 14:21 |
| whoami-rajat | i couldn't contact them after that but it seems good to get this in as well? | 14:21 |
| jungleboyj | Ugh. Ok. Guess we need to get some eyes on that. | 14:22 |
| rosmaita | yes, we should prioritize this one | 14:22 |
| rosmaita | i'll add a bp so we can track it | 14:22 |
| whoami-rajat | great, thanks! | 14:22 |
| rosmaita | let's not let the supported=true hold this up | 14:22 |
| rosmaita | we can un-support it if we have to at RC-1 time | 14:23 |
| rosmaita | anything else? | 14:23 |
| whoami-rajat | not from me | 14:23 |
| rosmaita | #topic Bug: Cinder Fail to extend attached volume using generic NFS driver | 14:24 |
| *** openstack changes topic to "Bug: Cinder Fail to extend attached volume using generic NFS driver (Meeting topic: cinder)" | 14:24 | |
| rosmaita | throne82: that's you | 14:24 |
| throne82 | Helo | 14:24 |
| throne82 | While I was enabling the extend attach tests for NetApp drivers we had some errors on qemu-img | 14:25 |
| rosmaita | #link https://bugs.launchpad.net/cinder/+bug/1870367 | 14:25 |
| openstack | Launchpad bug 1870367 in Cinder "Fail to extend attached volume using generic NFS driver" [Undecided,New] | 14:26 |
| throne82 | I tested with the generic NFS driver and it fails the same (since the implementation is the same) | 14:26 |
| *** enriquetaso has left #openstack-meeting-alt | 14:27 | |
| throne82 | so the extend fails due to the qemu-img cant lock the volume to write anything | 14:27 |
| *** enriquetaso has joined #openstack-meeting-alt | 14:27 | |
| rosmaita | yes, thanks for the detailed bug report | 14:27 |
| eharney | the failure here makes sense, i think this will need some work in the generic NFS driver | 14:27 |
| lseki | netapp ontap nfs drivers relies on image_utils, which is used by generic nfs driver as well | 14:28 |
| eharney | qemu-img can't get a lock to resize the file because it's in use by nova's qemu | 14:28 |
| lseki | I wonder if generic nfs had ever supported online extend in the past | 14:28 |
| lseki | I think it did, because ontap nfs driver also did some time ago | 14:29 |
| lseki | not sure what changed, the image_utils resize_image code didn't change for years | 14:29 |
| eharney | older nfs configs may have let it succeed if they didn't have the same nfs lock support configured | 14:30 |
| eharney | whether that is/was safe or not is another question | 14:30 |
| lseki | seems that currently nfs driver is skipping extend attached volume tests | 14:32 |
| lseki | #link https://b4a949e5f6fdf3010036-e4f20cff14b59b3a1c5b0d28b2b173f9.ssl.cf2.rackcdn.com/696626/2/check/devstack-plugin-nfs-tempest-full/098763c/testr_results.html | 14:32 |
| rosmaita | so i guess the question is, can this be made safe or do we not allow extending an attached volume for nfs? | 14:32 |
| eharney | i think we should block it in the driver for now, there should be some ways to do it safely with enough work | 14:33 |
| rosmaita | that makes sense to me | 14:33 |
| eharney | the problem is that extending attached volumes was added as a general cinder function w/o strict checking into what happens in each driver etc | 14:33 |
| rosmaita | that is definitely a problem | 14:33 |
| rosmaita | so we may see this again | 14:34 |
| eharney | it was quite a while ago, so not a huge worry, but maybe | 14:34 |
| rosmaita | well, any drivers currently skipping the tests should definitely take a look | 14:34 |
| rosmaita | i mean, their maintainers should take a look | 14:35 |
| smcginnis | I could have sworn we had required drivers to report if they supported extending attached volumes. | 14:35 |
| whoami-rajat | the support matrix says we support it for nfs | 14:35 |
| rosmaita | guess that will need an update | 14:36 |
| kaisers_ | There's a tempest flag for that test: @testtools.skipUnless(CONF.volume_feature_enabled.extend_attached_volume, | 14:36 |
| kaisers_ | "Attached volume extend is disabled.") | 14:36 |
| eharney | smcginnis: i was thinking the same thing, but i haven't found code for it | 14:36 |
| rosmaita | throne82: has your question been answered? | 14:36 |
| kaisers_ | (source: https://github.com/openstack/tempest/blob/348fa311fe031ff7d04f41aa9e6ac65f6f6391fe/tempest/api/volume/test_volumes_extend.py) | 14:36 |
| whoami-rajat | https://review.opendev.org/#/c/454287/ this is a generic implementation of the feature | 14:37 |
| LiangFang | I remember lots of drivers in os-brick not support extend volume feature | 14:38 |
| eharney | there's a difference between the os-brick extend_volume support and the cinder driver extend_volume | 14:39 |
| LiangFang | ok | 14:39 |
| eharney | i think the os-brick part is for updating Nova after the actual extend was done. the latter being what fails here | 14:39 |
| throne82 | rosmaita: can we have a discussion later to further discuss if there's a safe way to do this? | 14:39 |
| rosmaita | sure | 14:40 |
| rosmaita | it's a bug, so we have a bit of time | 14:40 |
| rosmaita | ok, throne82, thank you for bringing this up and working on it | 14:40 |
| throne82 | thanks! | 14:41 |
| rosmaita | #topic Review needed: NFS encrypted volume support | 14:41 |
| *** openstack changes topic to "Review needed: NFS encrypted volume support (Meeting topic: cinder)" | 14:41 | |
| rosmaita | request from enriquetaso | 14:41 |
| enriquetaso | Hi | 14:41 |
| rosmaita | hello! | 14:41 |
| enriquetaso | Hello cinder team, so this is a really old spec that we have "Support Cinder volume encryption with the NFS driver." | 14:41 |
| enriquetaso | #link https://blueprints.launchpad.net/cinder/+spec/nfs-volume-encryption | 14:41 |
| enriquetaso | I've been working on this : https://review.opendev.org/#/c/597148/ and I need some reviews :D | 14:41 |
| enriquetaso | I know it's a long patch but It's ready for opinions. | 14:41 |
| rosmaita | i've been seeing enriquetaso's reviews on several other patches lately | 14:42 |
| rosmaita | so it would be good to do her a solid and review her patch | 14:42 |
| enriquetaso | :D thanks rosmaita | 14:42 |
| whoami-rajat | rosmaita++ | 14:42 |
| rosmaita | this is in the list of cycle features, btw, so it is important to review it soon | 14:43 |
| rosmaita | anything else? | 14:43 |
| enriquetaso | nop | 14:43 |
| eharney | i think this one is pretty close (kind of a biased opinion), i mostly want to ensure it's being tested thoroughly | 14:43 |
| enriquetaso | eharney++ | 14:44 |
| rosmaita | ok, great | 14:44 |
| rosmaita | #topic cinder-tempest-plugin | 14:44 |
| *** openstack changes topic to "cinder-tempest-plugin (Meeting topic: cinder)" | 14:44 | |
| rosmaita | tosky: that's you | 14:44 |
| tosky | so | 14:44 |
| tosky | as you can see in the list, there are a few cinder-tempest-plugin reviews which are ready or almost-ready but in a reviewable state, and that would be nice to have | 14:45 |
| jungleboyj | I was going to say that we needed to make sure eharney looked at her patch, but then I see the owner. ;-) | 14:45 |
| tosky | I can quickly say a few words about each of them | 14:45 |
| *** igordc has joined #openstack-meeting-alt | 14:45 | |
| tosky | - Enable c-bak and switch to the storage blacklist: https://review.opendev.org/#/c/717379/ -> this is mostly complete IMHO, and the subject says it all | 14:46 |
| tosky | c-bak testing also with the cinder backend for cinder-tempest-plugin changes; it unblocks a few other tests | 14:46 |
| *** lpetrut has quit IRC | 14:46 | |
| tosky | same for Enable volume_revert tests https://review.opendev.org/#/c/717379/ , which unblocks revert tests on LVM | 14:47 |
| tosky | - Add Snapshot data integrity test https://review.opendev.org/#/c/702495/ - this is IMHO logically fine, I only suggested to move some common code in a separate function | 14:47 |
| tosky | - Add LVM+tgt tempest job https://review.opendev.org/537658 - originally proposed long time ago by Eric, it has been refactored; it looks fine now, even though it may conflict with https://review.opendev.org/#/c/717379/, we may need some reordering | 14:48 |
| *** Xuchu has joined #openstack-meeting-alt | 14:48 | |
| *** Xuchu has left #openstack-meeting-alt | 14:49 | |
| tosky | and finally, two other real tests about backups which are waiting on c-bak support: | 14:49 |
| tosky | - Extending testing scope of Incremental Backup https://review.opendev.org/#/c/652817/ | 14:49 |
| tosky | - Add test for check dependencies between incr backups https://review.opendev.org/#/c/652771/ | 14:49 |
| rosmaita | ok, thanks ... looks like you have them listed in order of priority | 14:49 |
| tosky | their results should be rechecked now that c-bak is active | 14:49 |
| eharney | awesome to see all this work being done | 14:49 |
| enriquetaso | tosky++ | 14:49 |
| tosky | I'm only nagging people, enriquetaso and whoami-rajat did most of the work here | 14:50 |
| rosmaita | tosky: is it ok to hold off on the multipath scenario questions until next week? | 14:50 |
| tosky | finally, I have a few questions about this patch, which was abandoned for a while, but it is interesting: | 14:50 |
| tosky | oh, sure | 14:50 |
| tosky | that was the one | 14:50 |
| tosky | but you can read the question there and answer anytime - I won't stop you! | 14:51 |
| rosmaita | thanks, just want to make sure we get to the other items | 14:51 |
| tosky | sure, EOF for now | 14:51 |
| rosmaita | #topic Allow removing NFS snapshots in error status is stuck. | 14:51 |
| *** openstack changes topic to "Allow removing NFS snapshots in error status is stuck. (Meeting topic: cinder)" | 14:51 | |
| whoami-rajat | thanks tosky | 14:51 |
| enriquetaso | Hello again ... So, I have 3 diff opinions in the patch: | 14:51 |
| enriquetaso | #link https://review.opendev.org/#/c/679138/ | 14:51 |
| enriquetaso | I'm not sure what to do, maybe we can discuss it here and get a conclusion. i'll do my best to summarize them: | 14:51 |
| enriquetaso | 1. Make the patch a partial bug fix because the root issue here is that the snapshot entries were created in the first place. But If snapshots were created, then the decision was made to disable snapshots, we should still allow those available snapshots to be deleted. Otherwise there is no way to get rid of them without re configuring the system. | 14:52 |
| enriquetaso | ^ smcginnis | 14:52 |
| enriquetaso | 2. Don't make it a partial bug and still call _check_snapshot_support() in delete, but catch the expected exception. | 14:52 |
| enriquetaso | Then log that exception, so admins have some trace of it, and then allow the delete to go through. | 14:52 |
| enriquetaso | ^ hemna_ | 14:52 |
| enriquetaso | 3. Replace the current check because the introduction of this check dates back to Ubuntu 14 issue with libvirt < 1.2.7 so not sure how useful is the check in current deployments (maybe used for other purposes). Since the volume is only created in db, my recommendation would still be to just delete it from db during create. | 14:53 |
| enriquetaso | It might be easily done by defining a new exception, raise it from from nfs driver and catch it in manager. This way, the snapshot is never created (as intended), the exception is raised stating the operation is not supported, there is no extra work to remove snapshot | 14:53 |
| enriquetaso | ^ whoami-rajat | 14:53 |
| enriquetaso | sorry for bothering you, but I'm confused | 14:53 |
| hemna_ | mep | 14:54 |
| eharney | both #1 and #3 point to an idea that it would be useful for a failed snapshot creation to be recorded as "nothing actually happened on the backend that needs to be deleted later" | 14:55 |
| eharney | but i think that's a larger scope thing to take on than the bug fix at hand here | 14:55 |
| whoami-rajat | my idea is just to remove the conflict of whether we can delete a snapshot if it's disabled, also manual cleanup would be required but it's ok if it's implemented any other way as far as it solves the issue | 14:57 |
| whoami-rajat | s/it's disabled/if support is disabled | 14:58 |
| eharney | there doesn't have to be a conflict there, the driver could do a check as to whether it needs to delete anything when delete_snapshot is called and if not, return success before it queries whether snapshots are enabled | 14:58 |
| rosmaita | we're down to 1 minute | 14:59 |
| rajinir | https://www.irccloud.com/pastebin/yPQfJRXw/ | 14:59 |
| rosmaita | ganso: will need to postpone until next meeting | 14:59 |
| rosmaita | rajinir: ty | 14:59 |
| ganso | rosmaita: it may be quick, we could chat in the channel | 14:59 |
| rajinir | rosmaita: https://etherpad.openstack.org/p/Dell_EMC_Driver_Feature_Patches_Ussuri | 14:59 |
| rosmaita | ok | 14:59 |
| rosmaita | thanks everyone, we have another meeting happening here in a minute | 15:00 |
| rosmaita | #endmeeting | 15:00 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 15:00 | |
| openstack | Meeting ended Wed Apr 8 15:00:18 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-04-08-14.00.html | 15:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-04-08-14.00.txt | 15:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-04-08-14.00.log.html | 15:00 |
| amotoki | #startmeeting horizon | 15:01 |
| *** LiangFang has quit IRC | 15:01 | |
| openstack | Meeting started Wed Apr 8 15:01:55 2020 UTC and is due to finish in 60 minutes. The chair is amotoki. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| amotoki | hi | 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:02 |
| *** tosky has left #openstack-meeting-alt | 15:02 | |
| vishalmanchanda | hi | 15:02 |
| e0ne | hi | 15:02 |
| *** ganso has left #openstack-meeting-alt | 15:02 | |
| rosmaita | e0ne: congratulations! | 15:03 |
| amotoki | okay, let's start | 15:03 |
| e0ne | rosmaita: thanks :) | 15:03 |
| amotoki | #topic notices/announcements | 15:04 |
| *** openstack changes topic to "notices/announcements (Meeting topic: horizon)" | 15:04 | |
| amotoki | this week is milestone-3 i.e. feature freeze | 15:04 |
| amotoki | I will prepare a release patch tomorrow or early Friday in local time | 15:04 |
| e0ne | amotoki: do we have anything we should merge before FF? | 15:05 |
| amotoki | I haven't checked the list yet. I would like to look thru pending reviews later in this meeting. | 15:05 |
| e0ne | ok | 15:06 |
| *** priteau has joined #openstack-meeting-alt | 15:06 | |
| *** yamamoto has quit IRC | 15:06 | |
| amotoki | another thing is TC voting period started. | 15:06 |
| amotoki | you can ask questions to TC candidates in the mailing list as you already see. | 15:07 |
| amotoki | one more | 15:08 |
| amotoki | virtual PTG planning is ongoing https://etherpad.openstack.org/p/Virtual_PTG_Planning | 15:08 |
| amotoki | you can find discussions in past meetings | 15:08 |
| amotoki | it is not specific to individual projects but we can give input to planning if you have any | 15:09 |
| amotoki | that's all from me as announcements | 15:10 |
| amotoki | anything other? | 15:10 |
| e0ne | nothing from me | 15:11 |
| vishalmanchanda | none | 15:11 |
| amotoki | moving on | 15:11 |
| amotoki | #topic reviews for FF | 15:11 |
| *** openstack changes topic to "reviews for FF (Meeting topic: horizon)" | 15:11 | |
| amotoki | I would like to discuss this topic first | 15:11 |
| amotoki | I failed to find enough time to review vishalmanchanda's error msg stuff https://review.opendev.org/#/c/708069/ | 15:12 |
| *** gyee has joined #openstack-meeting-alt | 15:13 | |
| amotoki | how do you think the status? | 15:13 |
| vishalmanchanda | hmm np. | 15:13 |
| vishalmanchanda | I will continue it in next cycle. | 15:13 |
| *** yamamoto has joined #openstack-meeting-alt | 15:15 | |
| *** yamamoto has quit IRC | 15:15 | |
| amotoki | I will look and test it tomorrow. we can measure its readiness and risk. | 15:15 |
| vishalmanchanda | I am not able to explore more from past two week due to poor internet connectivity in my area. | 15:16 |
| amotoki | another one is https://review.opendev.org/709025 | 15:16 |
| amotoki | already +2 from e0ne | 15:16 |
| amotoki | I looked thru it today | 15:16 |
| vishalmanchanda | i have seen this patch but i didn't have env. to test it. | 15:17 |
| amotoki | it is generally good, but it passes if-condition for display_name from the server side to client side | 15:17 |
| vishalmanchanda | will take a look after the meeting. | 15:17 |
| amotoki | that's the only point I am not a fan of, but it looks like a refactoring issue | 15:17 |
| amotoki | so if it works well I would like to have it in ussuri | 15:18 |
| amotoki | another one is https://review.opendev.org/#/c/630663/ | 15:18 |
| amotoki | e0ne is a co-author of this | 15:19 |
| e0ne | amotoki: oh.. I miised your last comment :( | 15:19 |
| amotoki | e0ne: my question does not block it. it is just a question. | 15:21 |
| amotoki | I totally forgot a comment on patch set 3 when I posted it, but hopefully it still works | 15:22 |
| amotoki | other two are maintenance stuffs https://review.opendev.org/714933 and https://review.opendev.org/#/c/715925/ | 15:23 |
| amotoki | requirements change might be better to be a part of ussuri-3 | 15:23 |
| amotoki | a release note is optional but nice to have it | 15:24 |
| e0ne | amotoki: answered to your comments in gerrit | 15:24 |
| amotoki | e0ne: thanks | 15:24 |
| amotoki | ah.... do you think we need a release note on pyScss 1.3.6 or later? | 15:25 |
| e0ne | according #714933 we can add a release note libpcre3-dev package but it's not mandatory anymore | 15:25 |
| amotoki | yeah, I think we need another release note on a known issue on the material theme | 15:26 |
| e0ne | amotoki: personally, I'm ok to wait 2 seconds more during deployment time | 15:26 |
| amotoki | e0ne: +1 | 15:26 |
| e0ne | amotoki: I'd got a bug assigned to me. I didn't have a time to take a look on it | 15:26 |
| e0ne | https://bugs.launchpad.net/horizon/+bug/1771559 | 15:26 |
| openstack | Launchpad bug 1771559 in OpenStack Dashboard (Horizon) "error while loading icon with pyscss 1.3.5 or later" [High,Confirmed] - Assigned to Ivan Kolodyazhny (e0ne) | 15:26 |
| e0ne | IMO, we can add a release not or a fix before RC1 | 15:27 |
| amotoki | exactly | 15:27 |
| e0ne | #715925 - I've already +2ed on it | 15:28 |
| amotoki | I found similar scss code in the material them, so perhaps we can apply similar workaround | 15:28 |
| amotoki | a final one in my list is https://review.opendev.org/#/c/711195/ (transition to pytest) | 15:29 |
| amotoki | it requires dependency change, so I raise it here | 15:29 |
| e0ne | I'm a co-author of this patch too, so I'm not feeling good to +2 on it | 15:30 |
| amotoki | looking at the code, it looks good | 15:30 |
| amotoki | I would like to run it locally before +2 | 15:30 |
| vishalmanchanda | will review it asap. | 15:30 |
| amotoki | IMHO it is almost ready to go | 15:30 |
| e0ne | thanks | 15:31 |
| amotoki | I already reviewed it in detail :) | 15:31 |
| vishalmanchanda | cool.. | 15:31 |
| amotoki | any other reviews good for ussuri feature freeze? | 15:32 |
| vishalmanchanda | https://review.opendev.org/#/c/640981/ | 15:33 |
| vishalmanchanda | any thoughts on this patch? | 15:33 |
| vishalmanchanda | or we can see it in next cycle. | 15:33 |
| amotoki | IMHO we need to defer it to Victoria | 15:33 |
| vishalmanchanda | sure. | 15:34 |
| amotoki | it is because it needs changes in language code in zanata and the infra scripts | 15:34 |
| e0ne | +1 | 15:34 |
| amotoki | even if we merge it, the infra script will override it and the patch will be reverted immediately | 15:34 |
| vishalmanchanda | ohk. | 15:35 |
| amotoki | okay, let's review them and hopefully we can have them in ussuri. | 15:36 |
| vishalmanchanda | +1. | 15:37 |
| amotoki | moving on | 15:37 |
| amotoki | #topic General Priorities | 15:37 |
| *** openstack changes topic to "General Priorities (Meeting topic: horizon)" | 15:37 | |
| amotoki | I would like to share the status on "Catch up the new policy mechanism" | 15:37 |
| amotoki | passing system_scope to the policy engine itself is simple, but it turns out it affects the horizon UI design | 15:38 |
| amotoki | as of now, in the identity panel, the policy engine automatically switches the token from a project-scoped to a domain-scoped | 15:39 |
| amotoki | so we can see the panels under the identity dashboard | 15:39 |
| amotoki | to support system-scope, we need a system-scoped token | 15:40 |
| amotoki | but we have a no way to switch to system-scoped token | 15:40 |
| amotoki | we always depends on a project token (and domain_id in a user info) | 15:40 |
| amotoki | we may need another scope "switch" mechanism | 15:41 |
| e0ne | seems to be a good discussion on the PTG | 15:41 |
| amotoki | I considered an option to automatically try a system-scoped token, but it looks too aggresive | 15:42 |
| amotoki | e0ne: yeah | 15:42 |
| amotoki | I will summarize what I investigated so far. | 15:42 |
| amotoki | that's the update on the policy stuff. | 15:43 |
| amotoki | next topic is about xstatic updates | 15:44 |
| amotoki | I recently proposed xstatic maintenance updates under horizon governance. | 15:44 |
| amotoki | two patches are pending https://review.opendev.org/#/q/project:%255E.*/xstatic-.*+status:open+owner:%22Akihiro+Motoki+%253Camotoki%2540gmail.com%253E%22 | 15:45 |
| amotoki | vishalmanchanda raised a question on whether we prepare a new version for the above two patches | 15:46 |
| vishalmanchanda | yeah. | 15:47 |
| rdopiera | I don't think a README change warrants a version bump | 15:47 |
| e0ne | rdopiera: +1 | 15:47 |
| amotoki | I personally am okay to release new versions to sync PyPI and openstack/releases instead of reverting existing updates in the repo | 15:47 |
| rdopiera | (also, I already made them start merging) | 15:47 |
| vishalmanchanda | cool:) | 15:48 |
| amotoki | thanks rdopiera | 15:48 |
| amotoki | they are now in the gate queue | 15:48 |
| amotoki | I will prepare a release patch after Ussuri-3 | 15:49 |
| amotoki | so that we can test them during RC period | 15:49 |
| e0ne | +1 | 15:50 |
| amotoki | rdopiera: what about xstatic-angular-fileupload? | 15:50 |
| rdopiera | I got distracted, I will release it next week | 15:51 |
| amotoki | np | 15:51 |
| e0ne | amotoki: there is one more patch from me: https://review.opendev.org/#/c/718379/ | 15:52 |
| e0ne | amotoki: I'm not sure how to treat it: a bug or a feature... | 15:52 |
| amotoki | e0ne: I think it is a bug | 15:52 |
| e0ne | ok | 15:52 |
| amotoki | we support custom templating for openrc but we don't provide enough info :( | 15:53 |
| e0ne | so I'm ok to get it landed after FF | 15:53 |
| amotoki | e0ne: bug report would be nice | 15:53 |
| *** rosmaita has left #openstack-meeting-alt | 15:53 | |
| e0ne | amotoki: will do it | 15:53 |
| amotoki | e0ne: thanks | 15:53 |
| amotoki | rdopiera and all: do we want to have xstatic-angular-fileupload under the horizon governance? | 15:53 |
| rdopiera | I think so | 15:54 |
| amotoki | we already have the repository under openstack/ namespace, so the process would be much simpler | 15:54 |
| rdopiera | what is involved? | 15:54 |
| amotoki | in addition, the PyPI repo already allows openstackci to publish deliverables | 15:54 |
| amotoki | https://pypi.org/project/XStatic-Angular-FileUpload/ | 15:54 |
| *** yamamoto has joined #openstack-meeting-alt | 15:55 | |
| amotoki | I think what we need are (1) to add the repo under horizon in the governance repo and (2) configure publish-to-pypi in project-config | 15:55 |
| amotoki | then it is ready to release a new version | 15:56 |
| amotoki | this is another way to publish a new version | 15:56 |
| amotoki | rdopiera: one question: how urgent do we need a new version of xstaic-angular-fileupload? | 15:56 |
| rdopiera | amotoki: not at all | 15:56 |
| rdopiera | I don't know any CVEs or bugfixes we need in it, I just upgraded it because it was alphebetically at the beginning of the list | 15:57 |
| amotoki | :) | 15:57 |
| rdopiera | however, it would be nice to have it cleaned up eventually | 15:57 |
| rdopiera | but it doesn't have to be ussuri | 15:57 |
| amotoki | so it looks better to add xstaic-angular-fileupload to OpenStack governance so that everyone can cut a release | 15:58 |
| amotoki | does it make sense? | 15:58 |
| e0ne | +1 | 15:58 |
| rdopiera | yes | 15:59 |
| amotoki | thanks, I will move the transition forward along with other xstatic releases. | 15:59 |
| amotoki | oh, we are out of time | 16:00 |
| amotoki | if anything other, let's continue in #-horizon | 16:00 |
| amotoki | thanks for joining | 16:00 |
| amotoki | #endmeeting | 16:00 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 16:00 | |
| openstack | Meeting ended Wed Apr 8 16:00:22 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-04-08-15.01.html | 16:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-04-08-15.01.txt | 16:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-04-08-15.01.log.html | 16:00 |
| amotoki | o/ | 16:00 |
| *** ttsiouts has quit IRC | 16:00 | |
| *** jcoufal has joined #openstack-meeting-alt | 16:04 | |
| *** yamamoto has quit IRC | 16:09 | |
| *** derekh has quit IRC | 16:59 | |
| *** priteau has quit IRC | 17:15 | |
| *** jcoufal has quit IRC | 17:34 | |
| *** linshiyx has quit IRC | 17:35 | |
| *** rdopiera has quit IRC | 17:40 | |
| *** diablo_rojo has quit IRC | 18:02 | |
| *** diablo_rojo has joined #openstack-meeting-alt | 18:06 | |
| *** e0ne has quit IRC | 18:21 | |
| *** kaisers_ has quit IRC | 18:31 | |
| *** diablo_rojo has quit IRC | 18:53 | |
| *** slaweq_ has joined #openstack-meeting-alt | 18:59 | |
| *** slaweq has quit IRC | 19:00 | |
| *** throne82 has quit IRC | 19:16 | |
| *** vishalmanchanda has quit IRC | 19:32 | |
| *** yamamoto has joined #openstack-meeting-alt | 20:07 | |
| *** gshippey has quit IRC | 20:07 | |
| *** ccamacho has quit IRC | 20:08 | |
| *** yamamoto has quit IRC | 20:13 | |
| *** igordc has quit IRC | 20:17 | |
| *** igordc has joined #openstack-meeting-alt | 20:18 | |
| *** jraju__ has quit IRC | 21:07 | |
| *** jtomasek has quit IRC | 21:25 | |
| *** slaweq_ has quit IRC | 21:38 | |
| *** enriquetaso has quit IRC | 21:48 | |
| *** slaweq_ has joined #openstack-meeting-alt | 22:13 | |
| *** rcernin has joined #openstack-meeting-alt | 22:32 | |
| *** gyee has quit IRC | 22:49 | |
| *** tmazur has joined #openstack-meeting-alt | 23:34 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!