opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/818037 | 02:21 |
---|---|---|
opendevreview | katari manoj kumar proposed openstack/cinder master: Doc: Typos in Spectrum Virtulize family user guide https://review.opendev.org/c/openstack/cinder/+/819082 | 09:25 |
yuhao_zhu | Hi core reviewers, if you have time, please look at https://review.opendev.org/c/openstack/cinder/+/768592 | 09:27 |
yuhao_zhu | thanks | 09:29 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 09:52 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 10:21 |
ignaziocassano | Hello, on qeens version ov cinder using netapp driver I want create 3 svm : svmgold1, svmcold2 and svmsilver. 11:25 svmgold1 and svmgold2 must be balanced basing on aggregate capacity. 11:28 So I created svmgold1 and svmgold2 with the same backend name (svmgold) under cinder.conf, and svmsilver with another backend name. I noted that the filter CapacityFilter is applied not also between svmgold1 an svmgold2 , infact volumes created with | 10:31 |
ignaziocassano | ted with svmgold volume type are created also under svmsilver. Is the filter applied on all bacakends ? It is strange | 10:32 |
ignaziocassano | any help please ? | 10:40 |
opendevreview | Andrei Yachmenev proposed openstack/cinder master: Nfs: manageable volumes https://review.opendev.org/c/openstack/cinder/+/819089 | 10:41 |
*** dviroel|afk is now known as dviroel | 11:00 | |
ignaziocassano | dviroel: can you help me ? | 11:03 |
dviroel | ignaziocassano: hey man, I'm not a cinder's scheduler specialist but my approach would be to enable debug logging and restart only scheduler service. You will be able to see which filters and weighers are being applied and which backends are being discarded | 12:01 |
ignaziocassano | dvirole: I'll do it | 12:01 |
ignaziocassano | thanks | 12:01 |
opendevreview | Felipe Rodrigues proposed openstack/cinder master: Generic NFS: skip qemu-img resize if volume is attached https://review.opendev.org/c/openstack/cinder/+/739079 | 12:55 |
opendevreview | Felipe Rodrigues proposed openstack/cinder master: Generic NFS: skip qemu-img resize if volume is attached https://review.opendev.org/c/openstack/cinder/+/739079 | 13:05 |
opendevreview | Felipe Rodrigues proposed openstack/cinder master: Generic NFS: skip qemu-img resize if volume is attached https://review.opendev.org/c/openstack/cinder/+/739079 | 13:38 |
rosmaita | Early reminder: today's cinder meeting is on video + #openstack-meeting-alt at 1400 UTC | 13:55 |
rosmaita | connection info: https://bluejeans.com/3228528973 | 13:55 |
rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira ^^ | 13:55 |
*** whoami-rajat__ is now known as whoami-rajat | 13:57 | |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 14:35 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 14:37 |
enriquetaso | #topic Honor multipath config everywhere | 14:56 |
enriquetaso | We have a fix thanks to gorka \o/ | 14:56 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/818994 | 14:56 |
* enriquetaso forgot to start the meeting facepalms | 14:57 | |
enriquetaso | #startmeeting cinder_bs | 14:57 |
opendevmeet | Meeting started Wed Nov 24 14:57:28 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:57 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:57 |
opendevmeet | The meeting name has been set to 'cinder_bs' | 14:57 |
enriquetaso | Welcome back | 14:57 |
enriquetaso | Only one topic today that includes multiple bug reports. | 14:57 |
enriquetaso | #topic Honor multipath config everywhere | 14:57 |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1951982 | 14:57 |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1951981 | 14:57 |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1951977 | 14:57 |
enriquetaso | We have a fix thanks to gorka \o/ | 14:57 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/818994 | 14:57 |
enriquetaso | Please feel free to review it when you have time. | 14:57 |
enriquetaso | happy thanksgiving! | 14:59 |
enriquetaso | #endmeeting | 14:59 |
opendevmeet | Meeting ended Wed Nov 24 14:59:36 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-11-24-14.57.html | 14:59 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-11-24-14.57.txt | 14:59 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-11-24-14.57.log.html | 14:59 |
eharney | geguileo: for https://bugs.launchpad.net/cinder/+bug/1951982 , i have to wonder if volume rekey actually gets any benefits from multipath | 15:01 |
geguileo | eharney: mmmmm, you are probably right | 15:05 |
eharney | lots of time spent setting up devices etc for a one-shot operation that will only write KBs of data... | 15:06 |
geguileo | it shouldn't be writing enough data to make it worth the extra time to do the attach | 15:06 |
eharney | exactly :) | 15:06 |
geguileo | eharney: I'll update the LP bug and the patch to reflect this (the patch will have a comment to explain why we don't do it) | 15:06 |
geguileo | eharney: thanks for the feedback :-) | 15:07 |
eharney | great, thanks | 15:07 |
enriquetaso | thanks! | 15:09 |
geguileo | eharney: oooooh, I just realized that we should still do this... | 15:13 |
geguileo | because a backend could return a single path and if that patch is down the connection would fail | 15:13 |
eharney | geguileo: hmm, yeah | 15:13 |
geguileo | s/patch/path | 15:13 |
geguileo | that would not be good, so maybe I'll leave it as it is and add a comment stating that even if performance-wise is less performant it should be more robust | 15:14 |
geguileo | as it is = passing the config options instead of False | 15:14 |
eharney | if we think it's a big perf difference, we could try single path first, and then retry with multipath if that fails, but that's getting rather involved... | 15:15 |
geguileo | eharney: by the way, there are many drivers that will do multipath even if that call is saying it doesn't want multipath | 15:18 |
geguileo | because they always return multipathed connection information since the single pathed os-brick connector can iterate through the paths until it finds one that works | 15:19 |
geguileo | and since the cinder call to os-brick's connect_volume during rekeying passes the multipath config option, then it ends up being a multipathed connection anyways | 15:20 |
geguileo | (that's why on many drivers things like backups were doing multipathing anyways) | 15:21 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Honor multipath config everywhere https://review.opendev.org/c/openstack/cinder/+/818994 | 15:22 |
opendevreview | Fábio Oliveira proposed openstack/cinder master: DNM: Fix SolidFire init after setup change https://review.opendev.org/c/openstack/cinder/+/819007 | 15:30 |
opendevreview | huangbingyan proposed openstack/cinder stable/ussuri: fix 'huawei dorado v6' create new volume form snapshot error https://review.opendev.org/c/openstack/cinder/+/818778 | 16:00 |
hemna | rosmaita sorry I had to bail right at my spot in the meeting, I had to run to an appointment. | 16:13 |
rosmaita | hemna: np, hopefully we'll get some eyes on your patch | 16:15 |
opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 16:58 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 17:22 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Reset state robustification for snapshot os-reset_status https://review.opendev.org/c/openstack/cinder/+/804035 | 17:24 |
opendevreview | Benny Kopilov proposed openstack/cinder-tempest-plugin master: Create volume from image delete the used image https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/819158 | 19:05 |
*** dviroel is now known as dviroel|afk | 19:53 | |
opendevreview | Merged openstack/cinder stable/wallaby: [Pure Storage] Add missing DB values when performing create CG from CG snap https://review.opendev.org/c/openstack/cinder/+/818579 | 23:02 |
opendevreview | Ghanshyam proposed openstack/cinder master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/cinder/+/819200 | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!