Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-54207

Update delete plugin flow to error out for anything other than file not found

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Won't Do
    • Icon: Minor Minor
    • evergreen

      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)

            jennyfive Jen Lijó
            asmith_cb Mandie Smith
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: