Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #344
The reason the volume was failing to be removed was due to a mismatch between the a
.env
file in thedocker
directory, and the source-controlled.env
file in the root directory. The deploy runs under thedocker
directory, and so environment files pertaining to the docker stack should live in that directory. I believe it's best practice to keep these files at the same level as the compose file anyway.The only reason I could see to keep this file in the root directory would be to run girder/other parts of the stack locally. However that's not a use case we currently support/endorse, so I don't see a reason to keep the
.env
file in the root directory.