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 over 2 years 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 over 2 years ago
- Has duplicate Bug #20714: augeas methods in audit mode do enforce things added
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 6.2.17 to 997
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 997 to 6.2.18
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 6.2.18 to 6.2.19
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 6.2.19 to 6.2.20
- Priority changed from 124 to 123
Updated by François ARMAND about 2 years ago
- Assignee set to Alexis Mousset
- Priority changed from 123 to 122
Updated by Alexis Mousset about 2 years ago
- Status changed from New to In progress
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 6.2.20 to old 6.2 issues to relocate
- Priority changed from 122 to 121
Updated by Benoît PECCATTE over 1 year ago
- Status changed from In progress to New
- Assignee deleted (
Alexis Mousset) - Priority changed from 121 to 109
Updated by Nicolas CHARLES over 1 year ago
- Target version changed from old 6.2 issues to relocate to 7.2.7
- Effort required set to Medium
- Priority changed from 109 to 93
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.7 to 7.2.8
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.8 to 7.2.9
- Priority changed from 93 to 91
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.9 to 7.2.10
- Priority changed from 91 to 89
Updated by Alexis Mousset over 1 year ago
- Target version changed from 7.2.10 to 7.2.11
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.2.11 to 1046
Updated by Alexis Mousset about 1 year ago
- Target version changed from 1046 to 7.3.8
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.3.8 to 7.3.9
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.3.9 to 7.3.10
Updated by Vincent MEMBRÉ 12 months ago
- Target version changed from 7.3.10 to 7.3.11
Updated by Vincent MEMBRÉ 10 months ago
- Target version changed from 7.3.11 to 7.3.12
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 7.3.12 to 7.3.13
Updated by Vincent MEMBRÉ 9 months ago
- Target version changed from 7.3.13 to 7.3.14
Updated by Vincent MEMBRÉ 7 months ago
- Target version changed from 7.3.14 to 7.3.15
Updated by Vincent MEMBRÉ 6 months ago
- Target version changed from 7.3.15 to 7.3.16
Updated by Vincent MEMBRÉ 5 months ago
- Target version changed from 7.3.16 to 7.3.17