Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make verify miss local helm #555

Closed
andrei-panov opened this issue Aug 9, 2022 · 1 comment
Closed

make verify miss local helm #555

andrei-panov opened this issue Aug 9, 2022 · 1 comment
Labels
area/dev-productivity Developer productivity related (how to improve development) kind/bug Bug status/closed Issue is closed (either delivered or triaged)

Comments

@andrei-panov
Copy link
Contributor

How to categorize this issue?

/area dev-productivity
/kind bug

What happened:
During execution of make verify it fails with missed helm

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

FROM ubuntu:xenial-20210804

RUN apt-get update && \
    apt-get install -y wget make git build-essential curl

RUN wget https://dl.google.com/go/go1.18.4.linux-amd64.tar.gz && \
    tar -C /usr/local -xzf go1.18.4.linux-amd64.tar.gz && \
    ln -s /usr/local/go/bin/go /usr/local/bin/go && \
    rm -rf go1.18.4.linux-amd64.tar.gz

#RUN curl -fsSL -o get_helm.sh https://raw.githubusercontent.com/helm/helm/main/scripts/get-helm-3 && \
#    chmod 700 get_helm.sh && \
#    ./get_helm.sh

RUN go env

WORKDIR /root/go/src/github.com/gardener/gardener-extension-provider-azure
COPY . .
RUN make install
RUN make verify

Anything else we need to know?:

$ docker build --file Dockerfile2 .
Sending build context to Docker daemon  167.5MB
Step 1/8 : FROM ubuntu:xenial-20210804
 ---> b6f507652425
Step 2/8 : RUN apt-get update &&     apt-get install -y wget make git build-essential curl
 ---> Using cache
 ---> a8cb4e5a6cfc
Step 3/8 : RUN wget https://dl.google.com/go/go1.18.4.linux-amd64.tar.gz &&     tar -C /usr/local -xzf go1.18.4.linux-amd64.tar.gz &&     ln -s /usr/local/go/bin/go /usr/local/bin/go &&     rm -rf go1.18.4.linux-amd64.tar.gz
 ---> Using cache
 ---> b6e8bfaec581
Step 4/8 : RUN go env
 ---> Using cache
 ---> 3e8962f9d205
Step 5/8 : WORKDIR /root/go/src/github.com/gardener/gardener-extension-provider-azure
 ---> Running in 98877d862c60
Removing intermediate container 98877d862c60
 ---> ce868acd4cb4
Step 6/8 : COPY . .
 ---> c523b276db32
Step 7/8 : RUN make install
 ---> Running in 135682ce141a
> Install
Removing intermediate container 135682ce141a
 ---> 255def5afce1
Step 8/8 : RUN make verify
 ---> Running in 435d06e6a72c
> Check
Executing golangci-lint
Executing gofmt/goimports
All checks successful
> Check Helm charts
Checking for chart symlink errors
Checking whether all charts can be rendered
/root/go/src/github.com/gardener/gardener-extension-provider-azure/vendor/github.com/gardener/gardener/hack/check-charts.sh: line 32: helm: command not found
make: *** [check] Error 127
Makefile:121: recipe for target 'check' failed
The command '/bin/sh -c make verify' returned a non-zero code: 2

related issue #27

@gardener-robot gardener-robot added area/dev-productivity Developer productivity related (how to improve development) kind/bug Bug labels Aug 9, 2022
@andrei-panov andrei-panov changed the title make verify miss local helm make verify miss local helm Aug 9, 2022
@andrei-panov
Copy link
Contributor Author

fix is merged: #556

@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dev-productivity Developer productivity related (how to improve development) kind/bug Bug status/closed Issue is closed (either delivered or triaged)
Projects
None yet
Development

No branches or pull requests

2 participants