Skip to content

Supporting Azure/storage and updating env vars for replication files #230

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

Merged
merged 8 commits into from
Feb 21, 2022

Conversation

Rub21
Copy link
Collaborator

@Rub21 Rub21 commented Feb 18, 2022

What I am changing

  • Adding Azure cli support
  • Updating REPLICATION_FOLDER path

How I did it

  • Installing Azure cli in osm-processor
  • Adding Code for download/upload files to azure storage
  • Updating values in Values.yaml and templates
  • Azure support for planer-dump, full-history and db-backup

How you can test it

Fro testing, we need to set up credentials in .env.cloudprovider from https://github.com/developmentseed/osm-seed/blob/azure/storage/envs/.env.cloudprovider.example
and then run

docker compose -f compose/replication.yml build
docker compose -f compose/replication.yml up

docker compose -f compose/planet-dump.yml build
docker compose -f compose/planet-dump.yml up

docker compose -f compose/full-history.yml build
docker compose -f compose/full-history.yml up

docker compose -f compose/db-backup-restore.yml build
docker compose -f compose/db-backup-restore.yml up

Related Issues

#228

cc. @geohacker @batpad

Copy link
Member

@geohacker geohacker left a comment

Choose a reason for hiding this comment

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

@Rub21 this looks great to me, I think we'll test along with a full setup on AKS. I left one note about the osm-processor version change in the Dockerfiles.

@Rub21
Copy link
Collaborator Author

Rub21 commented Feb 21, 2022

@geohacker @batpad , the changes here are working good:

  • Tested on Azure - storage
  • Tested in AWS -S3
  • Tested in GCP kubernetes - Cloud storage

@Rub21 Rub21 merged commit ca73477 into develop Feb 21, 2022
@Rub21
Copy link
Collaborator Author

Rub21 commented Feb 21, 2022

Testing contianers locally:

docker compose -f compose/replication.yml build &
docker compose -f compose/planet-dump.yml build &
docker compose -f compose/full-history.yml build &
docker compose -f compose/db-backup-restore.yml build


docker compose -f compose/replication.yml up &
docker compose -f compose/planet-dump.yml up &
docker compose -f compose/full-history.yml up &
docker compose -f compose/db-backup-restore.yml up


docker compose -f compose/replication.yml down &
docker compose -f compose/planet-dump.yml down &
docker compose -f compose/full-history.yml down &
docker compose -f compose/db-backup-restore.yml down

@geohacker geohacker deleted the azure/storage branch February 22, 2022 06:00
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.

2 participants