~thogarre/juju-backup-all:master

Last commit made on 2022-06-13
Get this branch:
git clone -b master https://git.launchpad.net/~thogarre/juju-backup-all
Only Garrett Neugent can upload to this branch. If you are Garrett Neugent please log in for upload directions.

Branch merges

Branch information

Name:
master
Repository:
lp:~thogarre/juju-backup-all

Recent commits

eab60c4... by Garrett Neugent

Address isort and black lint checks,upgrade black.

Pre-commit won't allow its yaml file in a separate commit, so the
upgrade to black 22.3.0 is included as well. This is to address bug 2964
where black fails locally with pre-commit.

https://github.com/psf/black/issues/2964

38267bb... by Garrett Neugent

Do not use -r for removing backup files.

Currently all backups are saved as a gzip file, so including the
-r flag for recursive deletion is unnecessary.

8952457... by Joe Guo

use local-fan addr as public addr for lxd

In libjuju[0], machine.dns_name will look for `public` or `local-cloud` address.
For lxd, neither exists, it will return None and cause ssh/scp functions fail.[1]

This patch use `local-fan` address as public address, when neither public nor local-cloud addr is available. so `machine.dns_name` can still get a valid address.

[0]: https://github.com/juju/python-libjuju/blob/master/juju/machine.py#L231
[1]: https://github.com/juju/python-libjuju/issues/611

Reviewed-on: https://code.launchpad.net/~guoqiao/juju-backup-all/+git/juju-backup-all/+merge/414135
Reviewed-by: Alvaro Uria <email address hidden>

34c65ca... by Ksawery Dziekoński

Fix TimeoutError exception type handling

Reviewed-on: https://code.launchpad.net/~ksdziekonski/juju-backup-all/+git/juju-backup-all/+merge/420729
Reviewed-by: Alvaro Uria <email address hidden>
Reviewed-by: Peter Sabaini <email address hidden>
Reviewed-by: Facundo Ciccioli <email address hidden>

99370c6... by Facundo Ciccioli

Bump libjuju version to current latest (2.9.9)

This is specifically to include https://github.com/juju/python-libjuju/pull/612, available since 2.9.6.

Reviewed-on: https://code.launchpad.net/~canonical-bootstack-cre/juju-backup-all/+git/juju-backup-all/+merge/421844
Reviewed-by: Peter Sabaini <email address hidden>
Reviewed-by: Ksawery Dziekoński <email address hidden>

0713fe2... by Facundo Ciccioli

Bump libjuju to current latest version

871d234... by Ksawery Dziekoński

Fix TimeoutError exception type handling

2a19f0d... by Joe Guo

use local-fan addr as public addr for lxd

In libjuju[0], machine.dns_name will look for `public` or `local-cloud` address.
For lxd, neither exists, it will return None and cause ssh/scp functions fail.[1]

This patch use `local-fan` address as public address,
so `machine.dns_name` can still get a valid address.

[0]: https://github.com/juju/python-libjuju/blob/master/juju/machine.py#L231
[1]: https://github.com/juju/python-libjuju/issues/611

0b320f2... by Jose Guedez

Add support for Python 3.6 (version bundled with bionic)

Reviewed-on: https://code.launchpad.net/~jfguedez/juju-backup-all/+git/juju-backup-all/+merge/412734
Reviewed-by: James Troup <email address hidden>

d7b9e06... by Jose Guedez

Add support for Python 3.6 (version bundled with bionic)

* Adds unit testing with two versions of Python (3.6 / 3.8)
* Replaces dataclasses with the attr library
* Marked the failing unit test from LP#1951914 to be skipped

Closes-Bug: 1953125