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.

Sign up for GitHub

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

Jump to bottom

Sync-FlashArrayHosts should have -WhatIf and -Confirm:$true/$false switches added #47

Open
mikenelson-io opened this issue Apr 10, 2023 · 1 comment
Open

Sync-FlashArrayHosts should have -WhatIf and -Confirm:$true/$false switches added #47

mikenelson-io opened this issue Apr 10, 2023 · 1 comment
Assignees
@aguzev @mikenelson-io @barkz
Labels
bug Something isn't working fixed Issue is fixed in current or future version Retest Retest fix

Comments

@mikenelson-io
Copy link
Member

This cmdlet should be able to show what would happen if the user ran this cmdlet against two arrays but not actually do the operation without confirmation. Without confirmation, a user could potentially inadvertently overwrite the target array configuration.
@mikenelson-io mikenelson-io changed the title Sync-FlashArrayHosts should have -WhatIf and -Confirm"$true/false switches added Sync-FlashArrayHosts should have -WhatIf and -Confirm:$true/$false switches added Apr 10, 2023
@barkz barkz added bug Something isn't working Retest Retest fix labels Apr 11, 2023
@barkz
Copy link
Contributor

barkz commented Apr 12, 2023

Implement Write-Warning noting destructive operation with confirm.

@aguzev aguzev added the working on it Actively being worked on or planned for next release label Apr 13, 2023
aguzev added a commit that referenced this issue Apr 13, 2023
@aguzev
April13 update
005550a 
- Remove $Creds from help GH-52
- Rename Exchange private functions GH-41
- Check OS version in Test-WindowsBestPractices GH-48
- Set Confirm Impact for  Test-WindowsBestPractices to High GH-64
- Add confirmation to Sync-FlashArrayHosts GH-47
-
-
@aguzev aguzev mentioned this issue Apr 13, 2023
Merged
@aguzev aguzev added fixed Issue is fixed in current or future version and removed working on it Actively being worked on or planned for next release labels Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aguzev aguzev

@mikenelson-io mikenelson-io

@barkz barkz

Labels
bug Something isn't working fixed Issue is fixed in current or future version Retest Retest fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@aguzev @mikenelson-io @barkz