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

[opampsupervisor] e2e test failing: Permission denied #33093

Closed
crobert-1 opened this issue May 16, 2024 · 7 comments · Fixed by #33095
Closed

[opampsupervisor] e2e test failing: Permission denied #33093

crobert-1 opened this issue May 16, 2024 · 7 comments · Fixed by #33095
Labels
bug Something isn't working cmd/opampsupervisor

Comments

@crobert-1
Copy link
Member

Component(s)

cmd/opampsupervisor

Describe the issue you're reporting

CI/CD failure links:
https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/9114560087/job/25059098980?pr=32177
https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/9115126246/job/25061035790

Failure output:

=== RUN   TestSupervisorAgentDescriptionConfigApplies
    e2e_test.go:143: 
        	Error Trace:	/home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/cmd/opampsupervisor/e2e_test.go:143
        	            				/home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/cmd/opampsupervisor/e2e_test.go:469
        	Error:      	Received unexpected error:
        	            	error creating storage dir: mkdir /var/lib/otelcol: permission denied
        	Test:       	TestSupervisorAgentDescriptionConfigApplies
    e2e_test.go:115: Shutting down
@crobert-1 crobert-1 added the needs triage New item requiring triage label May 16, 2024
@github-actions GitHub Actions
Copy link
Contributor

Pinging code owners:

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

@evan-bradley evan-bradley added bug Something isn't working and removed needs triage New item requiring triage labels May 16, 2024
@evan-bradley
Copy link
Contributor

@BinaryFissionGames I think this is related to #32618, could you take a look?

@crobert-1
Copy link
Member Author

+1 freq: https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/9115269204/job/25061515803?pr=31773

@crobert-1
Copy link
Member Author

This has failed every time on main since #32618 was merged.

@BinaryFissionGames
Copy link
Contributor

Taking a look...

@BinaryFissionGames BinaryFissionGames mentioned this issue May 16, 2024
Merged
@BinaryFissionGames
Copy link
Contributor

It looks like the merge of #32819 and #32618 resulted in failing tests.

#33095 should fix it.

evan-bradley pushed a commit that referenced this issue May 16, 2024
@BinaryFissionGames
[cmd/opampsupervisor]: Add storage dir to e2e test config to fix test…
c3b68d6 
… failure (#33095)

**Description:** <Describe what has changed.>
fixes test failure observed in #33093 (due to merge of #32819 and
#32618)

**Link to tracking Issue:** Closes #33093

**Testing:** run supervisor e2e tests
@evan-bradley
Copy link
Contributor

Tests pass now: https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/9115821831/job/25063271280

Thanks @crobert-1 and @BinaryFissionGames!

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 cmd/opampsupervisor
Projects
None yet
Milestone
No milestone
3 participants
@BinaryFissionGames @evan-bradley @crobert-1