Skip to content

tigris: document shadow write-through preservation on update - #2470

Draft
tripledoublev wants to merge 2 commits into
mainfrom
vincent/tigris-shadow-write-through-preserve-docs
Draft

tripledoublev wants to merge 2 commits into
mainfrom
vincent/tigris-shadow-write-through-preserve-docs

Conversation

@tripledoublev

@tripledoublev tripledoublev commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Clarifies that omitting --shadow-write-through on fly storage update preserves the bucket's current write-through setting, rather than resetting it to disabled.

Blocked — do not merge yet

This describes behavior from superfly/flyctl#5119, which has not merged. Today, omitting --shadow-write-through on an update still resets it to disabled. Do not merge this PR until #5119 is merged and released — otherwise this page will describe behavior that doesn't exist yet.

Opened as draft, and as a separate PR from #2469, specifically so the accurate-for-today edits there aren't held up by this one.

Scope note

This branch is stacked on #2469's commit, based on main — so the diff currently shows both PRs' changes together. It'll collapse to just this one sentence once #2469 merges; until then, treat #2469 as the source of truth for the other wording and rebase here if it changes.

Delete guidance advertised a nonexistent --force flag and had the
empty/non-empty deletion rule backwards. fly storage destroy has no
--force; it prompts for confirmation unless --yes is passed, and
deletion is unconditional regardless of bucket contents.

Shadow bucket create/update examples were missing the required
--shadow-name flag. flyctl rejects a partial shadow configuration
(all five of name, access key, secret key, endpoint, region are
required) before any provider call, so the old examples would fail
as written.
Clarifies that omitting --shadow-write-through on an update preserves
the bucket's current setting rather than resetting it.

DRAFT: depends on an in-progress flyctl fix that has not shipped yet.
Do not merge until the corresponding CLI change is released.
@tripledoublev
tripledoublev changed the base branch from main to vincent/fix-tigris-shadow-delete-docs August 24, 2026 16:23
@kcmartin

Copy link
Copy Markdown
Contributor

Heads up: the docs move to Mintlify next Wednesday, so this file becomes tigris/index.mdx and this PR won't merge cleanly afterwards. Since it's blocked on superfly/flyctl#5119 anyway, I'll carry the change into the new file once that lands and credit you. Nothing needed from you.

Base automatically changed from vincent/fix-tigris-shadow-delete-docs to main September 18, 2026 16:03
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.

2 participants