diff --git a/SingularityBase/src/main/java/com/hubspot/mesos/SingularityVolume.java b/SingularityBase/src/main/java/com/hubspot/mesos/SingularityVolume.java index 2dbd21bbcc..09c8edda1c 100644 --- a/SingularityBase/src/main/java/com/hubspot/mesos/SingularityVolume.java +++ b/SingularityBase/src/main/java/com/hubspot/mesos/SingularityVolume.java @@ -9,13 +9,13 @@ public class SingularityVolume { private final String containerPath; private final Optional hostPath; - private final Mode mode; + private final Optional mode; @JsonCreator public SingularityVolume( @JsonProperty("containerPath") String containerPath, @JsonProperty("hostPath") Optional hostPath, - @JsonProperty("mode") Mode mode) { + @JsonProperty("mode") Optional mode) { this.containerPath = containerPath; this.hostPath = hostPath; this.mode = mode; @@ -29,7 +29,7 @@ public Optional getHostPath() { return hostPath; } - public Mode getMode() { + public Optional getMode() { return mode; } diff --git a/SingularityService/src/main/java/com/hubspot/singularity/mesos/SingularityMesosTaskBuilder.java b/SingularityService/src/main/java/com/hubspot/singularity/mesos/SingularityMesosTaskBuilder.java index 8d41765fbd..90717dba24 100644 --- a/SingularityService/src/main/java/com/hubspot/singularity/mesos/SingularityMesosTaskBuilder.java +++ b/SingularityService/src/main/java/com/hubspot/singularity/mesos/SingularityMesosTaskBuilder.java @@ -220,7 +220,9 @@ private void prepareContainerInfo(final SingularityTaskId taskId, final TaskInfo if (volumeInfo.getHostPath().isPresent()) { volumeBuilder.setHostPath(fillInTaskIdValues(volumeInfo.getHostPath().get(), taskId)); } - volumeBuilder.setMode(volumeInfo.getMode()); + if (volumeInfo.getMode().isPresent()) { + volumeBuilder.setMode(volumeInfo.getMode().get()); + } containerBuilder.addVolumes(volumeBuilder); }