From: Anirudh Rayabharam <anrayabh@linux.microsoft.com>

Implement domainInterfaceAddresses for the Cloud Hypervisor driver.
Support VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_LEASE and
VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_ARP sources. Implementation is
similar to other drivers.

Signed-off-by: Anirudh Rayabharam <anrayabh@linux.microsoft.com>
Signed-off-by: Praveen K Paladugu <praveenkpaladugu@gmail.com>
---
 src/ch/ch_driver.c | 42 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 42 insertions(+)

diff --git a/src/ch/ch_driver.c b/src/ch/ch_driver.c
index 6a8da5f356..d284524337 100644
--- a/src/ch/ch_driver.c
+++ b/src/ch/ch_driver.c
@@ -2298,6 +2298,47 @@ chConnectDomainEventDeregisterAny(virConnectPtr conn,
     return 0;
 }
 
+static int
+chDomainInterfaceAddresses(virDomain *dom,
+                           virDomainInterfacePtr **ifaces,
+                           unsigned int source,
+                           unsigned int flags)
+{
+    virDomainObj *vm = NULL;
+    int ret = -1;
+
+    virCheckFlags(0, -1);
+
+    if (!(vm = virCHDomainObjFromDomain(dom)))
+        goto cleanup;
+
+    if (virDomainInterfaceAddressesEnsureACL(dom->conn, vm->def, source) < 0)
+        goto cleanup;
+
+    if (virDomainObjCheckActive(vm) < 0)
+        goto cleanup;
+
+    switch (source) {
+    case VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_LEASE:
+        ret = virDomainNetDHCPInterfaces(vm->def, ifaces);
+        break;
+
+    case VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_ARP:
+        ret = virDomainNetARPInterfaces(vm->def, ifaces);
+        break;
+
+    default:
+        virReportError(VIR_ERR_ARGUMENT_UNSUPPORTED,
+                       _("Unsupported IP address data source %1$d"),
+                       source);
+        break;
+    }
+
+ cleanup:
+    virDomainObjEndAPI(&vm);
+    return ret;
+}
+
 
 /* Function Tables */
 static virHypervisorDriver chHypervisorDriver = {
@@ -2316,6 +2357,7 @@ static virHypervisorDriver chHypervisorDriver = {
     .domainCreateXML = chDomainCreateXML,                   /* 7.5.0 */
     .domainCreate = chDomainCreate,                         /* 7.5.0 */
     .domainCreateWithFlags = chDomainCreateWithFlags,       /* 7.5.0 */
+    .domainInterfaceAddresses = chDomainInterfaceAddresses, /* 10.10.1 */
     .domainShutdown = chDomainShutdown,                     /* 7.5.0 */
     .domainShutdownFlags = chDomainShutdownFlags,           /* 7.5.0 */
     .domainReboot = chDomainReboot,                         /* 7.5.0 */
-- 
2.47.0

Bubbling up this patch for reviews.

-Praveen

On 12/12/2024 4:31 PM, Praveen K Paladugu wrote:
> From: Anirudh Rayabharam <anrayabh@linux.microsoft.com>
> 
> Implement domainInterfaceAddresses for the Cloud Hypervisor driver.
> Support VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_LEASE and
> VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_ARP sources. Implementation is
> similar to other drivers.
> 
> Signed-off-by: Anirudh Rayabharam <anrayabh@linux.microsoft.com>
> Signed-off-by: Praveen K Paladugu <praveenkpaladugu@gmail.com>
> ---
>   src/ch/ch_driver.c | 42 ++++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 42 insertions(+)
> 
> diff --git a/src/ch/ch_driver.c b/src/ch/ch_driver.c
> index 6a8da5f356..d284524337 100644
> --- a/src/ch/ch_driver.c
> +++ b/src/ch/ch_driver.c
> @@ -2298,6 +2298,47 @@ chConnectDomainEventDeregisterAny(virConnectPtr conn,
>       return 0;
>   }
>   
> +static int
> +chDomainInterfaceAddresses(virDomain *dom,
> +                           virDomainInterfacePtr **ifaces,
> +                           unsigned int source,
> +                           unsigned int flags)
> +{
> +    virDomainObj *vm = NULL;
> +    int ret = -1;
> +
> +    virCheckFlags(0, -1);
> +
> +    if (!(vm = virCHDomainObjFromDomain(dom)))
> +        goto cleanup;
> +
> +    if (virDomainInterfaceAddressesEnsureACL(dom->conn, vm->def, source) < 0)
> +        goto cleanup;
> +
> +    if (virDomainObjCheckActive(vm) < 0)
> +        goto cleanup;
> +
> +    switch (source) {
> +    case VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_LEASE:
> +        ret = virDomainNetDHCPInterfaces(vm->def, ifaces);
> +        break;
> +
> +    case VIR_DOMAIN_INTERFACE_ADDRESSES_SRC_ARP:
> +        ret = virDomainNetARPInterfaces(vm->def, ifaces);
> +        break;
> +
> +    default:
> +        virReportError(VIR_ERR_ARGUMENT_UNSUPPORTED,
> +                       _("Unsupported IP address data source %1$d"),
> +                       source);
> +        break;
> +    }
> +
> + cleanup:
> +    virDomainObjEndAPI(&vm);
> +    return ret;
> +}
> +
>   
>   /* Function Tables */
>   static virHypervisorDriver chHypervisorDriver = {
> @@ -2316,6 +2357,7 @@ static virHypervisorDriver chHypervisorDriver = {
>       .domainCreateXML = chDomainCreateXML,                   /* 7.5.0 */
>       .domainCreate = chDomainCreate,                         /* 7.5.0 */
>       .domainCreateWithFlags = chDomainCreateWithFlags,       /* 7.5.0 */
> +    .domainInterfaceAddresses = chDomainInterfaceAddresses, /* 10.10.1 */
>       .domainShutdown = chDomainShutdown,                     /* 7.5.0 */
>       .domainShutdownFlags = chDomainShutdownFlags,           /* 7.5.0 */
>       .domainReboot = chDomainReboot,                         /* 7.5.0 */

-- 
Regards,
Praveen K Paladugu