Skip to content

Conversation

@jbudz
Copy link
Contributor

@jbudz jbudz commented Nov 29, 2016

On Windows machines, there can be EPERM failures during plugin installation when trying to rename extracted plugin files. This usually seems to be a result of AV software temporarily locking new files. Instead of failing immediately, this retries renaming for up to 3 seconds, before giving up and throwing the original error.

Closes #9046

@LeeDr
Copy link

LeeDr commented Nov 29, 2016

LGTM - I tested installing x-pack with these changes on Windows, and it worked.
I didn't run the new tests/rename.js. I'm not set up to run the tests right now.

@jbudz jbudz added the blocker label Nov 30, 2016
@BigFunger
Copy link
Contributor

LGTM

@jbudz jbudz merged commit 6759e25 into elastic:master Nov 30, 2016
elastic-jasper added a commit that referenced this pull request Nov 30, 2016
Backports PR #9260

**Commit 1:**
[plugin cli] Retry folder rename on windows eperm

* Original sha: 398662f
* Authored by Jonathan Budzenski <[email protected]> on 2016-11-29T20:05:19Z
elastic-jasper added a commit that referenced this pull request Nov 30, 2016
Backports PR #9260

**Commit 1:**
[plugin cli] Retry folder rename on windows eperm

* Original sha: 398662f
* Authored by Jonathan Budzenski <[email protected]> on 2016-11-29T20:05:19Z
epixa pushed a commit that referenced this pull request Nov 30, 2016
Backports PR #9260

**Commit 1:**
[plugin cli] Retry folder rename on windows eperm

* Original sha: 398662f
* Authored by Jonathan Budzenski <[email protected]> on 2016-11-29T20:05:19Z
epixa pushed a commit that referenced this pull request Nov 30, 2016
Backports PR #9260

**Commit 1:**
[plugin cli] Retry folder rename on windows eperm

* Original sha: 398662f
* Authored by Jonathan Budzenski <[email protected]> on 2016-11-29T20:05:19Z
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
Backports PR elastic#9260

**Commit 1:**
[plugin cli] Retry folder rename on windows eperm

* Original sha: 398662f
* Authored by Jonathan Budzenski <[email protected]> on 2016-11-29T20:05:19Z

Former-commit-id: 0fd25ab
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants