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

Post-run failure trying to save cache if cache directory was never populated #8

Closed
heysokam opened this issue Jul 20, 2024 · 3 comments · Fixed by #14
Closed

Post-run failure trying to save cache if cache directory was never populated #8

heysokam opened this issue Jul 20, 2024 · 3 comments · Fixed by #14

Comments

@heysokam
Copy link

https://github.com/OpenQuake3/defrag/actions/runs/10017364370
I don't know what these errors are about
They worked fine with goto-bus-stop version, but they trigger errors when changing to this action.
I'm kinda lost in how to debug it. I wish I could provide more insight.
What could I do to solve the problems? Is my setup incorrect, or is something else happening?

@squeek502
Copy link

squeek502 commented Jul 29, 2024

I could be wrong, but I think what's going wrong is that your action isn't actually doing anything with Zig that populates the cache. This action likely doesn't handle that situation gracefully (i.e. it expects the zig cache directory to be created during the action).

Using use-cache: false would likely fix it for you.

@mlugg
Copy link
Owner

mlugg commented Sep 13, 2024

Yep, @squeek502's analysis is correct. While I don't think this use case is particularly important (there's no reason to fetch a Zig compiler if you aren't going to compile anything!), that it caused confusion is unfortunate, so the action should be patched to not error in this case.

@mlugg mlugg changed the title [bug] Weird errors when setting up and cleaning after running Post-run failure trying to save cache if cache directory was never populated Sep 13, 2024
@mlugg mlugg closed this as completed in #14 Sep 18, 2024
@heysokam
Copy link
Author

tysm @mlugg 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants