Skip to content

Commit 060126d

Browse files
Gupta, SuryaGupta, Surya
authored andcommitted
[CSTACKEX-204] Change the frequency min limit to 3 hours
1 parent a945023 commit 060126d

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/utils/OntapStorageConstants.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ public class OntapStorageConstants {
171171
public static final String ASUP_ENABLED_CONFIG_KEY = "ontap.asup.enabled";
172172
public static final String ASUP_ENABLED_DEFAULT = "true";
173173
public static final String ASUP_INTERVAL_CONFIG_KEY = "ontap.asup.interval";
174-
public static final int ASUP_MIN_INTERVAL_HOURS = 1; // 1 hour (demo; production is 3)
174+
public static final int ASUP_MIN_INTERVAL_HOURS = 3;
175175
public static final int ASUP_MAX_INTERVAL_HOURS = 24;
176176
public static final int ASUP_DEFAULT_INTERVAL_HOURS = 12;
177177

plugins/storage/volume/ontap/src/test/java/org/apache/cloudstack/storage/asup/OntapAsupManagerTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -540,6 +540,7 @@ void validateAsupInterval_acceptsMinMaxAndDefault() {
540540

541541
@Test
542542
void validateAsupInterval_rejectsOutOfRangeAndNonInteger() {
543+
assertThrows(InvalidParameterValueException.class, () -> OntapConfigurationManager.AsupIntervalHours.validateValue("2"));
543544
assertThrows(InvalidParameterValueException.class, () -> OntapConfigurationManager.AsupIntervalHours.validateValue("0"));
544545
assertThrows(InvalidParameterValueException.class, () -> OntapConfigurationManager.AsupIntervalHours.validateValue("25"));
545546
assertThrows(InvalidParameterValueException.class, () -> OntapConfigurationManager.AsupIntervalHours.validateValue("abc"));
@@ -552,7 +553,7 @@ void getAsupIntervalHours_fallsBackOutsideRange() {
552553
asupManager.getAsupIntervalHours(null));
553554
assertEquals(OntapStorageConstants.ASUP_DEFAULT_INTERVAL_HOURS,
554555
asupManager.getAsupIntervalHours(0));
555-
assertEquals(2,
556+
assertEquals(OntapStorageConstants.ASUP_DEFAULT_INTERVAL_HOURS,
556557
asupManager.getAsupIntervalHours(2));
557558
assertEquals(OntapStorageConstants.ASUP_DEFAULT_INTERVAL_HOURS,
558559
asupManager.getAsupIntervalHours(25));

0 commit comments

Comments
 (0)