summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHÃ¥kon Hallingstad <hakon@oath.com>2018-09-26 21:44:19 +0200
committerGitHub <noreply@github.com>2018-09-26 21:44:19 +0200
commit528489892c875b83dda2df8f61fb5acfe4760590 (patch)
treeae44745f2fbf8a87f1242b7c0569b82545ccd830
parent93e547e793ff05135689c8175daf464a8be1dcdd (diff)
parent2504c5624c781f53652117e2c05edb3f5664409b (diff)
Merge pull request #7117 from vespa-engine/hakonhall/fix-casing-of-serialized-cloud
Fix casing of serialized cloud
-rw-r--r--node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerNetworking.java2
-rw-r--r--node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/acl/AclMaintainerTest.java4
2 files changed, 3 insertions, 3 deletions
diff --git a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerNetworking.java b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerNetworking.java
index 7678ad8169a..7f0ade2e500 100644
--- a/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerNetworking.java
+++ b/node-admin/src/main/java/com/yahoo/vespa/hosted/node/admin/docker/DockerNetworking.java
@@ -28,7 +28,7 @@ public enum DockerNetworking {
}
public static DockerNetworking from(String cloud, NodeType nodeType, boolean hostAdmin) {
- if (cloud.equals("AWS")) {
+ if (cloud.equalsIgnoreCase("aws")) {
return DockerNetworking.NPT;
} else if (nodeType == NodeType.confighost || nodeType == NodeType.proxyhost) {
return DockerNetworking.HOST_NETWORK;
diff --git a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/acl/AclMaintainerTest.java b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/acl/AclMaintainerTest.java
index 56373dda2f8..9d5555e1f92 100644
--- a/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/acl/AclMaintainerTest.java
+++ b/node-admin/src/test/java/com/yahoo/vespa/hosted/node/admin/maintenance/acl/AclMaintainerTest.java
@@ -49,13 +49,13 @@ public class AclMaintainerTest {
@Before
public void before() {
when(dockerOperations.getAllManagedContainers()).thenReturn(containerList);
- when(env.getCloud()).thenReturn("AWS");
+ when(env.getCloud()).thenReturn("aws");
when(env.getDockerNetworking()).thenReturn(DockerNetworking.NPT);
}
@Test
public void no_redirect_in_yahoo() {
- when(env.getCloud()).thenReturn("YAHOO");
+ when(env.getCloud()).thenReturn("yahoo");
when(env.getDockerNetworking()).thenReturn(DockerNetworking.MACVLAN);
Container container = addContainer("container1", "container1.host.com", Container.State.RUNNING);