client: Wait for caps to flush when flushing metadata.

Embarrassingly, this conditional has been backwards since
I committed it in 818e7939. But we want to do the wait when
we did do the flush and are syncing metadata. As the code
previously stood, we only did the wait when we didn't care,
and it didn't wait anyway since it was waiting for 0. So,
bad all around.
Signed-off-by: Greg Farnum <greg@inktank.com>

diff --git a/src/client/Client.cc b/src/client/Client.cc
index feb3beb8a312b3659c63a74be4e81886cd20c445..705c162da3b4dfeafaa0473a12d39af328971317 100644 (file)
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -5900,7 +5900,7 @@ int Client::_fsync(Fh *f, bool syncdataonly)
     wait_on_list(in->waitfor_commit);
   }
 
-  if (!flushed_metadata) wait_sync_caps(wait_on_flush); //this could wait longer than strictly necessary,
+  if (flushed_metadata) wait_sync_caps(wait_on_flush); //this could wait longer than strictly necessary,
                                                     //but on a sync the user can put up with it
 
   ldout(cct, 10) << "ino " << in->ino << " has no uncommitted writes" << dendl;