Skip to content

Conversation

sutaakar
Copy link
Contributor

@sutaakar sutaakar commented Apr 15, 2024

Issue link

N/A

What changes have been made

Adjusted existing e2e test to use Ingress/Route created by operator to download job logs.
Currently the test doesn't work on OpenShift because oauth container is still created by SDK. Once oauth creation is part of operator then it should work on OpenShift.

Verification steps

Verified by running PR check. Manual verification limited due to OpenShift issues.
Can be verified by running e2e test against fully deployed KinD cluster with Kuberay and operator.

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • Testing is not required for this change

@sutaakar sutaakar marked this pull request as ready for review April 16, 2024 08:47
@sutaakar sutaakar requested review from astefanutti and ChristianZaccaria and removed request for anishasthana and tedhtchang April 16, 2024 08:47
@astefanutti
Copy link
Contributor

/lgtm

@astefanutti
Copy link
Contributor

/approve

Copy link

openshift-ci bot commented Apr 16, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: astefanutti

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

The pull request process is described 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

@openshift-merge-bot openshift-merge-bot bot merged commit f412445 into project-codeflare:main Apr 16, 2024
@sutaakar sutaakar deleted the dnsmasq branch April 16, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants