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

doc(Docker): document running in Docker #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bentolor
Copy link

While I failed to get the setup running for me (I have a SmartSolar and it seems it uses a different approach, See https://community.victronenergy.com/questions/187303/victron-bluetooth-advertising-protocol.html. And my LifePO4 BMS seems to be out of bluetooth reach), I still documented the necessary steps for clarity.

Watch out my change in run.sh. No clue how this is supposed to run.

@fl4p
Copy link
Owner

fl4p commented Dec 16, 2023

Why are you running this in Docker? batmon supports all mayor operating systems.

And whats the reason building this from homeassistant/amd64-base:latest? Batmon doesn’t use any HA specific binaries.
minimal alpine linux should be just fine. Whats the size of homeassistant/amd64-base:latest ?

The Dockerfile is meant to be for HA add-on interface. HA chooses the right base image, see
https://developers.home-assistant.io/docs/add-ons/configuration/

I don't recommend running batmon in Docker anyway. Better to run this directly on the host, it doesn't open any port.
Docker is often unnecessary.

@bentolor
Copy link
Author

bentolor commented Dec 16, 2023

Why are you running this in Docker?

All my production workloads are portable, decoupled from host environment, using isolated environments and lightweight.

And whats the reason building this from homeassistant/amd64-base:latest

Obviously I tried to adopt the existing blueprint. I remember some weird dependencies to S7 and it seemshomeassistant/amd64-base:latest provides that.

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.

2 participants