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

Move maap staging hub to use keycloak #5809

Merged
merged 5 commits into from
Apr 1, 2025
Merged

Conversation

yuvipanda
Copy link
Member

Ref #5754

@yuvipanda
Copy link
Member Author

Currently failing with

image

Copy link

github-actions bot commented Mar 27, 2025

Merging this PR will trigger the following deployment actions.

Support deployments

No support upgrades will be triggered

Staging deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
aws maap staging Following helm chart values files were modified: enc-staging.secret.values.yaml, staging.values.yaml

Production deployments

No production hub upgrades will be triggered

yuvipanda added a commit to yuvipanda/pilot-hubs that referenced this pull request Mar 27, 2025
While working on 2i2c-org#5809,
I finally got frustrated enough with our usage of `--wait` that I've
ripped it out.

The major issue with `--wait` is that if the deployment fails (because
of a typo in config for example), you *must* wait for the 5min timeout
or helm can get into an inconsistent state. This sucks when prototyping.

I stole the code we use in the mybinder.org deployment repo
instead (https://github.com/jupyterhub/mybinder.org-deploy/blob/main/deploy.py#L273).

It also seems faster, but the most important capability here is
that it's cancellable.
This must come directly from KeyCloak
@yuvipanda
Copy link
Member Author

All sorted out! There was an extra trailing slash in the config that was set up in veda-auth's CI system, and removing it got it to work

@yuvipanda yuvipanda merged commit 7b10746 into 2i2c-org:main Apr 1, 2025
11 checks passed
Copy link

github-actions bot commented Apr 1, 2025

🎉🎉🎉🎉

Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/14184031316

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