Write Master + Rendition manifests

[thomshutt]

No description provided.

[codecov]

Codecov Report

Merging #89 (2e4d0fb) into main (6fa7738) will increase coverage by 0.79123%.
The diff coverage is 59.37500%.

@@                 Coverage Diff                 @@
##                main         #89         +/-   ##
===================================================
+ Coverage   46.77996%   47.57119%   +0.79123%     
===================================================
  Files             21          21                 
  Lines           1118        1194         +76     
===================================================
+ Hits             523         568         +45     
- Misses           533         557         +24     
- Partials          62          69          +7     

Impacted Files	Coverage Δ
clients/object_store_client.go	50.00000% <0.00000%> (-50.00000%)	⬇️
handlers/misttriggers/recording_end.go	27.27273% <0.00000%> (ø)
handlers/transcode.go	42.30769% <0.00000%> (ø)
transcode/transcode.go	46.42857% <60.00000%> (-3.57143%)	⬇️
transcode/manifest.go	72.09302% <68.91892%> (-2.90698%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6fa7738...2e4d0fb. Read the comment docs.

Impacted Files	Coverage Δ
clients/object_store_client.go	50.00000% <0.00000%> (-50.00000%)	⬇️
handlers/misttriggers/recording_end.go	27.27273% <0.00000%> (ø)
handlers/transcode.go	42.30769% <0.00000%> (ø)
transcode/transcode.go	46.42857% <60.00000%> (-3.57143%)	⬇️
transcode/manifest.go	72.09302% <68.91892%> (-2.90698%)	⬇️

[emranemran]

Just one comment that can be addressed in follow-up if needed but otherwise LGTM.

[emranemran]

According to the Apple HLS spec, the master playlist is supposed to be in reverse descending order of bitrate then resolution. I think that's how the HLS players need the master playlist to be ordered (?). If so, I don't believe we're guaranteed to get the highest bitrate rendition first from the BOT flow. I added reverse sorting playlist logic in the previous flow that might help if the HLS spec is how it's supposed to be implemented.

[thomshutt]

Good point, I created #94 to do it as a follow-up once transcoding is in. I'll have to look at the spec, but I seem to remember that they should be generally ordered like that but that there's some more subtle stuff around which one comes first