ClusterEnvironmentInfo telemetry device unnecessarily calls nodes stats API to populate metadata fields in a metric store which duplicates ExternalEnvironmentInfo. As ClusterEnvironmentInfo is not tagged with serverless.Status.Internal status, the additional bonus of the simplification is unblocking Rally when ran with public serverless privileges.
I've tested the change comparing metric store content (rally-metrics-*) with/without simplification. On each run the amount of documents with the following fields was equal:
serverless cluster with operator privileges - meta.jvm_vendor field,
stateful ESS cluster with superuser privileges - meta.jvm_vendor and meta.attribute_region fields.
ClusterEnvironmentInfotelemetry device unnecessarily calls nodes stats API to populate metadata fields in a metric store which duplicatesExternalEnvironmentInfo. AsClusterEnvironmentInfois not tagged withserverless.Status.Internalstatus, the additional bonus of the simplification is unblocking Rally when ran with public serverless privileges.I've tested the change comparing metric store content (
rally-metrics-*) with/without simplification. On each run the amount of documents with the following fields was equal:meta.jvm_vendorfield,meta.jvm_vendorandmeta.attribute_regionfields.