Skip to content

Conversation

@adambarreiro
Copy link
Contributor

This PR adds vcfa_org_ldap resource and data source.

To test it uses vCenter configured in the testing JSON configuration as LDAP server, so nothing special is needed.

Signed-off-by: abarreiro <[email protected]>
@adambarreiro adambarreiro self-assigned this Jan 30, 2025
abarreiro added 4 commits January 30, 2025 14:26
#
Signed-off-by: abarreiro <[email protected]>
#
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
#
Signed-off-by: abarreiro <[email protected]>
@adambarreiro adambarreiro marked this pull request as ready for review January 30, 2025 13:51
Copy link

@lvirbalas lvirbalas left a comment

Choose a reason for hiding this comment

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

Tried manually. Thanks!

Signed-off-by: abarreiro <[email protected]>
Copy link
Contributor

@Didainius Didainius left a comment

Choose a reason for hiding this comment

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

We need a mechanism to trust certificate of LDAP server as otherwise it might not work. Validated to work otherwise

#
Signed-off-by: abarreiro <[email protected]>
@adambarreiro adambarreiro merged commit 2f4c5ca into vmware:main Jan 31, 2025
2 checks passed
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.

3 participants