12684: Stop retrying 422 responses in PySDK
authorBrett Smith <brett.smith@curii.com>
Thu, 18 May 2023 12:36:56 +0000 (08:36 -0400)
committerBrett Smith <brett.smith@curii.com>
Thu, 18 May 2023 12:36:56 +0000 (08:36 -0400)
commit11cf3f097fa7d9f10c0131791249c56aab6839a6
tree3f9732f9b405f7973514c477b24144f2aa0c226e
parente3962b41ad01780c1b173892f0a5c947449a0ed8
12684: Stop retrying 422 responses in PySDK

The original motivation for this was to retry when the API server was
having database connectivity problems. The feeling eight years later is
that things have changed enough that, on balance, this isn't worth
retrying anymore.

I don't think this will have any real impact on current Arvados
software. In the main branch as I write this,
`check_http_response_status` only gets called in five places. Three of
those are in the main `arvados` module for job and task utilities, which
presumably nobody is using anymore. The other two talk to Keep, which
only returns 422 for hash mismatches, where a retry will definitely
never succeed.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
sdk/python/arvados/retry.py
sdk/python/tests/test_api.py
sdk/python/tests/test_retry.py