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

[FEATURE] standard ingress with TLS #26

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

MichaelSp
Copy link
Contributor

@MichaelSp MichaelSp commented Oct 10, 2024

This will replace ingress with helm standard ingress, which also adds support for TLS configuration.

Recommended usage:

ingress:
  enabled: true
  annotations:
    kubernetes.io/tls-acme: "true"
  hosts:
    - perses.example.com
  tls:
    - secretName: ingress-cert
      hosts:
        - perses.example.com

in combination with cert-manager this will automatically generate certificates and configure HTTPS.

Description

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, IGNORE.
  • All commits have DCO signoffs.

This will replace ingress with helm standard ingress, which also adds support for TLS configuration. Recommended usage:

```yaml
    ingress:
      enabled: true
      annotations:
        kubernetes.io/tls-acme: "true"
      hosts:
        - perses.example.com
      tls:
        - secretName: ingress-cert
          hosts:
            - perses.example.com
```

in combination with [cert-manager](https://cert-manager.io/) this will automatically generate certificates and configure HTTPS.

Signed-off-by: Michael Sprauer <[email protected]>
@Nexucis
Copy link
Member

Nexucis commented Oct 23, 2024

@nicolastakashi would you mind to review this PR please ? I don't feel I have the knowledge to do it myself.

@nicolastakashi
Copy link
Contributor

Hey @MichaelSp thanks for your contribution, LGTM

@nicolastakashi nicolastakashi merged commit b6c261c into perses:main Oct 23, 2024
2 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