Skip to content

Commit

Permalink
Revert "ci: use apply rather than sync for deploys"
Browse files Browse the repository at this point in the history
This reverts commit 25c4e6f.
The "apply" action requires the helm-diff plugin to be installed.
Will revisit using an action to install and configure helmfile.
  • Loading branch information
conorsch committed Nov 2, 2023
1 parent 25c4e6f commit 155df60
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion deployments/ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,8 @@ function helm_uninstall() {
# as necessary. Will *not* replace certain durable resources like
# the LoadBalancer Service objects, which are annotated with helm.sh/resource-policy=keep.
function helm_install() {
helmfile apply -f "$HELMFILE_MANIFEST" --args \
# TODO: make sure helmfile is present in ci environemnt.
helmfile sync -f "$HELMFILE_MANIFEST" --args \
--set="image.tag=${PENUMBRA_VERSION}"
}

Expand Down

0 comments on commit 155df60

Please sign in to comment.