Skip to content

MINOR: Fix testTlsDefaults failure due to TLS 1.0/1.1 being disabled#11092

Merged
ijuma merged 1 commit into
2.8from
fix-test-tls-defaults-2.8
Jul 20, 2021
Merged

MINOR: Fix testTlsDefaults failure due to TLS 1.0/1.1 being disabled#11092
ijuma merged 1 commit into
2.8from
fix-test-tls-defaults-2.8

Conversation

@ijuma

@ijuma ijuma commented Jul 20, 2021

Copy link
Copy Markdown
Member

The latest JDKs no longer support TLS 1.0/1.1 causing the test to fail.
We have already fixed this in trunk and 3.0, so this is for 2.8 and older
branches.

The relevant trunk commit is 530224e. We had another test with
the same issue and it was fixed for all branches via #10922.

Committer Checklist (excluded from commit message)

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

The latest JDKs no longer support TLS 1.0/1.1 causing the test to fail.
We have already fixed this in trunk and 3.0, so this is for 2.8 and older
branches.

Trunk commits are 530224e and 12aa595.
@ijuma
ijuma requested a review from rajinisivaram July 20, 2021 13:24

@rajinisivaram rajinisivaram left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ijuma Thanks for the PR, LGTM

@ijuma

ijuma commented Jul 20, 2021

Copy link
Copy Markdown
Member Author

Test failures are unrelated.

@ijuma
ijuma merged commit b6071b2 into 2.8 Jul 20, 2021
@ijuma
ijuma deleted the fix-test-tls-defaults-2.8 branch July 20, 2021 20:42
ijuma added a commit that referenced this pull request Jul 20, 2021
…11092)

The latest JDKs no longer support TLS 1.0/1.1 causing the test to fail.
We have already fixed this in trunk and 3.0, so this is for 2.8 and older
branches.

The relevant trunk commit is 530224e. We had another test with
the same issue and it was fixed for all branches via #10922.

Reviewers: Rajini Sivaram <rajinisivaram@googlemail.com>
ijuma added a commit that referenced this pull request Jul 20, 2021
…11092)

The latest JDKs no longer support TLS 1.0/1.1 causing the test to fail.
We have already fixed this in trunk and 3.0, so this is for 2.8 and older
branches.

The relevant trunk commit is 530224e. We had another test with
the same issue and it was fixed for all branches via #10922.

Reviewers: Rajini Sivaram <rajinisivaram@googlemail.com>
ijuma added a commit that referenced this pull request Jul 20, 2021
…11092)

The latest JDKs no longer support TLS 1.0/1.1 causing the test to fail.
We have already fixed this in trunk and 3.0, so this is for 2.8 and older
branches.

The relevant trunk commit is 530224e. We had another test with
the same issue and it was fixed for all branches via #10922.

Reviewers: Rajini Sivaram <rajinisivaram@googlemail.com>
@ijuma

ijuma commented Jul 20, 2021

Copy link
Copy Markdown
Member Author

Merged to 2.8 and cherry-picked to 2.7, 2.6 and 2.5.

udaynpusa pushed a commit to mapr/kafka that referenced this pull request Jan 30, 2024
…pache#11092)

The latest JDKs no longer support TLS 1.0/1.1 causing the test to fail.
We have already fixed this in trunk and 3.0, so this is for 2.8 and older
branches.

The relevant trunk commit is 530224e. We had another test with
the same issue and it was fixed for all branches via apache#10922.

Reviewers: Rajini Sivaram <rajinisivaram@googlemail.com>
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