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

fix(byon): Use notebook-image label for visibility toggle #169

Merged
merged 1 commit into from
Apr 6, 2022

Conversation

tumido
Copy link
Member

@tumido tumido commented Apr 5, 2022

According to opendatahub-io-contrib/jupyterhub-singleuser-profiles#217 we'll use the opendatahub.io/notebook-image label as the Spawner visibility toggle directly so no Spawner changes are needed. Instead we need to adapt our pipelines and dashboard backend.

@tumido
Copy link
Member Author

tumido commented Apr 5, 2022

/cc @dlabaj

Copy link
Contributor

@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

Copy link

@oindrillac oindrillac left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@dlabaj dlabaj merged commit 8b060f9 into opendatahub-io:BYON Apr 6, 2022
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 8, 2022
Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 8, 2022
Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 8, 2022
Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 8, 2022
Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
Update backend port setting and makefile dev commands. (opendatahub-io#133)
Use webpack dev server proxy instead of CORS utilities (opendatahub-io#127)


Fixed issues with cherry picks.
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 8, 2022
Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
Update backend port setting and makefile dev commands. (opendatahub-io#133)
Use webpack dev server proxy instead of CORS utilities (opendatahub-io#127)


Fixed issues with cherry picks.
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 13, 2022
Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
Update backend port setting and makefile dev commands. (opendatahub-io#133)
Use webpack dev server proxy instead of CORS utilities (opendatahub-io#127)


Fixed issues with cherry picks.

Fixed additional git cherry pick issues.

Added ability to add packages, modify packages, etc.

Added software package on import.

Added additional table and fix some styling issues.


Fixed issue where form was submitted.

Updated import in notebooksUtils.ts

Updated code to work with image streams instead of pipelines.
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 21, 2022
[UPSTREAM] Fix to allow a user to enter an empty string.
Fixed linting error.

Updated to read userTrackingEnabled instead of clusterSettings.userTrackingEnabled.

[Upstream] Added BYON, webpack proxy, and CORS heading fix.

Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
Update backend port setting and makefile dev commands. (opendatahub-io#133)
Use webpack dev server proxy instead of CORS utilities (opendatahub-io#127)


Fixed issues with cherry picks.

Fixed additional git cherry pick issues.

Added ability to add packages, modify packages, etc.

Added software package on import.

Added additional table and fix some styling issues.


Fixed issue where form was submitted.

Updated import in notebooksUtils.ts

Updated code to work with image streams instead of pipelines.

Fixed linting errors.

Fixed empty software package bug.

Fixed bugs found during testing.

Notebook image table update.

Fixed error where not showing the empty state.

Fixed table issue.

Add another update of the notbook image table.
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 21, 2022
[UPSTREAM] Fix to allow a user to enter an empty string.
Fixed linting error.

Updated to read userTrackingEnabled instead of clusterSettings.userTrackingEnabled.

[Upstream] Added BYON, webpack proxy, and CORS heading fix.

Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
Update backend port setting and makefile dev commands. (opendatahub-io#133)
Use webpack dev server proxy instead of CORS utilities (opendatahub-io#127)


Fixed issues with cherry picks.

Fixed additional git cherry pick issues.

Added ability to add packages, modify packages, etc.

Added software package on import.

Added additional table and fix some styling issues.


Fixed issue where form was submitted.

Updated import in notebooksUtils.ts

Updated code to work with image streams instead of pipelines.

Fixed linting errors.

Fixed empty software package bug.

Fixed bugs found during testing.

Notebook image table update.

Fixed error where not showing the empty state.

Fixed table issue.

Add another update of the notbook image table.

Fixed issue where notebook would description disappeared.

Fixed missing notification.

Prevent duplicate names.

Fixed issue with notebook images not updating correctly.
dlabaj added a commit to dlabaj/odh-dashboard that referenced this pull request Apr 22, 2022
[UPSTREAM] Fix to allow a user to enter an empty string.
Fixed linting error.

Updated to read userTrackingEnabled instead of clusterSettings.userTrackingEnabled.

[Upstream] Added BYON, webpack proxy, and CORS heading fix.

Updated with REST api notebook image endpoints and new notebook image types.
Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>
Updated type.ts in front end with Succeeded
[byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>
Rebased BYON from master (opendatahub-io#162)

* Updated way that to detect an admin. (opendatahub-io#137)

Fixed linting error.

* Update backend port setting and makefile dev commands. (opendatahub-io#133)

* Update README.md (opendatahub-io#96)

Updated Readme to specify that a build is required prior to running the development server.

* Fix for input not allowing blank spaces.:

Fix to allow a user to enter an empty string.

Fixed with review comment.

* Add culler timeout settings feature (opendatahub-io#134)

* Added types for notebookimage and status (opendatahub-io#147)

Updated with REST api notebook image endpoints and new notebook image types.

* Updated REST API  (opendatahub-io#155)

* Updated with REST API design changes.

Fixed formatting error.

* Update backend/src/types.ts

Co-authored-by: Tom Coufal <[email protected]>

Co-authored-by: Tom Coufal <[email protected]>

* Updated type.ts in front end with Succeeded

* [byon] Implement backend (opendatahub-io#156)

* feat(byon): List all notebooks

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Get single notebook

Signed-off-by: Tomas Coufal <[email protected]>

* fix(byon): Update api spec to include software and id

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Schedule new notebook import

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Delete notebook

Signed-off-by: Tomas Coufal <[email protected]>

* feat(byon): Update notebook

Signed-off-by: Tomas Coufal <[email protected]>

Co-authored-by: Christopher Chase <[email protected]>
Co-authored-by: Chad Roberts <[email protected]>
Co-authored-by: Juntao Wang <[email protected]>
Co-authored-by: Tom Coufal <[email protected]>
fix(byon): Use notebook-image label for visibility toggle (opendatahub-io#169)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Feat add creator annotation (opendatahub-io#170)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow to patch name and description on notebook (opendatahub-io#172)

Signed-off-by: Tomas Coufal <[email protected]>
feat(byon): Allow python dependency visibility field changes (opendatahub-io#173)

Signed-off-by: Tomas Coufal <[email protected]>
Added code for importing byon image. (opendatahub-io#171)

Updated front end code with import and start of table.
Updated with byon changes

Updated code with byon ui changes.
Integrated import, delete, and get functionality for notebooks.

Added start of edit panel.

Updated build error fixes.

Updates to edit dialog and linting cleanup.

Added sort to table.

Added filtering.

Added failure warning to allow user to click

Add check disable enabling failed notebook images

Fixed with review comments.

Fixed time bug.

Removed admin
Update backend port setting and makefile dev commands. (opendatahub-io#133)
Use webpack dev server proxy instead of CORS utilities (opendatahub-io#127)


Fixed issues with cherry picks.

Fixed additional git cherry pick issues.

Added ability to add packages, modify packages, etc.

Added software package on import.

Added additional table and fix some styling issues.


Fixed issue where form was submitted.

Updated import in notebooksUtils.ts

Updated code to work with image streams instead of pipelines.

Fixed linting errors.

Fixed empty software package bug.

Fixed bugs found during testing.

Notebook image table update.

Fixed error where not showing the empty state.

Fixed table issue.

Add another update of the notbook image table.

Fixed issue where notebook would description disappeared.

Fixed missing notification.

Prevent duplicate names.

Fixed issue with notebook images not updating correctly.
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.

3 participants