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

Primaries and IPM #9

Merged
merged 2 commits into from
Sep 30, 2019
Merged

Primaries and IPM #9

merged 2 commits into from
Sep 30, 2019

Conversation

amy-hashi
Copy link
Contributor

When internal production mode is set, the number of primaries needs to be hard set at 3, due to scaling issues with ceph. Otherwise, the value of the count variable will be used.

When internal production mode is set, the number of primaries needs to be hard set at 3, due to scaling issues with ceph. Otherwise, the value of the count variable will be used.
primary.tf Show resolved Hide resolved
Inserted a comment explaining the count line on the primary instance resource.
Copy link
Contributor

@erindatkinson erindatkinson left a comment

Choose a reason for hiding this comment

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

LGTM 🌈

When we release this we should also add to the docs about this.

@amy-hashi amy-hashi merged commit 98a084d into master Sep 30, 2019
@amy-hashi amy-hashi deleted the primary-ipm branch September 30, 2019 17:51
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.

2 participants