Skip to content

Conversation

@atheo89
Copy link
Member

@atheo89 atheo89 commented Jul 21, 2025

Manual PR to fix the sync from upstream

As the nightly Build fails: https://github.com/red-hat-data-services/rhods-devops-infra/actions/runs/16405788249/job/46351529923

mtchoum1 and others added 11 commits July 17, 2025 13:35
[RHOAIENG-26264] commit*.env references updates automatically at 11:00 UTC using cron job
…ahub-io#1414)

This commit introduces tests for ROCm-enabled workbench images on OpenShift. These tests verify that the images can be deployed successfully on a cluster with AMD GPUs and that both PyTorch and TensorFlow can correctly detect the available accelerator.
To support the testing of large accelerator images, the following changes were made:
- The pod readiness timeout in the test framework has been increased to 10 minutes to allow sufficient time for image pulling.
- The utility was updated to allow for configurable timeouts. `ImageDeployment`
- Existing CUDA tests were updated to use this new configurable timeout.
…tahub-io#1414)

The best fix is to make the SocketProxy more robust so that it doesn't crash when a connection attempt fails. By catching the expected BrokenPipeError, the proxy can simply discard the failed connection and continue listening for the next attempt from the Wait.until loop. This turns your test from a "hope it works" scenario into a reliable polling check.
…atahub-io#1412)

Previously, the linux/s390x build would fail to install Podman if Podman was not yet in the GitHub Actions cache.

Generalize the non-native architecture build process by using `tonistiigi/binfmt` to install QEMU handlers. This enables building container images for `linux/s390x` and `linux/ppc64le` on amd64 runners.

The podman installation step is now also performed for these new platforms. This replaces the previous approach that used `docker/setup-qemu-action` and only supported `s390x`.
@openshift-ci openshift-ci bot requested review from caponetto and jiridanek July 21, 2025 09:23
@openshift-ci
Copy link

openshift-ci bot commented Jul 21, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign caponetto for approval. For more information see the Code Review Process.

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

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

@jiridanek jiridanek merged commit fd69f05 into red-hat-data-services:main Jul 21, 2025
13 of 15 checks passed
@jiridanek
Copy link
Member

/lgtm
/approve

@atheo89 atheo89 deleted the sync-downstream branch October 13, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants