Skip to content

Add extstore s3 mrap + crr documentation.#4622

Open
cconstable wants to merge 9 commits into
mainfrom
extstore-s3-mrap
Open

Add extstore s3 mrap + crr documentation.#4622
cconstable wants to merge 9 commits into
mainfrom
extstore-s3-mrap

Conversation

@cconstable
Copy link
Copy Markdown

@cconstable cconstable commented May 26, 2026

What does this PR do?

  • Adds documentation for add making S3 external storage more durable with MRAP and CRR.
  • Discusses how this relates to Temporal Cloud's High Availability offering.

Notes to reviewers

  • I am currently validating this with a real-world example. Will comment back with the result.

@cconstable cconstable requested a review from a team as a code owner May 26, 2026 20:08
@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
temporal-documentation Ready Ready Preview, Comment May 28, 2026 3:08pm

Request Review

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 26, 2026

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 26, 2026

📖 Docs PR preview links

of size.
payload data in infrastructure they control. Set the offload size threshold to zero to externalize all payloads
regardless of size.
- **Resilience to regional failures.** External Storage can be configured for cross-region durability so that payloads
Copy link
Copy Markdown
Contributor

@lennessyy lennessyy May 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a real benefit they get from using external storage 🤔 I feel like what we added allows their payload storage to be resistant to regional failures if they are already using external storage, but using external storage by itself doesn't really add resilience to their workloads vs not using it, so this reads a bit misleading to me

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. I'll remove this addition.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated.

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.

3 participants