querysub 0.294.0 → 0.295.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "querysub",
-  "version": "0.294.0",
+  "version": "0.295.0",
   "main": "index.js",
   "license": "MIT",
   "note1": "note on node-forge fork, see https://github.com/digitalbazaar/forge/issues/744 for details",

package/src/4-deploy/edgeClientWatcher.tsx

@@ -66,7 +66,8 @@ function onLiveHashChange(liveHash: string, refreshThresholdTime: number) {
     if (liveHash === curHash) return;
     let prevHash = curHash;
     // Don't notify the user right away. Hopefully they refresh naturally, and we never have to notify them at all!
-    let notifyIntervals = [0.4, 0.75, 1];
+    // Also, refresh BEFORE the server dies, not exactly when it is about to die
+    let notifyIntervals = [0.4, 0.75, 0.95];
     console.log(blue(`Client liveHash changed ${liveHash}, prev hash: ${prevHash}`));
     // If we are replacing an already existing notification, don't show immediately
     let skipFirst = false;

package/src/deployManager/components/ServiceDetailPage.tsx

@@ -359,9 +359,9 @@ type ServiceConfig = ${serviceConfigType}
                                 </div>
                                 {serviceInfo && (
                                     <>
-                                        <div title={formatVeryNiceDateTime(serviceInfo.lastLaunchedTime)}>
+                                        {!!serviceInfo.lastLaunchedTime && <div title={formatVeryNiceDateTime(serviceInfo.lastLaunchedTime)}>
                                             Uptime: {formatTime(now - serviceInfo.lastLaunchedTime)}
-                                        </div>
+                                        </div>}
                                         <div>
                                             Launches: {serviceInfo.totalTimesLaunched}
                                         </div>

package/src/deployManager/machineApplyMainCode.ts

@@ -349,6 +349,7 @@ const runScreenCommand = measureWrap(async function runScreenCommand(config: {
                 // REMOVE the nodeId file, so we the node isn't terminated!
                 await fs.promises.unlink(nodeIdPath);
                 try {
+                    console.log(green(`Triggering rolling update notification to ${nodeId} for ${screenName} at ${new Date().toLocaleString()}`));
                     await triggerRollingUpdate({
                         nodeId,
                         time: rollingFinalTime,
@@ -487,6 +488,13 @@ let lastLaunchedTimePerService = new Map<string, number>();
 
 async function quickIsOutdated() {
     let machineId = getOwnMachineId();
+    // If any rolling are ready to restart, we are outdated
+    for (let [rollingScreenName, rollingInfo] of rollingKeepScreenAlive) {
+        if (rollingInfo.pinnedTime + rollingInfo.pinnedDuration < Date.now()) {
+            console.log(red(`Rolling screen ${rollingScreenName} is finished, doing a full resync`));
+            return true;
+        }
+    }
     let configs = await serviceConfigs.values();
     let relevantConfigs = configs.filter(config => config.machineIds.includes(machineId)).filter(x => x.parameters.deploy);
     let screens = await getScreenState();
@@ -508,7 +516,6 @@ async function quickIsOutdated() {
             if (!fs.existsSync(parameterPath)) return true;
             let prevParameters = await fs.promises.readFile(parameterPath, "utf8");
             if (prevParameters !== newParametersString) return true;
-
         }
     }
     return false;