-
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
Setup NASA ESDIS cluster and hub #3245
Conversation
Merging this PR will trigger the following deployment actions. Support and Staging deployments
Production deployments
|
44ae4f7
to
3c40662
Compare
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Requested in 2i2c-org#3068 (comment)
88adeb5
to
dda1705
Compare
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.
This looks good to me. Thanks for working this through, @sgibson91.
I appreciate that the domain is not called nasa-esdis but just esdis - yay (#3029). I think in the future, we should also not use the word 'nasa' in the name of the cluster itself, so the name matches the domain. But I think it's probably too much hassle to rename and redeploy this one, so I think it's fine to leave this as is. But I'd love for us to put this someplace as a policy so we don't use the 'nasa' prefix next time. Do you think you can help with that as a followup, @sgibson91?
@yuvipanda I wrote up #3500 so @damianavila can track it, but I don't think I'll be committing to picking up extra ad hoc work this close to the end of year break, instead focussing on wrapping up what's already on my plate |
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/7100465094 |
ref: #3068