Skip to content

Conversation

@olevski
Copy link
Member

@olevski olevski commented Jul 16, 2025

/deploy

The notebook service has not been used in a long while. This fully removes parts of it that are not needed (regardless of whether we are using Renku legacy or v2).

@RenkuBot
Copy link
Collaborator

You can access the deployment of this PR at https://ci-renku-4095.dev.renku.ch

@olevski olevski force-pushed the feat-remove-notebook-svc branch from 653f680 to 7385e07 Compare July 16, 2025 13:50
@olevski olevski temporarily deployed to ci-renku-4095 July 16, 2025 13:50 — with GitHub Actions Inactive
@olevski olevski requested review from a team as code owners July 17, 2025 08:10
@olevski olevski force-pushed the feat-remove-notebook-svc branch from 7385e07 to 89b208b Compare July 22, 2025 11:44
@olevski olevski temporarily deployed to ci-renku-4095 July 22, 2025 11:44 — with GitHub Actions Inactive
Copy link
Member

@leafty leafty left a comment

Choose a reason for hiding this comment

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

LGTM

@olevski olevski force-pushed the feat-remove-notebook-svc branch from 89b208b to e41e109 Compare July 22, 2025 11:55
@olevski olevski enabled auto-merge (squash) July 22, 2025 11:56
@olevski olevski merged commit af436de into release-2.5.0 Jul 22, 2025
23 of 24 checks passed
@olevski olevski deleted the feat-remove-notebook-svc branch July 22, 2025 11:56
@RenkuBot
Copy link
Collaborator

Tearing down the temporary RenkuLab deployment for this PR.

olevski added a commit that referenced this pull request Jul 23, 2025
* chore: create release 2.5.0

* feat: upgrade amalthea to 0.20.0 (#4090)

* feat: make swagger-ui image configurable (#4089)

This allows to customize swagger in the same way
as other components. Useful for example when
deploying on OpenShift where the default image
cannot run due to security constraints.

* chore(deps): bump brace-expansion (#4048)

Bumps the npm_and_yarn group with 1 update in the /cypress-tests directory: [brace-expansion](https://github.com/juliangruber/brace-expansion).


Updates `brace-expansion` from 1.1.11 to 1.1.12
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@1.1.11...v1.1.12)

---
updated-dependencies:
- dependency-name: brace-expansion
  dependency-version: 1.1.12
  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>
Co-authored-by: Lorenzo Cavazzi <43481553+lorenzo-cavazzi@users.noreply.github.com>

* chore: swap swagger services order to show data service first (#4093)

* feat: update renku-ui to 4.4.0 (#4100)

* chore: bump renku-ui to 4.4.0
* test: update group acceptance test

* feat: update renku-gateway to 1.6.0 (#4094)

The gateway can now run without GitLab when `enableV1Services` is set to `false`.

The cypress acceptance tests are updated to support correctly running when `enableV1Services=false`.

* chore(deps): bump brace-expansion (#4101)

Bumps the npm_and_yarn group with 1 update in the /cypress-tests directory: [brace-expansion](https://github.com/juliangruber/brace-expansion).


Updates `brace-expansion` from 1.1.11 to 1.1.12
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@1.1.11...v1.1.12)

---
updated-dependencies:
- dependency-name: brace-expansion
  dependency-version: 1.1.12
  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: remove unused notebook service (#4095)

* feat: allow custom session egress network policies (#4097)

The policies we had hardcoded is not general enough to work on Openshift or
in cases where the Renku deployment is not exposed to the internet.

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Tasko Olevski <tasko.olevski@sdsc.ethz.ch>
Co-authored-by: Samuel Gaist <samuel.gaist@idiap.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>
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.

4 participants