Skip to content

Commit

Permalink
DNM: CI fix (#323)
Browse files Browse the repository at this point in the history
CI fix

SUMMARY
Signed-off-by: Abhijeet Kasurde akasurde@redhat.com
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
molecule/default/tasks/taint.yml

Reviewed-by: None <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Abhijeet Kasurde <None>
Reviewed-by: None <None>
  • Loading branch information
Akasurde authored Jan 12, 2022
1 parent b19ff9d commit 04e14c1
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 17 deletions.
14 changes: 2 additions & 12 deletions molecule/default/roles/helm/tasks/tests_helm_diff.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,20 +52,10 @@

- name: Check if helm diff check is correct
vars:
expected: "\u001b[0;33mhelm, test-chart-another-configmap, ConfigMap (v1) has been added:\u001b[0m\n\
\u001b[0;31m- \u001b[0m\n\
\u001b[0;32m+ # Source: test-chart/templates/anothermap.yaml\u001b[0m\n\
\u001b[0;32m+ # BEGIN ANSIBLE MANAGED BLOCK\u001b[0m\n\
\u001b[0;32m+ apiVersion: v1\u001b[0m\n\
\u001b[0;32m+ kind: ConfigMap\u001b[0m\n\
\u001b[0;32m+ metadata:\u001b[0m\n\
\u001b[0;32m+ name: test-chart-another-configmap\u001b[0m\n\
\u001b[0;32m+ data:\u001b[0m\n\
\u001b[0;32m+ foo: bar\u001b[0m\n\
\u001b[0;32m+ # END ANSIBLE MANAGED BLOCK\u001b[0m"
foo_bar_value: "+ foo: bar"
assert:
that:
- expected == diff_result.diff.prepared
- foo_bar_value in diff_result.diff.prepared

- name: Upgrade local chart with modifications
helm:
Expand Down
10 changes: 5 additions & 5 deletions molecule/default/tasks/taint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@
vars:
search_key: "{{ item.key}}"
search_effect: "{{ item.effect }}"
all_taints: "{{ taint_patch_1 }} + {{ taint_patch_2 }}"
all_taints: "{{ taint_patch_1 + taint_patch_2 }}"
with_items: "{{ _result.result.spec.taints }}"

- name: Remove taints from node (check_mode)
Expand Down Expand Up @@ -323,7 +323,7 @@
vars:
search_key: "{{ item.key}}"
search_effect: "{{ item.effect }}"
all_taints: "{{ taint_patch_1 }} + {{ taint_patch_2 }}"
all_taints: "{{ taint_patch_1 + taint_patch_2 }}"
with_items: "{{ _result.result.spec.taints }}"

- name: Add other taints and update
Expand All @@ -340,7 +340,7 @@
vars:
search_key: "{{ item.key}}"
search_effect: "{{ item.effect }}"
all_taints: "{{ taint_patch_3 }} + {{ taint_patch_2 }}"
all_taints: "{{ taint_patch_3 + taint_patch_2 }}"
with_items: "{{ _result.result.spec.taints }}"

- name: Remove taint using key:effect
Expand Down Expand Up @@ -373,7 +373,7 @@
vars:
search_key: "{{ item.key}}"
search_effect: "{{ item.effect }}"
all_taints: "{{ taint_patch_3 }} + {{ left_taint_patch_2 }}"
all_taints: "{{ taint_patch_3 + left_taint_patch_2 }}"
with_items: "{{ _result.resources[0].spec.taints }}"

- name: Remove taint using key
Expand Down Expand Up @@ -405,7 +405,7 @@
vars:
search_key: "{{ item.key}}"
search_effect: "{{ item.effect }}"
all_taints: "{{ left_taint_patch_2 }} + {{ left_taint_patch_3 }}"
all_taints: "{{ left_taint_patch_2 + left_taint_patch_3 }}"
with_items: "{{ _result.resources[0].spec.taints }}"

- name: Remove taints (including non existing ones)
Expand Down

0 comments on commit 04e14c1

Please sign in to comment.