Skip to content

internal/cli/server: update default cli values#503

Merged
0xsharma merged 4 commits into
developfrom
shivam/sealer-gasPrice
Aug 26, 2022
Merged

internal/cli/server: update default cli values#503
0xsharma merged 4 commits into
developfrom
shivam/sealer-gasPrice

Conversation

@0xsharma

@0xsharma 0xsharma commented Aug 24, 2022

Copy link
Copy Markdown
Member

This PR sets some of the default flags in the new-cli referencing from bor v0.2.16 defaults.

Some of the flags changed are

  • txpool.journal
  • miner.gaslimit
  • miner.gasprice (It was set to 30 gwei which is not ideal for testnets. It is also later translated to txpool.pricelimit on a miner node and hence txpool is not able get transactions from other nodes as well)
  • cors/vhost configurations for http, ws, graphql
  • disable-bor-wallet (This change was already done in the release qa branch but was not reflected in develop).

@manav2401

Copy link
Copy Markdown
Member

Should we also change here and in launch repo?

@temaniarpit27

Copy link
Copy Markdown
Contributor

Should we also change here and in launch repo?

isn't this deprecated?

@codecov-commenter

codecov-commenter commented Aug 24, 2022

Copy link
Copy Markdown

Codecov Report

Merging #503 (1ac5f09) into develop (95f551e) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #503      +/-   ##
===========================================
+ Coverage    56.84%   56.95%   +0.10%     
===========================================
  Files          607      610       +3     
  Lines        69977    69999      +22     
===========================================
+ Hits         39781    39870      +89     
+ Misses       26788    26724      -64     
+ Partials      3408     3405       -3     
Impacted Files Coverage Δ
internal/cli/server/config.go 73.44% <100.00%> (+0.46%) ⬆️
core/rawdb/chain_iterator.go 58.38% <0.00%> (-1.25%) ⬇️
les/fetcher/block_fetcher.go 84.07% <0.00%> (-0.53%) ⬇️
metrics/disk_nop.go 0.00% <0.00%> (ø)
common/fdlimit/fdlimit_darwin.go 52.38% <0.00%> (ø)
consensus/ethash/mmap_help_other.go 100.00% <0.00%> (ø)
consensus/bor/bor.go 8.21% <0.00%> (+0.01%) ⬆️
les/downloader/downloader.go 75.47% <0.00%> (+0.21%) ⬆️
eth/downloader/skeleton.go 77.21% <0.00%> (+0.25%) ⬆️
p2p/discover/v5_udp.go 81.21% <0.00%> (+0.26%) ⬆️
... and 25 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@0xsharma 0xsharma changed the title chg : sealer.gasPrice to 1 from 30Gwei chg : sealer.gasPrice to 1Gwei from 30Gwei Aug 25, 2022
@manav2401 manav2401 force-pushed the shivam/sealer-gasPrice branch from 63f31bf to eb16a3d Compare August 26, 2022 05:55
@manav2401 manav2401 requested review from a team and temaniarpit27 August 26, 2022 05:58
@manav2401 manav2401 changed the title chg : sealer.gasPrice to 1Gwei from 30Gwei internal/cli/server: update default cli values Aug 26, 2022
@0xsharma 0xsharma merged commit 9c370f4 into develop Aug 26, 2022
@0xsharma 0xsharma deleted the shivam/sealer-gasPrice branch August 26, 2022 12:51
manav2401 pushed a commit that referenced this pull request Aug 26, 2022
internal/cli/server: update default cli values
manav2401 added a commit that referenced this pull request Sep 1, 2022
internal/cli/server: update default cli values

Co-authored-by: SHIVAM SHARMA <shivam691999@gmail.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.

5 participants