Skip to content

[encryption] Fix WAL directory manipulation in windows#1740

Merged
aaronlehmann merged 1 commit intomoby:masterfrom
cyli:fix-raft-storage-windows
Nov 10, 2016
Merged

[encryption] Fix WAL directory manipulation in windows#1740
aaronlehmann merged 1 commit intomoby:masterfrom
cyli:fix-raft-storage-windows

Conversation

@cyli
Copy link
Copy Markdown
Contributor

@cyli cyli commented Nov 10, 2016

This fixes bugs introduced in #1598 in windows:

  1. the wal directory cannot already exist, or wal.Create will not be able to rename
  2. when migrating WALs from an older directory, close the temporary writer first. Otherwise, windows will not be able to rename the temporary directory because the temporary writer has a lock.

@cyli cyli force-pushed the fix-raft-storage-windows branch from 7bbdd60 to 91cadbf Compare November 10, 2016 06:19
@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 10, 2016

Current coverage is 55.13% (diff: 16.66%)

Merging #1740 into master will increase coverage by 0.05%

@@             master      #1740   diff @@
==========================================
  Files           101        101          
  Lines         16497      16499     +2   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           9086       9097    +11   
+ Misses         6280       6266    -14   
- Partials       1131       1136     +5   

Sunburst

Powered by Codecov. Last update ce07d9f...e9cea3a

@aaronlehmann
Copy link
Copy Markdown
Collaborator

LGTM

1 similar comment
@LK4D4
Copy link
Copy Markdown
Contributor

LK4D4 commented Nov 10, 2016

LGTM

@aaronlehmann aaronlehmann merged commit 0be0da2 into moby:master Nov 10, 2016
@cyli cyli deleted the fix-raft-storage-windows branch November 10, 2016 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants