Skip to content

Commit

Permalink
nit: fix the naming and add - to the create-oidc-token
Browse files Browse the repository at this point in the history
Signed-off-by: Leo Li <leoli@redhat.com>
  • Loading branch information
Leo6Leo committed Dec 18, 2023
1 parent a73a26b commit 8890c5d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions pkg/apis/sources/register.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ const (
// the CRD is a Source duck type.
SourceDuckLabelValue = "true"

//OIDCLabelKey is used to filter out all the informers that related to OIDC work
OIDCLabelKey = "oidc"

// OIDCTokenRoleLabelSelector is the label selector for the OIDC token creator role and rolebinding informers
Expand Down
4 changes: 2 additions & 2 deletions pkg/reconciler/apiserversource/resources/oidc_rolebinding.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,12 @@ import (

// GetOIDCTokenRoleName will return the name of the role for creating the JWT token
func GetOIDCTokenRoleName(sourceName string) string {
return kmeta.ChildName(sourceName, "create-oidc-token")
return kmeta.ChildName(sourceName, "-create-oidc-token")
}

// GetOIDCTokenRoleBindingName will return the name of the rolebinding for creating the JWT token
func GetOIDCTokenRoleBindingName(sourceName string) string {
return kmeta.ChildName(sourceName, "create-oidc-token")
return kmeta.ChildName(sourceName, "-create-oidc-token")
}

// MakeOIDCRole will return the role object config for generating the JWT token
Expand Down

0 comments on commit 8890c5d

Please sign in to comment.