-
Notifications
You must be signed in to change notification settings - Fork 281
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
Fix race condition with certain drives and certain disks #1114
Fix race condition with certain drives and certain disks #1114
Conversation
Nice fix |
The first code snippet is the befeore log
|
Cheers Good fix, only suggestion would be to amend the INFO statement to state something like [attempt x of n], otherwise users may see the line multiple times in the log and think there is another issue. Once you have added that, good to approve. |
Seconding this, better logging will be great |
added in run x of y for drive wait
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good fix, added in the x of y run numbers to provide more information to the end user
9b5f290
into
automatic-ripping-machine:main
Description
Noticed that with an external drive (an external LG SP80NB80) and certain disks (maddening!), there would be a race condition with the drive being ready. Adding a few retries and moving the abort check before we create the
Job()
fixed it on my machine.If I added the retries where the old "Drive appears to be empty" check was, it would bail at
ARM is trying to write a job to the empty.log, or NAS**.log
, so I moved it before.Anyway, not sure if this is the appropriate place to put this (maybe the issue is with that wrapper?), but it fixed it for me.
Type of change
How Has This Been Tested?
It didn't work with some disks (seemed like dual layer ones?), and it does now
Checklist:
Changelog:
Job
objectLogs
Attach logs from successful test runs here