Skip to content

Adjust build test images for multi-arch support#26598

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
yselkowitz:ma-test-images
Dec 8, 2021
Merged

Adjust build test images for multi-arch support#26598
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
yselkowitz:ma-test-images

Conversation

@yselkowitz
Copy link
Contributor

@yselkowitz yselkowitz commented Nov 12, 2021

This includes a test image rebase.

/cc @sttts
/cc @dmage

@openshift-ci openshift-ci bot requested review from dmage and sttts November 12, 2021 04:45
@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 12, 2021

/test help

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 12, 2021

@yselkowitz: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test e2e-aws-fips
  • /test e2e-aws-image-registry
  • /test e2e-aws-jenkins
  • /test e2e-aws-serial
  • /test e2e-gcp
  • /test e2e-gcp-builds
  • /test e2e-gcp-image-ecosystem
  • /test e2e-gcp-upgrade
  • /test extended_gssapi
  • /test extended_ldap_groups
  • /test extended_networking
  • /test images
  • /test verify
  • /test verify-deps

The following commands are available to trigger optional jobs:

  • /test e2e-agnostic-cmd
  • /test e2e-aws
  • /test e2e-aws-cgroupsv2
  • /test e2e-aws-csi
  • /test e2e-aws-csi-migration
  • /test e2e-aws-disruptive
  • /test e2e-aws-multitenant
  • /test e2e-aws-ovn
  • /test e2e-aws-proxy
  • /test e2e-aws-single-node
  • /test e2e-aws-single-node-serial
  • /test e2e-aws-upgrade
  • /test e2e-azure
  • /test e2e-gcp-csi
  • /test e2e-gcp-disruptive
  • /test e2e-gcp-fips-serial
  • /test e2e-metal-ipi
  • /test e2e-metal-ipi-ovn-dualstack
  • /test e2e-metal-ipi-ovn-ipv6
  • /test e2e-metal-ipi-serial
  • /test e2e-metal-ipi-virtualmedia
  • /test e2e-openstack
  • /test e2e-openstack-serial
  • /test e2e-vsphere
  • /test okd-e2e-gcp

Use /test all to run the following jobs that were automatically triggered:

  • pull-ci-openshift-origin-master-e2e-agnostic-cmd
  • pull-ci-openshift-origin-master-e2e-aws-cgroupsv2
  • pull-ci-openshift-origin-master-e2e-aws-csi
  • pull-ci-openshift-origin-master-e2e-aws-fips
  • pull-ci-openshift-origin-master-e2e-aws-serial
  • pull-ci-openshift-origin-master-e2e-aws-single-node
  • pull-ci-openshift-origin-master-e2e-gcp
  • pull-ci-openshift-origin-master-e2e-gcp-builds
  • pull-ci-openshift-origin-master-e2e-gcp-csi
  • pull-ci-openshift-origin-master-e2e-gcp-upgrade
  • pull-ci-openshift-origin-master-images
  • pull-ci-openshift-origin-master-verify
  • pull-ci-openshift-origin-master-verify-deps
Details

In response to this:

/test help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 12, 2021

/test e2e-aws-jenkins

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 12, 2021

/test e2e-gcp-image-ecosystem

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 12, 2021

FYI this is used in build and jenkins tests, and commented-out parts of image-eco.
/cc @gabemontero @adambkaplan @akram

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 12, 2021

/retest

@gabemontero
Copy link
Contributor

gabemontero commented Nov 12, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 12, 2021
@gabemontero
Copy link
Contributor

gabemontero commented Nov 12, 2021 •
edited
Loading 

