Skip to content

test: run the remote state tests against Garage instead of MinIO - #151

Merged
99-not-out merged 1 commit into
mainfrom
ci/garage-for-state-tests
Oct 7, 2026
Merged

99-not-out merged 1 commit into
mainfrom
ci/garage-for-state-tests

Conversation

@99-not-out

Copy link
Copy Markdown
Contributor

TL;DR

MinIO no longer publishes its image, so the integration stack can't start its S3 store. This swaps it for Garage, an open-source S3-compatible store, and the remote state tests run unchanged against it.

What & why

The remote state tests check that ctl saves and loads its state in an S3 bucket. They used MinIO as a local S3. Now that the image has gone, every integration run fails to bring up its stack. Garage works as a drop-in replacement for what these tests need. Its one difference is that the bucket and access key are created through its admin API rather than with mc.

Changes

  • The compose stack runs a single-node Garage plus a one-shot setup container. The setup assigns the node, applies the layout, imports a fixed test key and creates the conduktor-state bucket.
  • The remote state tests point at Garage's S3 port and key. Their helpers and test names now say S3 rather than MinIO.

Notes

  • Locally I brought up only the Garage services from this compose file. ctl's own remote backend then loaded a missing state as empty, saved a state and read it back. The full integration run is left to CI because its ports clash with a stack I have running.
  • docs/state_management.md still uses MinIO as an example, including a "MinIO for testing" recipe that pulls the old image. Those docs can move to Garage in a follow-up.

🤖 Generated with Claude Code

https://claude.ai/code/session_01B464sFQc8q6VkehfkByBaX

The MinIO image is no longer published, so the integration stack could not start
its S3 store. Garage is a single-node S3-compatible store set up through its admin
API by a one-shot container.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01B464sFQc8q6VkehfkByBaX
@99-not-out
99-not-out marked this pull request as ready for review October 7, 2026 08:50
@99-not-out
99-not-out merged commit 6e379d9 into main Oct 7, 2026
7 checks passed
@99-not-out
99-not-out deleted the ci/garage-for-state-tests branch October 7, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants