cleanup: remove unused defaultAutoDetectFrequency variable

[JaredTan95]

Description:

Link to tracking Issue(s):

• Resolves: #issue-number

Testing:

Documentation:

[swiatekm]

@JaredTan95 could you fix the lint failure?

On a related note, I do not understand why the unused linter doesn't flag this. @open-telemetry/operator-approvers does anyone have an idea?

[iblancasa]

@JaredTan95 could you fix the lint failure?

On a related note, I do not understand why the unused linter doesn't flag this. @open-telemetry/operator-approvers does anyone have an idea?

Because it doesn't check global unexported variables. We might want to enable deadcode.

[swiatekm]

Because it doesn't check global unexported variables. We might want to enable deadcode.

deadcode was deprecated some time ago and is not present in latest golangci-lint. Which does actually find unused constants, but not if they're grouped. I'll see if there's a bug report about that.