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.

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

[Bug] CI does not have permissions to download cryostat-core dependency #1796

Closed
andrewazores opened this issue Nov 28, 2023 · 5 comments · Fixed by #1798
Closed

[Bug] CI does not have permissions to download cryostat-core dependency #1796

andrewazores opened this issue Nov 28, 2023 · 5 comments · Fixed by #1798
Labels
bug Something isn't working ci

Comments

@andrewazores
Copy link
Member

          Workflow started at 11/28/2023, 10:18:13 AM. [View Actions Run](https://github.com/cryostatio/cryostat/actions/runs/7020847064).

Originally posted by @github-actions[bot] in https://github.com/cryostatio/cryostat/issues/1795#issuecomment-1830056454

@andrewazores andrewazores added bug Something isn't working ci labels Nov 28, 2023
@andrewazores
Copy link
Member Author

I suppose this is related to my recent org-level change to read-only-by-default secrets, though I'm not sure exactly how since this looks like it's only trying to read an artifact from the GitHub Maven Packages repository. But I don't think anything else has recently changed that would affect this.

@aali309 @mwangggg could one of you take a look?

@andrewazores
Copy link
Member Author

@andrewazores
Copy link
Member Author

Interestingly, it looks like the backport of the same PR above got past that stage:

https://github.com/cryostatio/cryostat/actions/runs/7021108433

@mwangggg
Copy link
Member

Although it looks like it, it's not a permissions problem- it's an error with the way we're caching the maven files. I'm not quite sure what's wrong with the skjolber/maven-cache-github-action@v1 action, but it just doesn't properly cache the files. I updated the workflows to use actions/cache@v3 instead and it seems to be getting past this error 😁

@andrewazores
Copy link
Member Author

I'll just tag this one here as well since I think it may actually be the original issue I thought I was reporting here :)

https://github.com/cryostatio/cryostat-core/actions/runs/7052673314

from cryostatio/cryostat-core#309

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants