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

Added new parameter to support CMK, removed Computers to Include parameter #521

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

tagolovina
Copy link
Contributor

@tagolovina tagolovina commented Feb 5, 2025

Overview/Summary

Replace this with a brief description of what this Pull Request fixes, changes, etc.

This PR adds/removes

  1. Added new parameter to use customer-managed storage accounts in scheduled query alert rules

  2. Removed Computers to Include parameter because of policy parameters limit.

  3. Created documentation about customer-managed storage accounts in scheduled query alert rules
    Breaking Changes

  4. No

As part of this Pull Request I have

  • Read the Contribution Guide and ensured this PR is compliant with the guide
  • Checked for duplicate Pull Requests
  • Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Ensured PR tests are passing
  • Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

@judyer28 judyer28 added the Pattern: ALZ 🚁 Issues / PR's related to the ALZ Pattern label Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pattern: ALZ 🚁 Issues / PR's related to the ALZ Pattern
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants