changelog

vatesfr · Nov 27, 2024 · 81feb6c · 81feb6c
1 parent a944fe9
commit 81feb6c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 12 deletions.
diff --git a/CHANGELOG.unreleased.md b/CHANGELOG.unreleased.md
@@ -11,6 +11,8 @@
 
 > Users must be able to say: “Nice enhancement, I'm eager to test it”
 
+- [REST/VM] When creating a VM, the template's VIFs are created. It is also possible to create more VIFs or delete/update template's VIFs (PR [#8137](https://github.com/vatesfr/xen-orchestra/pull/8137))
+
 ### Bug fixes
 
 > Users must be able to say: “I had this issue, happy to know it's fixed”
@@ -31,4 +33,6 @@
 
 <!--packages-start-->
 
+- xo-server minor
+
 <!--packages-end-->
diff --git a/packages/xo-server/src/api/vm.mjs b/packages/xo-server/src/api/vm.mjs
@@ -340,15 +340,6 @@ create.params = {
         },
         device: { type: 'string', optional: true },
         remove: { type: 'boolean', optional: true },
-
-        // device: {
-        //   optional: true,
-        //   type: 'string',
-        // },
-        // remove: {
-        //   optional: true,
-        //   type: 'boolean',
-        // },
       },
     },
   },

diff --git a/packages/xo-web/src/xo-app/new-vm/index.js b/packages/xo-web/src/xo-app/new-vm/index.js
@@ -485,9 +485,9 @@ export default class NewVm extends BaseComponent {
     const resourceSet = this._getResourceSet()
     const { template } = this.props
 
-    // In case the user remove some VIFs created by the template
+    // In case user deletes some VIF created by the template
     // We need to mark them with `remove:true`
-    // so that xo-server correctly remove it
+    // so that xo-server deletes them properly
     if (_VIFs.length < templateVifs.length) {
       const _vifByDevice = keyBy(_VIFs, 'device')
       templateVifs.forEach(templateVif => {
@@ -497,7 +497,6 @@ export default class NewVm extends BaseComponent {
       })
     }
 
-    // return
     // Either use `memory` OR `memory*` params
     let { memory, memoryStaticMax, memoryDynamicMin, memoryDynamicMax } = state
     if ((memoryStaticMax != null || memoryDynamicMin != null) && memoryDynamicMax == null) {