Repository navigation
Install cs---engine-shed-2, and stop the tests snapshotting INSTALLED - #28
Merged
Merged
Conversation
bazauto
enabled auto-merge (squash)
September 25, 2026 19:15
Goods Shed's detector is out of service, so cs---goods-shed leaves INSTALLED. The config tests now check properties of whatever is installed, so a bring-up no longer turns the suite red and blocks deploy.sh. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Installs
cs---engine-shed-2, the first newbazauto/block-detectionchannel after Goods Shed's. It replacescs---goods-shedinINSTALLED, whose LM-iD.1 was removed in #27.Consequence for Goods Shed
Goods Shed now has only its IR beam. An
ir_positionclearis a no-op in the orchestrator, so the block can go occupied but can't clear until a detector is back on board 1 pin 8. The orchestrator also stops hearingcs---goods-shedand degrades the block tounknown. Both failures leave the block treated as occupied. This is recorded indocs/pin-allocation.mdand in theINSTALLEDcomment.Tests no longer snapshot
INSTALLEDThree tests hard-coded the Goods Shed pair, so installing any other sensor turned the suite red.
deploy.shthen refused to deploy, which is how this came up. They now check properties of whatever is installed:cs---/ir---prefix names;A config-only bring-up no longer needs a test edit. Pin and id validity are still covered by
test_the_real_config_resolvesand the allocation tests.python -m pytest: 208 passed.Deployed
scripts/deploy.sh io-node(orchestrator/healthonline, manual mode). The broker afterwards showedcs---engine-shed-2publishing, goingclearthenoccupied, andir---goods-shed→clear. The onecs---goods-shedmessage seen was the retained reading from before.Docs
CLAUDE.mdcurrent state,README.mdstatus, anddocs/pin-allocation.md"Currently installed".🤖 Generated with Claude Code