fs: reduce unnecessary sync rimraf retries

rimraf should only retry if certain errors are encountered.
Additionally, there is no point sleeping if an error occurs
on the last try.
PR-URL: #30785Fixes: #30580
Refs: #30569
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>

Author: cjihrig

lib/internal/fs/rimraf.js

@@ -209,12 +209,19 @@ function _rmdirSync(path, options, originalErr) {
 rimrafSync(join(path,child),options);
 });
 
-for(leti=1;i<=options.maxRetries+1;i++){
+consttries=options.maxRetries+1;
+
+for(leti=1;i<=tries;i++){
 try{
 returnrmdirSync(path,options);
-}catch{
-if(options.retryDelay>0)
+}catch(err){
+// Only sleep if this is not the last try, and the delay is greater
+// than zero, and an error was encountered that warrants a retry.
+if(retryErrorCodes.has(err.code)&&
+i<tries&&
+options.retryDelay>0){
 sleep(i*options.retryDelay);
+}
 }
 }
 }