canonical-ubuntu-qa team mailing list archive

[Tim Andersson <mp+476789@xxxxxxxxxxxxxxxxxx>]

Tim Andersson has proposed merging ~andersson123/autopkgtest-cloud:cloudrcs-mkdir into autopkgtest-cloud:master.

Requested reviews:
  Canonical's Ubuntu QA (canonical-ubuntu-qa)

For more details, see:
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/476789

Quick fix - create cloudrcs dir if it doesn't already exist
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~andersson123/autopkgtest-cloud:cloudrcs-mkdir into autopkgtest-cloud:master.

diff --git a/charms/focal/autopkgtest-cloud-worker/reactive/autopkgtest_cloud_worker.py b/charms/focal/autopkgtest-cloud-worker/reactive/autopkgtest_cloud_worker.py
index 47fe98f..39ca0c7 100644
--- a/charms/focal/autopkgtest-cloud-worker/reactive/autopkgtest_cloud_worker.py
+++ b/charms/focal/autopkgtest-cloud-worker/reactive/autopkgtest_cloud_worker.py
@@ -360,6 +360,8 @@ def update_nova_rcs():
 
     clear_old_rcs()
     rc_dest = Path(os.path.expanduser("~ubuntu/cloudrcs/"))
+    if not rc_dest.exists():
+        rc_dest.mkdir(exist_ok=True)
 
     with tarfile.open(rcs_file_path) as tar:
         for member in tar: