[ 
https://issues.apache.org/jira/browse/CASSANDRA-16275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17233507#comment-17233507
 ] 

Sam Tunnicliffe commented on CASSANDRA-16275:
---------------------------------------------


bq.  It came from this change, which includes ServerError in retry logic.

Cool, that was my understanding also. 

Whichever order we do things in, we're going to have some breakage, to try and 
minimize that I propose the following:

1. Commit this fix. {{auth_test.py::TestAuth::test_handle_corrupt_role}} will 
start to fail on all CI jobs.
2. Update the {{cassandra-test}} branch to at least 
{{e1fc528a0deec67f725f066204e8c1ea8b26bbde}}. I expect all dtest CI jobs to 
start failing here due to the incompatibilities in the base docker image. 
3. Publish new docker images which install the updated driver in the base 
image, so no conflicting updates are pulled.
4. Update cassandra's .circle/config.yml to use the new docker images. Build 
should start to work again, but with a handful of new failures for V5 tests. 
{{auth_test.py}} should start passing again.
5. Commit CASSANDRA-15299 to trunk. This will fix the failing V5 tests.

Most of the disruption will occur between steps 2 and 5, so I suggest we wait 
until INFRA-21103 is resolved before starting.

[~aholmber] [~mck] does that sound like a plan to you?

> Update python driver used by cassandra-dtest
> --------------------------------------------
>
>                 Key: CASSANDRA-16275
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-16275
>             Project: Cassandra
>          Issue Type: Task
>          Components: Test/dtest/python
>            Reporter: Sam Tunnicliffe
>            Assignee: Sam Tunnicliffe
>            Priority: Normal
>
> In order to commit CASSANDRA-15299, the python driver used by the dtests 
> needs to include PYTHON-1258, support for V5 framing. 
> Updating the python driver's cassandra-test branch to latest trunk causes 1 
> additional dtest failure in 
> {{auth_test.py::TestAuth::test_handle_corrupt_role_data}} because the 
> {{ServerError}} response is now subject to the configured {{retry_policy}}. 
> This means the error ultimately returned from the driver is 
> {{NoHostAvailable}}, rather than {{ServerError}}. 
> I'll open a dtest pr to change the expectation in the test and we can commit 
> that when the cassandra-test branch is updated.
> cc [~aholmber] [~aboudreault]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to