Merge branch 'master' into fix_logging

Author: nmaludy

CHANGELOG.rst

@@ -94,6 +94,10 @@ Fixed
 * Fixed a regression in the ``linux.dig`` action on Python 3. (bug fix) #4993
 
   Contributed by @blag
+* Fixed a bug in pack installation logging code where unicode strings were not being
+  interpolated properly. (bug fix)
+
+  Contributed by @misterpah
 * Fixed a compatibility issue with the latest version of the ``logging`` library API
   where the ``find_caller()`` function introduced some new variables. (bug fix)
 

st2common/st2common/util/pack_management.py

@@ -327,7 +327,7 @@ def apply_pack_owner_group(pack_path):
     pack_group = utils.get_pack_group()
 
     if pack_group:
-        LOG.debug('Changing owner group of "%s" directory to %s' % (pack_path, pack_group))
+        LOG.debug('Changing owner group of "{}" directory to {}'.format(pack_path, pack_group))
 
         if SUDO_BINARY:
             args = ['sudo', 'chgrp', '-R', pack_group, pack_path]
@@ -339,8 +339,8 @@ def apply_pack_owner_group(pack_path):
 
         if exit_code != 0:
             # Non fatal, but we still log it
-            LOG.debug('Failed to change owner group on directory "%s" to "%s": %s' %
-                      (pack_path, pack_group, stderr))
+            LOG.debug('Failed to change owner group on directory "{}" to "{}": {}'
+                      .format(pack_path, pack_group, stderr))
 
     return True