Fix CI failures by adding apt-get update to Github Actions#3082
Fix CI failures by adding apt-get update to Github Actions#3082embg merged 5 commits intofacebook:devfrom
Conversation
|
Well looks like Travis is failing... will investigate tomorrow. Edit: this link should fix the issue, will test tomorrow. Expect some spam from this PR since I can't test this on embg/zstd (Travis CI doesn't give free credits for public repos anymore). (note to self: Travis debug instructions here) Edit: also used this link to fix Travis (there were multiple keys that had to be updated). |
b2d1a6a to
9e0b2aa
Compare
|
Fixed most of the Travis issues, but there is one that apparently can only be fixed by moving away from Trusty. So there are two options:
Here is the failing Travis job: https://app.travis-ci.com/github/facebook/zstd/jobs/561969471 |
|
The failing Bottom line : we would be better off removing that test from Oh, and btw, it seems we already have such a test : Just check it's "equivalent enough" to remove the |
Following advice from here and here, add
sudo apt-get updateto hopefully avoid this CI error:The first time I ran CI on this commit (on a PR in embg/zstd) I got a weird failure on a different job, but the second time everything passed. I propose we land this for now (if it passes CI on facebook/zstd), and if the issue persists there are some other things in the first link above that we can try.