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

Update-Metadata - Safer workflow #64

Closed
10 tasks done
TheTaylorLee opened this issue Nov 28, 2024 · 0 comments · Fixed by #66
Closed
10 tasks done

Update-Metadata - Safer workflow #64

TheTaylorLee opened this issue Nov 28, 2024 · 0 comments · Fixed by #66
Labels
enhancement New feature or request

Comments

@TheTaylorLee
Copy link
Owner

TheTaylorLee commented Nov 28, 2024

Topic Reason
Update-Metadata currently remuxes in the same directory as the source file. This presents some risks that would be mitigated if it followed similar workflows to the transcoding process.

Issues that would be mitigated

  1. NFS locations would not be at risk of corruption due to dropped packets.
  2. File watching applications like plex would not cause disruption during transcoding.
  3. Disruptions during the remux process would result in the process not continuing until handled.

Changes to implement

  • Create a temp database for logging file original locations. Needed for knowing where to put post processed file given update-metadata should not update the prod shows/movies tables beyond the existing contexts.
  • Foreach file confirm fftools directories are empty
  • Copy File into source process directory
  • Write a table entry in the temp db with filename, fullpath, & tempname to avoid conflicts.
  • Remux into target process directory
  • Remove file from source process directory
  • Move file from target process directory into processed directory.
  • Check source, target, and processed directories are empty on next clean run of dta. If they are and temp db exists, delete it.
  • Overwrite original file with file that is in processed directory. Match fullpath from the database to confirm target location where processed filename matches tempname.
  • Update readme to remove update-metadata warning and instead just keep a warning about keeping backups.
@TheTaylorLee TheTaylorLee added question enhancement New feature or request and removed question labels Nov 28, 2024
@TheTaylorLee TheTaylorLee linked a pull request Dec 6, 2024 that will close this issue
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant