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

Instruction to reduce disk usage of a node #61

Merged
merged 2 commits into from
Oct 22, 2021
Merged

Instruction to reduce disk usage of a node #61

merged 2 commits into from
Oct 22, 2021

Conversation

mturkia
Copy link
Contributor

@mturkia mturkia commented Oct 22, 2021

Description

Additional documentation. These instructions lower the disk usage of sentries to 20% of the
default configuration. Some of these configurations can safely be used also on a validator.


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

These instructions lower the disk usage of sentries to 20% of the
default configuration. Some of these configurations can safely be used
also on a validator.
@kinrokinro
Copy link
Contributor

Great, good job!

Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mturkia! Can you drop your evmos address on the comments?

@fedekunze fedekunze merged commit a601428 into evmos:main Oct 22, 2021
@mturkia
Copy link
Contributor Author

mturkia commented Oct 22, 2021

Thanks @mturkia! Can you drop your evmos address on the comments?

evmos1zk8ef40wrhm8z9sqcmfa0zhz8euyyra8lmuesf

faddat referenced this pull request in notional-labs/evmos Dec 9, 2021
* evm: fix sig verification

* fixes

* test fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants