5186 commits

This branch

This branch

All branches

Author	SHA1	Message	Date
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
George Bartolomey	0e5524dac7	[#1515] adm: Print address in base58 format in morph ape get-admin ... Signed-off-by: George Bartolomey <george@bh4.ru>	2024-11-25 10:38:05 +00:00
Airat Arifullin	3ebd560f42	[#1519] cli: Make descriptive help for--rule option ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-25 07:21:05 +00:00
Airat Arifullin	1ed7ab75fb	[#1517] cli: Print the reason of ape manager error ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-22 15:24:32 +03:00
Evgenii Stratonikov	99f9e59de9	[#1514] adm: Remove --alphabet-wallets flag from readonly commands ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-21 14:37:34 +00:00
Evgenii Stratonikov	256f96e252	[#1514] adm/nns: Rename getRPCClient() to nnsWriter() ... Make it more specific and similar to nnsReader(). Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-21 14:37:34 +00:00
Evgenii Stratonikov	e5ea95c045	[#1514] adm/nns: Do not return hash from getRPCClient() ... It was unused and we employ better abstractions now. gopatch: ``` @@ var a, b expression @@ -a, b, _ := getRPCClient(...) +a, b := getRPCClient(...) ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-21 14:37:34 +00:00
Evgenii Stratonikov	9073e555db	[#1514] adm/nns: Do not create actor for readonly commands ... `nns get-records` and `nns tokens` command do not need to sign anything, so remove useless actor and use invoker directly. `NewLocalActor()` is only used in `ape` and `nns` packages. `ape` package seem to use it correctly, only when alphabet wallets are provided, so no changes there. Also, remove --alphabet-wallets flag from commands that do not need it. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-21 14:37:34 +00:00
Evgenii Stratonikov	2771fdb8c7	[#1514] adm/nns: Use nns.GetAllRecords() wrapper ... It was not possible previously, because GetAllRecords() was not declared safe in frostfs-contract. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-21 14:37:34 +00:00
Evgenii Stratonikov	efa4ce00b8	[#1514] go.mod: Update frostfs-contract to v0.21.0-rc.3 ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-21 14:37:34 +00:00
Anton Nikiforov	f12f04199e	[#1516] traverser: Check for placement vector out of range ... Placement vector may contain fewer nodes count than it required by policy due to the outage of the one of the node. Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-11-21 14:18:55 +03:00
Evgenii Stratonikov	2e2c62147d	[#1513] adm: Move ProtoConfigPath from constants to commonflags package ... Refs #932 Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-21 09:39:06 +03:00
Evgenii Stratonikov	6ae8667fb4	[#1509] .forgejo: Run actions on push to master ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-20 11:42:12 +03:00
Evgenii Stratonikov	49a4e727fd	[#1507] timer/test: Use const for constants ... Make it easy to see what the test is about. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-20 08:36:25 +00:00
Evgenii Stratonikov	2e974f734c	[#1507] timer/test: Improve test coverage ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-20 08:36:25 +00:00
Evgenii Stratonikov	3042490340	[#1507] timer: Remove unused OnDelta() method ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-20 08:36:25 +00:00
Airat Arifullin	a339b52a60	[#1501] adm: Refactor APE-chains managing subcommands ... * Use `cmd/internal/common/ape` parser commands within `ape` subcommands * Use flag names from `cmd/internal/common/ape Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00
Airat Arifullin	4ab4ed6f96	[#1501] cli: Refactor bearer subcommand ... * Use `cmd/internal/common/ape` parser commands within `generate-ape-override` subcommand * Use flag names from `cmd/internal/common/ape` Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00
Airat Arifullin	3b1364e4cf	[#1501] cli: Refactor ape-manager subcommands ... * Refactor ape-manager subcommands * Use `cmd/internal/common/ape` parser commands within ape-manager subcommands * Use flag names from `cmd/internal/common/ape` Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00
Airat Arifullin	daff77b273	[#1501] cli: Refactor local override managing subcommands ... * Refactor local override managing subcommands * Use `cmd/internal/common/ape` parser commands within local override subcommands * Use flag names from `cmd/internal/common/ape` Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00
Airat Arifullin	7a7ee71a4d	[#1501] cmd: Introduce common APE-chain parser commands ... * Introduce common parsing commands to use them in `frostfs-cli` and `frostfs-adm` APE-related subcommands * Introduce common flags for these parsing commands Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00
Airat Arifullin	ffe9906266	[#1501] cli: Move APE-chain parser methods to pkg/util ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00
Airat Arifullin	ae31ef3602	[#1501] cli: Move PrintHumanReadableAPEChain to a common package ... * Both `frostfs-cli` and `frostfs-adm` APE-related subcommands use `PrintHumanReadableAPEChain` to print a parsed APE-chain. So, it's more correct to have it in a common package over `frostfs-cli` and `frostfs-adm` folders. Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00
Airat Arifullin	e2cb0640f1	[#1501] util: Move eACL-to-APE converter to pkg/util ... * `ConvertEACLToAPE` is useful method which couldn't be imported out of frostfs-node so far as it has been in `internal` * Since `ConvertEACLToAPE` and related structures and unit-tests are placed in `pkg/util` Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-20 07:58:32 +00:00