Skip to content

Conversation

DaveCTurner
Copy link
Contributor

Because of #93575 it's not sufficient to mark repositories with
readonly: true while taking a backup. The only safe way to avoid
writes is to completely unregister them.

Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
@DaveCTurner DaveCTurner added >docs General docs changes :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v9.0.0 v8.15.4 v8.16.1 v8.17.0 labels Oct 18, 2024
@DaveCTurner DaveCTurner requested a review from ywangd October 18, 2024 03:39
Copy link
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine added Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. Team:Docs Meta label for docs team labels Oct 18, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner DaveCTurner added the auto-backport Automatically create backport pull requests when merged label Oct 18, 2024
@DaveCTurner DaveCTurner merged commit f718c56 into elastic:main Oct 18, 2024
6 checks passed
@DaveCTurner DaveCTurner deleted the 2024/10/18/snapshot-repo-backup-reword branch October 18, 2024 06:51
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Oct 18, 2024
Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Oct 18, 2024
Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Oct 18, 2024
Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Oct 18, 2024
Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.15
7.17
8.16
8.x

elasticsearchmachine pushed a commit that referenced this pull request Oct 18, 2024
Because of #93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
elasticsearchmachine pushed a commit that referenced this pull request Oct 18, 2024
Because of #93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
elasticsearchmachine pushed a commit that referenced this pull request Oct 18, 2024
Because of #93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
lkts pushed a commit to lkts/elasticsearch that referenced this pull request Oct 18, 2024
Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
georgewallace pushed a commit to georgewallace/elasticsearch that referenced this pull request Oct 25, 2024
Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
elasticsearchmachine pushed a commit that referenced this pull request Oct 28, 2024
Because of #93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request Nov 4, 2024
Because of elastic#93575 it's not sufficient to mark repositories with
`readonly: true` while taking a backup. The only safe way to avoid
writes is to completely unregister them.
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jun 6, 2025
It's now ok to mark a repository as `readonly: true` while taking a
backup, so this commit reinstates the docs that say this.

Relates elastic#93575
Reverts elastic#115062
Reverts elastic#115071
Relates elastic#128296
DaveCTurner added a commit to DaveCTurner/elastic-docs-content that referenced this pull request Jun 6, 2025
It's now ok to mark a repository as `readonly: true` while taking a
backup, so this commit reinstates the docs that say this.

Relates elastic/elasticsearch#93575
Reverts elastic/elasticsearch#115062
Reverts elastic/elasticsearch#115071
Relates elastic/elasticsearch#128296
Relates elastic/elasticsearch#129040
DaveCTurner added a commit to elastic/docs-content that referenced this pull request Jun 6, 2025
It's now ok to mark a repository as `readonly: true` while taking a
backup, so this commit reinstates the docs that say this.

Relates elastic/elasticsearch#93575
Reverts elastic/elasticsearch#115062
Reverts elastic/elasticsearch#115071
Relates elastic/elasticsearch#128296
Relates elastic/elasticsearch#129040
elasticsearchmachine pushed a commit that referenced this pull request Jun 6, 2025
It's now ok to mark a repository as `readonly: true` while taking a
backup, so this commit reinstates the docs that say this.

Relates #93575 Reverts #115062 Reverts #115071 Relates #128296
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >docs General docs changes Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. Team:Docs Meta label for docs team v7.17.26 v8.15.4 v8.16.1 v8.17.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants