dash-image-processing icon indicating copy to clipboard operation
dash-image-processing copied to clipboard

Migration to DDS

Open xhluca opened this issue 6 years ago • 4 comments

This is an issue discussing how to migrate this app to DDS.

At the moment, this uses 2 Heroku features:

  • S3 Bucketeer: A heroku add-on that manages a AWS S3 Storage Bucket for a heroku app.
  • Heroku Redis: Heroku's redis platform.

At the moment the S3 storage is directly created by heroku. In order to migrate to DDS, we will need to create a bucket (using plotly's credentials), and update the credentials of the app using that bucket. We then will need to create a DDS deployment and setup Redis.

xhluca avatar Jan 18 '19 19:01 xhluca

@nicolaskruchten @cldougl I'll work on this one as soon as I finish the docs for cytoscape, unless anyone would like to pick it up as a practice, e.g. peter).

xhluca avatar Jan 18 '19 19:01 xhluca

Hi, I cloned the repo and the app does not run as is. I got the following error.

Invalid type for parameter Bucket, value: None, type: <class 'NoneType'>, valid types: <class 'str'>

Could you please provide some guidance (or point to soem resource) on how to set up S3 Bucketeer and Heroku Redis? Thank you very much!

lmpan avatar Jan 26 '19 07:01 lmpan

Hi @lmpan, I believe Heroku has some tutorial in setting up Bucketeer and Redis. However, this demo will be migrated soon to DDS, our in-house platform for hosting Dash apps, since the current Heroku setup is rather complicated and beginner unfriendly. With DDS, the process will become a lot simpler, since Redis is integrated by default.

xhluca avatar Jan 26 '19 16:01 xhluca

Updates about this issue:

https://github.com/plotly/dash-image-processing/pull/7 allows using interoperable key from GCP bucket with boto3 python client, so both s3 and gcp bucket are supported for now with only two line of edits(https://github.com/plotly/dash-image-processing/blob/b1a808984285478daa5f7b58e652d4c6b83a20d7/app.py#L56 and https://github.com/plotly/dash-image-processing/blob/b1a808984285478daa5f7b58e652d4c6b83a20d7/app.py#L311) , and switch of .env credentials. Changes will be added to README.

ycaokris avatar May 01 '19 02:05 ycaokris