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

E2E - TestKamelCLIRun constantly failing after secrets.GITHUB_TOKEN applied #3952

Closed
tadayosi opened this issue Dec 22, 2022 · 0 comments · Fixed by #3953
Closed

E2E - TestKamelCLIRun constantly failing after secrets.GITHUB_TOKEN applied #3952

tadayosi opened this issue Dec 22, 2022 · 0 comments · Fixed by #3953
Assignees
@squakez
Labels
area/test kind/bug Something isn't working

Comments

@tadayosi
Copy link
Member

https://github.com/apache/camel-k/actions/runs/3755484014/jobs/6380597042

❌ TestKamelCLIRun (3m38.34s)
❌ TestKamelCLIRun/Examples_from_GitHub (3m26.92s)
❌ TestKamelCLIRun/Examples_from_GitHub/Gist_(ID) (280ms)
  GITHUB_TOKEN env var detected, using it for GitHub APIs authentication
  Error: cannot read sources: failed to resolve sources: GET https://api.github.com/gists/e2c3f9a5fd0d9e79b21b04809786f17a: 403 Resource not accessible by integration []
      testing.go:1169: test executed panic(nil) or runtime.Goexit: subtest may have called FailNow on a parent test

Apparently it's because secrets.GITHUB_TOKEN doesn't have previlege to access gists. A similar issue explained here:
https://goreleaser.com/errors/resource-not-accessible-by-integration/
@squakez squakez added the kind/bug Something isn't working label Dec 22, 2022
@squakez squakez self-assigned this Dec 22, 2022
squakez added a commit to squakez/camel-k that referenced this issue Dec 22, 2022
@squakez
fix(test): temporary remove env var
dc1c8ed 
Close apache#3952
@squakez squakez mentioned this issue Dec 22, 2022
Merged
squakez added a commit that referenced this issue Jan 3, 2023
@squakez
fix(test): temporary remove env var
c5562c3 
Close #3952
claudio4j added a commit to claudio4j/camel-k that referenced this issue May 18, 2023
@claudio4j
Fix e2e/namespace/install/cli/run_test.go for cli run from gist
f4f2a70 
Fix apache#3952
@claudio4j claudio4j mentioned this issue May 18, 2023
Merged
claudio4j added a commit to claudio4j/camel-k that referenced this issue May 18, 2023
@claudio4j
Fix e2e/namespace/install/cli/run_test.go for cli run from gist
417f79a 
Fix apache#3952
claudio4j added a commit to claudio4j/camel-k that referenced this issue May 18, 2023
@claudio4j
Fix e2e/namespace/install/cli/run_test.go for cli run from gist
8c661ea 
Fix apache#3952
squakez pushed a commit that referenced this issue May 19, 2023
@claudio4j @squakez
Fix e2e/namespace/install/cli/run_test.go for cli run from gist
bc59774 
Fix #3952
squakez pushed a commit to jboss-fuse/camel-k that referenced this issue May 19, 2023
@claudio4j @github-actions
Fix e2e/namespace/install/cli/run_test.go for cli run from gist
6fa53ad 
Fix apache#3952

(cherry picked from commit apache/camel-k@bc59774e9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@squakez squakez

Labels
area/test kind/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(test): temporary remove env var squakez/camel-k
2 participants
@tadayosi @squakez