Can set the number of VCPUs

virtweb_backend/Cargo.lock

@@ -1624,6 +1624,40 @@ dependencies = [
  "winapi",
 ]
 
+[[package]]
+name = "num"
+version = "0.4.1"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "b05180d69e3da0e530ba2a1dae5110317e49e3b7f3d41be227dc5f92e49ee7af"
+dependencies = [
+ "num-bigint",
+ "num-complex",
+ "num-integer",
+ "num-iter",
+ "num-rational",
+ "num-traits",
+]
+
+[[package]]
+name = "num-bigint"
+version = "0.4.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "608e7659b5c3d7cba262d894801b9ec9d00de989e8a82bd4bef91d08da45cdc0"
+dependencies = [
+ "autocfg",
+ "num-integer",
+ "num-traits",
+]
+
+[[package]]
+name = "num-complex"
+version = "0.4.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1ba157ca0885411de85d6ca030ba7e2a83a28636056c7c699b07c8b6f7383214"
+dependencies = [
+ "num-traits",
+]
+
 [[package]]
 name = "num-integer"
 version = "0.1.45"
@@ -1634,6 +1668,17 @@ dependencies = [
  "num-traits",
 ]
 
+[[package]]
+name = "num-iter"
+version = "0.1.43"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "7d03e6c028c5dc5cac6e2dec0efda81fc887605bb3d884578bb6d6bf7514e252"
+dependencies = [
+ "autocfg",
+ "num-integer",
+ "num-traits",
+]
+
 [[package]]
 name = "num-rational"
 version = "0.4.1"
@@ -1641,6 +1686,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "0638a1c9d0a3c0914158145bc76cff373a75a627e6ecbfb71cbe6f453a5a19b0"
 dependencies = [
  "autocfg",
+ "num-bigint",
  "num-integer",
  "num-traits",
 ]
@@ -2583,6 +2629,7 @@ dependencies = [
  "lazy_static",
  "light-openid",
  "log",
+ "num",
  "rand",
  "reqwest",
  "serde",

virtweb_backend/Cargo.toml

@@ -39,4 +39,5 @@ rand = "0.8.5"
 bytes = "1.5.0"
 tokio = "1.32.0"
 futures = "0.3.28"
-ipnetwork = "0.20.0"
+ipnetwork = "0.20.0"
+num = "0.4.1"

virtweb_backend/src/controllers/server_controller.rs

@@ -91,6 +91,23 @@ pub async fn server_info(client: LibVirtReq) -> HttpResult {
     }))
 }
 
