Fix mock, move kibana stuff to hosted and add some doc

1 files changed, 21 insertions, 4 deletions

diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/zone/ZoneRegistry.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/zone/ZoneRegistry.java
index 4189ecb7c6e..9211fa97204 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/zone/ZoneRegistry.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/zone/ZoneRegistry.java
@@ -20,14 +20,31 @@ import java.util.Optional;
  */
 public interface ZoneRegistry {
 
-    SystemName system();
-    List<ZoneId> zones();
+    /** Returns whether the system of this registry contains the given zone. */
     boolean hasZone(ZoneId zoneId);
+
+    /** Returns a list containing the id of all zones in this registry. */
+    List<ZoneId> zones();
+
+    /** Returns the default region for the given environment. */
+    RegionName getDefaultRegion(Environment environment);
+
+    /** Returns a list with all known config servers in the given zone. */
     List<URI> getConfigServerUris(ZoneId zoneId);
-    Optional<URI> getLogServerUri(ZoneId zoneId);
+
+    /** Returns a URL with the logs for the given deployment, if loggin is configured for its zone. */
+    Optional<URI> getLogServerUri(DeploymentId deploymentId);
+
+    /** Returns the time to live for deployments in the given zone. */
     Duration getDeploymentTimeToLive(ZoneId zoneId);
-    RegionName getDefaultRegion(Environment environment);
+
+    /** Returns a URL pointing at monitoring resources for the given deployment. */
     URI getMonitoringSystemUri(DeploymentId deploymentId);
+
+    /** Returns the URL of the dashboard for the system of this registry. */
     URI getDashboardUri();
 
+    /** Returns the system of this registry. */
+    SystemName system();
+
 }