Skip to content
This repository has been archived by the owner on Oct 28, 2022. It is now read-only.

Repair-NsxEdge: Add Upgrade feature to Nsx-Edge #454

Merged
merged 2 commits into from
Jan 29, 2018

Conversation

alagoutte
Copy link
Contributor

It is now possible to upgrade Edge with a cmdlet !

It is now possible to upgrade Edge with a cmdlet !
Copy link
Contributor

@nmbradford nmbradford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just comment grammar corrections - ill test based on current code, but will wait to merge until you have updated. Thanks for PR

@@ -13695,7 +13695,7 @@ function Repair-NsxEdge {
up to 200 subinterfaces. Multiple external IP addresses can be configured
for load balancer, site‐to‐site VPN, and NAT services.

The Repair-NsxEdge cmdlet allows a Resync or Redploy operation to be
The Repair-NsxEdge cmdlet allows a Resync, Redploy or Upgrade operation to be
performed on the specified Edges appliance.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edges -> Edge.

@@ -13728,7 +13733,8 @@ function Repair-NsxEdge {
#Specify the repair operation to be performed on the Edge.
#If ForceSync - The edge appliance is rebooted
#If Redeploy - The Edge is removed and redeployed (if the edge is HA this causes failover, otherwise, an outage.)
[ValidateSet("ForceSync", "Redeploy")]
#If Upgrade - The Edge is upgrade to last release
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change to -> upgraded to latest release

@nmbradford
Copy link
Contributor

jenkins test this please

@powernsxbot
Copy link

Tests Successful

@alagoutte
Copy link
Contributor Author

typo fixed !

@alagoutte
Copy link
Contributor Author

Repair-NsxEdge is not very good name for upgrade..
may be add a alias ?

need also the same function for NsxLogicalRouter ... (there is no Repair-NsxLogicalRouter for the moment...)

@nmbradford
Copy link
Contributor

Ok - did you want to raise another PR for this, or leave this unmerged until you have done that?

@alagoutte
Copy link
Contributor Author

i will push a another PR later...

@nmbradford nmbradford merged commit 85c9bda into vmware-archive:master Jan 29, 2018
@alagoutte alagoutte deleted the edge_upgrade branch March 7, 2018 07:28
@alagoutte alagoutte mentioned this pull request Nov 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants