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

break: test-cmd broke because of image for ppc64le arch no longer being imported from docker hub #17436

Closed
smarterclayton opened this issue Nov 22, 2017 · 10 comments · Fixed by #17437
Assignees
Labels
area/tests component/imageregistry kind/test-flake Categorizes issue or PR as related to test flakes. priority/P0

Comments

@smarterclayton
Copy link
Contributor

smarterclayton commented Nov 22, 2017

Since 2:30 EST on 11/22:

https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/17314/test_pull_request_origin_cmd/6415/#githubcomopenshiftorigintestcmdimagesimagestreams-testcmdimagessh150-executing-oc-get-istag-test-nginx-multiarch-streamlinux-ppc64le---templateimagedockerimagemetadataarchitecture-expecting-success-and-text-ppc64le

=== BEGIN TEST CASE ===
test/cmd/images.sh:150: executing 'oc get istag test-nginx-multiarch-stream:linux-ppc64le --template={{.image.dockerImageMetadata.Architecture}}' expecting success and text 'ppc64le'
FAILURE after 0.233s: test/cmd/images.sh:150: executing 'oc get istag test-nginx-multiarch-stream:linux-ppc64le --template={{.image.dockerImageMetadata.Architecture}}' expecting success and text 'ppc64le': the output content test failed
Standard output from the command:
amd64
There was no error output from the command.
=== END TEST CASE ===

Could be #17391 (unlikely, but merged at same time) or could be something else like Docker changing the images on the hub

@openshift/sig-developer-experience @deads2k

@smarterclayton smarterclayton added component/imageregistry priority/P0 area/tests kind/test-flake Categorizes issue or PR as related to test flakes. labels Nov 22, 2017
@bparees
Copy link
Contributor

bparees commented Nov 23, 2017

docker manifest isn't providing anything except amd64 anymore:
https://paste.fedoraproject.org/paste/zAws11LEKRD-EtJxq-hg7A

looks like they reverted to schema2.v1 manifests?

we're just going to have to disable the test for now, unless you know of an image that does have other a v2 manifest.

@bparees
Copy link
Contributor

bparees commented Nov 23, 2017

well I did that wrong, but even once i request v2 + manifestlist schemas, i'm just getting back a regular manifest, not a manifest list.

@bparees
Copy link
Contributor

bparees commented Nov 23, 2017

well w/ some logging I am seeing manifestlists coming back. I don't know why my curl invocation won't give me one. investigating why we're not seeing ppc

@bparees
Copy link
Contributor

bparees commented Nov 23, 2017

ok, s390x exists as a secondary arch, but ppc no longer does. updated the test to use that for now, but we probably need to create our own manifestlist image on dockerhub in the future, if someone can help me figure out how to push a manifestlist.

@bparees bparees self-assigned this Nov 23, 2017
@bparees
Copy link
Contributor

bparees commented Nov 23, 2017

updated the test to use a repository under our control w/ a multiarch manifestlist, so this won't get broken in the future if docker changes random image manifests.

@smarterclayton
Copy link
Contributor Author

smarterclayton commented Nov 23, 2017 via email

@bparees
Copy link
Contributor

bparees commented Nov 23, 2017

oc image mirror should be able to easily copy

copy what? to where?

@smarterclayton
Copy link
Contributor Author

smarterclayton commented Nov 23, 2017 via email

@bparees
Copy link
Contributor

bparees commented Nov 23, 2017

oc image mirror SRC DST where SRC is a manifest list and DST is a repo
under our control on the hub

per my comments in the PR, i've already created a repo under our control w/ a proper manifest list. And i'm not sure where i would have SRCed a valid one from if i hadn't created one from scratch, since as already established, the one we were using doesn't contain ppc anymore.

@priyankakhetawat
Copy link

Hi,
I am trying to do oc cluster up, I am getting this same error on rhel 7 with sys architecture as ppc64le

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tests component/imageregistry kind/test-flake Categorizes issue or PR as related to test flakes. priority/P0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants