Add maintenance zombie commands -> v0.38.6 #1698

Open
dstepanov-yadro wants to merge 3 commits from dstepanov-yadro/frostfs-node:zombie_v0386 into support/v0.38
No description provided.
dstepanov-yadro added 1 commit 2025-03-27 12:54:24 +00:00
[#9999] adm: Add maintenance zombie commands
Some checks failed
DCO action / DCO (pull_request) Successful in 29s
Vulncheck / Vulncheck (pull_request) Failing after 1m10s
Tests and linters / Staticcheck (pull_request) Successful in 1m53s
Tests and linters / Lint (pull_request) Failing after 2m4s
Build / Build Components (pull_request) Successful in 2m31s
Tests and linters / Tests (pull_request) Successful in 2m40s
Tests and linters / Tests with -race (pull_request) Successful in 3m4s
Tests and linters / gopls check (pull_request) Successful in 3m21s
fc4e759ce5
Change-Id: I2e85169849e2b9b7f44817b93283fa2f2b903676
Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>
dstepanov-yadro force-pushed zombie_v0386 from fc4e759ce5 to bb9b598042 2025-03-27 14:48:14 +00:00 Compare
dstepanov-yadro force-pushed zombie_v0386 from bb9b598042 to 1d646fce80 2025-03-27 14:59:49 +00:00 Compare
dstepanov-yadro force-pushed zombie_v0386 from 1d646fce80 to 08a374db1b 2025-03-27 15:02:01 +00:00 Compare
dstepanov-yadro force-pushed zombie_v0386 from 08a374db1b to 4cb32170f3 2025-03-28 08:11:24 +00:00 Compare
dstepanov-yadro changed title from WIP: Add maintenance zombie commands to Add maintenance zombie commands -> v.038.6 2025-03-28 08:17:46 +00:00
requested reviews from storage-core-committers, storage-core-developers 2025-03-28 08:17:46 +00:00
dstepanov-yadro changed title from Add maintenance zombie commands -> v.038.6 to Add maintenance zombie commands -> v0.38.6 2025-03-28 08:17:59 +00:00
dstepanov-yadro force-pushed zombie_v0386 from 4cb32170f3 to 6278986d62 2025-03-28 10:49:02 +00:00 Compare
acid-ant approved these changes 2025-03-28 13:22:40 +00:00
dstepanov-yadro added 1 commit 2025-03-31 09:27:44 +00:00
[#9999] adm: Add dry-run flag to zombie scan command
Some checks failed
DCO action / DCO (pull_request) Successful in 26s
Tests and linters / Lint (pull_request) Failing after 1m38s
Vulncheck / Vulncheck (pull_request) Failing after 1m35s
Build / Build Components (pull_request) Successful in 2m5s
Tests and linters / Staticcheck (pull_request) Successful in 2m57s
Tests and linters / Tests with -race (pull_request) Successful in 3m13s
Tests and linters / Tests (pull_request) Successful in 3m23s
Tests and linters / gopls check (pull_request) Successful in 3m39s
9ca7818fad
This allows to just print object addresses that should be moved to quarantine.

Change-Id: I551979d8bffaf45fe21d92f6edadfaadcb5d6e25
Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>
dstepanov-yadro force-pushed zombie_v0386 from 9ca7818fad to 87c9e53c29 2025-03-31 09:46:30 +00:00 Compare
fyrchik reviewed 2025-03-31 11:13:58 +00:00
@ -0,0 +34,4 @@
if batchSize == 0 {
commonCmd.ExitOnErr(cmd, "invalid batch size: %w", errors.New("batch size must be positive value"))
}
dryRun, _ := cmd.Flags().GetBool(dryRunFlag)
Owner

I believe, there should be an explicit desire to perform destructive changes.
How about asking for confirmation if --dry-run is omitted OR have --yes-i-want-to-really-move-data flag?

I believe, there should be an explicit desire to perform destructive changes. How about asking for confirmation if `--dry-run` is omitted OR have `--yes-i-want-to-really-move-data` flag?
Author
Member

Good point. Replaced dry-run with move flag.

Good point. Replaced `dry-run` with `move` flag.
dstepanov-yadro force-pushed zombie_v0386 from 87c9e53c29 to 1a8acd0731 2025-03-31 11:23:27 +00:00 Compare
dstepanov-yadro force-pushed zombie_v0386 from 1a8acd0731 to e14a17be34 2025-03-31 11:24:42 +00:00 Compare
fyrchik approved these changes 2025-03-31 12:58:49 +00:00
dstepanov-yadro added 1 commit 2025-04-03 08:42:26 +00:00
[#9999] adm: Fix removed container case for zombie scan
Some checks failed
DCO action / DCO (pull_request) Successful in 27s
Vulncheck / Vulncheck (pull_request) Failing after 1m18s
Build / Build Components (pull_request) Successful in 1m29s
Tests and linters / Tests (pull_request) Successful in 2m16s
Tests and linters / Staticcheck (pull_request) Successful in 2m16s
Tests and linters / Lint (pull_request) Successful in 2m29s
Tests and linters / Tests with -race (pull_request) Successful in 3m4s
Tests and linters / gopls check (pull_request) Successful in 3m37s
358de18908
Change-Id: I4452942b3dac87a3bd92421683745f183c300e22
Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>
Some checks failed
DCO action / DCO (pull_request) Successful in 27s
Vulncheck / Vulncheck (pull_request) Failing after 1m18s
Build / Build Components (pull_request) Successful in 1m29s
Tests and linters / Tests (pull_request) Successful in 2m16s
Tests and linters / Staticcheck (pull_request) Successful in 2m16s
Tests and linters / Lint (pull_request) Successful in 2m29s
Tests and linters / Tests with -race (pull_request) Successful in 3m4s
Tests and linters / gopls check (pull_request) Successful in 3m37s
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u zombie_v0386:dstepanov-yadro-zombie_v0386
git checkout dstepanov-yadro-zombie_v0386
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-node#1698
No description provided.