Add basic dynamic configuration with volume labels - #258
Draft
rdelaage wants to merge 1 commit into
Draft
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
rdelaage
force-pushed
the
docker_discovery
branch
from
November 14, 2022 13:49
b8d6cae to
0ca59d4
Compare
rdelaage
force-pushed
the
docker_discovery
branch
from
November 14, 2022 14:55
0ca59d4 to
59b2d6f
Compare
Contributor
Author
|
I added all the relevant options for the volume type. The list of backend is |
|
Any progres? |
Contributor
Author
|
Hello @Akruidenberg I was supposed to do some refactoring first but I didn't found time, I am quite busy right now in my personal life. But feel free to test it and suggest things! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #256
This PR add a basic docker discovery system to autorestic. Autorestic can now connect to a docker daemon, check for volumes with a
autorestic.enableset to1ortrueand backup to the backend configured inautorestic.tolabel. You can also specify a cron inautorestic.cronlabel.This service discovery is only enabled when the
--docker-discoveryflag is set.Since I am used to work with podman and autorestic use the docker command, I needed to add a
--docker-hostflag used to change the host used in the docker command if specified (unix:///var/run/user/1000/podman/podman.sockfor me when working with the podman user socket)The app still need a static configuration file to configure backends and global options. You can also use location both in the static configuration file and docker labels. The new behavior just append the dynamic configuration to the config variable after parsing the config file.
This PR is in draft mode because I would like your suggestions about this feature.