-
Notifications
You must be signed in to change notification settings - Fork 76
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.
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
fix(k8s-infra): enable k8sattributes to match other information #488
Conversation
This should close #460 |
ae7826b
to
236ab52
Compare
WalkthroughThis pull request introduces updates to the Changes
Possibly related PRs
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Outside diff range and nitpick comments (1)
charts/k8s-infra/values.yaml (1)
185-191
: LGTM! Consider assessing the impact of additional metadata on performance and storage.Extracting additional metadata fields will provide more comprehensive monitoring and observability of the Kubernetes environment. However, please assess the impact of this change on the system's performance and storage requirements, as it may increase the size of the collected telemetry data.
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (4)
- charts/k8s-infra/Chart.yaml (1 hunks)
- charts/k8s-infra/templates/_config.tpl (1 hunks)
- charts/k8s-infra/templates/otel-agent/daemonset.yaml (1 hunks)
- charts/k8s-infra/values.yaml (3 hunks)
Files skipped from review due to trivial changes (1)
- charts/k8s-infra/Chart.yaml
Additional comments not posted (4)
charts/k8s-infra/templates/otel-agent/daemonset.yaml (1)
Line range hint
102-106
: LGTM! The changes align with the PR objective and should enhance observability.The removal of
k8s.pod.uid
andk8s.pod.ip
attributes and the inclusion ofk8s.node.name
attribute instead should resolve the interference issue with the k8sprocessor and improve its ability to match and enrich telemetry data accurately.Shifting the focus to node-specific attributes may provide a more stable and consistent view of the system, as nodes are less ephemeral compared to pods. This change should enhance the overall observability without negatively impacting the flexibility to append additional resource attributes based on user-defined presets.
charts/k8s-infra/templates/_config.tpl (1)
367-370
: LGTM! Thek8snode
detector is a valuable addition.The new
k8snode
detector in theresourcedetection/internal
processor enables automatic detection and addition of Kubernetes node metadata to the collected telemetry data. This enhances the context and usefulness of the telemetry for analysis and troubleshooting.The use of the
K8S_NODE_NAME
environment variable provides flexibility in specifying the node name, while theserviceAccount
authentication type ensures secure communication with the Kubernetes API server.charts/k8s-infra/values.yaml (2)
126-127
: Verify compatibility of thecontainer-parser
with different logging formats.The change to introduce a single
container-parser
aligns with the PR objective to streamline the logging configuration. However, please verify that this single parser can handle all the different container logging formats (Docker, CRI-O, Containerd) that were previously handled by separate parsers.
244-244
: Review release notes and verify compatibility before deploying the OpenTelemetry collector image upgrade.Upgrading to version
0.109.0
of the OpenTelemetry collector image is beneficial to leverage the latest features and bug fixes. However, please review the release notes of this version to understand the specific changes and their potential impact on the existing setup. Also, verify the compatibility of the new version with other components in the system before deploying this change.
Summary
New Features
k8snode
detector to add k8s.node.uid for hostmetrics.Fixes
Chores