5209 commits

This branch

This branch

All branches

Author	SHA1	Message	Date
Vitaliy Potyarkin	a641c91594	[#1550] Add CODEOWNERS ... Signed-off-by: Vitaliy Potyarkin <v.potyarkin@yadro.com>	2024-12-11 10:34:57 +00:00
Evgenii Stratonikov	b1614a284d	[#1546] morph/event: Export NotificationHandlerInfo fields ... Hiding them achieves nothing, as the struct has no methods and is not used concurrently. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-11 07:39:49 +00:00
Evgenii Stratonikov	d0ce835fbf	[#1546] morph/event: Merge notification parser and handlers ... They are decoupled, but it is an error to have a handler without a corresponding parser. Register them together on the code level and get rid of unreachable code. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-11 07:39:49 +00:00
Evgenii Stratonikov	dfa51048a8	[#1546] morph/event: Remove "is started" checks from event handler registrar ... This codepath hides possible bugs in code. All initialization function should run before init stage. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-11 07:39:49 +00:00
Evgenii Stratonikov	670305a721	[#1546] morph/event: Remove nil checks from event handler registrar ... This codepath hides possible bugs in code. We would rather panic then silently fail. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-11 07:39:49 +00:00
Dmitrii Stepanov	1f6cf57e30	[#1548] metabase: Check if EC parent is removed or expired ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-11 07:26:33 +00:00
Dmitrii Stepanov	386a12eea4	[#1548] engine: Rename parent -> ecParent ... Parent could mean split parent or EC parent. In this case it is EC parent only. Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-11 07:26:33 +00:00
Dmitrii Stepanov	15139d80c9	[#1548] policer: Do not replicate EC chunk if object already removed ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-11 07:26:33 +00:00
Dmitrii Stepanov	41da27dad5	[#1549] engine: Drop Async flag from evacuation parameters ... Now it is only async evacuation. Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-10 17:00:00 +03:00
Dmitrii Stepanov	ac0511d214	[#1549] controlSvc: Drop deprecated EvacuateShard rpc ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-10 16:59:52 +03:00
Airat Arifullin	7e542906ef	[#1539] go.mod: Bump frostfs-sdk-go version ... * Also fix placement unit-test in object manager Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-12-06 15:29:37 +03:00
Evgenii Stratonikov	d1bc4351c3	[#1545] morph/event: Simplify frostfs contract event parsing ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-06 14:00:23 +03:00
Evgenii Stratonikov	1c12f23b84	[#1541] morph/event: Simplify netmap contract event parsing ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-06 10:26:39 +00:00
Evgenii Stratonikov	a353d45742	[#1541] morph/event: Simplify container contract event parsing ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-06 10:26:39 +00:00
Evgenii Stratonikov	d5c46d812a	[#1541] go.mod: Update frostfs-contract ... New version contains more idiomatic types in the auto-generated code. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-06 10:26:39 +00:00
Evgenii Stratonikov	d5d5ce2074	[#1541] morph/event: Simplify balance contract event parsing ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-06 10:26:39 +00:00
Dmitrii Stepanov	7df3520d48	[#1540] getSvc: Drop redundant returns ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-05 12:39:49 +00:00
Dmitrii Stepanov	5fe78e51d1	[#1540] getSvc: Do not log context canceled errors during EC assemble ... Those errors are fired when it is enough chunks retrieved and error group cancels other requests. Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-05 12:39:49 +00:00
Evgenii Stratonikov	84b4051b4d	[#1538] morph/container: Make opts struct similar to that of other contracts ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 15:30:58 +03:00
Evgenii Stratonikov	6a51086030	[#1538] morph/client: Remove TryNotary() option from side-chain contracts ... The notary is always enabled and this option does always work. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 15:30:58 +03:00
Evgenii Stratonikov	5c3b2d95ba	[#1538] node: Assume notary is enabled ... Notaryless environments are not tested at all since a while. We use neo-go only and it has notary contract enabled. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 15:30:58 +03:00
Evgenii Stratonikov	2d5d4093be	[#1537] morph: Use (user.ID).ScriptHash() where possible ... Pick up changes from TrueCloudLab/frostfs-sdk-go#198. gopatch: ``` @@ var user expression @@ -address.StringToUint160(user.EncodeToString()) +user.ScriptHash() ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 13:25:44 +03:00
Evgenii Stratonikov	e3487d5af5	[#1535] morph: Unify test invoke error messages ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 09:50:20 +00:00
Evgenii Stratonikov	e37dcdf88b	[#1535] morph/netmap: Unify error messages for config retrieval ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 09:50:20 +00:00
Evgenii Stratonikov	6c679d1535	[#1535] morph: Unify client creation error messages ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 09:50:20 +00:00
Aleksey Savchuk	281d65435e	[#1450] engine: Group object by shard before Inhume ... ``` goos: linux goarch: amd64 pkg: git.frostfs.info/TrueCloudLab/frostfs-node/pkg/local_object_storage/engine cpu: 12th Gen Intel(R) Core(TM) i5-1235U │ old.txt │ new.txt │ │ sec/op │ sec/op vs base │ InhumeMultipart/objects=1-12 11.42m ± 1% 10.71m ± 0% -6.27% (p=0.000 n=10) InhumeMultipart/objects=10-12 113.5m ± 0% 100.9m ± 3% -11.08% (p=0.000 n=10) InhumeMultipart/objects=100-12 1135.4m ± 1% 681.3m ± 2% -40.00% (p=0.000 n=10) InhumeMultipart/objects=1000-12 11.358 ± 0% 1.089 ± 1% -90.41% (p=0.000 n=10) InhumeMultipart/objects=10000-12 113.251 ± 0% 1.645 ± 1% -98.55% (p=0.000 n=10) geomean 1.136 265.5m -76.63% ``` Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>	2024-12-04 10:09:00 +03:00
Aleksey Savchuk	b348b20289	[#1450] engine: Add benchmark for Inhume operation ... Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>	2024-12-04 10:08:34 +03:00
Aleksey Savchuk	748edd1999	[#1450] engine: Return shard-level error if object is expired on inhume ... Since we have errors defined on the shard-level, it looks strage that we check an error againt the shard-level error `ErrLockObjectRemoval`, but then return the metabase-level error. Let's return the same shard-level error instead. Since we have errors defined on the shard-level Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>	2024-12-04 10:06:57 +03:00
Evgenii Stratonikov	47dfd8840c	[#1532] node: Allow to omit metabase.path if shard is disabled ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-04 03:30:19 +00:00
Aleksey Savchuk	432042c534	[#1527] engine: Add tests for handling expired objects on inhume and lock ... Currently, it's allowed to inhume or lock an expired object. Consider the following scenario: 1) An user inhumes or locks an object 2) The object expires 3) GC hasn't yet deleted the object 4) The node loses the associated tombstone or lock 5) Another node replicates tombstone or lock to the first node In this case, the second node succeeds, which is the desired behavior. Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>	2024-12-03 12:29:45 +03:00
Aleksey Savchuk	9cabca9dfe	[#1527] engine/test: Move default metabase options to separate function ... Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>	2024-12-02 16:13:37 +03:00
Aleksey Savchuk	60feed3b5f	[#1527] engine/test: Allow to specify current epoch in epochState ... Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>	2024-12-02 15:37:25 +03:00
Anton Nikiforov	635a292ae4	[#1528] cli: Keep order for required nodes in the result of object nodes ... Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-12-02 13:58:24 +03:00
Anton Nikiforov	edfa3f4825	[#1528] node: Keep order for equal elements when sort priority metrics ... Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-12-02 13:58:19 +03:00
Aleksey Savchuk	e0ac3a583f	[#1523] metabase: Remove (*DB).IterateCoveredByTombstones ... Remove this method because it isn't used anywhere since 7799f8e4c. Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>	2024-11-29 10:49:24 +00:00
Airat Arifullin	00c608c05e	[#1524] tree: Make check APE error get wrapped to api status ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-29 10:48:16 +00:00
Airat Arifullin	bba1892fa1	[#1524] ape: Make APE checker return error without status ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-29 10:48:16 +00:00
Evgenii Stratonikov	01acec708f	[#1525] pilorama: Use AppendUint* helpers from stdlib ... gopatch: ``` @@ var slice, e expression @@ +import "encoding/binary" -append(slice, byte(e), byte(e >> 8)) +binary.LittleEndian.AppendUint16(slice, e) ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-28 09:40:20 +03:00
Evgenii Stratonikov	aac65001e5	[#1522] adm/frostfsid: Remove unreachable condition ... SendConsensusTx() modifies SendTxs field, if it is not the case, there is a bug in code. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	1170370753	[#1522] adm/helper: Rename createSingleAccounts() -> getSingleAccounts() ... It doesn't create any accounts, purely finds them in the wallet. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	9e275d44c8	[#1522] adm/helper: Unexport DefaultClientContext() ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	2469e0c683	[#1522] adm/helper: Remove NewActor() helper ... It is used once, it is used only internally and it is single-statement. I see no justification in having it as a separate function. It introduces confusion, because we also have NewLocalActor(). Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	a6ef4ab524	[#1522] adm/helper: Rename GetN3Client() -> NewRemoteClient() ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	49959c4166	[#1522] adm/helper: Unexport GetFrostfsIDAdmin() ... It is used in `helper` package only, besides unit-tests. Move unit-tests to the same package, where they belong. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	61ee1b5610	[#1522] adm: Simplify LocalClient.SendRawTransaction() ... The old code was there before Copy() method was introduced. It was also supposed to check errors, however, they are already checked server-side. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	b10c954377	[#1522] adm: Split NewLocalClient() into functions ... No functional changes. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	1605391628	[#1522] adm/helper: Simplify Client interface ... Just reuse `actor.RPCActor`. No functional changes. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	b1766e47c7	[#1522] adm/helper: Remove unused GetCommittee() method from the Client interface ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	caa4253249	[#1522] adm: Remove unnecessary variable declaration ... It is better to have small scope. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-26 08:13:35 +00:00
Evgenii Stratonikov	7eac5fb18b	Release v0.44.0 ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-25 14:41:36 +03:00