Skip to content
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

Add default blob data location #3613

Merged
merged 1 commit into from
Mar 29, 2023
Merged

Conversation

ChilledSlim
Copy link
Contributor

On a new installation, Stash will default to 'blobs' (which relative to the CWD of /, becomes '/blobs'). In our recommended docker installation, this is NOT mapped and as a result will NOT be persistent across the container being removed or updated.

So by default, we are populating blob data which will be deleted, likely leaving to some confused new users. This should also be noted for existing installations to make this a persistent path.

On a new installation, Stash will default to 'blobs' (which relative to the CWD of /, becomes '/blobs').
In our recommended installation, this is NOT mapped and as a result will NOT be persistent across the container being removed or updated.

So by default, we are populating blob data which will be deleted, likely leaving to some confused new users.
This should also be noted for existing installations to make this a persistent path.
@ChilledSlim
Copy link
Contributor Author

@WithoutPants Drawing attention to the bold part which really should be in the release notes as I'm sure there are many docker installations out there.

@WithoutPants WithoutPants added the chore Pull requests for refactoring and admin work label Mar 29, 2023
@WithoutPants WithoutPants merged commit 2d8b6e1 into stashapp:develop Mar 29, 2023
@WithoutPants
Copy link
Collaborator

Thanks for the pick up on this. In terms of existing installations, I'm not sure what exactly I should be noting. Existing installs will need to explicitly set the blobs path if they choose to change to filesystem-based blob storage, and much like the generated path this should be persistent.

@ChilledSlim
Copy link
Contributor Author

Thanks for the pick up on this. In terms of existing installations, I'm not sure what exactly I should be noting. Existing installs will need to explicitly set the blobs path if they choose to change to filesystem-based blob storage, and much like the generated path this should be persistent.

You're probably right. Maybe something in the release note of "Ensure the path set is persistent (such as a persistent volume in containerized installations)".

TBH, I would have probably missed it, set it to /blobs, migrated all of the images, deleted the images and found out a month from now that all of my images are gone. I think there's a high risk of that right now, even if it is user error.

@ChilledSlim ChilledSlim deleted the patch-1 branch March 29, 2023 05:10
@WithoutPants
Copy link
Collaborator

Yeah, I could see myself doing something like that as well. It's definitely a high risk. I can add it to the release notes, but I'm not sure if it's going to be enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for refactoring and admin work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants