-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make docker images rootless #173
Conversation
14c1c01
to
3e43f71
Compare
6b20767
to
90e17d5
Compare
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.
- a few changes against good practices to my mind
- two breaking changes
- some changes unecessary
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.
could you use this PR to delete the useless filebeat service from docekr-compose-prod please ?
90e17d5
to
c210177
Compare
c210177
to
59af920
Compare
300988d
to
629a160
Compare
629a160
to
fa2d000
Compare
@@ -18,8 +18,6 @@ services: | |||
image: basegun-backend:${TAG}-prod | |||
ports: | |||
- 5000:5000 | |||
volumes: | |||
- /var/log/basegun:/app/logs |
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.
good idea !
c21e211
to
75565ee
Compare
75565ee
to
ed97da0
Compare
9722e5f
to
8f72ec8
Compare
8f72ec8
to
bd03e18
Compare
* Make backend image rootless * Remove filebeat from docker-compose-prod.yml * Make frontend image rootless * Send logs to stdout * Remove tag question when using Makefile * Remove logs endpoint * Remove useless PATH_LOGS and CONTINUE=y
No description provided.