An SVG of an eye

Proposal overview

Rewards controller update across V3 pools

Executed

Executed on 

Dec 25, 2022

Simple Summary

This proposal updates the rewardsController to it's newest version improving the DX by adding new getters and resolving some minor issues:

  1. soften solidity versions to ^0.8.0 here
  2. adding a getter to fetch the current index here
  3. replacing totalSupply with totalScaledSupply on setEmissionPerSecond here
  4. pumping revision from 1 to 2 here
  5. fixing getUserReward to return correct rewards for users with 0 balance here
  6. remove unnecessary constructor that sets state on a contract always used behind a proxy here
  7. consistently uppercase license pragma agpl->AGPL here
  8. improving documentation on various structs here
  9. removing constructor of RewardsDistributor and RewardsController here

Motivation

The Aave v3 codebase is divided into 2 different parts, Aave v3 core, and Aave v3 periphery. While usually the concept of “protocol” is more related to the Aave v3 core, there are some complementary components of Aave v3 living on the periphery codebase, for example, the system managing external teams configuring rewards for Aave v3 activity (e.g. supply/borrow assets).

Similar to the majority of other Aave smart contracts, the ones on the periphery are upgradeable and controlled by the Aave governance, which means they are designed to be improved over time.

Given that the community has approved a new Aave v3 Ethereum, we have used the occasion to introduce light improvement/fixes on the periphery smart contracts and now will submit the upgrade to be approved via governance and it only makes sense to apply them to already existing v3 pools.

Specification

This proposal executes PoolAddressesProvider.setAddressAsProxy(keccak256("INCENTIVES_CONTROLLER"), rewardsControllerImpl) on Polygon V3 pool.

If this proposal succeeds, this will be seen as positive signaling to upgrade implementations for v3 pools on networks controlled by guardians as well.

The RewardsController contracts are deployed and initialized already:

Verification on harmony & fantom does not work due to bugs in the respective explorers. You can verify though that bytecode & initialize calls are 100% the same between the different networks.

For guardian controlled pools the transactions are executed via gnosis, so instead of deploying a payload the guardians will just sign the setAddressAsProxy(keccak256("INCENTIVES_CONTROLLER"), rewardsControllerImpl) transactions.

The pre-encoded calldata to be executed by guardians on the respective PoolAddressesProvider:

  • 0x5dcc528c0000000000000000000000005f4d15d761528c57a5c30c43c1dab26fc5452731

References

Copyright

Copyright and related rights waived via CC0.

Your voting info

Voting results

YAE

566,382

AAVE

100.00%

NAY

<1

AAVE

<0.01%

Top 10 addresses

Votes

State
Executed

Executed on 

Dec 25, 2022
Quorum
Reached
Current votes

Required

566.38K

320.00K

Differential
Reached
Current differential

Required

566.38K

80,000.00

Total voting power

16,000,000

Proposal details

Created

Block

20 Dec 2022, 17:43 UTC +00:00

16227478

Started

Block

21 Dec 2022, 17:50 UTC +00:00

16234678

Ended

Block

24 Dec 2022, 09:50 UTC +00:00

16253878

Executed

25 Dec 2022, 12:52 UTC +00:00

Author

BGD Labs (@bgdlabs)