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

Fixes #5468: add sensible size constraint to fix partition monitoring #478

Conversation

ncharles
Copy link
Member

@ncharles ncharles commented Sep 2, 2014

No description provided.

@@ -94,5 +94,9 @@ bundle agent monitor_partition_size {

body volume freespacecheck(threshold) {
freespace => "$(threshold)";
# see https://dev.cfengine.com/issues/3016
check_foreign => "false";
Copy link
Member

Choose a reason for hiding this comment

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

check_foreign already defaults to false

Copy link
Member Author

Choose a reason for hiding this comment

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

ok

@ncharles ncharles force-pushed the bug_5468/dev/correct_partition_monitoring_on_2_11 branch from c815e54 to 5cfeb16 Compare September 5, 2014 09:33
@ncharles
Copy link
Member Author

ncharles commented Sep 5, 2014

PR updated

@@ -94,5 +94,8 @@ bundle agent monitor_partition_size {

body volume freespacecheck(threshold) {
freespace => "$(threshold)";
# see https://dev.cfengine.com/issues/3016
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't seem to be indented correctly

Copy link
Member Author

Choose a reason for hiding this comment

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

updated

@jooooooon
Copy link
Member

@ncharles please fix the indentation and self-merge!

@ncharles ncharles force-pushed the bug_5468/dev/correct_partition_monitoring_on_2_11 branch from 5cfeb16 to f7ea7a2 Compare September 5, 2014 10:35
ncharles added a commit that referenced this pull request Sep 5, 2014
…monitoring_on_2_11

Fixes #5468: add sensible size constraint to fix partition monitoring
@ncharles ncharles merged commit 97dbcf2 into Normation:branches/rudder/2.11 Sep 5, 2014
amousset pushed a commit to amousset/rudder-techniques that referenced this pull request Dec 8, 2015
…service_log

Fixes #5500: Clear error log on rudder-upgrade
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants