Skip to content

Host ha and multiple management server internal load balancing - #223

Merged
andrijapanicsb merged 12 commits into
apache:4.15from
shapeblue:host_ha
Oct 4, 2021
Merged

Host ha and multiple management server internal load balancing#223
andrijapanicsb merged 12 commits into
apache:4.15from
shapeblue:host_ha

Conversation

@Spaceman1984

Copy link
Copy Markdown
Contributor

Added some documentation around Host HA and management-server internal load balancing

@Spaceman1984 Spaceman1984 changed the title Host ha an multiple management server internal load balancing Host ha and multiple management server internal load balancing Jun 22, 2021
Comment thread source/adminguide/reliability.rst
Comment thread source/adminguide/reliability.rst Outdated
Comment thread source/adminguide/reliability.rst Outdated
Comment thread source/adminguide/reliability.rst Outdated
background check interval is also sent and is dynamically reconfigured
in the background task without need to restart agents.

Note: The 'static' and 'roundrobin' algorithms, strictly checks for the

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.

can you move this algorithm details to top, where they are described ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok

Comment thread source/adminguide/reliability.rst Outdated
any disk activity on the VMs running on the specific host.

HA Resource Management Service
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

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.

resource management service is part of Host HA, I think, no need of a separate section for this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok, I'll remove the heading.

Comment thread source/adminguide/accounts.rst
Comment thread source/adminguide/reliability.rst

@sureshanaparti sureshanaparti 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.

changes LGTM

@sureshanaparti
sureshanaparti requested a review from shwstppr July 19, 2021 09:22

@shwstppr shwstppr 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.

Minor nit. Rest LGTM.
Will be good to have review from @andrijapanicsb

Comment thread source/adminguide/reliability.rst Outdated
Co-authored-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@andrijapanicsb

Copy link
Copy Markdown
Contributor

Will do after my vacation pls, yes. On my to-do list (among other doc update reviews)

@yadvr

yadvr commented Sep 22, 2021

Copy link
Copy Markdown
Member

ping @andrijapanicsb

@andrijapanicsb

Copy link
Copy Markdown
Contributor

@blueorangutan docbuild

@blueorangutan

Copy link
Copy Markdown

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

@blueorangutan

Copy link
Copy Markdown

Doc build preview: http://qa.cloudstack.cloud/docs/WIP-PROOFING/pr/223. (SL-JID 154)

@andrijapanicsb andrijapanicsb 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.

LGTM (even though most of the HA-Enabled Hosts section is a copy-paste from the CWIKI design document)

@andrijapanicsb
andrijapanicsb merged commit a6ce2d8 into apache:4.15 Oct 4, 2021
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