Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds an Azure-focused benchmark workflow catalog (plus an orchestrator + cleanup flow) to support OPS-3941 by enabling automated discovery of cost-optimization opportunities across common Azure resources.
Changes:
- Added an orchestrator workflow to run benchmark sub-workflows and persist monthly cost timeseries.
- Added multiple Azure benchmark workflows (VMs, disks, App Service Plans, SQL DBs/elastic pools) that create Opportunities entries.
- Added a cleanup workflow intended to remove Azure benchmark-generated data.
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Run Azure Benchmark - orchestrator.json | New orchestrator that triggers cleanup + benchmark subflows, queries cost, and writes Timeseries records. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Clean-up Azure Benchmark data.json | New cleanup flow intended to delete benchmark-created Timeseries/Opportunities data. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Azure Benchmark - Unused App Services Plans.json | Detects unused App Service Plans (based on app metrics) and creates Opportunities. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Azure Benchmark - Unattached Managed Disks.json | Detects unattached disks and creates Opportunities with estimated savings. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Azure Benchmark - Single Databases.json | Detects unused single SQL databases and creates Opportunities with estimated savings. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Azure Benchmark - Rightsizing App Service Plans.json | Detects underutilized plans and proposes smaller SKU + savings. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Azure Benchmark - Idle VMs.json | Detects idle running VMs using CPU/network metrics and creates Opportunities. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Azure Benchmark - Elastic Pools.json | Detects empty/unused elastic pools and creates Opportunities with estimated savings. |
| packages/server/api/src/app/benchmark/workflows-catalog/azure/Azure Benchmark - Elastic Pool Storage.json | Detects overprovisioned elastic pool storage and creates Opportunities for downsizing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…Azure Benchmark - Rightsizing App Service Plans.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
ravikiranvm
approved these changes
Mar 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes OPS-3941.