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.

Sign up for GitHub

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

Jump to bottom

Increase main db timeout + logging tweaks #1017

Merged
merged 24 commits into from
Feb 6, 2024
Merged

Increase main db timeout + logging tweaks #1017

merged 24 commits into from
Feb 6, 2024

Conversation

rkachowski
Copy link

@rkachowski rkachowski commented Jan 25, 2024 •
edited
Loading

Description

Changes db timeouts in order to help ingestion of heavy blocks + adds some info to failed log messages.

Other changes

  • suppresses annoying logs around nfts and block sanitisation
  • removes tx cache update from indexer
    • after investigation, this is only used within the web pod

Tested

  • running on rc11

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 25, 2024 •
edited
Loading

Unit Test Results

       5 files  ±0     363 suites  ±0   3m 45s ⏱️ +25s
2 637 tests ±0  2 563 ✔️ +1  74 💤 ±0  0 ❌  - 1 
2 651 runs  ±0  2 575 ✔️ +1  76 💤 ±0  0 ❌  - 1 

Results for commit 21306eb. ± Comparison against base commit 0652afb.

♻️ This comment has been updated with latest results.

@rkachowski rkachowski changed the title Dhutch/wip token balances fix Increase main db timeout + logging tweaks Feb 6, 2024
@rkachowski rkachowski marked this pull request as ready for review February 6, 2024 14:29
@rkachowski rkachowski requested a review from a team as a code owner February 6, 2024 14:29
@rkachowski rkachowski enabled auto-merge (squash) February 6, 2024 14:57
@rkachowski rkachowski merged commit 5c45da6 into master Feb 6, 2024
25 checks passed
@rkachowski rkachowski deleted the dhutch/wip_token_balances_fix branch February 6, 2024 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alvarof2 alvarof2 alvarof2 approved these changes

@lvpeschke lvpeschke Awaiting requested review from lvpeschke lvpeschke is a code owner automatically assigned from celo-org/devopsre

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rkachowski @alvarof2