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

Containers are not restarted on failure #43

Closed
bswinnerton opened this issue Sep 6, 2018 · 7 comments
Closed

Containers are not restarted on failure #43

bswinnerton opened this issue Sep 6, 2018 · 7 comments

Comments

@bswinnerton
Copy link

I've noticed that when a backup fails (in my case it's due to the underlying mount point being unavailable), the docker containers that are automatically stopped are not restarted:

Local and Remote metadata are synchronized, no sync needed.
Last full backup left a partial set, restarting.
Last full backup date: Thu Sep  6 16:51:36 2018
RESTART: The first volume failed to upload before termination.
         Restart is impossible...starting backup from beginning.
Local and Remote metadata are synchronized, no sync needed.
Last full backup date: none
No signatures found, switching to full backup.
Giving up after 1 attempts. IOError: [Errno 2] No such file or directory: '/backup/duplicity-full.20180906T165246Z.vol1.difftar.gz'

Would it be possible to ensure that docker containers are started back up even when there is an error backing up?

@blacklabelops
Copy link
Owner

Yes, that is a good idea.

@blacklabelops
Copy link
Owner

Implemented and available.

@ghost
Copy link

ghost commented Oct 20, 2018

Are you sure that it works? I'm using version 1.2.1 and last night the backup failed to push the data via FTPS (I'm not sure why) and the containers stayed down after the failed backup.

@blacklabelops blacklabelops reopened this Oct 23, 2018
@blacklabelops
Copy link
Owner

I will check.

@blacklabelops
Copy link
Owner

The solution has been refactored and tested after issue #56.

Old script already fails when one of the containers fails to either stop or start or being not available anymore.

New script ignores docker failures and starts available containers.

@ghost
Copy link

ghost commented Oct 25, 2018

The solution has been refactored and tested after issue #56.

Old script already fails when one of the containers fails to either stop or start or being not available anymore.

New script ignores docker failures and starts available containers.

Thanks a lot for your work! I will update and check.

@blacklabelops
Copy link
Owner

Sorry, found the culprit when implementing #57.

You will have to upgrade to version 1.4.2 at least.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants