Skip to content

Add --local when setting jh/Gemfile

Fu Zhang requested to merge gitlab-jh/jh-team/gitlab:jh-test-bundle-info into master

What does this MR do and why?

When optimizing cross-border bandwidth for JiHu, we found a weird bug that gitaly-rubygems cannot be cached.

It reads from update-setup-test-env-cache job that

Creating cache gitaly-ruby-gems-v1...
WARNING: vendor/gitaly-ruby/: no matching files    
Uploading cache.zip to https://cos.ap-shanghai.myqcloud.com/saas-gitlab-runner-1303695223/gitlab-runner/project/9148/gitaly-ruby-gems-v1 

which is obviously something wrong with bundle path.

We debugged a lot and found the root cause was when we were running bundle config set gemfile jh/Gemfile. It turned out to be a global setting and will affect gitaly. Adding --local will fix the problem.

Now update-setup-test-env-cache job is back to normal

Creating cache gitaly-ruby-gems-v1...
vendor/gitaly-ruby/: found 14819 matching files and directories 
Uploading cache.zip to https://cos.ap-shanghai.myqcloud.com/saas-gitlab-runner-1303695223/gitlab-runner/project/9148/gitaly-ruby-gems-v1 
Created cache

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Fu Zhang

Merge request reports

Loading