Make telemetry related variables const again in internal/manager/telemetry/manager.go #4119
Closed
2 tasks done
Labels
area/debt
area/maintenance
Cleanup, refactoring, and other maintenance improvements that don't change functionality.
area/tests
Is there an existing issue for this?
Problem Statement
PR #4058 made
variables non-const and global to allow adjustments for proper testing. Having mutable package level variables is not the best and brings some issues.
Proposed Solution
Changing them back into const and adding options to customise them. Probably manager's Config has to be used (or maybe it's a better way...). We don't have to expose the CLI's flag for user to make them configurable.
Additional information
No response
Acceptance Criteria
SplunkEndpoint
,SplunkEndpointInsecureSkipVerify
,TelemetryPeriod
are configurable without being package global variablesThe text was updated successfully, but these errors were encountered: