Skip to content

fix: update path and name#323

Merged
m-huertasp merged 2 commits into
mainfrom
update/322-s3-storage-instructions
May 4, 2026
Merged

fix: update path and name#323
m-huertasp merged 2 commits into
mainfrom
update/322-s3-storage-instructions

Conversation

@m-huertasp
Copy link
Copy Markdown
Contributor

This pull request updates the S3 documentation to match the naming of the rclone remote from irb-minio to the irbminio currently used.

It also updates the path to the credentials' generation script.

@m-huertasp m-huertasp requested a review from migrau April 20, 2026 08:34
@m-huertasp m-huertasp self-assigned this Apr 20, 2026
Copilot AI review requested due to automatic review settings April 20, 2026 08:34
Copy link
Copy Markdown
Contributor

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

Updates the S3/MinIO documentation so command examples match the currently used rclone remote name (irbminio) and the current location of the credentials generation script.

Changes:

  • Update credentials generation script path in the terminal setup example.
  • Rename rclone remote references from irb-minio to irbminio across the doc examples/config snippet.

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

Comment thread docs/Cluster_basics/s3.md Outdated
@migrau
Copy link
Copy Markdown
Member

migrau commented Apr 20, 2026

Thanks for the update, Marta! When creating the credentials, you see irbminio instead irb-minio in e.g. openOnDemand?

image

If that's the case I guess IT modified something in the script. Looks good for me to apply the changes then :)

Copy link
Copy Markdown
Member

@migrau migrau left a comment

Choose a reason for hiding this comment

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

Looks good!

@m-huertasp
Copy link
Copy Markdown
Contributor Author

m-huertasp commented Apr 20, 2026

Yes! It's irbminio instead of irb-minio now everywhere...

I think you need to approve the PR so I can merge though! 💻

@m-huertasp m-huertasp merged commit cfd63a7 into main May 4, 2026
2 checks passed
@m-huertasp m-huertasp deleted the update/322-s3-storage-instructions branch May 4, 2026 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants