Closed MPins closed 3 weeks ago
[!IMPORTANT]
Review skipped
Auto reviews are limited to specific labels.
Labels to auto review (1)
* llm-reviewPlease check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Hi @MPins thanks for the PR.
I've reviewed your work and it seems a new empty line is not added after the log "Shutdown complete" message. It was though before your PR as can be seen in the shots below:
Before
After
I think @C-Otto was right, this "\n" should not be needed at the end of the shutdown complete message anyway because it should be added by the logging library. I was not able to reproduce it but from your test it seems only to occur with systemd? Maybe we should figure out first if this is an lnd an issue and why exactly the new line does not work?
Hello @Abdulkbk and @Chinwendu20 ... thank you for your comments ... you only see the bug on the log file and you have to start and stop LND using "systemctl start lnd" and "systemctl stop lnd commands.
You can see a lot of tests I did before and after the PR at #8713
@Abdulkbk and @Chinwendu20 ... you can see some tests I did before and after the PR at https://github.com/lightningnetwork/lnd/issues/8713 ... to simulate the bug you have to start and stop the LND using the "systemctl start lnd" and "systemctl stop lnd" ... then you repeat the tests after the PR and can verify the log file the bug is solved
@mpins, remember to re-request review from reviewers when ready
Hello @saubyk this PR should be closed as the https://github.com/lightningnetwork/lnd/pull/9083 solve the issue https://github.com/lightningnetwork/lnd/issues/8713 .
Replaced by https://github.com/lightningnetwork/lnd/pull/9083.
Fixes #8713
Change Description
Insert a blank line log message after the "Shutdown complete" log message to ensure that a new line is printed.
This is actually a workaround as the '\n' character was not working properly in some situations. See the tests I did at #8713.
You can reproduce the bug starting and stopping the LND using the "systemctl start lnd" and "systemctl stop lnd".
Steps to Test
The bug fix was tested on Ubuntu using systemctl start/stop lnd, as the bug only manifested this way.
Original Flaw
2024-05-20 14:04:17.705 [WRN] LTND: Config 'bitcoin.active' is deprecated, please remove it 2024-05-20 14:04:17.705 [INF] LTND: Version: 0.18.0-beta.rc2 commit=v0.18.0-beta.rc1-48-g2a8ca878f, build=development, logging=default, debuglevel=info . . . 2024-05-20 14:05:01.948 [INF] LTND: Shutdown complete2024-05-20 14:05:19.906 [WRN] LTND: Config 'bitcoin.active' is deprecated, please remove it 2024-05-20 14:05:19.907 [INF] LTND: Version: 0.18.0-beta.rc2 commit=v0.18.0-beta.rc1-48-g2a8ca878f, build=development, logging=default, debuglevel=info
Test After the PR
2024-05-20 15:59:47.298 [WRN] LTND: Config 'bitcoin.active' is deprecated, please remove it 2024-05-20 15:59:47.298 [INF] LTND: Version: 0.18.0-beta.rc2 commit=v0.18.0-beta.rc1-48-g2a8ca878f-dirty, build=development, logging=default, debuglevel=info . . . 2024-05-20 16:00:29.366 [INF] LTND: Shutdown complete 2024-05-20 16:00:29.366 [INF] LTND: 2024-05-20 16:00:47.594 [WRN] LTND: Config 'bitcoin.active' is deprecated, please remove it 2024-05-20 16:00:47.594 [INF] LTND: Version: 0.18.0-beta.rc2 commit=v0.18.0-beta.rc1-48-g2a8ca878f-dirty, build=development, logging=default, debuglevel=info
Pull Request Checklist
Testing
Code Style and Documentation
[skip ci]
in the commit message for small changes.📝 Please see our Contribution Guidelines for further guidance.