Skip to content

Commit

Permalink
some cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
mathis-marcotte committed Jan 28, 2025
1 parent 1e9f38b commit 84742ad
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 25 deletions.
25 changes: 0 additions & 25 deletions cmd/webhook.go
Original file line number Diff line number Diff line change
Expand Up @@ -353,28 +353,3 @@ func (whsvr *WebhookServer) serve(w http.ResponseWriter, r *http.Request) {
http.Error(w, fmt.Sprintf("could not write response: %v", err), http.StatusInternalServerError)
}
}

// func getSvmInfoList(client *kubernetes.Clientset) (map[string]SvmInfo, error) {
// klog.Infof("Getting filers list...")

// filerListCM, err := client.CoreV1().ConfigMaps("das").Get(context.Background(), "filers-list", metav1.GetOptions{})
// if err != nil {
// klog.Errorf("Error occured while getting the filers list: %v", err)
// return nil, err
// }

// var svmInfoList []SvmInfo
// err = json.Unmarshal([]byte(filerListCM.Data["filers"]), &svmInfoList)
// if err != nil {
// klog.Errorf("Error occured while unmarshalling the filers list: %v", err)
// return nil, err
// }

// //format the data into something a bit more usable
// filerList := map[string]SvmInfo{}
// for _, svm := range svmInfoList {
// filerList[svm.Vserver] = svm
// }

// return filerList, nil
// }
1 change: 1 addition & 0 deletions deploy/clusterrole.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1
metadata:
name: kerberos-sidecar-injector
namespace: das
labels:
app: kerberos-sidecar-injector
rules:
Expand Down
1 change: 1 addition & 0 deletions deploy/clusterrolebinding.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
name: kerberos-sidecar-injector
namespace: das
labels:
app: kerberos-sidecar-injector
roleRef:
Expand Down

0 comments on commit 84742ad

Please sign in to comment.