fix Copilot review and CI fail

Author: youjie23

docs/en/setup/backend/backend-alarm.md

@@ -252,8 +252,8 @@ See the following example:
   "ruleName": "service_resp_time_rule",
   "alarmMessage": "alarmMessage xxxx",
   "startTime": 1560524171000,
-  "recoveryTime": 15596606810000, 
-    "tags": [{
+  "recoveryTime": 1560524351000,
+  "tags": [{
         "key": "level",
         "value": "WARNING"
      }]

oap-server/server-alarm-plugin/src/main/java/org/apache/skywalking/oap/server/core/alarm/provider/RunningRule.java

@@ -223,6 +223,7 @@ public List<AlarmMessage> check() {
                     log.trace("RuleName:{} AlarmEntity {} {} {} expired", ruleName, alarmEntity.getName(),
                             alarmEntity.getId0(), alarmEntity.getId1());
                 }
+                return;
             }
 
             Optional<AlarmMessage> alarmMessageOptional = window.checkAlarm();
@@ -496,10 +497,8 @@ public AlarmStateMachine(int silencePeriod, int recoveryObservationPeriod) {
 
             public void onMatch() {
                 if (log.isTraceEnabled()) {
-                    if (log.isTraceEnabled()) {
-                        log.trace("RuleName:{} AlarmEntity {} {} {} onMatch silenceCountdown:{} currentState:{}",
-                                ruleName, entity.getName(), entity.getId0(), entity.getId1(), silenceCountdown, currentState);
-                    }
+                    log.trace("RuleName:{} AlarmEntity {} {} {} onMatch silenceCountdown:{} currentState:{}",
+                            ruleName, entity.getName(), entity.getId0(), entity.getId1(), silenceCountdown, currentState);
                 }
                 silenceCountdown--;
                 switch (currentState) {

oap-server/server-alarm-plugin/src/main/java/org/apache/skywalking/oap/server/core/alarm/provider/grpc/GRPCCallback.java

@@ -56,6 +56,7 @@ public GRPCCallback(AlarmRulesWatcher alarmRulesWatcher) {
         this.alarmRulesWatcher = alarmRulesWatcher;
         this.alarmServiceStubMap = new HashMap<>();
         this.grpcClientMap = new HashMap<>();
+        this.alarmSettingMap = new HashMap<>();
         Map<String, GRPCAlarmSetting> alarmSettingMap = alarmRulesWatcher.getGrpchookSetting();
         if (CollectionUtils.isNotEmpty(alarmSettingMap)) {
             alarmSettingMap.forEach((name, alarmSetting) -> {

oap-server/server-alarm-plugin/src/test/java/org/apache/skywalking/oap/server/core/alarm/provider/RunningRuleTest.java

@@ -3,7 +3,7 @@
  * contributor license agreements.  See the NOTICE file distributed with
  * this work for additional information regarding copyright ownership.
  * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License");you may not use this file except in compliance with
+ * (the "License"); you may not use this file except in compliance with
  * the License.  You may obtain a copy of the License at
  *
  *     http://www.apache.org/licenses/LICENSE-2.0

oap-server/server-alarm-plugin/src/test/java/org/apache/skywalking/oap/server/core/alarm/provider/wechat/WechatHookCallbackTest.java

@@ -92,7 +92,7 @@ public void testWechatWebhook() throws Exception {
         AlarmRulesWatcher alarmRulesWatcher = new AlarmRulesWatcher(rules, null, null);
         WechatHookCallback wechatHookCallback = new WechatHookCallback(alarmRulesWatcher);
         List<AlarmMessage> alarmMessages = new ArrayList<>(2);
-        List<AlarmRecoveryMessage> alarmRecoveryMessages = new ArrayList<>(1);
+        List<AlarmMessage> alarmRecoveryMessages = new ArrayList<>(1);
         AlarmMessage alarmMessage = new AlarmMessage();
         alarmMessage.setScopeId(DefaultScopeDefine.SERVICE);
         alarmMessage.setRuleName("service_resp_time_rule");
@@ -108,7 +108,7 @@ public void testWechatWebhook() throws Exception {
         wechatHookCallback.doAlarm(alarmMessages);
         AlarmRecoveryMessage alarmRecoveryMessage = new AlarmRecoveryMessage(anotherAlarmMessage);
         alarmRecoveryMessages.add(alarmRecoveryMessage);
-        wechatHookCallback.doAlarmRecovery(alarmMessages);
+        wechatHookCallback.doAlarmRecovery(alarmRecoveryMessages);
         Assertions.assertTrue(IS_SUCCESS.get());
     }
 }