This is in reference to this comment
The error handling for plugin and directory deletions should fail silently if file or directory not found. However, if we get any other error when attempting deletions, then the method should return an error.
Acceptance criteria:
- a test triggering showing that we now return an error if the file cannot be deleted and is not already absent
- (removing Write Permission on the file during the test seems like a good candidate?)
- In such case, the logging should be as clear as possible.
Open question
Should we have our own deleteFile variant that would try many steps to delete something before abandoning? (I am inclined to think so)
- is blocked by
-
JENKINS-54545 Promisify the plugin deletion
- Resolved
- links to