Skip to content

Remove kafka support to reduce size - #67

Merged
mheffner merged 1 commit into
mainfrom
remove-rdkafka
Dec 10, 2025
Merged

Remove kafka support to reduce size#67
mheffner merged 1 commit into
mainfrom
remove-rdkafka

Conversation

@mheffner

@mheffner mheffner commented Dec 9, 2025

Copy link
Copy Markdown
Member

Removing the Kafka exporter/receiver from the default build to reduce size and cold start time. This support seems less common for most use cases, so we'll exclude from the default image. Going to open issues to discuss building a separate layer for Kafka use cases.

@mheffner
mheffner requested a review from rjenkins December 9, 2025 19:23

@rjenkins rjenkins left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This makes sense, but wanted to point out it will also disable aws_iam as it's in the default flag. Do you manually add it back when you build somewhere else?

@mheffner

Copy link
Copy Markdown
Member Author

This makes sense, but wanted to point out it will also disable aws_iam as it's in the default flag. Do you manually add it back when you build somewhere else?

We actually intentionally leave that out because it shouldn't be needed in the Lambda environment. The runtime environment manages IAM roles and sets temporary credentials with environment variables.

@rjenkins

Copy link
Copy Markdown
Member

This makes sense, but wanted to point out it will also disable aws_iam as it's in the default flag. Do you manually add it back when you build somewhere else?

We actually intentionally leave that out because it shouldn't be needed in the Lambda environment. The runtime environment manages IAM roles and sets temporary credentials with environment variables.

OK, cool I thought you might need it, but now I recall we added for the teleport ask.

@mheffner
mheffner merged commit cf9fcf5 into main Dec 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants