Renew domains before 10Y deadline #932

New Issue

Closed

opened 2024-01-25 10:47:21 +00:00 by alexvanin · 5 comments

alexvanin commented

2024-01-25 10:47:21 +00:00

Owner

In 10 years after initial deployment, all system TLDs will be expired. This will break storage network and existing applications on top of it.

Describe the solution you'd like

Call renew method on all domains owned by committee / consensus before its expiration.

Describe alternatives you've considered

Do nothing and renew domains manually from time to time with some tool (e.g. frostfs-adm command)

Additional context

Some history details:

## Is your feature request related to a problem? Please describe. In [10 years](https://git.frostfs.info/TrueCloudLab/frostfs-contract/commit/1a4fa7e421d73afcf7eb4fa5a3208a3391f071cd) after initial deployment, all system TLDs will be expired. This will break storage network and existing applications on top of it. ## Describe the solution you'd like Call [renew](https://git.frostfs.info/TrueCloudLab/frostfs-contract/src/commit/f2a82aa635aa57d9b05092d8cf15b170b53cc324/nns/nns_contract.go#L337) method on all domains owned by committee / consensus before its expiration. ## Describe alternatives you've considered Do nothing and renew domains manually from time to time with some tool (e.g. frostfs-adm command) ## Additional context Some history details: - https://github.com/nspcc-dev/neofs-contract/pull/296 - https://github.com/nspcc-dev/neofs-contract/issues/268

alexvanin added the

frostfs-ir

good first issue

triage

labels 2024-01-25 10:47:21 +00:00

fyrchik commented

2024-01-25 16:35:07 +00:00

Owner

frostfs-adm is not a viable alternative, because it requires having all private keys in one place.
However, we could extend frostfs-cli control ir with domain renewal as the first step.

Let's discuss whether how it could be done automatically.

Another option is to introduce infinite expiration in the NNS contract (also needs discussion):

Some special mark is stored in the contract (+infinity).
For this mark some default value is always returned (+year).

`frostfs-adm` is not a viable alternative, because it requires having all private keys in one place. However, we could extend `frostfs-cli control ir` with domain renewal as the first step. Let's discuss whether how it could be done automatically. Another option is to introduce infinite expiration in the NNS contract (also needs discussion): 1. Some special mark is stored in the contract (+infinity). 2. For this mark some default value is always returned (+year).

fyrchik added the

discussion

label 2024-01-25 16:35:11 +00:00

fyrchik added this to the vNext milestone 2024-01-25 16:37:35 +00:00

acid-ant self-assigned this 2024-01-29 11:11:08 +00:00

acid-ant commented

2024-01-30 12:15:45 +00:00

Collaborator

Also, it would be nice to have an ability in adm or cli utility register domains.

Also, it would be nice to have an ability in `adm` or `cli` utility register domains.

fyrchik commented

2024-01-30 12:26:36 +00:00

Owner

cli talks with node only, should be in adm

acid-ant commented

2024-02-02 11:42:38 +00:00

Collaborator

In scope of this task also necessary to use rpclicent package from frostfs-contract.

In scope of this task also necessary to use `rpclicent` package from `frostfs-contract`.

acid-ant referenced this issue

2024-02-02 11:44:22 +00:00

adm: Refactor `morph` command #950

fyrchik commented

2024-02-13 07:06:47 +00:00

Owner

Separate package for

nns helpers (contract names, domainOf, hash resolving)
polling (next poll interval -- could be private, awaitTx etc)
sendConsensusTx etc. in another package
separate packages for client

Separate package for 1. nns helpers (contract names, `domainOf`, hash resolving) 2. polling (next poll interval -- could be private, awaitTx etc) 3. sendConsensusTx etc. in another package 4. separate packages for client

acid-ant referenced this issue from a commit

2024-02-13 11:43:51 +00:00

[#932] adm: Refactor command `morph`

acid-ant referenced this issue from a commit

2024-02-13 11:43:51 +00:00

[#932] adm: Move `NNS` methods to `util` package

acid-ant referenced this issue from a commit

2024-02-13 11:43:51 +00:00

[#932] adm: Move `alphabet-wallets` flag to `util` package

acid-ant referenced this issue from a commit

2024-02-13 11:43:51 +00:00

[#932] adm: Move `DomainOf` to `util` package