Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Add dashboard config CRD #561

Merged
merged 3 commits into from
Jul 8, 2022

Conversation

maroroman
Copy link
Contributor

@maroroman maroroman commented May 13, 2022

Adds a odh-dashboard custom resource definition (and a default instance). Currently it replaces the values in the Dashboard configmap and adds notebook sizes, notebook controller configuration and notebook controller status.

The notebook controller part of the CR is supposed to essentially fill the role of the Jupyterhub Singleuser Profiles configmap, but for the Notebook controller.

The notebook controller status should be an array with the information about the user's previous choices and environment variables and a link to their envVar secrets (managed by the spawner UI)

Note: I do not know whether to also add other parts of the old JSP configmap, such as gpuTypes and profiles, although these can be added in together with the features that use them.

Related to: opendatahub-io/odh-dashboard#159

@maroroman maroroman force-pushed the dashboard/cr-config branch 2 times, most recently from b15eced to ab0917f Compare May 13, 2022 09:44
@maroroman
Copy link
Contributor Author

maroroman commented May 13, 2022

@anishasthana @VaishnaviHire This CR has a problem that it would be reconciled by the operator, I am considering creating it in a init container of the dashboard when it is deployed to work around it, but let me know what would be the best solution.

At the very least the notebook status needs to be updated on every action in the spawner UI

Copy link

@dlabaj dlabaj left a comment

Choose a reason for hiding this comment

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

lgtm

@openshift-ci openshift-ci bot removed the lgtm label May 16, 2022
@LaVLaS
Copy link
Contributor

LaVLaS commented May 16, 2022

@maroroman @dlabaj Will this be supported in the dashboard v2.0 release? If not, then we should put this on hold until it is included in a planned release

@maroroman maroroman force-pushed the dashboard/cr-config branch 3 times, most recently from 993561f to d0670e3 Compare May 18, 2022 15:03
@maroroman
Copy link
Contributor Author

@LaVLaS This PR should not break anything in Dashboard 2, but as soon as the backend is added it will replace the configmap that is currently used. With the additional options that @dlabaj requested it should work with dashboard 2 fine.

Copy link
Contributor

@lucferbux lucferbux left a comment

Choose a reason for hiding this comment

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

/lgtm

@LaVLaS
Copy link
Contributor

LaVLaS commented Jun 6, 2022

/hold

Putting this on hold until opendatahub-io/odh-dashboard#238 is finalized and merged

@openshift-ci openshift-ci bot added the do-not-merge/hold Hold off on merging (provide reason in comment) label Jun 6, 2022
Copy link
Contributor

@LaVLaS LaVLaS left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link
Contributor

@LaVLaS LaVLaS left a comment

Choose a reason for hiding this comment

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

/lgtm

@LaVLaS
Copy link
Contributor

LaVLaS commented Jul 7, 2022

/unhold

This is required for the Dashboard v2.0 release

@openshift-ci openshift-ci bot removed the do-not-merge/hold Hold off on merging (provide reason in comment) label Jul 7, 2022
@maroroman
Copy link
Contributor Author

/retest

@LaVLaS LaVLaS requested a review from dlabaj July 8, 2022 14:18
Copy link

@dlabaj dlabaj left a comment

Choose a reason for hiding this comment

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

lgtm

@openshift-ci openshift-ci bot added the lgtm label Jul 8, 2022
@openshift-ci
Copy link

openshift-ci bot commented Jul 8, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dlabaj, LaVLaS, lucferbux, maroroman

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-ci openshift-ci bot merged commit 7df1e6b into opendatahub-io:master Jul 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants