lodestar

lodestar copied to clipboard

[Prater] Validator misses attestation duties

1

**Describe the bug** Validator 258098 missed its duties at epoch 93025, slot 2976826 In the validator log ``` May-11 02:34:48.819[] ^[[34mdebug^[[39m: Signed attestation slot=2976774, index=23, head=0xb4e289c97354dfb8c2384b1b2bb1a52d2ba68aa92ebe8b8281646e94bbb6d361, validatorIndex=258098 ... May-11 02:48:04.081[]...

twoeths

[Prater] Submitted unaggregated attestation is not included in AggregateAndProof

1

**Describe the bug** Validator submitted correct head attestation, at the correct time, with a good number of sent peers: ``` May-11 00:19:27.763[] ^[[34mdebug^[[39m: Signed attestation slot=2976097, index=31, head=0x9f29c0e6e301bf8f4ff6c6f2ed5bbfa282adaa5bb62132111a2d0e590e53246b, validatorIndex=258998 ```...

twoeths

**Describe the bug** No log was captured since it's 5 days ago, and this happened with version v0.36.0 **Expected behavior** No error

twoeths

Store orphaned/non-canonical blocks

2

**Is your feature request related to a problem? Please describe.** Right now when there are non-canonical blocks, there is no way inspect them, refer to https://github.com/ChainSafe/lodestar/blob/18c504c016214db3f645228346fab0f80accc9e4/packages/lodestar/src/chain/archiver/archiveBlocks.ts#L57 **Describe the solution you'd...

twoeths

Consider using node-proper-lockfile instead of lockfile

2

See https://github.com/moxystudio/node-proper-lockfile there are arguments in favor of switching. (recommended here https://stackoverflow.com/questions/35616281/how-can-i-lock-a-file-while-writing-to-it-asynchronously) lockfile Github repo is archived too Tho both libraries rely in https://github.com/tapjs/signal-exit/blob/main/index.js which does ugly tricks to work...

dapplion

Since 4.7 supports node16 and nodenext compilerOptions.module option, should we change it? I'm not sure what practical implications does it have to us or if we should use it. https://devblogs.microsoft.com/typescript/announcing-typescript-4-7/?q=1#esm-nodejs

wemeetagain

Consider using snappy along with snappyjs

1

**Is your feature request related to a problem? Please describe.** Right now we only use snappyjs and it's good with current gossip message size, but after the merge the block...

twoeths

**Describe the bug** Requesting the snapshot succeeds: ``` curl http://localhost:9596/eth/v1/lightclient/snapshot/0x514c1b91bd66eb3d2418cd92aadba21c6508b8e30c5af93bdb2e4d2745e44f6d | jq % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed...

dadepo

- See https://github.com/ChainSafe/lodestar/pull/4212 - Add latter after merging https://github.com/ChainSafe/lodestar/pull/4201

wemeetagain

export only required public variables - no export *

1

- From https://github.com/ChainSafe/lodestar/issues/3116 export * is too vague, exposes too many items to consumers. The public API should be though of with care and only expose what we expect to...

dapplion