Skip to content

test: fix ray pipeline and lower casing pipeline names in upgrade tests#12766

Open
nsingla wants to merge 1 commit intokubeflow:masterfrom
nsingla:fix_tests
Open

test: fix ray pipeline and lower casing pipeline names in upgrade tests#12766
nsingla wants to merge 1 commit intokubeflow:masterfrom
nsingla:fix_tests

Conversation

@nsingla
Copy link
Contributor

@nsingla nsingla commented Feb 4, 2026

Description of your changes:

  • lower casing pipeline name in the upgrade tests to avoid issues with k8s resource naming guidelines based on the k8s version
  • Enhance Ray integration test robustness:
    • Replace unreliable wait_ready() with custom timeout logic
    • Add comprehensive error handling and fallback connection strategies
    • Improve TLS setup with graceful fallbacks for direct connections
    • Add detailed logging and debugging output
    • Ensure proper cluster cleanup procedures

Checklist:

@google-oss-prow
Copy link

[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 zazulam for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@nsingla nsingla force-pushed the fix_tests branch 2 times, most recently from 0a3e64b to f36f046 Compare February 4, 2026 20:45
cluster is up and running: \", ray.is_initialized())\n\n @ray.remote\n def
train_fn():\n return 100\n\n result = ray.get(train_fn.remote())\n assert
100 == result\n ray.shutdown()\n cluster.down()\n return result\n\n"],"image":"registry.access.redhat.com/ubi9/python-311:latest"}'
Updated to Ray 2.53.0 compatible image\n image=\"quay.io/modh/ray@sha256:6d076aeb38ab3c34a6a2ef0f58dc667089aa15826fa08a73273c629333e12f1e\",\n verify_tls=False\n )\n )\n\n #
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Updated to Ray 2.53.0 compatible image indicates that this yaml was not recompiled.

Signed-off-by: Nelesh Singla <117123879+nsingla@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants