-
Notifications
You must be signed in to change notification settings - Fork 84
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
Flkelly powershell greenfield #226
Conversation
$arsPrefix updated to $arsPrefix = "10.0.2.0/26" It was 10.0.2.0/16 which was invalid CIDR for this range
…ure/Enterprise-Scale-for-AVS into flkelly-powershell-greenfield
@khensler Could you please review the PR and approve for PowerShell module? |
@fskelly Is this PR still needed as you are able to push the code? |
@fskelly @shaunjacob is this PR still applicable? |
@fskelly is this PR still applicable? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed these changes and resolved the lone merge conflict. Will need to add an item to re-run testing on these changes to ensure that there aren't api or cli changes that may impact functionality since this was tested some time ago.
Overview/Summary
PowerShell version of AVS Deployment
Json based for variables
Allows for full or staggered deployment
Breaking Changes
Testing Evidence
Tested with bugbash and improved with feedback from CSU
As part of this Pull Request I have
main
branch