Skip to content

Commit 1862707

Browse files
fix: delete agent and dind secrets (#397)
1 parent 4f2c23d commit 1862707

File tree

4 files changed

+17
-11
lines changed

4 files changed

+17
-11
lines changed

charts/cf-runtime/Chart.yaml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
apiVersion: v2
22
description: A Helm chart for Codefresh Runner
33
name: cf-runtime
4-
version: 6.1.4
4+
version: 6.1.5
55
keywords:
66
- codefresh
77
- runner
@@ -15,9 +15,7 @@ maintainers:
1515
annotations:
1616
artifacthub.io/changes: |
1717
- kind: fixed
18-
description: Updated volumeProvisioner image to `1.34.0`
19-
- kind: added
20-
description: Add pod monitor for volumeProvisioner deployment
18+
description: Fix delete agent and dind secrets
2119
dependencies:
2220
- name: cf-common
2321
repository: https://chartmuseum.codefresh.io/cf-common

charts/cf-runtime/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
## Codefresh Runner
22

3-
![Version: 6.1.4](https://img.shields.io/badge/Version-6.1.4-informational?style=flat-square)
3+
![Version: 6.1.5](https://img.shields.io/badge/Version-6.1.5-informational?style=flat-square)
44

55
Helm chart for deploying [Codefresh Runner](https://codefresh.io/docs/docs/installation/codefresh-runner/) to Kubernetes.
66

charts/cf-runtime/files/cleanup-runtime.sh

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
#!/bin/bash
22

33
echo "-----"
4-
echo "API_HOST: ${API_HOST}"
5-
echo "AGENT_NAME: ${AGENT_NAME}"
6-
echo "RUNTIME_NAME: ${RUNTIME_NAME}"
7-
echo "AGENT: ${AGENT}"
4+
echo "API_HOST: ${API_HOST}"
5+
echo "AGENT_NAME: ${AGENT_NAME}"
6+
echo "RUNTIME_NAME: ${RUNTIME_NAME}"
7+
echo "AGENT: ${AGENT}"
8+
echo "AGENT_SECRET_NAME: ${AGENT_SECRET_NAME}"
9+
echo "DIND_SECRET_NAME: ${DIND_SECRET_NAME}"
810
echo "-----"
911

1012
auth() {
@@ -23,11 +25,13 @@ remove_agent() {
2325
codefresh delete agent ${AGENT_NAME} || true
2426
}
2527

26-
remove_finalizers() {
28+
remove_secrets() {
2729
kubectl patch secret $(kubectl get secret -l codefresh.io/internal=true | awk 'NR>1{print $1}' | xargs) -p '{"metadata":{"finalizers":null}}' --type=merge || true
30+
kubectl delete secret $AGENT_SECRET_NAME || true
31+
kubectl delete secret $DIND_SECRET_NAME || true
2832
}
2933

3034
auth
3135
remove_runtime
3236
remove_agent
33-
remove_finalizers
37+
remove_secrets

charts/cf-runtime/templates/hooks/pre-delete/job-cleanup-resources.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,10 @@ spec:
5353
{{- include "runtime.installation-token-env-var-value" . | indent 10}}
5454
- name: AGENT
5555
value: {{ .Values.runtime.agent | quote }}
56+
- name: AGENT_SECRET_NAME
57+
value: {{ include "runner.fullname" . }}
58+
- name: DIND_SECRET_NAME
59+
value: codefresh-certs-server
5660
{{- include (printf "%s.env-vars" $cfCommonTplSemver) (dict "Values" $values.env "context" .) | nindent 8 }}
5761
{{- with $values.nodeSelector }}
5862
nodeSelector:

0 commit comments

Comments
 (0)