For some reason cirrus's /tmp permissions are set to only allow write access by the root user, causing apt-get to fail.
ci: cirrus workaround #28398
pull ajtowns wants to merge 1 commits into bitcoin:master from ajtowns:202309-cirrustmpperm changing 1 files +1 −1-
ajtowns commented at 9:05 PM on September 3, 2023: contributor
-
079e699a73
ci: cirrus workaround
For some reason cirrus's /tmp permissions are set to only allow write access by the root user, causing apt-get to fail.
-
DrahtBot commented at 9:05 PM on September 3, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK sipa, jonatack Concept ACK ismaelsadeeq If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Tests on Sep 3, 2023
-
ismaelsadeeq commented at 9:09 PM on September 3, 2023: member
Concept ACK
-
sipa commented at 9:10 PM on September 3, 2023: member
ACK 079e699a73b620f211f9daf31f58f4a2a09474cd This appears to fix the lint CI job.
-
jonatack commented at 9:24 PM on September 3, 2023: member
ACK 079e699a73b620f211f9daf31f58f4a2a09474cd
Seems odd to need to do this, and I don't know if it is the best way to fix the issue, but the lint CI passes here and currently fails on master.
-
ajtowns commented at 9:44 PM on September 3, 2023: contributor
Correct fix would be for cirrus's vm/docker images to already have correct perms for /tmp before the job runs
-
maflcko commented at 7:07 AM on September 4, 2023: member
lgtm.
I think Cirrus is using GCE containers for this, and I wonder if there is a setting that recently changed on Google's side? @fkorotkov
-
maflcko commented at 7:11 AM on September 4, 2023: member
For reference, the failing task on master is https://cirrus-ci.com/task/6339605360803840, which uses compute credits
-
fkorotkov commented at 8:28 AM on September 4, 2023: none
We've tried to update from
pd-balancedtopd-ssddisks yesterday to have better IO and bigger storage but it appeared that these changed messed up permissions. We've reverted the change. Sorry for the inconviniences! -
maflcko commented at 8:30 AM on September 4, 2023: member
Thanks!
Closing for now
- maflcko closed this on Sep 4, 2023
- bitcoin locked this on Sep 3, 2024
