Azure / bicep-registry-modules

Bicep registry modules
MIT License
421 stars 277 forks source link

[AVM Module Issue]: avm/ptn/security/security-center - AMA/MMA support #2665

Open jikuja opened 3 days ago

jikuja commented 3 days ago

Check for previous/existing GitHub issues

Issue Type?

Feature Request

Module Name

avm/ptn/security/security-center

(Optional) Module Version

No response

Description

Microsoft.Security/workspaceSettings | AMA vs MMA

Is Microsoft.Security/workspaceSettings for AMA or MMA?

I think this should be optional resource: https://techcommunity.microsoft.com/t5/microsoft-defender-for-cloud/microsoft-defender-for-cloud-strategy-and-plan-towards-log/ba-p/3883341

Current documentation

IMO documentation should better than

AMA

Not sure if AMA support is good for this module. Some defender features might require it but MMA/AMA should be optional.

(Optional) Correlation Id

No response

microsoft-github-policy-service[bot] commented 3 days ago

[!IMPORTANT] The "Needs: Triage :mag:" label must be removed once the triage process is complete!

[!TIP] For additional guidance on how to triage this issue/PR, see the BRM Issue Triage documentation.

avm-team-linter[bot] commented 3 days ago

@jikuja, thanks for submitting this issue for the avm/ptn/security/security-center module!

[!IMPORTANT] A member of the @Azure/avm-ptn-security-securitycenter-module-owners-bicep or @Azure/avm-ptn-security-securitycenter-module-contributors-bicep team will review it soon!