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 Identity support to hdinsight cluster #1695

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

p3ck
Copy link
Collaborator

@p3ck p3ck commented Aug 30, 2024

SUMMARY

Adds Managed Identity support to the hdinsight cluster module

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/azure_rm_hdinsightcluster.py
plugins/modules/azure_rm_hdinsightcluster_info.py
tests/integration/targets/azure_rm_hdinsightcluster/tasks/main.yml
tests/integration/targets/azure_rm_hdinsightcluster/tasks/managedidentity.yml

ADDITIONAL INFORMATION

Test cases have been updated to use newer versions since the old versions are no longer available.

@p3ck p3ck added enhancement New feature or request ready_for_review The PR has been modified and can be reviewed and merged new_feature New feature requirments labels Aug 30, 2024
@p3ck p3ck requested a review from Fred-sun August 30, 2024 15:49
@p3ck p3ck force-pushed the identity-hdinsight branch from c5d23cf to 9c32ec4 Compare August 30, 2024 15:50
@p3ck p3ck force-pushed the identity-hdinsight branch from 9c32ec4 to 6b26d5b Compare September 3, 2024 17:20
@p3ck
Copy link
Collaborator Author

p3ck commented Sep 3, 2024

rebased on latest dev - please review. Thanks

@p3ck
Copy link
Collaborator Author

p3ck commented Sep 17, 2024

CI test run https://dev.azure.com/azclitools/public/_build/results?buildId=190234&view=results Passed

Can we please review and merge?

Thanks

@Fred-sun
Copy link
Collaborator

@p3ck Has been pushed, will merge as soon as possible. Thank you!

@xuzhang3 xuzhang3 merged commit 1fd45ee into ansible-collections:dev Sep 18, 2024
Justwmz pushed a commit to Justwmz/azure that referenced this pull request Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request new_feature New feature requirments ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants