-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
cache-read-only=true
for scheduled workflows on main branch
#383
Comments
@kkolman you are right - that's why I changed value of cache-read-only to: And now scheduled runs at least try to write cache. The problem is they are failing with following error: I think it may be related to scheduled workflow runs because when I tested action triggered by PR cache was written successfully. |
Thanks for the reports. I think we should be able to detect when we are running with a scheduled trigger and avoid setting However, I'm having trouble getting a scheduled workflow to execute: I added a Any ideas? |
@bigdaz I see now it is being scheduled :). However, my error with |
@AndrzejPw Can you please raise a separate issue for your problem, since it appears unrelated to this reported issue? If you can provide a link to a failing build that would be great, otherwise the full stack trace would help. |
Thank you - my bug with Save Gradle User Home does not appear anymore (maybe sth was wrong with github environment). If it happens again I will report it. |
@AndrzejPw FYI there was another report of the |
The default expression for
cache-read-only
value is:But
github.event.repository.default_branch
is not set by GA on scheduled (cron) workflow runs.Hence
cache-read-only
is resolved totrue
not writing to cache during scheduled runs on the main branch.For CI configrations where workflows on main line are run via scheduling exclusively (no manual workflow runs happening) cache will never be written to.
I believe this behavior is worth documenting.
Unless there's a solid replacement to
github.event.repository.default_branch
for obtaining the default branch name (didnt find one in ga contexts).The text was updated successfully, but these errors were encountered: