From 1bd68d87fc5cf2d03e4b43ef841512048d75ff65 Mon Sep 17 00:00:00 2001 From: Pavel Emelyanov Date: Tue, 5 Nov 2019 18:41:58 +0300 Subject: [PATCH] main: Do not carry pctx all over the code v2: - do not use struct initialization extention Signed-off-by: Pavel Emelyanov --- main.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/main.cc b/main.cc index 53fc54e3aa..1407065dc0 100644 --- a/main.cc +++ b/main.cc @@ -505,7 +505,6 @@ int main(int ac, char** av) { auto& mm = service::get_migration_manager(); api::http_context ctx(db, proxy); httpd::http_server_control prometheus_server; - prometheus::config pctx; directories dirs; sharded feature_service; @@ -534,7 +533,7 @@ int main(int ac, char** av) { tcp_syncookies_sanity(); - return seastar::async([cfg, ext, &db, &qp, &proxy, &mm, &ctx, &opts, &dirs, &pctx, + return seastar::async([cfg, ext, &db, &qp, &proxy, &mm, &ctx, &opts, &dirs, &prometheus_server, &cf_cache_hitrate_calculator, &feature_service] { try { ::stop_signal stop_signal; // we can move this earlier to support SIGINT during initialization @@ -604,14 +603,15 @@ int main(int ac, char** av) { uint16_t pport = cfg->prometheus_port(); std::any stop_prometheus; if (pport) { - pctx.metric_help = "Scylla server statistics"; - pctx.prefix = cfg->prometheus_prefix(); prometheus_server.start("prometheus").get(); stop_prometheus = ::make_shared(defer([&prometheus_server] { startlog.info("stopping prometheus API server"); prometheus_server.stop().get(); })); //FIXME discarded future + prometheus::config pctx; + pctx.metric_help = "Scylla server statistics"; + pctx.prefix = cfg->prometheus_prefix(); (void)prometheus::start(prometheus_server, pctx); with_scheduling_group(maintenance_scheduling_group, [&] { return prometheus_server.listen(socket_address{prom_addr, pport}).handle_exception([pport, &cfg] (auto ep) {