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

Added a policy for members of node clusters #84

Merged
merged 1 commit into from Dec 6, 2016
Merged

Added a policy for members of node clusters #84

merged 1 commit into from Dec 6, 2016

Conversation

ppetit
Copy link
Contributor

@ppetit ppetit commented Dec 2, 2016

Added new policy for cluster of nodes.

@@ -375,6 +375,39 @@ aggregator:
relational_operator: '=='
threshold: 100
- status: unknown
# A policy that is typically used for storage or compute clusters
Copy link
Contributor

@elemoine elemoine Dec 6, 2016

Choose a reason for hiding this comment

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

Indent

arguments: [ okay ]
relational_operator: '<='
threshold: 50
function: percent
Copy link
Contributor

@elemoine elemoine Dec 6, 2016

Choose a reason for hiding this comment

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

Note: if more than 50% of members are down or critical then it is not possible to also have more than 50% of members in the okay state, so I think the second rule is not necessary.

Copy link
Contributor Author

@ppetit ppetit Dec 6, 2016

Choose a reason for hiding this comment

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

yes that makes sense. I'll fix it.

@elemoine elemoine merged commit e7f15b3 into salt-formulas:master Dec 6, 2016
1 check passed
@ppetit ppetit deleted the policy branch Dec 6, 2016
openstack-gerrit pushed a commit to openstack-archive/salt-formula-nova that referenced this issue Dec 8, 2016
Depends on salt-formulas/salt-formula-heka#84

Change-Id: I0df8c9cdea44232db7835db78521046bde4fee12
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.

None yet

3 participants