From 91077da273215596f07b7ebc858633dca350ad30 Mon Sep 17 00:00:00 2001
From: "Jose M. Guisado" <jguisado@soleta.eu>
Date: Wed, 14 Sep 2022 09:54:06 +0200
Subject: utils: minor fix for ogReduceFs and ogExtendFs

Do not return the subprocess result for ogReduceFs/ogExtendFs.

ogReduceFs works with or without the target filesystem mounted.
ogExtendFs requires the target filesystem to be mounted.

'ogMount' legacy script invocation should be replaced by a better
mount/umount wrapper.
---
 src/utils/fs.py | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

(limited to 'src/utils/fs.py')

diff --git a/src/utils/fs.py b/src/utils/fs.py
index cd58383..bce6022 100644
--- a/src/utils/fs.py
+++ b/src/utils/fs.py
@@ -82,21 +82,21 @@ def ogReduceFs(disk, part):
     Bash function 'ogReduceFs' wrapper
     """
     proc = subprocess.run(f'ogReduceFs {disk} {part}',
-                          shell=True, stdout=PIPE)
-
-    if proc.returncode == 0:
-        subprocess.run(f'ogUnmount {disk} {part}',
-                       shell=True, stdout=PIPE)
-        return proc.stdout.decode().replace('\n', '')
-
-    logging.warn(f'ogReduceFS exited with code {proc.returncode}')
-    return None
+                          shell=True, stdout=PIPE,
+                          encoding='utf-8')
+    if proc.returncode != 0:
+        logging.warn(f'ogReduceFS exited with non zero code: {proc.returncode}')
+    subprocess.run(f'ogUnmount {disk} {part}',
+                   shell=True)
 
 
 def ogExtendFs(disk, part):
     """
     Bash function 'ogExtendFs' wrapper
     """
+    subprocess.run(f'ogMount {disk} {part}',
+                   shell=True)
     proc = subprocess.run(f'ogExtendFs {disk} {part}',
                           shell=True)
-    return proc.returncode
+    if proc.returncode != 0:
+        logging.warn(f'ogExtendFs exited with non zero code: {proc.returncode}')
-- 
cgit v1.2.3-18-g5258