Skip to content

Commit

Permalink
Fix errors in hack/quick-install.sh (#185)
Browse files Browse the repository at this point in the history
Fix issues:

1/ Command helm upgrade --install karpenter charts/karpenter fails.

2/ Uninstall: Current order of operation leaves deployment: karpenter and statefulset: prometheus-karpenter-monitor remaining:
  • Loading branch information
cameronsenese authored Dec 16, 2020
1 parent 85dd82d commit f8b0c36
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions hack/quick-install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,17 @@ set -eu -o pipefail
main() {
local command=${1:-'--apply'}
if [[ $command = "--apply" ]]; then
echo "Installing Karpenter & dependencies.."
apply
echo "Installation complete!"
elif [[ $command = "--delete" ]]; then
echo "Uninstalling Karpenter & dependencies.."
delete
echo "Uninstallation complete!"
else
echo "Error: invalid argument: $command" >&2
usage
exit 22 # EINVAL
exit 22 # EINVAL
fi
}

Expand All @@ -27,9 +29,9 @@ EOF
}

delete() {
helm delete karpenter || true
helm delete cert-manager --namespace cert-manager || true
helm delete kube-prometheus-stack --namespace monitoring || true
helm delete karpenter || true

kubectl delete namespace cert-manager monitoring || true
}
Expand Down Expand Up @@ -67,7 +69,7 @@ apply() {
--set nodeExporter.enabled=false \
--set prometheus.enabled=false

helm upgrade --install karpenter charts/karpenter
helm upgrade --install karpenter karpenter/karpenter
}

usage
Expand Down

0 comments on commit f8b0c36

Please sign in to comment.