Update CHANGELOG.md

GitHub limits a check run summary to 65535 bytes, not characters. The
shopware report (DevScripts PR 2478) had emoji-heavy logs and failed with
422. Cut by bytes at a character boundary and close an open log block.

.github/workflows/arm-script-test.yml

@@ -39,7 +39,7 @@ jobs:
     timeout-minutes: 120
     steps:
       # The harness from the default branch, as in pr-script-test.yml.
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v7
         with:
           sparse-checkout: .github/workflows/scripts
           persist-credentials: false
@@ -106,7 +106,7 @@ jobs:
           } >>"$GITHUB_STEP_SUMMARY"
 
       - if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: result-arm
           path: ${{ runner.temp }}/result

.github/workflows/pr-script-test.yml

@@ -95,9 +95,10 @@ jobs:
       trusted: ${{ steps.plan.outputs.trusted }}
       kind: ${{ steps.plan.outputs.kind }}
       platforms: ${{ steps.plan.outputs.platforms }}
+      base_sha: ${{ steps.plan.outputs.base_sha }}
     steps:
       - id: plan
-        uses: actions/github-script@v8
+        uses: actions/github-script@v9
         with:
           script: |
             const { owner, repo } = context.repo;
@@ -258,6 +259,31 @@ jobs:
             }
 
             const include = [];
+            // Upgrade test: install the ct/ script from the PR's base branch,
+            // then update with the PR, as a user who installed before the merge
+            // would. Only in ProxmoxVE, where users have installed it, and only
+            // when the PR changes update_script(): with the same update code an
+            // existing install updates exactly as it does today. New scripts
+            // have nothing to upgrade from. Proxmox only, once per script.
+            const UPGRADE_REPO = 'community-scripts/ProxmoxVE';
+            let baseSha = '';
+            if (!isIssue && `${owner}/${repo}` === UPGRADE_REPO) {
+              const { data: b } = await github.rest.repos.getBranch({ owner, repo, branch: pr.base.ref });
+              baseSha = b.commit.sha;
+            }
+            // From `function update_script() {` to the first `}` at the start of
+            // a line; null without one.
+            const updateCode = t => ((t || '').match(/^(function[ \t]+)?update_script[ \t]*\(\)[ \t]*\{[\s\S]*?^\}/m) || [null])[0];
+            const updateChanged = async (p, headText) => {
+              if (!baseSha) return false;
+              let baseText;
+              try {
+                const res = await github.rest.repos.getContent({ owner, repo, path: p, ref: baseSha });
+                baseText = Buffer.from(res.data.content || '', 'base64').toString('utf8');
+              } catch (e) { return false; }
+              const before = updateCode(baseText), after = updateCode(headText);
+              return before !== null && after !== null && before !== after;
+            };
             const fetchScript = async p => {
               try {
                 const res = await github.rest.repos.getContent({ owner: head.owner.login, repo: head.name, path: p, ref: headSha });
@@ -302,7 +328,11 @@ jobs:
                 platforms = platforms.filter(p => only.has(p));
               }
               for (const platform of platforms) {
-                for (const os of oses) include.push({ script, os, platform, arm64: platform === 'arm' ? arm : '', name: `${os ? `${name}-${os}` : name}-${platform}` });
+                for (const os of oses) include.push({ script, os, platform, arm64: platform === 'arm' ? arm : '', mode: '', name: `${os ? `${name}-${os}` : name}-${platform}` });
+              }
+              if (!isVm && platforms.includes('pve') && await updateChanged(script, text)) {
+                const os = oses[0];
+                include.push({ script, os, platform: 'pve', arm64: '', mode: 'upgrade', name: `${os ? `${name}-${os}` : name}-pve-upgrade` });
               }
             }
             for (const script of entries.slice(MAX)) notBuilt.push(`\`${script}\`: at most ${MAX} scripts per run, name it with \`/test ${script}\``);
@@ -327,7 +357,7 @@ jobs:
                 // full test, e.g. as a required check.
                 owner, repo, name: `Proxmox script test${onlyList.length ? ` (${onlyList.join(', ')} only)` : ''}`, head_sha: headSha, status: 'in_progress',
                 details_url: runUrl,
-                output: { title: `Building ${include.length}`, summary: include.map(i => `- \`${i.script}\` ${[i.platform, i.os].filter(Boolean).join(' · ')}`).join('\n') },
+                output: { title: `Building ${include.length}`, summary: include.map(i => `- \`${i.script}\` ${[i.platform, i.os, i.mode].filter(Boolean).join(' · ')}`).join('\n') },
               });
               checkId = String(check.id);
             }
@@ -339,13 +369,14 @@ jobs:
             core.setOutput('check_id', checkId);
             core.setOutput('not_built', JSON.stringify(notBuilt));
             core.setOutput('platforms', onlyList.join(' '));
+            core.setOutput('base_sha', baseSha);
             // Who asked: the /test commenter, or whoever dispatched the run.
             core.setOutput('actor', context.actor);
 
             await sticky([
               MARK, '### Proxmox script test', '',
               `⏳ Building ${include.length} at ${headSha.slice(0, 7)}${isIssue ? ' (default branch)' : ''}${onlyList.length ? `, only on ${onlyList.join(', ')}` : ''} — [watch it live](${runUrl}) (open the \`test\` job)`, '',
-              ...include.map(i => `- \`${i.script}\` (${[i.platform, i.os].filter(Boolean).join(' · ')})`),
+              ...include.map(i => `- \`${i.script}\` (${[i.platform, i.os, i.mode && 'upgrade from the base branch'].filter(Boolean).join(' · ')})`),
               ...(notBuilt.length ? ['', '**Not built:**', ...notBuilt.map(n => `- ${n}`)] : []),
             ].join('\n'));
 
@@ -363,7 +394,7 @@ jobs:
     continue-on-error: ${{ matrix.platform == 'arm' }}
     permissions:
       contents: read
-    timeout-minutes: 90 # install (45) + update (30) + checks
+    timeout-minutes: 120 # install (45) + update (30) + second update + checks
     strategy:
       fail-fast: false
       matrix:
@@ -371,7 +402,7 @@ jobs:
     name: test (${{ matrix.name }})
     steps:
       # The default branch, never the PR: the harness itself must be trusted.
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v7
         with:
           sparse-checkout: .github/workflows/scripts
           persist-credentials: false
@@ -392,6 +423,9 @@ jobs:
           # ARM is the Incus harness on another machine.
           PLATFORM: ${{ matrix.platform == 'arm' && 'incus' || matrix.platform }}
           VAR_ARM64: ${{ matrix.platform == 'arm' && 'yes' || '' }}
