Skip to content

p2p: fix logspam#9756

Merged
mergify[bot] merged 3 commits intotendermint:mainfrom
faddat:no-more-logspam
Nov 24, 2022
Merged

p2p: fix logspam#9756
mergify[bot] merged 3 commits intotendermint:mainfrom
faddat:no-more-logspam

Conversation

@faddat
Copy link
Contributor

@faddat faddat commented Nov 23, 2022

Since starting off as a wee validator, I've been mystified by the volume of p2p logspam, which often makes it impossible to monitor other tasks. Thus, routine p2p events, have been cast into the land of debug.


PR checklist

  • Tests written/updated, or no tests needed
  • CHANGELOG_PENDING.md updated, or no changelog entry needed
  • Updated relevant documentation (docs/) and code comments, or no
    documentation updates needed

@faddat faddat requested a review from ebuchman as a code owner November 23, 2022 16:11
@faddat faddat requested a review from a team November 23, 2022 16:11
@thanethomson thanethomson added the S:automerge Automatically merge PR when requirements pass label Nov 24, 2022
@thanethomson
Copy link
Contributor

@faddat could you please rebase? I don't have access to pull the latest changes from main here.

@mergify mergify bot merged commit f7bb065 into tendermint:main Nov 24, 2022
thanethomson pushed a commit that referenced this pull request Nov 24, 2022
Since starting off as a wee validator, I've been mystified by the volume of p2p logspam, which often makes it impossible to monitor other tasks. Thus, routine p2p events, have been cast into the land of debug.

---

#### PR checklist

- [x] Tests written/updated, or no tests needed
- [x] `CHANGELOG_PENDING.md` updated, or no changelog entry needed
- [x] Updated relevant documentation (`docs/`) and code comments, or no
      documentation updates needed
thanethomson pushed a commit that referenced this pull request Nov 24, 2022
Since starting off as a wee validator, I've been mystified by the volume of p2p logspam, which often makes it impossible to monitor other tasks. Thus, routine p2p events, have been cast into the land of debug.

---

#### PR checklist

- [x] Tests written/updated, or no tests needed
- [x] `CHANGELOG_PENDING.md` updated, or no changelog entry needed
- [x] Updated relevant documentation (`docs/`) and code comments, or no
      documentation updates needed
thanethomson added a commit that referenced this pull request Nov 30, 2022
Since starting off as a wee validator, I've been mystified by the volume of p2p logspam, which often makes it impossible to monitor other tasks. Thus, routine p2p events, have been cast into the land of debug.

---

#### PR checklist

- [x] Tests written/updated, or no tests needed
- [x] `CHANGELOG_PENDING.md` updated, or no changelog entry needed
- [x] Updated relevant documentation (`docs/`) and code comments, or no
      documentation updates needed

Co-authored-by: Jacob Gadikian <jacobgadikian@gmail.com>
thanethomson added a commit that referenced this pull request Dec 2, 2022
Since starting off as a wee validator, I've been mystified by the volume of p2p logspam, which often makes it impossible to monitor other tasks. Thus, routine p2p events, have been cast into the land of debug.

---

#### PR checklist

- [x] Tests written/updated, or no tests needed
- [x] `CHANGELOG_PENDING.md` updated, or no changelog entry needed
- [x] Updated relevant documentation (`docs/`) and code comments, or no
      documentation updates needed

Co-authored-by: Jacob Gadikian <jacobgadikian@gmail.com>
@adizere adizere linked an issue Dec 6, 2022 that may be closed by this pull request
@moul moul mentioned this pull request Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S:automerge Automatically merge PR when requirements pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tendermint should be a little less loud

2 participants