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

Deploy dev veda backend with openveda.cloud #291

Open
1 task done
smohiudd opened this issue Feb 5, 2024 · 5 comments
Open
1 task done

Deploy dev veda backend with openveda.cloud #291

smohiudd opened this issue Feb 5, 2024 · 5 comments
Assignees
@botanical

Comments

@smohiudd
Copy link
Contributor

smohiudd commented Feb 5, 2024 •
edited
Loading

What

Before taking on this Cloudfront ticket, should test out using the new openveda.cloud domain on a the dev instance. This will help formulate a pattern that can be used for the production deployment.

As part of this ticket, explore (possibly implement) the option to have no subdomain in veda-backend routes construct. Currently this configuration is not supported. Ultimately this config option will support the production instance which may have a urls pattern like this:

openveda.cloud = production browser
openveda.cloud/api/stac
openveda.cloud/api/raster
openveda.cloud/api/ingest

AC

  • dev instance is deployed with openveda.cloud domain (i.e. dev.openveda.cloud)
@botanical botanical self-assigned this Feb 8, 2024
@botanical
Copy link
Member

I created a certificate for the new domain openveda.cloud and tested that I configured correctly in dev with an updated deployment but I noticed the base url webpage throws an error. I discovered that the same error shows on https://dev.delta-backend.com/ so I'm working on debugging it.

Screenshot 2024-02-09 at 5 08 47 PM

@botanical
Copy link
Member

botanical commented Feb 14, 2024 •
edited
Loading

For anyone following along the issue, the following links are available:
https://dev.openveda.cloud/
https://dev.openveda.cloud/api/raster/docs
https://dev.openveda.cloud/api/stac/docs
https://dev.openveda.cloud/api/ingest/docs

I also will update the dev secret to contain the new values for the following variables, pending some confirmation that that's okay to do:

  • VEDA_DOMAIN_HOSTED_ZONE_ID
  • VEDA_DOMAIN_HOSTED_ZONE_NAME
  • VEDA_CERT_ARN
  • VEDA_CUSTOM_HOST

I'm currently working on migrating to Stadia maps for https://dev.openveda.cloud/api/stac/index.html to be functional

@botanical botanical mentioned this issue Feb 14, 2024
Merged
@botanical
Copy link
Member

For anyone following along the issue, the following links are available: https://dev.openveda.cloud/ https://dev.openveda.cloud/api/raster/docs https://dev.openveda.cloud/api/stac/docs https://dev.openveda.cloud/api/ingest/docs

I also will update the dev secret to contain the new values for the following variables, pending some confirmation that that's okay to do:

  • VEDA_DOMAIN_HOSTED_ZONE_ID
  • VEDA_DOMAIN_HOSTED_ZONE_NAME
  • VEDA_CERT_ARN
  • VEDA_CUSTOM_HOST

I'm currently working on migrating to Stadia maps for https://dev.openveda.cloud/api/stac/index.html to be functional

I updated my comment and reverted the changes to the dev secret until I get confirmation from the team that it's okay to update it 😅

botanical added a commit that referenced this issue Feb 15, 2024
@botanical
fix: migrate stamen map references to stadia maps (#301)
385e611 
## Description
- This PR migrates Stamen Map references to use Stadia Maps (full
migration notes
[here](https://docs.stadiamaps.com/guides/migrating-from-stamen-map-tiles/))
- The migration is setup with my DevSeed email and to work with
`*.delta-backend.com` domain

## Related Issues
#291

## Changes Made
- updates `stac-viewer.html` references from stamen maps to stadia maps

## Testing Done
[- Page now shows
map](https://dev.delta-backend.com/api/stac/index.html)
@botanical
Copy link
Member

The PR attached to this issue has been merged 🎉 Thanks for the review, @smohiudd!

Once we can successfully get dev deployed again, I will udpate the Stadia Maps account with the new domain (openveda.cloud)

@botanical botanical mentioned this issue Feb 17, 2024
Merged
@botanical
Copy link
Member

Writing here to note that I updated the Stadia Map account to use the new domain (openveda.cloud).

I also opened a PR to address an issue I found in the STAC viewer #311.

botanical added a commit that referenced this issue Feb 23, 2024
@botanical
fix: update stac viewer endpoint value (#311)
c672098 
### Issue
#291

### What?

- Updated the endpoint value for the STAC viewer in order to properly
render the collections on the webpage.
- Added a PR template so that all new PRs will have a template for
reference

### Why?

- The endpoint is https://dev.openveda.cloud/collections which is
incorrect which causes the collections filter to fail in the STAC
viewer.


https://github.com/NASA-IMPACT/veda-backend/assets/12633533/b272fb4c-23a9-41c2-8ce8-925c6760231a


### Testing?

- I didn't locally deploy or anything but here is [the working
collections link](https://dev.openveda.cloud/api/stac/collections)

### Notes
- I think we could technically also update
https://github.com/NASA-IMPACT/veda-backend/blob/develop/stac_api/runtime/src/app.py#L79
to be:
```{"request": request, "endpoint": str(request.url).replace("/index.html", "/api/stac")},``` if that's preferred
@botanical botanical mentioned this issue Feb 23, 2024
Merged
botanical added a commit that referenced this issue Feb 26, 2024
@botanical
fix: migrate stamen map references to stadia maps raster api (#321)
3cc699a 
### Issue
#291

### What?

- Update Stamen Maps to Stadia Maps 
### Why?

- Addressed this changed in a [previous
PR](#301) but didn't
realize that I needed to update the raster api as well

### Testing?

- Local testing on vscode without deploying to dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@botanical botanical

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@botanical @smohiudd