5179 commits

This branch

This branch

All branches

Author	SHA1	Message	Date
Evgenii Stratonikov	b8a3f3bbd9	[#1621] treesvc: Cancel background sync on failure ... If applyOperationStream() exits prematurely, other goroutines will block on send and errgroup will never finish waiting. In this commit we also check whether context is cancelled. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2025-01-31 06:36:13 +00:00
Evgenii Stratonikov	0ea602897b	[#1621] treesvc: Move variable initialization to top in mergeOperationStreams() ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2025-01-31 06:36:13 +00:00
Dmitrii Stepanov	e1c00956c4	[#1627] go.mod: Bump frostfs-sdk-go version ... Contains mem leak fix. Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2025-01-30 16:19:49 +03:00
Dmitrii Stepanov	f43ee57b9b	[#1622] adm: Return non zero return code if metabase upgrade failed ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2025-01-30 15:37:05 +03:00
Evgenii Stratonikov	74e11b3cf3	[#1586] objsvc: Allow to send search response in multiple messages ... Previously, `ln` was only set once, so search has really worked for small number of objects. Fix panic: ``` panic: runtime error: slice bounds out of range [:43690] with capacity 21238 goroutine 6859775 [running]: git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object.(*searchStreamMsgSizeCtrl).Send(0xc001eec8d0, 0xc005734000) git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/transport_splitter.go:173 +0x1f0 git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search/v2.(*streamWriter).WriteIDs(0xc000520320, {0xc00eb1a000, 0x4fd9c, 0x7fd6475a9a68?}) git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search/v2/streamer.go:28 +0x155 git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search.(*uniqueIDWriter).WriteIDs(0xc001386420, {0xc00eb1a000?, 0xc0013ea9c0?, 0x113eef3?}) git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search/util.go:62 +0x202 git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search.(*execCtx).writeIDList(0xc00011aa38?, {0xc00eb1a000?, 0xc001eec9f0?, 0xc0008f4380?}) git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search/exec.go:68 +0x91 git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search.(*execCtx).executeLocal(0xc0008f4380, {0x176c538, 0xc001eec9f0}) git.frostfs.info/TrueCloudLab/frostfs-node/pkg/services/object/search/local.go:18 +0x16b ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-28 09:23:54 +03:00
Evgenii Stratonikov	f527556735	Revert "[#1492] metabase: Ensure Unmarshal() is called on a cloned slice" ... This reverts commit 8ed7a676d5. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-26 16:25:13 +03:00
Evgenii Stratonikov	23b426ec4a	[#1584] go.mod: Update sdk-go ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-26 15:37:21 +03:00
Airat Arifullin	7e3f2fca47	[#1563] tree: Wrap only ChainRouterError erros with ObjectAccessDenied ... * Such wrapping helps to differentiate logical check errors and server internal errors. Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-12-16 16:19:04 +03:00
Airat Arifullin	25ed595354	[#1563] object: Wrap only ChainRouterError erros with ObjectAccessDenied ... * Such wrapping helps to differentiate logical check errors and server internal errors. Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-12-16 16:18:58 +03:00
Airat Arifullin	fb2f40aea1	[#1563] ape: Introduce ChainRouterError error type ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-12-16 16:18:51 +03:00
Dmitrii Stepanov	15b4288d80	[#1548] metabase: Check if EC parent is removed or expired ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-11 10:42:12 +03:00
Dmitrii Stepanov	6ea8f2b23c	[#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 10:41:32 +03:00
Dmitrii Stepanov	47f12f8440	[#1548] policer: Do not replicate EC chunk if object already removed ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-12-11 10:41:00 +03:00
Evgenii Stratonikov	edc1824c23	[#1544] go.mod: Update sdk-go ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-06 12:50:39 +03:00
Airat Arifullin	cf48dfd55e	[#1524] tree: Make check APE error get wrapped to api status ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-12-04 09:57:46 +03:00
Airat Arifullin	274ac61236	[#1524] ape: Make APE checker return error without status ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-12-04 09:57:41 +03:00
Evgenii Stratonikov	892542d6e3	[#1532] node: Allow to omit metabase.path if shard is disabled ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-12-03 13:11:16 +00:00
Anton Nikiforov	af3d6368b0	[#1530] cli: Keep order for required nodes in the result of object nodes ... Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-12-03 11:45:52 +03:00
Anton Nikiforov	36fe470956	[#1530] node: Keep order for equal elements when sort priority metrics ... Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-12-03 11:45:38 +03: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
Evgenii Stratonikov	9f4ce600ac	[#1505] adm: Allow to manage additional keys in frostfsid ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-19 16:25:16 +03:00
Evgenii Stratonikov	d82f0d1926	[#1496] node/control: Await until SetNetmapStatus() persists ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-15 16:36:07 +03:00
Evgenii Stratonikov	acd5babd86	[#1496] morph: Merge InvokeRes and WaitParams ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-15 16:36:07 +03:00
Evgenii Stratonikov	b65874d1c3	[#1496] morph: Return InvokeRes from all invoke*() methods ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-15 16:36:07 +03:00
Evgenii Stratonikov	69c63006da	[#1496] morph: Move tx waiter to morph package ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-15 16:36:07 +03:00
Evgenii Stratonikov	d77a218f7c	[#1493] metabase: Merge Inhume() and DropGraves() for tombstones ... DropGraves() is only used to drop gravemarks after a tombstone removal. Thus, it makes sense to do Inhume() and DropGraves() in one transaction. It has less overhead and no unexpected problems in case of sudden power failure. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-14 06:47:04 +00:00
Evgenii Stratonikov	44df67492f	[#1493] metabase: Split inhumeTx() into 2 functions ... No functional changes. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-11-14 06:47:04 +00:00