TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 105 Pull requests 13 Projects Releases 2 Activity Actions

metrics: Fix container_size_bytes for EC #1318

Merged
fyrchik merged 3 commits from acid-ant/frostfs-node:bugfix/ec-fix-container-size-b into master 2024-09-04 19:51:11 +00:00
Conversation 0 Commits 3 Files changed 3 +1
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 8319b59238 - Show all commits

1
pkg/services/object/metrics.go
View file

@ -213,6 +213,7 @@ func (s putStreamMetric) CloseAndRecv(ctx context.Context) (*object.PutResponse,
return res, err
}
fyrchik commented 2024-08-19 11:14:44 +00:00
Outdated
Review
Copy link

unacceptable

unacceptable
fyrchik commented 2024-08-19 11:15:46 +00:00
Outdated
Review
Copy link

ah, gofumpt
on a side-note -- we have added it in CI, doesn't it work now?
It seems this was introduced in patch, everything was green there.

ah, gofumpt on a side-note -- we have added it in CI, doesn't it work now? It seems this was introduced in patch, everything was green there.
func (s patchStreamMetric) Send(ctx context.Context, req *object.PatchRequest) error {
s.metrics.AddPayloadSize("Patch", len(req.GetBody().GetPatch().GetChunk()))