Address review comments

Author: sciascid

include/wsrep/provider.hpp

@@ -515,7 +515,7 @@ namespace wsrep
          * Create a new provider.
          *
          * @param provider_spec Provider specification
-         * @param provider_options Initial options to provider
+         * @param provider_options_cb Callback to get initial provider options
          * @param thread_service Optional thread service implementation.
          */
         static std::unique_ptr<provider>

include/wsrep/server_state.hpp

@@ -297,7 +297,8 @@ namespace wsrep
          * @return Zero on success, non-zero on error.
          */
         int load_provider(const std::string& provider,
-                          const std::function<std::string(const provider_options&)>&,
+                          const std::function<std::string(
+                              const provider_options&)>& provider_options_cb,
                           const wsrep::provider::services& services
                           = wsrep::provider::services());
 

src/config_service_v1.cpp

@@ -233,7 +233,7 @@ int wsrep::config_service_v2_fetch(struct wsrep_st* wsrep,
     }
     if (config_service_v2_probe(wsrep->dlh))
     {
-        wsrep::log_warning() << "Provider does not support config service v2";
+        wsrep::log_info() << "Provider does not support config service v2";
         return 1;
     }
     if (config_service_v2_init(wsrep->dlh))