Skip to content

Support STS web-token based access.#446

Merged
klauspost merged 5 commits intominio:masterfrom
ramondeklein:support-sts-token
Jan 8, 2026
Merged

Support STS web-token based access.#446
klauspost merged 5 commits intominio:masterfrom
ramondeklein:support-sts-token

Conversation

@ramondeklein
Copy link
Contributor

@ramondeklein ramondeklein commented Jan 6, 2026

Summary

Support authenticating using an web token (using STS).

Context

Running Warp should be possible from AIStor operator (using Warp job), but the operator is created in such a way that it doesn't require root credentials anymore. To enable running Warp using STS is most convenient.

An alternative solution is to fetch the STS issued credentials from the operator and pass it using access-key, secret-key and session-token. However, that is a bit more complicated (due to a stateless reconcile loop). Supporting STS from Warp is simpler and can't do much harm.

How to test

Invoke Warp with --signature sts_web_token and either --sts-web-token.

This comment was marked as outdated.

@ramondeklein ramondeklein self-assigned this Jan 6, 2026
@ramondeklein ramondeklein added enhancement New feature or request documentation Improvements or additions to documentation labels Jan 6, 2026
@ramondeklein
Copy link
Contributor Author

@ravindk89 Not sure if we want to document this. I think normal users will not use it, but I'll leave that up to you...

@ravindk89
Copy link

There's no harm in it, although we also don't yet have docs for warp in K8s yet. That would probably come first, and would require some coordination because I'm still a little lost on which operator model sould be used there.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@klauspost klauspost merged commit 9e9bba4 into minio:master Jan 8, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants