-
Notifications
You must be signed in to change notification settings - Fork 7
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: modify for custom CA certificates #788
Conversation
TODO: |
This is ready for review. |
You can access the deployment of this PR at https://renku-ci-nb-788.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.
Looks good, let's merge this!
I will merge this even though the integration tests are failing. The integration tests are failing due to connection issues with gitlab it seems rather than real problems. |
Adds the ability to specify and inject any CA certificate in the notebook service. This is needed when the notebook service has to communicate with other services that are using self-signed certificates.
In addition to the changes required for injecting the certificates I also pulled out all the json patches into a separate directory and separated them into functions. This was required because the init container that injects the certificates for Amalthea has to be done in the correct order. I.e. if the git-clone init container runs first for a session then the certificates required will not be present so that the repository can be clones. Because of this and to keep the code manageable it was better to separate all the patches from the definition of the server class.
closes SwissDataScienceCenter/renku#2323
/deploy