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

Remove database configs from hooks #139

Open
Subbeh opened this issue Sep 29, 2023 · 2 comments
Open

Remove database configs from hooks #139

Subbeh opened this issue Sep 29, 2023 · 2 comments
Assignees

Comments

@Subbeh
Copy link

Subbeh commented Sep 29, 2023

As of version 1.8.0, database configurations are not defined under hooks anymore and borgmatic fails if the old format is still in use.

Please change the role accordingly so that any database backups are defined at root-level.

https://torsion.org/borgmatic/docs/how-to/backup-your-databases/#database-dump-hooks

@agross
Copy link

agross commented Dec 17, 2023

There are even more breaking changes coming up. These warnings are currently logged for my installation.

WARNING /etc/borgmatic/config.yaml:

Configuration sections (like location:, storage:, retention:, consistency:, and hooks:) are deprecated and support will be removed from a future release. To prepare for this, move your options out of sections to the global scope.

The exclude_if_present option now expects a list value. String values for this option are deprecated and support will be removed from a future release.

The repositories option now expects a list of key/value pairs. Lists of strings for this option are deprecated and support will be removed from a future release.

@m3nu
Copy link
Collaborator

m3nu commented Dec 18, 2023

I'll update the template and do a major version bump to reflect this.

@m3nu m3nu self-assigned this Dec 18, 2023
@m3nu m3nu added the type:task label Dec 18, 2023
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

3 participants