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

Setup NASA ESDIS cluster and hub #3245

Merged
merged 17 commits into from
Dec 5, 2023
Merged

Conversation

sgibson91
Copy link
Member

ref: #3068

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

Merging this PR will trigger the following deployment actions.

Support and Staging deployments

Cloud Provider Cluster Name Upgrade Support? Reason for Support Redeploy Upgrade Staging? Reason for Staging Redeploy
aws nasa-esdis Yes Following helm chart values files were modified: enc-support.secret.values.yaml, support.values.yaml Yes Following helm chart values files were modified: common.values.yaml, staging.values.yaml, enc-staging.secret.values.yaml

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
aws nasa-esdis prod Following helm chart values files were modified: common.values.yaml, enc-prod.secret.values.yaml, prod.values.yaml

@sgibson91 sgibson91 force-pushed the nasa-esdis-hub branch 3 times, most recently from 44ae4f7 to 3c40662 Compare October 30, 2023 09:03
@sgibson91 sgibson91 marked this pull request as ready for review November 29, 2023 11:11
@sgibson91 sgibson91 requested a review from a team as a code owner November 29, 2023 11:11
Copy link
Member

@yuvipanda yuvipanda left a 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?

@sgibson91
Copy link
Member Author

@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

@sgibson91 sgibson91 merged commit 30826a5 into 2i2c-org:master Dec 5, 2023
7 checks passed
@sgibson91 sgibson91 deleted the nasa-esdis-hub branch December 5, 2023 12:06
Copy link

github-actions bot commented Dec 5, 2023

🎉🎉🎉🎉

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done 🎉
Development

Successfully merging this pull request may close these issues.

2 participants