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

Add RW volume for AG tls certificate #722

Conversation

aorcholski
Copy link
Contributor

AG fails to start if RO filesystem is enabled and custom TLS secret is set. In such a case new RW volume is mounted on /var/lib/dynatrace/gateway/ssl directory and the launching script of AG image is able to copy the certificate to the 'ssl' directory.

How can this be tested?
-create tls secret kubectl -n dynatrace create secret generic dk-cert --from-file=server.p12=cert.p12
-add it to your CR

spec:
  activegate:
    tlsSecretName: dk-cert

check out AG logs kctl logs pod/<your CR name>-activegate-0
the following message is reported at the very beginning of the log

Copying /var/lib/dynatrace/secrets//tls/server.p12 certificate file to /var/lib/dynatrace/gateway/ssl/server.p12

Checklist

  • Unit tests have been updated/added
  • PR is labeled accordingly

…olume if RO (#721)

filesystem enabled and the custom TLS secret is set.
@aorcholski aorcholski added bug Something isn't working activegate Changes related to Activegate labels Apr 13, 2022
@aorcholski aorcholski requested a review from a team as a code owner April 13, 2022 08:27
@chrismuellner chrismuellner changed the title AG /var/lib/dynatrace/gateway/ssl directory is a mount-point for RW v… Add RW volume for AG tls certificate Apr 13, 2022
@aorcholski aorcholski merged commit 55fb74f into release-0.6 Apr 13, 2022
@aorcholski aorcholski deleted the bugfix/ActiveGate-gateway-ssl-directory-rw-volume-rel-0-6 branch April 13, 2022 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
activegate Changes related to Activegate bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants