sean/seanfarm
Watch
1
0
Fork
You've already forked seanfarm
 0
Code Issues Pull requests Projects Releases 386 Packages Wiki Activity

fix(test): pin the Binjovi kustomization by resource name #146

Merged
binjovi-bot merged 1 commit from fix/kustomization-name-pin into trunk 2026-09-01 16:28:31 +00:00
Conversation 0 Commits 1 Files changed 2 +50 -1
sean commented 2026-09-01 15:45:47 +00:00
Owner
Copy link

tests/binjovi.sh pinned a raw count of the resources in the Binjovi kustomization. That pin failed twice in one evening under concurrent writers, and each failure carried no information beyond two numbers: the fix each time was to bump a literal. A count also passes silently when one added resource replaces one removed one.

The guard now diffs the ordered resource-name list. Proven red and green: adding a phantom resource makes it print the added name plus the instruction to update the list in the same commit, and removing the phantom makes it green again.

https://claude.ai/code/session_01LeVatedQ4sCA1u6LHjo7B5

tests/binjovi.sh pinned a raw count of the resources in the Binjovi kustomization. That pin failed twice in one evening under concurrent writers, and each failure carried no information beyond two numbers: the fix each time was to bump a literal. A count also passes silently when one added resource replaces one removed one. The guard now diffs the ordered resource-name list. Proven red and green: adding a phantom resource makes it print the added name plus the instruction to update the list in the same commit, and removing the phantom makes it green again. https://claude.ai/code/session_01LeVatedQ4sCA1u6LHjo7B5
fix(test): pin the Binjovi kustomization by resource name
All checks were successful
binjovi/ci Binjovi completed the frozen plan
Details
ed4f96ede0
 
The guard compared a raw count. A count stays correct when one added
resource replaces one removed resource, and it names nothing when it
fails: two concurrent changes hit it twice in one evening, and each
failure needed only a number. The guard now compares the ordered name
list and prints what was added and what was removed.

Claude-Session: https://claude.ai/code/session_01LeVatedQ4sCA1u6LHjo7B5
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
sean/seanfarm!146
No description provided.