Skip to content

Handle 404s from the _cat/indices api in FullClusterRestartIT #122537

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
joegallo merged 7 commits into from
Feb 14, 2025
Merged

Handle 404s from the _cat/indices api in FullClusterRestartIT #122537

joegallo merged 7 commits into from
Feb 14, 2025

Conversation

@joegallo
Copy link
Contributor

@joegallo joegallo commented Feb 13, 2025

Closes #121115

This is a workaround for #104371

@joegallo joegallo added >test Issues or PRs that are addressing/adding tests :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team auto-backport Automatically create backport pull requests when merged v9.0.0 v8.18.0 v8.19.0 v9.1.0 labels Feb 13, 2025
@joegallo joegallo requested a review from masseyke February 13, 2025 20:19
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Feb 13, 2025

Pinging @elastic/es-data-management (Team:Data Management)

@joegallo joegallo mentioned this pull request Feb 13, 2025
Merged
masseyke
masseyke approved these changes Feb 13, 2025
View reviewed changes
Copy link
Member

@masseyke masseyke left a comment

Choose a reason for hiding this comment

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

LGTM

@joegallo joegallo requested a review from masseyke February 13, 2025 22:59
@joegallo joegallo merged commit 5c00341 into elastic:main Feb 14, 2025
17 checks passed
@joegallo joegallo deleted the handle-404s-from-cat-apis branch February 14, 2025 02:14
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Feb 14, 2025

💔 Backport failed

Status Branch Result
❌ 9.0 Commit could not be cherrypicked due to conflicts
❌ 8.18 Commit could not be cherrypicked due to conflicts
❌ 8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 122537

@joegallo joegallo mentioned this pull request Feb 14, 2025
Closed
joegallo added a commit to joegallo/elasticsearch that referenced this pull request Feb 14, 2025
@joegallo
Handle 404s from the _cat/indices api in FullClusterRestartIT (elasti…
aa3152d 
…c#122537)
joegallo added a commit to joegallo/elasticsearch that referenced this pull request Feb 14, 2025
@joegallo
Handle 404s from the _cat/indices api in FullClusterRestartIT (elasti…
d7938c0 
…c#122537)
joegallo added a commit to joegallo/elasticsearch that referenced this pull request Feb 14, 2025
@joegallo
Handle 404s from the _cat/indices api in FullClusterRestartIT (elasti…
ba20790 
…c#122537)
@joegallo joegallo mentioned this pull request Feb 14, 2025
Merged
This was referenced Feb 14, 2025
Merged
Merged
@joegallo
Copy link
Contributor Author

joegallo commented Feb 14, 2025

Okay, manual backports are up at #122562, #122563, and #122564.

elasticsearchmachine pushed a commit that referenced this pull request Feb 14, 2025
@joegallo
Handle 404s from the _cat/indices api in FullClusterRestartIT (#122537)…
ed36694 
… (#122564)
joegallo added a commit that referenced this pull request Feb 14, 2025
@joegallo
Handle 404s from the _cat/indices api in FullClusterRestartIT (#122537)…
bf74b66 
… (#122563)
elasticsearchmachine pushed a commit that referenced this pull request Feb 14, 2025
@joegallo
Handle 404s from the _cat/indices api in FullClusterRestartIT (#122537)…
2a693a9 
… (#122562)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@masseyke masseyke masseyke approved these changes

Assignees

No one assigned

Labels

auto-backport Automatically create backport pull requests when merged :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v8.18.0 v8.19.0 v9.0.0 v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CI] FullClusterRestartIT testGeoIpSystemFeaturesMigration {cluster=UPGRADED} failing

3 participants

@joegallo @elasticsearchmachine @masseyke