-
Notifications
You must be signed in to change notification settings - Fork 342
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
use tags for docker #264
use tags for docker #264
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer to not pin the Matomo version here.
Co-authored-by: J0WI <J0WI@users.noreply.github.com>
@@ -26,7 +26,7 @@ | |||
You can run the Matomo container and service like so: | |||
|
|||
```bash | |||
docker run -d --link some-mysql:db matomo | |||
docker run -d --link some-mysql:db matomo:4-apache |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docker run -d --link some-mysql:db matomo:4-apache | |
docker run -d --link some-mysql:db matomo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The goal of this PR is to restrict the tag to major releases of Matomo and its services. This change would contradict this goal.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are just examples. IMHO users should pin tags individually if they need to.
@@ -13,7 +13,7 @@ services: | |||
- ./db.env | |||
|
|||
app: | |||
image: matomo | |||
image: matomo:4-apache |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
image: matomo:4-apache | |
image: matomo |
@@ -13,7 +13,7 @@ services: | |||
- ./db.env | |||
|
|||
app: | |||
image: matomo:fpm-alpine | |||
image: matomo:4-fpm-alpine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
image: matomo:4-fpm-alpine | |
image: matomo:fpm-alpine |
@@ -36,7 +36,7 @@ This assumes you've already launched a suitable MySQL or MariaDB database contai | |||
Use a Docker volume to keep persistent data: | |||
|
|||
```console | |||
docker run -d -p 8080:80 --link some-mysql:db -v matomo:/var/www/html matomo | |||
docker run -d -p 8080:80 --link some-mysql:db -v matomo:/var/www/html matomo:4-apache |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
docker run -d -p 8080:80 --link some-mysql:db -v matomo:/var/www/html matomo:4-apache | |
docker run -d -p 8080:80 --link some-mysql:db -v matomo:/var/www/html matomo |
Fixes #263