Skip to content

Commit 2e02d68

Browse files
committed
fix: update PromPgw option description and refactor MetricPusher initialization
1 parent bc1dae4 commit 2e02d68

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

slo/src/Internal/Cli.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ public static class Cli
1414

1515
private static readonly Option<string> PromPgwOption = new(
1616
"--prom-pgw",
17-
"minimum amount of partitions in table") { IsRequired = true };
17+
"prometheus push gateway") { IsRequired = true };
1818

1919
private static readonly Option<string> ResourceYdbPath = new(
2020
new[] { "-t", "--resource-ydb-path" },
@@ -113,4 +113,4 @@ public static async Task<int> Run<T>(SloContext<T> sloContext, string[] args) wh
113113

114114
return await RootCommand.InvokeAsync(args);
115115
}
116-
}
116+
}

slo/src/Internal/SloContext.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,8 @@ public async Task Run(RunConfig runConfig)
9292
{
9393
// Trace.Listeners.Add(new ConsoleTraceListener()); debug meterPusher
9494

95-
var promPgwEndpoint = $"{runConfig.PromPgw}/metrics";
9695
var client = await CreateClient(runConfig);
97-
using var prometheus = new MetricPusher(promPgwEndpoint, "workload-" + Job,
96+
using var prometheus = new MetricPusher(runConfig.PromPgw, "workload-" + Job,
9897
intervalMilliseconds: runConfig.ReportPeriod);
9998
prometheus.Start();
10099

0 commit comments

Comments
 (0)