[
https://issues.apache.org/jira/browse/CASSANDRA-18106?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17690832#comment-17690832
]
Ekaterina Dimitrova commented on CASSANDRA-18106:
-------------------------------------------------
{quote} java.supported is providing "1.8" instead of just "8.
{quote}
Thanks for spotting it
{quote}Also, trivial, but can we remove the JAVAx_HOME lines (in just trunk):
{quote}
Honestly, we should also take care of CASSANDRA_USE_JDK11
[here|https://github.com/apache/cassandra/blob/trunk/.circleci/config.yml#L231]
or to be more accurate in config-2_1.yml
One more thing, CCM has its own tests and [JDK
tests|https://github.com/riptano/ccm/blob/master/tests/test_lib.py] also exist.
I know some of the CCM tests had some trouble to be run and Ariel has a
suggested fix that we keep on forgetting (published in CASSANDRA-17861) so
might be a good time to look at that one and then update here also the JDK CCM
tests accordingly.
> Update CCM for JDK17 and revise current JDK detection strategy
> --------------------------------------------------------------
>
> Key: CASSANDRA-18106
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18106
> Project: Cassandra
> Issue Type: Task
> Components: CI
> Reporter: Ekaterina Dimitrova
> Assignee: Brandon Williams
> Priority: Normal
> Fix For: 4.x
>
>
> As part of CASSANDRA-16895 initial POC an initial version of CCM patch was
> created. This needs to be revisited and reviewed
> Recently we closed CASSANDRA-18039 which brought questions, probably we need
> to revise how we detect JDK versions in CCM and whether it is correct. To the
> best of my knowledge there are certain tests in the repo around that and they
> pass so my guess is we need to revise just the strategy and maybe document it
> explicitly or consider if we want any changes to be applied. Also, we need to
> be careful with breaking changes.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
