-
Notifications
You must be signed in to change notification settings - Fork 67
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
[New Hub] MUST on Catalyst Project Africa cluster #3399
Conversation
Merging this PR will trigger the following deployment actions. Support and Staging deployments
Production deployments
|
97b6c0f
to
ba72899
Compare
So visiting https://must.af.catalystproject.2i2c.cloud is returning a 404 for me 😕 |
Okay, so it will be available when it's merged? I edited the PR comment. Also, do I need to change any other configurations in the load balancer? 🤔 |
No, if you've run If you have deployed locally, then we need to do more investigation as to why this is not working, but I cannot see any obvious issues from the files changed (though I am on my phone). |
Fantastic! No, I didn't deploy it locally. Now it's running! Edit: I just noticed that the logo has white letters, which are not visible with the white background. |
Nice! Before sharing this with the Community Champion, we need to provide a jupyter/scipy image instead of a rocker based image. Something like
should do it. |
Aah! I didn't see that. Done! |
After logging in, I get bounced to https://must.af.catalystproject.2i2c.cloud/user/[email protected]/rstudio? (which gives a 404) I think I should be redirect to https://must.af.catalystproject.2i2c.cloud/user/[email protected]/lab (which does work). I haven't yet looked at why this occurring. |
I think now we have a hub on this cluster using a different image, it makes sense to move the R/RStudio image definition from Probably what is happening is that not everything related to the image and setting up R envs is being fully overwritten by the Python image spec. |
@jmunroe We are picking up rstudio as the default URL from these lines in common.values.yaml
@AIDEA775 I opened AIDEA775#1 which you should be able to merge into this PR. Then if you redeploy, I think the problem James reported will be fixed. |
Okay, redeployed, and now it's working properly. Note: The default url is https://must.af.catalystproject.2i2c.cloud/user/[email protected]/tree no |
@jmunroe @sgibson91 Any update on this? Can we proceed? |
@AIDEA775 Have you commented on the related issue and invited the community champion to try it and provide feedback? |
This is a shared cluster and each hub may want different profile options, so let's move this config out of common - since it is no longer common!
Co-authored-by: Sarah Gibson <[email protected]>
dffee0c
to
f3bfd61
Compare
The community champion gave the okay. I rebased the branch to maintain a clean history :D |
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.
LGTM, merge when you like :)
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/7034483750 |
This will be deployed to must.af.catalystproject.2i2c.cloud