Skip to content

MinIO updates in ECE docs #1996

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 13 commits into from
Jul 15, 2025
Merged

MinIO updates in ECE docs #1996

merged 13 commits into from
Jul 15, 2025

Conversation

eedugon
Copy link
Contributor

@eedugon eedugon commented Jul 2, 2025

In relation with https://github.com/elastic/cloud/issues/134936, we have realized that the doc /deploy-manage/tools/snapshot-and-restore/minio-on-premise-repository.md was really outdated and needed some love.

This is the main page to review:

MinIO self-managed repository

Applied changes:

  • Changed wording to MinIO across all docs. No minio or Minio anymore.
  • Fixed a lot of broken links to MinIO docs.
  • Removed screenshots (thanks @Kushmaro for pointing that out), and replacing them by proper links and instructions.
  • Adding a note to recommend installing MinIO in separate hosts, although we support installing it in ECE allocator hosts.
  • Removed the manual steps to restore a snapshot from another cluster as now that's offered and documented as an ECE feature. I've linked to the right doc for that.
  • Improved the structure of certain sections as they were hard to understand (the bucket creation step for example was incorrect).
  • Renamed offline to air-gapped (for consistency with the new IA and new docs).
  • Changed title from on-premise to self-managed (because ECE hosts or the customer minio hosts could be on-premises or on cloud VMs technically. What they are in reality is self-managed).
  • Added useful links to other parts of the documentation.
  • Removed the statements about 6.x and 5.x versions of Elasticsearch. These are unneeded as in ECE 4.0 it's mandatory for all deployments to be on 8.0.0 or above. So there's no need to mention it.

Closes https://github.com/elastic/cloud/issues/134936

@eedugon eedugon added the ece Elastic Cloud Enterprise label Jul 2, 2025
@eedugon eedugon requested a review from Kushmaro July 2, 2025 08:42
@eedugon eedugon marked this pull request as ready for review July 2, 2025 08:47
@eedugon eedugon requested a review from a team as a code owner July 2, 2025 08:47
Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🎸
Very nice!

…sitory.md

Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
@eedugon
Copy link
Contributor Author

eedugon commented Jul 15, 2025

@kilfoyle , would you mind re-reviewing this again? I've made significant changes after a conversation with Omer, Alex P and support folks.

The previous version was making the impression that it was ok to run MinIO directly within ECE hosts, and we have decided to recommend against it, as MinIO should run on its own infrastructure for multiple reasons.

@eedugon eedugon requested review from kilfoyle and Kushmaro July 15, 2025 10:08
Copy link

@Kushmaro Kushmaro left a comment

Choose a reason for hiding this comment

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

LGTM

@kilfoyle
Copy link
Contributor

LGTM! 🚀
I added a few super small suggestions but it looks go to go. Thanks @eedugon!

Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
@eedugon eedugon merged commit 25bae93 into main Jul 15, 2025
9 checks passed
@eedugon eedugon deleted the ece_minio_enhancement branch July 15, 2025 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ece Elastic Cloud Enterprise
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants