Skip to content

feat: allow to toggle off v1 services#770

Merged
leafty merged 7 commits into
masterfrom
leafty/toggle-v1-services
Jul 18, 2025
Merged

feat: allow to toggle off v1 services#770
leafty merged 7 commits into
masterfrom
leafty/toggle-v1-services

Conversation

@leafty

@leafty leafty commented Jul 16, 2025

Copy link
Copy Markdown
Member

Details:

  • add revproxy.enableV1Services and login.enableV1Services configuration fields
  • when v1 services are not active:
    • do not serve /api, /api/direct, etc. -> GitLab
    • do not serve /api/kg, etc. -> KG, webhook
    • do not serve /ui-server/* routes related to v1 features
    • do not login though GitLab
    • do not logout from GitLab
  • also, remove routes to /api/search (service is not deployed anymore)

Corresponding renku PR -> SwissDataScienceCenter/renku#4094
Renku PR without GitLab -> SwissDataScienceCenter/renku#4096 (same as above but with v1 de-activated)

@leafty leafty requested a review from a team as a code owner July 16, 2025 13:27

@olevski olevski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks Flora this looks really nice.

@olevski olevski self-requested a review July 16, 2025 13:43
olevski
olevski previously approved these changes Jul 16, 2025
Comment thread internal/revproxy/main.go Outdated
Comment thread internal/revproxy/main.go Outdated
@olevski olevski self-requested a review July 16, 2025 13:52
@leafty

leafty commented Jul 16, 2025

Copy link
Copy Markdown
Member Author

Thanks @olevski , I would like to first update the v2 acceptance tests to work without GItLab first before merging here (so that we should be sure that everything works fine).

@leafty leafty merged commit f80b669 into master Jul 18, 2025
12 checks passed
@leafty leafty deleted the leafty/toggle-v1-services branch July 18, 2025 07:15
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