Skip to content

Commit

Permalink
helm 2.14 -> 2.17; succeeded pod is always valid
Browse files Browse the repository at this point in the history
  • Loading branch information
meir.pechthalt committed May 5, 2021
1 parent 419bb86 commit f0ae454
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
7 changes: 4 additions & 3 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FROM alpine:3.9

ARG HELM_VERSION=v2.14.0
ARG HELM_VERSION=v2.17.0
ARG HELM_OS_ARCH=linux-amd64

RUN apk --no-cache add ca-certificates git bash curl jq \
Expand All @@ -20,7 +20,8 @@ WORKDIR /home/orca

ENV HELM_HOME /home/orca/.helm

RUN helm init -c \
RUN helm init --stable-repo-url=https://charts.helm.sh/stable --client-only \
&& helm repo add "stable" "https://charts.helm.sh/stable" \
&& helm plugin install https://github.com/chartmuseum/helm-push

CMD ["orca"]
CMD ["orca"]
2 changes: 1 addition & 1 deletion pkg/utils/validate.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ func validatePods(name, kubeContext string, envValid bool) (bool, error) {
if phase == "Running" {
continue
}
if phase == "Succeeded" && pod.OwnerReferences[0].Kind == "Job" {
if phase == "Succeeded" {
continue
}
log.Printf("pod %s is in phase \"%s\"", pod.Name, phase)
Expand Down

0 comments on commit f0ae454

Please sign in to comment.