RPC Standards #022
Transcript
Ask about this call
Answered from this call's summary and transcript only.
AI-generated — verify against the recording.
Call summary
Decisions
- •PR #761 approved; engine API error code change accepted — 00:24:36
- •eth_getStorageValues PR ready to merge with minor documentation update — 00:39:43
- •Engine API changes require CL consensus; deferred to ACDC/ACDE — 00:44:28
- •txpool_inspect will not be standardized; move to geth-specific namespace — 01:20:22
Highlights
- Open Prs:
- ·PR #761 (fork choice error code) approved; requires coordinated Hive merge — 00:23:52
- ·eth_getStorageValues implemented in Geth, Erigon, Reth; tests regenerated — 00:37:00
- ·eth_capabilities needs abstract resource model; subscription approach preferred over strategies — 00:46:46
- ·txpool namespace: inspect method not standardizable due to unstandardized string format — 01:10:39
- Engine Api:
- ·Engine API PRs require consensus layer approval; not decided on RPC call — 00:43:41
- Error Codes:
- ·Error codes PR updated to use Go inspection; validation tests added — 00:04:30
- ·Minor syntax issues identified; async discussion needed on error wrapping — 00:22:02
- Versioning Discussion:
- ·Versioning deferred; focus shifted to GitHub release (V1 or alpha) — 00:17:32
Action Items
- •Simsonraj — Address error wrapping syntax and group collision validation — 00:04:47
- •Felix — Felix to review error codes PR output after call — 00:08:34
- •Sina — Add non-empty requirement comment to eth_getStorageValues spec — 00:42:08
- •Sina — Document eth_capabilities resource model; consider subscription approach — 01:00:48
Meeting chat21
- Mercy Boma Naps-Nkari
Hello sorry am not sure if my mic is working?
- Chase Wright
You appear to be muted in zoom
- lightclient
hehe
- Chase Wright
Reacted to "hehe" with 😃
- Mercy Boma Naps-Nkari
return nil, fmt.Errorf("error-groups[%d]: %w", i, err)
- Mercy Boma Naps-Nkari
return nil, fmt.Errorf("failed to parse error groups: %v", err)
- Mercy Boma Naps-Nkari
- Mercy Boma Naps-Nkari
- Mercy Boma Naps-Nkari
- Mercy Boma Naps-Nkari
Hello @lightclient
- Chase Wright
- Felix (Geth)
sorry I didn't see the call is on!
- Mercy Boma Naps-Nkari
- Mercy Boma Naps-Nkari
Maybe we can bring it up to ACDE?
- Chase Wright
headBlock
- Chase Wright
- Chase Wright
- Mercy Boma Naps-Nkari
- Chase Wright
- Simsonraj Easvarasakthi
gotta drop, thanks everyone
- Mercy Boma Naps-Nkari
Reacted to "gotta drop, thanks e..." with ❤️
Key decisions
PR #761 approved; engine API error code change accepted
PR #761 (fork choice error code) approved; requires coordinated Hive mergeeth_getStorageValues PR ready to merge with minor documentation update
eth_getStorageValues implemented in Geth, Erigon, Reth; tests regeneratedEngine API changes require CL consensus; deferred to ACDC/ACDE
Engine API PRs require consensus layer approval; not decided on RPC calltxpool_inspect will not be standardized; move to geth-specific namespace
txpool namespace: inspect method not standardizable due to unstandardized string format

