-
-
Notifications
You must be signed in to change notification settings - Fork 733
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
Dockerfile to use with docker-compose. #159
Comments
Nevermind, I got it.
|
Will issue new pull request for addition of "simple installation" within the |
This would be great, however I don't want to hardcode build args, thank you! |
I understand the sentiment. What do you constitute as hardcoded build args? Tag, features, directories, permissions, configurations? Would you be open to a "quick start-up" |
Hi, first of all I have to say that I'm not a Docker expert so I could miss something, however I'm open to discuss and to learn. SFTPGo provides sample dockerfiles that use multi stage build, inside the
when you build the image you can set the
the runtime configuration parameters could be set using ENV vars, this seems quite similar, for example to official mysql docker image. The only restriction is that a viper requires a config file in order to use ENV vars, so we cannot have an env vars only configuration but this could be fixed in future. If these things can be improved I would be happy to merge your changes. If you think that docs are not clear enough a pull request would be great (english is not my native language I'm aware that docs can be improved). Also providing some quick start-up Dockerfiles would be a nice additions, it could help new users. Thank you |
please take a look at the new README I hope it clarifies the things a bit. However pull requests are welcome, thank you! |
Trust me when I say this: the documentation is outstanding, and the only reason I was able to solve my issue. Regarding Docker, the issue in itself wasn't The
To instantly have the I would be happy to modify the current
Sometimes all a user wants to do is give the project a shot before using it as well. Adding extra steps and host-modifying commands aren't always pleasing. :) |
Ok, now I understand what you mean and it makes sense. Please send a pull request with your proposed changes, so we can discuss about them. The review can take a while I don't know docker very well. Reading the section "Using a custom MySQL configuration file" here seems that the way to ship configuration files in docker is inside the image itself. I'll try to find some time the next weekend to learn more about this. If we find a way to make anything configurable I could also submit a sftpgo image on docker hub. There are already several user contributed images there |
Sounds good. Will likely need a few days myself -- project deadline for work is coming up, and I'm hard at work on that. Regarding the user contributed images, the two I had found failed and didn't work. Perhaps a good way to go about this would be to automate the DockerHub push via Travis. I can help out in this regard, but it would be best for you to be the maintainer of both DockerHub and Travis. The automation would make things more efficient, and it would certainly be nice to just be able to do:
|
Ok, no hurry
I recently siwtched from Travis to GitHub Actions, anyway we will discuss about the official docker image and the automatic push once we found a way to build a general usable and full customizable way to make available the image itself, thank you! |
Understood. Sounds like a plan. Expect a PR from me in the coming days! |
we probably could do something like this. This image has similar requirements, it needs to expose host keys, mount volumes etc. Thank you! |
Wondering if there is any available
Dockerfile
that containerizes the entire project without needing to run commands in the host machine, nor multipledocker run
to finalize setup. Currently looking to runsftpgo
with an externalPostgres
service withdocker-compose
, but having a hard time setting-up the project without needing to involve the host machine by modifying the providedDockerfile
s in the repo.Any help would be appreciated.
The text was updated successfully, but these errors were encountered: