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 azure monitor log profile module #54702
Conversation
@yungezz, just so you are aware we have a dedicated Working Group for azure. |
The test
The test
|
The test
|
HI @mattclay seems test account doesn't have permission to create log profile, pls see below error:
https://app.shippable.com/github/ansible/ansible/runs/117172/114/tests Could you pls grant necessary permission? Thanks. |
@yungezz I don't think we'll be able to test this with our current CI system. This feature appears to operate at the subscription level, rather than the resource group or resource level. We don't have a safe way to expose privileges for managing the subscription currently. You can still create tests and run them locally, but you'll need to mark the test |
got it. thanks Matt @mattclay . |
ready_for_review |
@Madhura-CSI @brusMX @caohai @cperrin88 @devigned @gustavomcarmo @haroldwongms @iphilpot @julienstroheker @lmazuel @obsoleted @sozercan @techknowlogick @trstringer @tstringer @xscript @yaweiw @yuwzho As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
@yuwzho @zikalino @nitzmahone Please take a review this when you're free? Thank you very much! |
@yuwzho @zikalino @nitzmahone review_need |
The test
|
ready_for_review |
@nitzmahone @jborean93 Please take a review this PR when you're free! Thank you very much! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple little docs things, otherwise LGTM
type: list | ||
categories: | ||
description: | ||
- List of categories of logs. These categories are created as is convinient to user. Some Values are Write, Delete and/or Action. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
convenient
service_bus_rule_id: | ||
description: | ||
- The service bus rule ID of the service bus namespace in which you would like to have Event Hubs created for streaming in the Activity Log. | ||
- format like {serice_bus_resource_id}/authorizationrules{key_name}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
service_bus_resource_id
@nitzmahone It has been updated, please help to review it again! thank you very much! |
all minor comments addressed now -- merging |
* add azure monitor log profile module * fix version * fix lint * mark test as unsupported * fix lint * fix lint * Fix the error prompted in the comments
* add azure monitor log profile module * fix version * fix lint * mark test as unsupported * fix lint * fix lint * Fix the error prompted in the comments
SUMMARY
add new module azure_rm_monitorlogprofile. mark as WIP to avoid be merged into 2.8.
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION