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

fix: support auto mount certs for minioJob #2288

Merged
merged 8 commits into from
Aug 26, 2024

Conversation

jiuker
Copy link
Contributor

@jiuker jiuker commented Aug 23, 2024

fix: support auto mount certs for minioJob

fix: support auto mount cert for minioJob
@jiuker jiuker marked this pull request as draft August 23, 2024 01:57
tls
tls
ramondeklein
ramondeklein previously approved these changes Aug 23, 2024
Copy link
Contributor

@ramondeklein ramondeklein left a comment

Choose a reason for hiding this comment

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

I think this is a great improvement and will reduce a lot of questions on Subnet too :-)

PS: Nice detail to disable certificate mounting when using --insecure 👍🏻

@ramondeklein
Copy link
Contributor

@dvaldivia This PR should also be merged downstream.

simply
jiuker added 4 commits August 23, 2024 14:35
mount
certs
update the $HOME
example
@ramondeklein ramondeklein marked this pull request as ready for review August 23, 2024 09:46
docs/examples.md Outdated Show resolved Hide resolved
@jiuker jiuker requested a review from ramondeklein August 23, 2024 10:11
example
docs/examples.md Show resolved Hide resolved
Copy link
Contributor

@shtripat shtripat left a comment

Choose a reason for hiding this comment

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

overall looks good. take care of comments from @ramondeklein

@jiuker jiuker merged commit 2c5289c into minio:master Aug 26, 2024
20 of 21 checks passed
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.

3 participants