Updated dockerfile to allow building without local repo, changed couchdb to have flexible hostname#7
Open
NewlineDotBlog wants to merge 2 commits intohonoki:mainfrom
Open
Updated dockerfile to allow building without local repo, changed couchdb to have flexible hostname#7NewlineDotBlog wants to merge 2 commits intohonoki:mainfrom
NewlineDotBlog wants to merge 2 commits intohonoki:mainfrom
Conversation
Owner
|
Hi @NewlineDotBlog - thanks for this PR. I like the changes that remove the requirements for a local repo and would like to merge those in. I'm not so sure about the hostname variable; as it stands it seems like a user would need to edit the hostname in 3 places still: the service |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Hi,
When I tried to install bbrf on my local server I ran into some issues installing it all through portainer. Despite cloning the entire repository to another location on my server, the set up would refuse to work. After some tinkering it seems the following two issues fixed in this pull request caused the issues.
For whatever reason, docker refuses to run
docker-entrypoint.sh(due to permission issues) if it is mapped as volume in the docker compose file. After some tinkering, it seems that replacing theCOPYcommands in the dockerfile toADDcommands which simply retrieve the files from the github repo seems to work just fine.Secondly, allowing to change the hostname of couchdb helps in situations where there are several other instances of couchdb running in the same network. Since the hostname was hardcoded in the
bbrf-init.shfile, I updated this to instead use a new local environment variableCOUCHDB_HOSTNAMEinstead. This seems to work and allow the server to start without a local repo. (save for a few locally copied config files)Apologies if this is a bad PR, there is a first time for everything. If you'd like to see things changed let me know.