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

Shortcut to enable all namespaces #2809

Merged
merged 1 commit into from
Jul 6, 2021

Conversation

dapplion
Copy link
Contributor

@dapplion dapplion commented Jul 6, 2021

Motivation

For testing and other usages it's very convenient to enable all namespaces without typing the full list manually

Description

Setting --api.rest.api '*' enables all available namespaces

lion@lion-gram:~/Code/eth2.0/lodestar/packages/cli$ ./bin/lodestar beacon --api.rest.api '*' --api.rest.enabled
Jul-06 12:28:53.267 []                 info: Lodestar version=0.25.1 dapplion/enable-all-namespaces 149f1ae6, network=mainnet
Jul-06 12:29:31.662 [API]              info: Started REST api server address=http://127.0.0.1:9596, namespaces=["beacon","config","debug","events","lightclient","lodestar","node","validator"]

@codeclimate
Copy link

codeclimate bot commented Jul 6, 2021

Code Climate has analyzed commit 149f1ae and detected 0 issues on this pull request.

View more on Code Climate.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2021

Performance Report

✔️ no performance regression detected

Full benchmark results
Benchmark suite Current: f7c9495 Previous: 9cc19c4 Ratio
getCommitteeAssignments - req 1000 vs - 250000 vc 9.2666 ms/op 9.9080 ms/op 0.94
Process block - 250000 vs - with 0 validator exit 246.76 us/op 327.27 us/op 0.75
Process block - 250000 vs - with 1 validator exit 21.567 ms/op 25.378 ms/op 0.85
Process block - 250000 vs - with 16 validator exits 22.646 ms/op 26.697 ms/op 0.85
epoch - 250000 vs - processJustificationAndFinalization 72.047 us/op 83.154 us/op 0.87
epoch - 250000 vs - processRewardsAndPenalties 289.39 ms/op 352.18 ms/op 0.82
epoch - 250000 vs - processRegistryUpdates 11.301 us/op 11.946 us/op 0.95
epoch - 250000 vs - processSlashings 37.048 us/op 47.788 us/op 0.78
epoch - 250000 vs - processFinalUpdates 43.118 ms/op 48.660 ms/op 0.89
epoch - 250000 vs - prepareEpochProcessState 591.56 ms/op 730.61 ms/op 0.81
getAttestationDeltas - 250000 vs 122.89 ms/op 136.41 ms/op 0.90
processSlots - 250000 vs - 32 empty slots 3.9376 s/op 4.3180 s/op 0.91
shuffle list - 512 els 159.16 us/op 187.25 us/op 0.85
shuffle list - 16384 els 1.9280 ms/op 2.0424 ms/op 0.94
shuffle list - 250000 els 27.239 ms/op 28.318 ms/op 0.96
aggregationBits - 2048 els - readonlyValues 251.87 us/op 275.09 us/op 0.92
aggregationBits - 2048 els - zipIndexesInBitList 47.349 us/op 60.597 us/op 0.78
ssz.Root.equals 1.3160 us/op 1.4550 us/op 0.90
getPubkeys - persistent - req 1000 vs - 200000 vc 18.677 us/op 21.472 us/op 0.87
BLS verify - blst-native 2.1341 ms/op 2.4876 ms/op 0.86
BLS verifyMultipleSignatures 3 - blst-native 4.3203 ms/op 5.0224 ms/op 0.86
BLS verifyMultipleSignatures 8 - blst-native 9.2724 ms/op 10.785 ms/op 0.86
BLS verifyMultipleSignatures 32 - blst-native 34.905 ms/op 41.163 ms/op 0.85
BLS aggregatePubkeys 32 - blst-native 53.971 us/op 64.068 us/op 0.84
BLS aggregatePubkeys 128 - blst-native 175.69 us/op 204.03 us/op 0.86
CheckpointStateCache - add get delete 40.282 us/op 55.072 us/op 0.73
validate gossip signedAggregateAndProof - struct 5.0411 ms/op 6.0184 ms/op 0.84
validate gossip signedAggregateAndProof - treeBacked 4.9971 ms/op 6.0250 ms/op 0.83
validate gossip attestation - struct 2.2887 ms/op 2.8001 ms/op 0.82
validate gossip attestation - treeBacked 2.3857 ms/op 2.8874 ms/op 0.83

by benchmarkbot/action

@wemeetagain wemeetagain merged commit 732f767 into master Jul 6, 2021
@wemeetagain wemeetagain deleted the dapplion/enable-all-namespaces branch July 6, 2021 14:17
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