search

allfro / device-mapping-manager

58 stars 9 forks source link

Checking containers on first run + cleanup #14

Open michaelfarrow opened 10 months ago

michaelfarrow commented 10 months ago

If this image is run after other stacks / containers, it won't fix the required devices as it only listens for newly running containers. This is likely to happen if restarting a swarm host.

This change adds an initial check of all running containers on first run.

Also cleaned up import order.

PS. My first time using go, sorry for any mistakes 😬

gitdeath commented 8 months ago

@allfro - This appears to solve a use case I've run into. Are you open to merging?

davehope commented 7 months ago

@allfro This seems to be working really well, albeit for this issue which Michael has kindly submitted a PR for. Is there anything further needed that's preventing it being merged?

Thanks!

Fma965 commented 4 months ago

@michaelfarrow as this doesn't seem to be being merged, could you post your fork to dockerhub or ghcr for ease of access

michaelfarrow commented 4 months ago

@Fma965 It's on dockerhub here: https://hub.docker.com/r/weyforth/device-mapping-manager