Skip to content

Commit 9ebeb01

Browse files
Merge pull request #2287 from HubSpot/od-omit
Omit on demand jobs from scale limit during deploy validation
2 parents 0b420cb + 44999f4 commit 9ebeb01

2 files changed

Lines changed: 10 additions & 1 deletion

File tree

SingularityService/src/main/java/com/hubspot/singularity/data/SingularityValidator.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1244,7 +1244,11 @@ public void checkScale(
12441244
}
12451245
}
12461246

1247-
if (previousScale.isPresent() && !largeScaleDownAcknowledged.orElse(false)) {
1247+
if (
1248+
previousScale.isPresent() &&
1249+
!largeScaleDownAcknowledged.orElse(false) &&
1250+
request.getRequestType() != RequestType.ON_DEMAND
1251+
) {
12481252
int absMaxScaleDown = singularityConfiguration.getMaxScaleDownWithoutAcknowledgement();
12491253
boolean scaleDownExceedsAbsoluteMax =
12501254
previousScale.get() - request.getInstancesSafe() > absMaxScaleDown;

SingularityUI/package-lock.json

Lines changed: 5 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)