aboutsummaryrefslogtreecommitdiffstats
path: root/node-admin
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2021-01-14 14:10:02 +0100
committerMartin Polden <mpolden@mpolden.no>2021-01-14 14:36:31 +0100
commit764b7125df272d2c5e93fe7a06e2d34446f39881 (patch)
tree5355298427eb764e0ae526a5e8ebe4d520abbfa6 /node-admin
parent4786e790d95bf80b96d8442ef4949235cd5b7f2a (diff)
Skip --quiet with Yum 4
Diffstat (limited to 'node-admin')
-rw-r--r--node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumCommand.java11
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumTest.java2
2 files changed, 11 insertions, 2 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumCommand.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumCommand.java
index 3ff960d88e2..9fba22a239f 100644
--- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumCommand.java
+++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumCommand.java
@@ -148,9 +148,18 @@ public abstract class YumCommand<T extends YumCommand<T>> {
Version yumVersion = version(context);
String targetVersionLockName = yumPackage.toVersionLockName(yumVersion);
+ List<String> command = new ArrayList<>(4);
+ command.add("yum");
+ // Using --quiet on Yum 4 always results in an empty list, even if locks exist...
+ if (yumVersion.getMajor() < 4) {
+ command.add("--quiet");
+ }
+ command.add("versionlock");
+ command.add("list");
+
boolean alreadyLocked = terminal
.newCommandLine(context)
- .add("yum", "--quiet", "versionlock", "list")
+ .add(command)
.executeSilently()
.getOutputLinesStream()
.map(YumPackageName::parseString)
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumTest.java
index a66fc0cceab..3ec2bc9c570 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumTest.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/YumTest.java
@@ -192,7 +192,7 @@ public class YumTest {
@Test
public void testWithVersionLockYum4() {
mockYumVersion(YumVersion.rhel8);
- terminal.expectCommand("yum --quiet versionlock list 2>&1",
+ terminal.expectCommand("yum versionlock list 2>&1",
0,
"Last metadata expiration check: 0:51:26 ago on Thu 14 Jan 2021 09:39:24 AM UTC.\n");
terminal.expectCommand("yum versionlock add --assumeyes \"openssh-0:8.0p1-4.el8_1.*\" 2>&1");