Skip to content

🌊 Streams: Fix classic streams ingest pipeline manipulation#230053

Merged
flash1293 merged 4 commits intoelastic:mainfrom
flash1293:flash1293/fix-streams-test
Jul 31, 2025
Merged

🌊 Streams: Fix classic streams ingest pipeline manipulation#230053
flash1293 merged 4 commits intoelastic:mainfrom
flash1293:flash1293/fix-streams-test

Conversation

@flash1293
Copy link
Contributor

Fixes #229733

With elastic/elasticsearch#130847 , we track the creation and modification date of ingest pipelines.
As part of adjusting processing through the streams API, the ingest pipeline object is loaded, modified and written back. This fails since it tries to set the modified/creation timestamps which isn't allowed.

This PR is stripping out the offending keys

@flash1293 flash1293 added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Jul 31, 2025
@flash1293 flash1293 requested a review from a team as a code owner July 31, 2025 10:29
@flash1293 flash1293 added Team:obs-onboarding Observability Onboarding Team Feature:Streams This is the label for the Streams Project v9.2.0 labels Jul 31, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

@flash1293 flash1293 enabled auto-merge (squash) July 31, 2025 11:07
@flash1293 flash1293 merged commit 4561993 into elastic:main Jul 31, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

The CI Stats report is too large to be displayed here, check out the CI build annotation for this information.

delanni pushed a commit to delanni/kibana that referenced this pull request Aug 5, 2025
…230053)

Fixes elastic#229733

With elastic/elasticsearch#130847 , we track the
creation and modification date of ingest pipelines.
As part of adjusting processing through the streams API, the ingest
pipeline object is loaded, modified and written back. This fails since
it tries to set the modified/creation timestamps which isn't allowed.

This PR is stripping out the offending keys
@wildemat wildemat mentioned this pull request Aug 7, 2025
10 tasks
@flash1293
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

flash1293 added a commit to flash1293/kibana that referenced this pull request Sep 12, 2025
…230053)

Fixes elastic#229733

With elastic/elasticsearch#130847 , we track the
creation and modification date of ingest pipelines.
As part of adjusting processing through the streams API, the ingest
pipeline object is loaded, modified and written back. This fails since
it tries to set the modified/creation timestamps which isn't allowed.

This PR is stripping out the offending keys

(cherry picked from commit 4561993)

# Conflicts:
#	x-pack/platform/plugins/private/translations/translations/de-DE.json
flash1293 added a commit that referenced this pull request Sep 12, 2025
…230053) (#234916)

Fixes #232445

# Backport

This will backport the following commits from `main` to `8.19`:
- [🌊 Streams: Fix classic streams ingest pipeline manipulation
(#230053)](#230053)

<!--- Backport version: 10.0.2 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Joe
Reuter","email":"johannes.reuter@elastic.co"},"sourceCommit":{"committedDate":"2025-07-31T12:58:52Z","message":"🌊
Streams: Fix classic streams ingest pipeline manipulation
(#230053)\n\nFixes
https://github.com/elastic/kibana/issues/229733\n\nWith
elastic/elasticsearch#130847 , we track
the\ncreation and modification date of ingest pipelines.\nAs part of
adjusting processing through the streams API, the ingest\npipeline
object is loaded, modified and written back. This fails since\nit tries
to set the modified/creation timestamps which isn't allowed.\n\nThis PR
is stripping out the offending
keys","sha":"4561993a2d8d50773ee1a2f1bf4ef7dca7908944","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:skip","Team:obs-ux-logs","Feature:Streams","v9.2.0"],"title":"🌊
Streams: Fix classic streams ingest pipeline
manipulation","number":230053,"url":"https://github.com/elastic/kibana/pull/230053","mergeCommit":{"message":"🌊
Streams: Fix classic streams ingest pipeline manipulation
(#230053)\n\nFixes
https://github.com/elastic/kibana/issues/229733\n\nWith
elastic/elasticsearch#130847 , we track
the\ncreation and modification date of ingest pipelines.\nAs part of
adjusting processing through the streams API, the ingest\npipeline
object is loaded, modified and written back. This fails since\nit tries
to set the modified/creation timestamps which isn't allowed.\n\nThis PR
is stripping out the offending
keys","sha":"4561993a2d8d50773ee1a2f1bf4ef7dca7908944"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/230053","number":230053,"mergeCommit":{"message":"🌊
Streams: Fix classic streams ingest pipeline manipulation
(#230053)\n\nFixes
https://github.com/elastic/kibana/issues/229733\n\nWith
elastic/elasticsearch#130847 , we track
the\ncreation and modification date of ingest pipelines.\nAs part of
adjusting processing through the streams API, the ingest\npipeline
object is loaded, modified and written back. This fails since\nit tries
to set the modified/creation timestamps which isn't allowed.\n\nThis PR
is stripping out the offending
keys","sha":"4561993a2d8d50773ee1a2f1bf4ef7dca7908944"}}]}] BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:Streams This is the label for the Streams Project release_note:skip Skip the PR/issue when compiling release notes Team:obs-onboarding Observability Onboarding Team v8.19.4 v9.2.0

Projects

None yet

4 participants