Skip to content

[LI-CHERRY-PICK] Backport default use all dns ips - #350

Merged
ZIDAZ merged 5 commits into
2.3-li-1from
backport_default_use_all_dns_ips
Jun 6, 2022
Merged

[LI-CHERRY-PICK] Backport default use all dns ips#350
ZIDAZ merged 5 commits into
2.3-li-1from
backport_default_use_all_dns_ips

Conversation

@ZIDAZ

@ZIDAZ ZIDAZ commented Jun 6, 2022

Copy link
Copy Markdown

Backport from 2.4-li

#315
#348
#116
#188

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

kehuum and others added 5 commits June 3, 2022 23:11
…tartup (#315)

* do not resolve bootstrap server during client bootstrap/startup

Co-authored-by: Ke Hu <kehu@kehu-mn2.linkedin.biz>
…ltipleIPsWithUseAll (#116)

TICKET = N/A
LI_DESCRIPTION = The test fails since the domain kafka.apache.org used to return 3 IPs and is now
only returning two IPs. Furthermore, the upstream fix identified below cannot be cleanly cherry
picked.
EXIT_CRITERIA = when the commit 131d475 is picked from upstream:
KAFKA-12193: Re-resolve IPs after a client disconnects apache#9902
TICKET = N/A
LI_DESCRIPTION = Several tests are failing since the domain kafka.apache.org that used to resolve to more than 1 IPv4 addresses are not only resolving to 1 IPv4 address.
The upstream code has overhauled the ClusterConnectionStatesTest. We are simply ignoring these tests for now, and will get the new logic from upstream after a major version rebase.
EXIT_CRITERIA = This hotfix can be removed in the next major version rebase
@ZIDAZ
ZIDAZ requested a review from kehuum June 6, 2022 17:14
@ZIDAZ
ZIDAZ merged commit 167f4b6 into 2.3-li-1 Jun 6, 2022
@ZIDAZ
ZIDAZ deleted the backport_default_use_all_dns_ips branch June 6, 2022 17:29
@ZIDAZ ZIDAZ changed the title Backport default use all dns ips [LI-HOTFIX] Backport default use all dns ips Jun 6, 2022
@ZIDAZ ZIDAZ changed the title [LI-HOTFIX] Backport default use all dns ips [LI-CHERRY-PICK] Backport default use all dns ips Jun 7, 2022
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.

3 participants