Skip to content

Commit

Permalink
warn user when there are too many he hosts
Browse files Browse the repository at this point in the history
Previous fix [1] used the wrong method to calculate the number of HE hosts

[1] https://gerrit.ovirt.org/c/ovirt-engine/+/118221/

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=2027087
  • Loading branch information
dangel101 authored and mwperina committed Jun 29, 2022
1 parent 3f13de6 commit 018b4e0
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ private void runAnsibleHostDeployPlaybook() {
hostedEngineAction =
getParameters().getHostedEngineDeployConfiguration().getDeployAction().name().toLowerCase();
if (hostedEngineAction.equals("deploy")) {
long numHEHosts = vdsDao.getAll().stream().filter(VDS::isHostedEngineHost).count();
long numHEHosts = vdsDao.getAll().stream().filter(VDS::isHostedEngineDeployed).count();
int maxHEHosts = EngineLocalConfig.getInstance().getInteger("MAX_RECOMMENDED_HE_HOSTS");
if (numHEHosts >= maxHEHosts) {
log.warn(
Expand Down

0 comments on commit 018b4e0

Please sign in to comment.