Merge pull request #12825 from vespa-engine/hakonhall/document-when---enablerepo-is-necessary

Document when --enablerepo is necessary

1 files changed, 3 insertions, 0 deletions

diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/Yum.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/Yum.java
index 53f3a6d4900..4fde82f6fd6 100644
--- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/Yum.java
+++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/task/util/yum/Yum.java
@@ -102,6 +102,9 @@ public class Yum {
 
         if (!alreadyLocked) {
             CommandLine commandLine = terminal.newCommandLine(context).add("yum", "versionlock", "add");
+            // If the targetVersionLockName refers to a package in a by-default-disabled repo,
+            // we must enable the repo unless targetVersionLockName is already installed.
+            // The other versionlock commands (list, delete) does not require --enablerepo.
             for (String repo : repos) commandLine.add("--enablerepo=" + repo);
             commandLine.add(targetVersionLockName).execute();
             modified = true;