Sheepdog: Implement abstract methods of manage volume #77
Conversation
We add the following SheepdogDriver method. - manage_existing - manage_existing_get_size
cinder/volume/drivers/sheepdog.py
Outdated
There was a problem hiding this comment.
typo: Failed to connect to sheep daemon
|
unmanageメソッドの実装が漏れている。追加します。 unmanage時にはvolume-XXXのVDI名競合の可能性を排除するため、unmanage済みであることが分かるVDI名にrename(snapshot&clone)したいと思います。
どのフォーマットを選択するかは悩ましいところですが、もし元々の「volume-XXX」でgrepした際に対象となる様に3.の選択肢は除外。 |
cinder/volume/drivers/sheepdog.py
Outdated
There was a problem hiding this comment.
処理完了後に該当ボリュームを削除してますが、この操作実施前に作成した既存のSheepdog Volumeのsnapshotは管理者が手動で消すという整理にということですよね。
そういう整理にするという認識が合っていれば良いです。
There was a problem hiding this comment.
スナップショットについてはcloneを実行するブロックのfinally句にてdelete_snapshotしていますので、管理者が運用で対処する必要はないと考えています。
snapshotを作成した場合はrename処理の成否に関わらず必ず削除し、rename(snap+clone)が成功した場合には元のボリュームを削除、途中で失敗した場合には元のボリュームは残す整理としています。
There was a problem hiding this comment.
前の回答が質問の意図とずれていましたので再回答。
manage対象のVDIが元々持つスナップショットについては、manage_existing及びunmanageでは関与(操作)としません。
そちらのスナップショットが不要な場合には、管理者に手動で削除して頂くという整理となります。
y-naga
changed the title
2 participants
We add the following SheepdogDriver method.