Skip to content

Forward port changes from backport of #125562 #126413

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.

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

Merged

Conversation

DaveCTurner
Copy link
Contributor

The backport to 8.x needed some changes to pass through CI; this
commit forward-ports the relevant bits of those changes back into main
to keep the branches aligned.

The backport to `8.x` needed some changes to pass through CI; this
commit forward-ports the relevant bits of those changes back into `main`
to keep the branches aligned.
@DaveCTurner DaveCTurner added >non-issue :Core/Infra/REST API REST infrastructure and utilities v9.1.0 labels Apr 7, 2025
@DaveCTurner DaveCTurner requested a review from ldematte April 7, 2025 15:15
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Apr 7, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

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

Curious how put and delete pipeline did not show in any test failures/needed no test change -- coverage?
Anyway, LGTM

@DaveCTurner
Copy link
Contributor Author

Those APIs are in the fragile Logstash shim plugin thing which appears not to have any tests in main.

@DaveCTurner DaveCTurner merged commit f6c1965 into elastic:main Apr 7, 2025
17 checks passed
@DaveCTurner DaveCTurner deleted the 2025/04/07/empty-response-fwd-port branch April 7, 2025 18:05
mridula-s109 pushed a commit that referenced this pull request Apr 7, 2025
The backport to `8.x` needed some changes to pass through CI; this
commit forward-ports the relevant bits of those changes back into `main`
to keep the branches aligned.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/REST API REST infrastructure and utilities >non-issue Team:Core/Infra Meta label for core/infra team v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants