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

[processor/k8sattributes] Failing test TestE2E_ClusterRBACCollectorStartAfterTelemetryGen/traces-deployment #37125

Open
crobert-1 opened this issue Jan 9, 2025 · 6 comments · Fixed by #37131
Open

[processor/k8sattributes] Failing test TestE2E_ClusterRBACCollectorStartAfterTelemetryGen/traces-deployment #37125

crobert-1 opened this issue Jan 9, 2025 · 6 comments · Fixed by #37131
Assignees
@bacherfl
Labels
flaky test a test is flaky processor/k8sattributes k8s Attributes processor

Comments

@crobert-1
Copy link
Member

Component(s)

processor/k8sattributes

Describe the issue you're reporting

Failing CI/CD action link

Failure output:

=== RUN   TestE2E_ClusterRBACCollectorStartAfterTelemetryGen/traces-deployment
    e2e_test.go:1602: 
        	Error Trace:	/home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/processor/k8sattributesprocessor/e2e_test.go:1602
        	            				/home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/processor/k8sattributesprocessor/e2e_test.go:1573
        	Error:      	Received unexpected error:
        	            	the following errors occurred:
        	            	 -  k8s.deployment.uid attribute not found
        	            	 -  k8s.deployment.name attribute not found
        	Test:       	TestE2E_ClusterRBACCollectorStartAfterTelemetryGen/traces-deployment

This failure was hit in a PR unrelated to this component.
@crobert-1 crobert-1 added the needs triage New item requiring triage label Jan 9, 2025
@github-actions github-actions bot added the processor/k8sattributes k8s Attributes processor label Jan 9, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Pinging code owners:

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

@TylerHelmuth TylerHelmuth added flaky test a test is flaky and removed needs triage New item requiring triage labels Jan 9, 2025
@TylerHelmuth
Copy link
Member

@bacherfl I believe this was caused by #37088

@crobert-1
Copy link
Member Author

+1 freq: https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/12700117150/job/35402366222?pr=35332

@bacherfl
Copy link
Contributor

@bacherfl I believe this was caused by #37088

will look into this today

This was referenced Jan 10, 2025
Merged
Open
@bacherfl
Copy link
Contributor

Created a PR to skip this test for now: #37131

@crobert-1
Copy link
Member Author

I'm going to reopen this issue for now for the purpose of investigating and fixing the test, as the test in question is only skipped for now, not fixed.

@crobert-1 crobert-1 reopened this Jan 10, 2025
AkhigbeEromo pushed a commit to sematext/opentelemetry-collector-contrib that referenced this issue Jan 13, 2025
@bacherfl @AkhigbeEromo
[chore] Skip flaky test `TestE2E_ClusterRBACCollectorStartAfterTeleme…
e91a6fc 
…tryGen` (open-telemetry#37131)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

The affected test was added to verify the solution implemented for
open-telemetry#37056. However, this seems to not have been fully solved yet.
Therefore, to avoid flaky test runs, this test is skipped for now, until
the issue is fully resolved

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#37125

<!--Describe what testing was performed and which tests were added.-->
#### Testing

Disabled the flaky test until open-telemetry#37056 is fully solved

---------

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bacherfl bacherfl

Labels
flaky test a test is flaky processor/k8sattributes k8s Attributes processor
Projects
None yet
Milestone
No milestone
3 participants
@bacherfl @TylerHelmuth @crobert-1