From 4698dcd935854c72cb3b3d5acc649b6f7ff86202 Mon Sep 17 00:00:00 2001
From: Dan Bungert <daniel.bungert@canonical.com>
Date: Mon, 14 Mar 2022 09:33:05 -0600
Subject: [PATCH] logging: raise warning when not adjusting perms

---
 subiquitycore/file_util.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/subiquitycore/file_util.py b/subiquitycore/file_util.py
index 5877f29b..0af78d2b 100644
--- a/subiquitycore/file_util.py
+++ b/subiquitycore/file_util.py
@@ -16,6 +16,7 @@
 import contextlib
 import datetime
 import grp
+import logging
 import os
 import tempfile
 
@@ -24,9 +25,13 @@ import yaml
 _DEF_PERMS_FILE = 0o640
 _DEF_GROUP = 'adm'
 
+log = logging.getLogger('subiquitycore.file_util')
+
 
 def set_log_perms(target, *, isdir=True, group_write=False, mode=None):
     if os.getuid() != 0:
+        log.warning('set_log_perms: running as non-root - not adjusting' +
+                    ' group owner or permissions for ' + target)
         return
     if mode is None:
         mode = _DEF_PERMS_FILE