esm-cache: fix permissions used for creating apt directories

canonical · Apr 15, 2024 · 6ef38a5 · 6ef38a5
1 parent 3ff325a
commit 6ef38a5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/uaclient/apt.py b/uaclient/apt.py
@@ -843,7 +843,7 @@ def _ensure_esm_cache_structure():
     for file in ESM_BASIC_FILE_STRUCTURE["files"]:
         system.create_file(file)
     for folder in ESM_BASIC_FILE_STRUCTURE["folders"]:
-        os.makedirs(folder, exist_ok=True, mode=755)
+        os.makedirs(folder, exist_ok=True, mode=0o755)
 
 
 def update_esm_caches(cfg) -> None:

diff --git a/uaclient/tests/test_apt.py b/uaclient/tests/test_apt.py
@@ -1610,10 +1610,12 @@ def test_create_when_necessary(
             mock.call(
                 tmpdir + "/var/cache/apt/archives/partial",
                 exist_ok=True,
-                mode=755,
+                mode=0o755,
             ),
             mock.call(
-                tmpdir + "/var/lib/apt/lists/partial", exist_ok=True, mode=755
+                tmpdir + "/var/lib/apt/lists/partial",
+                exist_ok=True,
+                mode=0o755,
             ),
         ]