Skip to content

Small resiliency status update #123497

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Feb 26, 2025
Merged

Small resiliency status update #123497

merged 1 commit into from
Feb 26, 2025

Conversation

DaveCTurner
Copy link
Contributor

No description provided.

@DaveCTurner DaveCTurner added >docs General docs changes :Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. auto-backport Automatically create backport pull requests when merged v8.18.1 v8.19.0 v9.0.1 v9.1.0 v8.17.4 labels Feb 26, 2025
@github-actions GitHub Actions
Copy link
Contributor

Documentation preview:

@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.

@elasticsearchmachine elasticsearchmachine added Team:Docs Meta label for docs team Team:Distributed Indexing Meta label for Distributed Indexing team labels Feb 26, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

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

henningandersen
henningandersen approved these changes Feb 26, 2025
View reviewed changes
Copy link
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

LGTM

@leemthompo
Copy link
Contributor

leemthompo commented Feb 26, 2025 •
edited
Loading

This is internal right? (Or at least it just lives in the repo)

Can ignore the asciidoc warning

@DaveCTurner DaveCTurner added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 26, 2025
@DaveCTurner
Copy link
Contributor Author

This is internal right? (Or at least it just lives in the repo)

No, this is the source for https://www.elastic.co/guide/en/elasticsearch/resiliency/current/index.html

pugnascotia
pugnascotia reviewed Feb 26, 2025
View reviewed changes
docs/resiliency/index.asciidoc
Comment on lines +121 to +124
We have ported the known scenarios in the Jepsen blogs that check loss of
acknowledged writes to our testing infrastructure. The new tests are run
continuously in our testing farm and are passing. We will also monitor for new
failure scenarios and adapt our test suite as needed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it still fair to say that the status of "Run Jepsen" is "ongoing", if we have no real intention of doing it (AFAICT)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Eh we could probably change this heading. It's fair to say that we monitor for new failure scenarios, whether they arise from new Jepsen tests or from some other source.

@elasticsearchmachine elasticsearchmachine merged commit 4be53f5 into elastic:main Feb 26, 2025
6 of 7 checks passed
@DaveCTurner DaveCTurner deleted the 2025/02/26/resiliency-status-update branch February 26, 2025 14:49
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (elastic#123497)
d9d6962
@DaveCTurner DaveCTurner mentioned this pull request Feb 26, 2025
Merged
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (elastic#123497)
5048a4c
@DaveCTurner DaveCTurner mentioned this pull request Feb 26, 2025
Merged
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (elastic#123497)
ae235f9
@DaveCTurner DaveCTurner mentioned this pull request Feb 26, 2025
Merged
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (elastic#123497)
a251f9f
@DaveCTurner DaveCTurner mentioned this pull request Feb 26, 2025
Merged
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
✅ 8.18
✅ 8.x
✅ 9.0
✅ 8.17

@leemthompo
Copy link
Contributor

No, this is the source for https://www.elastic.co/guide/en/elasticsearch/resiliency/current/index.html

That's a weird published-in-what-looks-like-a-vaccum page and I'm not sure what the plan is for it in the new system, making a note...

All else being equal, it's still true that anything in asciidoc on main or 9.0 will be lost

elasticsearchmachine pushed a commit that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (#123497) (#123501)
ff93aa3
elasticsearchmachine pushed a commit that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (#123497) (#123499)
54dc2db
elasticsearchmachine pushed a commit that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (#123497) (#123500)
f031ba6
elasticsearchmachine pushed a commit that referenced this pull request Feb 26, 2025
@DaveCTurner
Small resiliency status update (#123497) (#123502)
50a799b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pugnascotia pugnascotia pugnascotia left review comments

@henningandersen henningandersen henningandersen approved these changes

Assignees

No one assigned

Labels

auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. >docs General docs changes Team:Distributed Indexing Meta label for Distributed Indexing team Team:Docs Meta label for docs team v8.17.4 v8.18.1 v8.19.0 v9.0.1 v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@DaveCTurner @elasticsearchmachine @leemthompo @pugnascotia @henningandersen