Project

General

Profile

Actions

Bug #21520

open

Augeas methods do not support the audit mode

Added by Félix DALLIDET 4 months ago. Updated about 2 months ago.

Status:
In progress
Priority:
N/A
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:
Priority:
121
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 #1

Updated by François ARMAND 4 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
Actions #2

Updated by François ARMAND 4 months ago

  • Has duplicate Bug #20714: augeas methods in audit mode do enforce things added
Actions #3

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 6.2.17 to 997
Actions #4

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 997 to 6.2.18
Actions #5

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 6.2.18 to 6.2.19
Actions #6

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 6.2.19 to 6.2.20
  • Priority changed from 124 to 123
Actions #7

Updated by François ARMAND 2 months ago

  • Assignee set to Alexis Mousset
  • Priority changed from 123 to 122
Actions #8

Updated by Alexis Mousset 2 months ago

  • Status changed from New to In progress
Actions #9

Updated by Vincent MEMBRÉ about 2 months ago

  • Target version changed from 6.2.20 to 6.2.21
  • Priority changed from 122 to 121
Actions #10

Updated by François ARMAND about 2 months ago

  • Description updated (diff)
Actions

Also available in: Atom PDF