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 S3 upload to machinery and remove sync-s3 repository #16

Closed
cedricve opened this Issue Nov 4, 2015 · 3 comments

Comments

2 participants
@cedricve
Copy link
Member

cedricve commented Nov 4, 2015

For performance and architectural reasons, the uploading of images to a S3 bucket should be managed by the machinery and not longer exists as a separate service (sync-s3).

We will need to use the AWS C++ library, and run a seperate thread in the machinery that will upload images to the S3 bucket.

Following features are needed

  • Upload image to a S3 bucket
  • When no connection is available, images needs to be synced at a later time, when the connection is back online.
  • Uploading images has a lower priority than the image processing itself, this means that if the machinery is detecting motion and saving images, the sync-thread should halt or release it resources to the image processing.
@nicolas-f

This comment has been minimized.

Copy link

nicolas-f commented Dec 27, 2015

If cloud is not configured the process should not be started.

@cedricve cedricve added the in progress label Jan 3, 2016

@cedricve cedricve self-assigned this Jan 3, 2016

@cedricve

This comment has been minimized.

Copy link
Member

cedricve commented Jan 28, 2016

Did some research on how to do it in C++: https://blog.cedric.ws/how-to-use-aws-with-your-c-application. I will try to implement this in the machinery asap.

@cedricve

This comment has been minimized.

Copy link
Member

cedricve commented Feb 24, 2016

Feature added to develop branch

@cedricve cedricve closed this Feb 24, 2016

@cedricve cedricve removed the in progress label Feb 24, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment