Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Add support for subnetID option #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

phvalguima
Copy link
Contributor

@phvalguima phvalguima commented Aug 9, 2021

This change implements the features described on LP#1939248.
LBProviders can now issue a request with fixed ingress IP for
private LBs (point 1) and subnetID can be specified instead
of vnetName + subnetName. SubnetID covers the case where
subnets can be provided by vnets on different RGs (point 2).

Add a when_all check for manage_lbs() (LP#1937300).

This change implements the features described on LP#1939248.
LBProviders can now issue a request with fixed ingress IP for
private LBs (point 1) and subnetID can be specified instead
of vnetName + subnetName. SubnetID covers the case where
subnets can be provided by vnets on different RGs (point 2).

Add a gate for manage_lbs() (LP#1937300).
Copy link
Contributor

@johnsca johnsca left a comment

Choose a reason for hiding this comment

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

LGTM but I would like to see coverage for this added to the integration test. It doesn't run automatically yet due to credentials and concerns about automatic cleanup, but it would be good to see a manual pass of it with this covered.

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.

2 participants