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

Ignore create_service_account with NSX 9.0.0 #1471

Merged
merged 1 commit into from
Nov 26, 2024
Merged

Conversation

annakhm
Copy link
Collaborator

@annakhm annakhm commented Nov 26, 2024

In compute manager resource, NSX no longer accepts false value for create_service_account attribute.
Since terraform sdk can't distinguish between unset value and false value for the attribute, we will ignore the attribute entirely for NSX 9.0 and above

@vmwclabot
Copy link
Member

@annakhm, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

@vmwclabot vmwclabot added the dco-required DCO Required label Nov 26, 2024
In compute manager resource, NSX no longer accepts false value
for create_service_account attribute.
Since terraform sdk can't distinguish between unset value and false value for the
attribute, we will ignore the attribute entirely for NSX 9.0 and above

Signed-off-by: Anna Khmelnitsky <[email protected]>
@annakhm annakhm force-pushed the service-account-refix branch from b57ff81 to a04e6b4 Compare November 26, 2024 00:14
@vmwclabot vmwclabot removed the dco-required DCO Required label Nov 26, 2024
Copy link
Member

@salv-orlando salv-orlando left a comment

Choose a reason for hiding this comment

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

LGTM

@annakhm annakhm merged commit d36467c into master Nov 26, 2024
3 checks passed
@annakhm annakhm deleted the service-account-refix branch November 26, 2024 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants