adm: Add dump policy #783
No reviewers
Labels
No Label
P0
P1
P2
P3
badger
frostfs-adm
frostfs-cli
frostfs-ir
frostfs-lens
frostfs-node
good first issue
triage
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-node#783
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "achuprov/frostfs-node:dump-police"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Close #716
Signed-off-by: Alexander Chuprov a.chuprov@yadro.com
0ef29c4fbe
toac463a4bd2
@ -147,2 +147,4 @@
}
dumpPolicy = &cobra.Command{
Use: "dump-police",
s/police/policy/
fixed × 3x
@ -147,2 +147,4 @@
}
dumpPolicy = &cobra.Command{
Use: "dump-police",
s/police/policy/
@ -147,2 +147,4 @@
}
dumpPolicy = &cobra.Command{
Use: "dump-police",
s/police/policy/
ac463a4bd2
toaafe6206b0
aafe6206b0
to5f1d0d84dd
@ -55,0 +59,4 @@
func dumpPolicyCmd(cmd *cobra.Command, _ []string) error {
c, err := getN3Client(viper.GetViper())
if err != nil {
return fmt.Errorf("can't create N3 client: %w", err)
I don't mind if you return the error here (and below). But I also see you pass
cmd *cobra.Command
to this method.How about to consider using
commonCmd.ExitOnErr(cmd, "your txt", err)
. This allows to exit the program and you do not need to checkif err != nil
:)fixed
5f1d0d84dd
to87d6d8890f
It is better to use output snippets, instead of images in the description.