Skip to content

Commit

Permalink
Merge pull request open-horizon#3952 from LiilyZhang/zhangl/issue3951
Browse files Browse the repository at this point in the history
Issue open-horizon#3951 - Minor bug in combineRoleBindingSubjects fun…
  • Loading branch information
LiilyZhang authored Nov 17, 2023
2 parents 66fd586 + 8152936 commit 8da1a27
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion kube_operator/api_objects.go
Original file line number Diff line number Diff line change
Expand Up @@ -1330,7 +1330,7 @@ func combineRoleBindingSubjects(subjects1 []rbacv1.Subject, subjects2 []rbacv1.S
// remove duplicate in the subjects array
submap := make(map[string]rbacv1.Subject)
for _, sub := range subs {
key := fmt.Sprintf("%v/%v/%v", sub.Namespace, sub.Kind, sub.Kind) // key is <namespace>/<kind>/<name>
key := fmt.Sprintf("%v/%v/%v", sub.Namespace, sub.Kind, sub.Name) // key is <namespace>/<kind>/<name>
submap[key] = sub
}

Expand Down

0 comments on commit 8da1a27

Please sign in to comment.