*** brinzhang has joined #openstack-cinder | 00:26 | |
*** boxiang has joined #openstack-cinder | 00:40 | |
*** boxiang_ has quit IRC | 00:40 | |
*** ociuhandu has joined #openstack-cinder | 01:07 | |
*** ociuhandu has quit IRC | 01:12 | |
*** Liang__ has joined #openstack-cinder | 01:21 | |
*** andrebeltrami has quit IRC | 01:23 | |
*** jv|afk has joined #openstack-cinder | 01:27 | |
*** lvbin01 has quit IRC | 01:30 | |
*** lvbin01 has joined #openstack-cinder | 01:30 | |
*** root____ has joined #openstack-cinder | 01:41 | |
*** root____ is now known as ruffian_sheep | 01:41 | |
ruffian_sheep | whoami-rajat: hi,are u online? | 01:41 |
---|---|---|
*** sapd1 has joined #openstack-cinder | 02:02 | |
*** brinzhang_ has joined #openstack-cinder | 02:09 | |
*** brinzhang has quit IRC | 02:11 | |
*** rcernin has quit IRC | 02:24 | |
*** boxiang_ has joined #openstack-cinder | 02:30 | |
*** rcernin has joined #openstack-cinder | 02:31 | |
*** jv|afk has quit IRC | 02:32 | |
*** boxiang has quit IRC | 02:33 | |
*** brinzhang_ is now known as brinzhang | 03:00 | |
*** Xuchu has joined #openstack-cinder | 03:18 | |
*** rcernin has quit IRC | 03:23 | |
*** rcernin has joined #openstack-cinder | 03:31 | |
*** psachin has joined #openstack-cinder | 03:33 | |
*** rcernin has quit IRC | 03:46 | |
*** josephillips has joined #openstack-cinder | 03:59 | |
josephillips | hey question | 03:59 |
josephillips | before i can resize a root volume turning off the vm reset-state to available and later perform the resize | 04:00 |
josephillips | now im having a issue | 04:00 |
josephillips | Failed to schedule_extend_volume: Cannot place volume b309a788-9e28-42e3-9018-c999e0758df2 on cinder@cinder-netapp-1#10.10.10.253:/vol_cloud_openstack.: cinder.exception.NoValidBackend: Cannot place volume b309a788-9e28-42e3-9018-c999e0758df2 on cinder@cinder-netapp-1#10.10.10.253:/vol_cloud_openstack. | 04:01 |
josephillips | only work | 04:01 |
josephillips | when i perform a attach-state | 04:01 |
*** sapd1 has quit IRC | 04:12 | |
*** Xuchu_ has joined #openstack-cinder | 04:12 | |
*** Xuchu has quit IRC | 04:13 | |
*** Xuchu_ is now known as Xuchu | 04:13 | |
*** rcernin has joined #openstack-cinder | 04:23 | |
*** vishalmanchanda has joined #openstack-cinder | 04:26 | |
*** rcernin has quit IRC | 04:28 | |
*** rcernin has joined #openstack-cinder | 04:29 | |
*** ykarel|away is now known as ykarel | 04:31 | |
*** abdysn has joined #openstack-cinder | 04:57 | |
*** ociuhandu has joined #openstack-cinder | 05:09 | |
*** sapd1 has joined #openstack-cinder | 05:11 | |
*** ociuhandu has quit IRC | 05:14 | |
*** udesale has joined #openstack-cinder | 05:18 | |
whoami-rajat | ruffian_sheep, hey | 05:26 |
whoami-rajat | josephillips, hi, why did you perform reset-state ? | 05:27 |
whoami-rajat | josephillips, we can resize an in-use volume but that depends on the backend | 05:27 |
*** rcernin has quit IRC | 05:36 | |
*** rcernin has joined #openstack-cinder | 05:36 | |
*** vishakha has joined #openstack-cinder | 05:40 | |
*** brinzhang_ has joined #openstack-cinder | 05:47 | |
*** brinzhang has quit IRC | 05:48 | |
*** rcernin has quit IRC | 05:51 | |
*** rcernin has joined #openstack-cinder | 05:55 | |
*** rcernin has quit IRC | 06:10 | |
*** rcernin has joined #openstack-cinder | 06:13 | |
*** boxiang has joined #openstack-cinder | 06:17 | |
*** boxiang_ has quit IRC | 06:19 | |
*** rcernin has quit IRC | 06:28 | |
*** ociuhandu has joined #openstack-cinder | 06:45 | |
*** tkajinam has quit IRC | 06:56 | |
*** tkajinam has joined #openstack-cinder | 06:56 | |
*** rcernin has joined #openstack-cinder | 06:57 | |
*** rcernin has quit IRC | 07:02 | |
*** sapd1 has quit IRC | 07:03 | |
*** ociuhandu has quit IRC | 07:10 | |
*** sapd1 has joined #openstack-cinder | 07:21 | |
*** jawad_axd has joined #openstack-cinder | 07:23 | |
*** tosky has joined #openstack-cinder | 07:24 | |
*** baba-tunde has joined #openstack-cinder | 07:40 | |
openstackgerrit | wanghao proposed openstack/cinder master: Support to query snapshot filter by updated_at/created_at https://review.opendev.org/733062 | 07:51 |
*** brinzhang_ has quit IRC | 07:52 | |
openstackgerrit | Hervé Beraud proposed openstack/cinder master: Stop to use the __future__ module. https://review.opendev.org/732822 | 07:56 |
*** ociuhandu has joined #openstack-cinder | 07:59 | |
*** baba-tunde has quit IRC | 08:07 | |
*** pcheli has joined #openstack-cinder | 08:12 | |
*** ociuhandu has quit IRC | 08:17 | |
*** baba-tunde has joined #openstack-cinder | 08:22 | |
*** pcheli has quit IRC | 08:23 | |
*** m75abrams has joined #openstack-cinder | 08:34 | |
*** martinkennelly has joined #openstack-cinder | 08:35 | |
*** redrobot has quit IRC | 08:35 | |
*** yaawang has quit IRC | 08:37 | |
*** yaawang has joined #openstack-cinder | 08:38 | |
*** e0ne has joined #openstack-cinder | 08:49 | |
*** tkajinam has quit IRC | 08:53 | |
*** tkajinam has joined #openstack-cinder | 08:54 | |
*** sapd1 has quit IRC | 08:57 | |
*** yaawang has quit IRC | 09:00 | |
*** yaawang has joined #openstack-cinder | 09:02 | |
*** ociuhandu has joined #openstack-cinder | 09:04 | |
*** Xuchu has quit IRC | 09:08 | |
*** tkajinam has quit IRC | 09:09 | |
*** sapd1 has joined #openstack-cinder | 09:11 | |
*** ociuhandu has quit IRC | 09:30 | |
*** ociuhandu has joined #openstack-cinder | 09:31 | |
*** ociuhandu has quit IRC | 09:42 | |
*** ociuhandu has joined #openstack-cinder | 09:42 | |
*** udesale_ has joined #openstack-cinder | 09:53 | |
*** udesale has quit IRC | 09:56 | |
*** udesale_ has quit IRC | 10:00 | |
*** e0ne has quit IRC | 10:14 | |
*** e0ne has joined #openstack-cinder | 10:14 | |
*** Liang__ has quit IRC | 10:17 | |
*** m75abrams has quit IRC | 10:18 | |
*** ociuhandu_ has joined #openstack-cinder | 10:27 | |
*** ociuhandu_ has quit IRC | 10:28 | |
*** ociuhandu has quit IRC | 10:30 | |
*** ociuhandu has joined #openstack-cinder | 10:31 | |
*** ociuhandu has quit IRC | 10:32 | |
*** ociuhandu has joined #openstack-cinder | 10:33 | |
*** ociuhandu has quit IRC | 10:34 | |
*** e0ne_ has joined #openstack-cinder | 10:34 | |
*** e0ne has quit IRC | 10:34 | |
*** ociuhandu has joined #openstack-cinder | 10:35 | |
*** ociuhandu has quit IRC | 10:36 | |
*** ociuhandu has joined #openstack-cinder | 10:37 | |
*** ociuhandu has quit IRC | 10:37 | |
*** ociuhandu has joined #openstack-cinder | 10:38 | |
*** ociuhandu has quit IRC | 10:49 | |
*** priteau has joined #openstack-cinder | 10:59 | |
*** e0ne_ has quit IRC | 11:06 | |
*** e0ne has joined #openstack-cinder | 11:06 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/os-brick master: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733098 | 11:09 |
*** boxiang has quit IRC | 11:09 | |
*** boxiang has joined #openstack-cinder | 11:10 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/os-brick stable/ussuri: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733099 | 11:10 |
openstackgerrit | Ivan Pchelintsev proposed openstack/os-brick stable/train: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733100 | 11:11 |
*** sapd1 has quit IRC | 11:12 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/os-brick stable/stein: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733102 | 11:12 |
openstackgerrit | Ivan Pchelintsev proposed openstack/os-brick stable/rocky: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733103 | 11:12 |
openstackgerrit | Ivan Pchelintsev proposed openstack/os-brick stable/queens: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733104 | 11:13 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733105 | 11:14 |
rosmaita | geguileo: whoami-rajat: smcginnis: eharney: e0ne: hemna: ^^ | 11:14 |
*** spatel has joined #openstack-cinder | 11:14 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/ussuri: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733106 | 11:14 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/train: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733107 | 11:15 |
whoami-rajat | rosmaita, should we wait for third party CI? | 11:15 |
rosmaita | no | 11:16 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/ussuri: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733106 | 11:16 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/stein: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733108 | 11:17 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/rocky: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733109 | 11:17 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/queens: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733110 | 11:18 |
*** spatel has quit IRC | 11:19 | |
*** pcheli has joined #openstack-cinder | 11:21 | |
*** e0ne has quit IRC | 11:21 | |
*** pck has quit IRC | 11:21 | |
*** e0ne has joined #openstack-cinder | 11:22 | |
pcheli | @rosmaita @geguileo @whoami-rajat @eharney @smcginnis patches are ready for review. | 11:22 |
rosmaita | pcheli: ty | 11:22 |
*** TobbeCN has joined #openstack-cinder | 11:23 | |
*** ociuhandu has joined #openstack-cinder | 11:25 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/os-brick master: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733098 | 11:27 |
*** e0ne has quit IRC | 11:30 | |
*** pcheli has quit IRC | 11:31 | |
*** e0ne has joined #openstack-cinder | 11:32 | |
*** ociuhandu has quit IRC | 11:32 | |
*** pck has joined #openstack-cinder | 11:33 | |
*** e0ne_ has joined #openstack-cinder | 11:35 | |
*** e0ne has quit IRC | 11:35 | |
*** e0ne has joined #openstack-cinder | 11:36 | |
*** e0ne_ has quit IRC | 11:39 | |
*** e0ne has quit IRC | 11:42 | |
*** ociuhandu has joined #openstack-cinder | 11:45 | |
*** e0ne has joined #openstack-cinder | 11:47 | |
*** e0ne_ has joined #openstack-cinder | 11:50 | |
*** ociuhandu has quit IRC | 11:50 | |
*** jv|afk has joined #openstack-cinder | 11:50 | |
*** e0ne has quit IRC | 11:52 | |
*** andrebeltrami has joined #openstack-cinder | 11:58 | |
*** e0ne has joined #openstack-cinder | 11:59 | |
*** e0ne_ has quit IRC | 12:00 | |
*** e0ne has quit IRC | 12:02 | |
*** enriquetaso has joined #openstack-cinder | 12:02 | |
*** Xuchu has joined #openstack-cinder | 12:02 | |
*** e0ne has joined #openstack-cinder | 12:09 | |
*** Xuchu has quit IRC | 12:12 | |
*** e0ne has quit IRC | 12:19 | |
*** e0ne_ has joined #openstack-cinder | 12:19 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733105 | 12:25 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/train: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733107 | 12:26 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/stein: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733108 | 12:26 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/queens: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733110 | 12:27 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/rocky: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733109 | 12:28 |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder stable/ussuri: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733106 | 12:28 |
*** martinkennelly has quit IRC | 12:36 | |
*** martinkennelly has joined #openstack-cinder | 12:38 | |
openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: DNM test the newer ceph and also on CentOS 8 https://review.opendev.org/729766 | 12:41 |
*** martinkennelly has quit IRC | 12:45 | |
*** e0ne_ has quit IRC | 12:47 | |
*** xinliang has joined #openstack-cinder | 12:48 | |
*** martinkennelly has joined #openstack-cinder | 12:51 | |
*** e0ne has joined #openstack-cinder | 12:53 | |
*** tkajinam has joined #openstack-cinder | 12:57 | |
*** e0ne has quit IRC | 13:06 | |
*** e0ne has joined #openstack-cinder | 13:07 | |
*** e0ne has quit IRC | 13:11 | |
*** psachin has quit IRC | 13:14 | |
*** e0ne has joined #openstack-cinder | 13:15 | |
*** ociuhandu has joined #openstack-cinder | 13:16 | |
*** kaisers has joined #openstack-cinder | 13:19 | |
*** e0ne has quit IRC | 13:21 | |
*** ociuhandu has quit IRC | 13:21 | |
*** Guest27280 has joined #openstack-cinder | 13:22 | |
*** e0ne has joined #openstack-cinder | 13:24 | |
*** Guest27280 is now known as redrobot | 13:25 | |
*** e0ne has quit IRC | 13:30 | |
*** ociuhandu has joined #openstack-cinder | 13:31 | |
*** ociuhandu has quit IRC | 13:32 | |
*** ociuhandu has joined #openstack-cinder | 13:32 | |
*** e0ne has joined #openstack-cinder | 13:33 | |
*** Liang__ has joined #openstack-cinder | 13:46 | |
*** Liang__ is now known as LiangFang | 13:47 | |
*** vishakha has quit IRC | 13:57 | |
*** LiangFang has quit IRC | 14:06 | |
*** lpetrut has joined #openstack-cinder | 14:10 | |
*** Liang__ has joined #openstack-cinder | 14:13 | |
*** abdysn has quit IRC | 14:17 | |
*** ociuhandu has quit IRC | 14:18 | |
*** sapd1 has joined #openstack-cinder | 14:19 | |
*** kaisers has quit IRC | 14:20 | |
*** xinliang has quit IRC | 14:22 | |
*** lpetrut has quit IRC | 14:30 | |
*** boxiang_ has joined #openstack-cinder | 14:32 | |
*** boxiang has quit IRC | 14:35 | |
openstackgerrit | Merged openstack/os-brick master: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733098 | 14:46 |
openstackgerrit | Luigi Toscano proposed openstack/cinder-tempest-plugin master: WIP Skip a few c-bak tests broken by the new ceph https://review.opendev.org/733161 | 14:57 |
*** ociuhandu has joined #openstack-cinder | 14:57 | |
smcginnis | rosmaita: Umm, probably should do separate release patches for each cycle for that os-brick release. | 14:59 |
smcginnis | That would speed things up. | 14:59 |
*** ociuhandu has quit IRC | 15:08 | |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - Array capabilities extend fix https://review.opendev.org/733166 | 15:10 |
openstackgerrit | Simon O'Donovan proposed openstack/cinder master: PowerMax Driver - RDF State Validation Enhancements https://review.opendev.org/733167 | 15:14 |
*** jawad_axd has quit IRC | 15:20 | |
*** ociuhandu has joined #openstack-cinder | 15:31 | |
*** ociuhandu has quit IRC | 15:32 | |
*** ociuhandu_ has joined #openstack-cinder | 15:32 | |
*** jv|afk has quit IRC | 15:32 | |
*** ociuhandu_ has quit IRC | 15:34 | |
*** ociuhandu has joined #openstack-cinder | 15:34 | |
*** ociuhandu_ has joined #openstack-cinder | 15:51 | |
*** ociuhandu has quit IRC | 15:54 | |
*** jv|afk has joined #openstack-cinder | 15:55 | |
*** ociuhandu_ has quit IRC | 15:55 | |
*** jv|afk has quit IRC | 16:01 | |
josephillips | whoami-rajat: hi | 16:05 |
*** Liang__ has quit IRC | 16:05 | |
whoami-rajat | josephillips, hey | 16:05 |
josephillips | whoami-rajat: no is not possible | 16:06 |
josephillips | look this guide | 16:06 |
josephillips | https://mcwhirter.com.au/craige/blog/2014/Resizing_a_Root_Volume_for_an_Openstack_Instance/ | 16:06 |
josephillips | all volumes in-use can't be resized | 16:06 |
geguileo | whoami-rajat: still around? | 16:06 |
whoami-rajat | geguileo, yep | 16:07 |
geguileo | whoami-rajat: I think we have a bug related to the __DEFAULT__ volume type thingy | 16:07 |
whoami-rajat | geguileo, apart from the one brian was fixing? | 16:07 |
*** ykarel is now known as ykarel|away | 16:07 | |
josephillips | this practice work on rocky | 16:08 |
josephillips | now on train | 16:08 |
geguileo | whoami-rajat: yup, if you do "cinder type-default" you will only get the one defined in the config | 16:08 |
josephillips | i get this error | 16:08 |
josephillips | Failed to schedule_extend_volume: Cannot place volume b309a788-9e28-42e3-9018-c999e0758df2 on cinder@cinder-netapp-1#10.10.10.253:/vol_cloud_openstack.: cinder.exception.NoValidBackend: Cannot place volume b309a788-9e28-42e3-9018-c999e0758df2 on cinder@cinder-netapp-1#10.10.10.253:/vol_cloud_openstack. | 16:09 |
whoami-rajat | josephillips, i'm not aware of this document but this doesn't seem official | 16:09 |
josephillips | no its not official | 16:09 |
*** trident has quit IRC | 16:09 | |
whoami-rajat | josephillips, https://github.com/openstack/cinder/blob/master/cinder/api/openstack/rest_api_version_history.rst#342 | 16:09 |
josephillips | ohh | 16:09 |
whoami-rajat | geguileo, oh | 16:09 |
whoami-rajat | geguileo, i didn't consider that :/ | 16:09 |
geguileo | whoami-rajat: this is the API entry https://github.com/openstack/cinder/blob/master/cinder/api/v2/types.py#L55 | 16:10 |
geguileo | whoami-rajat: this is the code it calls for default https://github.com/openstack/cinder/blob/master/cinder/volume/volume_types.py#L177 | 16:10 |
geguileo | whoami-rajat: wait... | 16:10 |
geguileo | it does work | 16:10 |
geguileo | I'm blind XDXDXD | 16:10 |
geguileo | whoami-rajat: can you forget the last couple of minutes? ;-) | 16:11 |
whoami-rajat | geguileo, yep i checked :P | 16:11 |
whoami-rajat | it just shows the current default | 16:11 |
whoami-rajat | if it's the conf then that else the __DEFAULT__ one | 16:11 |
geguileo | whoami-rajat: yeah, I missed the else part | 16:11 |
whoami-rajat | but honestly, i didn't consider that during the changes | 16:11 |
whoami-rajat | thanks for mentioning that | 16:12 |
*** trident has joined #openstack-cinder | 16:12 | |
*** jv|afk has joined #openstack-cinder | 16:12 | |
whoami-rajat | josephillips, it is backend specific to which will support the extend in-use feature, which backend are you using? | 16:12 |
josephillips | netapp nfs | 16:13 |
geguileo | is that the conversation we had on the PTG? | 16:13 |
geguileo | NFS extend being broken? | 16:13 |
geguileo | when in use? | 16:13 |
josephillips | i will tried now | 16:14 |
josephillips | i was trying with api version 3.0 | 16:14 |
josephillips | i wil try now | 16:14 |
josephillips | with a updated api | 16:14 |
whoami-rajat | yeah, this seems the same | 16:14 |
geguileo | josephillips: extending an in-use NFS volume is currently broken | 16:14 |
josephillips | oh | 16:14 |
*** martinkennelly has quit IRC | 16:14 | |
geguileo | josephillips: you'll probably see a locking error in the logs | 16:14 |
josephillips | do you have the bug id? | 16:15 |
geguileo | josephillips: look at L179 for today's notes on the topic: https://etherpad.opendev.org/p/victoria-ptg-cinder | 16:15 |
whoami-rajat | i guess the netapp nfs uses the base NFS lseki ^ | 16:15 |
geguileo | whoami-rajat: that's what they said, that not only netapp, but other drivers as well | 16:15 |
geguileo | josephillips: https://bugs.launchpad.net/cinder/+bug/1870367 | 16:16 |
openstack | Launchpad bug 1870367 in Cinder "Fail to extend attached volume using generic NFS driver" [High,In progress] - Assigned to Silvan Kaiser (2-silvan) | 16:16 |
whoami-rajat | geguileo, yep, also mentioned quobyte but just confirming :) | 16:16 |
lseki | yes, unfortunately extend attached volume is currently broken for some NFS drivers | 16:19 |
josephillips | with the turned off vm | 16:24 |
josephillips | should word? | 16:24 |
josephillips | work? | 16:24 |
openstackgerrit | Merged openstack/os-brick stable/ussuri: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733099 | 16:26 |
geguileo | josephillips: yes, it should | 16:26 |
josephillips | ok let me try it | 16:27 |
josephillips | the problem is when is ON the vm right? | 16:27 |
geguileo | josephillips: yes, that is correct | 16:28 |
josephillips | ok i will try | 16:28 |
josephillips | Failed to schedule_extend_volume: Cannot place volume 04ddb9b7-2d92-443a-a3fe-4ef29f4605b3 on cinder@cinder-netapp-1#10.10.10.253:/vol_cloud_openstack.: cinder.exception.NoValidBackend: Cannot place volume 04ddb9b7-2d92-443a-a3fe-4ef29f4605b3 on cinder@cinder-netapp-1#10.10.10.253:/vol_cloud_openstack. | 16:33 |
josephillips | same error | 16:33 |
josephillips | using the api | 16:33 |
josephillips | 3.43 i can perform the resize without reset-state | 16:34 |
rosmaita | smcginnis: good idea, will separate patches | 16:35 |
geguileo | josephillips: reset state? but it's in available state, right? | 16:36 |
josephillips | now using the new api i dont have to reset the state | 16:36 |
josephillips | resetting the state | 16:36 |
josephillips | to available throw the same error | 16:36 |
josephillips | the one way this error don't show is when the attach-status is detached | 16:36 |
*** sapd1 has quit IRC | 16:36 | |
josephillips | and perform the resize | 16:36 |
geguileo | josephillips: I'm getting lost, weren't you going to test extend for an offline volume? | 16:40 |
geguileo | then it should already be available and detached | 16:41 |
josephillips | geguileo: no , im trying to extend a volume of a turned off VM i tried with extend and i get the error i write before , the way that the resize can be performed is when i perform cinder reset-state --attach-status detached | 16:42 |
josephillips | after that is possible to perform the resize | 16:42 |
josephillips | i can't detach the volume because is a root volume | 16:42 |
josephillips | this is why i had to do with reset-state | 16:43 |
josephillips | the problem with this is maybe idk how to doit the association of attached with the vm get lost | 16:43 |
geguileo | josephillips: if the volume is turned off, but nova doesn't detach, then the volume is not offline | 16:43 |
geguileo | I mean | 16:43 |
geguileo | josephillips: if the VM is turned off, but nova doesn't detach, then the volume is not offline | 16:43 |
josephillips | got it | 16:44 |
josephillips | so this error is because is still attach to the vm right | 16:45 |
geguileo | and probably the reason why it doesn't work with prior microversions is because originally we didn't have extend for in-use volumes | 16:45 |
geguileo | josephillips: yup, confirmed 3.42 is when we introduced volume extend in-use | 16:46 |
geguileo | josephillips: https://github.com/openstack/cinder/blob/master/cinder/api/microversions.py#L126 | 16:46 |
josephillips | i see and now i dont have to perform the reset-state --state available anymore | 16:47 |
josephillips | using the 3.42 api | 16:47 |
geguileo | josephillips: yup, as long as the VM is not running, you can use that microversion and you won't run in the other NFS bug | 16:47 |
josephillips | yes | 16:47 |
josephillips | the vm is not running | 16:48 |
josephillips | but i still get this error | 16:48 |
josephillips | this scnerario is | 16:48 |
*** boxiang_ has quit IRC | 16:49 | |
*** boxiang has joined #openstack-cinder | 16:49 | |
josephillips | using 3.42 api , | 16:49 |
josephillips | vm is turned off | 16:49 |
josephillips | when i perform the extend i get error i post before | 16:49 |
josephillips | mhmm | 16:49 |
smcginnis | The volume is still attached, even if the VM is turned off. | 16:50 |
geguileo | and on 3.43 it works? | 16:50 |
geguileo | smcginnis: it's attached, but Nova is not "using" it in the sense that the lock is not there, so Cinder should be able to do its thing | 16:52 |
geguileo | josephillips: I hadn't seen that error, but it looks like a scheduling problem, which should be unrelated to anything else and should fail regardless of the state fo the volume | 16:52 |
josephillips | now geguileo if i perform reset-state status detach | 16:53 |
josephillips | attach-status detach | 16:54 |
josephillips | work well | 16:54 |
geguileo | josephillips: the error you were getting was the scheduling one I saw? | 16:54 |
josephillips | yep | 16:55 |
josephillips | came from cinder-scheduler | 16:55 |
lseki | does the scheduler consider a volume as in-use, even if the vm is turned off? | 16:57 |
lseki | then it should filter out the netapp nfs back end | 16:57 |
geguileo | lseki: yes, it will be consider to be in use as long as Nova doesn't dettach | 16:58 |
openstackgerrit | Merged openstack/os-brick stable/train: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733100 | 16:58 |
openstackgerrit | Merged openstack/os-brick stable/stein: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733102 | 16:58 |
openstackgerrit | Merged openstack/os-brick stable/rocky: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733103 | 16:58 |
openstackgerrit | Merged openstack/os-brick stable/queens: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733104 | 16:58 |
geguileo | but I think here we have an issue with having different behavior between extend_attached_volume and exted methods | 16:58 |
geguileo | but as far as I can see they are mostly the same | 17:00 |
geguileo | josephillips: I believe you'll have to debug the scheduler to see why it's giving that error | 17:02 |
geguileo | I don't know if you have meaningful messages on the scheduler prior to that one for that same request | 17:02 |
josephillips | Backend doesn't support attached volume extend _satisfies_extra_specs /usr/lib/python3/dist-packages/cinder/scheduler/filters/capabilities_filter.py:45 | 17:03 |
josephillips | this happend before | 17:03 |
geguileo | josephillips: are you using default NFS or netapp? | 17:06 |
josephillips | netapp | 17:06 |
geguileo | josephillips: yeah, that's the issue here | 17:06 |
geguileo | josephillips: since they know that attached extend is broken, they have disabled it in their driver | 17:07 |
geguileo | so they are reporting online_extend_support set to false on the pool | 17:07 |
josephillips | lol i have a big issue now resizing root disk then :( | 17:08 |
geguileo | yup you do | 17:08 |
geguileo | you have to do the reset state to fake it | 17:08 |
josephillips | y do the reset state | 17:09 |
josephillips | the problem with this | 17:09 |
josephillips | is how i can associate again to the vm | 17:09 |
geguileo | are you running master? | 17:09 |
josephillips | nope stable/train | 17:09 |
geguileo | I think the less bad option is modifying the DB manually to set the volume's status | 17:10 |
geguileo | or changing the driver code | 17:11 |
geguileo | manually so it doesn't report that value | 17:11 |
lseki | driver code to be modified: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/dataontap/nfs_cmode.py#L276 | 17:11 |
geguileo | yup, that one | 17:12 |
josephillips | i guess the less invasive is set this value to true | 17:12 |
josephillips | what is the worst thing can happend? | 17:12 |
josephillips | i will turn off this vm | 17:12 |
geguileo | josephillips: but then if you run an extend without stopping the VM you will run into a problem | 17:13 |
lseki | if you forget turning off, the volume will get in error-extending state | 17:13 |
geguileo | ^ that | 17:13 |
josephillips | ok sound better | 17:13 |
geguileo | lseki: but we should be able to reset the state it to in-use , right? | 17:13 |
lseki | yes | 17:14 |
geguileo | then it's probably better than running a sql command to update the volume XDXD | 17:14 |
lseki | not sure what exactly happens to the vm while the root volume is in error-extending state | 17:15 |
geguileo | lseki: probably nothing, but yeah, josephillips should confirm it with a test volume | 17:16 |
josephillips | geguileo: yep is work putting these line on True | 17:17 |
lseki | josephillips: so if you don't forget turning the VM off, nothing bad should happen | 17:17 |
josephillips | yep | 17:17 |
josephillips | i will coordinate with cloud operators to perform this action and return back to False | 17:17 |
josephillips | thanks | 17:17 |
lseki | thank you josephillips, it's very good to know how this issue is impacting the customers | 17:19 |
josephillips | btw | 17:19 |
josephillips | can you guys review this | 17:19 |
josephillips | https://review.opendev.org/#/c/728215/ | 17:19 |
josephillips | is already on master ussuri | 17:20 |
josephillips | just need to be reviewed in train to close this bug | 17:21 |
*** e0ne_ has joined #openstack-cinder | 17:29 | |
*** e0ne has quit IRC | 17:29 | |
*** rmcall has quit IRC | 17:32 | |
*** jv|afk has quit IRC | 17:42 | |
*** e0ne_ has quit IRC | 17:44 | |
openstackgerrit | Sofia Enriquez proposed openstack/cinder master: Enable sparse image conversion https://review.opendev.org/681580 | 17:46 |
*** m75abrams has joined #openstack-cinder | 17:47 | |
openstackgerrit | Sofia Enriquez proposed openstack/cinder master: Enable sparse image conversion https://review.opendev.org/681580 | 17:59 |
*** baba-tunde has quit IRC | 18:28 | |
enriquetaso | Hello rosmaita, and cinder team: we need to merge ASAP https://review.opendev.org/#/c/732506/2 in order to merge the NFS encryption patch later | 18:34 |
rosmaita | enriquetaso: ack | 18:35 |
*** boxiang_ has joined #openstack-cinder | 18:45 | |
*** boxiang has quit IRC | 18:48 | |
*** jmlowe has quit IRC | 18:51 | |
*** jv|afk has joined #openstack-cinder | 19:09 | |
enriquetaso | thanks!! | 19:11 |
*** vishalmanchanda has quit IRC | 19:13 | |
*** TobbeCN has quit IRC | 19:24 | |
smcginnis | rosmaita: RSD appears to be testing an internal detail of os-brick. Looking at what needs to be done there to get the os-brick requirements update out. | 19:31 |
smcginnis | Luckily that just impacts master, which I think is less critical in this case than the stable branches. | 19:31 |
*** martinkennelly has joined #openstack-cinder | 19:36 | |
rosmaita | smcginnis: agree, thanks for looking into this | 19:41 |
rosmaita | smcginnis: zuul is estimating 1 hour 20 min before the u-c change will merge, i'll update the ussuri patch with the new requirements as soon as that lands | 19:43 |
*** martinkennelly has quit IRC | 19:44 | |
smcginnis | ++ | 19:45 |
*** jmlowe has joined #openstack-cinder | 19:52 | |
openstackgerrit | Arthur Nascimento Santos proposed openstack/cinder master: NetApp ONTAP: Fix extend volume for iSCSI/FCP https://review.opendev.org/729782 | 20:11 |
*** priteau has quit IRC | 20:12 | |
*** prometheanfire has quit IRC | 20:16 | |
*** prometheanfire has joined #openstack-cinder | 20:16 | |
*** smcginnis has quit IRC | 20:18 | |
*** smcginnis has joined #openstack-cinder | 20:19 | |
*** ChanServ sets mode: +o smcginnis | 20:19 | |
*** e0ne has joined #openstack-cinder | 20:27 | |
openstackgerrit | Sean McGinnis proposed openstack/os-brick master: Revert "Switch from retrying to tenacity" https://review.opendev.org/733396 | 20:28 |
smcginnis | rosmaita: I just gotta say, complete and utter BS ^ | 20:28 |
smcginnis | rosmaita: So we will need to do one more release off of master and use that as the update. | 20:29 |
smcginnis | Any maybe blacklist that version of os-brick. | 20:29 |
rosmaita | ok | 20:30 |
*** jv|afk has quit IRC | 20:38 | |
*** jv|afk has joined #openstack-cinder | 20:50 | |
smcginnis | rosmaita: Hmm, may be on to something. | 21:00 |
*** enriquetaso has quit IRC | 21:00 | |
*** e0ne has quit IRC | 21:18 | |
smcginnis | Winner winner chicken dinner. | 21:25 |
rosmaita | good job! just saw your comment on https://github.com/jd/tenacity/issues/25 | 21:28 |
smcginnis | Totally missed that they had added it. Based on the conversations on that bug, didn't sound like they were going to budge on providing a way to do it. | 21:33 |
smcginnis | Simplifies a lot of things. I can update os-brick now too, but that doesn't need to be done until later. Just a cinder patch to merge, then I think we can get the requirements update through.. | 21:33 |
smcginnis | Just running some final tests locally. | 21:33 |
openstackgerrit | Brian Rosmaita proposed openstack/cinder stable/ussuri: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733106 | 21:41 |
*** boxiang_ has quit IRC | 21:41 | |
*** boxiang_ has joined #openstack-cinder | 21:42 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Improve tenacity retry sleep mocking https://review.opendev.org/733412 | 21:42 |
smcginnis | rosmaita: ^^ | 21:43 |
smcginnis | jungleboyj, eharney, others: This is blocking us from updating the os-brick requirement for master that includes the security fix we had discussed. ^ | 21:43 |
rosmaita | smcginnis: will this allow https://review.opendev.org/#/c/733286/1 to pass? | 21:44 |
smcginnis | I believe so. I'm pretty sure that job checks out cinder master and runs tests using the new updates requirement. | 21:45 |
rosmaita | ok, cool. | 21:45 |
smcginnis | So once we land this change, we should be able to just recheck that patch and it should pick up the fixes that should allow it to pass. | 21:45 |
smcginnis | Which is nice - no need to quickly do a new release. | 21:45 |
rosmaita | very nice | 21:45 |
smcginnis | Yep, confirmed the job checks out cinder master: https://zuul.opendev.org/t/openstack/build/1887edc8ab1348d587eab9750d18d176/log/job-output.txt#269 | 21:46 |
rosmaita | \o/ | 21:48 |
rosmaita | smcginnis: like you don't have enough to do, care to look at https://review.opendev.org/#/c/733258/ ? | 21:50 |
rosmaita | (brick releases for stein and train) | 21:50 |
*** spatel has joined #openstack-cinder | 22:35 | |
*** spatel has quit IRC | 22:40 | |
*** jv|afk has quit IRC | 22:41 | |
*** m75abrams has quit IRC | 22:43 | |
openstackgerrit | Brian Rosmaita proposed openstack/cinder stable/queens: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733110 | 22:55 |
openstackgerrit | Brian Rosmaita proposed openstack/cinder stable/rocky: Remove VxFlex OS credentials from connection_properties https://review.opendev.org/733109 | 22:58 |
*** jv|afk has joined #openstack-cinder | 22:58 | |
*** rcernin has joined #openstack-cinder | 23:14 | |
*** spatel has joined #openstack-cinder | 23:18 | |
*** andrebeltrami has quit IRC | 23:37 | |
*** sorrison has quit IRC | 23:39 | |
*** tosky has quit IRC | 23:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!