Skip to content

Update README.md - #566

Open
shimada671 wants to merge 1 commit into
cloudnativedaysjp:mainfrom
shimada671:fix/kubevirt-readme
Open

Update README.md#566
shimada671 wants to merge 1 commit into
cloudnativedaysjp:mainfrom
shimada671:fix/kubevirt-readme

Conversation

@shimada671

Copy link
Copy Markdown

chapter_kubevirt README の以下を修正しました。

  • DataVolume の説明を補足

StorageClass が WaitForFirstConsumer の環境では、
VM を起動しないと DataVolume が Succeeded にならず
詰まったため、その補足説明を README に追記しました。

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the KubeVirt documentation to include the virtctl start command and explains the dependency on starting the VM when using WaitForFirstConsumer storage classes. The review feedback suggests clarifying that volume provisioning depends on Pod scheduling and warns that this addition may cause a conflict with a duplicate start command in the subsequent live migration section.

Comment on lines +299 to +305
# 仮想マシンを起動します。
virtctl start vm-with-datavolume

# StorageClass が WaitForFirstConsumer の場合、
# 仮想マシンを起動しないと DataVolume の PHASE は
# WaitForFirstConsumer から進みません。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

StorageClass の volumeBindingModeWaitForFirstConsumer に設定されている場合、PVC のプロビジョニングはそれを使用する Pod(この場合は仮想マシンの Pod)がスケジュールされるまで開始されません。この仕様により、仮想マシンを起動しない限り DataVolume のインポートも開始されないという点について、理由を添えて説明すると学習者にとってより分かりやすくなります。

また、この変更により後の「ライブマイグレーション」セクション(326行目付近)にある virtctl start コマンドが重複し、そのまま手順を進めるとエラー(VM が既に起動している旨)が発生する点に注意が必要です。必要に応じて、後のセクションの記述を調整することを検討してください。

Suggested change
# 仮想マシンを起動します。
virtctl start vm-with-datavolume
# StorageClass が WaitForFirstConsumer の場合、
# 仮想マシンを起動しないと DataVolume の PHASE は
# WaitForFirstConsumer から進みません。
# StorageClass が WaitForFirstConsumer の場合、仮想マシンを起動して Pod を
# スケジュールさせない限り、ボリュームのプロビジョニングおよびインポートが開始されません。
virtctl start vm-with-datavolume

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant