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

Add provider flag for a policy global manager env #316

Merged
merged 4 commits into from
May 20, 2020

Conversation

asarfaty
Copy link
Contributor

No description provided.

@ghost ghost added size/xs Relative Sizing: extra-small documentation Documentation labels May 19, 2020
@asarfaty asarfaty requested review from annakhm and enhaocui May 19, 2020 05:31
@annakhm annakhm requested a review from madhukark May 19, 2020 16:13
nsxt/provider.go Outdated
@@ -145,6 +146,12 @@ func Provider() terraform.ResourceProvider {
Description: "Enforcement Point for NSXT Policy",
DefaultFunc: schema.EnvDefaultFunc("NSXT_POLICY_ENFORCEMENT_POINT", "default"),
},
"policy_global_manager": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd suggest to call it just "global_manager" - Madhu, what do you think?

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree, just "global_manager" is good.

Copy link
Contributor

@enhaocui enhaocui left a comment

Choose a reason for hiding this comment

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

Generally LGTM apart from Anna's comment.

host = "192.168.110.41"
username = "admin"
password = "default"
policy_global_manager = true
Copy link
Contributor

Choose a reason for hiding this comment

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

global_manager

@@ -177,6 +189,8 @@ The following arguments are used to configure the VMware NSX-T Provider:
* `enforcement_point` - (Optional) Enforcement point, mostly relevant for policy
data sources. For VMC environment, this should be set to `vmc-enforcementpoint`.
For on-prem deployments, this setting should not be specified.
* `policy_global_manager` - (Optional) True if this is a policy global manager
Copy link
Contributor

Choose a reason for hiding this comment

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

Here too, just 'global_manager'. Also, can change the string 'policy global manager' to 'Global Manager'

password = "default"
policy_global_manager = true
host = "192.168.110.41"
usernam = "admin"
Copy link
Contributor

Choose a reason for hiding this comment

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

Oops, typo?

Copy link
Contributor

@enhaocui enhaocui left a comment

Choose a reason for hiding this comment

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

LTGM, one typo

@annakhm annakhm merged commit 5769c6f into federation May 20, 2020
@annakhm annakhm deleted the federation_flag branch September 3, 2020 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation size/xs Relative Sizing: extra-small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants