-
Notifications
You must be signed in to change notification settings - Fork 36
feat: prepare Legacy sessions phase out #4119
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
Conversation
|
You can access the deployment of this PR at https://ci-renku-4119.dev.renku.ch |
9887f72 to
1ccd765
Compare
Co-authored-by: Flora Thiebaut <flora.thiebaut@sdsc.ethz.ch>
leafty
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Flag is not working, see deployment
leafty
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😱 Thanks for spotting that! The fix is here SwissDataScienceCenter/renku-ui#3799 |
|
Tearing down the temporary RenkuLab deployment for this PR. |
Co-authored-by: Flora Thiebaut <flora.thiebaut@sdsc.ethz.ch>
* chore: create release 2.6.0 * feat: add support for git revision and context dir options to image builds (#4102) * fix: upgrade amalthea to 0.20.1 (#4123) Changes the order of the proxies so that sessions where the base URL path is stripped to "/" still works. * feat: upgrade data service to 0.53.0 (#4124) * chore: add buildpack base images (#4086) * chore: publish base images on tag events (#4125) * chore(deps): bump tmp (#4121) Bumps the npm_and_yarn group with 1 update in the /cypress-tests directory: [tmp](https://github.com/raszi/node-tmp). Updates `tmp` from 0.2.1 to 0.2.4 - [Changelog](https://github.com/raszi/node-tmp/blob/master/CHANGELOG.md) - [Commits](raszi/node-tmp@v0.2.1...v0.2.4) --- updated-dependencies: - dependency-name: tmp dependency-version: 0.2.4 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * feat: prepare Legacy sessions phase out (#4119) Co-authored-by: Flora Thiebaut <flora.thiebaut@sdsc.ethz.ch> * feat: disable legacy sessions by default --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Tasko Olevski <tasko.olevski@sdsc.ethz.ch> Co-authored-by: Rok Roškar <roskarr@ethz.ch> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Lorenzo Cavazzi <43481553+lorenzo-cavazzi@users.noreply.github.com> Co-authored-by: Flora Thiebaut <flora.thiebaut@sdsc.ethz.ch> Co-authored-by: Tasko Olevski <olevski90@gmail.com>

Add a new value
ui.client.supportLegacySessionsto turn off the Legacy sessions on the UI side -- see lorenzo/v1-sessions-warningsThis PR includes the latest UI version 4.6.0 with a bunch of other minor improvements and bugfixes
/deploy extra-values=ui.client.supportLegacySessions=false