specs icon indicating copy to clipboard operation
specs copied to clipboard

Cloud Recording command to dynamically accelerate access to recorded segments

Open jmelancondev opened this issue 1 year ago • 4 comments

This adds the RequestTemporarySegmentDurationOverride command which allows clients to request that a camera records shorter segments on demand. This allows a client to manage costs by configuring cameras for large segment sizes, but dynamically request shorter segments whenever a end-user needs access to the video that is being recorded right now.

This command is optional, and allows a device to offer a better experience for its cloud recording.

jmelancondev avatar Sep 05 '24 17:09 jmelancondev

Agree with principle of operation. Please find a shorter name or acronym to keep especially capabilities concise. Below I use ShortSegment but there may be a better term.

Suggest to add an optional element ShortSegment to RecordingConfiguration with elements

  • Duration and
  • Termination

Suggestion to rename method from Request to Set with clarification that calls overwrite previous requests. This would allow to terminate by setting duration to zero.

Editorial:

The method should be defined in the recording configuration section.

HansBusch avatar Sep 08 '24 17:09 HansBusch

Updated based on Hans & F2F feedback:

  • Renamed to OverrideSegmentDuration name
  • Added the SegmentDurationOverride readonly section in recording configuration target to show what is currently in effect.

jmelancondev avatar Sep 11 '24 07:09 jmelancondev

Reviewed and holding off until updates are made.

kieran242 avatar Oct 23 '24 14:10 kieran242

Thanks, OverrideSegmentDuration now compiles nicely with Visual Studio :-)

HansBusch avatar Oct 25 '24 09:10 HansBusch

Question regarding behavior over reboot. When clients use short expiration duration they have to frequently update it and storing in flash is problematic. Can we add a note like duration of expiration is expected to be in the range of seconds or minutes and may not persist after device restart. ?

HansBusch avatar Nov 27 '24 10:11 HansBusch

Question, what behavior is expected when a client changes the recording configuration and an override is active? For the sake of simplicity I would prefer that this deletes any previous override.

HansBusch avatar Nov 27 '24 11:11 HansBusch

Updated the element name without Request. SOAP conventions are pretty unclear to the uninitiated!

I agree with both statements regarding reboots & configuration upgrade. I've added a mention to those in the spec. I did not add the range part though, as we already added following the F2F that "The override duration shall not exceed 1 hour." in the spec. This seems like a similar, but stronger requirement so I kept that one.

jmelancondev avatar Nov 27 '24 13:11 jmelancondev

Already approved. Re-Reviewed updates and I am happy. Good catch by @HansBusch and you have already fixed it by the time I had re-reviewed.

kieran242 avatar Dec 03 '24 13:12 kieran242