Warn about deprecated commands

These commands still work against 3.6 servers, but we should start warning that they will stop working once 3.7 is used. Signed-off-by: Simo Sorce <[email protected]>
cucushift · Oct 11, 2017 · 7642124 · 7642124
1 parent 36865f9
commit 7642124
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/pkg/cmd/server/admin/overwrite_bootstrappolicy.go b/pkg/cmd/server/admin/overwrite_bootstrappolicy.go
@@ -70,6 +70,7 @@ func NewCommandOverwriteBootstrapPolicy(commandName string, fullName string, cre
 			}
 			kcmdutil.CheckErr(options.OverwriteBootstrapPolicy())
 		},
+		Deprecated: fmt.Sprintf("will not work against 3.7 servers"),
 	}
 
 	flags := cmd.Flags()

diff --git a/pkg/oc/admin/migrate/authorization/authorization.go b/pkg/oc/admin/migrate/authorization/authorization.go
@@ -71,6 +71,7 @@ func NewCmdMigrateAuthorization(name, fullName string, f *clientcmd.Factory, in
 			kcmdutil.CheckErr(options.Validate())
 			kcmdutil.CheckErr(options.Run())
 		},
+		Deprecated: fmt.Sprintf("will not work against 3.7 servers"),
 	}
 	return cmd
 }

diff --git a/pkg/oc/cli/cmd/create/policy_binding.go b/pkg/oc/cli/cmd/create/policy_binding.go
@@ -54,6 +54,7 @@ func NewCmdCreatePolicyBinding(name, fullName string, f *clientcmd.Factory, out
 			cmdutil.CheckErr(o.Validate())
 			cmdutil.CheckErr(o.Run())
 		},
+		Deprecated: fmt.Sprintf("will not work against 3.7 servers. Use (Cluster)RoleBindings instead."),
 	}
 	cmdutil.AddOutputFlagsForMutation(cmd)
 	return cmd
-Original file line number
+Diff line change
@@ Expand Up @@
     			}
     			kcmdutil.CheckErr(options.OverwriteBootstrapPolicy())
     		},
+    		Deprecated: fmt.Sprintf("will not work against 3.7 servers"),
     	}
     	flags := cmd.Flags()
@@ Expand Down @@