filestore: create but do not clobber current/commit_op_seq

If it exists, preserve its value.  Otherwise, create and set it to 1.

Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/os/FileStore.cc b/src/os/FileStore.cc
index 7bcd0b4b1c8dcd4919e6224a4d3f62c99fda91d1..15e887d1f1fb9b089bd47feadca119f58720751c 100644 (file)
--- a/src/os/FileStore.cc
+++ b/src/os/FileStore.cc
@@ -979,19 +979,21 @@ int FileStore::mkfs()
 
   // write initial op_seq
   {
-    uint64_t initial_seq;
+    uint64_t initial_seq = 0;
     int fd = read_op_seq(&initial_seq);
     if (fd < 0) {
       derr << "mkfs: failed to create " << current_op_seq_fn << ": "
           << cpp_strerror(fd) << dendl;
       goto close_fsid_fd;
     }
-    int err = write_op_seq(fd, 1);
-    if (err < 0) {
-      TEMP_FAILURE_RETRY(::close(fd));  
-      derr << "mkfs: failed to write to " << current_op_seq_fn << ": "
-          << cpp_strerror(err) << dendl;
-      goto close_fsid_fd;
+    if (initial_seq == 0) {
+      int err = write_op_seq(fd, 1);
+      if (err < 0) {
+       TEMP_FAILURE_RETRY(::close(fd));  
+       derr << "mkfs: failed to write to " << current_op_seq_fn << ": "
+            << cpp_strerror(err) << dendl;
+       goto close_fsid_fd;
+      }
     }
     TEMP_FAILURE_RETRY(::close(fd));  
   }