Revert queueName default value in chart

Since queueName is a new field in the chart, it should not have
a default. This will cause issues for users upgrading.

When queueName is not set, it will use the stream name for the
queue instead.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <[email protected]>

Author: alexellis

chart/queue-worker/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: queue-worker
 description: Dedicated queue-worker for OpenFaaS using JetStream
 type: application
-version: 0.2.17
+version: 0.2.18
 keywords:
 - openfaas
 - jetstream

chart/queue-worker/README.md

@@ -17,28 +17,25 @@ helm upgrade --install \
   slow-fns openfaas/queue-worker \
   --namespace openfaas \
   --set maxInflight=5 \
-  --set queueName=slow-fns \
   --set mode=static \
-  --set nats.stream.name=slow-fns-requests \
+  --set nats.stream.name=slow-fns \
   --set nats.consumer.durableName=slow-fns-workers \
   --set upstreamTimeout=15m
 ```
 
 It's recommended to take the NAME of the queue i.e. `slow-fns` and then use it as prefix in the following way for the above configuration:
 
-* `queueName` - NAME
 * `nats.stream.name` - NAME`-requests`
 * `nats.consumer.durableName` - NAME`-workers`
 
 As an alternative to using `--set`, you could also write your own YAML file. Below is the equivalent configuration in a values.yaml file, for instance `values-slow-fns.yaml`:
 
 ```yaml
 maxInflight: 5
-queueName: slow-fns
 mode: static
 nats:
   stream:
-    name: slow-fns-requests
+    name: slow-fns
   consumer:
     durableName: slow-fns-workers
 upstreamTimeout: 15m  
@@ -89,9 +86,8 @@ helm upgrade --install \
   slow-fns ./ \
   --namespace openfaas \
   --set maxInflight=5 \
-  --set queueName=slow-fns \
   --set mode=static \
-  --set nats.stream.name=slow-fns-requests \
+  --set nats.stream.name=slow-fns \
   --set nats.consumer.durableName=slow-fns-workers \
   --set upstreamTimeout=15m
 ```
@@ -102,7 +98,7 @@ helm upgrade --install \
 |-----------|-------------|---------|
 | `image` | The queue-worker image that should be deployed | See values.yaml |
 | `replicas` | Number of queue-worker replicas to create | `1` |
-| `queueName` | Name of the queue | `faas-request` |
+| `queueName` | Name of the queue if you want it to be different to the stream name | `""` - when empty, defaults to `nats.stream.name` |
 | `mode` | Queue operation mode: `static` (OpenFaaS Standard) or `function` (requires OpenFaaS for Enterprises) | `static` |
 | `maxInflight` | Control the concurrent invocations | `1` |
 | `queuePartitions` | Number of queue partitions | `1` |

chart/queue-worker/values.yaml

@@ -10,7 +10,7 @@ replicas: 1
 
 # Change this to your queue name.
 # Allowed characters (a - z), (0 - 9), - and _
-queueName: "custom-request"
+queueName: ""
 mode: static
 maxInflight: 1
 
@@ -43,10 +43,10 @@ nats:
   host: "nats.openfaas"
   port: "4222"
   stream:
-    name: "faas-request"
+    name: "custom-request"
     replicas: 1
   consumer:
-    durableName: "faas-workers"
+    durableName: "custom-workers"
     ackWait: "30s"
 
 logs: