Skip to content

Cleanup event-handler and fluentd template configs - #68563

Merged
kshi36 merged 1 commit into
masterfrom
kevin/event-handler-tpl
Jul 15, 2026
Merged

Cleanup event-handler and fluentd template configs#68563
kshi36 merged 1 commit into
masterfrom
kevin/event-handler-tpl

Conversation

@kshi36

@kshi36 kshi36 commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

This PR aims to fix up inconsistencies in event handler template config files generated from teleport-event-handler configure command. This only affects initial event handler setup for new users and not existing plugin instances. Docs also updated to be consistent.

Changelog: Improved teleport-event-handler configure command with more accurate boilerplate config files

Changes

teleport-event-handler binary and fluentd config:

  • Modified audit events endpoint for better clarity: /test.log -> /events.log (matches helm chart)

teleport-plugin-event-handler helm chart:

  • Modified session events endpoint to properly match against fluentd matchers: /session.log -> /session
  • Modified config's secret names to match the chart name (other access plugins already follow this convention):
    • teleport-event-handler-identity -> teleport-plugin-event-handler-identity
    • teleport-event-handler-client-tls -> teleport-plugin-event-handler-client-tls

Docs:

  • Reflect above changes

Manual Test Plan

Test Environment

Teleport Cloud and local Event Handler plugin (binary & helm chart)

Test Cases

  • Verify new setup of event handler + fluentd forwarder configs behaves normally
    • executable/docker
    • helm chart

@github-actions

github-actions Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
kevin/event-handler-tpl 3ace148 5 ✅SUCCEED kevin-event-handler-tpl 2026-07-15 17:04:41

Preview links for changed docs pages

@kshi36
kshi36 force-pushed the kevin/event-handler-tpl branch from 95064d5 to 1c32cc2 Compare July 11, 2026 00:28
@kshi36
kshi36 force-pushed the kevin/event-handler-tpl branch from 1c32cc2 to 41e4a42 Compare July 14, 2026 00:10
@kshi36
kshi36 marked this pull request as ready for review July 14, 2026 16:58
@github-actions
github-actions Bot requested a review from ptgott July 14, 2026 16:58
@kshi36
kshi36 force-pushed the kevin/event-handler-tpl branch from 3dcfc94 to 3ace148 Compare July 15, 2026 16:52
@kshi36
kshi36 added this pull request to the merge queue Jul 15, 2026
Merged via the queue into master with commit be0cf0d Jul 15, 2026
48 checks passed
@kshi36
kshi36 deleted the kevin/event-handler-tpl branch July 15, 2026 19:12
@backport-bot-workflows

Copy link
Copy Markdown
Contributor

@kshi36 See the table below for backport results.

Branch Result
branch/v18 Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants