Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Vote 2023-11-07 #190

Merged
merged 27 commits into from
Nov 10, 2023
Merged

Vote 2023-11-07 #190

merged 27 commits into from
Nov 10, 2023

Conversation

kovalgek
Copy link
Contributor

@kovalgek kovalgek commented Nov 7, 2023 •
edited
Loading

I. stETH transfers to RCC PML ATC

  1. Transfer 272 stETH to RCC 0xDE06d17Db9295Fa8c4082D4f73Ff81592A3aC437
  2. Transfer 434 stETH to PML 0x17F6b2C738a63a8D3A113a228cfd0b373244633D
  3. Transfer 380 stETH to ATC 0x9B1cebF7616f2BC73b47D226f90b01a7c9F86956

II. Change the on-chain name of node operator with id 27 from 'Prysmatic Labs' to 'Prysm Team at Offchain Labs'

  1. Grant NodeOperatorsRegistry.MANAGE_NODE_OPERATOR_ROLE to voting
  2. Change node operator name from Prysmatic Labs to Prysm Team at Offchain Labs
  3. Revoke MANAGE_NODE_OPERATOR_ROLE from Voting

Vote passed & executed on Nov-10-2023 02:43:23 PM +UTC, block 18542216
https://etherscan.io/tx/0x9c72c967a05268406a34f45d25538b7a0e7f7fed2ea8857ad1f3db7a3c9de146

kovalgek and others added 22 commits October 20, 2023 13:17
@kovalgek
add draft script and tests to remove 3 easy track factories for RCC, …
13ef4db 
…PML, ATC
@kovalgek
add script and tests for new factories but with old addresses
a24f584
@kovalgek
add usdt and usdc permissions
a68403d
@TheDZhon
test: fix negative report test cases
13af5d4 
The operators setup has been changed drastically.
Node Operator #1 is fully exited ATM.
@TheDZhon
test: fix vebo and ao edge cases
eb4ebf5
@kovalgek
add balance limit tests
6bcefd5
@kovalgek
check registry limits
7e3a825
@kovalgek
change node operator name
7a24f24
@kovalgek
add stETH to registries
b5074e5
@kovalgek
refactoring + remove node renaming
7fd8f52
@kovalgek
fixes: additional check for registiries, steth from agent to rcc, pml…
4604658 
…, atc tests
@kovalgek
update fabric addresses, fix tests
f8c36ea
@kovalgek
update description
e97169a
@kovalgek
remove factories, add node renaming
9c05fea
@kovalgek
update description, fix tests
36dc722
@kovalgek
fix tests for correct events when forward agent node renaming
616b3b3
@kovalgek
add node operator number
4bb7951
@kovalgek
update steth amounts
58335cd
@bulbozaur
update timeout gas limit
4923cf6
@bulbozaur
fix gas limit
c8ab7ff
@kovalgek
reject and archive voting
92904d3
@kovalgek
add new voting with node 27 renaming and top up ballance of RCC, PML,…
66e0be4 
… ATC
@kovalgek kovalgek marked this pull request as ready for review November 7, 2023 11:33
bulbozaur
bulbozaur approved these changes Nov 10, 2023
View reviewed changes
Copy link
Contributor

@bulbozaur bulbozaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bulbozaur bulbozaur merged commit 41e8e53 into master Nov 10, 2023
1 check passed
@bulbozaur bulbozaur deleted the vote_2023_11_07 branch November 10, 2023 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bulbozaur bulbozaur bulbozaur approved these changes

@zuzueeka zuzueeka Awaiting requested review from zuzueeka

@TheDZhon TheDZhon Awaiting requested review from TheDZhon

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kovalgek @bulbozaur @TheDZhon