| *** markstur has joined #openstack-cinder | 00:08 | |
| *** harlowja has quit IRC | 00:23 | |
| *** moshele has quit IRC | 00:25 | |
| *** markstur has quit IRC | 00:27 | |
| *** markstur has joined #openstack-cinder | 00:30 | |
| *** imacdonn has quit IRC | 00:49 | |
| *** imacdonn has joined #openstack-cinder | 00:49 | |
| *** markstur has quit IRC | 00:53 | |
| *** rcernin_ has joined #openstack-cinder | 01:23 | |
| *** gouthamr has quit IRC | 01:23 | |
| *** rcernin has quit IRC | 01:25 | |
| *** zhaochao has joined #openstack-cinder | 01:41 | |
| *** rcernin has joined #openstack-cinder | 02:00 | |
| *** rcernin_ has quit IRC | 02:03 | |
| *** gouthamr has joined #openstack-cinder | 02:36 | |
| *** edmondsw has quit IRC | 02:45 | |
| *** mmethot has quit IRC | 02:46 | |
| *** mmethot has joined #openstack-cinder | 02:49 | |
| *** psachin has joined #openstack-cinder | 02:51 | |
| openstackgerrit | Jay Bryant proposed openstack/os-brick stable/pike: Fix multipath disconnect with path failure https://review.openstack.org/594777 | 03:00 |
|---|---|---|
| *** brinzhang has joined #openstack-cinder | 03:14 | |
| *** nicolasbock has quit IRC | 03:32 | |
| openstackgerrit | Merged openstack/cinder stable/rocky: ScaleIO: Deprecate sio_allow_non_padded_thick_volumes https://review.openstack.org/594556 | 03:46 |
| *** Bhujay has joined #openstack-cinder | 04:16 | |
| *** cmart has quit IRC | 04:31 | |
| *** moshele has joined #openstack-cinder | 05:00 | |
| *** moshele has quit IRC | 05:02 | |
| *** sdake has quit IRC | 05:13 | |
| *** sdake has joined #openstack-cinder | 05:13 | |
| *** edmondsw has joined #openstack-cinder | 05:34 | |
| *** sdake has quit IRC | 05:50 | |
| *** Luzi has joined #openstack-cinder | 05:51 | |
| *** sdake has joined #openstack-cinder | 05:51 | |
| *** sdake has quit IRC | 06:02 | |
| *** sdake has joined #openstack-cinder | 06:04 | |
| *** moshele has joined #openstack-cinder | 06:06 | |
| *** dpawlik has joined #openstack-cinder | 06:08 | |
| *** dpawlik has quit IRC | 06:13 | |
| *** dpawlik has joined #openstack-cinder | 06:15 | |
| *** ircuser-1 has joined #openstack-cinder | 06:21 | |
| *** gkadam has joined #openstack-cinder | 06:31 | |
| *** pcaruana has joined #openstack-cinder | 06:41 | |
| *** rcernin has quit IRC | 07:07 | |
| *** evrardjp has joined #openstack-cinder | 07:26 | |
| *** alexchadin has joined #openstack-cinder | 07:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.openstack.org/594877 | 07:39 |
| *** e0ne has joined #openstack-cinder | 08:01 | |
| *** michalpelka has joined #openstack-cinder | 08:14 | |
| michalpelka | Developing volume driver for cinder, my zuul failed due to not known changes in os-brick. I added 'Depends-on' in my commit message. | 08:21 |
| *** pcaruana has quit IRC | 08:28 | |
| *** pcaruana has joined #openstack-cinder | 08:30 | |
| *** laurent\ has joined #openstack-cinder | 08:42 | |
| *** laurent\ has quit IRC | 08:42 | |
| *** dirk has joined #openstack-cinder | 08:44 | |
| *** laurent\ has joined #openstack-cinder | 08:47 | |
| openstackgerrit | Michał Pełka proposed openstack/cinder master: Add MooseFS/MooseFS Pro Volume Driver https://review.openstack.org/594359 | 08:47 |
| *** michalpelka has quit IRC | 08:59 | |
| *** josecastroleon has quit IRC | 09:00 | |
| *** josecastroleon has joined #openstack-cinder | 09:00 | |
| *** brinzhang has quit IRC | 09:01 | |
| *** brinzhang has joined #openstack-cinder | 09:01 | |
| *** moshele has quit IRC | 09:05 | |
| *** moshele has joined #openstack-cinder | 09:05 | |
| *** brinzhang has quit IRC | 09:12 | |
| *** alexchadin has quit IRC | 09:17 | |
| *** alexchadin has joined #openstack-cinder | 09:21 | |
| *** moshele has quit IRC | 09:34 | |
| *** moshele has joined #openstack-cinder | 09:45 | |
| *** josecastroleon has quit IRC | 09:56 | |
| *** alexchadin has quit IRC | 10:13 | |
| *** alexchadin has joined #openstack-cinder | 10:14 | |
| *** alexchadin has quit IRC | 10:14 | |
| *** alexchadin has joined #openstack-cinder | 10:15 | |
| *** alexchadin has quit IRC | 10:15 | |
| *** nicolasbock has joined #openstack-cinder | 10:44 | |
| *** alexchadin has joined #openstack-cinder | 10:48 | |
| *** ganso has joined #openstack-cinder | 10:51 | |
| *** alexchadin has quit IRC | 10:53 | |
| *** alexchadin has joined #openstack-cinder | 11:04 | |
| *** josecastroleon has joined #openstack-cinder | 11:08 | |
| *** erlon has joined #openstack-cinder | 11:12 | |
| *** alexchadin has quit IRC | 11:19 | |
| *** alexchadin has joined #openstack-cinder | 11:22 | |
| *** abishop has quit IRC | 11:28 | |
| *** luizbag has joined #openstack-cinder | 11:41 | |
| *** mriedem_away is now known as mriedem | 11:54 | |
| *** mriedem has left #openstack-cinder | 11:57 | |
| *** mriedem has joined #openstack-cinder | 11:57 | |
| *** gnufied has quit IRC | 11:58 | |
| *** mriedem has quit IRC | 12:01 | |
| *** mriedem has joined #openstack-cinder | 12:01 | |
| *** abishop has joined #openstack-cinder | 12:03 | |
| BlackDex | Hello there | 12:10 |
| BlackDex | The QoS *iops* options seem to be "static" | 12:10 |
| BlackDex | if i change them they won't apply runtime. | 12:10 |
| BlackDex | I need to re-attach a volume for this to work, or shutdown/poweron the instance | 12:11 |
| BlackDex | os there a way to set this during runtime? | 12:12 |
| jungleboyj | BlackDex: Correct. Those are set in the volume at creation time. | 12:12 |
| jungleboyj | Sorry, at attach time. | 12:12 |
| jungleboyj | It is set in libvirt. Not aware of a way to make that dynamic. | 12:13 |
| BlackDex | oke | 12:14 |
| BlackDex | good to know :) | 12:14 |
| BlackDex | thx jungleboyj | 12:15 |
| jungleboyj | BlackDex: Welcome. If it is like other functions in that area, making that dynamic would be very difficult. | 12:16 |
| BlackDex | for that i think kvm/libvirt need some changes | 12:16 |
| jungleboyj | BlackDex: Correct. | 12:17 |
| BlackDex | the only way for it to be dynamic i think is if the backend supports it | 12:17 |
| jungleboyj | BlackDex: Correct. That would be another way to tackle it. | 12:17 |
| *** psachin has quit IRC | 12:32 | |
| kaisers | Hoi! Quick Q: I want to change some default settings for a Cinder driver. Am i correct that i a) add a deprecation note to the next release and b) change the behaviour with a new note in the release following after the next. Is that the way to go? | 12:33 |
| jungleboyj | kaisers: Hey there. | 12:34 |
| smcginnis | kaisers: Changing a config option completely? Or just changing its default value. | 12:35 |
| kaisers | smcginnis: default value only | 12:35 |
| jungleboyj | smcginnis: That was what I was about to ask. | 12:36 |
| smcginnis | kaisers: Then you can do it in one release. You can change the default value and have a release note under the Upgrade section that calls out that the value has changed and when upgrading the deployer will need to modify it if they wish to keep the old value. | 12:36 |
| jungleboyj | Shouldn't be so complicated for just changing the default value. | 12:36 |
| jungleboyj | smcginnis: ++ | 12:36 |
| kaisers | smcginnis: jungleboyj: Ok, thanks! | 12:37 |
| jungleboyj | smcginnis: RC2: https://review.openstack.org/594948 | 12:40 |
| openstackgerrit | Michał Pełka proposed openstack/cinder master: Add MooseFS/MooseFS Pro Volume Driver https://review.openstack.org/594951 | 12:41 |
| smcginnis | jungleboyj: Got it, thanks! | 12:41 |
| jungleboyj | smcginnis: Welcome. | 12:42 |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Changed default behaviour of nas_secure_file_* for Quobyte driver https://review.openstack.org/594967 | 13:06 |
| openstackgerrit | Michał Pełka proposed openstack/cinder master: Add MooseFS/MooseFS Pro Volume Driver https://review.openstack.org/594981 | 13:17 |
| openstackgerrit | Merged openstack/cinder stable/queens: RemoteFS: Use summarize option for "du" https://review.openstack.org/593630 | 13:18 |
| openstackgerrit | Merged openstack/cinder master: Imported Translations from Zanata https://review.openstack.org/594877 | 13:18 |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Changed default behaviour of nas_secure_file_* for Quobyte driver https://review.openstack.org/594967 | 13:18 |
| *** Bhujay has quit IRC | 13:21 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Changed default behaviour of nas_secure_file_* for Quobyte driver https://review.openstack.org/594967 | 13:21 |
| *** eharney has joined #openstack-cinder | 13:21 | |
| *** dustins has joined #openstack-cinder | 13:24 | |
| *** tpsilva has joined #openstack-cinder | 13:25 | |
| *** gnufied has joined #openstack-cinder | 13:26 | |
| openstackgerrit | Michał Pełka proposed openstack/cinder master: Add MooseFS/MooseFS Pro Volume Driver https://review.openstack.org/594359 | 13:26 |
| *** josecastroleon has quit IRC | 13:44 | |
| *** josecastroleon has joined #openstack-cinder | 13:44 | |
| *** r-daneel has joined #openstack-cinder | 13:48 | |
| *** cmart has joined #openstack-cinder | 13:51 | |
| *** alexchadin has quit IRC | 13:59 | |
| *** gouthamr has quit IRC | 14:00 | |
| *** nicolasbock has quit IRC | 14:04 | |
| *** Luzi has quit IRC | 14:07 | |
| rajinir | junglenoyj,smcginnis: This one is ready , can you review pl https://review.openstack.org/#/c/566943 | 14:09 |
| rajinir | <jungleboyj> This is ready for review, <jungleboyj> | 14:09 |
| *** moshele has quit IRC | 14:10 | |
| *** alexchadin has joined #openstack-cinder | 14:10 | |
| *** jiapei has joined #openstack-cinder | 14:16 | |
| jungleboyj | rajinir: There is a mention of Rocky in there that needs to be updated. | 14:16 |
| *** dpawlik has quit IRC | 14:27 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder stable/rocky: VMAX Rocky doc - important known issue https://review.openstack.org/595020 | 14:30 |
| *** nicolasbock has joined #openstack-cinder | 14:31 | |
| *** jdillaman has joined #openstack-cinder | 14:32 | |
| *** alexchadin has quit IRC | 14:33 | |
| *** gouthamr has joined #openstack-cinder | 14:50 | |
| *** cmart has quit IRC | 14:56 | |
| *** tonyb has quit IRC | 15:04 | |
| *** cmart has joined #openstack-cinder | 15:08 | |
| *** zhaochao has quit IRC | 15:09 | |
| *** pcaruana has quit IRC | 15:10 | |
| *** r-daneel has quit IRC | 15:23 | |
| *** r-daneel has joined #openstack-cinder | 15:24 | |
| *** openstackgerrit has quit IRC | 15:31 | |
| *** nicolasbock has quit IRC | 15:43 | |
| *** openstackgerrit has joined #openstack-cinder | 15:47 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: LVM: Disable multiattach for LIO iSCSI target https://review.openstack.org/595231 | 15:47 |
| *** nicolasbock has joined #openstack-cinder | 15:49 | |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell EMC: Added excluded_domain_ips ListOpt to SC driver https://review.openstack.org/566943 | 15:49 |
| *** pcaruana has joined #openstack-cinder | 15:49 | |
| *** gkadam is now known as gkadam-afk | 15:52 | |
| *** erlon is now known as _erlon_ | 15:55 | |
| *** erlon_ has joined #openstack-cinder | 15:58 | |
| *** _erlon_ is now known as erlon | 15:58 | |
| jungleboyj | Reminder that we have a meeting now. :-) | 16:02 |
| *** gouthamr has quit IRC | 16:05 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595244 | 16:22 |
| *** moshele has joined #openstack-cinder | 16:28 | |
| *** masayukig has quit IRC | 16:28 | |
| *** jiapei has quit IRC | 16:35 | |
| *** cmart has quit IRC | 16:39 | |
| *** gouthamr has joined #openstack-cinder | 16:46 | |
| openstackgerrit | Merged openstack/cinder master: Make solidfire code py3-compatible https://review.openstack.org/591072 | 16:47 |
| *** gkadam-afk is now known as gkadam | 16:49 | |
| *** gkadam has quit IRC | 16:54 | |
| whoami-rajat | Hi jungleboyj smcginnis : i have a ques, does cinder support data deduplication? and if so is it source based or target based? | 16:59 |
| smcginnis | whoami-rajat: That would be up to the backend. Cinder isn't in the IO path. | 17:01 |
| *** cmart has joined #openstack-cinder | 17:04 | |
| KeithMnemonic | smcginnis: sorry for the noise, i figured out what happened, the 11a was not clean and that is why tox was messed up | 17:07 |
| KeithMnemonic | it is better now | 17:07 |
| *** e0ne has quit IRC | 17:07 | |
| jungleboyj | whoami-rajat: What smcginnis said. :-) | 17:13 |
| whoami-rajat | smcginnis: jungleboyj , ok, thanks for the info :) | 17:16 |
| jungleboyj | No problem. | 17:17 |
| *** gnufied has quit IRC | 17:25 | |
| *** dobson` has quit IRC | 17:25 | |
| *** tswanson has quit IRC | 17:25 | |
| *** cburgess has quit IRC | 17:25 | |
| *** mchlumsky has quit IRC | 17:25 | |
| *** sayalilunkad has quit IRC | 17:25 | |
| *** mtreinish has quit IRC | 17:25 | |
| *** N3l1x has quit IRC | 17:25 | |
| *** dulek has quit IRC | 17:25 | |
| *** ericyoung has quit IRC | 17:25 | |
| *** obre has quit IRC | 17:25 | |
| *** Reepicheep_ has quit IRC | 17:25 | |
| *** afazekas_ has quit IRC | 17:25 | |
| *** hemna has quit IRC | 17:25 | |
| *** dims has quit IRC | 17:25 | |
| *** y_storshoo has quit IRC | 17:25 | |
| *** Anticimex has quit IRC | 17:25 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/os-brick master: [DNM] Testing CI triggering https://review.openstack.org/595268 | 17:25 |
| *** gouthamr has quit IRC | 17:29 | |
| *** cburgess has joined #openstack-cinder | 17:32 | |
| *** dims_ has joined #openstack-cinder | 17:36 | |
| *** markvoelker has quit IRC | 17:39 | |
| *** woojay has quit IRC | 17:41 | |
| *** eharney has quit IRC | 18:01 | |
| *** cmart has quit IRC | 18:08 | |
| *** hemna has joined #openstack-cinder | 18:08 | |
| *** mriedem is now known as mriedem_lunch | 18:14 | |
| *** ChanServ has quit IRC | 18:16 | |
| *** ChanServ has joined #openstack-cinder | 18:22 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 18:22 | |
| smcginnis | Baaah! Just realized the scaleio zero padding config option was just added during Rocky. | 18:30 |
| smcginnis | So the whole multi-patch to avoid backporting deprecations was a waste of time. | 18:30 |
| smcginnis | And we could have just changed the existing config option and been done with it. | 18:30 |
| smcginnis | jungleboyj: If we want to do an RC3, I suppose we still have time to clean everything up. ^ | 18:33 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595307 | 18:37 |
| smcginnis | jungleboyj: I have two variations of that one now ^ | 18:38 |
| smcginnis | First one is if we keep it in rocky as new but deprecated and we want to remove the deprecation in stein. | 18:38 |
| jungleboyj | smcginnis: Ok. Let me catch up in a minute here. In another meeting. | 18:38 |
| smcginnis | This current one is if we want to just remove it in rocky right away. | 18:38 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595307 | 18:41 |
| *** gnufied has joined #openstack-cinder | 18:41 | |
| *** moshele has quit IRC | 18:46 | |
| *** dustins has quit IRC | 18:50 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder stable/rocky: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595314 | 18:50 |
| *** dustins has joined #openstack-cinder | 18:54 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595307 | 18:55 |
| *** mriedem_lunch is now known as mriedem | 18:55 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595307 | 18:57 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder stable/rocky: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595314 | 18:59 |
| *** dustins has quit IRC | 19:04 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated iSCSI target config opts https://review.openstack.org/595318 | 19:07 |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell EMC: Added excluded_domain_ips ListOpt to SC driver https://review.openstack.org/566943 | 19:14 |
| whoami-rajat | smcginnis: jungleboyj tommylikehu : please review these small python fixes, | 19:15 |
| whoami-rajat | https://review.openstack.org/#/c/591608/ | 19:15 |
| whoami-rajat | https://review.openstack.org/#/c/594130/ | 19:15 |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell EMC: Added excluded_domain_ips ListOpt to SC driver https://review.openstack.org/566943 | 19:17 |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell EMC: Added excluded_domain_ips ListOpt to SC driver https://review.openstack.org/566943 | 19:18 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove WindowsDriver mapping https://review.openstack.org/595326 | 19:19 |
| *** tdasilva has joined #openstack-cinder | 19:29 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated HPE Lefthand config opts https://review.openstack.org/595328 | 19:29 |
| *** mriedem has quit IRC | 19:35 | |
| *** mriedem has joined #openstack-cinder | 19:38 | |
| KeithMnemonic | jungleboyj: can you do the honors? https://review.openstack.org/#/c/589576 | 19:40 |
| jungleboyj | KeithMnemonic: It has been such a longer journey I am not sure if I am ready for it to be over. | 19:41 |
| KeithMnemonic | dont worry there are 3 more still ;-) | 19:41 |
| KeithMnemonic | and i have a vmare one i have to work on next | 19:42 |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell EMC: Added excluded_domain_ips ListOpt to SC driver https://review.openstack.org/566943 | 19:42 |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell EMC: Added excluded_domain_ips ListOpt to SC driver https://review.openstack.org/566943 | 19:44 |
| ganso | jungleboyj, smcginnis: Hey! Could any of you please take a quick look at this backport to pike? https://review.openstack.org/#/c/592769 it already one +2. Thanks in advance! | 19:45 |
| jungleboyj | ganso: The change ID isn't right, but it looks like the cherry pick commit is right. So I guess it is ok. | 19:46 |
| *** pcaruana has quit IRC | 19:47 | |
| jungleboyj | ganso: Done. | 19:47 |
| ganso | jungleboyj: hmmm shouldn't backports have a difference change-id ? I have never noticed if they were the same before | 19:48 |
| ganso | *different | 19:48 |
| jungleboyj | ganso: Nope, they are all supposed to have the same ID so you can click on it and see the who train of changes. | 19:48 |
| jungleboyj | ganso: I.E. https://review.openstack.org/#/q/Ib9da3278334b9b64e7acc36772c5bae660e86a34 | 19:49 |
| ganso | jungleboyj: oh right, gerrit then will know it is for a different branch, so it doesn't conflict with the original patch | 19:49 |
| ganso | jungleboyj: thanks! I will tell Lucio about this when he's back from vacation =) | 19:49 |
| ganso | jungleboyj: he probably created a new commit message and didn't know the change IDs had to be the same | 19:50 |
| jungleboyj | No problem. If you just do a Cherry-pick -x it happens properly. | 19:50 |
| whoami-rajat | jungleboyj: Hi, can you please take a look to my patches in the above links. Thanks | 19:50 |
| jungleboyj | whoami-rajat: Sure. Give me a little bit. | 19:51 |
| whoami-rajat | jungleboyj: ok :) | 19:52 |
| *** dustins has joined #openstack-cinder | 19:56 | |
| *** e0ne has joined #openstack-cinder | 19:57 | |
| *** luizbag has quit IRC | 20:07 | |
| *** erlon has quit IRC | 20:07 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove flashsystem_multipath_enabled opt https://review.openstack.org/595340 | 20:07 |
| *** dpawlik has joined #openstack-cinder | 20:12 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated HPE Lefthand config opts https://review.openstack.org/595328 | 20:15 |
| jungleboyj | Somebody is going on a cleaning rampage. | 20:18 |
| smcginnis | Rawr | 20:18 |
| jungleboyj | whoami-rajat: Ok, looked at your patches. Thanks. | 20:24 |
| jungleboyj | smcginnis: Ok, what are we doing with the ScaleIO thing? | 20:24 |
| smcginnis | jungleboyj: I think the best option is to merge this one - https://review.openstack.org/#/c/595307/ | 20:25 |
| smcginnis | jungleboyj: Backport it into stable/rocky - https://review.openstack.org/#/c/595314/ | 20:25 |
| smcginnis | jungleboyj: Then cut an RC3 with it. | 20:25 |
| smcginnis | jungleboyj: Then I will backport a clean fix for that issue to the stable branches. | 20:26 |
| smcginnis | jungleboyj: Since a new bug was filed on that rather than just reopening the old one, I missed that it was just something added in Rocky. So there was no need to be concerned about backporting deprecations. | 20:26 |
| *** tonyb has joined #openstack-cinder | 20:26 | |
| *** e0ne has quit IRC | 20:27 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder master: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595307 | 20:27 |
| smcginnis | jungleboyj: We can just make it right and backport the correct fix as far back as we can. | 20:27 |
| smcginnis | Oops | 20:27 |
| jungleboyj | :-) | 20:27 |
| jungleboyj | smcginnis: Ok, that makes sense. | 20:27 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder stable/rocky: Remove deprecated sio_allow_non_padded_thick_volumes https://review.openstack.org/595314 | 20:27 |
| smcginnis | Old habit on that one I guess. :) | 20:28 |
| jungleboyj | I can't believe we spent a whole two hours fighting about deprecation when it wasn't an issue. | 20:28 |
| smcginnis | Yeah, feel kind of dumb about that one. But also, no one pointed out that that was the case. | 20:28 |
| jungleboyj | My bad. I got so wrapped up in the issue I didn't think to see if it was an issue. | 20:29 |
| smcginnis | I think we all did that. | 20:30 |
| jungleboyj | Lesson learned. | 20:30 |
| smcginnis | Yeah | 20:31 |
| jungleboyj | Ok. Lets fix this right and spin another RC. | 20:31 |
| *** ericyoung has joined #openstack-cinder | 20:33 | |
| jungleboyj | smcginnis: Any issues with also merging Helen's doc change here ? https://review.openstack.org/#/c/595020/1 | 20:34 |
| smcginnis | jungleboyj: I suppose not. Would probably be better to just address the issue in master and backport it though. | 20:35 |
| jungleboyj | smcginnis: You want to ask that question and see what she says before we decide whether to include it or not? | 20:38 |
| *** harlowja has joined #openstack-cinder | 20:42 | |
| *** cmart has joined #openstack-cinder | 20:48 | |
| *** erlon_ has quit IRC | 20:59 | |
| *** abishop has quit IRC | 21:04 | |
| *** gouthamr has joined #openstack-cinder | 21:12 | |
| *** dansmith is now known as htimsnad | 21:34 | |
| mriedem | is there any reason os-migrate_volume isn't documented in the api-ref? or just no one has done it yet? | 21:44 |
| *** dustins has quit IRC | 21:44 | |
| smcginnis | Maybe because it is an API extension. | 21:48 |
| smcginnis | Though we updated it with a microversion, which should have been done along with moving it into the standard API. | 21:48 |
| smcginnis | os-migrate_volume: {'force_host_copy': False, 'lock_volume': False, 'cluster: None} | 21:49 |
| *** dustins has joined #openstack-cinder | 21:51 | |
| *** dustins is now known as dustins_ | 21:52 | |
| *** dpawlik has quit IRC | 21:53 | |
| *** dustins_ is now known as dustins | 21:53 | |
| *** mriedem has quit IRC | 21:55 | |
| *** dustins has quit IRC | 21:58 | |
| *** dustins has joined #openstack-cinder | 21:58 | |
| *** mriedem has joined #openstack-cinder | 21:59 | |
| mriedem | heh https://bugs.launchpad.net/cinder/+bug/1607539 | 22:00 |
| openstack | Launchpad bug 1607539 in Cinder "api documentation missing for many actions" [Undecided,In progress] - Assigned to lihaijing (lihaijing) | 22:00 |
| mriedem | maybe i'll start something | 22:00 |
| *** dustins has quit IRC | 22:03 | |
| *** dustins has joined #openstack-cinder | 22:04 | |
| *** cmart has quit IRC | 22:18 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Drop legacy backup service support https://review.openstack.org/595372 | 22:25 |
| *** ganso has quit IRC | 22:28 | |
| *** dustins has quit IRC | 22:32 | |
| *** dustins has joined #openstack-cinder | 22:32 | |
| *** rcernin has joined #openstack-cinder | 22:34 | |
| mriedem | so what does force_host_copy mean for volume migration? | 22:37 |
| mriedem | i've got all of the other params documented | 22:37 |
| openstackgerrit | Merged openstack/cinder stable/pike: NetApp ONTAP: Fix driver force detach operations https://review.openstack.org/592769 | 22:38 |
| mriedem | it looks to me like force_host_copy doesn't do anything in a volume migration, only retype? | 22:41 |
| mriedem | https://docs.openstack.org/cinder/latest/contributor/migration.html | 22:47 |
| mriedem | " Enables or disables generic host-based force-migration, which bypasses the driver optimization." ? | 22:47 |
| mriedem | so if force_host_copy=False, it goes through the driver to do the migration, else it does a generic migration not using the volume driver | 22:48 |
| mriedem | what's the difference? the driver can optimize for the backend? | 22:48 |
| *** ChanServ has quit IRC | 22:49 | |
| *** dustins has quit IRC | 22:52 | |
| openstackgerrit | Merged openstack/cinder master: Optimizing code (wrap_socket()) https://review.openstack.org/591608 | 22:57 |
| openstackgerrit | Merged openstack/cinder master: Make divisibility py3 compatible in DRBD driver https://review.openstack.org/594130 | 23:02 |
| *** ChanServ has joined #openstack-cinder | 23:03 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 23:03 | |
| *** dustins has joined #openstack-cinder | 23:05 | |
| *** dustins has quit IRC | 23:05 | |
| *** dustins has joined #openstack-cinder | 23:06 | |
| *** dustins has quit IRC | 23:08 | |
| *** tpsilva has quit IRC | 23:16 | |
| *** jdillaman has quit IRC | 23:19 | |
| *** gouthamr has quit IRC | 23:48 | |
| *** r-daneel has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!