Skip to content

feat(hubs): Draft FOCUS 1.5 SkuPrices preview - #2339

Draft
Michael Flanakin (flanakin) wants to merge 8 commits into
devfrom
flanakin/focus15-skuprices
Draft

Michael Flanakin (flanakin) wants to merge 8 commits into
devfrom
flanakin/focus15-skuprices

Conversation

@flanakin

@flanakin Michael Flanakin (flanakin) commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

🤖 [AI]

🛠️ Description

Draft, unwired preview of the FOCUS 1.5 SKU Price dataset (FOCUS PR 2424) built from Prices_final_v1_2. Parked here to pick up later; not ready to merge.

  • SkuPrices_v1_5: one row per SkuPriceId x price type (x_UnitPriceType: List, Base, Contracted, Effective)
  • SkuPricesWide_v1_5: same prices with price types as columns, for comparison
  • Validation script (18/19 checks pass) and 10 side-by-side scenarios (10/10 match)
  • Size on ftk-dev: full 14-month history ≈ 4.17M rows / ~0.70 GB vs. 18.8M rows / 2.51 GB in Prices_final_v1_2

Details, mapping decisions, results, and open gaps: src/templates/finops-hub/modules/Microsoft.FinOpsHubs/Analytics/preview/SkuPrices_v1_5/README.md

Known gaps: empty PricingUnit (no PricingUnits open data on ftk-dev), savings plan list price lost on ingestion, current prices only (no replaced prices), MCA untested, not wired into app.bicep / .build.config.

Fixes #

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests
  • 🤞 PS -WhatIf / az validate
  • 👍 Manually deployed + verified
  • 💪 Unit tests
  • 🙌 Integration tests

📦 Deploy to test?

  • Hubs + ADX (managed)
  • Hubs + Fabric (manual) — URI:
  • Hubs (manual)
  • Hubs (no data)
  • Workbooks
  • Alerts

🙋‍♀️ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

  • ✅ Updated changelog (required for dev PRs)
  • ➡️ Will add log in a future PR (feature branch PRs only)
  • ❎ Log not needed (small/internal change)

📖 Did you update documentation?

  • ✅ Public docs in docs (required for dev)
  • ✅ Public docs in docs-mslearn (required for dev)
  • ✅ Internal dev docs in docs-wiki (required for dev)
  • ✅ Internal dev docs in src (required for dev)
  • ➡️ Will add docs in a future PR (feature branch PRs only)
  • ❎ Docs not needed (small/internal change)

🤖 Generated with Claude Code

Adds a preview (unwired) FOCUS 1.5 SKU Price dataset built from Prices_final_v1_2,
a one-row-per-price comparison table, validation, and 10 side-by-side scenarios.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 8, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update the changelog in a future PR, but this PR is going into the dev branch. All dev branch PRs must have changelog entries to be ready for the next release. Either update the PR to target a feature branch or add a changelog entry to this PR.

@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update documentation in a future PR, but this PR is going into the dev branch. All dev branch PRs must have applicable documentation updated to be ready for the next release. Either update the PR to target a feature branch or update the documentation in this PR.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 8, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update the changelog in a future PR, but this PR is going into the dev branch. All dev branch PRs must have changelog entries to be ready for the next release. Either update the PR to target a feature branch or add a changelog entry to this PR.

@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update documentation in a future PR, but this PR is going into the dev branch. All dev branch PRs must have applicable documentation updated to be ready for the next release. Either update the PR to target a feature branch or update the documentation in this PR.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 8, 2026
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Hubs + ADX (managed) deployment failed. View logs.

@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Hubs + ADX (managed) deployed to pr-2339-adx.

Adds CostsWithSkuPriceIdv2 (Costs copy with SkuPriceIdv2 looked up by price type, meter, offer, and term),
real SkuPriceId + ContractId joins in S08/S09, 3 more PR 2595 scenarios (S11-S13), block-size handling,
and fixes stripped quotes in Build-SkuPrices.ps1.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 8, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update the changelog in a future PR, but this PR is going into the dev branch. All dev branch PRs must have changelog entries to be ready for the next release. Either update the PR to target a feature branch or add a changelog entry to this PR.

@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update documentation in a future PR, but this PR is going into the dev branch. All dev branch PRs must have applicable documentation updated to be ready for the next release. Either update the PR to target a feature branch or update the documentation in this PR.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 8, 2026
Adds S14 price backfill, unit-aware cost joins, server-measured CPU/memory/rows scanned (median of 3),
and per-measure winners. Optimizes queries (lookup, semi-joins, distinct eligibility evaluation).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 8, 2026
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Hubs + ADX (managed) deployed to pr-2339-adx.

…s HTML)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 16, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update the changelog in a future PR, but this PR is going into the dev branch. All dev branch PRs must have changelog entries to be ready for the next release. Either update the PR to target a feature branch or add a changelog entry to this PR.

@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update documentation in a future PR, but this PR is going into the dev branch. All dev branch PRs must have applicable documentation updated to be ready for the next release. Either update the PR to target a feature branch or update the documentation in this PR.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 16, 2026
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 16, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update the changelog in a future PR, but this PR is going into the dev branch. All dev branch PRs must have changelog entries to be ready for the next release. Either update the PR to target a feature branch or add a changelog entry to this PR.

@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update documentation in a future PR, but this PR is going into the dev branch. All dev branch PRs must have applicable documentation updated to be ready for the next release. Either update the PR to target a feature branch or update the documentation in this PR.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hubs + ADX (managed) deployment failed. View logs.

@github-actions

Copy link
Copy Markdown
Contributor

Hubs + ADX (managed) deployment failed. View logs.

@github-actions

Copy link
Copy Markdown
Contributor

Hubs + ADX (managed) deployed to pr-2339-adx.

@flanakin Michael Flanakin (flanakin) added this to the v16 milestone Oct 26, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 26, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update the changelog in a future PR, but this PR is going into the dev branch. All dev branch PRs must have changelog entries to be ready for the next release. Either update the PR to target a feature branch or add a changelog entry to this PR.

@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update documentation in a future PR, but this PR is going into the dev branch. All dev branch PRs must have applicable documentation updated to be ready for the next release. Either update the PR to target a feature branch or update the documentation in this PR.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 26, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 26, 2026
@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update the changelog in a future PR, but this PR is going into the dev branch. All dev branch PRs must have changelog entries to be ready for the next release. Either update the PR to target a feature branch or add a changelog entry to this PR.

@microsoft-github-policy-service

Copy link
Copy Markdown

@Michael Flanakin (@flanakin), I noticed you said you would update documentation in a future PR, but this PR is going into the dev branch. All dev branch PRs must have applicable documentation updated to be ready for the next release. Either update the PR to target a feature branch or update the documentation in this PR.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Needs: Documentation 📝 PR needs a changelog entry, public docs, or internal docs before it can be closed label Oct 26, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hubs + ADX (managed) deployed to pr-2339-adx.

This branch was successfully deployed

1 active deployment
ftk-pr — c6730964 Deployed Oct 26, 2026 by flanakin via Hubs + ADX (managed) #639
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Review 👀 PR that is ready to be reviewed Tool: FinOps hubs Data pipeline solution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants