-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Closed
Labels
discussion-neededCommunity discussion neededCommunity discussion needed
Description
During a meeting discussing #14099, inconsistent component naming across core and contrib repos was brought to attention:
- Some components use underscores:
memory_limiter,docker_stats,tail_sampling,k8s_cluster,headers_setter,receiver_creator - Others use all-lowercase:
otlphttp,k8sattributes,awsxray,basicauth
This inconsistency affects discoverability and user experience.
Current state: https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/coding-guidelines.md#module-organization provide guidance on module/package naming (e.g., memorylimiterprocessor), but there is no guidance on component type names used in configuration files.
Questions for discussion:
- Which pattern should we establish and document as a standard: with underscore (
memory_limiter) or without (memorylimiter)? - Should we migrate existing components to match the chosen pattern, or keep them as-is for backwards compatibility?
cc @open-telemetry/collector-approvers @open-telemetry/collector-contrib-approvers
andrzej-stencel and evan-bradley
Metadata
Metadata
Assignees
Labels
discussion-neededCommunity discussion neededCommunity discussion needed