+pub async fn number_vcpus() -> HttpResult {
+    let mut system = System::new();
+    system.refresh_cpu();
+    let number_cpus = system.cpus().len();
+    assert_ne!(number_cpus, 0, "Got invlid number of CPU!");
+
+    let mut possible_numbers = vec![1];
+
+    if number_cpus > 1 {
+        for i in 0..(number_cpus / 2) {
+            possible_numbers.push(2 + 2 * i);
+        }
+    }
+
+    Ok(HttpResponse::Ok().json(possible_numbers))
+}
+
 pub async fn networks_list() -> HttpResult {
     let mut system = System::new();
     system.refresh_networks_list();

virtweb_backend/src/libvirt_lib_structures.rs

@@ -192,6 +192,13 @@ pub struct DomainCPUTopology {
     pub threads: usize,
 }
 
+#[derive(serde::Serialize, serde::Deserialize)]
+#[serde(rename = "cpu")]
+pub struct DomainVCPUXML {
+    #[serde(rename = "$value")]
+    pub body: usize,
+}
+
 #[derive(serde::Serialize, serde::Deserialize)]
 #[serde(rename = "cpu")]
 pub struct DomainCPUXML {
@@ -221,6 +228,9 @@ pub struct DomainXML {
     /// The maximum allocation of memory for the guest at boot time
     pub memory: DomainMemoryXML,
 
+    /// Number of vCPU
+    pub vcpu: DomainVCPUXML,
+
     /// CPU information
     pub cpu: DomainCPUXML,
 

virtweb_backend/src/libvirt_rest_structures.rs

@@ -2,15 +2,17 @@ use crate::app_config::AppConfig;
 use crate::constants;
 use crate::libvirt_lib_structures::{
     DevicesXML, DiskBootXML, DiskDriverXML, DiskReadOnlyXML, DiskSourceXML, DiskTargetXML, DiskXML,
-    DomainCPUTopology, DomainCPUXML, DomainMemoryXML, DomainXML, FeaturesXML, GraphicsXML,
+    DomainCPUTopology, DomainCPUXML, DomainMemoryXML, DomainVCPUXML, DomainXML, FeaturesXML,
-    NetworkDHCPRangeXML, NetworkDHCPXML, NetworkDNSForwarderXML, NetworkDNSXML, NetworkDomainXML,
+    GraphicsXML, NetworkDHCPRangeXML, NetworkDHCPXML, NetworkDNSForwarderXML, NetworkDNSXML,
-    NetworkForwardXML, NetworkIPXML, NetworkXML, OSLoaderXML, OSTypeXML, XMLUuid, ACPIXML, OSXML,
+    NetworkDomainXML, NetworkForwardXML, NetworkIPXML, NetworkXML, OSLoaderXML, OSTypeXML, XMLUuid,
+    ACPIXML, OSXML,
 };
 use crate::libvirt_rest_structures::LibVirtStructError::StructureExtraction;
 use crate::utils::disks_utils::Disk;
 use crate::utils::files_utils;
 use ipnetwork::{Ipv4Network, Ipv6Network};
 use lazy_regex::regex;
+use num::Integer;
 use std::net::{IpAddr, Ipv4Addr, Ipv6Addr};
 use std::ops::{Div, Mul};
 
@@ -74,6 +76,8 @@ pub struct VMInfo {
     pub architecture: VMArchitecture,
     /// VM allocated memory, in megabytes
     pub memory: usize,
+    /// Number of vCPU for the VM
+    pub number_vcpu: usize,
     /// Enable VNC access through admin console
     pub vnc_access: bool,
     /// Attach an ISO file
@@ -81,7 +85,6 @@ pub struct VMInfo {
     /// Storage - https://access.redhat.com/documentation/fr-fr/red_hat_enterprise_linux/6/html/virtualization_administration_guide/sect-virtualization-virtualized_block_devices-adding_storage_devices_to_guests#sect-Virtualization-Adding_storage_devices_to_guests-Adding_file_based_storage_to_a_guest
     pub disks: Vec<Disk>,
     // TODO : network interfaces
-    // TODO : number of CPUs
 }
 
 impl VMInfo {
@@ -116,6 +119,10 @@ impl VMInfo {
             return Err(StructureExtraction("VM memory is invalid!").into());
         }
 
+        if self.number_vcpu == 0 || (self.number_vcpu != 1 && self.number_vcpu.is_odd()) {
+            return Err(StructureExtraction("Invalid number of vCPU specified!").into());
+        }
+
         let mut disks = vec![];
 
         if let Some(iso_file) = &self.iso_file {
@@ -245,12 +252,22 @@ impl VMInfo {
                 memory: self.memory,
             },
 
+            vcpu: DomainVCPUXML {
+                body: self.number_vcpu,
+            },
+
             cpu: DomainCPUXML {
                 mode: "host-passthrough".to_string(),
                 topology: Some(DomainCPUTopology {
                     sockets: 1,
-                    cores: 1,
+                    cores: match self.number_vcpu {
-                    threads: 1,
+                        1 => 1,
+                        v => v / 2,
+                    },
+                    threads: match self.number_vcpu {
+                        1 => 1,
+                        _ => 2,
+                    },
                 }),
             },
 
@@ -285,6 +302,7 @@ impl VMInfo {
                     .into());
                 }
             },
+            number_vcpu: domain.vcpu.body,
             memory: convert_to_mb(&domain.memory.unit, domain.memory.memory)?,
             vnc_access: domain.devices.graphics.is_some(),
             iso_file: domain

virtweb_backend/src/main.rs

@@ -102,6 +102,10 @@ async fn main() -> std::io::Result<()> {
                 "/api/server/info",
                 web::get().to(server_controller::server_info),
             )
+            .route(
+                "/api/server/number_vcpus",
+                web::get().to(server_controller::number_vcpus),
+            )
             .route(
                 "/api/server/networks",
                 web::get().to(server_controller::networks_list),

virtweb_frontend/src/api/ServerApi.ts

@@ -172,6 +172,18 @@ export class ServerApi {
     ).data;
   }
 
+  /**
+   * Get host supported vCPUs configurations
+   */
+  static async NumberVCPUs(): Promise<number[]> {
+    return (
+      await APIClient.exec({
+        method: "GET",
+        uri: "/server/number_vcpus",
+      })
+    ).data;
+  }
+
   /**
    * Get host networks card list
    */

virtweb_frontend/src/api/VMApi.ts

@@ -39,6 +39,7 @@ interface VMInfoInterface {
   boot_type: "UEFI" | "UEFISecureBoot";
   architecture: "i686" | "x86_64";
   memory: number;
+  number_vcpu: number;
   vnc_access: boolean;
   iso_file?: string;
   disks: VMDisk[];
@@ -52,6 +53,7 @@ export class VMInfo implements VMInfoInterface {
   description?: string;
   boot_type: "UEFI" | "UEFISecureBoot";
   architecture: "i686" | "x86_64";
+  number_vcpu: number;
   memory: number;
   vnc_access: boolean;
   iso_file?: string;
@@ -65,6 +67,7 @@ export class VMInfo implements VMInfoInterface {
     this.description = int.description;
     this.boot_type = int.boot_type;
     this.architecture = int.architecture;
+    this.number_vcpu = int.number_vcpu;
     this.memory = int.memory;
     this.vnc_access = int.vnc_access;
     this.iso_file = int.iso_file;
@@ -77,6 +80,7 @@ export class VMInfo implements VMInfoInterface {
       boot_type: "UEFI",
       architecture: "x86_64",
       memory: 1024,
+      number_vcpu: 1,
       vnc_access: true,
       disks: [],
     });

virtweb_frontend/src/routes/VNCRoute.tsx

@@ -80,7 +80,7 @@ function VNCInner(p: { vm: VMInfo }): React.ReactElement {
       }}
     >
       <VncScreen
-        url={token!.url}
+        url={token.url}
         onDisconnect={() => {
           console.info("VNC disconnected " + token?.url);
           disconnected();

virtweb_frontend/src/widgets/vms/VMDetails.tsx

@@ -22,10 +22,14 @@ interface DetailsProps {
 }
 
 export function VMDetails(p: DetailsProps): React.ReactElement {
-  const [list, setList] = React.useState<IsoFile[] | any>();
+  const [isoList, setIsoList] = React.useState<IsoFile[] | any>();
+  const [vcpuCombinations, setVCPUCombinations] = React.useState<
+    number[] | any
+  >();
 
   const load = async () => {
-    setList(await IsoFilesApi.GetList());
+    setIsoList(await IsoFilesApi.GetList());
+    setVCPUCombinations(await ServerApi.NumberVCPUs());
   };
 
   return (
@@ -33,13 +37,19 @@ export function VMDetails(p: DetailsProps): React.ReactElement {
       loadKey={"1"}
       load={load}
       errMsg="Failed to load the list of ISO files"
-      build={() => <VMDetailsInner isoList={list} {...p} />}
+      build={() => (
+        <VMDetailsInner
+          isoList={isoList}
+          vcpuCombinations={vcpuCombinations}
+          {...p}
+        />
+      )}
     />
   );
 }
 
 function VMDetailsInner(
-  p: DetailsProps & { isoList: IsoFile[] }
+  p: DetailsProps & { isoList: IsoFile[]; vcpuCombinations: number[] }
 ): React.ReactElement {
   return (
     <Grid container spacing={2}>
@@ -146,6 +156,19 @@ function VMDetailsInner(
           }
         />
 
+        <SelectInput
+          editable={p.editable}
+          label="Number of vCPU"
+          options={p.vcpuCombinations.map((v) => {
+            return { label: v.toString(), value: v.toString() };
+          })}
+          value={p.vm.number_vcpu.toString()}
+          onValueChange={(v) => {
+            p.vm.number_vcpu = Number(v ?? "0");
+            p.onChange?.();
+          }}
+        />
+
         <CheckboxInput
           editable={p.editable}
           label="Enable VNC access"