Skip to content

Revert log severity change for updating IOC channel - #187

Open
simon-ess wants to merge 2 commits into
masterfrom
revert_a_log_severity
Open

simon-ess wants to merge 2 commits into
masterfrom
revert_a_log_severity

Conversation

@simon-ess

Copy link
Copy Markdown
Contributor

Unfortunately this gets caught on every channel after a restart since the iocid is different (host:port) in each case. This would flood the info level logs for a typical IOC restart.

Note that the log was being done twice, once before the call to _update_existing_channel_diff_iocid and once during.

Unfortunately this gets caught on every channel after a restart since the
iocid is different (host:port) in each case. This would _flood_ the info
level logs for a typical IOC restart.

Note that the log was being done twice, once before the call to
_update_existing_channel_diff_iocid and once during.
@simon-ess
simon-ess requested a review from jacomago October 13, 2026 13:40
@simon-ess simon-ess self-assigned this Oct 13, 2026
Iterating over records then env vars meant (due to increased logging)
that we logged _every_ record/env var combination. This change means
that we log each environment variable that is missing only a single
time.
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants