(cloudstack) branch 4.20 updated: mgmt: add back serviceip in ManagementServerResponse (#10891)

[pearl11594]

This is an automated email from the ASF dual-hosted git repository.

pearl11594 pushed a commit to branch 4.20
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/4.20 by this push:
     new d0dc6d8093a mgmt: add back serviceip in ManagementServerResponse 
(#10891)
d0dc6d8093a is described below

commit d0dc6d8093a17222b683d4657307ff87f922c3de
Author: Wei Zhou <weiz...@apache.org>
AuthorDate: Wed May 21 06:25:52 2025 +0200

    mgmt: add back serviceip in ManagementServerResponse (#10891)
    
    * mgmt: add back serviceip in ManagementServerResponse
    
    * api: add Deprecated annotation to serviceip
---
 .../main/java/org/apache/cloudstack/api/ApiConstants.java   |  1 +
 .../cloudstack/api/response/ManagementServerResponse.java   | 13 +++++++++++++
 .../src/main/java/com/cloud/api/query/QueryManagerImpl.java |  1 +
 3 files changed, 15 insertions(+)

diff --git a/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java 
b/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java
index efb1ca929ca..89c9a194e3f 100644
--- a/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java
+++ b/api/src/main/java/org/apache/cloudstack/api/ApiConstants.java
@@ -448,6 +448,7 @@ public class ApiConstants {
     public static final String SENT = "sent";
     public static final String SENT_BYTES = "sentbytes";
     public static final String SERIAL = "serial";
+    public static final String SERVICE_IP = "serviceip";
     public static final String SERVICE_OFFERING_ID = "serviceofferingid";
     public static final String SESSIONKEY = "sessionkey";
     public static final String SHOW_CAPACITIES = "showcapacities";
diff --git 
a/api/src/main/java/org/apache/cloudstack/api/response/ManagementServerResponse.java
 
b/api/src/main/java/org/apache/cloudstack/api/response/ManagementServerResponse.java
index 4165ea25778..d3f1096d39e 100644
--- 
a/api/src/main/java/org/apache/cloudstack/api/response/ManagementServerResponse.java
+++ 
b/api/src/main/java/org/apache/cloudstack/api/response/ManagementServerResponse.java
@@ -74,6 +74,11 @@ public class ManagementServerResponse extends BaseResponse {
     @Param(description = "the running OS kernel version for this Management 
Server")
     private String kernelVersion;
 
+    @Deprecated
+    @SerializedName(ApiConstants.SERVICE_IP)
+    @Param(description = "the IP Address for this Management Server. This is 
deprecated, please use 'ipaddress' instead.")
+    private String serviceIp;
+
     @SerializedName(ApiConstants.IP_ADDRESS)
     @Param(description = "the IP Address for this Management Server")
     private String ipAddress;
@@ -122,6 +127,10 @@ public class ManagementServerResponse extends BaseResponse 
{
         return lastBoot;
     }
 
+    public String getServiceIp() {
+        return serviceIp;
+    }
+
     public String getIpAddress() {
         return ipAddress;
     }
@@ -170,6 +179,10 @@ public class ManagementServerResponse extends BaseResponse 
{
         this.kernelVersion = kernelVersion;
     }
 
+    public void setServiceIp(String serviceIp) {
+        this.serviceIp = serviceIp;
+    }
+
     public void setIpAddress(String ipAddress) {
         this.ipAddress = ipAddress;
     }
diff --git a/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java 
b/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java
index 729c65cdc63..89f50b6b2e7 100644
--- a/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java
+++ b/server/src/main/java/com/cloud/api/query/QueryManagerImpl.java
@@ -5445,6 +5445,7 @@ public class QueryManagerImpl extends 
MutualExclusiveIdsManagerBase implements Q
                 
mgmtResponse.addPeer(createPeerManagementServerNodeResponse(peer));
             }
         }
+        mgmtResponse.setServiceIp(mgmt.getServiceIP());
         mgmtResponse.setIpAddress(mgmt.getServiceIP());
         mgmtResponse.setObjectName("managementserver");
         return mgmtResponse;