Project

General

Profile

Actions

Bug #21520

open

Augeas methods do not support the audit mode

Added by Félix DALLIDET almost 2 years ago. Updated 15 days ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Generic methods
Target version:
Severity:
Critical - prevents main use of Rudder | no workaround | data loss | security
UX impact:
I hate Rudder for that
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Medium
Priority:
89
Name check:
To do
Fix check:
To do
Regression:
No

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.


Related issues 1 (0 open1 closed)

Has duplicate Rudder - Bug #20714: augeas methods in audit mode do enforce thingsRejectedActions
Actions

Also available in: Atom PDF