Bug #5468
closedWeird behaviours of partitionMonitoring technique on ZFS on 2.11
Description
CFEngine 3.6 (so rudder-agent 2.11) outputs useless and silly warnings like this on some partition sizes:
2013-06-20T17:55:32+0000 error: /main/methods/'rabbitmq'/rabbitmq/storage/'/var/lib/rabbitmq': File system '/var/lib/rabbitmq' is suspiciously small! (4116 bytes)
This was reported in CFEngine ticket https://dev.cfengine.com/issues/3016
Following IRC discussion, here's the modification to add, to make it work correctly on ZFS
diff --git a/systemSettings/misc/partitionSizeMonitoring/1.0/monitorPartitionSize.st b/systemSettings/misc/partitionSizeMonitoring/1.0/monitorPartitionSize.st index 62da9d8..d13a12c 100644 --- a/systemSettings/misc/partitionSizeMonitoring/1.0/monitorPartitionSize.st +++ b/systemSettings/misc/partitionSizeMonitoring/1.0/monitorPartitionSize.st @@ -94,5 +94,8 @@ bundle agent monitor_partition_size { body volume freespacecheck(threshold) { freespace => "$(threshold)"; + check_foreign => "false"; + sensible_size => "0"; + sensible_count => "0"; }
Updated by Nicolas CHARLES over 9 years ago
- Status changed from New to In progress
- Assignee set to Nicolas CHARLES
- Target version set to 2.11.3
Thank you Olivier !
Updated by Nicolas CHARLES over 9 years ago
see https://dev.cfengine.com/issues/3016
i target this bug to 2.11 as it has been reported only in 2.11 (could reproduce in 2.10)
Updated by Nicolas CHARLES over 9 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Nicolas CHARLES to Jonathan CLARKE
- Pull Request set to https://github.com/Normation/rudder-techniques/pull/478
Updated by François ARMAND over 9 years ago
Nicolas, did you mean "i target this bug to 2.11 as it has been reported only in 2.11 (could NOT reproduce in 2.10)" ? Else, it doesn't make sens to target in 2.11.
Updated by Nicolas CHARLES over 9 years ago
oh yes, i meant could not reproduce in 2.10
bad case of sloppy finger
Updated by Jonathan CLARKE over 9 years ago
- Description updated (diff)
OK, looks good to me (except for some indentation). However the "check_foreign" already defaults to false, so no need to add that.
Updated by Nicolas CHARLES over 9 years ago
- Status changed from Pending technical review to Pending release
- % Done changed from 0 to 100
Applied in changeset policy-templates:commit:f7ea7a23584680315d726add4fc40ea80227af5f.
Updated by Nicolas CHARLES over 9 years ago
Applied in changeset policy-templates:commit:97dbcf29936ec428e00aee896b0a7fafe7e3d81f.
Updated by Vincent MEMBRÉ over 9 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 2.11.3, which was released on the 25/09/2014.
- Announcement
- Changelog
- Download information: https://www.rudder-project.org/site/get-rudder/downloads/