Skip to content

Conversation

@michael-redpanda
Copy link
Contributor

The method_whitelist parameter in urllib3's Retry class has been deprecated in favor of allowed_methods. This change ensures compatibility with newer versions of urllib3.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v25.3.x
  • v25.2.x
  • v25.1.x

Release Notes

  • none

The method_whitelist parameter in urllib3's Retry class has been
deprecated in favor of allowed_methods. This change ensures compatibility
with newer versions of urllib3.

Signed-off-by: Michael Boquard <[email protected]>
@michael-redpanda michael-redpanda self-assigned this Jan 7, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the deprecated method_whitelist parameter to allowed_methods in urllib3's Retry class initialization. The change ensures compatibility with newer versions of urllib3 where the old parameter name has been deprecated.

Key changes:

  • Replaced deprecated method_whitelist parameter with allowed_methods in the Retry class instantiation
  • Added a clarifying comment that the None value means all HTTP methods will be retried

@michael-redpanda michael-redpanda changed the title admin: update deprecated method_whitelist to allowed_methods [v25.2.x] admin: update deprecated method_whitelist to allowed_methods Jan 7, 2026
@vbotbuildovich
Copy link
Collaborator

CI test results

test results on build#78658
test_class test_method test_arguments test_kind job_url test_status passed reason test_history
cluster_metadata_uploader_fixture test_upload_in_term unit https://buildkite.com/redpanda/redpanda/builds/78658#019b99e5-682c-4cb7-9d74-2ecb92655017 FAIL 0/1
NodesDecommissioningTest test_recommissioning_node_finishes null integration https://buildkite.com/redpanda/redpanda/builds/78658#019b9a16-800e-4327-b568-1a97f02ac557 FLAKY 10/11 Test PASSES after retries.No significant increase in flaky rate(baseline=0.0000, p0=1.0000, reject_threshold=0.0100. adj_baseline=0.1000, p1=0.3487, trust_threshold=0.5000) https://redpanda.metabaseapp.com/dashboard/87-tests?tab=142-dt-individual-test-history&test_class=NodesDecommissioningTest&test_method=test_recommissioning_node_finishes

@michael-redpanda michael-redpanda merged commit 1ec2e4d into v25.2.x Jan 8, 2026
20 checks passed
@michael-redpanda michael-redpanda deleted the fix-admin-retry branch January 8, 2026 12:11
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.

4 participants