[275] Implement ExposeService method

[jiripetrlik]

Issue link

Issue: #275

PR to distributed-workloads with changes to tests will follow.

[jiripetrlik]

@sutaakar
Thank you for the review. I've tried to address both your comments.

[sutaakar]

Please use the new function in https://github.com/project-codeflare/codeflare-operator/blob/a14914da37d61dd2e3de160c7033e3b2fe6d500e/test/e2e/mnist_rayjob_mcad_raycluster_test.go to make sure it works properly.
Also the KUBERNETES_HOSTNAME needs to be set in PR check workflow.

[jiripetrlik]

Hello @sutaakar , I've tried to address all the comments. In the final version I removed KUBERNETES_HOSTNAME parameter at all.

[jiripetrlik]

Thank you @sutaakar for the review. I think I've addressed all your comments.

[sutaakar]

When I think about it again, this is not actually needed, as above we construct HTTP route anyway, so we are sure it is HTTP.
Can you please remove this code?

[jiripetrlik]

/retest

[dimakis]

/retest

[ChristianZaccaria]

/retest

[sutaakar]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 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 [sutaakar]

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