Skip to content

Commit 0f4eae5

Browse files
committed
Migration -All support code-review fixes
1 parent 07acc17 commit 0f4eae5

File tree

4 files changed

+8
-15
lines changed

4 files changed

+8
-15
lines changed

src/ServiceManagement/StorSimple/Commands.StorSimple/Cmdlets/Migration/ConfirmAzureStorSimpleLegacyVolumeContainerStatus.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@ namespace Microsoft.WindowsAzure.Commands.StorSimple.Cmdlets
2626
[Cmdlet(VerbsLifecycle.Confirm, "AzureStorSimpleLegacyVolumeContainerStatus")]
2727
public class ConfirmAzureStorSimpleLegacyVolumeContainerStatus : StorSimpleCmdletBase
2828
{
29-
public const string AllContainers = "ALL";
30-
public const string SpecificContainers = "DCSpecific";
31-
3229
[Parameter(Mandatory = true, Position = 0, HelpMessage = StorSimpleCmdletHelpMessage.MigrationConfigId)]
3330
[ValidateNotNullOrEmpty]
3431
public string LegacyConfigId { get; set; }
@@ -37,11 +34,11 @@ public class ConfirmAzureStorSimpleLegacyVolumeContainerStatus : StorSimpleCmdle
3734
[ValidateSet("Commit", "Rollback", IgnoreCase = true)]
3835
public string MigrationOperation { get; set; }
3936

40-
[Parameter(Mandatory = true, Position = 2, ParameterSetName = SpecificContainers,
37+
[Parameter(Mandatory = true, Position = 2, ParameterSetName = StorSimpleCmdletParameterSet.MigrateSpecificContainer,
4138
HelpMessage = StorSimpleCmdletHelpMessage.MigrationLegacyDataContainers)]
4239
public string[] LegacyContainerNames { get; set; }
4340

44-
[Parameter(Mandatory = true, Position = 2, ParameterSetName = AllContainers,
41+
[Parameter(Mandatory = true, Position = 2, ParameterSetName = StorSimpleCmdletParameterSet.MigrateAllContainer,
4542
HelpMessage = StorSimpleCmdletHelpMessage.MigrationAllContainers)]
4643
public SwitchParameter All { get; set; }
4744

src/ServiceManagement/StorSimple/Commands.StorSimple/Cmdlets/Migration/ImportAzureStorSimpleLegacyVolumeContainer.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,15 @@ namespace Microsoft.WindowsAzure.Commands.StorSimple.Cmdlets
2525
[Cmdlet(VerbsData.Import, "AzureStorSimpleLegacyVolumeContainer")]
2626
public class ImportAzureStorSimpleLegacyVolumeContainer : StorSimpleCmdletBase
2727
{
28-
public const string AllContainers = "ALL";
29-
public const string SpecificContainers = "DCSpecific";
30-
3128
[Parameter(Mandatory = true, Position = 0, HelpMessage = StorSimpleCmdletHelpMessage.MigrationConfigId)]
3229
[ValidateNotNullOrEmpty]
3330
public string LegacyConfigId { get; set; }
3431

35-
[Parameter(Mandatory = true, Position = 1, ParameterSetName = SpecificContainers,
32+
[Parameter(Mandatory = true, Position = 1, ParameterSetName = StorSimpleCmdletParameterSet.MigrateSpecificContainer,
3633
HelpMessage = StorSimpleCmdletHelpMessage.MigrationLegacyDataContainers)]
3734
public string[] LegacyContainerNames { get; set; }
3835

39-
[Parameter(Mandatory = true, Position = 1, ParameterSetName = AllContainers,
36+
[Parameter(Mandatory = true, Position = 1, ParameterSetName = StorSimpleCmdletParameterSet.MigrateAllContainer,
4037
HelpMessage = StorSimpleCmdletHelpMessage.MigrationAllContainers)]
4138
public SwitchParameter All { get; set; }
4239

src/ServiceManagement/StorSimple/Commands.StorSimple/Cmdlets/Migration/StartAzureStorSimpleLegacyVolumeContainerMigrationPlan.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,15 @@ namespace Microsoft.WindowsAzure.Commands.StorSimple.Cmdlets
2525
[Cmdlet(VerbsLifecycle.Start, "AzureStorSimpleLegacyVolumeContainerMigrationPlan")]
2626
public class StartAzureStorSimpleLegacyVolumeContainerMigrationPlan : StorSimpleCmdletBase
2727
{
28-
public const string AllContainers = "ALL";
29-
public const string SpecificContainers = "DCSpecific";
30-
3128
[Parameter(Mandatory = true, Position = 0, HelpMessage = StorSimpleCmdletHelpMessage.MigrationConfigId)]
3229
[ValidateNotNullOrEmpty]
3330
public string LegacyConfigId { get; set; }
3431

35-
[Parameter(Mandatory = true, Position = 1, ParameterSetName = SpecificContainers,
32+
[Parameter(Mandatory = true, Position = 1, ParameterSetName = StorSimpleCmdletParameterSet.MigrateSpecificContainer,
3633
HelpMessage = StorSimpleCmdletHelpMessage.MigrationLegacyDataContainers)]
3734
public string[] LegacyContainerNames { get; set; }
3835

39-
[Parameter(Mandatory = true, Position = 1, ParameterSetName = AllContainers,
36+
[Parameter(Mandatory = true, Position = 1, ParameterSetName = StorSimpleCmdletParameterSet.MigrateAllContainer,
4037
HelpMessage = StorSimpleCmdletHelpMessage.MigrationAllContainers)]
4138
public SwitchParameter All { get; set; }
4239

src/ServiceManagement/StorSimple/Commands.StorSimple/StorSimpleCmdletParameterSet.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,5 +28,7 @@ internal static class StorSimpleCmdletParameterSet
2828
public const string IdentifyByStatus = "IdentifyByStatus";
2929
public const string UseExistingStorageAccount = "UseExistingStorageAccount";
3030
public const string CreateNewStorageAccount = "CreateNewStorageAccount";
31+
public const string MigrateAllContainer = "MigrateAllContainer";
32+
public const string MigrateSpecificContainer = "MigrateSpecificContainer";
3133
}
3234
}

0 commit comments

Comments
 (0)