Skip to content

Conversation

mashhurs
Copy link
Contributor

This PR adds restrictions/acceptance note on Logstash pipeline ID, used through pipeline create REST API.

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against main? Unless there is a good reason otherwise, we prefer pull requests against main and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS and architecture that we support?
  • If you are submitting this code for a class then read our policy for that.

Copy link
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v8.19.5 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Sep 26, 2025
@mashhurs mashhurs self-assigned this Sep 26, 2025
@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Sep 26, 2025
@mashhurs mashhurs added the >docs General docs changes label Sep 26, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the Team:Docs Meta label for docs team label Sep 26, 2025
@mashhurs mashhurs added auto-backport Automatically create backport pull requests when merged v8.18.8 labels Sep 26, 2025
Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

LGTM, might want @karenzone to keep me honest though :)

@mashhurs
Copy link
Contributor Author

mashhurs commented Oct 1, 2025

FYI: main and 9.x changes - elastic/elasticsearch-specification#5354

@mashhurs mashhurs merged commit 83072b0 into elastic:8.19 Oct 1, 2025
6 of 7 checks passed
@mashhurs
Copy link
Contributor Author

mashhurs commented Oct 1, 2025

💔 Some backports could not be created

Status Branch Result
main Conflict resolution was aborted by the user
8.18

Manual backport

To create the backport manually run:

backport --pr 135557

Questions ?

Please refer to the Backport tool documentation

@mashhurs
Copy link
Contributor Author

mashhurs commented Oct 1, 2025

Who added these labels?

(Also, good to see you!)

👋 @justzh,
I labelled them but feel free to adjust if it requires.

Also, I am not sure automation didn't backport to 8.18, manually raised PR - #135790

@mashhurs mashhurs deleted the update-logstash-pipeline-id-doc-desc branch October 1, 2025 17:49
mashhurs added a commit to mashhurs/elasticsearch that referenced this pull request Oct 1, 2025
mashhurs added a commit that referenced this pull request Oct 1, 2025
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 >docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team :Logstash/CPM Team:Docs Meta label for docs team Team:Logstash Logstash team v8.18.8 v8.19.5
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants