Project

General

Profile

Actions

Bug #22060

open

file_augeas_commands method fails to detect failures and changes

Added by Félix DALLIDET over 1 year ago. Updated 2 days ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Generic methods
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
I dislike using that feature
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Priority:
47
Name check:
To do
Fix check:
To do
Regression:
No

Description

The method uses the execresult keyword to call augtool and captures its output to the given variable name.
But the method always return a success even when executing augeas commands with syntax errors.
execresult is not able to catch the exit code, we should at least properly document that the method only reports success and try to fix it in 7.3 when we should be able to
better interact with third-party tools using custom resources.

To reproduce, just try to execute the method with:

set /files/etc/network/interfaces/autooo[child::1 = 'vmbr0']/1 vmbr0
save

See how the report is still a success.


Related issues 1 (0 open1 closed)

Has duplicate Rudder - Bug #22059: file_augeas_commands method fails to detect failures and changesRejectedActions
Actions #1

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.2 to 7.2.3
Actions #2

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.3 to 7.2.4
Actions #3

Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 7.2.4 to 7.2.5
Actions #4

Updated by Alexis Mousset about 1 year ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • UX impact set to I dislike using that feature
  • User visibility set to Operational - other Techniques | Rudder settings | Plugins
  • Priority changed from 0 to 54
Actions #5

Updated by Alexis Mousset about 1 year ago

  • Has duplicate Bug #22059: file_augeas_commands method fails to detect failures and changes added
Actions #6

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 7.2.5 to 7.2.6
  • Priority changed from 54 to 53
Actions #7

Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 7.2.6 to 7.2.7
  • Priority changed from 53 to 52
Actions #8

Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 7.2.7 to 7.2.8
Actions #9

Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 7.2.8 to 7.2.9
  • Priority changed from 52 to 51
Actions #10

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 7.2.9 to 7.2.10
  • Priority changed from 51 to 50
Actions #11

Updated by Alexis Mousset 9 months ago

  • Target version changed from 7.2.10 to 7.2.11
  • Priority changed from 50 to 49
Actions #12

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 7.2.11 to 1046
  • Priority changed from 49 to 48
Actions #13

Updated by Alexis Mousset 7 months ago

  • Target version changed from 1046 to 7.3.8
Actions #14

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 7.3.8 to 7.3.9
  • Priority changed from 48 to 47
Actions #15

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 7.3.9 to 7.3.10
Actions #16

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 7.3.10 to 7.3.11
Actions #17

Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 7.3.11 to 7.3.12
Actions #18

Updated by Vincent MEMBRÉ about 2 months ago

  • Target version changed from 7.3.12 to 7.3.13
Actions #19

Updated by Vincent MEMBRÉ about 2 months ago

  • Target version changed from 7.3.13 to 7.3.14
Actions #20

Updated by Vincent MEMBRÉ 2 days ago

  • Target version changed from 7.3.14 to 7.3.15
Actions

Also available in: Atom PDF