diff options
author | Jon Bratseth <bratseth@oath.com> | 2018-11-30 05:28:28 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-11-30 05:28:28 -0800 |
commit | c6bd76ac92e97f6273ebf70a7d93751198d66774 (patch) | |
tree | 58002ddf50ce57355731da1030cb2e99576e854c | |
parent | 21bf7a18cbe61b680b328265f9556eb0967f7bb5 (diff) | |
parent | ba142416b7cdba9031365dc641fedbce1be777ed (diff) |
Merge pull request #7820 from vespa-engine/jvenstad/add-source-info-to-app-response
Add git info for last submitted package to app response
6 files changed, 33 insertions, 0 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java index 26495178c9b..2c5bbef1e70 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java @@ -391,6 +391,14 @@ public class ApplicationApiHandler extends LoggingRequestHandler { "/instance/" + application.id().instance().value() + "/job/", request.getUri()).toString()); + application.deploymentJobs().statusOf(JobType.component) + .flatMap(status -> status.lastSuccess()) + .map(run -> run.application().source()) + .ifPresent(source -> sourceRevisionToSlime(source, object.setObject("source"))); + + application.deploymentJobs().projectId() + .ifPresent(id -> object.setLong("projectId", id)); + // Currently deploying change if (application.change().isPresent()) { toSlime(object.setObject("deploying"), application.change()); diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiTest.java index 33ac089439c..e65e0b62e1b 100644 --- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiTest.java +++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiTest.java @@ -97,6 +97,7 @@ import static org.junit.Assert.assertTrue; * @author bratseth * @author mpolden * @author bjorncs + * @author jonmv */ public class ApplicationApiTest extends ControllerContainerTest { diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application-without-change-multiple-deployments.json b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application-without-change-multiple-deployments.json index 7bb78aba459..417fdce7550 100644 --- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application-without-change-multiple-deployments.json +++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application-without-change-multiple-deployments.json @@ -2,6 +2,12 @@ "application": "application1", "instance": "default", "deployments": "http://localhost:8080/application/v4/tenant/tenant1/application/application1/instance/default/job/", + "source": { + "gitRepository": "repository1", + "gitBranch": "master", + "gitCommit": "commit1" + }, + "projectId": 1, "deployedInternally": false, "deploymentJobs": [ { diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application.json b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application.json index 2f25e9471aa..1fc61d92f40 100644 --- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application.json +++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application.json @@ -2,6 +2,12 @@ "application": "application1", "instance": "default", "deployments": "http://localhost:8080/application/v4/tenant/tenant1/application/application1/instance/default/job/", + "source": { + "gitRepository": "repository1", + "gitBranch": "master", + "gitCommit": "commit1" + }, + "projectId": 123, "deploying": { "revision": { "hash": "(ignore)", diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application1-recursive.json b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application1-recursive.json index da6bf455857..75dd484866f 100644 --- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application1-recursive.json +++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application1-recursive.json @@ -2,6 +2,12 @@ "application": "application1", "instance": "default", "deployments": "http://localhost:8080/application/v4/tenant/tenant1/application/application1/instance/default/job/", + "source": { + "gitRepository": "repository1", + "gitBranch": "master", + "gitCommit": "commit1" + }, + "projectId": 123, "deploying": { "revision": { "hash": "(ignore)", diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application2.json b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application2.json index 56abc8f9d82..2c34e5ae712 100644 --- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application2.json +++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/restapi/application/responses/application2.json @@ -2,6 +2,12 @@ "application": "application2", "instance": "default", "deployments": "http://localhost:8080/application/v4/tenant/tenant2/application/application2/instance/default/job/", + "source": { + "gitRepository": "repository1", + "gitBranch": "master", + "gitCommit": "commit1" + }, + "projectId": 456, "deploying": { "version": "(ignore)" }, |