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

Initial PR : Create a public demonstration repository named cdk-eoapi #1

Merged
merged 11 commits into from
Jul 12, 2023

Conversation

emileten
Copy link
Contributor

@emileten emileten commented Jun 30, 2023

Ref issue developmentseed/eoapi-cdk#47

This PR creates a demonstration of how to use the constructs from https://github.com/developmentseed/cdk-pgstac. It is based on https://github.com/MAAP-Project/maap-cdk-pgstac itself mostly a copy of https://github.com/developmentseed/aws-asdi-pgstac, with the exception of a few bugs removed (back when it was created) and a tiler application that was added.

Most of the work done on top of just copy pasting https://github.com/MAAP-Project/maap-cdk-pgstac is :

  • increased configurability (more options are added to the top configuration model)
  • cleaned up the configuration management by using pydantic
  • cleaned up the configuration management by using a single configuration YAML file. In the two example repos above, one needs three files (a dotenv file, one yaml file for the bastion host user data and another file with a list of ip addresses).
  • converted the whole wrapper to python for accessibility.

@sharkinsspatial I did a basic testing of this PR (synth, deployment, and collection ingest work). I need to do more next week but a first round of reviews would be helpful.

Also, I am not addressing testing and documentation, I am leaving that to next PRs.

Thanks !

@emileten emileten changed the title Initial PR Initial PR : Create a public demonstration repository named cdk-eoapi Jun 30, 2023
@emileten emileten self-assigned this Jun 30, 2023
.gitignore Show resolved Hide resolved
cdk_eoapi/pgStacInfra.py Show resolved Hide resolved
cdk_eoapi/pgStacInfra.py Show resolved Hide resolved
cdk_eoapi/vpc.py Show resolved Hide resolved
cdk_eoapi/pgStacInfra.py Show resolved Hide resolved
config.py Show resolved Hide resolved
app.py Show resolved Hide resolved
cdk_eoapi/pgStacInfra.py Show resolved Hide resolved
config.py Show resolved Hide resolved
.gitignore Show resolved Hide resolved
… relationship to be manually established if the data access role is injected, and if not injected move the creation of the trust relationship policy to after the stac ingestor creation to be able to use the exact arn
@emileten
Copy link
Contributor Author

emileten commented Jul 7, 2023

note to myself. I would like to :

  • add the option to create the database from a snapshot. We don't have that and I think it's important (I currently need that for MAAP, and VEDA has that option)
  • have the database in a stack that's different from the applications, i.e separating stateful and stateless resources deployments in different stacks, because we are likelly to redeploy the apps much more often than the database. CSDA does that..

Thoughts @sharkinsspatial ?

@emileten emileten merged commit 0a32caf into main Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants