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

FLIP GOV-291: Removing decommissioned nodes #292

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

FLIP GOV-291: Removing decommissioned nodes #292

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3e78508
Create 20240930-removing-decommisioned-nodes.md
vishalchangrani Sep 30, 2024
9a4b77a
Update 20240930-removing-decommisioned-nodes.md
vishalchangrani Sep 30, 2024
7cbc25f
Update governance/20240930-removing-decommisioned-nodes.md
vishalchangrani Oct 6, 2024
f20207c
adding transaction ID
vishalchangrani Nov 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions governance/20240930-removing-decommisioned-nodes.md
View file
Open in desktop
Copy link
Member

Choose a reason for hiding this comment

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

What transaction are you planning on using for this? I don't see that mentioned anywhere

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Plan to use this transaction: https://github.com/onflow/service-account/blob/main/templates/remove_approved_nodes.cdc

Copy link
Member

Choose a reason for hiding this comment

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

Can you put that in the FLIP?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added link to transaction script and the actual transaction that was submitted

Copy link
Contributor Author

Choose a reason for hiding this comment

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

can you preview review?

Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
---
status: Proposed
flip: GOV-291
authors: Vishal Changrani
sponsors: Vishal Changrani
updated: 2024-09-30
---

# FLIP GOV-291: Removing decommissioned nodes

### Proposal

1. The verification node with node ID `1cf032dd5b3cf9a63943e1f04472bb57cc1240d5cdd86c5fde05b4dabf8b1e7a` is no longer running.
The node was staked through BlockDaemon. The customer needs to unstake the node but is unreachable as per BlockDaemon.
The proposal is to remove the node from the approved list of node IDs via a transaction submitted by the service committee.
This will remove the node from the network in the next epoch.

2. The access node with node ID `6b9038227f6774f632f7481a2d4d82d6da5efc5fbceb1756fbfc590112231a5b` is no longer running.
The node was staked by Metrika who no longer intends to run the node.
Even though access nodes no longer have to be added to the approved list of nodes, this node was added to the network before the launch of permissionless access nodes and therefore had to be added to the approved list back then.

### [Issue](https://github.com/onflow/flips/issues/291)
vishalchangrani marked this conversation as resolved.
Show resolved Hide resolved

### [Forum Post](fourm post: https://forum.flow.com/t/flip-gov-291-remove-two-nodes-from-the-approved-list-of-nodes/6599)