Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nsxt_policy_lb_pool to support a list for active_monitor_path #845

Closed
declan-morris opened this issue Mar 1, 2023 · 0 comments
Closed
Assignees
Labels
enhancement Enhancement
Milestone

Comments

@declan-morris
Copy link

Is your feature request related to a problem? Please describe.

nsxt_policy_lb_pool can have an active monitor attached to it. NSXT allows you to add multiple active monitors in the UI but this doesn't appear to be supported via terraform. Adding one via terraform allows management of the first one in the list, more can be added in the NSXT UI and terraform won't see that change.

Describe the solution you'd like

I think the active_monitor_path should support a list of string for the different paths for a list of active monitors

Describe alternatives you've considered

Separate out the virtual server to have one for each port and use a check on each virtual server

Additional context

No response

@declan-morris declan-morris added the enhancement Enhancement label Mar 1, 2023
@salv-orlando salv-orlando self-assigned this Nov 3, 2023
salv-orlando added a commit to salv-orlando/terraform-provider-nsxt that referenced this issue Nov 22, 2023
The active_monitor_path supports a single attribute whereas
the NSX API allows for multiple monitor paths to be specified.

This commit introduces a list attribute called active_monitor_paths
and deprecates active_monitor_path. The two attributes are
mutually exclusive.
salv-orlando added a commit to salv-orlando/terraform-provider-nsxt that referenced this issue Nov 22, 2023
The active_monitor_path supports a single attribute whereas
the NSX API allows for multiple monitor paths to be specified.

This commit introduces a list attribute called active_monitor_paths
and deprecates active_monitor_path. The two attributes are
mutually exclusive.

Signed-off-by: Salvatore Orlando <[email protected]>
@salv-orlando salv-orlando added this to the v3.4.1 milestone Nov 24, 2023
salv-orlando added a commit to salv-orlando/terraform-provider-nsxt that referenced this issue Nov 28, 2023
The active_monitor_path supports a single attribute whereas
the NSX API allows for multiple monitor paths to be specified.

This commit introduces a list attribute called active_monitor_paths
and deprecates active_monitor_path. The two attributes are
mutually exclusive.

Signed-off-by: Salvatore Orlando <[email protected]>
salv-orlando added a commit to salv-orlando/terraform-provider-nsxt that referenced this issue Nov 30, 2023
The active_monitor_path supports a single attribute whereas
the NSX API allows for multiple monitor paths to be specified.

This commit introduces a list attribute called active_monitor_paths
and deprecates active_monitor_path. The two attributes are
mutually exclusive.

Signed-off-by: Salvatore Orlando <[email protected]>
salv-orlando added a commit to salv-orlando/terraform-provider-nsxt that referenced this issue Dec 1, 2023
The active_monitor_path supports a single attribute whereas
the NSX API allows for multiple monitor paths to be specified.

This commit introduces a list attribute called active_monitor_paths
and deprecates active_monitor_path. The two attributes are
mutually exclusive.

Signed-off-by: Salvatore Orlando <[email protected]>
salv-orlando added a commit that referenced this issue Dec 4, 2023
Policy LB pool: introduce active_monitor_paths attribute (#845)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement
Projects
None yet
Development

No branches or pull requests

3 participants