fix: subscription-manager list --installed call failure

The `subscription-manager list --installed` was failing with RC 1, after printing stdout it visible that it reported that it was called incorrectly and thus printed help text. Issue was caused by the way how parameters were passed, now each is passed separately as the exec.Command expects it. https://issues.redhat.com/browse/HMS-3279 Signed-off-by: Petr Vobornik <[email protected]>
pvoborni · Dec 13, 2023 · b26c6cd · b26c6cd
1 parent 9885785
commit b26c6cd
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/hostinfo/subscription.go b/hostinfo/subscription.go
@@ -60,7 +60,7 @@ func GetSocketCount(facts SubManValues) (string, error) {
 }
 
 func GetProduct(facts SubManValues) ([]string, error) {
-	output, _ := execSubManCommand("list --installed")
+	output, _ := execSubManCommand("list", "--installed")
 	values := parseSubManOutputMultiVal(output)
 	return values.get("Product ID")
 }
@@ -104,8 +104,8 @@ func GetBillingInfo(facts SubManValues) (BillingInfo, error) {
 	return BillingInfo{}, err
 }
 
-func execSubManCommand(command string) (string, error) {
-	cmd := exec.Command("subscription-manager", command)
+func execSubManCommand(command ...string) (string, error) {
+	cmd := exec.Command("subscription-manager", command...)
 	logger.Debugf("Executing `subscription-manager %s`...\n", command)
 
 	var stdout, stderr bytes.Buffer

diff --git a/mocks/subscription-manager b/mocks/subscription-manager
@@ -215,10 +215,12 @@ hard_coded() {
       show_facts
       ;;
     list)
-      show_list_installed
-      ;;
-    "list --installed")
-      show_list_installed
+      if [ "${2}" == "--installed" ]; then
+        show_list_installed
+      else
+        echo "Unsupported command: ${command}" >&2
+        exit 1
+      fi
       ;;
     *)
       echo "Unsupported command: ${command}" >&2