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

change chart to be cluster instead of tenant; make client and server certs completely optional; also make backups optional #17

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

jessebot
Copy link
Contributor

@jessebot jessebot commented Nov 6, 2023

now client and server certs have their own respective sections that can be enabled and generation is optional for both:

certificates:
  server:
    # -- enable using server certificates
    enabled: false
    # -- generate server certs using cert-manager. if true the following
    # are ignored: certificates.serverTLSSecret, certificates.serverCASecret
    generate: false
    # -- name of existing Kubernetes Secret for the postgresql server TLS cert,
    # ignored if certificates.generate is true
    serverTLSSecret: ""
    # -- name of existing Kubernetes Secret for the postgresql server Certificate
    # Authority cert, ignored if certificates.generate is true
    serverCASecret: ""
  client:
    # -- enable using client certificates
    enabled: false
    # -- generate client certs using cert-manager. if true the following are
    # ignored: certificates.clientCASecret, certificates.replicationTLSSecret
    generate: false
    # -- name of existing Kubernetes Secret for the postgresql client Certificate
    # Authority cert, ignored if certificates.generate is true
    clientCASecret: ""
    # -- name of existing Kubernetes Secret for the postgresql replication TLS
    # cert ignored if certificates.generate is true
    replicationTLSSecret: ""

@jessebot jessebot added the enhancement New feature or request label Nov 6, 2023
@jessebot jessebot requested a review from cloudymax November 6, 2023 07:34
@jessebot jessebot self-assigned this Nov 6, 2023
@jessebot
Copy link
Contributor Author

jessebot commented Nov 6, 2023

tested locally with the ct install --target-branch main command it worked fine. not sure why it's breaking in ci 🤷

@jessebot jessebot changed the title make client and server certs completely optional; also make backups optional change chart to be cluster instead of tenant; make client and server certs completely optional; also make backups optional Nov 6, 2023
@cloudymax cloudymax merged commit 59a1cd5 into main Nov 6, 2023
1 of 2 checks passed
@cloudymax cloudymax deleted the only-generate-some-certs branch November 6, 2023 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants