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

Add upload speed limit for cloud recording #401

Draft
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

jflevesque-genetec
Copy link
Contributor

This PR introduces a new upload limit for cloud recording, to help prevent network spikes when a device uploads a segment to the object storage. Without this type of limit, having numerous devices uploading their segments at the same time can greatly degrade the network quality of a customer.

<title>Uploading to an Object Storage</title>
<para>
A device signaling support for recording to an external target shall limit the upload speed so that a segment takes at least half as long as its duration to upload.
If the upload fails on the first try, the device shall retry uploading to the external target as fast as possible, to prevent loss of recording.
Copy link
Contributor

Choose a reason for hiding this comment

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

do we have ONVIF API to control the segment upload speed against segment size & duration? Or this PR is about only recording control doc update.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At the moment, only update to the documentation. Some other use cases were raised during the F2F and the proposal will have to be reviewed and adapted.

Copy link
Contributor

Choose a reason for hiding this comment

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

@ jflevesque-genetec , thanks for clarifying it.

Copy link
Contributor

Choose a reason for hiding this comment

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

@jflevesque-genetec Is there any further update to this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, we still need to make some updates to it, but we haven't had the time.

Copy link
Contributor

Choose a reason for hiding this comment

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

@jflevesque-genetec thanks for the response I am sure it will be discussed and reviewed at the F2F.

@jflevesque-genetec jflevesque-genetec marked this pull request as draft July 16, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants