search

rackspace / gophercloud

A Go SDK for OpenStack. IN FEATURE FREEZE. See Issue #592
http://gophercloud.io
Other
456 stars 181 forks source link

fix cinder detach parameter #596

Closed bergwolf closed 8 years ago

bergwolf commented 8 years ago

It is "os-froce_detach" per http://developer.openstack.org/api-ref-blockstorage-v2.html

Signed-off-by: Peng Tao bergwolf@gmail.com

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 81.084% when pulling 574d1768639e35fa3060e339ac4df8b2d8dc969a on bergwolf:detach into 985a863d6dd5f928b485dbc8ef440813aafa39ad on rackspace:master.

jrperritt commented 8 years ago

detach is correct, as well. They are 2 different actions.

bergwolf commented 8 years ago

hmm, just checked cinder code, there is indeed "os-detach" action.

Too bad only "os-force_detach" is documented in the cinder api reference page. Sorry for the noise.

jrperritt commented 8 years ago

That's quite alright; it's reasonable to expect documentation. After working with OpenStack for a few years now, though, I've learned to go straight to the source like you eventually did.