Skip to content

fix: bucket-events sensor parameters indent & service account TDE-1879#1183

Merged
paulfouquet merged 2 commits intomasterfrom
fix/bucket-event-sensor-parameters-tde-1879
Apr 7, 2026
Merged

fix: bucket-events sensor parameters indent & service account TDE-1879#1183
paulfouquet merged 2 commits intomasterfrom
fix/bucket-event-sensor-parameters-tde-1879

Conversation

@paulfouquet
Copy link
Copy Markdown
Collaborator

@paulfouquet paulfouquet commented Apr 7, 2026

Motivation

Triggers parameters

The Argo Events sensor trigger was failing to create workflows due to an invalid Workflow specification.
The triggers.template.parameters were incorrectly defined (under workflowTemplateRef), which is not supported by Argo Workflows and caused a strict decoding error at runtime:

│ {"level":"error","ts":"2026-04-07T02:35:48.356028899Z","logger":"argo-events.sensor","caller":"sensors/listener.go:380","msg":"Failed to execute a trigger","sensorName":"wf-bucket-events","error":"failed to execute trigger, failed to execute │
│ time="2026-04-07T02:35:48.359Z" level=fatal msg="Failed to parse workflow: strict decoding error: unknown field \"spec.workflowTemplateRef.parameters\""   

This resulted in the sensor failing to execute triggers when processing events from the EventBus.

Service Account

A cross namespace (Cluster scope) service account needs to be used by the sensor to trigger the workflow on the argo namespace.

Modifications

Indent correctly parameters
Specify service account

Verification

@paulfouquet paulfouquet self-assigned this Apr 7, 2026
@paulfouquet paulfouquet marked this pull request as ready for review April 7, 2026 18:29
@paulfouquet paulfouquet requested review from a team as code owners April 7, 2026 18:29
amfage
amfage previously approved these changes Apr 7, 2026
@paulfouquet paulfouquet changed the title fix: bucket-events sensor parameters indent TDE-1879 fix: bucket-events sensor parameters indent & service account TDE-1879 Apr 7, 2026
@paulfouquet paulfouquet added this pull request to the merge queue Apr 7, 2026
Merged via the queue into master with commit 55a9a93 Apr 7, 2026
5 checks passed
@paulfouquet paulfouquet deleted the fix/bucket-event-sensor-parameters-tde-1879 branch April 7, 2026 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants