Skip to content
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

🌊 Add support for existing data streams #202057

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

🌊 Add support for existing data streams #202057

wants to merge 2 commits into from
+62 −6

Conversation

simianhacker
Copy link
Member

🍒 Summary

This PR introduces the initial support for working with existing data streams. This is done by reading the /_data_steram API endpoint then converting those results to stream definitions with the managed flag set to false, and then mixing them in with the "managed" streams results. This PR has the following changes:

  • Add managed field to the Stream definition
  • Set managed: true on streams created through the system
  • Update listStreams to return both managed and un-managed streams
  • Update readStream to fallback to "un-managed" stream if the managed stream is not found

@simianhacker simianhacker marked this pull request as ready for review November 27, 2024 21:53
@simianhacker simianhacker added v9.0.0 backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) v8.18.0 Team:Observability Team label for Observability Team (for things that are handled across all of observability) labels Nov 27, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/unified-observability (Team:Observability)

@simianhacker simianhacker added the release_note:skip Skip the PR/issue when compiling release notes label Nov 27, 2024
@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 27, 2024 •
edited
Loading

💔 Build Failed

Failed CI Steps

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@flash1293 flash1293 Awaiting requested review from flash1293 flash1293 is a code owner

@dgieselaar dgieselaar Awaiting requested review from dgieselaar dgieselaar is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Observability Team label for Observability Team (for things that are handled across all of observability) v8.18.0 v9.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@simianhacker @elasticmachine