Bug #21520
openAugeas methods do not support the audit mode
Description
When trying to use an augeas method in audit mode I endup with the following error message:
Repaired previous component while in dry-run mode, this is a bug. Aborting immediately.
After looking at the method source code, it looks like it the method is not designed at all to support audit mode.
This should at least be documented and skipped in audit as it "breaks" the agent run.
Analysis¶
variable_string_from_augeas
ok, no modification
file_augeas_set
Understands audit mode when a file is given in parameter.
But when no file is given, audit mode is ignored. But actually, that method can't work in audit mode, we need to skip it with a n/a
report.
file_augeas_commands
It's a method that takes a script and get its output: flexible, but we can't assume anything about the given script.
For that file, we need to just skip with an n/a
report.
It also points to the need to perhaps have a "augeas audit only" method where we would authorize that with a "be careful, danger for you feet" message in the description.
Updated by François ARMAND 10 months ago
- Severity set to Critical - prevents main use of Rudder | no workaround | data loss | security
- Priority changed from 0 to 124
- Regression changed from Yes to No
Updated by François ARMAND 10 months ago
- Has duplicate Bug #20714: augeas methods in audit mode do enforce things added
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 6.2.17 to 997
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 997 to 6.2.18
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 6.2.18 to 6.2.19
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 6.2.19 to 6.2.20
- Priority changed from 124 to 123
Updated by François ARMAND 8 months ago
- Assignee set to Alexis Mousset
- Priority changed from 123 to 122
Updated by Vincent MEMBRÉ 8 months ago
- Target version changed from 6.2.20 to 6.2.21
- Priority changed from 122 to 121
Updated by Benoît PECCATTE about 1 month ago
- Status changed from In progress to New
- Assignee deleted (
Alexis Mousset) - Priority changed from 121 to 109
Updated by Nicolas CHARLES 19 days ago
- Target version changed from 6.2.21 to 7.2.7
- Effort required set to Medium
- Priority changed from 109 to 93
Updated by Vincent MEMBRÉ 14 days ago
- Target version changed from 7.2.7 to 7.2.8