Add RayCLuster SDK Oauth Authentication test

[Srihari1192]

Issue link

closes https://issues.redhat.com/browse/RHOAIENG-55

What changes have been made

• Added tests for RayCluster SDK Oauth Authentication scenarios with and without login using python
• Added pytest marker for each tests for selecting tests to run on kind or openshift cluster environment
• Refactored setup methods kube configuration , create and delete namespace in support.py

Verification steps

Checks

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

[Srihari1192]

Will convert this test to python tests similar to PR #451

[Bobbins228]

I ran through these changes with poetry run pytest -v -s ./tests/e2e/mnist_raycluster_sdk_oauth_test.py -m openshift and I received this error.

tests/e2e/mnist_raycluster_sdk_oauth_test.py:65: in run_mnist_raycluster_sdk_oauth
    self.assert_jobsubmit_withlogin(cluster)
tests/e2e/mnist_raycluster_sdk_oauth_test.py:105: in assert_jobsubmit_withlogin
    status = job.status()
../.local/lib/python3.11/site-packages/codeflare_sdk/job/jobs.py:201: in status
    return self._runner.status(self._app_handle)
../.local/lib/python3.11/site-packages/torchx/runner/api.py:419: in status
    desc = scheduler.describe(app_id)
../.local/lib/python3.11/site-packages/torchx/schedulers/ray_scheduler.py:383: in describe
    job_status_info = self._get_job_status(app_id)
../.local/lib/python3.11/site-packages/torchx/schedulers/ray_scheduler.py:376: in _get_job_status
    client = self._get_ray_client(job_submission_netloc=addr)

With the final pytest error being:
FAILED tests/e2e/mnist_raycluster_sdk_oauth_test.py::TestRayClusterSDKOauth::test_mnist_ray_cluster_sdk_auth - ValueError: client netloc (ray-dashboard-mnist-test-ns-6t2pj.apps.rosa.mcampbel.n399.p3.openshiftapps.com) does not match job netloc (ray)

Did you come across something like this when testing @Srihari1192

[Srihari1192]

poetry run pytest -v -s ./tests/e2e/mnist_raycluster_sdk_oauth_test.py

@Bobbins228 no it works fine on my cluster
I didnt added import ray as assumed not required , is it failing because of this?

[Bobbins228]

/lgtm

[ChristianZaccaria]

Just one nitpick there. After that, lgtm

[ChristianZaccaria]

It seems to just need a rebase, afterwards, /lgtm

[ChristianZaccaria]

/lgtm

[sutaakar]

@Srihari1192 Can you adjust https://github.com/project-codeflare/codeflare-sdk/blob/main/.github/workflows/e2e_tests.yaml#L126 to run all tests with proper kind marker?

[sutaakar]

Can you add timeout for tests?
From my local experiments it seems that https://pypi.org/project/pytest-timeout/ works quite well.

[sutaakar]

/lgtm

[Bobbins228]

Awesome stuff Srihari works like a charm 🥇
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Bobbins228, ChristianZaccaria, sutaakar

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:

• OWNERS [Bobbins228]

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