Skip to content

Conversation

mathis-marcotte
Copy link

Small fix to improve consistency in the default workspace volume name when creating a new Notebook.

I noticed that on initial load of the crud-web-apps/jupyter form, the default workspace volume name was "{notebook-name}-workspace".
But if you deleted that workspace volume, and then re-added a workspace volume to the Notebook you are creating, then the workspace volume name was "{notebook-name}-volume".

This PR is to FIX the default name of the workspace volume to always follow the same standard of "{notebook-name}-workspace" to match with the default configs.

Original PR here: kubeflow/kubeflow#7705

Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign orfeas-k for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@andyatmiami
Copy link
Contributor

/area v1

@google-oss-prow google-oss-prow bot added the area/v1 area - version - kubeflow notebooks v1 label Sep 9, 2025
@andyatmiami
Copy link
Contributor

Hi @mathis-marcotte - do you think as part of this change you could add an assertion (or provide a new test entirely) here to validate this change as well?

Thanks for the contribution as well as re-opening on this "new" repo!
🙇

@mathis-marcotte
Copy link
Author

Hi @andyatmiami,
Yes I can totally add that assertion! Once I have some extra time soon, I will get that done.

@google-oss-prow google-oss-prow bot added area/backend area - related to backend components area/frontend area - related to frontend components size/S and removed size/XS labels Sep 17, 2025
@mathis-marcotte
Copy link
Author

@andyatmiami

Added the new test case for validating this change. Tested locally with npm run test and everything passed.

image

Let me know if there is anything else I can do to improve this PR

@andyatmiami
Copy link
Contributor

@mathis-marcotte

thank you for making this addition.. i'm a little back-logged on my PR reviewing - but would expect to be able to focus on this mid-next week.

will be in touch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/backend area - related to backend components area/frontend area - related to frontend components area/v1 area - version - kubeflow notebooks v1 size/S

Projects

Status: Needs Triage

Development

Successfully merging this pull request may close these issues.

2 participants