VR: fix warning Expected X answers while executing SetXXXCommand but Y (#7050)

Author: weizhouapache

core/src/main/java/com/cloud/agent/api/routing/IpAssocCommand.java

@@ -40,9 +40,4 @@ public boolean executeInSequence() {
     public IpAddressTO[] getIpAddresses() {
         return ipAddresses;
     }
-
-    @Override
-    public int getAnswersCount() {
-        return ipAddresses.length;
-    }
 }

core/src/main/java/com/cloud/agent/api/routing/IpAssocVpcCommand.java

@@ -29,10 +29,4 @@ protected IpAssocVpcCommand() {
     public IpAssocVpcCommand(IpAddressTO[] ips) {
         super(ips);
     }
-
-    @Override
-    public int getAnswersCount() {
-        //Count private gateway to maximum value
-        return ipAddresses.length * 2;
-    }
 }

core/src/main/java/com/cloud/agent/api/routing/NetworkElementCommand.java

@@ -80,7 +80,7 @@ public void setRouterAccessIp(final String routerAccessIp) {
     }
 
     public int getAnswersCount() {
-        return 1;
+        return 2;
     }
 
     public boolean isQuery() {

core/src/main/java/com/cloud/agent/api/routing/SetFirewallRulesCommand.java

@@ -111,9 +111,4 @@ public String[][] generateFwRules() {
 
         return result;
     }
-
-    @Override
-    public int getAnswersCount() {
-        return rules.length;
-    }
 }

core/src/main/java/com/cloud/agent/api/routing/SetIpv6FirewallRulesCommand.java

@@ -47,9 +47,4 @@ public FirewallRuleTO[] getRules() {
     public String getGuestIp6Cidr() {
         return guestIp6Cidr;
     }
-
-    @Override
-    public int getAnswersCount() {
-        return rules.length;
-    }
 }

core/src/main/java/com/cloud/agent/api/routing/SetMonitorServiceCommand.java

@@ -95,4 +95,9 @@ public boolean shouldDeleteFromProcessedCache() {
     public void setDeleteFromProcessedCache(boolean deleteFromProcessedCache) {
         this.deleteFromProcessedCache = deleteFromProcessedCache;
     }
+
+    @Override
+    public int getAnswersCount() {
+        return 2 + (reconfigureAfterUpdate ? 1 : 0);
+    }
 }

core/src/main/java/com/cloud/agent/api/routing/SetNetworkACLCommand.java

@@ -109,9 +109,4 @@ public int compare(final NetworkACLTO acl1, final NetworkACLTO acl2) {
     public NicTO getNic() {
         return nic;
     }
-
-    @Override
-    public int getAnswersCount() {
-        return rules.length;
-    }
 }

core/src/main/java/com/cloud/agent/api/routing/SetPortForwardingRulesCommand.java

@@ -40,10 +40,5 @@ public SetPortForwardingRulesCommand(List<? extends PortForwardingRuleTO> pfRule
     public PortForwardingRuleTO[] getRules() {
         return rules;
     }
-
-    @Override
-    public int getAnswersCount() {
-        return rules.length;
-    }
 }
 

core/src/main/java/com/cloud/agent/api/routing/SetStaticNatRulesCommand.java

@@ -47,9 +47,4 @@ public StaticNatRuleTO[] getRules() {
     public Long getVpcId() {
         return vpcId;
     }
-
-    @Override
-    public int getAnswersCount() {
-        return rules.length;
-    }
 }

core/src/main/java/com/cloud/agent/api/routing/SetStaticRouteCommand.java

@@ -59,9 +59,4 @@ public String[] generateSRouteRules() {
         }
         return toAdd.toArray(new String[toAdd.size()]);
     }
-
-    @Override
-    public int getAnswersCount() {
-        return staticRoutes.length;
-    }
 }