Actions
Bug #22060
openfile_augeas_commands method fails to detect failures and changes
Pull Request:
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.
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 7.2.2 to 7.2.3
Updated by Vincent MEMBRÉ about 2 years ago
- Target version changed from 7.2.3 to 7.2.4
Updated by Vincent MEMBRÉ almost 2 years ago
- Target version changed from 7.2.4 to 7.2.5
Updated by Alexis Mousset almost 2 years 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
Updated by Alexis Mousset almost 2 years ago
- Has duplicate Bug #22059: file_augeas_commands method fails to detect failures and changes added
Updated by Vincent MEMBRÉ almost 2 years ago
- Target version changed from 7.2.5 to 7.2.6
- Priority changed from 54 to 53
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.6 to 7.2.7
- Priority changed from 53 to 52
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 52 to 51
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.9 to 7.2.10
- Priority changed from 51 to 50
Updated by Alexis Mousset over 1 year ago
- Target version changed from 7.2.10 to 7.2.11
- Priority changed from 50 to 49
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.11 to 1046
- Priority changed from 49 to 48
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
- Priority changed from 48 to 47
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.3.9 to 7.3.10
Updated by Vincent MEMBRÉ about 1 year ago
- Target version changed from 7.3.10 to 7.3.11
Updated by Vincent MEMBRÉ 11 months ago
- Target version changed from 7.3.11 to 7.3.12
Updated by Vincent MEMBRÉ 10 months ago
- Target version changed from 7.3.12 to 7.3.13
Updated by Vincent MEMBRÉ 10 months ago
- Target version changed from 7.3.13 to 7.3.14
Updated by Vincent MEMBRÉ 8 months ago
- Target version changed from 7.3.14 to 7.3.15
Updated by Vincent MEMBRÉ 7 months ago
- Target version changed from 7.3.15 to 7.3.16
Updated by Vincent MEMBRÉ 6 months ago
- Target version changed from 7.3.16 to 7.3.17
Actions