Skip to content

Commit 96287c6

Browse files
Gupta, SuryaGupta, Surya
authored andcommitted
[CSTACKEX-204] Address Review Comments
1 parent 504c4c7 commit 96287c6

3 files changed

Lines changed: 3 additions & 6 deletions

File tree

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,7 @@ public class OntapConfigurationManager implements Configurable {
3737
OntapStorageConstants.ASUP_INTERVAL_CONFIG_KEY,
3838
String.valueOf(OntapStorageConstants.ASUP_DEFAULT_INTERVAL_HOURS),
3939
OntapStorageConstants.ASUP_INTERVAL_DESCRIPTION,
40-
true, ConfigKey.Scope.Global, null,
41-
OntapStorageConstants.ASUP_INTERVAL_DISPLAY_TEXT, asupIntervalValidator());
40+
true, ConfigKey.Scope.Global, null, asupIntervalValidator());
4241

4342
public static final ConfigKey<?>[] CONFIG_KEYS = new ConfigKey<?>[] {
4443
AsupIntervalHours

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,6 @@ public class OntapStorageConstants {
174174
public static final int ASUP_MIN_INTERVAL_HOURS = 1;
175175
public static final int ASUP_MAX_INTERVAL_HOURS = 168; // once a week
176176
public static final int ASUP_DEFAULT_INTERVAL_HOURS = 24; // once a day
177-
public static final String ASUP_INTERVAL_DISPLAY_TEXT = "NetApp ONTAP AutoSupport interval";
178177

179178
public static final String ASUP_INTERVAL_DESCRIPTION =
180179
"The interval in hours between telemetry reports sent to the ONTAP cluster. "

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -520,11 +520,10 @@ void asupIntervalHours_defaultIsProductionValue() {
520520
}
521521

522522
@Test
523-
void asupIntervalHours_keyAndDisplayText() {
523+
void asupIntervalHours_keyIsAutosupportInterval() {
524524
assertEquals(OntapStorageConstants.ASUP_INTERVAL_CONFIG_KEY,
525525
OntapConfigurationManager.AsupIntervalHours.key());
526-
assertEquals(OntapStorageConstants.ASUP_INTERVAL_DISPLAY_TEXT,
527-
OntapConfigurationManager.AsupIntervalHours.displayText());
526+
assertEquals("ontap.autosupport.interval", OntapConfigurationManager.AsupIntervalHours.key());
528527
}
529528

530529
@Test

0 commit comments

Comments
 (0)