Azure / azure-monitor-baseline-alerts

Azure Monitor Baseline Alerts

MIT License

167 stars 243 forks source link

Standardization on param usage for failingPeriods and evaluationPeriods #362

Closed Brunoga-MS closed 1 month ago

Brunoga-MS commented 1 month ago

Overview/Summary

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

This PR fixes/adds/changes/removes

Make use of parameters for failingPeriods and evaluationPeriods settings instead of hardcoded values

Breaking Changes

NONE

As part of this Pull Request I have

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