Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add debug logging configuration and env option for eventhandler #42169

Merged
merged 9 commits into from
Jun 24, 2024
Merged

Add debug logging configuration and env option for eventhandler #42169

merged 9 commits into from
Jun 24, 2024

Conversation

stevenGravy
Copy link
Contributor

@stevenGravy stevenGravy commented May 30, 2024 •
edited
Loading

fixes #42423

Provides a debug setting within the Teleport Event handler toml config and the environment variable FDFWD_DEBUG. Also enables this to be set within the helm chart.

storage = "./storage"
timeout = "10s"
batch = 20
debug = true

Will follow up with docs post this merging.

changelog: Debug setting available for event-handler via configuration or FDFWD_DEBUG environment variable.

@stevenGravy stevenGravy added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v15 backport/branch/v16 labels May 30, 2024
@stevenGravy stevenGravy removed the no-changelog Indicates that a PR does not require a changelog entry label May 30, 2024
tigrato
tigrato approved these changes Jun 22, 2024
View reviewed changes
Copy link
Contributor

@tigrato tigrato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stevenGravy can you please test my latest changes?

@stevenGravy
Copy link
Contributor Author

@stevenGravy can you please test my latest changes?

Yes, looks good!

espadolini
espadolini approved these changes Jun 24, 2024
View reviewed changes
integrations/event-handler/testdata/config-debug.toml Outdated Show resolved Hide resolved
integrations/event-handler/main.go Outdated Show resolved Hide resolved
@stevenGravy stevenGravy added this pull request to the merge queue Jun 24, 2024
@stevenGravy stevenGravy removed this pull request from the merge queue due to a manual request Jun 24, 2024
stevenGravy and others added 2 commits June 24, 2024 08:39
@stevenGravy @espadolini
line changes
ace2ef3 
Co-authored-by: Edoardo Spadolini <edoardo.spadolini@goteleport.com>
@stevenGravy
Merge branch 'master' into stevenGravy/eventhandlerdebug
83f3338
@stevenGravy stevenGravy added this pull request to the merge queue Jun 24, 2024
Merged via the queue into master with commit d1dde0c Jun 24, 2024
40 checks passed
@stevenGravy stevenGravy deleted the stevenGravy/eventhandlerdebug branch June 24, 2024 13:17
@public-teleport-github-review-bot
Copy link

@stevenGravy See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR

This was referenced Jun 24, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@espadolini espadolini espadolini approved these changes

@tigrato tigrato tigrato approved these changes

Assignees
No one assigned
Labels
backport/branch/v15 backport/branch/v16 helm size/sm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

event handler chart should have debug mode
3 participants
@stevenGravy @espadolini @tigrato