Prysm: Allow Static ENRs to Be Created

[nisdas]

🚀 Feature Request

Description

Currently the ENRs created by a Beacon Node are dynamic, in the sense that they can constantly
change with new information added to it. For the purposes of running a bootnode/static node using a normal Prysm Node this is undesirable as we would have ENRs which are changing every few minutes. For this reason there should be a flag to allow static ENRs to be created.

Describe the solution you'd like

Static ENRs would require discoveryV5 to stop endpoint prediction . Also new information should not be written to ENRs once the ENR has been instantiated. Another Action Item for this issue would be to allow the host ip address to be the only address added to an ENR in the case the flag is defined (do not add in the internal ip to the ENR).

Describe alternatives you've considered

N.A

[prestonvanloon]

Can you elaborate on the use case? In what environment is this functionality desireable?

[nisdas]

Mostly, if you want to run a Prysm Node as a bootnode. The prysm node's enr constantly changes which is undesirable for a bootnode. Our constantly changing bitfield affects the ENR, so we can disable it for a user who desires a static ENR

[djrtwo]

Curious, do you change the bitfield just for the long-term subnet subscriptions or do you change per-epoch based on beacon committee subscriptions?

[djrtwo]

just caught up in the discord

Going to add this clarifying comment to the spec!

"Note: Short lived beacon committee assignments should not translate into updating attnets field in the ENR"

[shayzluf]

@nisdas seems like this change will not be specific but will reflect in all cases .
spec is going to change if i understand right so i am adding blocked label to the issue . please remove if its not blocked by spec update.
can you please update the issue to description?

[nisdas]

This doesn't really have anything to do with the spec. Its just a matter of keeping the ENR fixed upon initalization, not accounting for any additional information added to it.

[shayzluf]

@nisdas thanks

[rauljordan]

Is this still relevant @nisdas ?