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

Create rpc startup script #1249

Merged
merged 3 commits into from
Feb 9, 2024
Merged

Create rpc startup script #1249

merged 3 commits into from
Feb 9, 2024

Conversation

philipsu522
Copy link
Contributor

Describe your changes and provide context

Create an easy tool for starting a sei node. This will ultimately help with documentation as we no longer need to maintain a lot of resources on the web page. Inspiration comes mostly from https://docs.osmosis.zone/osmosis-core/osmosisd

Testing performed to validate your change

Tested locally
image

Copy link

codecov bot commented Jan 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ea58ea9) 63.45% compared to head (05391e0) 63.25%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1249      +/-   ##
==========================================
- Coverage   63.45%   63.25%   -0.21%     
==========================================
  Files         257      257              
  Lines       16740    16740              
==========================================
- Hits        10623    10589      -34     
- Misses       5611     5645      +34     
  Partials      506      506              

see 4 files with indirect coverage changes

@philipsu522 philipsu522 requested review from yzang2019 and removed request for stevenlanders January 17, 2024 23:22
return env

def get_state_sync_params(rpc_url):
trust_height_delta = 40000 # may need to tune
Copy link
Contributor

Choose a reason for hiding this comment

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

We can make this an optional params

@philipsu522 philipsu522 enabled auto-merge (squash) February 9, 2024 01:47
@philipsu522 philipsu522 merged commit ef3554f into main Feb 9, 2024
36 checks passed
@philipsu522 philipsu522 deleted the create-rpc-startup-script branch February 9, 2024 01:57
udpatil added a commit that referenced this pull request Feb 9, 2024
* Keep v prefix for seid version (#1311)

keep v prefix for seid version

* Release v3.6.3 (#1317)

* [SeiDB] Add integration test for historical bank send (#1329)

* Add integration test for historical bank send

* Fix go mod

* Ignore error for vote already exist (#1325)

* Ignore error for vote already exist

* Fix go mod

* Add error log

* Create rpc startup script (#1249)

* finish startup script

* remove validator and moniker, add local env

---------

Co-authored-by: Philip Su <philip.su.522@gmail.com>
Co-authored-by: Yiming Zang <50607998+yzang2019@users.noreply.github.com>
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.

2 participants