[Open Data] Update commitment discount eligibility (September 2026) - #2299
Conversation
microsoft-github-policy-service
Bot
added
Needs: Review 👀
There was a problem hiding this comment.
🟡 Changes recommended
The PR description asserts “no meters removed” and “no eligibility flips,” but the updated CSV shows at least one meter ID missing and at least two existing meter IDs with changed eligibility, so either the data generation or the PR description needs correction.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR updates the open data snapshot for commitment discount eligibility (generated from the Azure Retail Prices API) and refreshes associated rollup counts used for reporting/validation.
Changes:
- Adds new meter eligibility rows to
CommitmentDiscountEligibility.csv. - Updates service-family rollup counts in
CommitmentDiscountEligibility.familycounts.json. - Includes at least one removed meter ID and at least two eligibility changes for existing meter IDs, which conflicts with the PR description’s “additive-only / no flips” claims.
File summaries
| File | Description |
|---|---|
| src/open-data/CommitmentDiscountEligibility.familycounts.json | Updates rollup counts (e.g., Compute counts) to reflect the latest eligibility dataset. |
| src/open-data/CommitmentDiscountEligibility.csv | Updates the eligibility table with newly added meter IDs and other row-level changes. |
Review details
- Files reviewed: 1/2 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Michael Flanakin (flanakin)
added
the
Status: ▶️ Ready
ac8daaf
into
dev
GitHub Actions
Inactive
|
All Contributors (@all-contributors) please add Roland Krummenacher (@RolandKrummenacher) for code |
|
@microsoft-github-policy-service[bot] Roland Krummenacher (@RolandKrummenacher) already contributed before to code |
Summary
Automated update of commitment discount eligibility data from the Azure Retail Prices API, produced by the Update Commitment Discount Eligibility workflow (run 33650494002).
Net +269 meters (92,624 → 92,893), composed of:
Resulting distribution:
The three non-additive rows, verified against the live API
All three were checked directly against
prices.azure.comwithapi-version=2023-01-01-previewandmeterRegion='primary'— the same parameters the fetch script uses.savingsPlanis only returned on that api-version, so a check without it reports no savings plan for every meter.99031618-6869-5409-9d5d-617a392fa088dc5d07a3-7915-5e95-921d-8beabf3c0c5ad0e8cf9a-e52a-5e56-ae17-318d9a1ad5c1The two VM meters gained savings plan pricing upstream, so
Eligible / Eligibleis now correct anddevis stale. The Foundry Models meter no longer has reservation or savings plan pricing, so it is eligible for neither; the script intentionally drops such meters rather than emittingNot Eligible / Not Eligible("Meters not seen this run are dropped (so retired meters age out instead of accumulating forever)"). That combination does not exist anywhere in the dataset, before or after.Notes
This run was dispatched manually rather than using the branch from the 2026-08-31 scheduled run (
opendata/commitment-eligibility-20260831-29). That branch was cut before #2284 merged, so it still carried the pre-swap column orientation and would have reverted the FOCUSCommitmentDiscountCategoryalignment across ~36k rows. It has been deleted, along with five older superseded snapshot branches.Test plan
🤖 Generated by the Update Commitment Discount Eligibility workflow