215 commits

Author	SHA1	Message	Date
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
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
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	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
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	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
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
Airat Arifullin	d336f2d487	[#1393] adm: Make NewLocalActor receive accout name ... * Some RPC-clients for contracts require different wallet account types. Since, `Policy` contract gets `consensus` accounts while `NNS` gets `committee` accounts. Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-11-08 13:57:51 +00:00
Evgenii Stratonikov	7ac0852364	[#1459] .golangci.yml: Add intrange linter, fix issues ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-10-30 15:18:22 +00:00
Alexander Chuprov	07ce40e119	[#1430] adm/morph: Add NNS address display in 'deploy' ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-10-15 13:56:37 +00:00
Alexander Chuprov	42bf03e5cc	[#1411] adm/nns: Add 'delRecord' ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-10-11 11:33:56 +03:00
Evgenii Stratonikov	94302235d0	[#1425] adm: Remove eACL fetching from dump-containers ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-10-09 10:56:48 +03:00
Evgenii Stratonikov	4190fba86d	[#1425] Remove SetEACL-related code ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-10-09 10:56:48 +03:00
Dmitrii Stepanov	8093e145b3	[#1412] adm: Resolve container type by metabase upgrade ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-10-08 11:41:51 +03:00
Evgenii Stratonikov	f83f7feb8c	[#1391] adm: Properly check whether transfers were made ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-10-02 11:41:33 +03:00
Evgenii Stratonikov	62028cd7ee	[#1409] adm: Uncommonize DeltaFlag ... It is used only in `force-new-epoch`, it is not _common_ between multiple commands. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-10-02 11:41:27 +03:00
Evgenii Stratonikov	f45e75e3eb	[#1409] adm: Do not bind DeltaFlag to viper ... We bind flag that could be specified in config. This is not a config flag, just a command option. Also fix TestInitialize failures: ``` Error: Received unexpected error: number of epochs cannot be less than 1 Test: TestInitialize/16_nodes/force-new-epoch ``` Refs #1372 (945b7c740b) Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-10-02 11:41:22 +03:00
Anton Nikiforov	7f8a1dcf8e	[#1400] adm: Support flag alphabet-wallets for commands proxy-add/remove-account ... Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-09-30 14:15:13 +03:00
Vitaliy Potyarkin	f71418b73c	[#1386] frostfs-adm: Add info to error messages ... These error messages bubble up to human users - adding more context helps to find the cause of the issue faster. Signed-off-by: Vitaliy Potyarkin <v.potyarkin@yadro.com>	2024-09-23 07:47:03 +00:00
Alexander Chuprov	53a90634fc	[#1301] adm/morph: Add 'delete' domains ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-09-20 11:54:18 +03:00
Alexander Chuprov	1361db91ee	[#1301] adm/morph: Add flag -v to 'Tokens' ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-09-20 11:54:18 +03:00
Alexander Chuprov	945b7c740b	[#1372] adm/morph: Add delta flag to 'force-new-epoch' ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-09-20 11:42:04 +03:00
Vitaliy Potyarkin	ec8da40567	[#1369] Update obsolete URLs ... Signed-off-by: Vitaliy Potyarkin <v.potyarkin@yadro.com>	2024-09-11 11:31:10 +00:00
Dmitrii Stepanov	654d970fad	[#1355] adm: Run metabase upgrade concurrently ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-09-06 15:54:40 +03:00
Ekaterina Lebedeva	a685fcdc96	[#1317] go.mod: Use range over int ... Since Go 1.22 a "for" statement with a "range" clause is able to iterate through integer values from zero to an upper limit. gopatch script: @@ var i, e expression @@ -for i := 0; i <= e - 1; i++ { +for i := range e { ... } @@ var i, e expression @@ -for i := 0; i <= e; i++ { +for i := range e + 1 { ... } @@ var i, e expression @@ -for i := 0; i < e; i++ { +for i := range e { ... } Signed-off-by: Ekaterina Lebedeva <ekaterina.lebedeva@yadro.com>	2024-09-03 13:00:54 +03:00
Dmitrii Stepanov	6c2146bbc1	[#1334] metabase: Add upgrade from v2 to v3 ... Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>	2024-08-30 09:06:20 +03:00
Evgenii Stratonikov	03976c6ed5	[#1341] .golangci.yml: Replace exportloopref with copyloopvar ... exportloopref is deprecated. gopatch: ``` @@ var index, value identifier var slice expression @@ for index, value := range slice { ... -value := value ... } @@ var index, value identifier var slice expression @@ for index, value := range slice { ... -index := index ... } @@ var value identifier var channel expression @@ for value := range channel { ... -value := value ... } ``` Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-08-28 15:44:41 +00:00
Airat Arifullin	85a77b7c21	[#1279] adm: Interpret "root" name as empty for namespace target type ... Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>	2024-07-29 12:57:40 +00:00
Alexander Chuprov	08953a2f94	[#1239] adm/morph: Fix transfer GAS to alphabet nodes ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-07-11 18:28:32 +03:00
Alexander Chuprov	36956db123	[#1159] adm/frostfsid: Remove wallet requirement for list-* ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-07-11 14:04:37 +00:00
Alexander Chuprov	74842e7f43	[#1210] adm: Fix error handling when contract not found ... Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>	2024-07-02 13:03:40 +00:00
Ekaterina Lebedeva	56eeb630b6	[#1217] Fix grammar mistakes and misspelling ... Signed-off-by: Ekaterina Lebedeva <ekaterina.lebedeva@yadro.com>	2024-07-01 19:14:25 +03:00
Anton Nikiforov	36eab4059c	[#218] adm: Refactor helper in part of reading alphabet wallets ... Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-07-01 06:56:02 +00:00
Anton Nikiforov	72ab373b71	[#218] adm: Update doc for morph generate-alphabet ... Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>	2024-07-01 06:56:02 +00:00
Evgenii Stratonikov	a0e49fa5a5	[#1170] adm: Support morph mTLS ... Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>	2024-06-14 09:27:06 +03:00