kubernetes
/
kubectl
mirror of https://github.com/kubernetes/kubectl.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Clean up nits in delete cascade 

Kubernetes-commit: f5a035a7e66b25c932a82fc8bc7cde195dd05884
This commit is contained in:
zhouya0 2020-09-25 11:38:28 +08:00 committed by Kubernetes Publisher
parent e3458f49bf
commit fd930e3995
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
pkg/cmd/delete/delete_flags.go
View File

@ -26,7 +26,6 @@ import (
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/cli-runtime/pkg/genericclioptions" "k8s.io/cli-runtime/pkg/genericclioptions"
"k8s.io/client-go/dynamic" "k8s.io/client-go/dynamic"
"k8s.io/klog/v2"
) )
// DeleteFlags composes common printer flag structs // DeleteFlags composes common printer flag structs
@ -79,7 +78,7 @@ func (f *DeleteFlags) ToOptions(dynamicClient dynamic.Interface, streams generic
} }
if f.CascadingStrategy != nil { if f.CascadingStrategy != nil {
var err error var err error
options.CascadingStrategy, err = getCascadingStrategy(*f.CascadingStrategy) options.CascadingStrategy, err = parseCascadingFlag(streams, *f.CascadingStrategy)
if err != nil { if err != nil {
return nil, err return nil, err
} }
@ -226,8 +225,8 @@ func NewDeleteFlags(usage string) *DeleteFlags {
} }
} }
func getCascadingStrategy(cascadingFlag string) (metav1.DeletionPropagation, error) { func parseCascadingFlag(streams genericclioptions.IOStreams, cascadingFlag string) (metav1.DeletionPropagation, error) {
b, err := strconv.ParseBool(cascadingFlag) boolValue, err := strconv.ParseBool(cascadingFlag)
// The flag is not a boolean // The flag is not a boolean
if err != nil { if err != nil {
switch cascadingFlag { switch cascadingFlag {
@ -238,14 +237,14 @@ func getCascadingStrategy(cascadingFlag string) (metav1.DeletionPropagation, err
case "background": case "background":
return metav1.DeletePropagationBackground, nil return metav1.DeletePropagationBackground, nil
default: default:
return metav1.DeletePropagationBackground, fmt.Errorf(`Invalid cascade value (%v). Must be "background", "foreground", or "orphan".`, cascadingFlag) return metav1.DeletePropagationBackground, fmt.Errorf(`invalid cascade value (%v). Must be "background", "foreground", or "orphan"`, cascadingFlag)
} }
} }
// The flag was a boolean // The flag was a boolean
if b { if boolValue {
klog.Warningf(`--cascade=%v is deprecated (boolean value) and can be replaced with --cascade=%s.`, cascadingFlag, "background") fmt.Fprintf(streams.ErrOut, "warning: --cascade=%v is deprecated (boolean value) and can be replaced with --cascade=%s.\n", cascadingFlag, "background")
return metav1.DeletePropagationBackground, nil return metav1.DeletePropagationBackground, nil
} }
klog.Warningf(`--cascade=%v is deprecated (boolean value) and can be replaced with --cascade=%s.`, cascadingFlag, "orphan") fmt.Fprintf(streams.ErrOut, "warning: --cascade=%v is deprecated (boolean value) and can be replaced with --cascade=%s.\n", cascadingFlag, "orphan")
return metav1.DeletePropagationOrphan, nil return metav1.DeletePropagationOrphan, nil
} }