Skip to content

Primary storage LUN capacity correction#599

Merged
DaanHoogland merged 2 commits intoapache:4.22from
shapeblue:primary-storage-lun-capacity-update
Nov 26, 2025
Merged

Primary storage LUN capacity correction#599
DaanHoogland merged 2 commits intoapache:4.22from
shapeblue:primary-storage-lun-capacity-update

Conversation

@sureshanaparti
Copy link
Contributor

@sureshanaparti sureshanaparti commented Nov 13, 2025 •
edited by github-actions bot
Loading

@sureshanaparti sureshanaparti changed the base branch from main to 4.22 November 13, 2025 07:48
Copilot AI reviewed Nov 13, 2025
View reviewed changes
Copy link

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

This PR updates the documentation regarding primary storage capacity recommendations by replacing a specific size limit with more general guidance.

  • Removes the specific "6 TB" capacity limit for primary storage mountpoints/LUNs
  • Replaces it with the more general phrase "limited to a manageable capacity"

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

@sureshanaparti
Copy link
Contributor Author

sureshanaparti commented Nov 13, 2025

@blueorangutan docbuild

@blueorangutan
Copy link

blueorangutan commented Nov 13, 2025

@sureshanaparti a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan
Copy link

blueorangutan commented Nov 13, 2025

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/599. (QA-JID 460)

borisstoyanov
borisstoyanov approved these changes Nov 25, 2025
View reviewed changes
Copy link
Contributor

@borisstoyanov borisstoyanov left a comment

Choose a reason for hiding this comment

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

LGTM

@DaanHoogland DaanHoogland merged commit 76d9008 into apache:4.22 Nov 26, 2025
1 check passed
@DaanHoogland DaanHoogland deleted the primary-storage-lun-capacity-update branch November 26, 2025 12:09
@alexandremattioli
Copy link

alexandremattioli commented Dec 3, 2025

@sureshanaparti @DaanHoogland
" Primary storage mountpoints or LUNs should be limited to a manageable
capacity as the CloudStack does not impose any limit on them."

should be

" Primary storage mountpoints or LUNs should be limited to a manageable
capacity as CloudStack does not impose any limits on them."

@harikrishna-patnala harikrishna-patnala added this to the 4.22.1 milestone Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@DaanHoogland DaanHoogland DaanHoogland approved these changes

@borisstoyanov borisstoyanov borisstoyanov approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

4.22.1

Development

Successfully merging this pull request may close these issues.

6 participants

@sureshanaparti @blueorangutan @alexandremattioli @DaanHoogland @borisstoyanov @harikrishna-patnala