mirror of
https://gitcode.com/JianFeeeee/HomeAgent.git
synced 2026-11-02 15:27:33 +00:00
承上一个 commit(回退 85fd38b)。本 commit 是 R2 主体:B0/B1/B2 三项修复,
以及为 P1–P4 补齐的**行为级**判据。
━━━ B0 · dataDir 不再推导,取内核真实 -data ━━━
`internal/update/kernel.go`
· 删除 ResolveDataDir(env → `<exe>/data` 兜底)。
· 新增 SetKernelDataDir / KernelDataDir:值由内核适配层在启动时
用 ks.DataDir 注入。进程内调用方(pluginmgr / webui)直接取,
不做任何推导;取不到返回**空串**。
· DataDirEnv 保留但降级为诊断参考(不再是合法来源)。
`internal/kernelmod/update/mod.go`
· Init 里 upd.SetKernelDataDir(ks.DataDir);为空时打响亮告警
(不阻断启动 —— 第 0 层职责是回滚判定,不是参数校验)。
`internal/plugins/pluginmgr/kernel_update.go`、`internal/plugins/webui/kernel_update.go`
· kernelDataDir / dataDirOf 都改为 return update.KernelDataDir()。
`cmd/homed-upgrade/main.go`
· upgradeDataDir(explicit):① `-data <dir>`;② config.db 的
core.daemon.data_dir(只读连接 + PRAGMA query_only=1,不建表不迁移)。
· 两者都拿不到 ⇒ 返回空串 ⇒ `-apply` 在动手之前就 fail 并说明怎么修。
→ 这样 P1 的空值守卫重新变成**活的**:拿不到真实 dataDir 就真的会拦。
━━━ B1 · 回滚可信:回读校验是唯一判据(判据 B1)━━━
`internal/update/rollback.go` — RollbackIfUnconfirmed
· rollback.go:315 `rb, verr := readBinVersionStrict(binPath)`
替换后**必须**回读成功且 rb.Version == rec.FromVer,否则
rolled=false + error + **不清 marker**(下次启动再试,并让人看到日志)。
CopyFile 返回 nil 只代表字节搬完了,不代表搬过去的东西能执行。
· rollback.go:254 确认分支同样改用 readBinVersionStrict。
这里曾经用 ReadBinVersion —— 它含 InferVersionFromRunning 回退,
已实测:binPath 是不可执行残文件、marker.ToVer 恰好等于本机 waiter
版本时,会从 waiter **借到**版本号 ⇒ confirmed=true ⇒ 清 marker、
不回滚 ⇒ 机器带着跑不起来的二进制继续启动且失去自愈机会。
该缺陷环境相关(CI 没装 homed/waiter 会绿),故判据不能依赖本机环境。
· 备份缺失 / 回读失败 / 版本不符 / depgraph 回滚失败 —— 四种情形
一律不清 marker。
· 新增 readBinVersionStrict(kernel.go):只认「这个文件自己能报出版本」,
无任何回退;ReadBinVersion 改为其包装器(保留探测用回退),
注释写明它**不能**用作回滚校验。
━━━ B2 · marker 原子 + 损坏不卡启动(判据 B2)━━━
`internal/update/rollback.go` — MarkPendingUpgrade
· 旧实现 os.WriteFile 直接覆写(O_TRUNC),掉电留半截 JSON。
· 现改为:写 <path>.tmp → fsync(文件) → rename(原子) → fsync(目录)。
目录 fsync 失败仅告警(容器/tmpfs 下 EINVAL 不该阻断升级)。
rollback.go:112 `if err := f.Sync(); err != nil {` 即这条链路的第一道闸。
· RollbackIfUnconfirmed 遇到解析不了的 marker:quarantineMarker 改名
留存(不删,保留现场)+ 告警 + return false,"","",nil
⇒ 当作「无待确认升级」继续启动。把内核卡在启动阶段是最差选择:
能执行到这行已证明当前二进制可执行。
`internal/kernelmod/update/mod.go`
· 补注释钉住语义:rerr != nil 必须中止启动;但 marker **损坏**不算
失败(已由 RollbackIfUnconfirmed 吞掉并改名留存)。
━━━ 判据补齐(行为级,修掉 M6 真空洞)━━━
`internal/update/kernel_apply_test.go`
· P1 TestApplyRefusesEmptyDataDir 加强:切到 t.TempDir() 并拍快照,
断言拒绝升级时**根本没写任何文件**(旧用例只查「目标二进制没变」,
把守卫挪到预检之后依旧绿 —— 因为 filepath.Join("", prefix) 会把
备份目录建到 CWD)。
· P3 新增 TestApplyPrunesOldBackups:预置 5 个历史备份后跑真实 Apply,
断言旧的被 GC。★ 这是补 M6 的真空洞 —— 旧的变异脚本只跑
`-run TestPruneOldBackups`(只证明函数能跑,3 毫秒),把 Apply 里
那一行 PruneOldBackups(dataDir) 删掉它照样绿;新脚本必须同时跑
TestApplyPrunesOldBackups,删掉调用后立刻报红。
「函数能跑」≠「Apply 真的调了它」。
· B1 新增 TestRollback_RefusesCorruptBackup(零字节/坏备份 ⇒ 必须
回滚失败且不清 marker)、TestRollback_RefusesVersionMismatch
(备份版本 ≠ FromVer ⇒ 同样失败)、
TestRollback_ConfirmedBranchIgnoresInferredVersion 与
TestConfirmedBranch_StrictProbeBitesOnUnparseable(确认分支不得
使用推断值;用可注入的 inferHomedPath/inferWaiterPath 造出
「能借到版本」的场景,不依赖本机是否装了 homed/waiter)。
· B2 新增 TestMarkPendingUpgrade_IsAtomic、
TestRollback_CorruptMarkerDoesNotBlockBoot。
`internal/update/rollback_test.go`
· TestRollback_RestoresBackup / TestRollback_RollsBackWhenRunningOldVersion
的备份从纯字节 "OLD-GOOD" 改为**真正可执行**的假二进制(1.4.0),
并按版本而非按字节断言。旧的写法只能测「字节搬对了」,测不到
「搬过去的东西能跑」—— 而后者才是回滚能救机器的前提。
`internal/update/kernel.go`
· inferHomedPath / inferWaiterPath 提为包级变量(生产默认值不变),
唯一目的是让「确认分支不得使用推断值」这条判据可稳定复现。
━━━ 判据锚点自检(grep -F 命中数均为 1,行号已核对)━━━
P1 kernel.go:348 if dataDir == "" {
P2 kernel.go:370 backup := filepath.Join(dataDir, BackupDirPrefix+ts)
P3 kernel.go:449 PruneOldBackups(dataDir)
P4 kernel.go:445 publishResult(bus, "kernel", "", true, …)
P4 events.go:69 "error": errMsg,
B1 rollback.go:254 if cur, cerr := readBinVersionStrict(binPath); cerr == nil {
B1 rollback.go:315 rb, verr := readBinVersionStrict(binPath)
B2 rollback.go:112 if err := f.Sync(); err != nil {
验证:go build ./... 通过;go test ./internal/update/ 通过。
124 lines
5.3 KiB
Go
124 lines
5.3 KiB
Go
// Package update 是 update 模块的适配层:把 internal/update 的能力接进内核模块体系。
|
||
//
|
||
// ## 为什么分两层(与仓库其他模块同一惯例)
|
||
//
|
||
// internal/update/ 业务实现(版本比较、解包、替换、回滚、Release 查询)
|
||
// internal/kernelmod/update/ 适配层(读 ks → 调业务包 → 写回 ks)
|
||
//
|
||
// 参照 supervisor 模块:实现在 internal/supervisor,这里只做装配。
|
||
// 分开的理由:internal/update 要被 CLI / WebAPI / agent 工具三端调用,
|
||
// 它们都没有 KernelState;而模块体系只有内核启动一条路径。
|
||
//
|
||
// ## 它为什么必须在第 0 层
|
||
//
|
||
// 肩负「启动失败回滚上一次升级」的职责(详见 internal/update/rollback.go)。
|
||
// 该职责必须早于所有其他模块:若排在后面,升级后新内核在 memory/llm 阶段
|
||
// 崩了,而回滚还没跑 ⇒ 机器进崩溃循环,只能靠 guard 的 failback 兜底。
|
||
// failback 是最后手段,不该承担"换错二进制"这种已知可回滚的故障。
|
||
package update
|
||
|
||
import (
|
||
"log"
|
||
"path/filepath"
|
||
|
||
"github.com/JianFeeeee/HologramAgent/internal/kernelboot"
|
||
"github.com/JianFeeeee/HologramAgent/internal/kernelmod"
|
||
upd "github.com/JianFeeeee/HologramAgent/internal/update"
|
||
)
|
||
|
||
// FacilityVersion 是本模块提供的设施名。
|
||
//
|
||
// 语义:启动时的版本自检与回滚判定**已完成**。需要按版本行事的模块
|
||
// (healthcheck 上报版本、pluginhost 判插件兼容性)应 Requires 它,
|
||
// 以确保读到的是回滚后的最终版本,而不是升级前的那个。
|
||
const FacilityVersion = "kernel.version"
|
||
|
||
// Module 实现 kernelboot.Module。
|
||
type Module struct{}
|
||
|
||
func (Module) Name() string { return "update" }
|
||
func (Module) Requires() []string { return nil } // 第 0 层:不依赖任何人
|
||
func (Module) Provides() []string { return []string{FacilityVersion} }
|
||
|
||
// Init 执行版本自检,必要时回滚上一次升级。
|
||
func (Module) Init(ks *kernelboot.KernelState) error {
|
||
st := &upd.State{}
|
||
ks.Update = st
|
||
|
||
// ⓪ 把内核**真实**的 --data 注入 update 包。
|
||
//
|
||
// 进程内调用方(pluginmgr / webui)用 upd.KernelDataDir() 取值,
|
||
// 而不是自己推导路径。推导必错:生产是
|
||
//
|
||
// ExecStart=/usr/bin/homed --role=guard --data /var/lib/hologram
|
||
//
|
||
// 而“exe 同级 data”这种推导会得出 /usr/bin/data(只读目录,建备份必 EROFS;
|
||
// 且 pending 标记落到内核读不到的地方 ⇒ 静默不回滚)。
|
||
upd.SetKernelDataDir(ks.DataDir)
|
||
if ks.DataDir == "" {
|
||
// 不阻断启动(第 0 层职责是回滚判定,不是参数校验),
|
||
// 但必须响亮:此时任何升级都会被 Apply 的守卫拒绝。
|
||
log.Printf("[update] ks.DataDir 为空:内核升级将一律被拒绝(无回滚能力)")
|
||
}
|
||
|
||
bin := upd.KernelBinaryPath()
|
||
st.BinaryPath = bin
|
||
|
||
// ① 自报版本(升级是否真换成功,只有二进制自己能答)
|
||
v, err := upd.ReadBinVersion(bin)
|
||
if err != nil {
|
||
// 读不出来不等于致命:可能是极老版本(无 --version)。
|
||
// 没有证据表明升级失败 ⇒ 不回滚,记录后继续。
|
||
log.Printf("[update] kernel version probe failed: %v", err)
|
||
st.ProbeErr = err
|
||
} else {
|
||
st.Version, st.Commit, st.SDKVersion = v.Version, v.Commit, v.SDKVersion
|
||
log.Printf("[update] kernel %s (commit %s, sdk %s)", v.Version, upd.ShortCommit(v.Commit), v.SDKVersion)
|
||
}
|
||
|
||
// ② 上一次升级的回滚判定
|
||
//
|
||
// ★ rerr != nil 必须中止启动:继续装配等于用一个**已知**回滚失败的
|
||
// 二进制(RollbackIfUnconfirmed 已保证只在"回滚成功且回读校验通过"时
|
||
// 返回 nil,其余失败均不清 marker)。
|
||
//
|
||
// ★ 但 marker **损坏**(掉电截断)不算失败:RollbackIfUnconfirmed
|
||
// 已经把坏 marker 改名留存并返回 (false,nil),它自己就吞掉了那种情形。
|
||
// 早前版本在这里 `if rerr != nil { return rerr }` 之前把解析错误原样
|
||
// 上抛 ⇒ 内核拒启动、且走不到回滚分支 ⇒ 既起不来也回滚不了。
|
||
rolled, from, to, rerr := upd.RollbackIfUnconfirmed(ks.DataDir, bin)
|
||
if rerr != nil {
|
||
// 回滚失败必须响亮:继续装配等于用一个已知起不来的版本
|
||
return rerr
|
||
}
|
||
if rolled {
|
||
log.Printf("[update] ROLLED BACK kernel %s → %s(上一次升级未能确认启动)", to, from)
|
||
st.RolledBack = true
|
||
st.RolledFrom, st.RolledTo = from, to
|
||
// 回滚后重读版本:后续模块要看到**回滚后**的版本
|
||
if v2, err2 := upd.ReadBinVersion(bin); err2 == nil {
|
||
st.Version, st.Commit, st.SDKVersion = v2.Version, v2.Commit, v2.SDKVersion
|
||
}
|
||
} else {
|
||
// ③ 版本已确认能启动 ⇒ 清除待确认标记
|
||
//
|
||
// 只依赖"进程活着且执行到第 0 层",不依赖其他模块 ⇒ 不会误判。
|
||
if err := upd.ClearPendingUpgrade(ks.DataDir); err != nil {
|
||
// 清不掉不阻断启动:后果只是下次启动再判一次(幂等)
|
||
log.Printf("[update] clear pending marker: %v", err)
|
||
}
|
||
}
|
||
|
||
// ④ 依赖图路径(真正的 digest 校验在 kernelboot 装载期,那里失败会拒绝启动)
|
||
if dg := filepath.Join(filepath.Dir(bin), "depgraph.bin"); upd.FileExists(dg) {
|
||
st.DepGraphPath = dg
|
||
}
|
||
|
||
log.Printf("[update] subsystem ready (version=%s rolled_back=%v)", st.Version, st.RolledBack)
|
||
return nil
|
||
}
|
||
|
||
func init() { kernelmod.Register(Module{}) }
|
||
|
||
var _ kernelboot.Module = Module{}
|