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

Cannot restore only the database from a backup of both database and active config. #166

Open
izmeez opened this issue Jun 20, 2024 · 4 comments

Comments

@izmeez
Copy link

izmeez commented Jun 20, 2024

If a manual backup is done of Database and Active Config Directory and then a restore of the backup is done selecting only Database it fails with the error:

Unable to restore from file /tmp/backup_migrate_....db_config.tar because a tar file can't be restored to this database.

Restoring both database and active config works but may not be what the user wants to do.

@argiepiano
Copy link
Collaborator

argiepiano commented Jun 20, 2024

Yes, this is expected. You can't do a Database restore from a file that contains something else. Perhaps there is lack of clarity in the choice - this choice is not supposed to give you options of what to restore, but is rather a way for the user to indicate the type of backup file you are trying to restore. The file type and the destinations must match.

BTW this confusion has been pointed out in other issues.

Perhaps there should be a way to prevent the fatal error and instead indicated the mismatch before trying to process the backup restore.

@izmeez
Copy link
Author

izmeez commented Jun 23, 2024

@argiepiano Thank you for the reply. It sounds like it is "By design".

Perhaps there should be a way to prevent the fatal error and instead indicated the mismatch before trying to process the backup restore.

  1. It may make more sense if there is a notice rather than an error.

  2. It may require something more to help users understand this restriction: that combined backups can only be used for combined restores. It may affect the workflow of different use cases.

Previously bam only did one thing, the database. Now with the option to include the active config with the database the implications of restore is evident.

@argiepiano
Copy link
Collaborator

I agree this is confusing. I think the way to go here is to check the type of source in a form validation. Also perhaps add some help text to the form.

@izmeez
Copy link
Author

izmeez commented Jul 13, 2024

Now, with a better understanding of the way the combined backup/restore works we have changed our workflow. Some extra help text is a good idea.

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

No branches or pull requests

2 participants