Skip to content

Use LRS replication for bosh storage account#678

Merged
beyhan merged 1 commit into
mainfrom
stephanme-patch-1
May 21, 2026
Merged

Use LRS replication for bosh storage account#678
beyhan merged 1 commit into
mainfrom
stephanme-patch-1

Conversation

@stephanme
Copy link
Copy Markdown
Member

LRS (Locally redundant storage) is cheaper than GRS (geo-redundant storage).

We observed nearly $500 cost per month for "Geo-Replication v2 Data Transfer".

GRS is not necessary for bbl envs used in CF validation (the dominate use case of bbl I would say). Replication type can be overridden if bbl is used for setting up a productive foundation.

LRS (Locally redundant storage) is cheaper than GRS (geo-redundant storage).

We observed nearly $500 cost per month for "Geo-Replication v2 Data Transfer".

GRS is not necessary for bbl envs used in CF validation (the dominate use case of bbl I would say). Replication type can be overridden if bbl is used for setting up a productive foundation.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4969aace-7934-46ff-8863-de059b8eef03

📥 Commits

Reviewing files that changed from the base of the PR and between 489cb2b and c3c6727.

📒 Files selected for processing (1)
  • terraform/azure/templates/storage.tf

Walkthrough

The PR updates the Azure storage account configuration for BOSH by changing the replication strategy from Geo-Redundant Storage (GRS) to Locally Redundant Storage (LRS). This single-line change modifies the account_replication_type attribute in the azurerm_storage_account.bosh Terraform resource definition.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly describes the main change: switching BOSH storage replication from GRS to LRS, which aligns with the file modification in storage.tf.
Description check ✅ Passed The description clearly relates to the changeset by explaining the cost rationale and use case for changing from GRS to LRS replication.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch stephanme-patch-1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@beyhan beyhan merged commit a77b5da into main May 21, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants