autonity / aut

A command-line RPC client for Autonity
MIT License
11 stars 10 forks source link

Ensure compatibility with Yamuna protocol #155

Closed aiman closed 1 month ago

aiman commented 3 months ago

The Autonity Yamuna upgrade brings some new protocol features, we should ensure it doesn't break existing commands and we should also aim to support any new features.

aiman commented 3 months ago

@cmjc or @yazzaoui can advise on breaking changes and/or what commit to use as a baseline

cmjc commented 2 months ago

@aiman - suggest using this tag commit - https://github.com/autonity/autonity/releases/tag/v0.14.0-pre

For breaking changes, I have found the beneath. @yazzaoui please advise if I have missed anything, TIA.


aut breaking changes.

Calls broken as ABI changed (https://github.com/autonity/autonity/compare/v0.13.0...v0.14.0-pre):

Examples:

Note also, new for v0.14.0 (vesting, inflation, impact on liquid ntn rewards):

Calls broken due to BLS aggregation

Breaks aut because:

New functions for existing command line groups

szemate commented 2 months ago

@cmjc

Calls broken due to BLS aggregation

Could you please give examples of calls that break due to BLS aggregation?

cmjc commented 2 months ago

Hi @szemate I'm investigating.

aut block lgtm

aut validator lgtm

aut node lgtm

aut config is broken

aut protocol Commands related to Autonity-specific protocol operations.. I've found one broken so far:

aut protocol total-redistributed
eth_abi.exceptions.InsufficientDataBytes: Tried to read 32 bytes, only got 0 bytes.

The above exception was the direct cause of the following exception:

web3.exceptions.BadFunctionCallOutput: Could not decode contract function call to totalRedistributed with return data: b'', output_types: ['uint256']
szemate commented 2 months ago

aut protocol total-redistributed is broken because the contract function has been renamed. I haven't been able to find any command so far that is broken due to BLS aggregation.

cmjc commented 2 months ago

aut protocol total-redistributed is broken because the contract function has been renamed. I haven't been able to find any command so far that is broken due to BLS aggregation.

OK, great. Neither have I (aut block get returns the quorumCertificate fine.)