Skip to content

Commit a6595bb

Browse files
author
Stan Lagun
committed
Minor improvements to etcd app
Fixes small issues that caused warnings to be printed (including "deployment failed" result, which is incorrect)
1 parent ab95c24 commit a6595bb

File tree

4 files changed

+9
-1
lines changed

4 files changed

+9
-1
lines changed

examples/etcd/dependencies/etcd-recover.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ parent: flow/etcd-recover
88
child: job/delete-node-$NAME-job
99
args:
1010
NAME: $nodeSuffix
11+
SELF_DELETE: "true"
1112
---
1213
apiVersion: appcontroller.k8s/v1alpha1
1314
kind: Dependency

examples/etcd/dependencies/etcd-scale.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -35,4 +35,5 @@ parent: flow/etcd-scale
3535
child: job/delete-node-$NAME-job
3636
args:
3737
NAME: $AC_NAME
38+
SELF_DELETE: "false"
3839

examples/etcd/resources/delete-node-job.yaml

+6-1
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,8 @@ job:
2525
value: $clusterName
2626
- name: REPLICA_NAME
2727
value: $NAME
28+
- name: SELF_DELETE
29+
value: $SELF_DELETE
2830
command:
2931
- "/bin/sh"
3032
- "-c"
@@ -38,5 +40,8 @@ job:
3840
kubectl delete replica "replica-$REPLICA_NAME"
3941
kubectl delete service "etcd-$REPLICA_NAME"
4042
kubectl delete replica -l"replicaspace=etcd-recover-$CLUSTERNAME"
41-
kubectl delete job "delete-node-$REPLICA_NAME-job"
43+
if [ $SELF_DELETE = "true" ]
44+
then
45+
kubectl delete job "delete-node-$REPLICA_NAME-job"
46+
fi
4247
restartPolicy: Never

examples/etcd/resources/monitor-job.yaml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ metadata:
44
name: monitor-clustername-job
55
meta:
66
timeout: 0
7+
on-error: ignore
78
job:
89
apiVersion: batch/v1
910
kind: Job

0 commit comments

Comments
 (0)