adm: Exclude group.frostfs key from output of the dump-hashes #127

acid-ant · 2023-03-09T12:55:34Z

acid-ant commented

2023-03-09 12:55:34 +00:00

Close #126

Signed-off-by: Anton Nikiforov an.nikiforov@yadro.com

Close #126 Signed-off-by: Anton Nikiforov <an.nikiforov@yadro.com>

acid-ant requested review from fyrchik 2023-03-09 12:55:53 +00:00

acid-ant requested review from carpawell 2023-03-09 12:55:53 +00:00

acid-ant requested review from dstepanov-yadro 2023-03-09 12:55:54 +00:00

acid-ant requested review from ale64bit 2023-03-09 12:55:54 +00:00

dstepanov-yadro approved these changes 2023-03-09 13:25:17 +00:00

fyrchik reviewed 2023-03-09 14:45:13 +00:00

cmd/frostfs-adm/internal/modules/morph/dump_hashes.go Outdated

													
				@ -124,3 +125,3 @@

						}

						if !bytes.HasSuffix(bs, []byte(zone)) {

						if !bytes.HasSuffix(bs, []byte(zone)) || bytes.HasSuffix(bs, []byte(morphClient.NNSGroupKeyName)) {

fyrchik commented

2023-03-09 14:45:13 +00:00

May be HasPrefix?

May be `HasPrefix`?

acid-ant commented

2023-03-09 15:30:20 +00:00

Poster

oops, you right, changed.

acid-ant force-pushed bugfix/126-dump-hashes from 75d9f1152f to 9929dcf50b

2023-03-09 15:29:55 +00:00

Compare

carpawell approved these changes 2023-03-09 17:27:31 +00:00

fyrchik approved these changes 2023-03-09 20:26:52 +00:00

fyrchik merged commit 9929dcf50b into master

2023-03-09 20:26:59 +00:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

4 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#127

adm: Exclude group.frostfs key from output of the dump-hashes #127

adm: Exclude `group.frostfs` key from output of the `dump-hashes` #127