Skip to content

Commit 460eb7d

Browse files
authored
Redis port description state fix (#293)
* Redis Port issue fix * clean up
1 parent c16f32d commit 460eb7d

File tree

2 files changed

+1
-15
lines changed

2 files changed

+1
-15
lines changed

src/VirtualClient/VirtualClient.Actions/Memtier/MemtierBenchmarkClientExecutor.cs

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ public class MemtierBenchmarkClientExecutor : MemcachedExecutor
3636
new Regex(@"connection\s+refused", RegexOptions.IgnoreCase | RegexOptions.Compiled)
3737
};
3838

39-
// private List<Metric> aggregatedMetrics;
4039
private List<ProcessOutputDescription> processOutputDescriptions;
41-
// private int startingServerPort;
4240

4341
/// <summary>
4442
/// Initializes a new instance of the <see cref="MemtierBenchmarkClientExecutor"/> class.
@@ -135,18 +133,6 @@ public bool WarmUp
135133
}
136134
}
137135

138-
/// <summary>
139-
/// Parameter defines true/false whether the client action should also emit metric per each server process separately.
140-
/// Default value is false and collects only aggregate metrics across all the redis/memcached server processes running in the system.
141-
/// </summary>
142-
public bool PerProcessMetric
143-
{
144-
get
145-
{
146-
return this.Parameters.GetValue<bool>(nameof(this.PerProcessMetric), false);
147-
}
148-
}
149-
150136
/// <summary>
151137
/// True/false whether TLS should be enabled. Default = false.
152138
/// </summary>

src/VirtualClient/VirtualClient.Actions/Redis/RedisServerExecutor.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ private Task SaveStateAsync(EventContext telemetryContext, CancellationToken can
328328
ports.Add(new PortDescription
329329
{
330330
CpuAffinity = this.BindToCores ? i.ToString() : null,
331-
Port = this.Port
331+
Port = this.Port + i
332332
});
333333
}
334334

0 commit comments

Comments
 (0)