Replace subprocess with python-libjuju calls

Bug #1941918 reported by Dariusz Smigiel
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-backup-all
Fix Released
Undecided
Unassigned

Bug Description

There are few places in the code, which rely on running `subprocess` and using `juju`.

It needs to be replaced with appropriate `python-libjuju` calls.

Revision history for this message
Zachary Zehring (zzehring) wrote :

Currently, python-libjuju does not implement a couple of key functions we need for the tool: retrieving all controllers and creating controller backups.

Here are the GH Issues for implementing those features:

controllers: https://github.com/juju/python-libjuju/issues/529
controller backup: https://github.com/juju/python-libjuju/issues/529

Once those are implemented, we can remove the subprocess calls and add the lib calls.

Revision history for this message
Jose Guedez (jfguedez) wrote :

The upstream issues have been resolved. The current code in master uses the API for creating controller backups.

Changed in juju-backup-all:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.