-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Vagrantfile: Add support for SHARE_PARENT=2 #14559
Conversation
Add support for value "2" for "SHARE_PARENT" environment variable, which causes the parent of the parent of the current directory to be shared with the Dev VM. This is useful to share all the github.com repositories in the developer's directory tree with the dev VM without using "USER_MOUNTS". This also fixes the NFS mount error on macOS when one exported NFS directory is within another. This would happen if the cilium repo is in the same tree that would be shared with USER_MOUNTS. Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
test-me-please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit, LGTM otherwise!
if ENV["SHARE_PARENT"] == "2" then | ||
cilium_dir = '../..' | ||
cilium_path = '/home/vagrant/go/src/github.com' | ||
elsif ENV["SHARE_PARENT"] then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be worth checking for == "1"
here, in case some contributors see the SHARE_PARENT=2
and try to disable by explicitely passing SHARE_PARENT=0
.
retest-gke |
1 similar comment
retest-gke |
We don't use the Vagrantfiles for GKE, so I don't think we need to wait for that CI pipeline. Marking as ready to merge. |
Add support for value "2" for "SHARE_PARENT" environment variable,
which causes the parent of the parent of the current directory to be
shared with the Dev VM. This is useful to share all the github.com
repositories in the developer's directory tree with the dev VM without
using "USER_MOUNTS".
This also fixes the NFS mount error on macOS when one exported NFS
directory is within another. This would happen if the cilium repo is
in the same tree that would be shared with USER_MOUNTS.
Signed-off-by: Jarno Rajahalme jarno@covalent.io