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

VirtualServer with SSL services? #620

Closed
trungdq88 opened this issue Jul 15, 2019 · 3 comments · Fixed by #621
Closed

VirtualServer with SSL services? #620

trungdq88 opened this issue Jul 15, 2019 · 3 comments · Fixed by #621
Assignees
Labels
proposal An issue that proposes a feature request

Comments

@trungdq88
Copy link

trungdq88 commented Jul 15, 2019

Is your feature request related to a problem? Please describe.
Currently, I don't see there is an option to create a VirtualServer with secured service. When define the upstream with port 443, I see this error:
image

Describe the solution you'd like
Be able to use SSL services in VirtualServer, maybe use the same annotation as in Ingress?

apiVersion: k8s.nginx.org/v1alpha1
kind: VirtualServer
metadata:
  name: secure-app-virtual-server
  annotations:
    nginx.org/ssl-services: "secure-app"
spec:
  host: example.com
  tls:
    secret: secure-app-certs
  upstreams:
  - name: secure-app
    service: secure-app
    port: 443
  routes:
  - path: /
    upstream: secure-app
@Dean-Coakley Dean-Coakley added the proposal An issue that proposes a feature request label Jul 15, 2019
@Dean-Coakley
Copy link
Contributor

Hi @trungdq88

Thanks for the feature suggestion.

TLS support is a planned feature to be included in the next upcoming ingress controller release. We'll be sure to update this issue once it is available via the :edge tag too.

@Rulox
Copy link
Contributor

Rulox commented Jul 16, 2019

Hi @trungdq88 ,

Upstreams now can configure TLS for requests to services with SSL. This is available in the edge version of the Ingress Controller.

Have a look at the Upstream.TLS docs for more info.

I hope it helps!

@trungdq88
Copy link
Author

Thank you @Rulox !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal An issue that proposes a feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants