Skip to content

Commit 27a8901

Browse files
committed
preflight: by default, replicas=0 to avoid failing deployment
The UX was bad: the user would immediately see a failing deployment and think that the click-to-deploy process failed. Closes #41 Signed-off-by: Maël Valais <[email protected]>
1 parent c96aa81 commit 27a8901

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

README.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,11 +102,12 @@ When you are done, click the "Deploy" button:
102102
This will install Jetstack Secure for cert-manager, and will redirect to
103103
the [Applications](https://console.cloud.google.com/kubernetes/application) page:
104104

105-
<img src="https://user-images.githubusercontent.com/2195781/112529074-28e2bb00-8da5-11eb-9aca-f347fbfe0330.png" width="600" alt="The Application view on GKE after deploying the application test-1 onto the test-1 namespace. This screenshot is stored in this issue: https://github.com/jetstack/jetstack-secure-gcm/issues/21">
105+
<img src="https://user-images.githubusercontent.com/2195781/110795922-a96acd00-8277-11eb-959e-bf7ea51ae992.png" width="500" alt="The application page for test-1 shows that all the deployments are green. This screenshot is stored in this issue: https://github.com/jetstack/jetstack-secure-gcm/issues/21">
106106

107-
**Note:** the preflight deployment is expected to be failing when the
108-
application is first deployed. After registering your cluster on
109-
<https://platform.jetstack.io>, the deployment will start working. To register your cluster, keep reading the [next section](#step-2-log-into-the-jetstack-secure-dashboard).
107+
**Note:** by default, the `preflight` deployment is scaled to 0. After
108+
completing the steps in the [next
109+
section](#step-2-log-into-the-jetstack-secure-dashboard), the deployment will
110+
start working.
110111

111112
### Step 2: log into the Jetstack Secure dashboard
112113

@@ -192,6 +193,7 @@ You can then apply the Jetstack Secure agent configuration to your cluster:
192193

193194
```sh
194195
sed '/namespace:/d' agent-config.yaml | kubectl -n $NAMESPACE apply -f-
196+
kubectl -n $NAMESPACE scale --replicas=1 $(kubectl -n $NAMESPACE get deploy -oname | grep preflight)
195197
kubectl -n $NAMESPACE rollout restart $(kubectl -n $NAMESPACE get deploy -oname | grep preflight)
196198
```
197199

chart/jetstack-secure-gcm/values.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,9 @@ google-cas-issuer:
5656
tag: 1.1.0-gcm.1
5757

5858
preflight:
59+
# By default, the preflight deployment is "disabled" by setting replicas=0.
60+
# See https://github.com/jetstack/jetstack-secure-gcm/issues/41
61+
replicas: 0
5962
nameOverride: jetstack-secure-gcm
6063
fullnameOverride: preflight
6164
serviceAccount:

0 commit comments

Comments
 (0)