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.

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

feat: AWS MinIO distributed directpv #304

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

davidspek
Copy link
Contributor

Summary

This PR updates our distributed MinIO deployment on AWS to use minio directpv together with nodes that have local HDDs.

Test Plan

local linking

Signed-off-by: DavidSpek <[email protected]>
@@ -0,0 +1,33 @@
-----BEGIN CERTIFICATE-----
Copy link
Member

Choose a reason for hiding this comment

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

should we check this in?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sadly I think we have to. Their CLI generates the certificates at installation time, which were for a different namespace so I needed to generate new ones manually. I don't think we'll be able to use cert-manager since the operator is coded to look for ca.pem, cert.pem and key.pem and cert-manager will create ca.crt, tls.crt and tls.key.

If they move away from only being able to deploy directpv with their CLI that would make me very happy haha.

@davidspek
Copy link
Contributor Author

Just need to improve the startup script so it waits for all the local disks to be discovered somehow.

Signed-off-by: DavidSpek <[email protected]>
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