Description of problem:
LVMS currently relies on https://github.com/openshift/lvm-operator/blob/main/internal/controllers/vgmanager/exec/exec.go#L84 to determine stdErr output, but this is not filled when using streaming output. We will need to adjust and add the error logging buffer when the close of a command fails.
Version-Release number of selected component (if applicable):
4.16 release candidate up to 4.14, below stderr is also not logged but not because of the streaming of stdout, but because we ignore it completely 4.13.z and below are still affected but might be even harder to fix
How reproducible:
100%, any failing lvm command is affected
Steps to Reproduce:
1. Configure LVMS with a deviceSelector for a broken device (e.g. a device with 0B)
Actual results:
observe the failure message is done only with an exit status but not the actual output
Expected results:
failure should indicate the stderr output in the status
Additional info:
- links to
-
RHBA-2024:138414 LVMS 4.16.3 Bug Fix Update
- mentioned on