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 Management to log_analytics #1683

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

p3ck
Copy link
Collaborator

@p3ck p3ck commented Aug 21, 2024

SUMMARY

Added Identity Management support to log_analytics module.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/azure_rm_loganalyticsworkspace.py
requirements.txt
tests/integration/targets/azure_rm_loganalyticsworkspace/tasks/main.yml

ADDITIONAL INFORMATION

Identity support is only in the beta release of azure.mgmt.loganalytics

@p3ck p3ck added enhancement New feature or request ready_for_review The PR has been modified and can be reviewed and merged labels Aug 21, 2024
@p3ck p3ck requested review from Fred-sun and nirarg August 21, 2024 17:55
Added Identity Management support to log_analytics module.
@p3ck p3ck force-pushed the identity-log_analytics branch from a00cac4 to acfd74e Compare September 3, 2024 17:20
@p3ck
Copy link
Collaborator Author

p3ck commented Sep 3, 2024

rebased on latest dev - please review

@p3ck
Copy link
Collaborator Author

p3ck commented Sep 17, 2024

CI test results https://dev.azure.com/azclitools/public/_build/results?buildId=190233&view=results Pass

Can we review and merge please?

@Fred-sun
Copy link
Collaborator

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

@xuzhang3 xuzhang3 merged commit 394eac4 into ansible-collections:dev Sep 18, 2024
Justwmz pushed a commit to Justwmz/azure that referenced this pull request Nov 4, 2024
Added Identity Management support to log_analytics module.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 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