Co-authored-by: YaacovHazan <yaacov.hazan@redis.com>

… argument (#234)

* Enable specifying the tls protocol version to use via --tls-protocols argument

* Added tls-protocols tlsv1.2 and tlsv1.3 tests to CI

* Removed spurious comment on tls protocol parsing

* Updated comment on TLS_PROTOCOLS env variable usage on tests

* Add rate limiting to control the number of requests per second

A new 'rate-limiting' option was added to control the number of
request per second.

The rate limiting is based on the 'Token Bucket' algorithm, and
according to the configured rate, on each interval, a "new" amount
of requests allowed to be sent to the server.

The rate-limiting is at the connection level. Therefore, in cluster mode,
the limitation is for each shard, so if, for example, the cluster has three
shards and the user configured one request per second. On every second,
memtier-benchmark will send three requests, one for each shard.

* Added tests to cover --rate-limiting option. Ensured the help message explains the per connection rate-limit

* Fixed per cluster rps test limits

---------

Co-authored-by: YaacovHazan <yaacov.hazan@redis.com>
Co-authored-by: filipecosta90 <filipecosta.90@gmail.com>

* Fixed tls arg tests

* Fixed tls arg tests

* Verbose output on CI runs

* Verbose output on CI runs

* Fixed tls-protocols parsing

* using c++ std::strtok instead of c strktok

* Skipping rate-limit + test time on cluster setups

* see #238, data import broken

The PR includes some refactoring and cleanup around the following:
object_generator
    - generate_key(), a new function that generates a key and stores it
      inside internal buffer (no need for a buffer at client level)
import_object_generator
    - Align it to object_generator APIs of getting key/value/expire
    - read_next_item - new function for reading the next item before
      creating SET command
    - read_next_key - new function for reading (pointing) to the next
      key before creating the GET command
data_object
    - removed.
      We were already getting key/value/expire separately. Now that
      data-import and verify-data also moved to do it like that,
      we could remove it completely.
verify_client
    - unify the craate_request with its base class, and derive the
      create_x_request functions.

* Included data import test cases

* Cluster mode cannot be specified when importing

* Fixed merge of tests from master

---------

Co-authored-by: YaacovHazan <yaacov.hazan@redis.com>
Co-authored-by: filipecosta90 <filipecosta.90@gmail.com>

* Fix upstream missing USE_TLS macro guard

* Added --disable-tls build variation to prevent this issue in the future

---------

Co-authored-by: filipecosta90 <filipecosta.90@gmail.com>

…W and also raw values per second (#246)

…248)

* Specifying the C++ standard version to avoid deprecation/failures on build in case of new standard usage

* Fixed deprecated and casting build errors

* Removed spurious change

* Update memtier_benchmark.cpp

Co-authored-by: Yossi Gottlieb <yossigo@gmail.com>

---------

Co-authored-by: Yossi Gottlieb <yossigo@gmail.com>

* fixed build for tls v1.3 on openssl v1.0.2. Added CI check

* fixed ci step name

* Include test for openssl v3.0

* Make reference to openssl@3.0 on the Readme given its the LTS version

…he latency quantiles (#251)

* Refresh Red Hat build steps and drop 6.x.
* Add APT repository install steps.

* Release 2.1.0

* Removed CHANGES in favour of release notes of github

* continue on failure on openssl macos matrix

* using brew prefix instead of static path for openssl libs on macos

* Adjusted pkgconfig path in macos to use brew --prefix tool

* Increased codecov upload timeout