Skip to content

upgrade argo-workflows to 3.4 #144

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Dec 13, 2022
Original file line number Diff line number Diff line change
@@ -1,18 +1,9 @@
apiVersion: kustomize.config.k8s.io/v1alpha1
kind: Component
resources:
- https://github.com/codefresh-io/argo-workflows/releases/download/v3.2.6-cap-CR-8697/install.yaml
patches:
# reset the crbs to `subject.namespace: default`, so that argo-cd will later change them to the actual ns
- target:
group: rbac.authorization.k8s.io
version: v1
kind: ClusterRoleBinding
patch: |-
- op: replace
path: /subjects/0/namespace
value: default
- https://github.com/codefresh-io/argo-workflows/releases/download/v3.4.4-cf/install.yaml

patches:
- target:
group: apps
version: v1
Expand All @@ -22,29 +13,22 @@ patches:
- op: add
path: /spec/template/spec/containers/0/args/-
value: --auth-mode=server

patchesStrategicMerge:
- |
apiVersion: v1
kind: ConfigMap
metadata:
- target:
version: v1
kind: ConfigMap
name: workflow-controller-configmap
data:
# Fields reference can be found here: https://argoproj.github.io/argo-workflows/workflow-controller-configmap.yaml
workflowDefaults: |
spec:
ttlStrategy:
secondsAfterCompletion: 86400
secondsAfterSuccess: 86400
secondsAfterFailure: 86400
podGC:
strategy: OnWorkflowCompletion
labelSelector:
matchLabels:
should-be-deleted: "true"

configMapGenerator:
- name: workflow-controller-configmap
behavior: merge
literals:
- containerRuntimeExecutor=emissary
patch: |-
- op: add
path: /data
value:
workflowDefaults: |
spec:
ttlStrategy:
secondsAfterCompletion: 86400
secondsAfterSuccess: 86400
secondsAfterFailure: 86400
podGC:
strategy: OnWorkflowCompletion
labelSelector:
matchLabels:
should-be-deleted: "true"