Merge pull request #2060 from jestabro/warning-boot-config-err

T5320: warn on entering config mode if boot config errors present

op-mode-definitions/configure.xml.in

@@ -11,7 +11,12 @@
         echo "Please do it as an administrator level VyOS user instead."
     else
         if grep -q -e '^overlay.*/filesystem.squashfs' /proc/mounts; then
-            echo "WARNING: You are currently configuring a live-ISO environment, changes will not persist until installed"
+        echo "WARNING: You are currently configuring a live-ISO environment, changes will not persist until installed"
+        else
+            if grep -q -s '1' /tmp/vyos-config-status; then
+            echo "WARNING: There was a config error on boot: saving the configuration now could overwrite data."
+            echo "You may want to check and reload the boot config"
+            fi
         fi
         history -w
         export _OFR_CONFIGURE=ok

python/vyos/util.py

@@ -1139,6 +1139,19 @@ def boot_configuration_complete() -> bool:
         return True
     return False
 
+def boot_configuration_success() -> bool:
+    from vyos.defaults import config_status
+
+    try:
+        with open(config_status) as f:
+            res = f.read().strip()
+    except FileNotFoundError:
+        return False
+
+    if int(res) == 0:
+        return True
+    return False
+
 def sysctl_read(name):
     """ Read and return current value of sysctl() option """
     tmp = cmd(f'sysctl {name}')

src/op_mode/powerctrl.py

@@ -104,8 +104,9 @@ def cancel_shutdown():
 
 def check_unsaved_config():
     from vyos.config_mgmt import unsaved_commits
+    from vyos.util import boot_configuration_success
 
-    if unsaved_commits():
+    if unsaved_commits() and boot_configuration_success():
         print("Warning: there are unsaved configuration changes!")
         print("Run 'save' command if you do not want to lose those changes after reboot/shutdown.")
     else: