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.

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

cli: Add warnings to ETH/LPT accounts #2127

Merged
merged 3 commits into from
Dec 9, 2021

Conversation

leszko
Copy link
Contributor

@leszko leszko commented Dec 7, 2021

What does this pull request do? Explain your changes. (required)
Add warning to ETH/LPT accounts to make sure users won't send their currencies into these addresses.

Specific updates (required)

  • Add FOR REFERENCE - DO NOT SEND TOKENS to all contract addresses
  • Change ETH account to Broadcaster Account (Orchestrator Account)
  • Add YOUR WALLET FOR ETH & LPT to Broadcaster/Orchestrator Account

Here's how the whole node stats look like right now:

+-----------+
|NODE STATS|
+-----------+
*------------------------------------*--------------------------------------------*
|                     Node's version |                      0.5.22-d7c75365-dirty |
*------------------------------------*--------------------------------------------*
|          Node's GO runtime version |                                   go1.17.3 |
*------------------------------------*--------------------------------------------*
|                Node's architecture |                                      arm64 |
*------------------------------------*--------------------------------------------*
|            Node's operating system |                                     darwin |
*------------------------------------*--------------------------------------------*
|                          HTTP Port |                                       7935 |
*------------------------------------*--------------------------------------------*
|                 Controller Address | 0x77A0865438f2EfD65667362D4a8937537CA7a5EF |
| FOR REFERENCE - DO NOT SEND TOKENS |                                            |
*------------------------------------*--------------------------------------------*
|              LivepeerToken Address | 0xb822E7f01dA0A84b5E0df314B05A7f20F95a2b34 |
| FOR REFERENCE - DO NOT SEND TOKENS |                                            |
*------------------------------------*--------------------------------------------*
|        LivepeerTokenFaucet Address | 0x1c77a83Eb9c2f646aF722F44bb1D5dB97eE92E56 |
| FOR REFERENCE - DO NOT SEND TOKENS |                                            |
*------------------------------------*--------------------------------------------*
|                Broadcaster Account | 0xE9CC42d8e4885c29118B209a9FD2e775Fb234036 |
|          YOUR WALLET FOR ETH & LPT |                                            |
*------------------------------------*--------------------------------------------*
|                        LPT Balance |                                     0 LPTU |
*------------------------------------*--------------------------------------------*
|                        ETH Balance |                 633.999891797999242586 ETH |
*------------------------------------*--------------------------------------------*
|                      Max Gas Price |                                        n/a |
*------------------------------------*--------------------------------------------*
|                      Min Gas Price |                                     0 GWei |
*------------------------------------*--------------------------------------------*
+-----------------+
|BROADCASTER STATS|
+-----------------+
*-------------------------------*-------------------------------*
|           Max Price Per Pixel |                           n/a |
*-------------------------------*-------------------------------*
| Broadcast Transcoding Options | P240p30fps16x9,P360p30fps16x9 |
*-------------------------------*-------------------------------*
|                       Deposit |                       100 ETH |
*-------------------------------*-------------------------------*
|                       Reserve |                       100 ETH |
*-------------------------------*-------------------------------*
+---------------+
|DELEGATOR STATS|
+---------------+
*------------------*--------------------------------------------*
|           Status |                                   Unbonded |
*------------------*--------------------------------------------*
|            Stake |                                     0 LPTU |
*------------------*--------------------------------------------*
|   Collected Fees |                                      0 WEI |
*------------------*--------------------------------------------*
|    Pending Stake |                                     0 LPTU |
*------------------*--------------------------------------------*
|     Pending Fees |                                      0 WEI |
*------------------*--------------------------------------------*
|  Delegated Stake |                                     0 LPTU |
*------------------*--------------------------------------------*
| Delegate Address | 0x0000000000000000000000000000000000000000 |
*------------------*--------------------------------------------*
| Last Claim Round |                                          0 |
*------------------*--------------------------------------------*
|      Start Round |                                          0 |
*------------------*--------------------------------------------*
CURRENT ROUND: 83

How did you test each of these updates (required)
Executed livepeer_cli locally and checked stats

Does this pull request close any open issues?
fix #2120

Checklist:

Copy link
Member

@yondonfu yondonfu left a comment

Choose a reason for hiding this comment

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

LGTM!

@leszko leszko merged commit bf1c025 into livepeer:master Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clear warning on CLI Address List
3 participants