[receiver/kubeletstats] Enable re-aggregation feature#46455
[receiver/kubeletstats] Enable re-aggregation feature#46455rite7sh wants to merge 12 commits intoopen-telemetry:mainfrom
Conversation
rite7sh
requested review from
a team,
ChrsMark,
TylerHelmuth and
dmitryax
as code owners
|
Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib. Important reminders:
A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better! |
|
/workflow-approve |
|
hey @paulojmdias i am pretty sure this does not require changelog entry can it be mentioned under skip changelog lable? |
|
/workflow-approve |
|
@rite7sh I think this deserves a changelog entry, please add it |
|
/workflow-approve |
rite7sh
force-pushed
the
kubeletstats-enable-reaggregation
branch
from
e0dedd0 to
2747011
Compare
|
/workflow-approve |
|
I tried implementing the re-aggregation changes by adding |
rite7sh
force-pushed
the
kubeletstats-enable-reaggregation
branch
3 times, most recently
from
006b01d to
ebccfa8
Compare
rite7sh
force-pushed
the
kubeletstats-enable-reaggregation
branch
from
de80626 to
8021921
Compare
|
@dmitryax please let me know if the changes look good to you whenever you can :) |
|
|
||
| description: | | ||
| The Kubelet Stats Receiver pulls node, pod, container, and volume metrics from the API server on a kubelet | ||
| and sends it down the metric pipeline for further processing. |
There was a problem hiding this comment.
oh my bad, i will revert it back to as needed.
| aws.volume.id: | ||
| description: "[DEPRECATED] The id of the AWS Volume" | ||
| enabled: true | ||
| requirement_level: opt_in |
There was a problem hiding this comment.
We don't need requirement_level on resource_attributes yet. Only on attributes
|
It needs another merge from main and re-generating the go files |
rite7sh
force-pushed
the
kubeletstats-enable-reaggregation
branch
from
b8bf339 to
d138e00
Compare
|
@dmitryax done with the requested changes hopefully its good to go :)? |
|
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
rite7sh
force-pushed
the
kubeletstats-enable-reaggregation
branch
from
ac0b951 to
316012a
Compare
3 participants
Description
Enables re-aggregation support for the kubeletstats receiver by updating metadata configuration.
This change adds the required reaggregation block and classifies all resource attributes and metric attributes with appropriate aggregation levels. The update is limited strictly to metadata and does not modify any metric definitions, names, or stability levels.
Link to tracking issue
Fixes #46363
Testing
Ran receiver module tests:
go test ./receiver/kubeletstatsreceiver/...Ran vet checks:
go vet ./receiver/kubeletstatsreceiver/...Verified metadata builds and parses correctly with no schema or structural errors.
No additional tests were required as this change only affects metadata configuration.
Documentation
No documentation changes required.
This update only enables re-aggregation support via metadata and does not change user-facing configuration or receiver behavior beyond aggregation capability.