Skip to content
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

Streamline deployment of GESIS acceptance server #3178

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

rgaiacs
Copy link
Collaborator

@rgaiacs rgaiacs commented Jan 21, 2025

This is related with #2797 and #3087.

SSH access to GESIS Kubernetes cluster requires access to GESIS VPN. This makes the use of GitHub actions challenge.

Different than #3090, this PR assumes that the GESIS Kubernetes is configured. GESIS Kubernetes runs on bare-metal and is configured following MetalLB + Ingress NGINX Controller.

GESIS operates a on-premise GitLab. This PR addes a GitLab CI/CD configuration to deploy the mybinder.org Helm chart to the GESIS Kubernetes using GitLab agent for Kubernetes.

After #3159, any the commit on main is pushed to GESIS GitLab and will trigger a new deployment of this mybinder.org Helm chart.

At the moment, the Helm chart deploy fails with

Error: Unable to continue with install: IngressClass "nginx" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal "interactive-environment": current value is "ingress-nginx"; annotation validation error: key "meta.helm.sh/release-namespace" must equal "interactive-environment": current value is "ingress-nginx"

My understanding of the error is that GESIS Kubernetes already has a Ingress Class nginx that was configured for MetalLB + Ingress NGINX Controller but the Binder Hub and Jupyter Hub upstream Helm chart tries to create the same Ingress Class nginx and this operation is blocked. Is my understanding correct? @manics @yuvipanda @minrk

@rgaiacs rgaiacs self-assigned this Jan 21, 2025
@rgaiacs rgaiacs requested review from minrk and manics January 22, 2025 13:00
@rgaiacs
Copy link
Collaborator Author

rgaiacs commented Jan 22, 2025

This is ready for review. If it looks good, please merge.

The mybinder's Helm chart was changed to allow the static server to be disabled.

@arnim this does not include the custom GESIS page. This will go in another PR.

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.

1 participant