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

rebase: use minikube v1.18 #1898

Merged
merged 1 commit into from
Mar 3, 2021
Merged

Conversation

nixpanic
Copy link
Member

@nixpanic nixpanic commented Mar 2, 2021

Minikube v1.18 has been released. This conains a fix for our losetup
workaround, so that can be removed now.

Updates: #1840


Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)
  • /retest all: run this in case the CentOS CI failed to start/report any test
    progress or results

@nixpanic nixpanic added the rebase update the version of an external component label Mar 2, 2021
@nixpanic nixpanic requested a review from Madhu-1 March 2, 2021 12:24
@nixpanic nixpanic added the component/testing Additional test cases or CI work label Mar 2, 2021
@nixpanic
Copy link
Member Author

nixpanic commented Mar 2, 2021

ci/centos/containerized-tests failed while building the image with dependencies for testing:

cd /opt/build/go/src/github.com/ceph/ceph-csi && make containerized-test TARGET=mod-check CONTAINER_CMD=podman ENV_CSI_IMAGE_NAME=registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi USE_PULLED_IMAGE=no'
Warning: Permanently added 'n28.crusty.ci.centos.org,172.19.2.28' (ECDSA) to the list of known hosts.
cephcsi image settings: registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi version canary
[ ! -f .test-container-id ] || podman rmi registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi:test
podman build  -t registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi:test -f ./scripts/Dockerfile.test .
STEP 1: FROM registry.fedoraproject.org/fedora:latest
Getting image source signatures
Copying blob sha256:157ab801145489f145f258148bd135102a3294e420f1859a39e824e7cda56b2f
Copying config sha256:9f2a56037643a68ea81711a8eeb4501428eefd40b000c866ad9745a581c0464d
Writing manifest to image destination
Storing signatures
STEP 2: ARG GOPATH=/go
--> Using cache 31de5ecfc421c7d12748a3c3cf48a2cd6bbd77b588e4b753e1d79d3cf39ceb2e
--> 31de5ecfc42
STEP 3: ENV  GOPATH=${GOPATH}  GO111MODULE=on  PATH="${GOPATH}/bin:/opt/commitlint/node_modules/.bin:${PATH}"
Error: error checking if cached image exists from a previous build: error getting history of "4770638bf2e01c123662b7dade27aaf1925fc2fe473628834729a4309b2d4131": error creating new image from reference to image "4770638bf2e01c123662b7dade27aaf1925fc2fe473628834729a4309b2d4131": error locating item named "manifest" for image with ID "4770638bf2e01c123662b7dade27aaf1925fc2fe473628834729a4309b2d4131": file does not exist
make: *** [Makefile:207: .test-container-id] Error 125
script returned exit code 2

Not sure what the problem can be, it seems something related to podman or maybe the fedora image?

@nixpanic
Copy link
Member Author

nixpanic commented Mar 2, 2021

/retest ci/centos/containerized-tests

@nixpanic
Copy link
Member Author

nixpanic commented Mar 2, 2021

ci/centos/containerized-tests failed while building the image with dependencies for testing:

cd /opt/build/go/src/github.com/ceph/ceph-csi && make containerized-test TARGET=mod-check CONTAINER_CMD=podman ENV_CSI_IMAGE_NAME=registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi USE_PULLED_IMAGE=no'
Warning: Permanently added 'n28.crusty.ci.centos.org,172.19.2.28' (ECDSA) to the list of known hosts.
cephcsi image settings: registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi version canary
[ ! -f .test-container-id ] || podman rmi registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi:test
podman build  -t registry-ceph-csi.apps.ocp.ci.centos.org/ceph-csi:test -f ./scripts/Dockerfile.test .
STEP 1: FROM registry.fedoraproject.org/fedora:latest
Getting image source signatures
Copying blob sha256:157ab801145489f145f258148bd135102a3294e420f1859a39e824e7cda56b2f
Copying config sha256:9f2a56037643a68ea81711a8eeb4501428eefd40b000c866ad9745a581c0464d
Writing manifest to image destination
Storing signatures
STEP 2: ARG GOPATH=/go
--> Using cache 31de5ecfc421c7d12748a3c3cf48a2cd6bbd77b588e4b753e1d79d3cf39ceb2e
--> 31de5ecfc42
STEP 3: ENV  GOPATH=${GOPATH}  GO111MODULE=on  PATH="${GOPATH}/bin:/opt/commitlint/node_modules/.bin:${PATH}"
Error: error checking if cached image exists from a previous build: error getting history of "4770638bf2e01c123662b7dade27aaf1925fc2fe473628834729a4309b2d4131": error creating new image from reference to image "4770638bf2e01c123662b7dade27aaf1925fc2fe473628834729a4309b2d4131": error locating item named "manifest" for image with ID "4770638bf2e01c123662b7dade27aaf1925fc2fe473628834729a4309b2d4131": file does not exist
make: *** [Makefile:207: .test-container-id] Error 125
script returned exit code 2

Not sure what the problem can be, it seems something related to podman or maybe the fedora image?

This problem should get addressed with #1900 . Once that one is merged and the CI jobs are updated, a /retest can be attempted.

@nixpanic
Copy link
Member Author

nixpanic commented Mar 2, 2021

/retest ci/centos/containerized-tests

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Mar 3, 2021

/test ci/centos/containerized-tests

Minikube v1.18 has been released. This conains a fix for our `losetup`
workaround, so that can be removed now.

Updates: ceph#1840
Signed-off-by: Niels de Vos <ndevos@redhat.com>
@ceph-csi-bot ceph-csi-bot force-pushed the rebase/minikube/v1.18 branch from bc0dba4 to 03e2d9d Compare March 3, 2021 03:58
@mergify mergify bot merged commit ec691fd into ceph:devel Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/testing Additional test cases or CI work rebase update the version of an external component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants