Add notebook annotation to RayCluster

[Ygnas]

Issue link

https://issues.redhat.com/browse/RHOAIENG-6301

What changes have been made

Added the notebook prefix annotation to RayCluster when it was created from the workbench. That should allow to construct a workbench url for the metrics

Verification steps

Create a RayCluster from the Data Science Projects Workbench, the created cluster should have an annotation app.kubernetes.io/managed-by:

Checks

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

[Bobbins228]

/lgtm

[Bobbins228]

Perhaps we could improve the annotation to remove the path to the notebook?
For example my RC was given app.kubernetes.io/managed-by: /notebook/mark-dsp/mark-wb

Instead we should aim for app.kubernetes.io/managed-by: mark-wb

[Fiona-Waters]

Perhaps we could improve the annotation to remove the path to the notebook? For example my RC was given app.kubernetes.io/managed-by: /notebook/mark-dsp/mark-wb

Instead we should aim for app.kubernetes.io/managed-by: mark-wb

Is there any reason not to use the path here? The reason for the addition of this annotation is to allow the UI team to get the notebook URL for use on the frontend.

[Bobbins228]

@Fiona-Waters @Ygnas
My bad for not reading the issue correctly.
/lgtm

[Bobbins228]

/lgtm

[Fiona-Waters]

Ran through this on my rosa cluster and it works as expected. Looks great to me, thanks Ignas!
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Bobbins228, Fiona-Waters

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,Fiona-Waters]

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