-
Notifications
You must be signed in to change notification settings - Fork 593
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[v24.2.x] [CORE-7851] Adjust cipher strings to include ECDSA ciphers #24209
Conversation
This was a miss when #19792 landed. Only RSA based cipher strings were included in the list. This wasn't caught because our integration tests only use RSA based certificates. Also this may have taken some time for customers to find as this bug didn't effect TLSv1.3. Signed-off-by: Michael Boquard <michael@redpanda.com> (cherry picked from commit f0c141b)
Conflict in |
non flaky failures in https://buildkite.com/redpanda/redpanda/builds/58365#01934acd-42ab-499a-a545-f6dbfb5674b2:
non flaky failures in https://buildkite.com/redpanda/redpanda/builds/58365#01934acd-42a6-4241-9e4f-c558dfb76151:
non flaky failures in https://buildkite.com/redpanda/redpanda/builds/58365#01934acd-42ae-4a1d-b228-007f90c97180:
non flaky failures in https://buildkite.com/redpanda/redpanda/builds/58365#01934acd-42a9-4cbb-b54f-22795522537b:
|
Retry command for Build#58365please wait until all jobs are finished before running the slash command
|
This change permits users of the TLSCertManager to select which type of key to use in certificates. If one isn't selected, a random one is chosen, in the hope this increases our test coverage over time. Additionally, the tls_version_test has been updated to test both ECDSA and RSA based certificates. Signed-off-by: Michael Boquard <michael@redpanda.com> (cherry picked from commit 1760e8d)
57f8e28
to
0967f6a
Compare
Force push
|
ducktape was retried in https://buildkite.com/redpanda/redpanda/builds/58392#01934b77-7754-48e7-8054-65796aba191c |
Backport of PR #24191
Fixes: CORE-8274
Fixes: #24207