Skip to content

feat: adds CollectionAgent scaffolding#124

Open
Nageshbansal wants to merge 9 commits into
mainfrom
feat/casting/ca/scaffolding
Open

feat: adds CollectionAgent scaffolding#124
Nageshbansal wants to merge 9 commits into
mainfrom
feat/casting/ca/scaffolding

Conversation

@Nageshbansal
Copy link
Copy Markdown
Member

@Nageshbansal Nageshbansal commented May 22, 2026

Summary

Scaffolds the CollectionAgent Kind on the Planner contract from #122. Lands the API types, the agent-kind collector molding, a list-aware config merge primitive, and a shared pourer staging buffer.

Changes

  • API: CollectorStatus carries the collector config as a config map; adds CollectorKind.ConfigKey and TypeConfig.Set.
  • CollectionAgent Casting interface + Planner route Forge/Cast through a *pourer.Pourer.
  • Collector molding under internal/molding/collectionagent/collectormolding/ renders the base agent OTel config and merges the enricher's contribution at the kind's config key.
  • New internal/pourer package: staging buffer that hides domain.Material from castings.
  • domain.Merge / domain.MergeYAML: deep-merge with a per-path ListMerge strategy (k8s listType), reusable across moldings.

Related: https://github.com/SigNoz/platform-pod/issues/1971, https://github.com/SigNoz/platform-pod/issues/1972

Base automatically changed from feat/casting/collection-agent to main May 26, 2026 10:02
Comment thread api/v1alpha1/collectionagent/collector.go Outdated
- domain: add Merge/MergeYAML with a per-path ListMerge strategy (k8s listType)
- pourer: add staging buffer a casting fills during Forge
- collectormolding: render the base config and merge the enricher's
  contribution at the kind's config key
- api: add TypeConfig.Set and CollectorKind.ConfigKey
@Nageshbansal Nageshbansal force-pushed the feat/casting/ca/scaffolding branch from e1908d4 to 637a889 Compare May 29, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants