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
Copy link
Copy Markdown
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.

changes LGTM

@sureshanaparti sureshanaparti requested a review from shwstppr July 19, 2021 09:22
Copy link
Copy Markdown
Contributor

@shwstppr shwstppr left a comment

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
Copy link
Copy Markdown
Member

yadvr commented Sep 22, 2021

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)

Copy link
Copy Markdown
Contributor

@andrijapanicsb andrijapanicsb left a comment

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