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: allow session launcher parameters to be reset #434

Open
wants to merge 11 commits into
base: release-amaltheas-migration
Choose a base branch
from

Conversation

olevski
Copy link
Member

@olevski olevski commented Oct 1, 2024

Allows the API to accept None as input for args, command and the session launcher resource class ID so that they can be reset to their defaults in patch endpoints.

olevski and others added 6 commits September 25, 2024 17:16
This is a breaking change in the API. It requires changes in the UI for it to be fully useful.
Co-authored-by: Samuel Gaist <samuel.gaist@idiap.ch>

squashme: resolve package version conflicts
This work brings the notebook related endpoints to a working state
to serve as replacement for the renku-notebooks external service.

Short summary:
- API tests have been added
- Code has been fixed to answer / handle exception
- Automated test cluster creation has been added using k3d

---------

Co-authored-by: Tasko Olevski <tasko.olevski@sdsc.ethz.ch>
Co-authored-by: Samuel Gaist <samuel.gaist@idiap.ch>
@olevski olevski requested a review from a team as a code owner October 1, 2024 22:36
@olevski
Copy link
Member Author

olevski commented Oct 1, 2024

This change is part of the following stack:

Change managed by git-spice.

@olevski olevski force-pushed the reset-session-launcher-params branch 2 times, most recently from 1a841b3 to 659480c Compare October 1, 2024 22:39
olevski and others added 5 commits October 2, 2024 00:42
squashme: use the amalthea session cache
Allows the API to accept None as input for args, command and the session
launcher resource class ID so that they can be reset to their defaults
in patch endpoints.
@olevski olevski force-pushed the reset-session-launcher-params branch from 659480c to 4ccfc03 Compare October 1, 2024 22:44
@olevski olevski force-pushed the release-amaltheas-migration branch from 7fcfa17 to be5272b Compare October 2, 2024 13:09
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.

2 participants