dkropachev commented on PR #799: URL: https://github.com/apache/cassandra-ccm/pull/799#issuecomment-3684061794
> > https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch-before-5-dtest/42/ – same failure as on 5.0 https://github.com/apache/cassandra-ccm/pull/799#issuecomment-3652331364 > > @dkropachev , if i patch https://github.com/thelastpickle/cassandra-dtest/commit/aada32c94a565b441d2ac72b6c0bd6aed5dce71a) so `requirements.txt` to also `packaging<21` > it fixes the failures: http://ci-cassandra.infra.datastax.com/job/cassandra-5.0/61/ > > This includes applying the same change of `LooseVersion` in all of cassandra-dtest, something that needs to be done for [CASSANDRA-18321](https://issues.apache.org/jira/browse/CASSANDRA-18321). > > Unfortunately, this changes a compatibility breakage (complicated because both cassandra-dtest and cassandra-ccm are used unpinned). This will be documented. > > But it means I will need to merge both this PR and the cassandra-test PR at the same time, and I'm waiting on a review on the latter: > - https://github.com/apache/cassandra-dtest/pull/283 Not really, you can pin ccm in dtest and then merge thus guy and then make pr to move to new ccm commit or back to master on dtest -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
