From: Samuel Just <sam.just@inktank.com>
Date: Tue, 20 Aug 2013 00:23:44 +0000 (-0700)
Subject: PG: remove old log when we upgrade log version
X-Git-Tag: v0.67.2~8
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=b3a9a8c4e5edff5431d8da71033047eced6bf985;p=ceph.git

PG: remove old log when we upgrade log version

Otherwise the log_oid will be non-empty and the next
boot will cause us to try to upgrade again.

Fixes: #6057
Signed-off-by: Samuel Just <sam.just@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
(cherry picked from commit 1f851cb2489a95526de932ec6734ebf413e43490)
---

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index 00c0c15d5573..e06ddbb0e343 100644
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -2438,7 +2438,9 @@ void PG::read_state(ObjectStore *store, bufferlist &bl)
     /* We don't want to leave the old format around in case the next log
      * write happens to be an append_log()
      */
+    pg_log.mark_log_for_rewrite();
     ObjectStore::Transaction t;
+    t.remove(coll_t(), log_oid); // remove old version
     pg_log.write_log(t, log_oid);
     int r = osd->store->apply_transaction(t);
     assert(!r);
diff --git a/src/osd/PGLog.h b/src/osd/PGLog.h
index 955b9c18937d..552f9b0cee96 100644
--- a/src/osd/PGLog.h
+++ b/src/osd/PGLog.h
@@ -180,6 +180,13 @@ protected:
     divergent_priors.insert(make_pair(version, obj));
     dirty_divergent_priors = true;
   }
+public:
+  void mark_log_for_rewrite() {
+    mark_dirty_to(eversion_t::max());
+    mark_dirty_from(eversion_t());
+    touched_log = false;
+  }
+protected:
 
   /// DEBUG
   set<string> log_keys_debug;