Meeting Summary:
The meeting focused on reviewing pending pull requests (PRs) and discussing blockers for the execution APIs, particularly related to Amsterdam and HiveChain updates. Chase explained that most PRs and issues in execution APIs were blocked by the need to update HiveChain to Amsterdam compatibility, as the current HiveChain hasn’t been updated in a long time and the only person who works on it, Felix, is out of office. Lightclient agreed to review and merge two ready PRs related to HiveChain, which would help unblock other related changes. The team discussed several other PRs including EIP-1898 testing, duplicate named tests, and block access lists, with Chase noting that some items needed more community consensus before merging. Boma expressed frustration about the slow progress on error codes, which has been ongoing for over a year, and suggested creating a PR to prompt more developer discussion and feedback. The conversation ended with agreement that the major blocker is updating HiveChain to Amsterdam compatibility, which would unblock many other development tasks.
Click to expand detailed summary
Chase reported that two PRs are ready for review related to Amsterdam execution APIs, but there is nobody available to review them due to Felix being out of office. Lightclient agreed to review and merge the two PRs, which will allow Chase to rebase the draft PR on top of them. The team also discussed a frame transactions issue (PR 860) which was determined to be a placeholder that doesn’t need immediate attention since frame transactions aren’t currently supported in HiveChain.
Boma and Chase discussed updates on a PR related to deeper goodness, noting that Geth cannot currently pass the spec as written. Boma mentioned leaving a message on the EL channel to inform clients about needed updates for proper alignment with the PR. Chase then explained that their next task involves addressing duplicate named tests and regenerating fixtures in a separate PR, which they plan to merge after receiving more approvals.
Chase explained that PR 862 is blocking the merge of BuildBlock V1 testing due to an issue with GoEthereum not honoring the target gas limit in the API. Chase opened a PR in GoEthereum to patch this, as other clients already support it. Boma acknowledged understanding the issue after Chase’s explanation.
Chase and Boma discussed merging a block access list feature, but decided to wait since there are open client issues referencing it and Hive chain doesn’t currently support block access lists. They also reviewed an e-subscribe PR from Sena, which Chase agreed to merge since the upgrade issue had been resolved. The conversation ended with plans to discuss error codes in a future meeting when more team members could attend.
Boma and Chase discussed the need to move forward on an issue that has been ongoing for over a year, with previous work done by Felix and Sina on air codes changes. They considered getting more opinions from the EL team but acknowledged that previous discussions on this topic were not recent. The conversation ended with Boma suggesting they wait for Felix’s return to get updates on the ongoing work and decide on the release timeline.
The team discussed moving forward with a release, with Boma suggesting creating a PR to prompt developer feedback, while Chase emphasized the need for more discussion before making a 1.00 release. They identified HiveChain integration as a major blocker preventing testing on Glamsterdam. The team agreed to point out these concerns to the Discord channel and to have Klein review the high PRs, with Chase confirming he would be okay with this approach.
Next Steps:
- lightclient: Review and merge the two HiveChain PRs (debugs and spec test/spec gen) today.
- Chase: Rebase the draft PR (tracking/Amsterdam) on top of the merged HiveChain PRs and update as needed.
- Chase: Post the EIP-1898 tests PR in Discord to get more thumbs up before merging.
- Chase: Merge the e-subscribe PR from Sena.
- Boma: Point out the release discussion and HiveChain Amsterdam blockers in the Discord channel and notify Felix.
- Chase: Follow up on the Go Ethereum issue regarding target gas limit for BuildBlock V1 before merging the related testing PR.
Recording Access: