Skip to content

feat(migrations): Add migration for inputs.kafka_consumer connection_strategy option#19121

Merged
srebhan merged 1 commit into
influxdata:masterfrom
skartikey:feat/migrate-kafka-consumer-connection-strategy
Jun 18, 2026
Merged

feat(migrations): Add migration for inputs.kafka_consumer connection_strategy option#19121
srebhan merged 1 commit into
influxdata:masterfrom
skartikey:feat/migrate-kafka-consumer-connection-strategy

Conversation

@skartikey

@skartikey skartikey commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

The connection_strategy option in inputs.kafka_consumer was deprecated in v1.33.0 for removal in v1.40.0, replaced by startup_error_behavior. This adds a migration that maps connection_strategy = "defer" to startup_error_behavior = "retry" on telegraf config migrate, dropping the "startup" default and erroring on contradicting values. The option removal ships separately (#19131) so users can run the migration while still on v1.39.x.

Checklist

  • No AI generated code was used in this PR

Related issues

related to #19085

@telegraf-tiger telegraf-tiger Bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Jun 17, 2026
@skartikey skartikey added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Jun 18, 2026
…strategy option

The connection_strategy option was deprecated in v1.33.0 for removal in
v1.40.0, replaced by startup_error_behavior. Remove it and add a migration
mapping "defer" to startup_error_behavior = "retry", dropping the "startup"
default and erroring on contradicting values.
@telegraf-tiger

Copy link
Copy Markdown
Contributor

@skartikey skartikey added this to the v1.39.1 milestone Jun 18, 2026

@srebhan srebhan 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.

Thanks @skartikey!

@srebhan
srebhan merged commit 629a223 into influxdata:master Jun 18, 2026
29 checks passed
skartikey added a commit that referenced this pull request Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants