Skip to content

Conversation

@lorenzo-cavazzi
Copy link
Member

@lorenzo-cavazzi lorenzo-cavazzi commented Jul 15, 2025

By default, we still load the renku-python "core" service as default when opening Swagger, although we mostly use the data services now.
This change makes Data Service the default selected service.

P.S. We might want to remove some entries from the list -- probably Notebooks and Search?

image

/deploy

@RenkuBot
Copy link
Collaborator

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

@olevski
Copy link
Member

olevski commented Jul 16, 2025

P.S. We might want to remove some entries from the list -- probably Notebooks and Search?

I will remember to add a followup PR that when the enableV1Services is set to false then the swagger page should not even show anything else other than the data service.

@lorenzo-cavazzi lorenzo-cavazzi merged commit a7ba6e4 into release-2.5.0 Jul 16, 2025
27 of 34 checks passed
@lorenzo-cavazzi lorenzo-cavazzi deleted the lorenzo/swagger-data-service-default branch July 16, 2025 08:54
@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