-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Added a comment about Dockerfile deployment #4808
Conversation
I found issues while trying to deploy an app from a dockerfile and I found that the expose instruction causes problems with the dokku proxy.
When you say it is exposed on the incorrect port, what did you find it was exposed on? |
Hello, The default HTTP port is 80 and if you specify in the dockerfile that you expose the port 8888 you'll have to set up the proxy manually. That means the default nginx page is displayed. However, if you don't add the |
Do you have an example Dockerfile app that doesn't work as expected and the change that you performed to make it work as expected? |
Hello, See this dockerfile:
And here is the procfile:
This app won't be presented in the default HTTP port. Insthead the default nginx page will be presented. To fix this issue, I just removed the |
Right, when you set an That is documented here under port management (and that section is linked to from here. Unfortunately, nginx does fallback routing, so if there is already a site set (which is usually something like I think your doc link might need a bit more explanation as to why not using an EXPOSE might help. I'll update the PR. |
I see. What happen if you push changes to the app with the port mapping? |
I think the doc update I wrote clarifies things. Can you check the diff? |
Thank you, seems better than mine. |
Awesome, I'll merge it then. |
# History ## 0.25.5 Install/update via the bootstrap script: ```shell wget https://raw.githubusercontent.com/dokku/dokku/v0.25.5/bootstrap.sh sudo DOKKU_TAG=v0.25.5 bash bootstrap.sh ``` ### Bug Fixes - #4787: @josegonzalez Do not require double quotes when issuing dokku run commands - #4816: @josegonzalez Set the DOKKU_HOST_ROOT on docker container start - #4810: @josegonzalez Handle udp ports when fetching network config - #4812: @josegonzalez Silence stderr when fetching a command from the Procfile - #4800: @josegonzalez Add help2man os package for copyfiles make target to devcontainer - #4796: @josegonzalez Remove the --restart flag from pre-deploy chown containers ### New Features - #4809: @josegonzalez Implement storage:ensure-directory command - #4801: @josegonzalez Expose git-from-archive and git-from-image plugin triggers - #4785: @josegonzalez Add support for VSCode Dev Containers ### Documentation - #4819: @stephenheron Fixed typo in tar documentation - #4824: @dy3l Fix GitLab case - #4822: @josegonzalez Add a note about using the registry:login command for private image deployments - #4808: @francipvb Added a comment about Dockerfile deployment - #4807: @FinnWoelm Docs: Fix instructions for enabling Docker Buildkit - #4786: @josegonzalez Add missing argument from trigger documentation - #4780: @adam12 Update source for `dokku-update` ### Tests - #4828: @josegonzalez Update golang in test apps to latest version - #4815: @josegonzalez tests: use python3 shebang for shellcheck-to-junit script - #4814: @josegonzalez Add wget to devcontainer to fix shfmt installation - #4791: @josegonzalez Fix test running in devcontainer ### Other - #4813: @dependabot[bot] chore(deps-dev): bump heroku/heroku-buildpack-php from 197 to 198 in /tests/apps/php - #4802: @dependabot[bot] chore(deps): bump django from 3.1.12 to 3.1.13 in /tests/apps/dockerfile-release - #4784: @josegonzalez Upgrade vector image to 0.16.x
I found issues while trying to deploy an app from a dockerfile and I found that the expose instruction causes problems with the dokku proxy. I found the solution and I am contributing it.