-
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
Move nasa-veda hub URL #3037
Move nasa-veda hub URL #3037
Conversation
Merging this PR will trigger the following deployment actions. Support and Staging deployments
Production deployments
|
Let's wait for explicit approval in #3029 (comment) before going through with this. |
@freitagb is going to use this as an opportunity to use a brand new domain. So let's hold off for a bit more. |
- from: nasa-veda.2i2c.cloud | ||
to: veda.2i2c.cloud | ||
- from: staging.nasa-veda.2i2c.cloud | ||
to: staging.veda.2i2c.cloud |
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.
to: staging.veda.2i2c.cloud | |
to: staging.veda.2i2c.cloud | |
- from: grafana.nasa-veda.2i2c.cloud | |
to: grafana.veda.2i2c.cloud |
Let's get grafana updated as well (and the associated callback URL for the oauth2 application)
I think there could be merge conflicts or config-conflicts caused by #3075, where I added a redirect from staging.nasa-veda.2i2c.cloud to staging.veda.2i2c.cloud. The reason I did that was because the hub was currently running at staging.veda.2i2c.cloud and it seemed like the most non-breaking change for now, prompted by having support request saying staging.nasa-veda.2i2c.cloud had issues (because it had moved). |
Ping @freitagb - what do we do with this one? |
Ping @freitagb - should we move forward on this? |
I've re-pinged @freitagb on slack now. |
yes, can we move forward with migrating to openveda.cloud? |
@yuvipanda can we use hub.openveda.cloud |
Can you put the following CNAMEs in place? hub.openveda.cloud -> nasa-veda.2i2c.cloud Let me know once done, I'll update the PR. |
@yuvipanda done! |
a989759
to
b136135
Compare
@slesaad great, I've updated the PR. Am waiting for the DNS change to propagate ( Thank you for the quick response. |
@slesaad do you know what the TTLs on those CNAME entries are? I still can't see them, even if I force |
@yuvipanda hmm.. 🤔 |
@slesaad hmm, I'm not sure. I guess I'll just wait a little more and see? |
@slesaad they still don't resolve for me - are they resolving for you? |
@yuvipanda they are not 🤔 i wonder why. let me delete and recreate them and see if that helps. I don't know what else to try 😅 |
@slesaad yeah i don't see it still. I'm not sure what the problem is. I still see it return an I also see that |
@yuvipanda we're putting in a ticket with AWS to see what's up, I'll let you know when we have more info |
should be working now @yuvipanda ! |
We keep redirects in place from the old URL, in case there are external links already present. The OAuth callback URL needs to be changed at https://github.com/organizations/2i2c-org/settings/applications/2096065 just before merging. Ref 2i2c-org#3029
Also remove repetitive comments
79c6c53
to
c53a556
Compare
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/7229399532 |
Thanks @slesaad! |
Follow-up to 2i2c-org#3037
We keep redirects in place from the old URL, in case there are external links already present.
The OAuth callback URL needs to be changed at
https://github.com/organizations/2i2c-org/settings/applications/2096065 just before merging.
Ref #3029