Skip to content

[Misc] Update various OpenTelemetry config options

Description

This MR moves the username from the file pointed to in metricsExportOtlpBasicAuthFile to a separate config option. In doing so, it:

  • Creates a new config option metricsExportOtlpAuthBasicUsername
  • Changes metricsExportOtlpBasicAuthFile to metricsExportOtlpAuthBasicPasswordFile
    • (note the change from BasicAuth to AuthBasic; this is to follow the namespacing more closely)
  • Removed the word export from various of the config options

Both changes happen similarly to the options in the frontend.

New taped conf:

telemetry retainInstanceIdOnRestart false

telemetry metricsBackend NOOP

telemetry metricsExportInterval 15000

telemetry metricsExportTimeout 3000

telemetry metricsOtlpEndpoint endpoint:port

# New
telemetry metricsOtlpAuthBasicUsername username
# Changed from metricsExportOtlpBasicAuthFile (name change + expects password only now)
telemetry metricsOtlpAuthBasicPasswordFile /path/to/password/file

# Renamed from metricsExportFileEndpoint
telemetry metricsFileEndpoint path

New frontend conf:

cta.telemetry.retain_instance_id_on_restart false

cta.telemetry.metrics.backend NOOP

cta.telemetry.metrics.export.interval 15000
cta.telemetry.metrics.export.timeout 3000

# Renamed from cta.telemetry.metrics.export.otlp.endpoint
cta.telemetry.metrics.otlp.endpoint endpoint:port

# New
cta.telemetry.metrics.otlp.auth.basic.username username
# Changed from cta.telemetry.metrics.export.otlp.basic_auth_file (name change + expects password only now)
cta.telemetry.metrics.otlp.auth.basic.password_file /path/to/password/file

# Renamed from cta.telemetry.metrics.export.file.endpoint
cta.telemetry.metrics.file.endpoint path

Checklist

  • Documentation reflects the changes made.
  • Merge Request title is clear, concise, and suitable as a changelog entry. See our contributing docs

References

Closes #1357 (closed)

Edited by Niels Alexander Buegel

Merge request reports

Loading