Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[receiver/k8scluster] No longer adding container state metadata on container.id #32676

Closed
dloucasfx opened this issue Apr 24, 2024 · 1 comment · Fixed by #32677
Closed

[receiver/k8scluster] No longer adding container state metadata on container.id #32676

dloucasfx opened this issue Apr 24, 2024 · 1 comment · Fixed by #32677
Labels
bug Something isn't working needs triage New item requiring triage receiver/k8scluster

Comments

@dloucasfx
Copy link
Contributor

dloucasfx commented Apr 24, 2024 •
edited

Component(s)

receiver/k8scluster

What happened?

Description

Starting in v0.80.0 and above, the k8scluster receiver is no longer capturing and sending container state metadata against the resource id container.id

Steps to Reproduce

Run the latest version of the collector with k8scluster receiver

Expected Result

Container metadata, example container.status:running should be displayed added on the container.id attribute

Actual Result

container.status:running is missing

Collector version

v0.80.0 and above

Additional context

A fix has been identified and shortly it will be linked to this issue
@dloucasfx dloucasfx added bug Something isn't working needs triage New item requiring triage labels Apr 24, 2024
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners:

%s See Adding Labels via Comments if you do not have permissions to add labels yourself.

@dloucasfx dloucasfx mentioned this issue Apr 24, 2024
Merged
dmitryax pushed a commit that referenced this issue Apr 24, 2024
@dloucasfx
[receiver/k8scluster] Fix container state metadata (#32677)
873ad9c 
**Description:** 
The
[transform](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/k8sclusterreceiver/internal/pod/pods.go#L37)
function is stripping out the container state which is needed in the
container
[GetMetadata](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/k8sclusterreceiver/internal/container/containers.go#L96)

This PR adds the state field back.

**Link to tracking Issue:**
This PR fixes
#32676

Signed-off-by: Dani Louca <dlouca@splunk.com>
rimitchell pushed a commit to rimitchell/opentelemetry-collector-contrib that referenced this issue May 8, 2024
@dloucasfx @rimitchell
[receiver/k8scluster] Fix container state metadata (open-telemetry#32677
8785cf2 
)

**Description:** 
The
[transform](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/k8sclusterreceiver/internal/pod/pods.go#L37)
function is stripping out the container state which is needed in the
container
[GetMetadata](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/k8sclusterreceiver/internal/container/containers.go#L96)

This PR adds the state field back.

**Link to tracking Issue:**
This PR fixes
open-telemetry#32676

Signed-off-by: Dani Louca <dlouca@splunk.com>
jlg-io pushed a commit to jlg-io/opentelemetry-collector-contrib that referenced this issue May 14, 2024
@dloucasfx @jlg-io
[receiver/k8scluster] Fix container state metadata (open-telemetry#32677
0fa0840 
)

**Description:** 
The
[transform](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/k8sclusterreceiver/internal/pod/pods.go#L37)
function is stripping out the container state which is needed in the
container
[GetMetadata](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/k8sclusterreceiver/internal/container/containers.go#L96)

This PR adds the state field back.

**Link to tracking Issue:**
This PR fixes
open-telemetry#32676

Signed-off-by: Dani Louca <dlouca@splunk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working needs triage New item requiring triage receiver/k8scluster
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[receiver/k8scluster] Fix container state metadata dloucasfx/opentelemetry-collector-contrib
1 participant
@dloucasfx