-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Temporarily remove step to save ccache from scheduled jobs #9433
Temporarily remove step to save ccache from scheduled jobs #9433
Conversation
Saving ccache is resulting in some errors finalizing the upload which is preventing scheduled jobs from running. Removing this step temporarily till its resolved.
✅ Deploy Preview for meta-velox canceled.
|
@bikramSingh91 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@bikramSingh91 merged this pull request in a9897b1. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
This is caused by an underlying, flakey issue of the artifacts back end, there is nothing we can do see actions/upload-artifact#543 Removing the step is a bit heavy handed we can just add |
Actually it seems the latest release updated the npm package used to work with the backend, I'll drop the version used in the action down to 4.3.0 for now. edit: done, the fix is in the v1 tag so will apply automatically here |
…ncubator#9433) Summary: Saving ccache is resulting in some errors finalizing the upload which is preventing scheduled jobs from running. Removing this step temporarily till its resolved. Pull Request resolved: facebookincubator#9433 Reviewed By: kgpai Differential Revision: D55987674 Pulled By: bikramSingh91 fbshipit-source-id: bc5655df1ef96dbd710871b0f1ed30b06cf41e4a
Saving ccache is resulting in some errors finalizing the upload
which is preventing scheduled jobs from running. Removing this
step temporarily till its resolved.