From: Sage Weil <sage@redhat.com>
Date: Sat, 7 Apr 2018 18:36:32 +0000 (-0500)
Subject: osd/osd_types: fix pg_pool_t encoding for hammer
X-Git-Tag: v13.1.0~363^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=37c22b273959c6ca54808f3cb8d22a2d250d0c1b;p=ceph.git

osd/osd_types: fix pg_pool_t encoding for hammer

If we are missing the hammer feature, we will also be missing the
luminous feature, but we should still encode like hammer.

Analogous fix to e28e0c617af8825ae92cced5d87cc4f403709e48

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

diff --git a/src/osd/osd_types.cc b/src/osd/osd_types.cc
index ca5ddc0bea3..ff6adb50062 100644
--- a/src/osd/osd_types.cc
+++ b/src/osd/osd_types.cc
@@ -1557,8 +1557,7 @@ void pg_pool_t::encode(bufferlist& bl, uint64_t features) const
     // this was the first post-hammer thing we added; if it's missing, encode
     // like hammer.
     v = 21;
-  }
-  if (!HAVE_FEATURE(features, SERVER_LUMINOUS)) {
+  } else if (!HAVE_FEATURE(features, SERVER_LUMINOUS)) {
     v = 24;
   }