Skip to content

Update lhv2 volume expansion doc#1048

Open
brandboat wants to merge 1 commit into
harvester:mainfrom
brandboat:HARV-10597
Open

Update lhv2 volume expansion doc#1048
brandboat wants to merge 1 commit into
harvester:mainfrom
brandboat:HARV-10597

Conversation

@brandboat
Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Harvester’s “Edit a Volume” documentation to refine guidance around online volume expansion with Longhorn and to improve readability across the unversioned and versioned docs.

Changes:

  • Simplifies the Longhorn online expansion note to focus on the V1 Data Engine.
  • Adds an informational note advising users to resolve Longhorn UI volume warnings before expanding.
  • Fixes a malformed/indented numbered list item in the StorageClass restore steps.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
docs/volume/edit-volume.md Updates Longhorn online expansion guidance and adds a warning note; fixes list formatting.
versioned_docs/version-v1.7/volume/edit-volume.md Mirrors the Longhorn guidance update and list formatting fix for v1.7 docs.
versioned_docs/version-v1.8/volume/edit-volume.md Mirrors the Longhorn guidance update and list formatting fix for v1.8 docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/volume/edit-volume.md
Comment thread versioned_docs/version-v1.7/volume/edit-volume.md
Comment thread versioned_docs/version-v1.8/volume/edit-volume.md
Signed-off-by: Cooper Tseng <cooper.tseng@suse.com>
@github-actions
Copy link
Copy Markdown

Name Link
🔨 Latest commit 89f3f71
😎 Deploy Preview https://6a0ebde959a4ff39b8d4abb3--harvester-preview.netlify.app

Copy link
Copy Markdown
Contributor

@jillian-maroket jillian-maroket left a comment

Choose a reason for hiding this comment

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

Review done

Comment on lines +28 to +34
- **Longhorn**: Longhorn volumes support online volume expansion.

:::info

Before expanding a volume, check the volume status on the Volumes UI and resolve any warnings such as `Precheck new replica failed: disks are unavailable`. Volume expansion might not work as expected while these warnings are present.

:::
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.

Suggested change
- **Longhorn**: Longhorn volumes support online volume expansion.
:::info
Before expanding a volume, check the volume status on the Volumes UI and resolve any warnings such as `Precheck new replica failed: disks are unavailable`. Volume expansion might not work as expected while these warnings are present.
:::
- **Longhorn**: Both the V1 and V2 Data Engines support online volume expansion.
:::info important
Before expanding a volume, check its status on the **Volumes** screen of the Harvester UI and resolve any warnings, such as `Precheck new replica failed: disks are unavailable`. Volume expansion may fail if these warnings are not addressed.
:::

Comment on lines +28 to +34
- **Longhorn**: Longhorn volumes support online volume expansion.

:::info

Before expanding a volume, check the volume status on the Volumes UI and resolve any warnings such as `Precheck new replica failed: disks are unavailable`. Volume expansion might not work as expected while these warnings are present.

:::
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.

Suggested change
- **Longhorn**: Longhorn volumes support online volume expansion.
:::info
Before expanding a volume, check the volume status on the Volumes UI and resolve any warnings such as `Precheck new replica failed: disks are unavailable`. Volume expansion might not work as expected while these warnings are present.
:::
- **Longhorn**: Both the V1 and V2 Data Engines support online volume expansion.
:::info important
Before expanding a volume, check its status on the **Volumes** screen of the Harvester UI and resolve any warnings, such as `Precheck new replica failed: disks are unavailable`. Volume expansion may fail if these warnings are not addressed.
:::

Comment on lines +28 to +34
- **Longhorn**: Longhorn volumes support online volume expansion.

:::info

Before expanding a volume, check the volume status on the Volumes UI and resolve any warnings such as `Precheck new replica failed: disks are unavailable`. Volume expansion might not work as expected while these warnings are present.

:::
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.

Suggested change
- **Longhorn**: Longhorn volumes support online volume expansion.
:::info
Before expanding a volume, check the volume status on the Volumes UI and resolve any warnings such as `Precheck new replica failed: disks are unavailable`. Volume expansion might not work as expected while these warnings are present.
:::
- **Longhorn**: Both the V1 and V2 Data Engines support online volume expansion.
:::info important
Before expanding a volume, check its status on the **Volumes** screen of the Harvester UI and resolve any warnings, such as `Precheck new replica failed: disks are unavailable`. Volume expansion may fail if these warnings are not addressed.
:::

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.

3 participants