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

test: Fix local tests #15130

Merged
merged 1 commit into from
Feb 26, 2021
Merged

test: Fix local tests #15130

merged 1 commit into from
Feb 26, 2021

Conversation

pchaigno
Copy link
Member

Local ginkgo execution is failing with:

    k8s1-1.20: building cilium container image...
    k8s1-1.20: Using Docker Buildx builder "default" with build flags " --load".
    k8s1-1.20: docker buildx build -f images/cilium/Dockerfile  --load --build-arg BASE_IMAGE=scratch --build-arg NOSTRIP= --build-arg LOCKDEBUG=1 --build-arg RACE= --build-arg V= --build-arg LIBNETWORK_PLUGIN= --build-arg CILIUM_SHA=0df8078e9 -t quay.io/cilium/cilium:latest .
    k8s1-1.20: #1 [internal] load build definition from Dockerfile
    [...]
    k8s1-1.20: #26 writing image sha256:1ec75dbc4cc595cc0700cf3070f2ba18c7c06cef5542c763fc4da4acd23aa275 done
    k8s1-1.20: #26 naming to quay.io/cilium/cilium:latest done
    k8s1-1.20: #26 DONE 2.1s
    k8s1-1.20: Define "DOCKER_FLAGS=--push" to push the build results.
    k8s1-1.20: tagging cilium image...
    k8s1-1.20: Error response from daemon: No such image: cilium/cilium:latest
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

André suggested to define DOCKER_REGISTRY and export the environment variables.

Co-authored-by: André Martins [email protected]
Signed-off-by: Paul Chaignon [email protected]

Local ginkgo execution is failing with:

        k8s1-1.20: building cilium container image...
        k8s1-1.20: Using Docker Buildx builder "default" with build flags " --load".
        k8s1-1.20: docker buildx build -f images/cilium/Dockerfile  --load --build-arg BASE_IMAGE=scratch --build-arg NOSTRIP= --build-arg LOCKDEBUG=1 --build-arg RACE= --build-arg V= --build-arg LIBNETWORK_PLUGIN= --build-arg CILIUM_SHA=0df8078e9 -t quay.io/cilium/cilium:latest .
        k8s1-1.20: #1 [internal] load build definition from Dockerfile
        [...]
        k8s1-1.20: #26 writing image sha256:1ec75dbc4cc595cc0700cf3070f2ba18c7c06cef5542c763fc4da4acd23aa275 done
        k8s1-1.20: #26 naming to quay.io/cilium/cilium:latest done
        k8s1-1.20: #26 DONE 2.1s
        k8s1-1.20: Define "DOCKER_FLAGS=--push" to push the build results.
        k8s1-1.20: tagging cilium image...
        k8s1-1.20: Error response from daemon: No such image: cilium/cilium:latest
    The SSH command responded with a non-zero exit status. Vagrant
    assumes that this means the command failed. The output for this command
    should be in the log above. Please read the output to determine what
    went wrong.

André suggested to define DOCKER_REGISTRY and export the environment
variables.

Co-authored-by: André Martins <[email protected]>
Signed-off-by: Paul Chaignon <[email protected]>
@pchaigno pchaigno added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. sig/contributing Impacts contribution workflow, guidelines, and tools. labels Feb 26, 2021
@pchaigno pchaigno requested a review from a team as a code owner February 26, 2021 14:57
@pchaigno pchaigno requested a review from nbusseneau February 26, 2021 14:57
@pchaigno pchaigno requested a review from aanm February 26, 2021 14:57
@aanm aanm removed their assignment Feb 26, 2021
@pchaigno
Copy link
Member Author

test-me-please

@pchaigno
Copy link
Member Author

@aanm Where did maintainer's little helper go?

@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 26, 2021
@christarazi christarazi merged commit a3bb073 into cilium:master Feb 26, 2021
@pchaigno pchaigno deleted the fix-local-tests branch February 26, 2021 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI. sig/contributing Impacts contribution workflow, guidelines, and tools.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants