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

feat: add action to handle SSL values as secrets for TLS configuration (#394) #397

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

DnPlas
Copy link
Contributor

@DnPlas DnPlas commented Mar 26, 2024

  • feat: add action to handle SSL values as secrets for TLS configuration

This commits introduces actions that allow users to configure the TLS ingress gateway for a single host directly passing the SSL cert and key to the charm.

  • save-tls-secret: allows users to pass the ssl-key and ssl-crt values, which the charm saves in a juju secret (owned by the charm) and uses them to reconcile the ingress Gateway with such information.
  • remove-tls-secret: a handy action that allows users to remove the TLS secret, which in turn removes the TLS configuration from the ingress Gateway.

This commit also adds unit and integration tests to increase the coverage due to the recent changes.

WARNING: please note this feature is only supported in 1.17 and 1.18, and it will be removed after releasing 1.18 in favour of the TLS provider method.

Fixes #380

#394)

* feat: add action to handle SSL values as secrets for TLS configuration

This commits introduces actions that allow users to configure the TLS
ingress gateway for a single host directly passing the SSL cert and key
to the charm.
- save-tls-secret: allows users to pass the ssl-key and ssl-crt values,
  which the charm saves in a juju secret (owned by the charm) and uses
  them to reconcile the ingress Gateway with such information.
- remove-tls-secret: a handy action that allows users to remove the
  TLS secret, which in turn removes the TLS configuration from the
  ingress Gateway.

This commit also adds unit and integration tests to increase the
coverage due to the recent changes.

WARNING: please note this feature is only supported in 1.17 and 1.18,
and it will be removed after releasing 1.18 in favour of the TLS
provider method.

Fixes #380
@DnPlas DnPlas requested a review from a team as a code owner March 26, 2024 09:00
@DnPlas DnPlas added the backport Backport a change from one track to another. label Mar 26, 2024
@DnPlas DnPlas merged commit 5c86667 into track/1.17 Mar 26, 2024
19 checks passed
@DnPlas DnPlas deleted the KF-5346-cherry-pick-tls-commits branch March 26, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Backport a change from one track to another. Libraries: Out of sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants