Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Multi-replica minio #589

Merged
merged 3 commits into from
Feb 20, 2023
Merged

Multi-replica minio #589

merged 3 commits into from
Feb 20, 2023

Conversation

MarioUhrik
Copy link
Contributor

@MarioUhrik MarioUhrik commented Feb 17, 2023 •
edited

closes #182

Works standalone, but we need the E2E to determine if it works with Claudie

@MarioUhrik MarioUhrik marked this pull request as ready for review February 17, 2023 13:49
@MarioUhrik
Copy link
Contributor Author

Hi @MiroslavRepka @Despire , the E2E tests passed here, pls check
https://github.com/berops/claudie/actions/runs/4204354918

MiroslavRepka
MiroslavRepka approved these changes Feb 17, 2023
View reviewed changes
Copy link
Contributor

@MiroslavRepka MiroslavRepka left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @MarioUhrik

@MarioUhrik MarioUhrik added this pull request to the merge queue Feb 20, 2023
Merged via the queue into master with commit 7fbe0bf Feb 20, 2023
@MarioUhrik MarioUhrik deleted the feature/multi-replica-minio branch February 20, 2023 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MiroslavRepka MiroslavRepka MiroslavRepka approved these changes

@Despire Despire Awaiting requested review from Despire

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature: Production readiness of Minio
2 participants
@MarioUhrik @MiroslavRepka