DryRun causes UnboundLocalError: local variable 'status' referenced before assignment

ShahriyarR / MySQL-AutoXtraBackup

MySQL-AutoXtraBackup commandline tool written in Python 3 based on Percona XtraBackup

https://autoxtrabackup.azepug.az/

MIT License

139 stars 79 forks source link

DryRun causes UnboundLocalError: local variable 'status' referenced before assignment #431

Open WernerMairl opened 3 years ago

WernerMairl commented 3 years ago

Finding on master: error thrown here: https://github.com/ShahriyarR/MySQL-AutoXtraBackup/blob/72ca0a834f12331c3ed8cde17ea77a8ce573a774/backup_prepare/prepare.py#L630

caused/impacted by the following line in case dryrun! https://github.com/ShahriyarR/MySQL-AutoXtraBackup/blob/72ca0a834f12331c3ed8cde17ea77a8ce573a774/backup_prepare/prepare.py#L539

Assumtion: some "else" for dryrun should initialize status (status not initialzed for dry-run)!

regards Werner

ShahriyarR commented 3 years ago

@WernerMairl As I have released version 2.0, this error should not be encountered anymore. Could you please check out out the latest version of our tool? Thanks :)