This repository was archived by the owner on Oct 28, 2022. It is now read-only.
Fixing bug in New-NsxLoadBalancerPool#611
Open
Nick-Andreev wants to merge 1 commit intovmware-archive:masterfrom
Open
Fixing bug in New-NsxLoadBalancerPool#611Nick-Andreev wants to merge 1 commit intovmware-archive:masterfrom
Nick-Andreev wants to merge 1 commit intovmware-archive:masterfrom
Conversation
alagoutte
reviewed
Jan 8, 2020
|
|
||
| # Test to cover a bug where New-NsxLoadBalancerPool fails to create a new pool on the Edge, | ||
| # when Edge already has one existing pool and one existing VIP. | ||
| Context "Load Balancer Pool (Extended)" { |
Contributor
There was a problem hiding this comment.
Load Balancer Pool (Add Multiple Pool) ?
Author
There was a problem hiding this comment.
Yeah, this bug happens only under certain circumstances, when:
- You already have one pool
- You already have one VIP
- And you trying to add a second pool - that's when the bug manifests itself
I had a long post under #434 explaining why this happens.
This test is more for demonstration purposes and for confirming that the issue is fixed by the submitted commits. It's up to you if you want to keep this test moving forward, to make sure this bug never get reintroduced into the code again in future versions.
From pure testing purposes, it doesn't test anything more than the already existing "Load Balancer Pool" test.
Hope that makes sense.
Contributor
|
LGTM @dcoghlan can you launch test suite ? |
Contributor
|
jenkins test this please |
|
Tests Successful |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixing a bug in New-NsxLoadBalancerPool, where it fails to create a new pool on the Edge, when Edge already has one existing pool and one existing VIP.
This is the error message you get:
All credit goes to @neoxinth who fixed this in pull request #435. I just added a Pester test for this bug for regression testing purposes.
There is a full explanation of why this issue happens in the first place, which you can find in the discussion thread for #435.