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

enable FIPs mode on New-NSXEdge #579

Closed
wants to merge 1 commit into from
Closed

Conversation

rlakey
Copy link

@rlakey rlakey commented May 4, 2019

Added option to enable FIPs mode on New-NSXEdge command resolve issue 578

Added option to enable FIPs mode on New-NSXEdge command
@vmwclabot
Copy link

@rlakey, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.

@vmwclabot
Copy link

@rlakey, your company's legal contact has approved your signed contributor license agreement. It will also be reviewed by VMware, but the merge can proceed.

@alagoutte
Copy link
Contributor

Look good but need a better name (for PR)

Do you have look if it is not possible to include on testsuite ?

@vmwclabot
Copy link

@rlakey, VMware has rejected your signed contributor license agreement. The merge can not proceed until the agreement has been resigned. Click here to resign the agreement.

@dcoghlan dcoghlan changed the title Update PowerNSX.psm1 enable FIPs mode on New-NSXEdge Jul 11, 2019
@dcoghlan dcoghlan added the Test Required Tests are required before this request will be merged. label Jul 11, 2019
@rlakey
Copy link
Author

rlakey commented Dec 16, 2019

Sorry i was not notified of the comments. I am part of the VMware github org now so I shouldn't need to sign the CLA.

@alagoutte
Copy link
Contributor

@rlakey, i think you always need to sign CLA

@rlakey
Copy link
Author

rlakey commented Dec 17, 2019

@rlakey, i think you always need to sign CLA

Not according to the link. "As a VMware employee there's no need for you to sign the VMware CLA."

@dcoghlan dcoghlan mentioned this pull request Apr 7, 2021
@dcoghlan dcoghlan closed this in #648 Apr 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-rejected Test Required Tests are required before this request will be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants