-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Download large test repo once, then store in image #850
Conversation
This allows us to skip the lengthy step of downloading elawa.zip from Google Drive every time `skaffold delete` gets run, or the hg-repos volume gets deleted.
I suspect the |
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.
the init-repos component (kustomize terminology) is used in staging and develop to init repos, this change would break that.
This restores the develop and staging environments to their previous state.
If running as non-root, mkdir /init-repos will fail, so we will instead download files (if necessary) into /tmp. The alternative would be to run this initContainer as root, then do a chown -R www-data:www-data on the newly-created repos.
There's no reason to unzip the sena-3.zip (or elawa.zip) files into the dev's Git worktree any more. We had been doing this to populate the repos volume when using Docker Compose, but now that we've switched to Skaffold, this step is useless, so we'll remove it.
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.
nice job
This allows us to skip the lengthy step of downloading elawa.zip from Google Drive every time
skaffold delete
gets run, or the hg-repos volume gets deleted.Fixes #758.