Skip to content

Commit

Permalink
Update docker compose to specify influxdb:1.8.4
Browse files Browse the repository at this point in the history
  • Loading branch information
@samwiseg0
samwiseg0 committed Jun 3, 2021
1 parent 01b7d2c commit c47b7c2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docker-compose.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ services:
influxdb:
hostname: influxdb
container_name: influxdb
image: influxdb
image: influxdb:1.8.4
networks:
- internal
volumes:
Expand Down Expand Up @@ -128,4 +128,4 @@ services:
depends_on:
- influxdb
- varken
restart: unless-stopped
restart: unless-stopped

3 comments on commit c47b7c2

@modem7
Copy link

@modem7 modem7 commented on c47b7c2 Sep 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it better to use the 1.8-alpine tag as the image is half the size?

@samwiseg0
Copy link
Member Author

@samwiseg0 samwiseg0 commented on c47b7c2 Sep 9, 2021 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it better to use the 1.8-alpine tag as the image is half the size?

Alpine only has one arch for its images. amd64. If you want to change over to that you may. I would prefer to keep the image that is most compatible with more systems as the example.

https://hub.docker.com/layers/influxdb/library/influxdb/1.8.9-alpine/images/sha256-dc04dbf431ffb4a415ab34fb2c619e4fed559590f41f3a8245a4e232fc16c933?context=explore

@modem7
Copy link

@modem7 modem7 commented on c47b7c2 Sep 9, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Derp — didn't even notice that. Good spot. Agreed.

Please sign in to comment.