Skip to content

Upgrade to Maven 3.8.8 for CASSJAVA-102 #117

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 18, 2025

Conversation

aratno
Copy link
Contributor

@aratno aratno commented Jun 4, 2025

This is the latest Maven version that still supports Java 8 and newer revapi versions.

Tested with:

$ docker build -t apache/cassandra-java-driver-testing-ubuntu2204 -f docker/testing/ubuntu2204_java_driver_testing.docker .
$ docker run -it <image_id> bash
$ mvn --version

@aratno aratno self-assigned this Jun 4, 2025
Copy link
Contributor

@SiyaoIsHiding SiyaoIsHiding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirm that docker build and

        container_id=\$(docker run -td -e TEST_JAVA_VERSION=${TEST_JAVA_VERSION} -e SERVER_VERSION=${SERVER_VERSION} -e TEST_JAVA_MAJOR_VERSION=${testJavaMajorVersion} -v \$(pwd):/home/docker/cassandra-java-driver apache.jfrog.io/cassan-docker/apache/cassandra-java-driver-testing-ubuntu2204 'sleep 2h')
        docker exec --user root \$container_id bash -c \"sudo bash /home/docker/cassandra-java-driver/ci/create-user.sh docker \$(id -u) \$(id -g) /home/docker/cassandra-java-driver\"
        docker exec --user docker \$container_id './cassandra-java-driver/ci/run-tests.sh'

Works find in my local environment.

Copy link
Member

@michaelsembwever michaelsembwever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

would be better with the download checksummed.

@aratno
Copy link
Contributor Author

aratno commented Jun 10, 2025

Thanks - will add the checksum

@aratno aratno requested a review from michaelsembwever June 10, 2025 22:53
@aratno
Copy link
Contributor Author

aratno commented Jun 10, 2025

Added checksum validation for Maven - @michaelsembwever could you take a look and deploy images when you're ready? Accompanying PR has been approved as well: apache/cassandra-java-driver#2042

This is the latest Maven version that still supports Java 8 and newer
revapi versions.

Tested with:

  $ docker build -t apache/cassandra-java-driver-testing-ubuntu2204 -f docker/testing/ubuntu2204_java_driver_testing.docker .
  $ docker run -it <image_id> bash
  $ mvn --version
@aratno aratno force-pushed the CASSJAVA-102-upgrade-maven-in-ci branch from 3e8a120 to ff8b734 Compare June 18, 2025 14:58
@aratno aratno merged commit eab5b74 into apache:trunk Jun 18, 2025
@aratno aratno deleted the CASSJAVA-102-upgrade-maven-in-ci branch June 18, 2025 14:59
@michaelsembwever
Copy link
Member

michaelsembwever commented Jun 27, 2025

has been published to both dockerhub and apache.frog.io

(including the ninja fixes from f155210 and edd01db )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants