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

Adding useful information to trust cert-manager cert for minio #1631

Merged
merged 2 commits into from
Jun 27, 2023
Merged

Adding useful information to trust cert-manager cert for minio #1631

merged 2 commits into from
Jun 27, 2023

Conversation

cniackz
Copy link
Contributor

@cniackz cniackz commented Jun 9, 2023 •
edited
Loading

Objective:

To help users to trust MinIO Cert when cert-manager is used.

Issue:

AWS SDK exposed: Empty issuer DN not allowed in X509Certificates

Solution:

  • In the Certificate from cert-manager add spec.subject.organization and/or spec.commonName
  subject:
    organizations:
      - jetstack
  # The use of the common name field has been deprecated since 2000 and is
  # discouraged from being used.
  commonName: example.com

Documentation:

https://cert-manager.io/docs/usage/certificate/

@cniackz cniackz self-assigned this Jun 9, 2023
@harshavardhana harshavardhana merged commit f0cafa9 into minio:master Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dvaldivia dvaldivia dvaldivia approved these changes

@jiuker jiuker jiuker approved these changes

@harshavardhana harshavardhana Awaiting requested review from harshavardhana

@pjuarezd pjuarezd Awaiting requested review from pjuarezd

@reivaj05 reivaj05 Awaiting requested review from reivaj05

@cesnietor cesnietor Awaiting requested review from cesnietor

@dilverse dilverse Awaiting requested review from dilverse

Assignees

@cniackz cniackz

Labels
documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cniackz @jiuker @dvaldivia @harshavardhana