Skip to content
This repository has been archived by the owner on May 18, 2019. It is now read-only.

Commit

Permalink
Set mergeStrategy to 'merge' for ArmadaChart patches
Browse files Browse the repository at this point in the history
This depends on argoproj/argo-workflows#1269.
Custom Resources can not be patched using the default merge strategy of
'strategic'. Once 1269 is merged, Argo will have that option.
  • Loading branch information
ian-howell committed Mar 20, 2019
1 parent b3abf04 commit b2edb92
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions armada-operator/examples/argo/wf-blog-group.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ spec:
- name: enable-blog-1-template
resource:
action: patch
successCondition: status.succeeded == 1
failureCondition: status.succeeded == 0
mergeStrategy: merge
successCondition: status.succeeded == true
manifest: |
apiVersion: armada.airshipit.org/v1alpha1
kind: ArmadaChart
Expand All @@ -33,8 +33,7 @@ spec:
- name: blog-1-ready-template
resource:
action: get
successCondition: status.succeeded == 1
failureCondition: status.succeeded == 0
successCondition: status.succeeded == true
manifest: |
apiVersion: armada.airshipit.org/v1alpha1
kind: ArmadaChart
Expand All @@ -44,8 +43,8 @@ spec:
- name: enable-blog-2-template
resource:
action: patch
successCondition: status.succeeded == 1
failureCondition: status.succeeded == 0
mergeStrategy: merge
successCondition: status.succeeded == true
manifest: |
apiVersion: armada.airshipit.org/v1alpha1
kind: ArmadaChart
Expand All @@ -57,8 +56,7 @@ spec:
- name: blog-2-ready-template
resource:
action: get
successCondition: status.succeeded == 1
failureCondition: status.succeeded == 0
successCondition: status.succeeded == true
manifest: |
apiVersion: armada.airshipit.org/v1alpha1
kind: ArmadaChart
Expand Down

0 comments on commit b2edb92

Please sign in to comment.