Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only considering StorageScaleUnit as Fault Domain Type #1

Open
DarrylvanderPeijl opened this issue Sep 30, 2019 · 4 comments
Open

Only considering StorageScaleUnit as Fault Domain Type #1

DarrylvanderPeijl opened this issue Sep 30, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@DarrylvanderPeijl
Copy link

S2D-Maintenance.ps1 does not consider other fault domain types then 'StorageScaleUnit'

$ScaleUnit = Get-StorageFaultDomain -Type StorageScaleUnit -CimSession $Name | Where-Object { $_.FriendlyName -eq $Name }

@comnam90 comnam90 self-assigned this Oct 4, 2019
@comnam90 comnam90 added the enhancement New feature or request label Oct 4, 2019
@comnam90
Copy link
Owner

comnam90 commented Oct 4, 2019

@DarrylvanderPeijl
Does a storage node have a type other than StorageScaleUnit?

@comnam90
Copy link
Owner

comnam90 commented Oct 4, 2019

Alternative code would be to put the enclosure related to the storage node into maintenance mode, but that assumes anyone using the script has hardware that supports SES.

Code:

$Enclosure = Get-StorageNode -CimSession $Name | `
    Where-Object {$_.Name -ilike "$Name*"} | `
    Get-StorageEnclosure -PhysicallyConnected
$Enclosure | Enable-StorageMaintenanceMode

@DarrylvanderPeijl
Copy link
Author

I get what you're saying.
In theory, if fault domains is configured as "Rack", you can put all nodes in a rack in maintenance. (and update them all at once)
That was what I was targeting.

@comnam90
Copy link
Owner

comnam90 commented Oct 7, 2019

Ah right, compromise then, I'll build a different command like Enable-S2DFDMaintenance

It'll have a parameter for FaultDomainType like Rack, Enclosure, Site, Etc, and the Name of the Fault Domain they want to put into maintenance mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants