Merge pull request #314 from reasonerjt/allow-skip-nginx-svc-1.1.0

Skip unknown expose.type -- 1.1.0
goharbor · Aug 5, 2019 · fa38270 · fa38270
2 parents 30af256 + ceaf710
commit fa38270
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -88,7 +88,7 @@ The following table lists the configurable parameters of the Harbor chart and th
 | Parameter                                                                   | Description                                                                                                                                                                                                                                                                                                                                     | Default                         |
 | --------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ------------------------------- |
 | **Expose**                                                                  |
-| `expose.type`                                                               | The way how to expose the service: `ingress`, `clusterIP`, `nodePort` or `loadBalancer`                                                                                                                                                                                                                                                                         | `ingress`                       |
+| `expose.type`                                                               | The way how to expose the service: `ingress`, `clusterIP`, `nodePort` or `loadBalancer`, other values will be ignored and the creation of service will be skipped.                                                                                                                                                                                                                                                                         | `ingress`                       |
 | `expose.tls.enabled`                                                        | Enable the tls or not                                                                                                                                                                                                                                                                                                                           | `true`                          |
 | `expose.ingress.controller` | The ingress controller type. Currently supports `default`, `gce` and `ncp` | `default` |
 | `expose.tls.secretName`                                                     | Fill the name of secret if you want to use your own TLS certificate and private key. The secret must contain keys named `tls.crt` and `tls.key` that contain the certificate and private key to use for TLS. The certificate and private key will be generated automatically if it is not set                                                   |                                 |

diff --git a/templates/nginx/service.yaml b/templates/nginx/service.yaml
@@ -1,4 +1,4 @@
-{{- if ne .Values.expose.type "ingress" }}
+{{- if or (eq .Values.expose.type "clusterIP") (eq .Values.expose.type "nodePort") (eq .Values.expose.type "loadBalancer") }}
 apiVersion: v1
 kind: Service
 metadata: