Skip to content

Commit

Permalink
Fix a pre-check condition for Helm binary (#169)
Browse files Browse the repository at this point in the history
* Fixed a condition on precheck that since V3.0.0 would cause helm version check not to run

* Updated condidition based on the comment

* Explicitly check for >0

Explicitly check for >0

Co-authored-by: Øystein Bedin <[email protected]>
  • Loading branch information
jfilipcz and oybed authored Feb 7, 2022
1 parent 4d99933 commit a140756
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion roles/openshift-applier/tasks/pre-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@
ignore_errors: true
register: helm_version_check
when:
- openshift_cluster_content | join(', ') | regex_search('helm_chart')
- openshift_cluster_content | sum(attribute='content', start=[]) | selectattr('helm', 'defined') | list | count > 0

# Block to handle helm command output
# - only proceed if the helm command returned any output
Expand Down

0 comments on commit a140756

Please sign in to comment.