TrueCloudLab/frostfs-contract
24
0
Fork 19
Code Issues 16 Pull requests Projects Releases 3 Activity Actions

frostfsid: Remove old data on update #95

Merged
fyrchik merged 2 commits from fyrchik/frostfs-contract:fix-frostfsid-update into support/v0.19 2024-09-04 19:51:18 +00:00
Conversation 0 Commits 2 Files changed 3 +13 -2
3 changed files with 13 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
VERSION
View file

@ -1 +1 @@
v0.19.1 v0.19.3

2
common/version.go
View file

@ -5,7 +5,7 @@ import "github.com/nspcc-dev/neo-go/pkg/interop/native/std"
const ( const (
major = 0 major = 0
minor = 19 minor = 19
patch = 1 patch = 3
// Versions from which an update should be performed. // Versions from which an update should be performed.
// These should be used in a group (so prevMinor can be equal to minor if there are // These should be used in a group (so prevMinor can be equal to minor if there are

11
frostfsid/frostfsid_contract.go
View file

@ -73,6 +73,17 @@ const (
func _deploy(data any, isUpdate bool) { func _deploy(data any, isUpdate bool) {
ctx := storage.GetContext() ctx := storage.GetContext()
if isUpdate {
storage.Delete(ctx, "netmapScriptHash")
storage.Delete(ctx, "containerScriptHash")
dkirillov marked this conversation as resolved Outdated
dkirillov commented 2024-06-04 08:34:26 +00:00
Outdated
Review
Copy link

Should we also delete notary key and o prefix?

Should we also delete `notary` key and `o` prefix?
fyrchik commented 2024-06-04 08:42:20 +00:00
Outdated
Review
Copy link

I believe notary was actually removed during the previous update (see support/v0.18 branch)
Added o prefix removal.

I believe `notary` was actually removed during the previous update (see support/v0.18 branch) Added `o` prefix removal.
dkirillov commented 2024-06-04 09:35:18 +00:00
Outdated
Review
Copy link

I don't see we explicitly deleted such key #50/files

Or we just didn't set it?

I don't see we explicitly deleted such key https://git.frostfs.info/TrueCloudLab/frostfs-contract/pulls/50/files#diff-0854a7a4bf47e1957519dccfa5d54a2c5c8d08ee Or we just didn't set it?
fyrchik commented 2024-06-04 10:54:55 +00:00
Outdated
Review
Copy link

common.RmAndCheckNotaryDisabledKey(data, notaryDisabledKey) -- this line was present in the previous release.

`common.RmAndCheckNotaryDisabledKey(data, notaryDisabledKey)` -- this line was present in the previous release.
👍 1
it := storage.Find(ctx, "o", storage.KeysOnly)
for iterator.Next(it) {
key := iterator.Value(it)
storage.Delete(ctx, key)
}
}
args := data.(struct { args := data.(struct {
admin interop.Hash160 admin interop.Hash160
}) })