Skip to content

fix: grant the pipeline deploy step keda.sh ScaledObjects and Trigger Authentications - #486

Closed
gauron99 wants to merge 1 commit into
openshift:mainfrom
gauron99:push-xnvtrtmrqmnn
Closed

gauron99 wants to merge 1 commit into
openshift:mainfrom
gauron99:push-xnvtrtmrqmnn

Conversation

@gauron99

Copy link
Copy Markdown

Since knative/func#4069 func's keda deployer reads ScaledObjects on every deploy, to catch a switch between the http and kafka scalers, and creates ScaledObjects and TriggerAuthentications for kafka triggers. The deploy step of the on-cluster build runs that deployer as the namespace's default service account, with the Role this operator creates. That Role had no keda.sh rules, so, as knative/func#4077 describes for func's own roles, every keda deploy is refused. The E2E keda jobs have failed on every PR since 2026-10-05.

The deploy step runs ghcr.io/knative/func-utils:v2, a tag func republishes from its main branch, so the change reached the operator without a func bump.

Grant the verbs func grants itself in knative/func#4077. The operator's ClusterRole needs the same rule, or it may not create the Role. Existing Roles get the rule on the next rebuild in their namespace, because ensureDeployFunctionRole rewrites rules that differ.

…Authentications

Since knative/func#4069 func's keda deployer reads ScaledObjects on
every deploy, to catch a switch between the http and kafka scalers, and
creates ScaledObjects and TriggerAuthentications for kafka triggers.
The deploy step of the on-cluster build runs that deployer as the
namespace's default service account, with the Role this operator
creates. That Role had no keda.sh rules, so, as knative/func#4077
describes for func's own roles, every keda deploy is refused. The E2E
keda jobs have failed on every PR since 2026-10-05.

The deploy step runs ghcr.io/knative/func-utils:v2, a tag func
republishes from its main branch, so the change reached the operator
without a func bump.

Grant the verbs func grants itself in knative/func#4077. The operator's
ClusterRole needs the same rule, or it may not create the Role.
Existing Roles get the rule on the next rebuild in their namespace,
because ensureDeployFunctionRole rewrites rules that differ.
@gauron99 gauron99 closed this Oct 13, 2026
@openshift-ci openshift-ci Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 13, 2026
@openshift-ci
openshift-ci Bot requested a review from maschmid October 13, 2026 09:01
@openshift-ci

openshift-ci Bot commented Oct 13, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gauron99

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant