Uh oh!
There was an error while loading. Please reload this page.
Fix npm root cache issue on test-discovery-notifications - #11251
Conversation
|
| COPY package.json package-lock.json ./ | ||
| # Quick fix for CI where we don't run as root user |
There was a problem hiding this comment.
why needed on only these containers?
There was a problem hiding this comment.
These containers run npm in the dockerfile (so as root) and then execute as user
* origin/main: [QA-1960] Update harmony in create-react-app (#11242) Fix npm root cache issue on test-discovery-notifications (#11251) Audius Protocol v0.7.83 [PAY-3884] Fix plaid verification layout (#11250) Remove @pedalboard/backfill-audio-analyses (#11247) Add fixed-decimal to identity test cache in CI (#11249) rm remaining audiusd code (#11230) [PAY-3866] Listen Streak Challenge Web UI (#11241) Allow multiple services to be pushed at once (#11248) Allow testing multiple services at once in CI (#11246) Fix test-contracts and test-eth-contracts in ci (#11245) Improve history table performance (#11243) Fix auth for purchase and sale count (#11238) [C-5736] Migrate explore featured content to tan-query (#11236) Make pushing DN images into one single job instead of 10 (#11240) Clean up env before running tests (#11239)
Moves the npm cache from the root so that the CircleCI user can access it.
Tested by manually changing this live on the worker and seeing the repro go away:
sudo su circlecito ensure you're acting as the same user as CIcd ~/audius-protocoland ensure the repository is up to date with main. ~/.profile; export DOCKER_UID=$(id -u); export DOCKER_GID=$(id -g); audius-compose test run "discovery-provider-notifications"