Skip to content

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
@jiuker jiuker requested a review from ramondeklein August 23, 2024 10:11
example
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