Skip to content

Add missing type = api to streaming parent config examples#21739

Merged
ktsaou merged 1 commit intomasterfrom
fix/streaming-docs-add-type-api
Feb 11, 2026
Merged

Add missing type = api to streaming parent config examples#21739
ktsaou merged 1 commit intomasterfrom
fix/streaming-docs-add-type-api

Conversation

@ktsaou
Copy link
Member

@ktsaou ktsaou commented Feb 11, 2026

Summary

  • The stock stream.conf marks type = api as mandatory (YOU MUST SET THIS FIELD ON ALL API KEYS), but all parent-side configuration examples in the streaming README omitted it
  • This caused AI support agents (Nedi) and users copying from docs to produce incomplete parent configurations
  • Added type = api to all 5 parent-side [API_KEY]/[UUID] examples: quick reference table, basic setup, HA setup (both keys), and the step-by-step guide

Test plan

  • Verify all parent config examples in src/streaming/README.md now include type = api
  • Verify generated integration docs render correctly

Summary by cubic

Added type = api to all Parent-side streaming configuration examples in src/streaming/README.md. This aligns with stream.conf’s mandatory requirement and prevents users from creating invalid parent configs when copying from the docs.

Written for commit 21b1d57. Summary will update on new commits.

The stock stream.conf marks type = api as mandatory ("YOU MUST SET
THIS FIELD ON ALL API KEYS"), but all examples in the README omitted
it. This caused AI support agents and users copying from docs to
produce incomplete configurations.
@ktsaou ktsaou merged commit b41cd87 into master Feb 11, 2026
22 of 23 checks passed
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Ancairon pushed a commit to Ancairon/netdata that referenced this pull request Feb 12, 2026
…1739)

Add missing type = api to all streaming parent config examples

The stock stream.conf marks type = api as mandatory ("YOU MUST SET
THIS FIELD ON ALL API KEYS"), but all examples in the README omitted
it. This caused AI support agents and users copying from docs to
produce incomplete configurations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant