Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

Commit

Permalink
Merge pull request #657 from kaczyns/triggersRBAC
Browse files Browse the repository at this point in the history
tekton triggers apis moved to triggers.tekton.dev
  • Loading branch information
mchenggit authored Apr 29, 2020
2 parents 54286c9 + b9b7cd9 commit 3c07d5f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion deploy/kabanero-customresources.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -300,7 +300,7 @@ metadata:
kabanero.io/install: 25-triggers-role
rules:
- apiGroups:
- tekton.dev
- triggers.tekton.dev
resources:
- triggerbindings
- triggertemplates
Expand Down
4 changes: 2 additions & 2 deletions pkg/controller/stack/stack_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -531,9 +531,9 @@ func reconcileActiveVersions(stackResource *kabanerov1alpha2.Stack, c client.Cli
// Only allow Group: tekton.dev
allowed := true
for _, resource := range resources {
if resource.GroupVersionKind().Group != "tekton.dev" {
if (resource.GroupVersionKind().Group != "tekton.dev") && (resource.GroupVersionKind().Group != "triggers.tekton.dev") {
value.ActiveAssets[index].Status = assetStatusFailed
value.ActiveAssets[index].StatusMessage = "Manifest rejected: contains a Group not equal to tekton.dev"
value.ActiveAssets[index].StatusMessage = "Manifest rejected: contains a Group not equal to tekton.dev or triggers.tekton.dev"
allowed = false
}
}
Expand Down

0 comments on commit 3c07d5f

Please sign in to comment.