MINOR: Correct the variable name in remote storage's test (#22349)

This is a follow-up PR for
#20913 (comment)
Thanks.  cc @kamalcph

Reviewers: Murali Basani <muralidhar.basani@aiven.io>, Ken Huang
 <s7133700@gmail.com>, Chia-Ping Tsai <chia7712@gmail.com>

Author: jiafu1115

core/src/test/scala/unit/kafka/server/DynamicBrokerConfigTest.scala

@@ -771,35 +771,35 @@ class DynamicBrokerConfigTest {
   }
 
   @Test
-  def testDynamicRemoteCopyLagThrowsOnIncorrectConfig(): Unit = {
-    // remote copy lag ms cannot exceed effective local retention ms
-    verifyIncorrectRemoteCopyLagProps(
+  def testDynamicLogRemoteCopyLagThrowsOnIncorrectConfig(): Unit = {
+    // log remote copy lag ms cannot exceed effective log local retention ms
+    verifyIncorrectLogRemoteCopyLagProps(
       retentionMs = 1000L,
       logLocalRetentionMs = -2L,
-      remoteCopyLagMs = 1001L,
+      logRemoteCopyLagMs = 1001L,
       retentionBytes = 1000L,
       logLocalRetentionBytes = -2L,
-      remoteCopyLagBytes = 100L
+      logRemoteCopyLagBytes = 100L
     )
 
-    // remote copy lag bytes cannot exceed effective local retention bytes
-    verifyIncorrectRemoteCopyLagProps(
+    // log remote copy lag bytes cannot exceed effective log local retention bytes
+    verifyIncorrectLogRemoteCopyLagProps(
       retentionMs = 1000L,
       logLocalRetentionMs = -2L,
-      remoteCopyLagMs = 100L,
+      logRemoteCopyLagMs = 100L,
       retentionBytes = 1000L,
       logLocalRetentionBytes = -2L,
-      remoteCopyLagBytes = 1001L
+      logRemoteCopyLagBytes = 1001L
     )
 
   }
 
-  def verifyIncorrectRemoteCopyLagProps(retentionMs: Long,
-                                        logLocalRetentionMs: Long,
-                                        remoteCopyLagMs: Long,
-                                        retentionBytes: Long,
-                                        logLocalRetentionBytes: Long,
-                                        remoteCopyLagBytes: Long): Unit = {
+  def verifyIncorrectLogRemoteCopyLagProps(retentionMs: Long,
+                                           logLocalRetentionMs: Long,
+                                           logRemoteCopyLagMs: Long,
+                                           retentionBytes: Long,
+                                           logLocalRetentionBytes: Long,
+                                           logRemoteCopyLagBytes: Long): Unit = {
     val props = TestUtils.createBrokerConfig(0, port = 8181)
     props.put(ServerLogConfigs.LOG_RETENTION_TIME_MILLIS_CONFIG, retentionMs.toString)
     props.put(ServerLogConfigs.LOG_RETENTION_BYTES_CONFIG, retentionBytes.toString)
@@ -810,9 +810,9 @@ class DynamicBrokerConfigTest {
 
     val newProps = new Properties()
     newProps.put(RemoteLogManagerConfig.LOG_LOCAL_RETENTION_MS_PROP, logLocalRetentionMs.toString)
-    newProps.put(RemoteLogManagerConfig.LOG_REMOTE_COPY_LAG_MS_PROP, remoteCopyLagMs.toString)
+    newProps.put(RemoteLogManagerConfig.LOG_REMOTE_COPY_LAG_MS_PROP, logRemoteCopyLagMs.toString)
     newProps.put(RemoteLogManagerConfig.LOG_LOCAL_RETENTION_BYTES_PROP, logLocalRetentionBytes.toString)
-    newProps.put(RemoteLogManagerConfig.LOG_REMOTE_COPY_LAG_BYTES_PROP, remoteCopyLagBytes.toString)
+    newProps.put(RemoteLogManagerConfig.LOG_REMOTE_COPY_LAG_BYTES_PROP, logRemoteCopyLagBytes.toString)
     // validate default config
     assertThrows(classOf[ConfigException], () => config.dynamicConfig.validate(newProps, perBrokerConfig = false))
     // validate per broker config