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

Updating Consul DNS example #105

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

apcurrier
Copy link

Updated the base images we are using, as the old ones are not working. Also modified the NGINX dockerfile to reflect current practices in our documentation: https://docs.nginx.com/nginx/admin-guide/installing-nginx/installing-nginx-docker/.

With these changes, the Consul DNS example with NGINX Plus is working again.

Changing to a more recent vm box to keep the example up to date.

Signed-off-by: Adam P Currier <[email protected]>
Updated image to be more current and modified the nginx plus setup to reflect practices today.

Signed-off-by: Adam P Currier <[email protected]>
Signed-off-by: Adam P Currier <[email protected]>
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.

1 participant