-
Notifications
You must be signed in to change notification settings - Fork 29
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
feat(service): restore optimized migration check #2854
Conversation
- handle the GitLab library specific errors. - fall back to full repository checkout in case of unforseen errors. fix #2546
You can access the deployment of this PR at https://renku-ci-rp-2854.dev.renku.ch |
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.
Thank you, looks a lot better now 🙂 I just had one really nitpicky comment
use `BaseException` instead of `Exception` to catch all the remaining exceptions Co-authored-by: Ralf Grubenmann <ralf.grubenmann@sdsc.ethz.ch>
I added a test with a public repo, to be sure we don't break that for the optimized migrations checks (although that |
This PR restores the optimized migration check for the service.
The problem was mainly caused by invalid GitLab tokens that trigger errors when passed to the
gitlab
python library -- quite unexpected considering the different outcomes when invoking similar GitLab APIs with the same expired/invalid tokens.In short, I added logic to:
gitlab
library common errors./deploy
fix #2546