Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-28727

LVMS ignores stderr from command output when running host commands

XMLWordPrintable

    • Quality / Stability / Reliability
    • False
    • Hide

      None

      Show
      None
    • 5
    • Low
    • No
    • None
    • OCPEDGE Sprint 249, OCPEDGE Sprint 250
    • 2
    • Done
    • Bug Fix
    • Hide
      Previously, LVMS provided limited error information when encountering issues during Node setup while interacting with lvm2, wipefs, and lsblk. This fix introduces more detailed error status messages, helping users diagnose and resolve setup issues more effectively.
      Show
      Previously, LVMS provided limited error information when encountering issues during Node setup while interacting with lvm2, wipefs, and lsblk. This fix introduces more detailed error status messages, helping users diagnose and resolve setup issues more effectively.
    • None
    • None
    • None
    • None

      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:

      
          

              rh-ee-jmoller Jakob Moeller (Inactive)
              rh-ee-jmoller Jakob Moeller (Inactive)
              None
              None
              Rahul Deore Rahul Deore
              None
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: