Jason Fehr has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23440 )
Change subject: IMPALA-14455: Cleanup OpenTelemetry Tracing Startup Flags ...................................................................... IMPALA-14455: Cleanup OpenTelemetry Tracing Startup Flags Fixes several issues with the OpenTelemetry tracing startup flags: 1. otel_trace_beeswax -- Removes this hidden flag which enabled tracing of queries submitted over Beeswax. Since this protocol is deprecated and no tests assert the traces generated by Beeswax queries, this flag was removed to eliminate an extra check when determining if OpenTelemetry tracing should be enabled. 2. otel_trace_tls_minimum_version -- Fixes parsing of this flag's value. This flag is in the format "tlsv1.2" or "tlsv1.3", but the OpenTelemetry C++ SDK expects the minimum TLS version to be in the format "1.2" or "1.3". The code now removes the "tlsv" prefix before passing the value to the OpenTelemetry C++ SDK. 3. otel_trace_tls_insecure_skip_verify -- Fixes the guidance to only set this flag to true in dev/testing. Adds ctest tests for the functions that configure the TraceProvider singleton to ensure startup flags are correctly parsed and applied. Modifies the http_exporter_config and init_otel_tracer function signatures in otel.cc to return the actual object they create instead of a Status since these functions only ever returned OK. Updates the OpenTelemetry collector docker-compose file to support the collector receiving traces over both HTTP and HTTPS. This setup is used to manually smoke test the integration from Impala to an OpenTelemetry collector. Change-Id: Ie321fa37c0fd260f783dc6cf47924d53a06d82ea Reviewed-on: http://gerrit.cloudera.org:8080/23440 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Joe McDonnell <[email protected]> --- M be/src/observe/otel-flags-trace.cc M be/src/observe/otel-test.cc M be/src/observe/otel.cc M be/src/observe/otel.h M be/src/runtime/exec-env.cc A be/src/testutil/localhost.key A be/src/testutil/localhost.pem M testdata/bin/otel-collector/README.md M testdata/bin/otel-collector/docker-compose.yml R testdata/bin/otel-collector/otel-config-http.yml A testdata/bin/otel-collector/otel-config-https.yml 11 files changed, 497 insertions(+), 85 deletions(-) Approvals: Impala Public Jenkins: Verified Joe McDonnell: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23440 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie321fa37c0fd260f783dc6cf47924d53a06d82ea Gerrit-Change-Number: 23440 Gerrit-PatchSet: 17 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]>
