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 Keystore field to CRD to support HTTPS #17

Merged
merged 1 commit into from
Jun 17, 2024
Merged

Add Keystore field to CRD to support HTTPS #17

merged 1 commit into from
Jun 17, 2024

Conversation

jhunzik
Copy link
Contributor

@jhunzik jhunzik commented Jun 4, 2024

About

Adds the ability to mount a keystore.jks file for setting up routes that host an HTTP server over TLS.

Hero Instructions

  1. Start with a fresh Minikube cluster or equivalent
  2. From the operator/webhook directory: make build && minikube image load ghcr.io/octoconsulting/keip/route-webhook:0.6.0
  3. In the operator directory, run make all to deploy the integration route controller.
  4. Install CertManager kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/v1.15.0/cert-manager.yaml
  5. Create the JKS keystore secret: k create secret generic --from-literal=password=password jks-password
  6. Deploy the example route: kubectl apply -k operator/example
  7. Port forward the pod and verify it can be hit in the browser: kubectl port-forward testroute-74d574bf85-tbv9m 8443:8443

operator/crd/crd.yaml Outdated Show resolved Hide resolved
operator/crd/crd.yaml Outdated Show resolved Hide resolved
operator/crd/crd.yaml Show resolved Hide resolved
operator/webhook/introute/sync.py Outdated Show resolved Hide resolved
operator/webhook/introute/sync.py Outdated Show resolved Hide resolved
operator/crd/crd.yaml Outdated Show resolved Hide resolved
@jhunzik jhunzik changed the title Add TLS config for server routes Add Keystore field to CRD to support HTTPS Jun 14, 2024
@mcalcote
Copy link
Contributor

Hero'd on b2faec5 👍

@jhunzik jhunzik merged commit d1b1a53 into main Jun 17, 2024
5 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.

5 participants