lei.chen/myxce
1
0
Fork 1
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore(wolf): checkpoint buglog.json (951 bugs)

Browse source
This commit is contained in:
Rongsong Shen 2026-08-17 23:45:24 +00:00
parent 2954f697a3
commit a336884fc4
Signed by: rshen
SSH key fingerprint: SHA256:8Vf2pZ3h6rJa7EhGK+WaAQ9Mta/apZvteW6g9U/WKKk
1 changed files with 23 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

23
.wolf/buglog.json
View file

@ -16995,6 +16995,29 @@
],
"occurrences": 4,
"last_seen": "2026-08-17T12:32:00Z"
},
{
"id": "bug-1570",
"timestamp": "2026-08-17T23:45:00Z",
"error_message": "resource-value-verification task 2.1 round-1 REJECTED: `resupdate -n mem -v 5 -v 8` silently overwrote to 8 (last-write-wins) instead of rejecting the duplicate -v and keeping the first value 5",
"file": "src/controller/testsuite/test.c",
"root_cause": "cmd_resupdate()'s deferred-finalization refactor (needed so a trailing -t can still apply to its pending -n/-v pair) rewrote the 'v' case to `if (resname != NULL) { resval_str = optarg; }` -- it dropped the pre-existing guard that rejected a second -v for the same pending -n. Baseline (pre-task-2.1, git show 98b8c34b^) fired set_resval() immediately on -v and reset resname=NULL, so a second -v naturally hit the 'no res name found' else-branch; the deferred model keeps resname non-NULL across multiple -v's, so the missing resval_str==NULL check let the second -v silently clobber the first.",
"fix": "Changed the 'v' case guard to `if (resname != NULL && resval_str == NULL)` -- only accept a value into the pending slot if a name is pending AND no value has already been accepted for it; a duplicate -v now prints the same `ignore res value \\`%s' as no res name found` warning and the first value is kept. Verified via full case-by-case trace against the git-show baseline (duplicate -v, two full -n/-v pairs, -n flushing a valueless prior -n, stray -v with no -n, trailing bare -n, and the new -t override case) plus a live single-node controller smoke test: `resupdate -n mem -v 5 -v 8` on a real joined worker prints the ignore-warning for `8` and `resval -n mem -m localhost` confirms the stored value is 5.000000, not 8.",
"tags": [
"resupdate",
"cmd_resupdate",
"getopt",
"duplicate-flag",
"last-write-wins",
"resource-value-verification",
"task-2.1",
"round-2-remediation"
],
"related_bugs": [
"bug-1569"
],
"occurrences": 1,
"last_seen": "2026-08-17T23:45:00Z"
}
]
}