Merge pull request #28128 from vespa-engine/jonmv/build-number-in-submit-response

Return build number with submit response

1 files changed, 4 insertions, 4 deletions

diff --git a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java
index 29e1d494ffc..7303320a4f7 100644
--- a/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java
+++ b/controller-api/src/main/java/com/yahoo/vespa/hosted/controller/api/integration/deployment/ApplicationVersion.java
@@ -95,8 +95,8 @@ public class ApplicationVersion implements Comparable<ApplicationVersion> {
     public String stringId() {
         return source.map(SourceRevision::commit).map(ApplicationVersion::abbreviateCommit)
                 .or(this::commit)
-                .map(commit -> String.format("%s.%d-%s", majorVersion, buildNumber().getAsLong(), commit))
-                .orElseGet(() -> majorVersion + "." + buildNumber().getAsLong());
+                .map(commit -> String.format("%s.%d-%s", majorVersion, buildNumber(), commit))
+                .orElseGet(() -> majorVersion + "." + buildNumber());
     }
 
     /**
@@ -105,8 +105,8 @@ public class ApplicationVersion implements Comparable<ApplicationVersion> {
      */
     public Optional<SourceRevision> source() { return source; }
 
-    /** Returns the build number that built this version */
-    public OptionalLong buildNumber() { return OptionalLong.of(id.number()); }
+    /** Returns the build number of this version */
+    public long buildNumber() { return id.number(); }
 
     /** Returns the email of the author of commit of this version, if known */
     public Optional<String> authorEmail() { return authorEmail; }