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.

Sign up for GitHub

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

Jump to bottom

Add initial config for Temple university #1664

Merged
merged 4 commits into from
Sep 8, 2022
Merged

Add initial config for Temple university #1664

merged 4 commits into from
Sep 8, 2022

Conversation

yuvipanda
Copy link
Member

Fixes #1648

@github-actions
Copy link

github-actions bot commented Aug 31, 2022 •
edited
Loading

Support and Staging deployments

Cloud Provider Cluster Name Upgrade Support? Reason for Support Redeploy Upgrade Staging? Reason for Staging Redeploy
gcp 2i2c No Yes Following prod hubs require redeploy: temple

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
gcp 2i2c temple Following helm chart values files were modified: temple.values.yaml, enc-temple.secret.values.yaml

@yuvipanda yuvipanda mentioned this pull request Aug 31, 2022
Closed
7 tasks
@yuvipanda yuvipanda requested a review from a team August 31, 2022 01:22
GeorgianaElena
GeorgianaElena approved these changes Aug 31, 2022
View reviewed changes
Copy link
Member

@GeorgianaElena GeorgianaElena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@GeorgianaElena
Copy link
Member

Hmm, for some reason I don't see myself as an admin on this hub. Wondering what's that about

@GeorgianaElena GeorgianaElena mentioned this pull request Aug 31, 2022
Closed
@GeorgianaElena
Copy link
Member

@yuvipanda, I opened #1671 since I think it might be a real, more general issue. But I couldn't find a fix to it today though :(

@yuvipanda
Copy link
Member Author

@GeorgianaElena I think the issue seems to have fixed itself somehow? I'm definitely not seeing it anymore :)

@yuvipanda yuvipanda merged commit 2829e7a into 2i2c-org:master Sep 8, 2022
@github-actions
Copy link

github-actions bot commented Sep 8, 2022

🎉🎉🎉🎉

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

@GeorgianaElena
Copy link
Member

Yes, I noticed it too. Damian has been added too on the other hubs that I've deleted him from, so somehow this resolved itself? I'm really confused about what happened 😅 Will close the specific issue too and hopefully it doesn't resurface again.

@sgibson91
Copy link
Member

I did merge a PR yesterday that forced a deploy across all hubs. While that explains why people who are missing are back, it doesn't explain why they were missing in the first place.

@GeorgianaElena
Copy link
Member

it doesn't explain why they were missing in the first place.

For the temple hub, it remains a mystery. But, I was the one deleting Damian from three of our hubs to see if he gets re-added with a new hub deploy and at the moment when I opened the issue above, a redeploy didn't add him back.

And somehow, now he was, and the issue resolved itself for temple too 🤷🏼 no idea what this was about though. I didn't find anything strange when I investigated this, so still a mystery

@damianavila
Copy link
Contributor

And somehow, now he was,

I am a mystery... 😜

@GeorgianaElena GeorgianaElena mentioned this pull request Dec 5, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GeorgianaElena GeorgianaElena GeorgianaElena approved these changes

Assignees

@yuvipanda yuvipanda

Labels
None yet
Projects
No open projects
Sprint Board
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[New Hub] Temple University Education Hub
4 participants
@yuvipanda @GeorgianaElena @sgibson91 @damianavila