Skip to content

Conversation

@olevski
Copy link
Member

@olevski olevski commented Jul 16, 2025

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

/deploy

@olevski olevski requested a review from a team as a code owner July 16, 2025 14:49
@olevski olevski temporarily deployed to ci-renku-4097 July 16, 2025 14:49 — with GitHub Actions Inactive
@RenkuBot
Copy link
Collaborator

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

@olevski olevski requested review from a team as code owners July 17, 2025 08:10
sgaist
sgaist previously requested changes Jul 21, 2025
Copy link
Collaborator

@sgaist sgaist left a comment

Choose a reason for hiding this comment

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

Beside the cleanup for the various image version and the suggestion from @leafty, the egress part LGTM.

@olevski olevski force-pushed the feat-allow-custom-session-netpol branch from 8362a79 to e407b86 Compare July 22, 2025 11:40
@olevski olevski temporarily deployed to ci-renku-4097 July 22, 2025 11:40 — with GitHub Actions Inactive
@olevski olevski requested review from leafty and sgaist July 22, 2025 11:43
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

The stuff we had hardcoded is not general enough to work on Openshift or
in cases where the Renku deployment is not exposed to the internet.
@olevski olevski dismissed sgaist’s stale review July 22, 2025 12:16

The changes have been addressed. It is holding up the mernging of the PR so I dismissed this.

@olevski olevski merged commit 8dbc9a8 into release-2.5.0 Jul 22, 2025
25 checks passed
@olevski olevski deleted the feat-allow-custom-session-netpol branch July 22, 2025 12:16
@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.

5 participants