Skip to content

[8.x] 🌊 Streams: Prepare API for publishing (#213127)#214415

Merged
flash1293 merged 3 commits intoelastic:8.xfrom
flash1293:backport/8.x/pr-213127
Mar 13, 2025
Merged

[8.x] 🌊 Streams: Prepare API for publishing (#213127)#214415
flash1293 merged 3 commits intoelastic:8.xfrom
flash1293:backport/8.x/pr-213127

Conversation

@flash1293
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Add streams API to documentation as an experimental feature

<img width="2555" alt="Screenshot 2025-03-07 at 11 44 54"
src="https://github.com/user-attachments/assets/f54e5e6e-0c20-4bad-9cff-27747d0f76e2"
/>

There are a couple of changes in here:
* Split streams API in internal and public and mark the public parts as
experimental
* Add the public parts to the Kibana documentation
* Add description and summary
* Adjust the server repository wrapper to pass through summary and
description

# To test

* Generate OAS bundle: `node scripts/capture_oas_snapshot --include-path
/api/streams --update`
* Apply overlays `cd oas_docs && make api-docs`
* Make sure bump.sh is installed (`npm install -g bump-cli`)
* Run for preview: `cd oas_docs && bump preview output/kibana.yaml`

# Open questions

* Does the split into public and internal make sense?
* Is it a problem if this is visible in the user-facing documentation
page before we actually release streams? Or would it be OK if the API is
marked as experimental? (mostly a question for @LucaWintergerst )

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 4681b6c)

# Conflicts:
#	.buildkite/scripts/steps/checks/capture_oas_snapshot.sh
#	oas_docs/bundle.serverless.json
#	oas_docs/overlays/kibana.overlays.serverless.yaml
#	x-pack/platform/plugins/shared/streams/server/routes/streams/crud/route.ts
#	x-pack/platform/plugins/shared/streams/server/routes/streams/management/route.ts
@flash1293 flash1293 added the backport This PR is a backport of another PR label Mar 13, 2025
…t --include-path /api/status --include-path /api/alerting/rule/ --include-path /api/alerting/rules --include-path /api/actions --include-path /api/security/role --include-path /api/spaces --include-path /api/dashboards --update --no-serverless'
@flash1293 flash1293 enabled auto-merge (squash) March 13, 2025 16:42
@flash1293 flash1293 merged commit 0f2bde5 into elastic:8.x Mar 13, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
streamsApp 367.5KB 367.7KB +199.0B

History

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

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants