TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 101 Pull requests 6 Projects Releases 1 Activity Actions

[#9999] shard: Check storage id on delete
Some checks failed
DCO action / DCO (pull_request) Successful in 1m33s
Details
Vulncheck / Vulncheck (pull_request) Successful in 2m7s
Details
Tests and linters / Lint (pull_request) Failing after 2m15s
Details
Tests and linters / Staticcheck (pull_request) Successful in 2m14s
Details
Build / Build Components (pull_request) Successful in 2m27s
Details
Tests and linters / gopls check (pull_request) Successful in 2m58s
Details
Tests and linters / Tests (pull_request) Successful in 3m18s
Details
Tests and linters / Tests with -race (pull_request) Successful in 3m49s
Details

Browse source 
Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>
This commit is contained in:
Dmitrii Stepanov 2024-11-05 16:05:23 +03:00
parent c06ab2595b
commit 718f2bb5d6
Signed by: dstepanov-yadro
GPG key ID: 237AF1A763293BC0
1 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
pkg/local_object_storage/shard/delete.go
View file

@ -117,6 +117,12 @@ func (s *Shard) deleteFromBlobstor(ctx context.Context, addr oid.Address) error
return err
}
storageID := res.StorageID()
if storageID == nil {
// if storageID is nil it means:
// 1. there is no such object
// 2. object stored by writecache: should not happen, as `validateWritecacheDoesntContainObject` called before `deleteFromBlobstor`
return nil
}
var delPrm common.DeletePrm
delPrm.Address = addr