| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: uploading test(3) https://review.opendev.org/c/openstack/cinder/+/869695 | 05:35 |
|---|---|---|
| *** bhagyashris|brb is now known as bhagyashris | 06:39 | |
| *** bhagyashris is now known as bhagyashris|afk | 06:39 | |
| Mounika | Hi Reviewers, Do we have the cinder meeting today? | 06:39 |
| Mounika | I don't see the etherpad updated with today's date | 06:40 |
| Sathya | Hi Reviewers, please review merge request for FC 2.0 config parameter https://review.opendev.org/c/openstack/cinder/+/869302 | 07:54 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: uploading test(4) https://review.opendev.org/c/openstack/cinder/+/869699 | 08:11 |
| *** geguileo is now known as Guest1064 | 09:26 | |
| akawai | Hi members: Pls help me how to solve "remote unpack failed: error Missing tree 4bd9c8e8429b2d12d0f894af690ed2bc1806e9e5" when executing "git review" command. | 10:17 |
| akawai | I wanna upload two patches with relation chain to https://review.opendev.org/c/openstack/cinder/+/796170 . | 10:18 |
| akawai | One https://review.opendev.org/c/openstack/cinder/+/867340 was succeed, but another is always failed with above error. | 10:18 |
| akawai | "git push --no-thin" and "git fsck" did not work. But I could upload without relation chain, or with relation chain to another patch. what should i do next? | 10:19 |
| opendevreview | Lukas Piwowarski proposed openstack/cinder stable/zed: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/cinder/+/869805 | 10:19 |
| *** geguileor is now known as geguileo | 10:20 | |
| opendevreview | Lukas Piwowarski proposed openstack/cinder stable/zed: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/cinder/+/869805 | 10:39 |
| *** bhagyashris|afk is now known as bhagyashris | 11:32 | |
| opendevreview | Jean-Sébastien Bevilacqua proposed openstack/cinder master: Add cinder Driver for Lustre https://review.opendev.org/c/openstack/cinder/+/853785 | 13:49 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare | 13:59 |
| opendevreview | saikrishnagk proposed openstack/cinder master: [SVf] Adding Support for --delete-volumes flag for delete volume group https://review.opendev.org/c/openstack/cinder/+/869704 | 14:20 |
| opendevreview | Lukas Piwowarski proposed openstack/cinder stable/zed: DNM: Test change in run-tempest role https://review.opendev.org/c/openstack/cinder/+/869805 | 14:27 |
| andrewbogott_ | Who can I add as a reviewer who has +submit? | 14:35 |
| *** senrique is now known as enriquetaso | 14:48 | |
| enriquetaso | #startmeeting cinder_bs | 15:02 |
| opendevmeet | Meeting started Wed Jan 11 15:02:44 2023 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:02 |
| enriquetaso | Hello, only one bug for today's meeting and it's already assigned | 15:03 |
| enriquetaso | # svf : if pool attribute is specified in volume type during retype along with --migration-policy defaults to cinder generic migration | 15:03 |
| enriquetaso | #topic svf : if pool attribute is specified in volume type during retype along with --migration-policy defaults to cinder generic migration | 15:03 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2001619 | 15:03 |
| enriquetaso | svf driver needs to implement retype with migrations on their driver | 15:04 |
| Sathya | yes | 15:04 |
| Sathya | we are trying to implement non disruptive migration | 15:05 |
| happystacker | I have opened a new bug today, do I need to add it somewhere? | 15:06 |
| enriquetaso | thanks Sathya, i think there's not fix proposed upstream yet | 15:06 |
| Sathya | when the pool parameter is mentioned in the volume type it's entering cinder generic migration instead of driver specific implementation | 15:06 |
| enriquetaso | happystacker, sure, let's mentioned it on the open discussion, I'll add it on the next week email report | 15:07 |
| happystacker | excellent! thank you | 15:07 |
| enriquetaso | Sathya++ anything else to share? | 15:08 |
| Sathya | i had discussion with rajat , he said we could take this up as pool attribute is concerned for most of storage drivers | 15:09 |
| Sathya | whoami-rajat you there? | 15:10 |
| Sathya | Our customer requirement needs the pool also for the migration | 15:11 |
| enriquetaso | i think he's in a meeting | 15:11 |
| enriquetaso | ohh, so it may involve some code change on the manager.py | 15:12 |
| Sathya | is there any concerns if we could add this pool attribute in the manager.py | 15:12 |
| enriquetaso | well.. If the change is justified I don't see the problem.. if you proposed a patch for the mid cycle we can discuss it there and get cinder team attention | 15:13 |
| enriquetaso | okay, any other comments? | 15:16 |
| Sathya | we will try to make the changes and test, and to raise the patch | 15:16 |
| enriquetaso | thanks Sathya ! | 15:16 |
| eharney | i think i fixed our py311 failures we discussed last week, just waiting for CI issues to shake out | 15:16 |
| enriquetaso | \o/ | 15:16 |
| enriquetaso | yay | 15:16 |
| enriquetaso | eharney++ | 15:16 |
| enriquetaso | okay, moving to open discussion | 15:16 |
| enriquetaso | #topic open discussion | 15:16 |
| enriquetaso | happystacker, do you mind sharing the bug link? | 15:17 |
| happystacker | I have opened this bug | 15:17 |
| happystacker | https://bugs.launchpad.net/cinder/+bug/2002535 | 15:17 |
| sfv880_ | Hello reviewers, I addressed all the comments on https://review.opendev.org/c/openstack/cinder/+/852369 and https://review.opendev.org/c/openstack/cinder/+/864287 - it is very important for us and our customers are waiting for these fixes to be merged. Also, these changes are required to pass the RHOSP certification. Could you please help me and review it ? Thank you so much! | 15:17 |
| happystacker | this occurs when resizing a server | 15:17 |
| happystacker | it's an NFS env | 15:17 |
| happystacker | it seems that qemu-img is converting a qcow2 to raw | 15:17 |
| eharney | is this related to the NFS bug we were already working on re: qcow2/raw format handling? | 15:18 |
| eharney | enriquetaso: ^ | 15:18 |
| happystacker | it sounds similar | 15:18 |
| happystacker | that happens with image volume caching enabled | 15:18 |
| happystacker | don't understand why this conversion of raw happens | 15:18 |
| enriquetaso | i think it's not, but i need to look at it a big more | 15:19 |
| happystacker | and the attachment doesn't get updated, so attachment is waiting for qcow2 and file is raw | 15:19 |
| enriquetaso | good one: i need to try my patch with cache enabled | 15:19 |
| happystacker | if we manually convert it back to qcow2, then resize work | 15:19 |
| eharney | it would be good if the bug described what cinder operations happen when that nova resize occurs | 15:20 |
| enriquetaso | that's strange | 15:20 |
| happystacker | if I disable cache, it works fine | 15:20 |
| eharney | sounds worth digging into, for sure | 15:21 |
| happystacker | I can update and add more information | 15:21 |
| happystacker | and do a comparaison nova/cinder at the time of the failure | 15:21 |
| enriquetaso | that sounds good, would be nice to have the cinder operations | 15:22 |
| enriquetaso | happystacker++ | 15:22 |
| enriquetaso | just wondering: do you have a patch for this bug happystacker or you just faced it ? | 15:23 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: storwize: Work around bug in unit test https://review.opendev.org/c/openstack/cinder/+/869853 | 15:25 |
| roquej | no patch yet | 15:25 |
| enriquetaso | looks like happystacker is gone lol | 15:25 |
| roquej | i'm back lol | 15:26 |
| enriquetaso | oh cool | 15:26 |
| roquej | i'd like to understand the reason of that concersion | 15:26 |
| roquej | conversion | 15:26 |
| roquej | qcow2 to raw | 15:26 |
| enriquetaso | sounds good, the generic nfs do some raw conversion when getting the image from glance and then converts to qcow2 but that shouldnt be affecting this | 15:27 |
| enriquetaso | okay, only 3 minutes left | 15:27 |
| enriquetaso | any other bug to discuss? | 15:28 |
| eharney | i just submitted this patch above to close out a six year old bug ^ | 15:28 |
| enriquetaso | sfv880_, i'll add the patches to my review list | 15:28 |
| roquej | It seems that this conversion happens from qcow2 to raw | 15:28 |
| roquej | but never got back to qcow2 | 15:28 |
| roquej | which is an issue | 15:28 |
| roquej | I'll move on digging it | 15:28 |
| enriquetaso | thanks eharney | 15:29 |
| roquej | I have to drop. Thank you guys | 15:29 |
| enriquetaso | thanks! | 15:29 |
| enriquetaso | Argonauts please review patches | 15:30 |
| enriquetaso | thanks | 15:30 |
| enriquetaso | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Wed Jan 11 15:30:46 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-01-11-15.02.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-01-11-15.02.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-01-11-15.02.log.html | 15:30 |
| sfv880_ | enriquetaso - thank you so much! | 15:40 |
| whoami-rajat | stephenfin, hey, do we have any document about how the new enginefacade works? | 15:52 |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: Get ready for tox 4 https://review.opendev.org/c/openstack/python-cinderclient/+/868317 | 17:14 |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: DNM: check functional jobs https://review.opendev.org/c/openstack/python-cinderclient/+/869226 | 17:21 |
| opendevreview | Merged openstack/cinder master: Doc: IBM Spectrum Virtualize family [SVf] Volume driver document correction https://review.opendev.org/c/openstack/cinder/+/869691 | 17:29 |
| Tony_Saad | Hello everyone, I am looking for a review on my first patch https://review.opendev.org/c/openstack/cinder/+/867753 | 18:39 |
| *** zaitcev is now known as zaitcev|afk | 18:41 | |
| rosmaita | Tony_Saad: left a comment for you | 18:52 |
| *** zaitcev|afk is now known as zaitcev | 19:55 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!