Nov 12 13:15:19.751: INFO: At 2021-11-12 13:10:21 +0000 UTC - event for nfs-server: {multus } AddedInterface: Add eth0 [10.129.2.14/23] from openshift-sdn
Nov 12 13:15:19.751: INFO: At 2021-11-12 13:10:21 +0000 UTC - event for nfs-server: {kubelet ip-10-0-250-249.us-east-2.compute.internal} Pulling: Pulling image "quay.io/openshift/community-e2e-images:e2e-quay-io-redhat-developer-nfs-server-1-1-dlXGfzrk5aNo8EjC"
Nov 12 13:15:19.751: INFO: At 2021-11-12 13:10:21 +0000 UTC - event for nfs-server: {kubelet ip-10-0-250-249.us-east-2.compute.internal} Failed: Failed to pull image "quay.io/openshift/community-e2e-images:e2e-quay-io-redhat-developer-nfs-server-1-1-dlXGfzrk5aNo8EjC": rpc error: code = Unknown desc = reading manifest e2e-quay-io-redhat-developer-nfs-server-1-1-dlXGfzrk5aNo8EjC in quay.io/openshift/community-e2e-images: manifest unknown: manifest unknown
Nov 12 13:15:19.751: INFO: At 2021-11-12 13:10:21 +0000 UTC - event for nfs-server: {kubelet ip-10-0-250-249.us-east-2.compute.internal} Failed: Error: ErrImagePull
Nov 12 13:15:19.751: INFO: At 2021-11-12 13:10:22 +0000 UTC - event for nfs-server: {kubelet ip-10-0-250-249.us-east-2.compute.internal} BackOff: Back-off pulling image "quay.io/openshift/community-e2e-images:e2e-quay-io-redhat-developer-nfs-server-1-1-dlXGfzrk5aNo8EjC"
Nov 12 13:15:19.751: INFO: At 2021-11-12 13:10:22 +0000 UTC - event for nfs-server: {kubelet ip-10-0-250-249.us-east-2.compute.internal} Failed: Error: ImagePullBackOff

^^ @yselkowitz last I dealt with this I think you need to engage with either @soltysh or @smarterclayton to manually pull the trigger on getting the mirrored image pushed to quay.io

https://github.com/openshift/origin/blob/master/test/extended/util/image/README.md

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 12, 2021

@sttts is also listed in OWNERS there, but I'm not picky :-)

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 16, 2021

/cc @smarterclayton @soltysh @sttts
for a test image update.

@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Nov 18, 2021
@yselkowitz yselkowitz changed the title Bump nfs-server version for multi-arch support Adjust build test images for multi-arch support Nov 18, 2021
@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 18, 2021

Ping @smarterclayton @soltysh @sttts for a test image update per https://github.com/openshift/origin/blob/master/test/extended/util/image/README.md

@smarterclayton
Copy link
Contributor

smarterclayton commented Nov 18, 2021

@soltysh can you do the image mirror for this PR? If not I'll do it (hopfeully tomorrow, but might get delayed)

@soltysh
Copy link
Contributor

soltysh commented Nov 19, 2021

Should be mirrored by now, let's see
/retest

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 19, 2021

/test e2e-gcp-image-ecosystem

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Nov 19, 2021

/test e2e-gcp-builds

@soltysh
Copy link
Contributor

soltysh commented Nov 19, 2021

/test e2e-gcp

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 7, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 7, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adambkaplan, gabemontero, soltysh, yselkowitz

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 7, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

8 similar comments
@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 7, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 7, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Dec 8, 2021

/retest

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@yselkowitz
Copy link
Contributor Author

yselkowitz commented Dec 8, 2021

/retest-required

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

4 similar comments
@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 8, 2021 •
edited
Loading

@yselkowitz: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-single-node 141d957 link false /test e2e-aws-single-node

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

6 similar comments
@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

openshift-bot commented Dec 8, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit d04e24f into openshift:master Dec 8, 2021
@yselkowitz yselkowitz deleted the ma-test-images branch April 29, 2022 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adambkaplan adambkaplan adambkaplan approved these changes

@dmage dmage Awaiting requested review from dmage

@sttts sttts Awaiting requested review from sttts

@akram akram Awaiting requested review from akram

@gabemontero gabemontero Awaiting requested review from gabemontero

@smarterclayton smarterclayton Awaiting requested review from smarterclayton

+1 more reviewer

@soltysh soltysh soltysh approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@soltysh soltysh

@adambkaplan adambkaplan

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@yselkowitz @gabemontero @smarterclayton @soltysh @akram @adambkaplan @openshift-bot @openshift-merge-robot

Comments