Skip to content

test(s3): replace MinIO with Garage - #1001

Merged
NickLarsenNZ merged 6 commits into
mainfrom
test/use-garage-s3
Oct 7, 2026
Merged

NickLarsenNZ merged 6 commits into
mainfrom
test/use-garage-s3

Conversation

@NickLarsenNZ

@NickLarsenNZ NickLarsenNZ commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

MinIO no longer has images available.

Note: Garage doesn't natively support TLS, and therefore needs a sidecar proxy to handle that (we could explore using tlsproxy, but nginx was a known quantity).

The same work is being done in demos: stackabletech/demos#459

Decision

  • Use garage+nginx like this PR in integration-tests and demos?

MinIO no longer has images available.

Note: Garage doesn't natively support TLS, and therefore needs a sidecar proxy to handle that (we could explore using tlsproxy, but nginx was a known quantity).
@NickLarsenNZ NickLarsenNZ self-assigned this Sep 29, 2026
@NickLarsenNZ

Copy link
Copy Markdown
Member Author

Running tests. It worked for 2.12.0. Running on other versions.

@NickLarsenNZ

This comment was marked as resolved.

@Techassi

Copy link
Copy Markdown
Member

We basically decided that Garage is the way forwarding during the on-site.

@NickLarsenNZ

NickLarsenNZ commented Sep 30, 2026 •

Copy link
Copy Markdown
Member Author

We basically decided that Garage is the way forwarding during the on-site.

In that case, I can now fix the broken demos that are blocking me :D

@sbernauer sbernauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM otherwise, thanks!

Comment thread tests/templates/kuttl/iceberg-rest/README.md Outdated
Comment thread tests/templates/kuttl/iceberg-rest/01_s3-connection.yaml Outdated
Comment thread tests/templates/kuttl/iceberg-rest/01_s3-connection.yaml Outdated
@NickLarsenNZ
NickLarsenNZ requested a review from sbernauer October 7, 2026 08:46

@sbernauer sbernauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@NickLarsenNZ
NickLarsenNZ enabled auto-merge October 7, 2026 08:49
@NickLarsenNZ
NickLarsenNZ added this pull request to the merge queue Oct 7, 2026
Merged via the queue into main with commit fe9d4f0 Oct 7, 2026
13 checks passed
@NickLarsenNZ
NickLarsenNZ deleted the test/use-garage-s3 branch October 7, 2026 08:59
@sbernauer

Copy link
Copy Markdown
Member

We went with garage in test and demos, so I guess this decision is accepted

@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants