Skip to content

Conversation

@sudo87
Copy link
Contributor

@sudo87 sudo87 commented Nov 10, 2025

This PR adds documentation related to Deployment Planners in CloudStack.


📚 Documentation preview 📚: https://cloudstack-documentation--594.org.readthedocs.build/en/594/

sudo87 and others added 6 commits November 10, 2025 15:11
Adding a section on deployment planners to include detailed descriptions of available planners and their behaviors.
Improving host.capacityType.to.order.clusters to include Deployment Planners
Adding deployment planner diagram
adding deployment planner diagram
@sudo87
Copy link
Contributor Author

sudo87 commented Nov 10, 2025

@blueoragnutan docbuild

@sudo87
Copy link
Contributor Author

sudo87 commented Nov 10, 2025

@harikrishna-patnala please review this pr

@prashanthr2
Copy link
Contributor

prashanthr2 commented Nov 11, 2025

I would suggest replacing.

Deployment planners determine how and where instances are placed across clusters within a zone
with
Deployment planners determine how instances are allocated within a zone based on configuration settings.

As deployment planner itself can work at the pod level and also the actual deployment is on particular host. I think it is better to just say zone.

Except that everything else lgtm

@sudo87 sudo87 force-pushed the deploymentPlanners branch from cecc1d2 to eccf2fd Compare November 12, 2025 04:54
@rajujith
Copy link
Contributor

@prashanthr2 based on my discussion with @sudo87 offline, we decided the retain the original statement as it's the correct one as per the functionality.

Update descriptions for VM volume allocation algorithms
@rajujith
Copy link
Contributor

@blueoragnutan docbuild

@vishesh92 vishesh92 requested a review from Copilot November 14, 2025 05:38
Copilot finished reviewing on behalf of vishesh92 November 14, 2025 05:40
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 adds comprehensive documentation for Deployment Planners in CloudStack, explaining how instances are placed across clusters within a zone.

Key changes:

  • New documentation file explaining deployment planner concepts and available planners (FirstFitPlanner, UserDispersingPlanner, UserConcentratedPodPlanner)
  • Updated VM volume allocators documentation to reference deployment planners
  • Added visual diagram illustrating the deployment planner workflow

Reviewed Changes

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

File Description
source/adminguide/deployment_planners.rst New documentation file providing detailed explanations of deployment planners, their behaviors, and configuration options
source/adminguide/index.rst Added deployment_planners to the table of contents under "Managing VM and Volume Allocation" section
source/adminguide/vm_volume_allocators.rst Updated to clarify when host.capacityType.to.order.clusters applies and added reference to deployment planners
source/_static/images/deployment-planner-diagram.png New diagram visualizing the deployment planner decision flow

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

@sudo87
Copy link
Contributor Author

sudo87 commented Nov 14, 2025

@blueoragnutan docbuild

Copy link
Contributor

@sureshanaparti sureshanaparti left a comment

Choose a reason for hiding this comment

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

lgtm

@sureshanaparti
Copy link
Contributor

@sudo87 can you target to 4.22 branch?

@sudo87 sudo87 changed the base branch from main to 4.22 November 14, 2025 08:28
@sudo87
Copy link
Contributor Author

sudo87 commented Nov 14, 2025

@sudo87 can you target to 4.22 branch?

Sure @sureshanaparti, please review and merge.

@sudo87 sudo87 force-pushed the deploymentPlanners branch from 6ee46a9 to ae721bc Compare November 18, 2025 11:40
@sudo87 sudo87 force-pushed the deploymentPlanners branch from ae721bc to 766d79f Compare November 18, 2025 11:42
Copy link
Contributor

@harikrishna-patnala harikrishna-patnala left a comment

Choose a reason for hiding this comment

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

LGTM

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.

6 participants