Skip to content

Commit

Permalink
serve-java-app: use eval cmd to work around weird formatting issue
Browse files Browse the repository at this point in the history
  • Loading branch information
robertchoi80 committed Jan 4, 2023
1 parent 8e6d3ef commit 81b9614
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion app_serving/serve-java-app.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -418,10 +418,11 @@ spec:
## TODO: this might be temporary. Once everything is confirmed,
## the helm chart can be pulled from internal helm repo.
helm upgrade --install --kubeconfig /etc/kubeconfig_temp -f /apps/app-serve-template/tomcat-vo.yaml -n {{workflow.parameters.namespace}} {{workflow.parameters.app_name}} . 2> >(tee -a $DEPLOY_LOG >&2)
## For springboot app case ##
else
cd /apps/app-serve-template/chart
helm template test . --debug
helm upgrade --install --kubeconfig /etc/kubeconfig_temp -f values-{{workflow.parameters.resource_spec}}.yaml $extra_env_str -n {{workflow.parameters.namespace}} {{workflow.parameters.app_name}} . 2> >(tee -a $DEPLOY_LOG >&2)
eval "helm upgrade --install --kubeconfig /etc/kubeconfig_temp -f values-{{workflow.parameters.resource_spec}}.yaml $extra_env_str -n {{workflow.parameters.namespace}} {{workflow.parameters.app_name}} ." 2> >(tee -a $DEPLOY_LOG >&2)
fi
# Just make sure helm release is deployed before status cmd
Expand Down

0 comments on commit 81b9614

Please sign in to comment.