Skip to content

build(deps): bump franz-go and set 9m conn idle timeout default#749

Merged
carsonip merged 2 commits intomainfrom
chore/bump-franz-go-conn-idle-timeout
Feb 23, 2026
Merged

build(deps): bump franz-go and set 9m conn idle timeout default#749
carsonip merged 2 commits intomainfrom
chore/bump-franz-go-conn-idle-timeout

Conversation

@carsonip
Copy link
Member

@carsonip carsonip commented Feb 23, 2026

Summary

  • bump github.com/twmb/franz-go and related franz-go modules to latest versions in go.mod / go.sum
  • add ConnIdleTimeout to kafka.CommonConfig
  • default to 9m (instead of franz-go default) when ConnIdleTimeout is unset, while allowing explicit override
  • update kafka tests for franz-go/kfake telemetry behavior changes:
    • include new expected manager metrics datapoints
    • allow additional debug UNKNOWN_TOPIC_OR_PARTITION entries
    • filter internal operation=GetTelemetrySubscriptions datapoints from producer metric assertions

Test plan

  • go test ./kafka/...
  • go test ./...

Upgrade franz-go modules to latest versions and make ConnIdleTimeout configurable in CommonConfig, defaulting to 9m instead of franz-go's default when unset.

Co-authored-by: Cursor <cursoragent@cursor.com>
@carsonip carsonip requested a review from a team February 23, 2026 09:28
@carsonip carsonip marked this pull request as draft February 23, 2026 09:30
Adjust manager and producer metrics tests to account for new telemetry datapoints emitted after the franz-go upgrade while preserving coverage of expected lag, assignment, and log behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
@carsonip carsonip marked this pull request as ready for review February 23, 2026 10:07
@carsonip carsonip merged commit bbaac04 into main Feb 23, 2026
6 checks passed
@carsonip carsonip deleted the chore/bump-franz-go-conn-idle-timeout branch February 23, 2026 10:08
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