Skip to content

Conversation

@sravotto
Copy link
Collaborator

Introduce a new s3 subcommand that performs validation tests against an S3-compatible object store. The command:

  • Initializes an S3 store from environment variables.
  • Runs validation checks via the validate package.
  • Outputs a formatted report to stdout.

This provides users with an easy way to confirm connectivity and configuration of their S3 object storage directly from the CLI.

@sravotto sravotto marked this pull request as ready for review August 22, 2025 18:46
@sravotto sravotto requested a review from BramGruneir August 22, 2025 18:46
Copy link
Collaborator

@BramGruneir BramGruneir left a comment

Choose a reason for hiding this comment

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

LGTM!

import (
"io"

"github.com/jedib0t/go-pretty/v6/table"
Copy link
Collaborator

Choose a reason for hiding this comment

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

'm surprised you didn't just use a template. But all good.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I used the same library in tempo. I found it useful, although a template would have worked just fine.

Introduce a new `s3` subcommand that performs validation tests against an
S3-compatible object store. The command:
- Initializes an S3 store from environment variables.
- Runs validation checks via the `validate` package.
- Outputs a formatted report to stdout.

This provides users with an easy way to confirm connectivity and
configuration of their S3 object storage directly from the CLI.
@sravotto sravotto merged commit af869a7 into main Aug 22, 2025
5 checks passed
@sravotto sravotto deleted the sr8_cli branch August 22, 2025 19:29
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