Fix Agent for CC5 (#706)

* Update CC5 version

This patch updates the CC5 version to a more recent 5.0.4
and updates the Message.Request.execute() call so it
matches the updated method signature in CC5.

Also adds Docker login step to pull private base images for DSE

Author: emerkle826

.github/workflows/ci.yaml

@@ -246,6 +246,11 @@ jobs:
           image: tonistiigi/binfmt:qemu-v7.0.0-28
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3
+      - name: Login to DockerHub
+        uses: docker/login-action@v3
+        with:
+          username: ${{ secrets.DOCKER_HUB_USERNAME }}
+          password: ${{ secrets.DOCKER_HUB_PASSWORD }}
       - name: Build DSE ${{ matrix.dse-version }}-${{ matrix.base-platform }}
         id: docker_build
         uses: docker/build-push-action@v5

CHANGELOG.md

@@ -10,6 +10,7 @@ Changelog for Management API, new PRs should update the `main / unreleased` sect
 ```
 
 ## unreleased
+* [BUGFIX] [#705](https://github.com/k8ssandra/management-api-for-apache-cassandra/issues/705) Fix Agent for CC5
 
 ## v0.1.110 [2025-11-20]
 * [FEATURE] [#695](https://github.com/k8ssandra/management-api-for-apache-cassandra/issues/695) Add Cassandra 5.0.6 to the build matrix

management-api-agent-hcd-cc5/pom.xml

@@ -79,7 +79,7 @@
         <dependency>
           <groupId>com.datastax.dse</groupId>
           <artifactId>dse-db-all</artifactId>
-          <version>5.0.0-beta2.b8452cef5a56</version>
+          <version>5.0.4.0-c64ed2865363</version>
           <exclusions>
             <exclusion>
               <groupId>commons-codec</groupId>

management-api-agent-hcd-cc5/src/main/java/com/datastax/mgmtapi/shim/RpcStatement.java

@@ -10,6 +10,7 @@
 import org.apache.cassandra.cql3.QueryOptions;
 import org.apache.cassandra.service.ClientState;
 import org.apache.cassandra.service.QueryState;
+import org.apache.cassandra.transport.Dispatcher.RequestTime;
 import org.apache.cassandra.transport.messages.ResultMessage;
 
 public class RpcStatement implements RpcStatementShim {
@@ -33,6 +34,15 @@ public ResultMessage execute(QueryState queryState, QueryOptions queryOptions, l
     return new ResultMessage.Void();
   }
 
+  /**
+   * as of Cassandra 4.1.6, CASSANDRA-19534, org.apache.cassandra.cql3.CQLStatement now has the
+   * method signature below, but it does not exist in Cassandra < 4.1.6. For Cassandra 5.0, this fix
+   * was ported between 5.0-rc1 and 5.0-rc2.
+   */
+  public ResultMessage execute(QueryState queryState, QueryOptions queryOptions, RequestTime rt) {
+    return new ResultMessage.Void();
+  }
+
   @Override
   public ResultMessage executeLocally(QueryState queryState, QueryOptions queryOptions) {
     return new ResultMessage.Void();

management-api-agent-hcd-cc5/src/main/java/org/apache/cassandra/transport/UnixSocketServerHcd.java

@@ -81,7 +81,6 @@ protected void channelRead0(ChannelHandlerContext ctx, Message.Request request)
         throws Exception {
 
       final UnixSocketConnection connection;
-      long queryStartNanoTime = System.nanoTime();
 
       try {
         assert request.connection() instanceof UnixSocketConnection;
@@ -101,7 +100,7 @@ protected void channelRead0(ChannelHandlerContext ctx, Message.Request request)
         // make the correct calls based on which version of 4.1.x we are. For Cassandra 5.0, this
         // patch was ported between 5.0-rc1 and 5.0-rc2.
         request
-            .execute(qstate, queryStartNanoTime)
+            .execute(qstate, Dispatcher.RequestTime.forImmediateExecution())
             .addCallback(
                 (Message.Response response, Throwable ignore) -> {
                   processMessageResponse(response, request, connection, ctx);