Bill commented on a change in pull request #136:
URL: https://github.com/apache/geode-benchmarks/pull/136#discussion_r525406468
##########
File path: README.md
##########
@@ -150,32 +152,21 @@ public class PutTask extends BenchmarkDriverAdapter
implements Serializable {
}
```
-## SNI
+## SNI Proxy
-On AWS, you can run any benchmark on a topology that routes all client-server
communication through an SNI proxy (HAproxy).
+You can run any benchmark on a topology that routes all client-server
communication through an SNI proxy.
+
+The `withSniProxy` property accepts:
+ * `HAProxy` for HAProxy based SNI proxy (default).
+ * `Envoy` for Envoy based SNI proxy.
+ * `Manual` for providing your own SNI proxy and managing its lifecycle.
+
+The `withSniProxyImage` property can be used to provide an alternative Docker
image to one of the
+supported proxy implementations. The value should be set to a valid Docker
image coordinate.
-To run a test, e.g. `PartitionedGetBenchmark`, with SNI:
-
-`./run_tests.sh -t anytagname -- -PwithSniProxy
'--tests=PartitionedGetBenchmark'`
-
-Since SNI is a feature of TLS, running with the SNI topology incurs TLS
overheads.
-
-### TODO for SNI
-* ~~verify `StartSniProxy` runs on proxy node~~
-* ~~don't require operator to supply `-PwithSSL`/`-DwithSSL=true` when running
SNI tests~~
-* ~~generate `haproxy.cfg` with client-visible SNI hostnames~~
-* ~~turn on SNI via `setPoolSocketFactory` in a new `StartClientSNI` task~~
-* ~~set `--hostname-for-clients` on locator and servers for SNI~~
-* ~~reinstate thread-per-core in `PrePopulateRegion.run()` and in
`PartitionedPutBenchmark[SNI]` ya~~
-* ~~set `keyRange` back to 1e6 in `PartitionedPutBenchmark[SNI]` after
client-server connections are healthy~~
-* ~~make topology orthogonal to tests so all tests can run with SNI; have a
`-PwithSniProxy`/`-DwithSniProxy=true` flag~~
-* Potential performance improvement: HAproxy as configured runs one process
with the max threads-per-process of 64 threads, ostensibly using 64/72 cores
(89%.) We might be able to improve performance by configuring HAproxy to run in
daemon mode where we can run two processes, each multithreaded, to run more
than 64 threads, thereby utilizing 100% of our cores.
-
-## TODO (General)
Review comment:
I'ma capture some of these at tickets.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
