Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run command to restart HA on a cluster #276

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

gungazoo
Copy link
Contributor

This is for the ADO item: Product Backlog Item 26090017: Some HA Alerts seem to require clearing by Reconfigure for HA

The Cx will input the cluster and this will disable and then re-enable HA to clear all of the alerts.

@gungazoo gungazoo requested review from Zsoldier and et1975 August 15, 2024 21:02
throw "Cluster $cluster_name not found."
}

$ha_status = ""
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we evaluate whether HA is enabled first to ascertain what action should be taken? As in, if previous run command failed for whatever reason, couldn't it be reasoned that our default action should be to turn on HA if we see that it is disabled?

Also, if we enable this, we should have monitoring in place to catch any SDDC's that have HA disabled for an extended period of time. If nothing, but to have a point of reference should customer suffer from an actual HA event, but were disabled when such event occurred.

Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have this as an internal geneva action?

Copy link
Member

Choose a reason for hiding this comment

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

This commandlet could be exposed as a Geneva Action if needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@et1975 I've been asking if Geneva and Run commands could share code but hadn't found a way. How can I do that?

Copy link
Member

Choose a reason for hiding this comment

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

@gungazoo Let's take that to Teams, this is a public repo.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Zsoldier -- Since it is the customer cycling it, I would say continue if HA is disabled. We should have a monitor if HA is disabled for a period of time.

Copy link
Contributor

Choose a reason for hiding this comment

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

As long as we have a monitor prior to this, I'm ok with proceeding.

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.

3 participants