Skip to content

Make admin storage pricing region-aware and sortable - #1153

Merged
benben merged 1 commit into
mainfrom
feat/admin-region-pricing-sort
Sep 3, 2026
Merged

Make admin storage pricing region-aware and sortable#1153
benben merged 1 commit into
mainfrom
feat/admin-region-pricing-sort

Conversation

@benben

@benben benben commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

  • select the storage customer-pricing schedule from the control plane's effective AWS region and remove the manual region switch
  • make every storage-economics column sortable, including gross margin
  • add per-organization and total prices needed to reach an 80% storage gross margin
  • document region selection, pricing behavior, local setup, and failure recovery

Pricing model

The 80%-margin what-if price is calculated from the unrounded allocated AWS cost:

target price = allocated cost / (1 - 0.80) = 5 * allocated cost

Tests

  • just ui-test
  • GOFLAGS=-run=TestMonthlyUsageReportsDeploymentPricingRegion just test-controlplane-k8s
  • just lint (currently reaches six pre-existing Staticcheck SA5011 findings in unrelated test files)

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

Test Impact Plan

Deterministic summary of how this PR changes tests, CI runners, and coverage-risk signals.

Summary

Area Added Changed Deleted
Test files 0 1 0
E2E/journey files 0 0 0
Workflow files 0 0 0

Signals

  • Test cases: +1 / -0
  • Assertions: +2 / -0
  • Skips or known failures added: 0
  • Workflow continue-on-error added: 0
  • Workflow path filters added: 0
  • Test commands removed from justfile: 0
  • E2E/journey retry lines added: 0

Coverage risk: neutral or increased

No coverage-reduction warnings detected.

@benben
benben merged commit b5ee4b0 into main Sep 3, 2026
35 checks passed
@benben
benben deleted the feat/admin-region-pricing-sort branch September 3, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant