PWX-38589: Fix the CPU spike and excessive logging in telemetry pods #1652
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
This PR contains 2 changes which combine to greatly reduce the log spam in the
px-telemetry-phonehome
pod and 1 change to slightly reduce thepx-telemetry-registration
pod's log spam:createDaemonSetTelemetryPhonehome
, we are now seedingAPPLIANCE_ID
andAPPLIANCE_NAME
.reconcile
functions, we now seedAPPLIANCE_NAME
uploadAll
timerange from the last 8760 hours (1 year of hours) to 744 (31 days).With the old config, we were logging 8760 hours * 11 logs-per-run = ~96k log lines
With the new config, we log 744 * 6 = ~3.8k log lines.
Which issue(s) this PR fixes (optional)
Closes PWX-38589
Special notes for your reviewer:
I ran this changeset on my local cluster, the (truncated) output for the log now looks like: