Create function to validate current persist_logs state prior to updat…

…es, then set to true if it is false with sane minimum auto-off delay

CHANGELOG.md

@@ -1,5 +1,9 @@
 # AmpliPi Software Releases
 
+## 0.4.6
+* System
+  * Automatically persist logs during (and for a short time after) updates
+
 ## 0.4.5
 * Web App
   * Ensure that abnormally-shaped album art is still horizontally centered

amplipi/updater/asgi.py

@@ -442,11 +442,27 @@ def progress_sse(tasks):
 @router.get('/update/install')
 def install():
   """ Start the install after update is downloaded """
+  persist_log_during_update()
   t = threading.Thread(target=install_thread)
   t.start()
   return {}
 
 
+def persist_log_during_update():
+  """Used during system updates to ensure persist logs is activated and has a minimum delay"""
+  persist_data = get_log_persist_state()
+  existing_persist = persist_data.persist_logs
+  existing_delay = persist_data.auto_off_delay
+  # If persist logs is already on and has a larger delay, keep that delay; otherwise ensure it has our sane minimum for support
+  if existing_persist and (existing_delay > 3 or existing_delay == 0):
+    data = Persist_Logs(persist_logs=True, auto_off_delay=existing_delay)
+    toggle_persist_logs(data=data)
+  else:
+    # Three days is an arbitrary number, picked to ensure the next few days of usage post-update are captured for support cases
+    data = Persist_Logs(persist_logs=True, auto_off_delay=3)
+    toggle_persist_logs(data=data)
+
+
 class PasswordInput(BaseModel):
   password: str
 

amplipi/updater/static/index.html

@@ -44,7 +44,7 @@ <h1><span class="text-white">Ampli</span><span class="text-danger">Pi</span></h1
               <a class="nav-link" id="manual-update-tab" data-toggle="tab" href="#manual-update" role="tab" aria-controls="manual-update" aria-selected="false">Custom Update</a>
             </li>
             <li class="nav-item">
-              <a class="nav-link" id="admin-settings-tab" data-toggle="tab" href="#admin-settings" role="tab" aria-controls="admin-settings" aria-selected="false">Admin Settings</a>
+              <a class="nav-link" id="admin-settings-tab" data-toggle="tab" href="#admin-settings" role="tab" aria-controls="admin-settings" aria-selected="false" onClick="getPersist();">Admin Settings</a>
             </li>
             <li class="nav-item">
               <a class="nav-link" id="support-tunnel-tab" data-toggle="tab" href="#support-tunnel" role="tab" aria-controls="support-tunnel" aria-selected="false">Support Tunnel</a>
@@ -153,6 +153,8 @@ <h1><span class="text-white">Ampli</span><span class="text-danger">Pi</span></h1
                   const textbox = document.getElementById("persist-input");
                   checkbox.checked = data.persist_logs;
                   textbox.value = data.auto_off_delay;
+
+                  return {"persist_logs": data.persist_logs, "auto_off_delay": data.auto_off_delay}
                 }
 
                 async function setPersist() {
@@ -190,7 +192,6 @@ <h1><span class="text-white">Ampli</span><span class="text-danger">Pi</span></h1
                   }, 1500);
                 }
 
-                window.onload = getPersist;
               </script>
 
               <div id="admin-settings-dialog">