+          # upgrade: install from the base branch, update with the PR.
+          TEST_MODE: ${{ matrix.mode || '' }}
+          BASE_SHA: ${{ needs.gate.outputs.base_sha }}
           UPSTREAM: ${{ github.repository }}
           CTID_BASE: ${{ github.repository == 'community-scripts/ProxmoxVE' && '9050' || '9000' }}
           # Only so release lookups are not rate limited: unauthenticated, all
@@ -402,14 +436,14 @@ jobs:
         # The self-hosted runners run as root; the GitHub one needs sudo (pct
         # and incus do).
         run: >-
-          ${{ matrix.platform == 'arm' && 'sudo --preserve-env=PR_REPO,PR_SHA,SCRIPT,OS,PLATFORM,VAR_ARM64,UPSTREAM,CTID_BASE,GH_API_TOKEN,OUT_DIR,GITHUB_RUN_ID,GITHUB_RUN_ATTEMPT' || '' }}
+          ${{ matrix.platform == 'arm' && 'sudo --preserve-env=PR_REPO,PR_SHA,SCRIPT,OS,PLATFORM,VAR_ARM64,TEST_MODE,BASE_SHA,UPSTREAM,CTID_BASE,GH_API_TOKEN,OUT_DIR,GITHUB_RUN_ID,GITHUB_RUN_ATTEMPT' || '' }}
           bash .github/workflows/scripts/pve-run-script.sh
 
       - if: always() && matrix.platform == 'arm'
         run: sudo chmod -R a+rX "$RUNNER_TEMP/result" 2>/dev/null || true
 
       - if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: result-${{ matrix.name }}
           path: ${{ runner.temp }}/result
@@ -427,12 +461,12 @@ jobs:
       issues: write
       pull-requests: write
     steps:
-      - uses: actions/download-artifact@v5
+      - uses: actions/download-artifact@v8
         with:
           pattern: result-*
           path: results
 
-      - uses: actions/github-script@v8
+      - uses: actions/github-script@v9
         env:
           MATRIX: ${{ needs.gate.outputs.matrix }}
           PR: ${{ needs.gate.outputs.pr }}
@@ -477,6 +511,12 @@ jobs:
                     if (r.status === 'pass' && r.update_status === 'error') { r.status = 'error'; r.step = `update ${r.update_step}`; }
                     const ulog = path.join(dir, 'update.log');
                     r.update_log = fs.existsSync(ulog) ? fs.readFileSync(ulog, 'utf8') : '';
+                    const u2log = path.join(dir, 'update2.log');
+                    r.update2_log = fs.existsSync(u2log) ? fs.readFileSync(u2log, 'utf8') : '';
+                    if (!['upgrade', ''].includes(r.mode || '')) r.mode = '';
+                    r.mode = r.mode || '';
+                    r.notes = Array.isArray(r.notes) ? r.notes.filter(n => typeof n === 'string').slice(0, 8) : [];
+                    r.resources = typeof r.resources === 'string' ? r.resources : '';
                     const ushot = path.join(dir, 'screenshot-update.png');
                     r.shot_update_file = fs.existsSync(ushot) ? ushot : '';
                     // ARM runs the Incus harness on an arm64 machine; result.json
@@ -484,7 +524,7 @@ jobs:
                     // unknown one would never match a matrix row anyway.
                     if (r.arch === 'aarch64') r.platform = 'arm';
                     else if (!['pve', 'incus'].includes(r.platform)) r.platform = 'pve';
-                    found.set(`${r.script}|${r.platform}|${r.os}`, r);
+                    found.set(`${r.script}|${r.platform}|${r.os}|${r.mode}`, r);
                   } catch (err) { core.warning(`unreadable ${p}: ${err.message}`); }
                 }
               }
@@ -492,19 +532,20 @@ jobs:
             walk('results');
 
             // A job that died before writing result.json still gets a row.
-            const rows = matrix.map(m => ({ ...(found.get(`${m.script}|${m.platform || 'pve'}|${m.os || 'default'}`) || {
-              script: m.script, os: m.os || 'default', platform: m.platform || 'pve',
+            const rows = matrix.map(m => ({ ...(found.get(`${m.script}|${m.platform || 'pve'}|${m.os || 'default'}|${m.mode || ''}`) || {
+              script: m.script, os: m.os || 'default', platform: m.platform || 'pve', mode: m.mode || '', notes: [], resources: '', update2_log: '',
               status: process.env.TEST_RESULT === 'cancelled' ? 'cancelled' : 'error',
               step: 'no result (runner or job failure)', duration: 0, log_tail: '', install_log: '', shot_file: '',
               update_status: '', update_log: '', shot_update_file: '',
             }), name: m.name }));
-            // Where a row ran: "pve", "incus · alpine", "arm64", or "VM" (Proxmox only).
+            // Where a row ran: "pve", "incus · alpine", "arm64", "pve · upgrade", or "VM" (Proxmox only).
             const isVm = r => r.script.startsWith('vm/');
             const isArm = r => r.platform === 'arm';
             const where = r => {
               if (isVm(r)) return 'VM';
               const p = isArm(r) ? 'arm64' : r.platform;
-              return r.os && r.os !== 'default' ? `${p} · ${r.os}` : p;
+              const w = r.os && r.os !== 'default' ? `${p} · ${r.os}` : p;
+              return r.mode === 'upgrade' ? `${w} · upgrade` : w;
             };
             // ARM is information only: shown, never counted. The result, the
             // check, the review and the approval come from Proxmox and Incus.
@@ -616,6 +657,15 @@ jobs:
             // and a log that does not fit loses its beginning: the end is where
             // a failure shows.
             const budget = Math.floor(52000 / rows.length);
+            // A check run summary holds 65535 BYTES, not characters, and log
+            // emoji take 3-4 bytes each. Cut at a character boundary and close
+            // a ~~~~ block left open.
+            const fitBytes = (text, max) => {
+              if (Buffer.byteLength(text) <= max) return text;
+              let cut = Buffer.from(text).subarray(0, max - 300).toString('utf8').replace(/�+$/, '');
+              if ((cut.match(/^~~~~/gm) || []).length % 2) cut += '\n~~~~';
+              return `${cut}\n\n… cut to fit GitHub's size limit; the full report is in the PR comment and the logs in the run artifacts.`;
+            };
             const tail = (text, max) => {
               text = fence(text).trimEnd();
               if (text.length <= max) return text;
@@ -647,6 +697,9 @@ jobs:
                 const lines = r.update_log.trimEnd().split('\n').length;
                 parts.push(details(`🔄 ${r.script} (${where(r)}) — update log (${lines} lines)`, r.update_log, updateShare));
               }
+              if (r.update2_log) {
+                parts.push(details(`🔁 ${r.script} (${where(r)}) — second update log`, r.update2_log, Math.min(r.update2_log.length, Math.floor(budget / 8))));
+              }
               return parts.join('\n\n');
             });
 
@@ -661,6 +714,22 @@ jobs:
               `${headline} at \`${process.env.SHA.slice(0, 7)}\``, '',
               table, '',
               ...(armLine ? [`<sub>${armLine}</sub>`, ''] : []),
+              ...(rows.some(r => r.mode === 'upgrade')
+                ? ['<sub>`upgrade`: installed from the base branch as it is now, then updated with this PR, as a user who installed before the merge would.</sub>', '']
+                : []),
+              ...(() => {
+                const noted = rows.filter(r => r.notes.length);
+                return noted.length
+                  ? ['**Notes** (information only, not counted):', ...noted.flatMap(r => r.notes.map(n => `- \`${r.script}\` (${where(r)}): ${clean(n)}`)), '']
+                  : [];
+              })(),
+              ...(() => {
+                const measured = rows.filter(r => r.resources);
+                return measured.length
+                  ? ['<details><summary>📊 Resources</summary>', '', '| Script | Platform | Usage |', '|---|---|---|',
+                     ...measured.map(r => `| \`${r.script}\` | ${where(r)} | ${clean(r.resources)} |`), '', '</details>', '']
+                  : [];
+              })(),
               ...(partial ? [`<sub>Built only on ${process.env.PLATFORMS.split(' ').join(', ')}; comment \`/test\` for every platform.</sub>`, ''] : []),
               ...shots, '',
               ...(() => { try { const n = JSON.parse(process.env.NOT_BUILT || '[]'); return n.length ? ['**Not built:**', ...n.map(x => `- ${x}`), ''] : []; } catch (e) { return []; } })(),
@@ -679,7 +748,7 @@ jobs:
                   title: armOnly
                     ? `ARM64 ${armPassed}/${armRows.length} passed (information only)`
                     : `${passed}/${coreRows.length} passed${armRows.length ? ` (+ ARM ${armPassed}/${armRows.length}, not counted)` : ''}`,
-                  summary: summary.slice(0, 65000),
+                  summary: fitBytes(summary, 65000),
                 },
               });
             }

.github/workflows/scripts/ci-controller-setup.sh

@@ -62,6 +62,7 @@ EOF
 }
 conf 200 10.201.146.232 pve-ci     proxmox-test community-scripts/DevScripts
 conf 201 10.201.146.231 pve-ci-2   proxmox-test community-scripts/ProxmoxVE
+conf 204 10.201.146.235 pve-ci-3   proxmox-test community-scripts/ProxmoxVE
 conf 202 10.201.146.233 incus-ci   incus-test   community-scripts/DevScripts
 conf 203 10.201.146.234 incus-ci-2 incus-test   community-scripts/ProxmoxVE
 

.github/workflows/scripts/pve-run-script.sh

@@ -48,6 +48,16 @@ URL_WAIT="${URL_WAIT:-120}"
 # script that leaves var_arm64 unset; the ARM run builds those anyway, to see.
 VAR_ARM64="${VAR_ARM64:-}"
 [[ -z "$VAR_ARM64" || "$VAR_ARM64" == "yes" ]] || { echo "bad VAR_ARM64" >&2; exit 2; }
+# upgrade: install the script as it is on the base branch (BASE_SHA of
+# UPSTREAM), then update with the PR. That is what a user does who installed
+# before the PR is merged and runs `update` after it. Empty: install and
+# update both from the PR.
+TEST_MODE="${TEST_MODE:-}"
+[[ -z "$TEST_MODE" || "$TEST_MODE" == "upgrade" ]] || { echo "bad TEST_MODE" >&2; exit 2; }
+if [[ -n "$TEST_MODE" ]]; then
+  [[ "${BASE_SHA:-}" =~ ^[0-9a-f]{40}$ ]] || { echo "bad BASE_SHA" >&2; exit 2; }
+  [[ "$SCRIPT" == ct/* ]] || { echo "upgrade tests are for ct/ scripts" >&2; exit 2; }
+fi
 [[ "$PLATFORM" == "pve" || "$SCRIPT" == ct/* ]] || { echo "vm/ scripts run on pve only" >&2; exit 2; }
 [[ "$CTID_BASE" =~ ^[0-9]+$ ]] || { echo "bad CTID_BASE" >&2; exit 2; }
 
@@ -59,12 +69,17 @@ INSTALL_LOG_OUT="$OUT_DIR/install.log"
 UPDATE_RAW="$OUT_DIR/update.raw.log"
 UPDATE_LOG="$OUT_DIR/update.log"
 UPDATE_TIMEOUT="${UPDATE_TIMEOUT:-30m}"
+UPDATE2_RAW="$OUT_DIR/update2.raw.log"
+UPDATE2_LOG="$OUT_DIR/update2.log"
+# How long the app has to keep running without a service restarting.
+STABLE_WAIT="${STABLE_WAIT:-60}"
 
 kind="${SCRIPT%%/*}" # ct or vm
 app=$(basename "$SCRIPT" .sh)
-job="${GITHUB_RUN_ID:-local}-${GITHUB_RUN_ATTEMPT:-1}-${PLATFORM}-${kind}-${app}-${OS:-default}"
+job="${GITHUB_RUN_ID:-local}-${GITHUB_RUN_ATTEMPT:-1}-${PLATFORM}-${kind}-${app}-${OS:-default}${TEST_MODE:+-$TEST_MODE}"
 tree="$SERVE_ROOT/$job"
 base="$SERVE_URL/$job"
+src_tree="$tree" # what gets installed; the base branch's tree in an upgrade test
 started=$(date +%s)
 
 status="fail"
@@ -102,6 +117,16 @@ failed_units=""
 # fail: the update broke; empty: not attempted (the install failed).
 update_status=""
 update_step=""
+# install: building and checking what was installed; update: from the first
+# update on. In an upgrade test a failure while installing is the base
+# branch's, not the PR's.
+phase="install"
+# Findings that do not fail the build, shown under the table on the PR.
+notes=()
+# Disk, RAM and OOM kills of the container, measured at the end.
+resources=""
+# Second `update` right after the first: no-update, updated-again, or empty.
+update2_status=""
 
 # Spinner frames: the engine redraws "⠋ Starting VM" a few times a second
 # without a tty too. An alternation, not a bracket: these are multibyte and
@@ -163,13 +188,61 @@ classify_cpu_failure() {
   step="needs a newer CPU than the test host has (no ${feature}): ${line}"
 }
 
+# Disk used against var_disk, RAM in use at rest (anonymous memory, without
+# the page cache that fills any limit) against the limit, and how often the
+# kernel killed something for lack of memory, over the container's whole life,
+# the build included. A dir-backed Incus root has no size of its own: df shows
+# the host's disk. Then what the container holds is counted with du, and the
+# script's var_disk is the size to compare with.
+measure_resources() {
+  local out disk_size disk_used anon limit oom want_disk want_ram line=()
+  out=$(guest_exec sh -c 'df -Pm / | awk "NR==2 {print \$2, \$3}"
+    awk "/^anon / {print int(\$2 / 1048576)}" /sys/fs/cgroup/memory.stat 2>/dev/null || echo -
+    cat /sys/fs/cgroup/memory.max 2>/dev/null || echo -
+    awk "/^oom_kill / {print \$2}" /sys/fs/cgroup/memory.events 2>/dev/null || echo -' 2>/dev/null) || return 0
+  read -r disk_size disk_used <<<"$(sed -n 1p <<<"$out")"
+  anon=$(sed -n 2p <<<"$out"); limit=$(sed -n 3p <<<"$out"); oom=$(sed -n 4p <<<"$out")
+  want_disk=$(grep -m1 -oE 'var_disk="\$\{var_disk:-[0-9]+\}"' "$src_tree/$SCRIPT" | grep -oE '[0-9]+' | head -n1)
+  want_ram=$(grep -m1 -oE 'var_ram="\$\{var_ram:-[0-9]+\}"' "$src_tree/$SCRIPT" | grep -oE '[0-9]+' | head -n1)
+  [[ "$disk_size" =~ ^[0-9]+$ && "$disk_used" =~ ^[0-9]+$ ]] || return 0
+  if [[ -n "$want_disk" ]] && ((disk_size > want_disk * 1024 * 3 / 2)); then
+    disk_size=$((want_disk * 1024))
+    disk_used=$(guest_exec sh -c 'du -sxm / 2>/dev/null | cut -f1' 2>/dev/null)
+    [[ "$disk_used" =~ ^[0-9]+$ ]] || return 0
+  fi
+  line+=("disk $(awk -v u="$disk_used" 'BEGIN {printf "%.1f", u / 1024}') of $(awk -v s="$disk_size" 'BEGIN {printf "%.0f", s / 1024}') GB")
+  if ((disk_used * 100 >= disk_size * 90)); then
+    notes+=("disk ${disk_used} of ${disk_size} MB used after the install: var_disk may be too small")
+  fi
+  [[ "$limit" =~ ^[0-9]+$ ]] && limit=$((limit / 1048576)) || limit="${want_ram:-}"
+  if [[ "$anon" =~ ^[0-9]+$ ]]; then
+    line+=("RAM at rest ${anon}${limit:+ of ${limit}} MB")
+    if [[ "$limit" =~ ^[0-9]+$ ]] && ((anon * 100 >= limit * 80)); then
+      notes+=("${anon} of ${limit} MB RAM in use at rest: var_ram may be too small")
+    fi
+  fi
+  if [[ "$oom" =~ ^[0-9]+$ ]]; then
+    line+=("OOM kills ${oom}")
+    ((oom > 0)) && notes+=("the kernel killed ${oom} process(es) for lack of memory: var_ram is too small for the build or the app")
+  fi
+  resources=$(
+    IFS='|'
+    echo "${line[*]}"
+  )
+  resources=${resources//|/ · }
+}
+
 finish() {
   [[ -f "$RAW_LOG" ]] && strip_ansi <"$RAW_LOG" >"$LOG"
   [[ -f "$RAW_LOG" ]] && collect_install_log
   [[ -f "$UPDATE_RAW" ]] && strip_ansi <"$UPDATE_RAW" >"$UPDATE_LOG"
-  rm -f "$UPDATE_RAW"
+  [[ -f "$UPDATE2_RAW" ]] && strip_ansi <"$UPDATE2_RAW" >"$UPDATE2_LOG"
+  rm -f "$UPDATE_RAW" "$UPDATE2_RAW"
+  [[ "$kind" == ct && -n "$ctid" && -z "$resources" ]] && guest_running && measure_resources
   jq -n \
     --arg script "$SCRIPT" --arg kind "$kind" --arg platform "$PLATFORM" --arg os "${OS:-default}" --arg status "$status" \
+    --arg mode "$TEST_MODE" --arg resources "$resources" --arg update2_status "$update2_status" \
+    --arg notes "$(printf '%s\n' "${notes[@]}")" \
     --arg step "$step" --arg exit_code "$exit_code" --arg sha "$PR_SHA" \
     --arg ctid "$ctid" --arg url "$url" --arg http_code "$http_code" \
     --arg failed_units "$failed_units" --arg ip "$ct_ip" --arg arch "$(uname -m)" \
@@ -179,6 +252,8 @@ finish() {
     '{script: $script, kind: $kind, platform: $platform, arch: $arch, os: $os, status: $status, step: $step, exit_code: $exit_code,
       sha: $sha, ctid: $ctid, ip: $ip, url: $url, http_code: $http_code,
       failed_units: $failed_units, update_status: $update_status, update_step: $update_step,
+      mode: $mode, resources: $resources, update2_status: $update2_status,
+      notes: ($notes | split("\n") | map(select(. != ""))),
       duration: $duration, log_tail: $tail}' >"$RESULT"
   echo "::group::result"
   jq 'del(.log_tail)' "$RESULT"
@@ -262,7 +337,7 @@ cleanup() {
     done
   fi
   rm -f "$RAW_LOG.start"
-  rm -rf -- "$tree"
+  rm -rf -- "$tree" "$tree-base"
 }
 trap cleanup EXIT
 # A cancelled job sends TERM; without this the EXIT trap would not run.
@@ -281,6 +356,10 @@ fail() {
   step="$1"
   status="${2:-fail}"
   [[ "$status" == "unsupported" ]] || status="fail"
+  if [[ "$TEST_MODE" == "upgrade" && "$phase" == "install" && "$status" == "fail" ]]; then
+    status="error"
+    step="the install before this PR (${UPSTREAM}@${BASE_SHA:0:7}) failed, so the upgrade was not tested: $1"
+  fi
   finish
   echo "::warning title=${SCRIPT}::${step}"
   exit 0
@@ -335,20 +414,37 @@ upstream_re="https://raw\.githubusercontent\.com/${UPSTREAM//./\\.}/(refs/heads/
 grep -rlE "$upstream_re" "$tree" --include='*.sh' --include='*.func' |
   xargs -r sed -i -E "s#${upstream_re}#${base}#g"
 
+# What gets installed: the PR, or in an upgrade test the base branch.
+src_tree="$tree"
+src_base="$base"
+if [[ "$TEST_MODE" == "upgrade" ]]; then
+  src_tree="$tree-base"
+  src_base="$base-base"
+  mkdir -p "$src_tree"
+  curl -fsSL --retry 3 "https://codeload.github.com/${UPSTREAM}/tar.gz/${BASE_SHA}" |
+    tar -xz -C "$src_tree" --strip-components=1 || setup_error "could not download ${UPSTREAM}@${BASE_SHA:0:7}"
+  [[ -f "$src_tree/$SCRIPT" ]] || fail "${SCRIPT} does not exist at ${UPSTREAM}@${BASE_SHA:0:7}"
+  grep -rlE "$upstream_re" "$src_tree" --include='*.sh' --include='*.func' |
+    xargs -r sed -i -E "s#${upstream_re}#${src_base}#g"
+fi
+
 if [[ "$kind" == "vm" ]]; then
   vm_build_and_check
   exit 0
 fi
 
 # --- Build ----------------------------------------------------------------------
-echo "Building ${SCRIPT} (${OS:-default OS}) on ${PLATFORM} as CT ${ctid} from ${PR_REPO}@${PR_SHA:0:7}"
+if [[ "$TEST_MODE" == "upgrade" ]]; then
+  echo "Upgrade test: installing ${SCRIPT} from ${UPSTREAM}@${BASE_SHA:0:7} (before this PR), then updating with ${PR_REPO}@${PR_SHA:0:7}"
+fi
+echo "Building ${SCRIPT} (${OS:-default OS}) on ${PLATFORM} as CT ${ctid} from $([[ "$TEST_MODE" == upgrade ]] && echo "${UPSTREAM}@${BASE_SHA:0:7}" || echo "${PR_REPO}@${PR_SHA:0:7}")"
 hostname_safe="ci-$(tr -c 'a-z0-9-\n' '-' <<<"${app,,}" | cut -c1-50)"
 build_env=(
   PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
   HOME=/root TERM=dumb LANG=C.UTF-8
   mode=default MODE=default PHS_SILENT=1 DIAGNOSTICS=no
   var_ctid="$ctid" var_hostname="$hostname_safe" var_brg="$BRIDGE"
-  "dev_mode=timing,keep" COMMUNITY_SCRIPTS_URL="$base"
+  "dev_mode=timing,keep" COMMUNITY_SCRIPTS_URL="$src_base"
 )
 if [[ "$PLATFORM" == "incus" ]]; then
   build_env+=(var_container_storage=default)
@@ -387,7 +483,7 @@ ci_value() {
   esac
 }
 declare -A answers=()
-json="$tree/json/${app}.json"
+json="$src_tree/json/${app}.json"
 if [[ -f "$json" ]]; then
   while IFS=$'\t' read -r name default; do
     answers[$name]="$default"
@@ -419,8 +515,8 @@ expand_templates() {
 
 # The engine names the install script after APP (lower case, no spaces), which
 # need not match the ct/ file name: ct/bluesky-pds.sh -> blueskypds-install.sh.
-nsapp=$(grep -m1 -oE '^APP="[^"]+"' "$tree/$SCRIPT" | cut -d'"' -f2 | tr -d ' ' | tr '[:upper:]' '[:lower:]')
-for f in "$tree/install/${app}-install.sh" "$tree/install/${nsapp}-install.sh" "$tree/$SCRIPT"; do
+nsapp=$(grep -m1 -oE '^APP="[^"]+"' "$src_tree/$SCRIPT" | cut -d'"' -f2 | tr -d ' ' | tr '[:upper:]' '[:lower:]')
+for f in "$src_tree/install/${app}-install.sh" "$src_tree/install/${nsapp}-install.sh" "$src_tree/$SCRIPT"; do
   [[ -f "$f" ]] || continue
   while read -r name; do
     [[ -v "answers[$name]" ]] || answers[$name]=""
@@ -443,7 +539,7 @@ done
 # In the background so that a TERM from a cancelled job is handled at once:
 # bash defers traps until a foreground command returns.
 env -i "${build_env[@]}" timeout --kill-after=1m "$BUILD_TIMEOUT" \
-  bash -c 'bash -c "$(curl -fsSL "$1")"' _ "$base/$SCRIPT" </dev/null >"$RAW_LOG" 2>&1 &
+  bash -c 'bash -c "$(curl -fsSL "$1")"' _ "$src_base/$SCRIPT" </dev/null >"$RAW_LOG" 2>&1 &
 build_pid=$!
 start_streams
 wait "$build_pid"
@@ -522,8 +618,12 @@ take_screenshot() {
 # open, e.g. http://${IP}:1337/admin, answers 200 (after redirects, so a jump
 # to /login is fine). A script that prints no URL has no web UI to check.
 # Run after the install and again after the update. Sets health_problem.
-pattern=$(grep -oE 'https?://\$\{IP\}(:[0-9]+)?(/[A-Za-z0-9._~/?=&%-]*)?' "$tree/$SCRIPT" | head -n1)
-[[ -n "$pattern" && -n "$ct_ip" ]] && url="${pattern/\$\{IP\}/$ct_ip}"
+app_url() { # the URL the script at $1 tells the user to open
+  local pattern
+  pattern=$(grep -oE 'https?://\$\{IP\}(:[0-9]+)?(/[A-Za-z0-9._~/?=&%-]*)?' "$1" | head -n1)
+  [[ -n "$pattern" && -n "$ct_ip" ]] && echo "${pattern/\$\{IP\}/$ct_ip}"
+}
+url=$(app_url "$src_tree/$SCRIPT")
 health_problem=""
 check_health() {
   health_problem=""
@@ -554,10 +654,10 @@ check_health screenshot.png || fail "$health_problem"
 # A fresh install is already on the latest release, so a plain `update` would
 # only say "No update available" and never run update_script's real path. The
 # version files the install wrote (~/.<app>, one per fetch_and_deploy_gh_release)
-# that update_script checks are set back to 0.0.0-ci first; check_for_gh_release then sees an update and
-# runs it all. Apps deployed from a branch (fetch_and_deploy_gh_branch, checked
-# with check_for_gh_branch) store a commit hash instead of a version; the
-# engine compares both as plain strings, so 0.0.0-ci works for them too. `update` is the /usr/bin/update the install generated: it pulls
+# that update_script checks are set back first (see below); check_for_gh_release
+# then sees an update and runs it all. Apps deployed from a branch
+# (fetch_and_deploy_gh_branch, checked with check_for_gh_branch) store a commit
+# hash instead of a version. `update` is the /usr/bin/update the install generated: it pulls
 # ct/<app>.sh from the URL this container was built from, i.e. the PR's tree.
 UPDATE_FAKE="0.0.0-ci"
 update_fail() {
@@ -581,30 +681,59 @@ tracked=$(grep -oE 'check_for_[a-z]*_?release[[:space:]]+("[^"]*"|[^[:space:]]+)
 if [[ -n "$tracked" && "$tracked" != *'$'* ]]; then
   version_files=$(grep -xF -f <(sed 's|^|/root/.|' <<<"$tracked") <<<"$version_files" || true)
 fi
+# A version keeps its digits and gets "-dev" (6.7.16.0 -> 6.7.16.0-dev): the
+# engine ranks that below the release, so the update runs, and an
+# update_script that reads the installed series (shopware: composer
+# "~6.7.0" from ~/.shopware) still gets the real one. 0.0.0-ci broke that.
+# A commit hash (or a bare number) has no series and gets 0.0.0-ci.
+declare -A update_fake=()
 for f in $version_files; do
-  guest_exec sh -c "echo '$UPDATE_FAKE' > '$f'; rm -f '$f.pending'" ||
+  cur=$(guest_exec cat "$f" 2>/dev/null | tr -d '\r\n')
+  if [[ "$cur" =~ ^v?[0-9]+(\.[0-9A-Za-z_+-]+)+$ ]]; then
+    update_fake[$f]="${cur}-dev"
+  else
+    update_fake[$f]="$UPDATE_FAKE"
+  fi
+  guest_exec sh -c "echo '${update_fake[$f]}' > '$f'; rm -f '$f.pending'" ||
     update_fail "could not reset $f"
+  echo "Update test: set back $f from ${cur} to ${update_fake[$f]}"
 done
-[[ -n "$version_files" ]] && echo "Update test: set back $(tr '\n' ' ' <<<"$version_files")to ${UPDATE_FAKE}"
 
-: >"$UPDATE_RAW"
-update_token=$(head -c 16 /dev/urandom | od -An -tx1 | tr -d ' \n')
-echo "──────── update test (live) ────────"
-echo "::stop-commands::${update_token}"
-# timeout runs a program, not a shell function, so spell out the exec here.
-if [[ "$PLATFORM" == "incus" ]]; then guest_cmd=(incus exec "$ctid" --); else guest_cmd=(pct exec "$ctid" --); fi
-timeout --kill-after=1m "$UPDATE_TIMEOUT" "${guest_cmd[@]}" env PHS_SILENT=1 mode=default MODE=default \
-  TERM=dumb DIAGNOSTICS=no var_ignore_breaking_changes=yes ${GH_API_TOKEN:+GITHUB_TOKEN="$GH_API_TOKEN"} update </dev/null \
-  > >(tee "$UPDATE_RAW" | live_filter) 2>&1 &
-update_pid=$!
-wait "$update_pid"
-update_rc=$?
-update_pid=""
-sleep 1
-echo "::${update_token}::"
+phase="update"
+# Upgrade test: the container pulls its update from where it was installed
+# from (COMMUNITY_SCRIPTS_URL in /usr/bin/update); point that at the PR, as
+# merging it would.
+if [[ "$TEST_MODE" == "upgrade" ]]; then
+  guest_exec sed -i -E "s#^export COMMUNITY_SCRIPTS_URL=.*#export COMMUNITY_SCRIPTS_URL=\"${base}\"#" /usr/bin/update &&
+    guest_exec grep -qxF "export COMMUNITY_SCRIPTS_URL=\"${base}\"" /usr/bin/update ||
+    setup_error "could not point /usr/bin/update at the PR"
+  echo "Upgrade test: /usr/bin/update now pulls from ${PR_REPO}@${PR_SHA:0:7}"
+fi
+
+# run_update <raw log> <title>: runs `update` in the container, live in the
+# job log; sets update_rc.
+run_update() {
+  local raw=$1 token
+  : >"$raw"
+  token=$(head -c 16 /dev/urandom | od -An -tx1 | tr -d ' \n')
+  echo "──────── $2 (live) ────────"
+  echo "::stop-commands::${token}"
+  # timeout runs a program, not a shell function, so spell out the exec here.
+  if [[ "$PLATFORM" == "incus" ]]; then guest_cmd=(incus exec "$ctid" --); else guest_cmd=(pct exec "$ctid" --); fi
+  timeout --kill-after=1m "$UPDATE_TIMEOUT" "${guest_cmd[@]}" env PHS_SILENT=1 mode=default MODE=default \
+    TERM=dumb DIAGNOSTICS=no var_ignore_breaking_changes=yes ${GH_API_TOKEN:+GITHUB_TOKEN="$GH_API_TOKEN"} update </dev/null \
+    > >(tee "$raw" | live_filter) 2>&1 &
+  update_pid=$!
+  wait "$update_pid"
+  update_rc=$?
+  update_pid=""
+  sleep 1
+  echo "::${token}::"
+}
+run_update "$UPDATE_RAW" "update test"
 
 update_failure_line() {
-  strip_ansi <"$UPDATE_RAW" | grep -E '✖|\[ERROR\]|error:' | tail -n 1 | sed -E 's/^\s*(✖️?|\s)*//' | cut -c1-180
+  strip_ansi <"${1:-$UPDATE_RAW}" | grep -E '✖|\[ERROR\]|error:' | tail -n 1 | sed -E 's/^\s*(✖️?|\s)*//' | cut -c1-180
 }
 if ((update_rc == 124 || update_rc == 137)); then
   update_fail "timed out after ${UPDATE_TIMEOUT}"
@@ -629,8 +758,8 @@ fi
 if strip_ansi <"$UPDATE_RAW" | grep -qi 'updated successfully'; then
   update_status="updated"
   for f in $version_files; do
-    [[ "$(guest_exec cat "$f" 2>/dev/null)" != "$UPDATE_FAKE" ]] ||
-      update_fail "reported success, but $f is still ${UPDATE_FAKE}"
+    [[ "$(guest_exec cat "$f" 2>/dev/null)" != "${update_fake[$f]}" ]] ||
+      update_fail "reported success, but $f is still ${update_fake[$f]}"
   done
 else
   # Ran cleanly without updating anything: no version file to fake (apt/npm
@@ -639,7 +768,69 @@ else
   update_step=$(strip_ansi <"$UPDATE_RAW" | grep -iE 'no update|up to date|up-to-date|already' | tail -n 1 | sed -E 's/^\s*(✔️?|ℹ️?|\s)*//' | cut -c1-160)
 fi
 
+url=$(app_url "$tree/$SCRIPT")
 check_health screenshot-update.png || update_fail "after the update: $health_problem"
 
+# --- Second update ------------------------------------------------------------------
+# Right after an update, `update` has nothing to do: it must say so and leave
+# the app alone. One that breaks on a second run breaks for every user on
+# their next update; one that updates again although nothing changed
+# rebuilds on every run (often a version file it never writes).
+if [[ "$update_status" == "updated" ]]; then
+  run_update "$UPDATE2_RAW" "second update (must have nothing to do)"
+  if ((update_rc == 124 || update_rc == 137)); then
+    update_fail "second run timed out after ${UPDATE_TIMEOUT}"
+  elif ((update_rc != 0)); then
+    last=$(update_failure_line "$UPDATE2_RAW")
+    update_fail "second run exited ${update_rc}${last:+: $last}"
+  fi
+  update_error=$(strip_ansi <"$UPDATE2_RAW" | grep -E '✖|\[ERROR\]' | tail -n 1 | sed -E 's/^\s*(✖️?|\s)*//' | cut -c1-180)
+  if [[ -n "$update_error" && "${update_error,,}" != *"rate limit"* ]]; then
+    update_fail "second run: $update_error"
+  fi
+  if strip_ansi <"$UPDATE2_RAW" | grep -qi 'updated successfully'; then
+    update2_status="updated-again"
+    notes+=("a second \`update\` right after the first updated again: update_script does not notice it is current (e.g. a version file it never writes)")
+  else
+    update2_status="no-update"
+  fi
+  check_health screenshot-update.png || update_fail "after the second update: $health_problem"
+fi
+
+# --- Does it stay up? -----------------------------------------------------------
+# A service that crash-loops can be "active" at every single look. Count the
+# restarts of every service (and Docker container) over STABLE_WAIT seconds:
+# any that went up was restarted while nobody touched the container.
+restart_counts() {
+  guest_exec sh -c '
+    if command -v systemctl >/dev/null; then
+      systemctl list-units --type=service --all --no-legend --plain | awk "{print \$1}" |
+        while read -r u; do echo "$u $(systemctl show -p NRestarts --value "$u" 2>/dev/null)"; done
+    fi
+    if command -v docker >/dev/null; then
+      docker ps -aq | xargs -r docker inspect -f "docker:{{.Name}} {{.RestartCount}}" 2>/dev/null
+    fi' 2>/dev/null | awk '$2 ~ /^[0-9]+$/' | sort
+}
+before=$(restart_counts)
+sleep "$STABLE_WAIT"
+after=$(restart_counts)
+restarting=$(join <(echo "$before") <(echo "$after") | awk '$3 > $2 {printf "%s%s (+%d)", sep, $1, $3 - $2; sep=", "}')
+[[ -z "$restarting" ]] || fail "not stable: restarted within ${STABLE_WAIT}s without anyone touching it: ${restarting}"
+check_health screenshot-update.png || fail "not stable: ${STABLE_WAIT}s later, ${health_problem}"
+
+# Error lines in the journal since boot: often harmless, sometimes the reason
+# a feature is broken. Information only.
+if guest_exec sh -c 'command -v journalctl' &>/dev/null; then
+  # Without the noise every container produces: udev may not trigger devices
+  # in an unprivileged container, and logind finds no VTs.
+  journal_errors=$(guest_exec journalctl -b -p err -q -o cat --no-pager 2>/dev/null | grep -vE '^\s*$' |
+    grep -vE "Failed to write 'add' to '/sys/.*/uevent'|Seat has no VTs but VT number not 0" | sort | uniq -c | sort -rn)
+  if [[ -n "$journal_errors" ]]; then
+    n=$(awk '{s += $1} END {print s}' <<<"$journal_errors")
+    top=$(head -n 2 <<<"$journal_errors" | sed -E 's/^\s*[0-9]+\s+//' | cut -c1-120 | paste -sd'|' - | sed 's/|/; /g')
+    notes+=("${n} error line(s) in the journal since boot, e.g. ${top}")
+  fi
+fi
+
 status="pass"
 finish

CHANGELOG.md

@@ -591,8 +591,19 @@ Exercise vigilance regarding copycat or coat-tailing sites that seek to exploit
 
   - #### 🐞 Bug Fixes
 
+    - OnlyOffice: change ct port to 8083 [@MickLesk](https://github.com/MickLesk) ([#18642](https://github.com/community-scripts/ProxmoxVE/pull/18642))
+    - dotnetaspwebapi: start the service only once an app is deployed [@MickLesk](https://github.com/MickLesk) ([#18639](https://github.com/community-scripts/ProxmoxVE/pull/18639))
     - Whisparr-eros: set the first login and save it [@MickLesk](https://github.com/MickLesk) ([#18617](https://github.com/community-scripts/ProxmoxVE/pull/18617))
 
+  - #### 💥 Breaking Changes
+
+    - Updated ASP.NET Web App script to Debian and adds a .Net selection [@kris701](https://github.com/kris701) ([#18628](https://github.com/community-scripts/ProxmoxVE/pull/18628))
+
+### 📚 Documentation
+
+  - github: bump workflow actions to their current majors [@MickLesk](https://github.com/MickLesk) ([#18637](https://github.com/community-scripts/ProxmoxVE/pull/18637))
+- pocketbase-bot: apply website metadata requests from Accept/Decline boxes [@MickLesk](https://github.com/MickLesk) ([#18631](https://github.com/community-scripts/ProxmoxVE/pull/18631))
+
 ## 2026-11-11
 
 ### 🚀 Updated Scripts

ct/dotnetaspwebapi.sh

@@ -4,17 +4,17 @@ source "$_cs_boot" 2>/dev/null || source <(curl -fsSL "${COMMUNITY_SCRIPTS_CORE_
 # Copyright (c) 2021-2026 community-scripts ORG
 # Author: Kristian Skov
 # License: MIT | https://github.com/community-scripts/ProxmoxVE/raw/main/LICENSE
-# Source: https://learn.microsoft.com/en-us/aspnet/core/host-and-deploy/linux-nginx?view=aspnetcore-9.0&tabs=linux-ubuntu
+# Source: https://learn.microsoft.com/en-us/dotnet/core/install/linux-debian
 
 APP="Dotnet ASP Web API"
 var_tags="${var_tags:-web}"
 var_cpu="${var_cpu:-1}"
 var_ram="${var_ram:-1024}"
 var_disk="${var_disk:-8}"
-var_os="${var_os:-ubuntu}"
-var_version="${var_version:-24.04}"
+var_os="${var_os:-debian}"
+var_version="${var_version:-13}"
 var_arm64="${var_arm64:-yes}"
-var_unprivileged="${var_unprivileged:-0}"
+var_unprivileged="${var_unprivileged:-1}"
 
 header_info "$APP"
 variables
@@ -43,4 +43,4 @@ description
 msg_ok "Completed successfully!\n"
 echo -e "${CREATING}${GN}${APP} setup has been successfully initialized!${CL}"
 echo -e "${INFO}${YW}Access it using the following IP:${CL}"
-echo -e "${GATEWAY}${BGN}${IP}:80${CL}"
+echo -e "${GATEWAY}${BGN}${IP}:80${CL}"

ct/onlyoffice.sh

@@ -52,4 +52,4 @@ description
 msg_ok "Completed successfully!\n"
 echo -e "${CREATING}${GN}${APP} setup has been successfully initialized!${CL}"
 echo -e "${INFO}${YW}Access it using the following URL:${CL}"
-echo -e "${GATEWAY}${BGN}http://${IP}${CL}"
+echo -e "${GATEWAY}${BGN}http://${IP}:8083${CL}"

install/dotnetaspwebapi-install.sh

@@ -3,7 +3,7 @@
 # Copyright (c) 2021-2026 community-scripts ORG
 # Author: Kristian Skov
 # License: MIT | https://github.com/community-scripts/ProxmoxVE/raw/main/LICENSE
-# Source: https://learn.microsoft.com/en-us/aspnet/core/host-and-deploy/linux-nginx?view=aspnetcore-9.0&tabs=linux-ubuntu
+# Source: https://learn.microsoft.com/en-us/dotnet/core/install/linux-debian
 
 source /dev/stdin <<<"$FUNCTIONS_FILE_PATH"
 color
@@ -14,21 +14,36 @@ network_check
 update_os
 
 msg_info "Installing Dependencies"
-apt_update_safe
 $STD apt install -y \
-  ssh \
-  software-properties-common
-
-$STD add-apt-repository -y ppa:dotnet/backports
-$STD apt install -y \
-  dotnet-sdk-9.0 \
   vsftpd \
   nginx
 msg_ok "Installed Dependencies"
 
-var_project_name="${var_project_name:-}"
-[[ -n "$var_project_name" ]] || read -r -p "${TAB3}Type the assembly name of the project: " var_project_name || true
-[[ -n "$var_project_name" ]] || { msg_error "No assembly name given; set var_project_name or answer the prompt"; exit 64; }
+setup_deb822_repo \
+  "microsoft" \
+  "https://packages.microsoft.com/keys/microsoft-2025.asc" \
+  "https://packages.microsoft.com/debian/13/prod/" \
+  "trixie" \
+  "main"
+
+if [[ -z "${var_dotnet_version:-}" && -t 0 ]]; then
+  read -r -t 60 -p "${TAB3}.NET SDK version, 10 (LTS) or 11 [10]: " var_dotnet_version || echo
+fi
+var_dotnet_version="${var_dotnet_version:-10}"
+var_dotnet_version="${var_dotnet_version%%.*}"
+if ! apt-cache show "dotnet-sdk-${var_dotnet_version}.0" &>/dev/null; then
+  msg_warn ".NET SDK ${var_dotnet_version} is not available for this system, using 10"
+  var_dotnet_version="10"
+fi
+
+if [[ -z "${var_project_name:-}" && -t 0 ]]; then
+  read -r -t 60 -p "${TAB3}Assembly name of the project [testproject]: " var_project_name || echo
+fi
+var_project_name="${var_project_name:-testproject}"
+
+msg_info "Installing .NET SDK ${var_dotnet_version}"
+$STD apt install -y "dotnet-sdk-${var_dotnet_version}.0"
+msg_ok "Installed .NET SDK ${var_dotnet_version}"
 
 msg_info "Setting up FTP Server"
 useradd ftpuser
@@ -55,7 +70,7 @@ msg_ok "FTP server setup completed"
 msg_info "Setting up Nginx Server"
 rm -f /var/www/html/index.nginx-debian.html
 
-sed "s/\$var_project_name/$var_project_name/g" >myfile <<'EOF' >/etc/nginx/sites-available/default
+sed "s/\$var_project_name/$var_project_name/g" <<'EOF' >/etc/nginx/sites-available/default
 map $http_connection $connection_upgrade {
   "~*Upgrade" $http_connection;
   default keep-alive;
@@ -82,12 +97,12 @@ msg_info "Creating Service"
 cat <<EOF >/etc/systemd/system/kestrel-aspnetapi.service
 [Unit]
 Description=.NET Web API App running on Linux
+ConditionPathExists=/var/www/html/$var_project_name.dll
 
 [Service]
 WorkingDirectory=/var/www/html
 ExecStart=/usr/bin/dotnet /var/www/html/$var_project_name.dll
 Restart=always
-# Restart service after 10 seconds if the dotnet service crashes:
 RestartSec=10
 KillSignal=SIGINT
 SyslogIdentifier=dotnet-${var_project_name}
@@ -98,7 +113,18 @@ Environment=DOTNET_NOLOGO=true
 [Install]
 WantedBy=multi-user.target
 EOF
-systemctl enable -q --now kestrel-aspnetapi
+cat <<EOF >/etc/systemd/system/kestrel-aspnetapi.path
+[Unit]
+Description=Start the .NET Web API App once it is deployed
+
+[Path]
+PathExists=/var/www/html/$var_project_name.dll
+
+[Install]
+WantedBy=multi-user.target
+EOF
+systemctl enable -q kestrel-aspnetapi
+systemctl enable -q --now kestrel-aspnetapi.path
 msg_ok "Created Service"
 
 motd_ssh