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

Fixed New-NsxLoadBalancerPool #435

Closed
wants to merge 2 commits into from

Conversation

StefanPahrmann
Copy link

#434 Fixes New-NsxLoadBalancerPool - fixed building right XML-file, right path to the API and change to POST from PUT in invoke-nsxwebrequest

@vmwclabot
Copy link

@neoxinth, 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

@neoxinth, VMware has approved your signed contributor license agreement.

@dcoghlan dcoghlan added the Test Required Tests are required before this request will be merged. label Jul 10, 2019
@jendm
Copy link

jendm commented Aug 6, 2019

I tested this and got this to work. Thank you. We were having issues with the version in master and this saved us a lot of time.

However I had to change 1 line in order for it to work.
I changed this -- $body = $_LoadBalancer.OuterXml
To this -- $body = $xmlPool.OuterXml

@Nick-Andreev
Copy link

Hey, guys. Could you let me know why this hasn't been merged yet? We are having the same issue. Happy to write a test for you, if that's what's blocking it. @dcoghlan @alagoutte

@alagoutte
Copy link
Contributor

Hi Nick,

Because missing test..

Do you known how to reproduce this issue ?

@Nick-Andreev
Copy link

Yes, I'll add a load balancer test.

@dcoghlan
Copy link
Contributor

Addressed as part of #611

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

Successfully merging this pull request may close these issues.

6 participants