fix: Check for HA as part of estimating max app count. - Githubissues

cloudfoundry-incubator / kubecf

Cloud Foundry on Kubernetes

Apache License 2.0

115 stars 62 forks source link

fix: Check for HA as part of estimating max app count. #1591

Closed andreas-kupries closed 4 years ago

andreas-kupries commented 4 years ago

Description

Replaced fixed value of 1 with a ternary returning either 1 or 3 based on the HA flag.

Motivation and Context

See #1479

How Has This Been Tested?

Local minikube. Deploy only as far as needed to see the NOTES.txt in the terminal.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code has security implications.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.