Skip to content

Commit

Permalink
Bug fix: Render actions fail when non Helm values yamls are present i…
Browse files Browse the repository at this point in the history
…n the examples directory (#686)

* Bug Fix: Make the render-examples.sh script only renders values.yaml files

* Rename example helm values files to end in "values.yaml"
  • Loading branch information
jvoravong authored Mar 6, 2023
1 parent e604320 commit d1d49e6
Show file tree
Hide file tree
Showing 8 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion examples/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

Each example has a directory where each of the following is included.
- README.md: A short description about the example.
- A Helm values configuration file to demonstrate the example.
- A Helm values configuration file to demonstrate the example, the file name always ends in values.yaml.
- A rendered_manifests directory that contains the rendered Kubernetes manifests for the example.
- Search for "CHANGEME" to find the values that must be changed in order to use the rendered manifests directly.

Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion examples/render-examples.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )
render_task(){
example_dir=$1
rendered_manifests_dir=$example_dir"rendered_manifests"
values_yaml=$example_dir`ls $example_dir | grep yaml`
values_yaml=$example_dir`ls $example_dir | grep values.yaml`

# Clear out all rendered manifests
rm -rf $rendered_manifests_dir
Expand Down

0 comments on commit d1d49e6

Please sign in to comment.