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

clip: add more logging to debug failed clips #1318

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Commits on Jul 14, 2024

  1. clip: add more logging to debug failed clips

    Clips are failing reporting 'start time specified exceeds duration of
    manifest'. This may happen for webrtc streams where the clip request
    comes in but the segments and manifest hasn't been written yet to S3. I
    n this case, the start time would fall past the latest segment and this
    error case would be hit -- however, we have several retries for this
    exact scenario.
    
    Based on logs, there are 10 retries every 5 seconds in
    addition to 3 studio retries - this should be sufficient for the latest
    segments to be written to S3 and the clip request would eventually
    succeed. This is not happening so we're adding the additional logging to
    ensure latest manifests are being downloaded on retry attempts.
    emranemran committed Jul 14, 2024
    Configuration menu
    Copy the full SHA
    5169c4e View commit details
    Browse the repository at this point in the history