Jason Fehr has uploaded a new patch set (#14). ( 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 function that configures the OtlpHttpExporter instance to ensure startup flags are correctly parsed and applied. 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 --- 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 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 10 files changed, 280 insertions(+), 46 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/40/23440/14 -- 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: newpatchset Gerrit-Change-Id: Ie321fa37c0fd260f783dc6cf47924d53a06d82ea Gerrit-Change-Number: 23440 Gerrit-PatchSet: 14 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
