make output message more concise

pkg/cmd/admin/policy/modify_roles.go

@@ -422,12 +422,11 @@ existingLoop:
 
 // prints affirmative output for role modification commands
 func printSuccessForCommand(role string, didAdd bool, targetName string, targets []string, isNamespaced bool, out io.Writer) {
-	preposition := "from"
 	verb := "removed"
-	clusterScope := " cluster "
+	clusterScope := "cluster "
 	allTargets := fmt.Sprintf("%q", targets)
 	if isNamespaced {
-		clusterScope = " "
+		clusterScope = ""
 	}
 	if len(targets) > 1 {
 		targetName = fmt.Sprintf("%ss", targetName)
@@ -436,8 +435,7 @@ func printSuccessForCommand(role string, didAdd bool, targetName string, targets
 	}
 	if didAdd {
 		verb = "added"
-		preposition = "to"
 	}
 
-	fmt.Fprintf(out, "Successfully %s the %q%srole %s %s %s.\n", verb, role, clusterScope, preposition, targetName, allTargets)
+	fmt.Fprintf(out, "%srole %q %s: %s\n", clusterScope, role, verb, allTargets)
 }