Copilot commented on code in PR #599:
URL:
https://github.com/apache/cloudstack-documentation/pull/599#discussion_r2522102615
##########
source/conceptsandterminology/choosing_deployment_architecture.rst:
##########
@@ -299,7 +299,7 @@ Setup Best Practices
- Use multiple clusters per pod if you need to achieve a certain switch
density.
-- Primary storage mountpoints or LUNs should not exceed 6 TB in size.
+- Primary storage mountpoints or LUNs should be limited to a manageable
capacity.
Review Comment:
The replacement of "should not exceed 6 TB in size" with "should be limited
to a manageable capacity" makes the documentation significantly less actionable
and informative. The original text provided specific, concrete guidance (a 6 TB
limit), while the new text is vague and subjective.
If the 6 TB limit is outdated or incorrect, consider replacing it with an
updated specific recommendation or explaining what factors determine
"manageable capacity" (e.g., backup/recovery time, performance considerations,
rebuild time). Vague guidance like "manageable capacity" doesn't help
administrators make informed decisions.
```suggestion
- Primary storage mountpoints or LUNs should not exceed 6 TB in size. If
your environment requires larger storage, consider the impact on
backup/recovery time, performance, and rebuild time, and consult your storage
vendor's recommendations.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
