Skip to content

feat(charts): option to purge geth mempool#1182

Merged
mycodecrafting merged 3 commits intomainfrom
mycodecrafting/geth-purge-mempool
Jun 17, 2024
Merged

feat(charts): option to purge geth mempool#1182
mycodecrafting merged 3 commits intomainfrom
mycodecrafting/geth-purge-mempool

Conversation

@mycodecrafting
Copy link
Contributor

@mycodecrafting mycodecrafting commented Jun 14, 2024

Summary

Add a purgeMempool flag to the rollup config which will force purge the geth mempool on startup if set to true.

Testing

  • Started local dev cluster w/ storage enabled and purgeMempool disabled.
  • Forced two (2) queued tx in mempool by sending tx with higher nonce than account's current nonce
  • Checked 2 queued tx were in mempool using txpool_status rpc call (✅ passed)
  • Deleted geth pod to force it to restart
  • Checked 2 queued tx were in mempool using txpool_status rpc call (✅ passed)
  • Enabled purgeMempool and updated helm chart
  • Deleted geth pod to force it to restart
  • Checked pending and queued tx were 0 using txpool_status rpc call (✅ passed)

@mycodecrafting mycodecrafting requested a review from a team as a code owner June 14, 2024 14:19
@mycodecrafting mycodecrafting requested a review from aajimal June 14, 2024 14:19
@github-actions github-actions bot added the cd label Jun 14, 2024
@mycodecrafting mycodecrafting added this pull request to the merge queue Jun 17, 2024
Merged via the queue into main with commit 16befd0 Jun 17, 2024
@mycodecrafting mycodecrafting deleted the mycodecrafting/geth-purge-mempool branch June 17, 2024 13:33
steezeburger added a commit that referenced this pull request Jun 19, 2024
* main:
  chore(bridge-withdrawer): add missing errors and clean up names (#1178)
  feat(sequencer): add ttl and invalid cache to app mempool (#1138)
  chore(astria-merkle): add benchmarks (#1179)
  chore(sequencer-relayer): add timeout to gRPCs to Celestia app (#1191)
  refactor(core): parse ics20 denoms as ibc or trace prefixed variants (#1181)
  Mycodecrafting/sequencer seed node (#1188)
  chore: register all metrics during startup (#1144)
  feat(charts): option to purge geth mempool (#1182)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants