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

TODO: Add note about mount path for backup pvc to restore AWX from existing backup files #409

Open
neofright opened this issue Mar 3, 2025 · 3 comments
Labels
exempt This issue is never marked as stale question Further information is requested

Comments

@neofright
Copy link
Contributor

In the restore documentation the manual restoration examples use the directory name of /data/backups but in the backup documentation the folder is /data/backup.

I spent more time than I'd like to admit before I finally diagnosed the stat: cannot statx '/data/backups/tower-openshift-backup-2024-08-04-130151 errors and realised why.

These directory names should probably be consistent in the documentation and yaml.

@neofright neofright added the question Further information is requested label Mar 3, 2025
@kurokobo
Copy link
Owner

kurokobo commented Mar 3, 2025

I can't find the string /data/backups in my repository 🤔
Could you please clarify which part of the description is making you concerned?

@neofright
Copy link
Contributor Author

Sure. In the manual restore instructions it says:

If the AWXBackup object no longer exists, place the backup files under /data/backup

Then afterwards specifies backup_dir: /backups/.

Should it be backup_dir: /data/backup/ or backup_dir: /backup/ in order to use the existing backups?

@kurokobo
Copy link
Owner

kurokobo commented Mar 4, 2025

Thanks for the update, I understand your situation.

It's indeed a difficult point to grasp. Unfortunately, the AWX Operator is designed to mount the specified PVC at /backups. It is hard coded so it is impossible to change the /backups in backup_dir to another path. Therefore, the only thing that can be done to align the directory names is to change /data/backup used in this repository to /data/backups (with trailing s). However, renaming the directory used in the PV would have a significant impact on existing users of this repository, so I don't want to do that.

That said, I have written about the mounted path in backup_dir in some issues in this repo and forums several times in the past, so I will also add similar comments to my guide.

Thanks for your report :)

@kurokobo kurokobo changed the title Restore instruction discrepencies TODO: Add note about mount path for backup pvc to restore AWX from existing backup files Mar 4, 2025
@kurokobo kurokobo added the exempt This issue is never marked as stale label Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exempt This issue is never marked as stale question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants