Commit ff6113df authored by Sean Sullivan's avatar Sean Sullivan

Removes dependency on RBAC within kubernetes core

parent 652cebcb
...@@ -19,7 +19,6 @@ go_library( ...@@ -19,7 +19,6 @@ go_library(
importpath = "k8s.io/kubernetes/pkg/kubectl/cmd/set", importpath = "k8s.io/kubernetes/pkg/kubectl/cmd/set",
visibility = ["//build/visible_to:pkg_kubectl_cmd_set_CONSUMERS"], visibility = ["//build/visible_to:pkg_kubectl_cmd_set_CONSUMERS"],
deps = [ deps = [
"//pkg/apis/rbac:go_default_library",
"//pkg/kubectl:go_default_library", "//pkg/kubectl:go_default_library",
"//pkg/kubectl/cmd/set/env:go_default_library", "//pkg/kubectl/cmd/set/env:go_default_library",
"//pkg/kubectl/cmd/templates:go_default_library", "//pkg/kubectl/cmd/templates:go_default_library",
......
...@@ -27,7 +27,6 @@ import ( ...@@ -27,7 +27,6 @@ import (
"k8s.io/apimachinery/pkg/types" "k8s.io/apimachinery/pkg/types"
utilerrors "k8s.io/apimachinery/pkg/util/errors" utilerrors "k8s.io/apimachinery/pkg/util/errors"
"k8s.io/apimachinery/pkg/util/sets" "k8s.io/apimachinery/pkg/util/sets"
"k8s.io/kubernetes/pkg/apis/rbac"
"k8s.io/kubernetes/pkg/kubectl/cmd/templates" "k8s.io/kubernetes/pkg/kubectl/cmd/templates"
cmdutil "k8s.io/kubernetes/pkg/kubectl/cmd/util" cmdutil "k8s.io/kubernetes/pkg/kubectl/cmd/util"
"k8s.io/kubernetes/pkg/kubectl/genericclioptions" "k8s.io/kubernetes/pkg/kubectl/genericclioptions"
...@@ -183,7 +182,7 @@ func (o *SubjectOptions) Validate() error { ...@@ -183,7 +182,7 @@ func (o *SubjectOptions) Validate() error {
} }
for _, info := range o.Infos { for _, info := range o.Infos {
_, ok := info.Object.(*rbac.ClusterRoleBinding) _, ok := info.Object.(*rbacv1.ClusterRoleBinding)
if ok && tokens[0] == "" { if ok && tokens[0] == "" {
return fmt.Errorf("serviceaccount must be <namespace>:<name>, namespace must be specified") return fmt.Errorf("serviceaccount must be <namespace>:<name>, namespace must be specified")
} }
...@@ -198,16 +197,16 @@ func (o *SubjectOptions) Run(fn updateSubjects) error { ...@@ -198,16 +197,16 @@ func (o *SubjectOptions) Run(fn updateSubjects) error {
subjects := []rbacv1.Subject{} subjects := []rbacv1.Subject{}
for _, user := range sets.NewString(o.Users...).List() { for _, user := range sets.NewString(o.Users...).List() {
subject := rbacv1.Subject{ subject := rbacv1.Subject{
Kind: rbac.UserKind, Kind: rbacv1.UserKind,
APIGroup: rbac.GroupName, APIGroup: rbacv1.GroupName,
Name: user, Name: user,
} }
subjects = append(subjects, subject) subjects = append(subjects, subject)
} }
for _, group := range sets.NewString(o.Groups...).List() { for _, group := range sets.NewString(o.Groups...).List() {
subject := rbacv1.Subject{ subject := rbacv1.Subject{
Kind: rbac.GroupKind, Kind: rbacv1.GroupKind,
APIGroup: rbac.GroupName, APIGroup: rbacv1.GroupName,
Name: group, Name: group,
} }
subjects = append(subjects, subject) subjects = append(subjects, subject)
...@@ -220,7 +219,7 @@ func (o *SubjectOptions) Run(fn updateSubjects) error { ...@@ -220,7 +219,7 @@ func (o *SubjectOptions) Run(fn updateSubjects) error {
namespace = o.namespace namespace = o.namespace
} }
subject := rbacv1.Subject{ subject := rbacv1.Subject{
Kind: rbac.ServiceAccountKind, Kind: rbacv1.ServiceAccountKind,
Namespace: namespace, Namespace: namespace,
Name: name, Name: name,
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment