1 files changed, 4 insertions, 28 deletions

diff --git a/config-provisioning/src/main/java/com/yahoo/config/provision/zone/ZoneId.java b/config-provisioning/src/main/java/com/yahoo/config/provision/zone/ZoneId.java
index 5e664e00b4c..b0ac59718aa 100644
--- a/config-provisioning/src/main/java/com/yahoo/config/provision/zone/ZoneId.java
+++ b/config-provisioning/src/main/java/com/yahoo/config/provision/zone/ZoneId.java
@@ -20,30 +20,16 @@ public class ZoneId {
 
     private final Environment environment;
     private final RegionName region;
-    private final SystemName system;
 
-    private ZoneId(Environment environment, RegionName region, CloudName cloud, SystemName system) {
+    private ZoneId(Environment environment, RegionName region) {
         this.environment = Objects.requireNonNull(environment, "environment must be non-null");
         this.region = Objects.requireNonNull(region, "region must be non-null");
-        this.system = Objects.requireNonNull(system, "system must be non-null");
-    }
-
-    private ZoneId(Environment environment, RegionName region) {
-        this(environment, region, CloudName.defaultName(), SystemName.defaultSystem());
     }
 
     public static ZoneId from(Environment environment, RegionName region) {
         return new ZoneId(environment, region);
     }
 
-    public static ZoneId from(SystemName system, Environment environment, RegionName region) {
-        return new ZoneId(environment, region, CloudName.defaultName(), system);
-    }
-
-    public static ZoneId from(Environment environment, RegionName region, CloudName cloud, SystemName system) {
-        return new ZoneId(environment, region, cloud, system);
-    }
-
     public static ZoneId from(String environment, String region) {
         return from(Environment.from(environment), RegionName.from(region));
     }
@@ -55,20 +41,14 @@ public class ZoneId {
             case 2:
                 return from(parts[0], parts[1]);
             case 4:
-                return from(parts[2], parts[3], parts[0], parts[1]);
+                // Deprecated: parts[0] == cloud, parts[1] == system
+                // TODO: Figure out whether this can be removed
+                return from(parts[2], parts[3]);
             default:
                 throw new IllegalArgumentException("Cannot deserialize zone id '" + value + "'");
         }
     }
 
-    public static ZoneId from(Environment environment, RegionName region, CloudName cloud) {
-        return new ZoneId(environment, region, cloud, SystemName.defaultSystem());
-    }
-
-    public static ZoneId from(String environment, String region, String cloud, String system) {
-        return new ZoneId(Environment.from(environment), RegionName.from(region), CloudName.from(cloud), SystemName.from(system));
-    }
-
     public static ZoneId defaultId() {
         return new ZoneId(Environment.defaultEnvironment(), RegionName.defaultName());
     }
@@ -81,10 +61,6 @@ public class ZoneId {
         return region;
     }
 
-    public SystemName system() {
-        return system;
-    }
-
     /** Returns the serialised value of this. Inverse of {@code ZoneId.from(String value)}. */
     public String value() {
         return environment + "." + region;