Skip to content

Remove throughput bucketing feature registration check - #2604

Merged
asier-isayas merged 3 commits into
masterfrom
users/aisayas/remove-throughputbucket-feature-reg-check
Sep 22, 2026
Merged

Remove throughput bucketing feature registration check#2604
asier-isayas merged 3 commits into
masterfrom
users/aisayas/remove-throughputbucket-feature-reg-check

Conversation

@asier-isayas

@asier-isayas asier-isayas commented Sep 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

Removes the ThroughputBucketing subscription feature-registration check and determines throughput bucket availability from the authentication and request path that supports reading and updating the settings.

Throughput bucket settings are now available for SQL containers when:

  • The user is authenticated with Microsoft Entra ID
  • ARM operations are enabled
  • Data Explorer is not running in Fabric
  • The container has dedicated throughput

The eligibility check now lives in SettingsComponent, alongside the UI and save behavior it controls.

Preview this branch

Asier Isayas
Remove throughput bucketing feature registration check
ad26ebc 
Derive throughput bucket availability in container settings from the supported AAD ARM path, and remove obsolete global and URL feature state.
@asier-isayas
asier-isayas requested a review from a team as a code owner September 21, 2026 13:41
@github-actions

github-actions Bot commented Sep 21, 2026

Copy link
Copy Markdown

Playwright tests ❌ failed

Passed Failed Flaky Duration
669 2 16 1043s

📁 Report: 35607213027-1/report.zip
Open container (Azure sign-in required) → click into 35607213027-1 folder → click report.zip → Download → unzip → open index.html · Workflow run

⚠️ 2 shard(s) failed before tests ran (infra/auth issue). Stats below reflect only shards that executed.

@github-actions

github-actions Bot commented Sep 21, 2026

Copy link
Copy Markdown

Playwright tests ❌ failed

Passed Failed Flaky Duration
672 2 14 1023s

📁 Report: 35607502762-1/report.zip
Open container (Azure sign-in required) → click into 35607502762-1 folder → click report.zip → Download → unzip → open index.html · Workflow run

⚠️ 2 shard(s) failed before tests ran (infra/auth issue). Stats below reflect only shards that executed.

@github-actions

github-actions Bot commented Sep 21, 2026

Copy link
Copy Markdown

Playwright tests ❌ failed

Passed Failed Flaky Duration
674 2 11 898s

📁 Report: 35631117614-1/report.zip
Open container (Azure sign-in required) → click into 35631117614-1 folder → click report.zip → Download → unzip → open index.html · Workflow run

⚠️ 2 shard(s) failed before tests ran (infra/auth issue). Stats below reflect only shards that executed.

@github-actions

github-actions Bot commented Sep 21, 2026

Copy link
Copy Markdown

Playwright tests ✅ passed

Passed Failed Flaky Duration
65 0 1 556s

📁 Report: 35631117614-2/report.zip
Open container (Azure sign-in required) → click into 35631117614-2 folder → click report.zip → Download → unzip → open index.html · Workflow run

@asier-isayas
asier-isayas merged commit 575ebd6 into master Sep 22, 2026
60 of 62 checks passed
@asier-isayas
asier-isayas deleted the users/aisayas/remove-throughputbucket-feature-reg-check branch September 22, 2026 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vchske vchske vchske approved these changes
@jawelton74 jawelton74 jawelton74 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@asier-isayas @vchske @jawelton74