mon: Optimized EC pools preprocess_pgtemp incorrectly rejecting pgtemp as nop

Optimized EC pools store pgtemp with primary shards first, this was not
being taken into account by OSDMonitor::preprocess_pgtemp which meant
that the change of pgtemp from [None,2,4] to [None,4,2] for a 2+1 pool
was being rejected as a nop because the primary first encoded version
of [None,2,4] is [None,4,2].

Signed-off-by: Bill Scales <bill_scales@uk.ibm.com>

diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc
index bd7f60594b3124a1d7a81cb3133a317fda692c7c..97c632cd7e971ff29cf648153512e26513673250 100644 (file)
--- a/src/mon/OSDMonitor.cc
+++ b/src/mon/OSDMonitor.cc
@@ -4142,9 +4142,11 @@ bool OSDMonitor::preprocess_pgtemp(MonOpRequestRef op)
     // change?
     //  NOTE: we assume that this will clear pg_primary, so consider
     //        an existing pg_primary field to imply a change
+    std::vector<int> acting_set;
+    osdmap.pg_to_acting_osds(p->first, acting_set);
     if (p->second.size() &&
        (osdmap.pg_temp->count(p->first) == 0 ||
-        osdmap.pg_temp->get(p->first) != p->second ||
+        acting_set != p->second ||
         osdmap.primary_temp->count(p->first)))
       return false;
   }