Perform asymetric Vespa home mount for vespa/ci image

1 files changed, 22 insertions, 0 deletions

diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentContextImplTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentContextImplTest.java
index e84d8345815..250c005566b 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentContextImplTest.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/nodeagent/NodeAgentContextImplTest.java
@@ -1,10 +1,13 @@
 // Copyright 2020 Oath Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.vespa.hosted.node.admin.nodeagent;
 
+import com.yahoo.config.provision.DockerImage;
 import com.yahoo.vespa.test.file.TestFileSystem;
 import org.junit.Test;
 
 import java.nio.file.FileSystem;
+import java.nio.file.Path;
+import java.nio.file.Paths;
 
 import static org.junit.Assert.assertEquals;
 
@@ -69,4 +72,23 @@ public class NodeAgentContextImplTest {
     public void path_under_vespa_home_must_be_relative() {
         context.pathInNodeUnderVespaHome("/home");
     }
+
+    @Test
+    public void rewrites_vespa_home_mount_point() {
+        assertRewrite("docker.tld/vespa/ci:1.2.3", "/var/log", "/var/log");
+        assertRewrite("docker.tld/vespa/ci:1.2.3", "/home/y/log", "/home/y/log");
+        assertRewrite("docker.tld/vespa/ci:1.2.3", "/opt/vespa/log", "/home/y/log");
+
+        assertRewrite("docker.tld/vespa/hosted:1.2.3", "/var/log", "/var/log");
+        assertRewrite("docker.tld/vespa/hosted:1.2.3", "/home/y/log", "/home/y/log");
+        assertRewrite("docker.tld/vespa/hosted:1.2.3", "/opt/vespa/log", "/opt/vespa/log");
+    }
+
+    private static void assertRewrite(String dockerImage, String path, String expected) {
+        NodeAgentContext context = new NodeAgentContextImpl.Builder("node123")
+                .nodeSpecBuilder(ns -> ns.wantedDockerImage(DockerImage.fromString(dockerImage)))
+                .build();
+        Path actual = context.rewritePathInNodeForWantedDockerImage(Paths.get(path));
+        assertEquals(Paths.get(expected), actual);
+    }
 }