Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

channeld: Splice resume check should log to DEBUG #7090

Merged

Conversation

wtogami
Copy link
Contributor

@wtogami wtogami commented Feb 19, 2024

You otherwise get thousands of these zero count Splice resume checks spamming your logs while most of the time it is only informing you of "nothing is happening".

Reduces log noise.

You otherwise get thousands of these zero count Splice resume checks
spamming your logs while most of the time it is only informing you
of "nothing is happening".

Reduces log noise.

Signed-off-by: Warren Togami <[email protected]>
Copy link
Collaborator

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 1264716

@ddustin
Copy link
Collaborator

ddustin commented Feb 19, 2024

ACK 1264716

Worth noting this only happens when peers reconnect. If it's spamming your log you may have a looping disconnect & reconnect problem

Copy link
Contributor

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack 1264716

@rustyrussell rustyrussell merged commit 897ed22 into ElementsProject:master Feb 20, 2024
36 checks passed
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.

4 participants