From fc0ad5f09e79c5f5b1cd98d64f4eff5b98abaa56 Mon Sep 17 00:00:00 2001 From: "anthony.wen" Date: Tue, 24 Mar 2026 22:32:58 -0400 Subject: [PATCH] Add kernel lookup to ATVM status template - update the ATVM status template so the HOSTS: table includes a Kernel column after Host - document that kernel values should be resolved by cross-referencing hostnames in atvm/inventory/vm-inventory.md - document that unknown should be used when a kernel value cannot be verified from the VM inventory - align the ATVM automation guide so local status output and Mattermost posts use the kernel-aware host table --- atvm/docs/automation/guide.md | 3 +++ atvm/docs/automation/status-template.md | 15 +++++++++------ 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/atvm/docs/automation/guide.md b/atvm/docs/automation/guide.md index acdf217..78d711d 100644 --- a/atvm/docs/automation/guide.md +++ b/atvm/docs/automation/guide.md @@ -210,8 +210,11 @@ Status-report expectations: - Use the same display layout for every ATVM automation status response regardless of test type (`e2e`, `systemOS`, `reboot`, `migrateops`, and others). - Use `/home/aw/code/cds/atvm/docs/automation/status-template.md` as the default template for both local status output and Mattermost status posts. - The default ATVM status template uses flat bullet-list sections for `COVERAGE:` and `FUNCTIONALLY:`, Markdown tables for `SUMMARY:`, `HOSTS:`, and `TIMING:`, and uses `NOTES:` for flat operator-facing notes. +- The `HOSTS:` table includes `Host`, `Kernel`, `Status`, and `Detail` columns in that order. - In `COVERAGE:`, describe the template, datastore/config family, migration style, and plugin/integration path, but do not list target hosts there. - In `FUNCTIONALLY:`, summarize what the run is intended to do at a high level, even when the run failed before reaching those steps. +- For the `Kernel` column, cross-reference the host name against `/home/aw/code/cds/atvm/inventory/vm-inventory.md`. +- If the hostname is not present in `vm-inventory.md`, report the kernel value as `unknown`. - Treat references to the "ATVM automation run" or "automation run" as referring to this ATVM folder workflow and the automation VM at `192.168.3.190`, not to Cirrus project operations such as the `atvm - cypress` project. - Treat a status request as a request for live status by default. - Unless the operator explicitly asks to send the status to Mattermost, print the status only in the local terminal response. diff --git a/atvm/docs/automation/status-template.md b/atvm/docs/automation/status-template.md index fece4d0..2041405 100644 --- a/atvm/docs/automation/status-template.md +++ b/atvm/docs/automation/status-template.md @@ -33,12 +33,12 @@ Use this as the default ATVM automation run-status template for: **HOSTS:** -| Host | Status | Detail | -|---|---|---| -| | ✅ PASS | completed | -| | ⚠️ FAIL | | -| | ⏳ RUN | in progress | -| | ⏭️ SKIP | | +| Host | Kernel | Status | Detail | +|---|---|---|---| +| | | ✅ PASS | completed | +| | | ⚠️ FAIL | | +| | | ⏳ RUN | in progress | +| | | ⏭️ SKIP | | **TIMING:** @@ -64,6 +64,7 @@ Use this as the default ATVM automation run-status template for: - Use flat bullet lists for `COVERAGE:` and `FUNCTIONALLY:`. - Use Markdown tables for `SUMMARY:`, `HOSTS:`, and `TIMING:`. - Use one host per row in the `HOSTS:` section. +- Include a `Kernel` column immediately after `Host` in the `HOSTS:` table. - For completed hosts, prefer: - `✅ PASS` - `⚠️ FAIL` @@ -74,4 +75,6 @@ Use this as the default ATVM automation run-status template for: - Put broader context under `NOTES:`, not in the host table. - `COVERAGE:` should describe what the run was intended to cover without listing target hosts. - `FUNCTIONALLY:` should describe the intended workflow steps at a high level, even if the run failed before completing them. +- Resolve kernel values by cross-referencing hostnames against `/home/aw/code/cds/atvm/inventory/vm-inventory.md`. +- If no kernel value can be verified from `vm-inventory.md`, use `unknown`. - Use the same template for Mattermost and local operator-visible status output.