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 a Bicep Azure Sentinel module #385

Merged
merged 7 commits into from
Sep 7, 2021
Merged

add a Bicep Azure Sentinel module #385

merged 7 commits into from
Sep 7, 2021

Conversation

vidyambala
Copy link
Contributor

@vidyambala vidyambala commented Sep 2, 2021

Description

Azure Sentinel resource deployment in TF converted to Bicep module.

Issue reference

The issue this PR will close: #359

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles or validates correctly
  • BASH scripts have been validated using shellcheck
  • All tests pass (manual and automated)
  • The documentation is updated to cover any new or changed features
  • Markdown files have been linted using the recommended linter. (See .vscode/extensions.json.)
  • Relevant issues are linked to this PR

@vidyambala vidyambala added the bicep Related to Bicep code label Sep 2, 2021
@vidyambala vidyambala self-assigned this Sep 2, 2021
@glennmusa glennmusa self-assigned this Sep 2, 2021
@glennmusa glennmusa changed the title Provision Azure Sentinel with the Bicep deployment #359 add a Bicep Azure Sentinel module Sep 2, 2021
Copy link
Contributor

@glennmusa glennmusa left a comment

Choose a reason for hiding this comment

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

Thanks @vidyambala! Was able to get this working with some changes documented in this review.

src/bicep/modules/SentinelSolution.bicep Outdated Show resolved Hide resolved
src/bicep/modules/SentinelSolution.bicep Outdated Show resolved Hide resolved
src/bicep/modules/SentinelSolution.bicep Outdated Show resolved Hide resolved
src/bicep/modules/SentinelSolution.bicep Outdated Show resolved Hide resolved
src/bicep/modules/SentinelSolution.bicep Outdated Show resolved Hide resolved
@vidyambala
Copy link
Contributor Author

Updated the module with recommendation.

@vidyambala vidyambala linked an issue Sep 2, 2021 that may be closed by this pull request
src/bicep/mlz.bicep Outdated Show resolved Hide resolved
Co-authored-by: Glenn Musa <4622125+glennmusa@users.noreply.github.com>
Copy link
Contributor

@glennmusa glennmusa left a comment

Choose a reason for hiding this comment

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

looks great, thanks @vidyambala 🎉

@glennmusa glennmusa merged commit ce4e928 into bicep Sep 7, 2021
@glennmusa glennmusa deleted the vidbala-sentinel branch September 7, 2021 19:02
Breanna-Stryker added a commit that referenced this pull request Sep 17, 2021
* prefer local backends for terraform

* add a Bicep Azure Sentinel module (#385)

* Remove Client/Client Secret/Tenant Vars from Terraform Templates

Co-authored-by: Bree Stryker <b-s-no-reply@microsoft.com>

* add workflows to lint and build .bicep modules (#400)

* disable verbose lint output (#402)

* optionally deploy built-in Policy Initiatives for NIST 800-53, CMMC Level 3, or DOD IL5 (#397)

* log activities from subscriptions used in a deployment into the Operations log analytics workspace (#412)

* optionally deploy Azure Bastion Host from the base deployment (#406)

Co-authored-by: Vidya Bala <vidbala@microsoft.com>
Co-authored-by: Bree Stryker <b-s-no-reply@microsoft.com>
Co-authored-by: Glenn Musa <4622125+glennmusa@users.noreply.github.com>
Co-authored-by: Shawn Gibbs <shawngib@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bicep Related to Bicep code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provision Azure Sentinel with the Bicep deployment
2 participants