[
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
