Daft draft to share -- don't stare

2 files changed, 10 insertions, 12 deletions

diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/LogStore.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/LogStore.java
index 07cdff9a9f4..959ac2b8680 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/LogStore.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/LogStore.java
@@ -1,8 +1,7 @@
 package com.yahoo.vespa.hosted.controller.api.integration;
 
-import com.yahoo.config.provision.ApplicationId;
 import com.yahoo.vespa.hosted.controller.api.integration.configserver.PrepareResponse;
-import com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType;
+import com.yahoo.vespa.hosted.controller.api.integration.deployment.RunId;
 
 /**
  * @author freva
@@ -10,24 +9,24 @@ import com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType;
 public interface LogStore {
 
     /** @return the test log of the given deployment job. */
-    String getTestLog(ApplicationId applicationId, JobType jobType, long buildId);
+    String getTestLog(RunId id);
 
     /** Stores the given test log for the given deployment job. */
-    void setTestLog(ApplicationId applicationId, JobType jobType, long buildId, String testLog);
+    void setTestLog(RunId id, String testLog);
 
     /** @return the convergence log of the given deployment job. */
-    String getConvergenceLog(ApplicationId applicationId, JobType jobType, long buildId);
+    String getConvergenceLog(RunId id);
 
     /** Stores the given convergence log for the given deployment job. */
-    void setConvergenceLog(ApplicationId applicationId, JobType jobType, long buildId, String convergenceLog);
+    void setConvergenceLog(RunId id, String convergenceLog);
 
     /** @return the result of prepare of the test application for the given deployment job. */
-    PrepareResponse getPrepareResponse(ApplicationId applicationId, JobType jobType, long buildId);
+    PrepareResponse getPrepareResponse(RunId id);
 
     /** Stores the given result of prepare of the test application for the given deployment job. */
-    void setPrepareResponse(ApplicationId applicationId, JobType jobType, long buildId, PrepareResponse prepareResponse);
+    void setPrepareResponse(RunId id, PrepareResponse prepareResponse);
 
     /** Deletes all data associated with test of a given deployment job */
-    void deleteTestData(ApplicationId applicationId, JobType jobType, long buildId);
+    void deleteTestData(RunId id);
 
 }
diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/RunId.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/RunId.java
index d78dbd6e636..a46cec1bb40 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/RunId.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/RunId.java
@@ -1,12 +1,11 @@
-package com.yahoo.vespa.hosted.controller.deployment;
+package com.yahoo.vespa.hosted.controller.api.integration.deployment;
 
 import com.yahoo.config.provision.ApplicationId;
-import com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType;
 
 import java.util.Objects;
 
 /**
- * Immutable ID of a job run by an {@link InternalBuildService}.
+ * Immutable ID of a job run by a {@link com.yahoo.vespa.hosted.controller.api.integration.BuildService}.
  *
  * @author jonmv
  */