-
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 hubs] Dandi #3879
[New hubs] Dandi #3879
Conversation
Merging this PR will trigger the following deployment actions. Support and Staging deployments
Production deployments
|
I don't know what's wrong, but I can't get the autoscaler to trigger when starting a new server. It's just stuck in:
I even recreated the cluster, thinking I missed something during the first phases :| |
@GeorgianaElena i can take a look at this today. |
On AWS, we unfortunately have to manually install and manage the cluster autoscaler. I looked to see if it was crashing, with
I have opened #3881 to automate this. Otherwise, I think if you enable the cluster autoscaler in the support chart and redeploy it, that should work! And sorry for not catching it when I was reviewing #3866 |
4c15dab
to
38b5aa1
Compare
Thank you @yuvipanda. I think this bit was recently removed from docs probably under the assumption that was already automated. Will work on fixing #3881. Thank you 🙏🏼 |
I believe this PR is now ready to be reviewed/deployed 🚀 |
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.
Two changes but otherwise looks good to me!
Co-authored-by: Yuvi Panda <[email protected]>
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.
Good to go!
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/8527749373 |
Looks like I missed spotting the equivalent of #3895 here too! |
Closes #3824
Also, updates the eksctl jsonnet file to not have dask nodes (I assumed there will be daskhubs on setup) and the terraform to use the requested buckets name (staging and prod were assumed by the template).