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

Remove instructions from readme about customizing files #487

Merged
merged 2 commits into from
Sep 15, 2023
Merged

Conversation

leighmcculloch
Copy link
Member

What

Remove instructions from readme about customizing files on persistent volumes.

Why

The instructions currently set someone up to fail because they describe a use case that is misaligned with the goals of the image, and misaligned with the actual maintenance that takes place on this image.

This image is for development and testing, both in the local form, and in the form of being able to deploy the image to simple PaaS environments in a short lived form. (Note that helm charts and other methods for installing and hosting the Stellar services exist for longer form and more complex environments.)

The organization of the files is particularly brittle. When maintaining the image for its primary purpose it is common that we need to change the organization of the files in ways that make the image incompatible with users who have customized the files.

Given that we've broken how those files work in the past and nobody has spoken up there's some signal, or lack of signal, that suggests very few people are actually customizing the files.

Nothing about this change prevents someone from customizing the files. The files are still stored in the location that becomes persistent when used in persistent mode.

This change serves to stop promoting it as a recommended way to use the image.

The change also adds a note clarifying how there is no compatibility guarantee on the configuration files.

Close #478

Copy link
Contributor

@sreuland sreuland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@leighmcculloch leighmcculloch enabled auto-merge (squash) September 15, 2023 20:00
@leighmcculloch leighmcculloch merged commit a2bad54 into master Sep 15, 2023
59 checks passed
@leighmcculloch leighmcculloch deleted the i478 branch September 15, 2023 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

consolidation for development and test purposes
2 participants