[stable/seq] Version change to GA and add baseURI setting (helm#14392)

* SEQ version change and baseURI setting - Bump SEQ to GA 5.0 image tag - Add setting for BASE_URI - Needed for Ingress and Azure Active Directory Signed-off-by: Jon Stelly <[email protected]> * Bump chart and app version Signed-off-by: Jon Stelly <[email protected]>
madifaya · Jun 3, 2019 · e9e9066 · e9e9066
1 parent dd0cb41
commit e9e9066
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 5 deletions.
diff --git a/stable/seq/Chart.yaml b/stable/seq/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v1
 name: seq
-version: 1.0.0
-appVersion: 5.0.832-pre
+version: 1.0.1
+appVersion: 5
 description: Seq is the easiest way for development teams to capture, search and visualize structured log events! This page will walk you through the very quick setup process.
 keywords:
 - seq

diff --git a/stable/seq/README.md b/stable/seq/README.md
@@ -46,9 +46,10 @@ The following table lists the configurable parameters of the Seq chart and their
 |           Parameter           |                Description                        |           Default            |
 |-------------------------------|-------------------------------------------------- |------------------------------|
 | `image.repository`         | Image repository                    | `datalust/seq`                                          |
-| `image.tag`                | Seq image tag. Possible values listed [here](https://hub.docker.com/r/datalust/seq/tags/).| `5.0.832-pre`|
+| `image.tag`                | Seq image tag. Possible values listed [here](https://hub.docker.com/r/datalust/seq/tags/).| `5`|
 | `image.pullPolicy`         | Image pull policy                   | `IfNotPresent`                                          |
-| `acceptEULA`               | Accept EULA                         | `Y`                                                 |
+| `acceptEULA`               | Accept EULA                         | `Y`                                                     |
+| `baseURI`                  | Base URL for ingress/AAD (see values.yaml)|                                                   |
 | `service.type`             | Kubernetes service type             | `ClusterIP`                                             |
 | `service.port`             | Kubernetes port where service is exposed| `5341`                                              |
 | `persistence.enabled`      | Use persistent volume to store data | `true`                                                  |

diff --git a/stable/seq/templates/deployment.yaml b/stable/seq/templates/deployment.yaml
@@ -25,6 +25,10 @@ spec:
           env:
             - name: "ACCEPT_EULA"
               value: "{{ .Values.acceptEULA }}"
+{{- if .Values.baseURI }}
+            - name: "BASE_URI"
+              value: "{{ .Values.baseURI }}"
+{{- end }}
           ports:
             - name: ingestion
               containerPort: 5341

diff --git a/stable/seq/values.yaml b/stable/seq/values.yaml
@@ -4,7 +4,7 @@
 
 image:
   repository: datalust/seq
-  tag: 5.0.832-pre
+  tag: 5
   pullPolicy: IfNotPresent
 
 # By passing the value Y in the ACCEPT_EULA environment variable,
@@ -13,6 +13,13 @@ image:
 # that you intend to use.
 acceptEULA: "Y"
 
+# Set this URL if you enable ingress and/or AAD authentication.
+# Without this URL set to include HTTPS, SEQ will try to set a login redirect
+# URL with HTTP instead of HTTPS and AAD's registration requires HTTPS.
+# The result is that you'll get an error during login:
+#   AADSTS50011: The reply url specified in the request does not match the reply urls configured for the application
+# baseURI: https://my.public.url/
+
 service:
   type: ClusterIP
   port: 5341