Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support notebook volume (PVC) management #20

Merged
merged 4 commits into from
Oct 6, 2020

Conversation

wg102
Copy link
Contributor

@wg102 wg102 commented Sep 11, 2020

Fixes #18

Note:
Since the data is now passed through main-table:

  • makes similar changes for the resource-table
  • moves the Confirmation Dialog into the main-table folder.

@wg102 wg102 marked this pull request as ready for review September 17, 2020 19:05
@wg102 wg102 force-pushed the 18-destroy-old-notebooks branch from ae76680 to 421ad7f Compare September 17, 2020 19:21
@wg102 wg102 force-pushed the 18-destroy-old-notebooks branch from 421ad7f to 2095188 Compare September 17, 2020 19:24
* Add icons to headers of the two tables under the main-table component
* Refactor code for cleaner use of promises
* Show user size of volumes in table
* Show user what notebooks volumes are mounted to
@saffaalvi saffaalvi requested a review from brendangadd October 1, 2020 23:17
@brendangadd brendangadd changed the title Destroy old notebooks Support notebook volume (PVC) management Oct 2, 2020
Copy link
Contributor

@brendangadd brendangadd left a comment

Choose a reason for hiding this comment

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

I don't see any cleanup of PVC deletion tracking on completion.

@saffaalvi saffaalvi force-pushed the 18-destroy-old-notebooks branch from a3a4bee to 48ecf01 Compare October 2, 2020 21:28
@saffaalvi saffaalvi requested a review from brendangadd October 2, 2020 21:29
@saffaalvi saffaalvi force-pushed the 18-destroy-old-notebooks branch from aba9c16 to 49fc2b3 Compare October 2, 2020 21:35
Switch to a tristate PVC status so that the volume management table can
indicate whether a volume is mounted, unmounted, or being deleted.
@brendangadd brendangadd merged commit 2e9daf6 into master Oct 6, 2020
@brendangadd brendangadd deleted the 18-destroy-old-notebooks branch October 6, 2020 16:29
saffaalvi pushed a commit that referenced this pull request Feb 11, 2021
Support notebook volume (PVC) management
@wg102 wg102 mentioned this pull request Sep 12, 2022
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Procedure to destroy old notebook volumes
3 participants