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

Logging (pathfind): Don't log [WRN] CRTR: Channel x has zero cltv delta #6069

Closed
C-Otto opened this issue Dec 7, 2021 · 3 comments · May be fixed by #6856
Closed

Logging (pathfind): Don't log [WRN] CRTR: Channel x has zero cltv delta #6069

C-Otto opened this issue Dec 7, 2021 · 3 comments · May be fixed by #6856
Labels
beginner Issues suitable for new developers good first issue Issues suitable for first time contributors to LND logging Related to the logging / debug output functionality P3 might get fixed, nice to have

Comments

@C-Otto
Copy link
Contributor

C-Otto commented Dec 7, 2021

Background

Inspired by C-Otto/rebalance-lnd#241.

As part of pathfinding, lnd might encounter edges that have a policy specifying a zero CLTV delta (possibly because of no/outdated gossip?). Every time an edge like this is used in the pathfinding code, a warning is logged:

log.Warnf("Channel %v has zero cltv delta",

For nodes that query a lot of routes (for example when running rebalance-lnd), countless warnings are logged. These warnings don't seem to serve a proper purpose, as the user cannot do anything about it, and in my experience this also didn't cause any issues.

Your environment

  • lnd-0.14.1-beta
  • Linux server 5.10.0-8-amd64 #​1 SMP Debian 5.10.46-4 (2021-08-03) x86_64 GNU/Linux
  • bitcoind v22

Steps to reproduce

Run QueryRoutes a few times.

Expected behaviour

Warning from issue title is not logged.

Actual behaviour

Warning from issue title is logged quite often.

@Roasbeef Roasbeef added beginner Issues suitable for new developers good first issue Issues suitable for first time contributors to LND logging Related to the logging / debug output functionality P3 might get fixed, nice to have labels Dec 7, 2021
@ErikEk
Copy link
Contributor

ErikEk commented Dec 9, 2021

I can fix it if we want this change.

@sloorush
Copy link

Hey!
I am interested in working on this issue, but could someone please explain what exactly needs to be done as PR #6242 was not accepted, which removes the check and the log.

@ziggie1984
Copy link
Collaborator

Closing with #7267

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beginner Issues suitable for new developers good first issue Issues suitable for first time contributors to LND logging Related to the logging / debug output functionality P3 might get fixed, nice to have
Projects
None yet
5 participants