Do not print a failure message when remote bootstrap is started succe…

…ssfully

Summary: Currently we print a warning message "Start remote bootstrap failed: OK". Fixing this.

Test Plan: Started a local cluster, added a new node, verified that the message doesn't appear anymore.

Reviewers: sergei, venkatesh, bharat

Reviewed By: bharat

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D4357

src/yb/tserver/tablet_service.cc

@@ -1298,7 +1298,7 @@ void ConsensusServiceImpl::StartRemoteBootstrap(const StartRemoteBootstrapReques
     return;
   }
   Status s = tablet_manager_->StartRemoteBootstrap(*req);
-  LOG_IF(WARNING, s.ok()) << "Start remote bootstrap failed: " << s;
+  LOG_IF(WARNING, !s.ok()) << "Start remote bootstrap failed: " << s;
   RETURN_UNKNOWN_ERROR_IF_NOT_OK(s, resp, &context);
   context.RespondSuccess();
 }