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

Move nasa-veda hub URL #3037

Merged
merged 6 commits into from
Dec 16, 2023
Merged

Move nasa-veda hub URL #3037

merged 6 commits into from
Dec 16, 2023

Conversation

yuvipanda
Copy link
Member

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

@github-actions
Copy link

github-actions bot commented Aug 25, 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-veda Yes Following helm chart values files were modified: support.values.yaml Yes Following helm chart values files were modified: staging.values.yaml

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
aws nasa-veda prod Following helm chart values files were modified: prod.values.yaml

@yuvipanda
Copy link
Member Author

Let's wait for explicit approval in #3029 (comment) before going through with this.

@yuvipanda
Copy link
Member Author

@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
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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)

@consideRatio consideRatio reopened this Sep 5, 2023
@consideRatio
Copy link
Contributor

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).

@yuvipanda
Copy link
Member Author

Ping @freitagb - what do we do with this one?

@consideRatio
Copy link
Contributor

Ping @freitagb - should we move forward on this?

@yuvipanda
Copy link
Member Author

I've re-pinged @freitagb on slack now.

@freitagb
Copy link
Contributor

yes, can we move forward with migrating to openveda.cloud?

@freitagb
Copy link
Contributor

@yuvipanda can we use hub.openveda.cloud

@yuvipanda
Copy link
Member Author

Can you put the following CNAMEs in place?

hub.openveda.cloud -> nasa-veda.2i2c.cloud
staging.hub.openveda.cloud -> staging.nasa-veda.2i2c.cloud

Let me know once done, I'll update the PR.

@slesaad
Copy link
Contributor

slesaad commented Nov 30, 2023

@yuvipanda done!

@yuvipanda
Copy link
Member Author

@slesaad great, I've updated the PR. Am waiting for the DNS change to propagate (dig doesn't see it yet), will deploy and merge after.

Thank you for the quick response.

@yuvipanda
Copy link
Member Author

@slesaad do you know what the TTLs on those CNAME entries are? I still can't see them, even if I force dig to look at the authoritative nameserver

@slesaad
Copy link
Contributor

slesaad commented Nov 30, 2023

@yuvipanda hmm.. 🤔
Screenshot 2023-11-30 at 3 53 54 PM
the ttl is 300 seconds.
anything else that i needed to do on my side?

@yuvipanda
Copy link
Member Author

@slesaad hmm, I'm not sure. I guess I'll just wait a little more and see?

@yuvipanda
Copy link
Member Author

@slesaad they still don't resolve for me - are they resolving for you?

@slesaad
Copy link
Contributor

slesaad commented Dec 4, 2023

@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 😅

@yuvipanda
Copy link
Member Author

@slesaad yeah i don't see it still. I'm not sure what the problem is. I still see it return an NXDOMAIN, which means the authoritative DNS server doesn't think it exists still. Maybe there's an extra 'rollout' step or something? I'm not sure.

I also see that openveda.cloud itself doesn't really resolve to anything, so maybe that's also part of the issue? Whoever is managing that entire domain may be able to help.

@slesaad
Copy link
Contributor

slesaad commented Dec 5, 2023

@yuvipanda we're putting in a ticket with AWS to see what's up, I'll let you know when we have more info

@slesaad
Copy link
Contributor

slesaad commented Dec 7, 2023

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
@yuvipanda yuvipanda merged commit 284761a into 2i2c-org:master Dec 16, 2023
6 checks passed
Copy link

🎉🎉🎉🎉

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

@yuvipanda
Copy link
Member Author

Thanks @slesaad!

yuvipanda added a commit to yuvipanda/pilot-hubs that referenced this pull request Dec 16, 2023
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.

4 participants