update readme and componentstory to use new menu components

.eslintrc.js

@@ -171,6 +171,7 @@ module.exports = {
         'vue/no-empty-component-block': 'error',
         'vue/no-multiple-objects-in-class': 'error',
         'vue/no-ref-object-reactivity-loss': 'error',
+        'vue/no-required-prop-with-default': 'error',
         'vue/no-static-inline-styles': 'error',
         'vue/no-template-target-blank': 'error',
         'vue/no-undef-components': ['error', { ignorePatterns: ['RouterLink', 'RouterView', 'I18nT'] }],
@@ -191,14 +192,18 @@ module.exports = {
     // Specific rules for XO i18n-related files
     {
       files: [
-        '@xen-orchestra/lite/**/*.{vue,ts}',
-        '@xen-orchestra/web/**/*.{vue,ts}',
+        '@xen-orchestra/lite/**/*.{vue,ts,json}',
+        '@xen-orchestra/web/**/*.{vue,ts,json}',
         '@xen-orchestra/web-core/**/*.{vue,ts,json}',
       ],
       extends: ['plugin:@intlify/vue-i18n/base'],
       settings: {
         'vue-i18n': {
-          localeDir: ['@xen-orchestra/web-core/lib/locales/*.json'],
+          localeDir: [
+            '@xen-orchestra/lite/src/locales/*.json',
+            '@xen-orchestra/web/src/locales/*.json',
+            '@xen-orchestra/web-core/lib/locales/*.json',
+          ],
           messageSyntaxVersion: '^9.9.0',
         },
       },
@@ -216,25 +221,13 @@ module.exports = {
         '@intlify/vue-i18n/no-raw-text': [
           'error',
           {
-            ignoreText: [
-              'Xen Orchestra Lite',
-              'Xen Orchestra',
-              'Xen Orchestra Appliance',
-              'XO Lite',
-              'XCP-ng',
-              'XO 5',
-              '404',
-              '⚠️',
-            ],
+            ignoreText: ['Xen Orchestra Lite', 'Xen Orchestra', 'XO Lite', 'XCP-ng', 'XO 5', '404', '⚠️'],
           },
         ],
         '@intlify/vue-i18n/no-v-html': 'error',
         '@intlify/vue-i18n/valid-message-syntax': 'error',
         '@intlify/vue-i18n/no-duplicate-keys-in-locale': 'error',
-        '@intlify/vue-i18n/no-missing-keys-in-other-locales': [
-          'error',
-          { ignoreLocales: ['de', 'fa', 'cs', 'es', 'sv', 'it', 'ru', 'uk', 'nl', 'pt_BR'] },
-        ],
+        '@intlify/vue-i18n/no-missing-keys-in-other-locales': ['error', { ignoreLocales: ['de', 'fa', 'cs'] }],
       },
     },
     // Specific rules for XO dev-related files

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -40,8 +40,8 @@ body:
   - type: markdown
     attributes:
       value: |
-        - Don't forget to [read this first](https://docs.xen-orchestra.com/community)
-        - As well as follow [this guide](https://docs.xen-orchestra.com/community#report-a-bug)
+        - Don't forget to [read this first](https://xen-orchestra.com/docs/community.html)
+        - As well as follow [this guide](https://xen-orchestra.com/docs/community.html#report-a-bug)
   - type: input
     id: xo-sources-commit-number
     attributes:

.github/workflows/ci.yml

@@ -1,4 +1,4 @@
-name: Continuous Integration
+name: Continous Integration
 on: push
 jobs:
   CI:

.gitignore

@@ -7,8 +7,6 @@
 /@vates/*/node_modules/
 /@xen-orchestra/*/dist/
 /@xen-orchestra/*/node_modules/
-/@xen-orchestra/rest-api/open-api/spec/swagger.json
-/@xen-orchestra/rest-api/src/open-api/routes/routes.ts
 /packages/*/dist/
 /packages/*/node_modules/
 

@vates/async-each/.USAGE.md

@@ -16,7 +16,7 @@ Returns a promise wich rejects as soon as a call to `iteratee` throws or a promi
 
 - `concurrency`: a number which indicates the maximum number of parallel call to `iteratee`, defaults to `10`. The value `0` means no concurrency limit.
 - `signal`: an abort signal to stop the iteration
-- `stopOnError`: whether to stop iteration of first error, or wait for all calls to finish and throw an `AggregateError`, defaults to `true`
+- `stopOnError`: wether to stop iteration of first error, or wait for all calls to finish and throw an `AggregateError`, defaults to `true`
 
 ```js
 import { asyncEach } from '@vates/async-each'

@vates/async-each/README.md

@@ -34,7 +34,7 @@ Returns a promise wich rejects as soon as a call to `iteratee` throws or a promi
 
 - `concurrency`: a number which indicates the maximum number of parallel call to `iteratee`, defaults to `10`. The value `0` means no concurrency limit.
 - `signal`: an abort signal to stop the iteration
-- `stopOnError`: whether to stop iteration of first error, or wait for all calls to finish and throw an `AggregateError`, defaults to `true`
+- `stopOnError`: wether to stop iteration of first error, or wait for all calls to finish and throw an `AggregateError`, defaults to `true`
 
 ```js
 import { asyncEach } from '@vates/async-each'

@vates/async-each/index.js

@@ -11,8 +11,8 @@ class AggregateError extends Error {
 
 /**
  * @template Item
- * @param {Iterable<Item>|AsyncIterable<Item>} iterable
- * @param {(item: Item, index: number, iterable: Iterable<Item>|AsyncIterable<Item>) => Promise<void>} iteratee
+ * @param {Iterable<Item>} iterable
+ * @param {(item: Item, index: number, iterable: Iterable<Item>) => Promise<void>} iteratee
  * @returns {Promise<void>}
  */
 exports.asyncEach = function asyncEach(iterable, iteratee, { concurrency = 10, signal, stopOnError = true } = {}) {

@vates/diff/index.js

@@ -7,7 +7,7 @@
  * @param {Array|Buffer|string} data1 - The first data array or buffer to compare.
  * @param {T} data2 - The second data array or buffer to compare.
  * @param {(index: number, diff: T) => void} [cb] - The callback function to invoke for each difference. If not provided, an array of differences will be returned.
- * @returns {Array<number|T>|undefined} - An array of differences if no callback is provided; otherwise, undefined.
+ * @returns {Array<number|T>|undefined} - An array of differences if no callback is provided, otherwise undefined.
  */
 module.exports = function diff(data1, data2, cb) {
   let result

@vates/disposable/.USAGE.md

@@ -14,7 +14,7 @@ import { deduped } from '@vates/disposable/deduped'
 // the connection with the passed host will be established once at the first call, then, it will be shared with the next calls
 const getConnection = deduped(async function (host)) {
   const connection = new Connection(host)
-  return new Disposable(connection, () => connection.close())
+  return new Disposabe(connection, () => connection.close())
 }, host => [host])
 ```
 

@vates/disposable/README.md

@@ -32,7 +32,7 @@ import { deduped } from '@vates/disposable/deduped'
 // the connection with the passed host will be established once at the first call, then, it will be shared with the next calls
 const getConnection = deduped(async function (host)) {
   const connection = new Connection(host)
-  return new Disposable(connection, () => connection.close())
+  return new Disposabe(connection, () => connection.close())
 }, host => [host])
 ```
 

@vates/fatfs/.gitignore

@@ -0,0 +1,3 @@
+.DS_Store
+.vscode/
+node_modules/

@vates/fatfs/README.md

@@ -28,7 +28,7 @@ fs.stat('autoexec.bat', function (e, stats) {
 
 ## API
 
-- `fs = fatfs.createFileSystem(vol, [opts], [cb])` — Simply pass in a block driver (see below) mapped to a FAT partition somewhere, and get back an instance of an [`fs`](http://nodejs.org/api/fs.html). An options dictionary can be provided, details are in the next section. You may also optionally provide a callback `cb(err)` which will be automatically registered for the on `'ready'` or `'error'` event.
+- `fs = fatfs.createFileSystem(vol, [opts], [cb])` — Simply pass in a block driver (see below) mapped to a FAT partition somewhere, and get back the API documented [here](http://nodejs.org/api/fs.html). An options dictionary can be provided, details are in the next section. You may also optionally provide a callback `cb(err)` which will be automatically registered for the on `'ready'` or `'error'` event.
 - `'ready'` event — fired on `fs` when initial volume information has been determined and the API is ready to use. It is safe to call other `fs` methods before this fires **only if** you are sure the first sector will be readable and represents a valid FAT volume.
 - `'error'` event — fired if initialization fails for whatever reason.
 
@@ -36,7 +36,7 @@ fs.stat('autoexec.bat', function (e, stats) {
 
 The `opts` dictionary you pass to `fatfs.createFileSystem` can contain any of the following options:
 
-- `ro` — Enables readonly mode if `true`. It defaults to `false`, but if your volume driver does not provide a `writeSectors` method it will be overridden to `true`.
+- `ro` — Enables readonly mode if `true`. It defaults to `false`, but if your volume driver does not provide a `writeSectors` method it will be overriden to `true`.
 - `noatime` — The FAT filesystem can track the last access time (just a date, actually) but this means every read operation would also incur some write overhead. Defaults to `true`, meaning by default access times will **not** be updated on reads. Set this to `false` to track access times.
 - `modmode` — chooses how `fs.chmod` (and the mode field from `fs.stat`–family calls) should map FAT attributes to POSIX permissions. Set to the number `0111` to map the readonly flag to the user's write bit being unset, and the archive/system/hidden flags to the user/group/other executable bits respectively. Set to the number `07000` to map the readonly flag to _all_ write bits being unset, and the archive/system/hidden flags to the sticky/setgid/setuid bits respectively. Set to `null` for readonly mapping. Defaults to `0111`.
 - `umask` — any bits _set_ in this octal number will be _unset_ in the 'mode' field from `fs.stat`–family calls. It does not affect anything else. Defaults to `process.umask()`, or `0022` if that is unavailable.
@@ -53,7 +53,7 @@ Well, sort of…
 
 ### Temporary
 
-- **BETA** **BETA** **BETA**. Seriously, this is a _brand new_, _from scratch_, _completely unproven_ filesystem implementation. It does not have full automated test coverage, and it has not been manually tested very much either. Please, please, please, **make sure you have a backup** of any important drive/image/card you unleash this upon.
+- **BETA** **BETA** **BETA**. Seriously, this is a _brand new_, _from scratch_, _completely unproven_ filesystem implementation. It does not have full automated test coverage, and it has not been manually tested very much either. Please please please **make sure you have a backup** of any important drive/image/card you unleash this upon.
 - A few methods are not quite implemented, either: `fs.rename`, `fs.unlink` and `fs.rmdir`, as well as `fs.watchFile`/`fs.unwatchFile` and `fs.watch`. These are Coming Soon™.
 - There are several internal housekeeping items (redundant FAT tables, extra FAT32 information, etc.) that are not done. These do not seem to affect interop, but you may see warnings when repairing a filesystem written by this module.
 - Oh, and not to scare you, but if an IO error happens while writing, the library usually just bails — bubbling an error up to your callback as if it were a hot potato. Although some attempt has been made to do separate writes in the safest order (e.g. allocating an additional file cluster, then appending data into it, and then finally updating the file's size), but this behavior has not been thoroughly audited for all operations. There's certainly no attempt to retry/cleanup/rollback if a multi-step change runs into trouble partway through.

@vates/fatfs/package.json

@@ -8,9 +8,8 @@
     "postversion": "npm publish --access public"
   },
   "repository": {
-    "directory": "@vates/fatfs",
     "type": "git",
-    "url": "https://github.com/vatesfr/xen-orchestra.git"
+    "url": "https://github.com/vatesfr/xen-orchestra"
   },
   "keywords": [
     "filesystem",
@@ -21,14 +20,13 @@
   ],
   "author": "Vates SAS",
   "license": "Apache-2.0",
-  "bugs": "https://github.com/vatesfr/xen-orchestra/issues",
-  "homepage": "https://github.com/vatesfr/xen-orchestra/tree/master/@vates/fatfs",
+  "bugs": {
+    "url": "https://github.com/vatesfr/xen-orchestra/issues"
+  },
+  "homepage": "https://github.com/vatesfr/xen-orchestra",
   "dependencies": {
     "fifolock": "^1.0.0",
     "struct-fu": "^1.2.1",
     "xok": "^1.0.0"
-  },
-  "engines": {
-    "node": ">=18"
   }
 }

@vates/fuse-vhd/.USAGE.md

@@ -13,7 +13,7 @@ await mount(handler, diskId, mountPoint)
 
 ### cli
 
-From the install folder:
+From the install folder :
 
 ```
 cli.mjs <remoteUrl> <vhdPathInRemote> <mountPoint>
@@ -25,25 +25,4 @@ After installing the package
 xo-fuse-vhd <remoteUrl> <vhdPathInRemote> <mountPoint>
 ```
 
-The `remoteUrl` can be found using the following command from the XOA CLI, and then picking the right `url` property:
-
-`xo-server-db ls remote`
-
-The `vhdPathInRemote` follows a file structure described in [VM backups](https://github.com/vatesfr/xen-orchestra/blob/master/%40xen-orchestra/backups/docs/VM%20backups/README.md).
-
-## Restore a file from a VHD using `fuse-vhd` CLI
-
-1. Mount a VHD to the Filesystem (see **Usage**).
-2. Verify the VHD is Correctly Mounted:
-
-`mount | grep fuse`
-
-3. Get the `START` and `SIZE` of the disk whose file you want to restore. Multiply `START` by 512 (block size) to get the offset value. (`mountedVhdPath` is the `mountPoint` value from earlier, followed by `/vhd0`, for example). Depending on the partition type, you may need to do some additional setup to discover the partitions.
-
-`partx --bytes --output=NR,START,SIZE,NAME,UUID,TYPE --pairs <mountedVhdPath>`
-
-4.  Mount the disk. Depending on the partition type, you may need to add some additional options. The `norecovery` option is used for ext3/ext4/xfs file systems; otherwise, remove this option.
-
-`mount --options=loop,ro,norecovery,sizelimit=<SIZE>,offset=<START*512>  --source=<mountedVhdPath> --target=<diskMountPoint>`
-
-5. Copy your files from the mounted partition, then unmount the partition.
+remoteUrl can be found by using cli in `@xen-orchestra/fs` , for example a local remote will have a url like `file:///path/to/remote/root`

@vates/fuse-vhd/README.md

@@ -29,7 +29,7 @@ await mount(handler, diskId, mountPoint)
 
 ### cli
 
-From the install folder:
+From the install folder :
 
 ```
 cli.mjs <remoteUrl> <vhdPathInRemote> <mountPoint>
@@ -41,28 +41,7 @@ After installing the package
 xo-fuse-vhd <remoteUrl> <vhdPathInRemote> <mountPoint>
 ```
 
-The `remoteUrl` can be found using the following command from the XOA CLI, and then picking the right `url` property:
-
-`xo-server-db ls remote`
-
-The `vhdPathInRemote` follows a file structure described in [VM backups](https://github.com/vatesfr/xen-orchestra/blob/master/%40xen-orchestra/backups/docs/VM%20backups/README.md).
-
-## Restore a file from a VHD using `fuse-vhd` CLI
-
-1. Mount a VHD to the Filesystem (see **Usage**).
-2. Verify the VHD is Correctly Mounted:
-
-`mount | grep fuse`
-
-3. Get the `START` and `SIZE` of the disk whose file you want to restore. Multiply `START` by 512 (block size) to get the offset value. (`mountedVhdPath` is the `mountPoint` value from earlier, followed by `/vhd0`, for example). Depending on the partition type, you may need to do some additional setup to discover the partitions.
-
-`partx --bytes --output=NR,START,SIZE,NAME,UUID,TYPE --pairs <mountedVhdPath>`
-
-4.  Mount the disk. Depending on the partition type, you may need to add some additional options. The `norecovery` option is used for ext3/ext4/xfs file systems; otherwise, remove this option.
-
-`mount --options=loop,ro,norecovery,sizelimit=<SIZE>,offset=<START*512>  --source=<mountedVhdPath> --target=<diskMountPoint>`
-
-5. Copy your files from the mounted partition, then unmount the partition.
+remoteUrl can be found by using cli in `@xen-orchestra/fs` , for example a local remote will have a url like `file:///path/to/remote/root`
 
 ## Contributions
 

@vates/fuse-vhd/package.json

@@ -19,12 +19,12 @@
   },
   "main": "./index.mjs",
   "dependencies": {
-    "@xen-orchestra/fs": "^4.6.3",
+    "@xen-orchestra/fs": "^4.3.0",
     "@xen-orchestra/log": "^0.7.1",
     "fuse-native": "^2.2.6",
     "lru-cache": "^7.14.0",
     "promise-toolbox": "^0.21.0",
-    "vhd-lib": "^4.14.1"
+    "vhd-lib": "^4.11.2"
   },
   "bin": {
     "xo-fuse-vhd": "./cli.mjs"

@vates/generator-toolbox/.USAGE.md

@@ -1,58 +0,0 @@
-A toolbox to ease the use of generator
-
-### Timeout
-
-wrap a source async generator to have it throw an error when timeout is reached
-timeout is a positive number in milliseconds
-
-```js
-import { Timeout } from '@vates/generator-toolbox'
-
-const wrappedGenerator = new Timeout(sourceGenerator, timeout)
-```
-
-### Throttle
-
-wrap a source async generator to have it respect a max speed ( in bytes per seconds).
-speed is either a strictly positive number or a function returning a strictly positive number. A speed change will be used for the next emitted packet.
-
-The source generator must yield object with a length property.
-
-Optimized for small yields regarding to the speed, since it won't split incoming packet.
-
-If the generator reached the max speed it will be paused, limiting memory consumption.
-
-```js
-import { Throttle } from '@vates/generator-toolbox'
-
-const wrappedGenerator = new Throttle(sourceGenerator, speed)
-```
-
-### Synchronized
-
-Fork a generator. The rules are:
-
-- if the source returns, all the forks return
-- if the forks error, all the forks error with the same error
-- if a fork returns, it is stopped, but the generator continue with the other
-- if a fork errors, it is stopped, but the generator continue with the other
-- if all the forks return, the source is stopped
-- if all the forks error, the source is errored with the last error
-- the source start producing a packet when the fastest forked ask for it
-- the source forks get the packet only when all the forks asked for it, no buffer stores in memory
-
-```ts
-import { Synchronized } from '@vates/generator-toolbox'
-
-async function consume(generator: AsyncGenerator) {
-  for await (const val of generator) {
-    console.log({ val })
-  }
-}
-const forker = new Synchronized(generator)
-const first = forker.fork('first')
-const second = forker.fork('second')
-await Promise.all([consume(first), consume(second)])
-```
-
-Note: you can stop early a generator by calling `generator.return()`, and you can stop in in error by calling `generator.throw(error)`

@vates/generator-toolbox/.npmignore

@@ -1 +0,0 @@
-../../scripts/npmignore

@vates/generator-toolbox/README.md

@@ -1,89 +0,0 @@
-<!-- DO NOT EDIT MANUALLY, THIS FILE HAS BEEN GENERATED -->
-
-# @vates/generator-toolbox
-
-[![Package Version](https://badgen.net/npm/v/@vates/generator-toolbox)](https://npmjs.org/package/@vates/generator-toolbox) ![License](https://badgen.net/npm/license/@vates/generator-toolbox) [![PackagePhobia](https://badgen.net/bundlephobia/minzip/@vates/generator-toolbox)](https://bundlephobia.com/result?p=@vates/generator-toolbox) [![Node compatibility](https://badgen.net/npm/node/@vates/generator-toolbox)](https://npmjs.org/package/@vates/generator-toolbox)
-
-## Install
-
-Installation of the [npm package](https://npmjs.org/package/@vates/generator-toolbox):
-
-```sh
-npm install --save @vates/generator-toolbox
-```
-
-## Usage
-
-A toolbox to ease the use of generator
-
-### Timeout
-
-wrap a source async generator to have it throw an error when timeout is reached
-timeout is a positive number in milliseconds
-
-```js
-import { Timeout } from '@vates/generator-toolbox'
-
-const wrappedGenerator = new Timeout(sourceGenerator, timeout)
-```
-
-### Throttle
-
-wrap a source async generator to have it respect a max speed ( in bytes per seconds).
-speed is either a strictly positive number or a function returning a strictly positive number. A speed change will be used for the next emitted packet.
-
-The source generator must yield object with a length property.
-
-Optimized for small yields regarding to the speed, since it won't split incoming packet.
-
-If the generator reached the max speed it will be paused, limiting memory consumption.
-
-```js
-import { Throttle } from '@vates/generator-toolbox'
-
-const wrappedGenerator = new Throttle(sourceGenerator, speed)
-```
-
-### Synchronized
-
-Fork a generator. The rules are:
-
-- if the source returns, all the forks return
-- if the forks error, all the forks error with the same error
-- if a fork returns, it is stopped, but the generator continue with the other
-- if a fork errors, it is stopped, but the generator continue with the other
-- if all the forks return, the source is stopped
-- if all the forks error, the source is errored with the last error
-- the source start producing a packet when the fastest forked ask for it
-- the source forks get the packet only when all the forks asked for it, no buffer stores in memory
-
-```ts
-import { Synchronized } from '@vates/generator-toolbox'
-
-async function consume(generator: AsyncGenerator) {
-  for await (const val of generator) {
-    console.log({ val })
-  }
-}
-const forker = new Synchronized(generator)
-const first = forker.fork('first')
-const second = forker.fork('second')
-await Promise.all([consume(first), consume(second)])
-```
-
-Note: you can stop early a generator by calling `generator.return()`, and you can stop in in error by calling `generator.throw(error)`
-
-## Contributions
-
-Contributions are _very_ welcomed, either on the documentation or on
-the code.
-
-You may:
-
-- report any [issue](https://github.com/vatesfr/xen-orchestra/issues)
-  you've encountered;
-- fork and create a pull request.
-
-## License
-
-[MIT](https://spdx.org/licenses/MIT) © [Vates SAS](https://vates.fr)

@vates/generator-toolbox/eslint.config.mjs

@@ -1,6 +0,0 @@
-// @ts-check
-
-import eslint from '@eslint/js'
-import tseslint from 'typescript-eslint'
-
-export default tseslint.config(eslint.configs.recommended, tseslint.configs.recommended)

@vates/generator-toolbox/package.json

@@ -1,38 +0,0 @@
-{
-  "name": "@vates/generator-toolbox",
-  "version": "1.0.4",
-  "main": "dist/index.mjs",
-  "license": "MIT",
-  "private": false,
-  "type": "module",
-  "devDependencies": {
-    "@eslint/js": "^9.19.0",
-    "rimraf": "^5.0.1",
-    "@types/node": "^20.6",
-    "typescript": "~5.6",
-    "typescript-eslint": "^8.23.0"
-  },
-  "scripts": {
-    "build": "tsc",
-    "dev": "tsc --watch",
-    "postversion": "npm publish --access public",
-    "prebuild": "rimraf dist/",
-    "predev": "yarn run prebuild",
-    "test": "tsc && node --test **/*.test.mjs",
-    "clean": "rimraf dist/"
-  },
-  "homepage": "https://github.com/vatesfr/xen-orchestra/tree/master/@vates/generator-toolbox",
-  "bugs": "https://github.com/vatesfr/xen-orchestra/issues",
-  "repository": {
-    "directory": "@vates/generator-toolbox",
-    "type": "git",
-    "url": "https://github.com/vatesfr/xen-orchestra.git"
-  },
-  "author": {
-    "name": "Vates SAS",
-    "url": "https://vates.fr"
-  },
-  "engines": {
-    "node": ">=20.18"
-  }
-}

@vates/generator-toolbox/src/index.mts

@@ -1,3 +0,0 @@
-export { Synchronized } from './synchronized.mjs'
-export { Throttle } from './throttle.mjs'
-export { Timeout } from './timeout.mjs'

@vates/generator-toolbox/src/synchronized.mts

@@ -1,157 +0,0 @@
-import assert from 'node:assert'
-
-export class Synchronized<T, TReturn, TNext> {
-  #source: AsyncGenerator<T, TReturn, TNext>
-  #forks = new Map<string, Forked<T, TReturn, TNext>>()
-  #removedForks = new Set<string>()
-  #waitingForks = new Set<string>()
-  #started = false
-
-  #nextValueForksReady?: {
-    promise: Promise<IteratorResult<T>>
-    forksWaitingReject: (error: Error) => void
-    forksWaitingResolve: () => void
-  }
-
-  constructor(source: AsyncGenerator<T, TReturn, TNext>) {
-    this.#source = source
-  }
-
-  fork(uid: string): AsyncGenerator<T, TReturn, TNext> {
-    assert.strictEqual(this.#started, false, `can't create a fork after consuming the data`)
-    assert.strictEqual(this.#forks.has(uid), false, `a fork with id ${uid} already exists`)
-
-    const fork = new Forked<T, TReturn, TNext>(this, uid)
-    this.#forks.set(uid, fork)
-    return fork
-  }
-
-  async #resolveWhenAllForksReady(): Promise<IteratorResult<T>> {
-    if (!this.#nextValueForksReady) {
-      throw new Error('Can t wait forks if there are noone waiting')
-    }
-    const { promise, forksWaitingResolve } = this.#nextValueForksReady
-    if (this.#waitingForks.size === this.#forks.size) {
-      // reset value
-      this.#waitingForks.clear()
-      this.#nextValueForksReady = undefined
-      forksWaitingResolve() // for the other forks waiting
-    }
-    return promise
-  }
-
-  async next(uid: string): Promise<IteratorResult<T>> {
-    if (this.#removedForks.has(uid)) {
-      return { done: true, value: undefined }
-    }
-    if (!this.#forks.has(uid)) {
-      throw new Error(`trying to advance fork ${uid} that is not a fork of this one`)
-    }
-
-    if (this.#waitingForks.has(uid)) {
-      throw new Error(`Fork ${uid} is already waiting`)
-    }
-
-    this.#started = true
-    if (this.#nextValueForksReady === undefined) {
-      let forksWaitingResolve = () => {}
-      let forksWaitingReject: (reason?: Error) => void = () => {}
-      const next = this.#source.next().catch(async error => {
-        const e = new Error(`Error in the source generator ${error.message}`, { cause: error })
-        forksWaitingReject(e)
-        // source has failed, kill everything, and stop the forks
-        for (const uid of [...this.#forks.keys()]) {
-          await this.remove(uid, error)
-        }
-      })
-      const promise = Promise.all([
-        next,
-        new Promise((_resolve, _reject) => {
-          forksWaitingResolve = () => _resolve(undefined)
-          forksWaitingReject = _reject
-        }),
-      ]).then(([_]) => _ as IteratorResult<T>)
-
-      this.#nextValueForksReady = { promise, forksWaitingResolve, forksWaitingReject }
-    }
-    this.#waitingForks.add(uid)
-    return this.#resolveWhenAllForksReady()
-  }
-
-  async remove(uid: string, error?: Error): Promise<IteratorResult<T>> {
-    const fork = this.#forks.get(uid)
-    if (fork === undefined) {
-      if (this.#removedForks.has(uid)) {
-        // already removed
-        return { done: true, value: undefined }
-      }
-      throw new Error(`trying to remove fork with uid ${uid} that is not a fork of this one`)
-    }
-    this.#forks.delete(uid)
-    this.#waitingForks.delete(uid)
-    this.#removedForks.add(uid)
-    try {
-      if (error === undefined) {
-        await fork.return()
-      } else {
-        await fork.throw(error)
-      }
-    } catch (cleaningError) {
-      console.error('Error while cleaning the forked', {
-        cleaningError,
-        sourceError: error,
-      })
-    }
-
-    if (this.#forks.size === 0) {
-      if (error === undefined) {
-        await this.#source.return(undefined as TReturn)
-      } else {
-        await this.#source.throw(error)
-      }
-      // Reject any pending forks waiting for the next value
-      if (this.#nextValueForksReady) {
-        this.#nextValueForksReady.forksWaitingReject(new Error('Source generator terminated.', { cause: error }))
-        this.#nextValueForksReady = undefined
-      }
-      // clear state
-      this.#removedForks.clear()
-      this.#waitingForks.clear()
-    } else {
-      // this fork was maybe blocking the others
-      if (this.#nextValueForksReady) {
-        await this.#resolveWhenAllForksReady()
-      }
-    }
-    return { done: true, value: undefined }
-  }
-}
-
-class Forked<T, TReturn, TNext> implements AsyncGenerator<T, TReturn, TNext> {
-  #parent: Synchronized<T, TReturn, TNext>
-  #uid: string
-  constructor(parent: Synchronized<T, TReturn, TNext>, uid: string) {
-    this.#parent = parent
-    this.#uid = uid
-  }
-  next(): Promise<IteratorResult<T>> {
-    return this.#parent.next(this.#uid)
-  }
-  async return(): Promise<IteratorResult<T>> {
-    return this.#parent.remove(this.#uid)
-  }
-  async throw(e: Error): Promise<IteratorResult<T>> {
-    return this.#parent.remove(this.#uid, e)
-  }
-
-  async *[Symbol.asyncIterator](): AsyncGenerator<T> {
-    while (true) {
-      const res = await this.next()
-      if (res.done) {
-        break
-      }
-
-      yield res.value
-    }
-  }
-}

@vates/generator-toolbox/src/synchronized.test.mts

@@ -1,150 +0,0 @@
-import assert from 'node:assert'
-import { suite, test } from 'node:test'
-import { Synchronized } from './synchronized.mjs'
-
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
-async function* makeRangeGenerator(end = Infinity, progress = { yielded: 0 }, onYielded = (val: unknown) => {}) {
-  for (let i = 0; i < end; i++) {
-    await new Promise(resolve => setTimeout(resolve, 10))
-    yield i
-    onYielded(i)
-    progress.yielded = i
-  }
-  return progress
-}
-
-async function consume(
-  iterable: AsyncGenerator,
-  delay = 2500,
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  onConsumed = (val: unknown, iterable: AsyncGenerator) => Promise.resolve(false)
-) {
-  for await (const val of iterable) {
-    await new Promise(resolve => setTimeout(resolve, delay))
-    if (await onConsumed(val, iterable)) {
-      iterable.return(undefined)
-    }
-  }
-}
-
-suite('success', () => {
-  test('if works with multiple consumer', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(3, progress)
-    const forker = new Synchronized(generator)
-    const first = forker.fork('first')
-    const second = forker.fork('second')
-    await Promise.all([consume(first, 50), consume(second, 500)])
-    assert.strictEqual(progress.yielded, 2)
-  })
-
-  test('if works with one consumer returning', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(10, progress)
-    const forker = new Synchronized(generator)
-    const first = forker.fork('first')
-    const second = forker.fork('second')
-    await new Promise(resolve => setTimeout(resolve, 500))
-    await Promise.all([
-      consume(first, 1),
-      consume(second, 1, async (val: unknown) => {
-        return Promise.resolve(val === 2)
-      }),
-    ])
-    assert.strictEqual(progress.yielded, 9)
-  })
-  test('if works with all consumers returning', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(10, progress)
-    const forker = new Synchronized(generator)
-    const first = forker.fork('first')
-    const second = forker.fork('second')
-    await new Promise(resolve => setTimeout(resolve, 500))
-    await Promise.all([
-      consume(first, 1, async (val: unknown) => {
-        return Promise.resolve(val === 5)
-      }),
-      consume(second, 1, async (val: unknown) => {
-        return Promise.resolve(val === 2)
-      }),
-    ])
-    assert.strictEqual(progress.yielded, 4)
-  })
-})
-
-suite('error', () => {
-  test('it fails all if error on source', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(5, progress, (val: unknown) => {
-      if (val === 2) {
-        throw new Error('source stop')
-      }
-    })
-    const forker = new Synchronized(generator)
-    const first = forker.fork('first')
-    const second = forker.fork('second')
-    await assert.rejects(Promise.all([consume(first, 1), consume(second, 1)]))
-
-    assert.strictEqual(progress.yielded, 1)
-  })
-  test('it flows until the end if one consumer throws', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(10, progress)
-    const forker = new Synchronized(generator)
-    const first = forker.fork('first')
-    const second = forker.fork('second')
-
-    const firstConsume = consume(first, 1)
-
-    const secondConsume = assert.rejects(() =>
-      consume(second, 1, async (val: unknown, iterable) => {
-        if (val === 1) {
-          await iterable.throw(new Error('error on second'))
-        }
-        return false
-      })
-    )
-    await Promise.allSettled([firstConsume, secondConsume])
-    assert.strictEqual(progress.yielded, 9)
-  })
-  test('it throw if ALL consumer throws', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(5, progress)
-    const forker = new Synchronized(generator)
-    const first = forker.fork('first')
-    const second = forker.fork('second')
-
-    const firstConsume = consume(first, 1, async (val: unknown, iterable) => {
-      if (val === 3) {
-        await iterable.throw(new Error('error on first'))
-      }
-      return false
-    })
-    const secondConsume = consume(second, 1, async (val: unknown, iterable) => {
-      if (val === 1) {
-        await iterable.throw(new Error('error on second'))
-      }
-      return false
-    })
-
-    await assert.rejects(Promise.all([firstConsume, secondConsume]))
-    assert.strictEqual(progress.yielded, 2)
-  })
-
-  test('It should not allow a new fork after the start', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(5, progress)
-    const forker = new Synchronized(generator)
-    const first = forker.fork('first')
-    await first.next()
-    assert.throws(() => forker.fork('second'))
-  })
-  test('It should not allow fork with the same uid', async () => {
-    const progress = { yielded: 0 }
-    const generator = makeRangeGenerator(5, progress)
-    const forker = new Synchronized(generator)
-    forker.fork('first')
-    forker.fork('second')
-    assert.throws(() => forker.fork('first'))
-  })
-})

@vates/generator-toolbox/src/throttle.mts

@@ -1,66 +0,0 @@
-import assert from 'node:assert'
-
-/**
- *
- * This class will throttle the production of a group of generators
- *
- * limits: depends on the event loop, so don't expect it to be very precise
- * works better with more small blocks
- * The source generator must yield object with a length property
- * Changing the speed will only be takin into account fot the next packets asked
- */
-export class Throttle {
-  #previousSlot = 0
-  #bytesPerSecond: number | (() => number)
-  get speed(): number {
-    let speed: number
-    if (typeof this.#bytesPerSecond === 'function') {
-      speed = this.#bytesPerSecond()
-    } else {
-      speed = this.#bytesPerSecond
-    }
-    assert.ok(speed > 0, `speed must be greater than zero, ${speed} computed`)
-    return speed
-  }
-  constructor(speed: number | (() => number)) {
-    this.#bytesPerSecond = speed
-  }
-
-  getNextSlot(length: number): { timeout?: ReturnType<typeof setTimeout>; promise?: Promise<unknown> } {
-    assert.notStrictEqual(length, undefined, `throttled stream need to expose a length property }`)
-    assert.ok(length > 0, `throttled stream must expose a positive length property , ${length} given }`)
-
-    const previous = this.#previousSlot
-    const nextSlot = Math.round(previous + (length * 1000) / this.speed)
-    if (nextSlot < Date.now()) {
-      // we're above the limit, go now
-      this.#previousSlot = Date.now()
-      return {}
-    }
-    this.#previousSlot = nextSlot
-    // wait till the next slot
-    // it won't be extremely precise since the event loop is not
-    let timeout: ReturnType<typeof setTimeout> | undefined = undefined
-    const promise = new Promise(function (resolve) {
-      timeout = setTimeout(resolve, nextSlot - Date.now())
-    })
-    return { promise, timeout }
-  }
-
-  async *createThrottledGenerator(source: AsyncGenerator<{ length: number }>): AsyncGenerator<{ length: number }> {
-    let timeout: ReturnType<typeof setTimeout> | undefined
-    try {
-      for await (const value of source) {
-        const res = this.getNextSlot(value.length)
-        timeout = res.timeout
-        // wait for the time slot before yielding the data
-        if (res.promise !== undefined) {
-          await res.promise
-        }
-        yield value
-      }
-    } finally {
-      clearTimeout(timeout)
-    }
-  }
-}

@vates/generator-toolbox/src/throttle.test.mts

@@ -1,63 +0,0 @@
-import { Throttle } from './throttle.mjs'
-import { suite, test } from 'node:test'
-import assert from 'node:assert'
-
-async function* makeGenerator(chunkSize = 1024 * 1024, nbChunks = 100) {
-  for (let i = 0; i < nbChunks; i++) {
-    yield Buffer.allocUnsafe(chunkSize)
-  }
-}
-
-async function consumes(generator: AsyncGenerator) {
-  // eslint-disable-next-line no-empty,@typescript-eslint/no-unused-vars
-  for await (const data of generator) {
-  }
-}
-
-suite('it throttle one', () => {
-  test('base test', async () => {
-    // 10MB/s
-    const throttler = new Throttle(10 * 1024 * 1024)
-    const throttled = throttler.createThrottledGenerator(makeGenerator(1024 * 1024, 20))
-
-    const start = Date.now()
-    await consumes(throttled)
-    const end = Date.now()
-    assert.strictEqual(Math.round((end - start) / 1000), 2)
-  })
-  test('variable speed test', async () => {
-    // 10MB/s
-    const throttler = new Throttle(() => 10 * 1024 * 1024)
-    const throttled = throttler.createThrottledGenerator(makeGenerator(1024 * 1024, 20))
-
-    const start = Date.now()
-    await consumes(throttled)
-    const end = Date.now()
-    assert.strictEqual(Math.round((end - start) / 1000), 2)
-  })
-  test('multiple generator test', async () => {
-    // 10MB/s
-    const throttler = new Throttle(10 * 1024 * 1024)
-    const first = throttler.createThrottledGenerator(makeGenerator(1024 * 1024, 10))
-    const second = throttler.createThrottledGenerator(makeGenerator(1024 * 1024, 10))
-    const start = Date.now()
-
-    await Promise.all([consumes(first), consumes(second)])
-    const end = Date.now()
-    assert.strictEqual(Math.round((end - start) / 1000), 2)
-  })
-  test(`it doesn't accept negative value`, async () => {
-    // 10MB/s
-    let throttler = new Throttle(-10 * 1024 * 1024)
-    let generator = throttler.createThrottledGenerator(makeGenerator(1024 * 1024, 10))
-    await assert.rejects(() => consumes(generator))
-
-    let index = 5
-    throttler = new Throttle(() => {
-      index--
-      return index > 0 ? 10 * 1024 * 1024 : -10 * 1024 * 1024
-    })
-    generator = throttler.createThrottledGenerator(makeGenerator(1024 * 1024, 10))
-    await assert.rejects(() => consumes(generator))
-  })
-})

@vates/generator-toolbox/src/timeout.mts

@@ -1,37 +0,0 @@
-import assert from 'node:assert'
-export class Timeout<T> implements AsyncGenerator {
-  #source: AsyncGenerator<T>
-  #timeout: number
-  constructor(source: AsyncGenerator<T>, timeout: number) {
-    assert.ok(timeout > 0, `Timeout value must be positive, ${timeout} received`)
-    this.#source = source
-    this.#timeout = timeout
-  }
-  async next(): Promise<IteratorResult<T>> {
-    let timeout: ReturnType<typeof setTimeout>
-    const promiseTimeout = new Promise((_, reject) => {
-      timeout = setTimeout(() => {
-        reject(new Error('Timeout reached '))
-      }, this.#timeout)
-    })
-    const promiseNext = new Promise((resolve, reject) => {
-      this.#source.next().then(res => {
-        // ensure timetout won't fire later
-        clearTimeout(timeout)
-        resolve(res)
-      }, reject)
-    })
-    // promiseTimeout will never resolve
-    return Promise.race([promiseNext, promiseTimeout]) as Promise<IteratorResult<T>>
-  }
-  return(): Promise<IteratorResult<T>> {
-    return this.#source.return(undefined)
-  }
-  throw(e: Error): Promise<IteratorResult<T>> {
-    return this.#source.throw(e)
-  }
-
-  [Symbol.asyncIterator](): AsyncGenerator<T> {
-    return this
-  }
-}

@vates/generator-toolbox/src/timeout.test.mts

@@ -1,67 +0,0 @@
-import { describe, it } from 'node:test'
-import assert from 'node:assert'
-import { Timeout } from './timeout.mjs'
-
-describe('Timeout class', () => {
-  it('should reject a timeout with a negative or zero value ', async () => {
-    // Create a mock async generator that never resolves
-    const mockGenerator = (async function* () {
-      yield 1
-    })()
-
-    assert.throws(() => new Timeout(mockGenerator, 0))
-    assert.throws(() => new Timeout(mockGenerator, -10))
-  })
-  it('should resolve with the value from the source generator if it completes before the timeout', async () => {
-    // Create a mock async generator that resolves immediately
-    const mockGenerator = (async function* () {
-      yield 1
-    })()
-
-    const timeout = new Timeout(mockGenerator, 100) // 100ms timeout
-
-    const result = await timeout.next()
-    assert.strictEqual(result.value, 1)
-    assert.strictEqual(result.done, false)
-  })
-
-  it('should reject with a timeout error if the source generator takes too long', async () => {
-    // Create a mock async generator that never resolves
-    // eslint-disable-next-line require-yield
-    const mockGenerator = (async function* () {
-      await new Promise(() => {}) // Never resolves
-    })()
-
-    const timeout = new Timeout(mockGenerator, 10) // 10ms timeout
-
-    await assert.rejects(timeout.next(), 'Expected timeout error was not thrown')
-  })
-  it('should allow iteration using for-await-of', async () => {
-    // Create a mock async generator that yields values
-    const mockGenerator: AsyncGenerator<number> = (async function* () {
-      yield 1
-      yield 2
-      yield 3
-    })()
-
-    const timeout = new Timeout(mockGenerator, 100) // 100ms timeout
-
-    const results: Array<number> = []
-    for await (const value of timeout) {
-      results.push(value as number)
-    }
-
-    assert.deepStrictEqual(results, [1, 2, 3])
-  })
-
-  it('should handle the throw method correctly', async () => {
-    // Create a mock async generator with a throw method
-    const mockGenerator = (async function* () {
-      yield 1
-    })()
-
-    const timeout = new Timeout(mockGenerator, 100) // 100ms timeout
-
-    await assert.rejects(timeout.throw(new Error('Test error')), 'Expected error was not thrown')
-  })
-})

@vates/generator-toolbox/tsconfig.json

@@ -1,12 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "es2023",
-    "module": "ESNext",
-    "moduleResolution": "bundler",
-    "strict": true,
-    "declaration": true, // Generates `.d.ts` files for type safety
-    "outDir": "dist" // Outputs compiled code to the `dist` directory
-  },
-  "include": ["src/**/*"],
-  "exclude": ["node_modules", "dist"]
-}

@vates/json-hash/.USAGE.md

@@ -1,4 +1,4 @@
-The use for this library is to create a composite key from a JSON value, ignoring objects identity and properties order.
+The use for this librariy is to create a composite key from a JSON value, ignoring objects identity and properties order.
 
 The hash algorithm used is intended to be fast and with low collisions and is not guaranteed to be be secure.
 

@vates/json-hash/README.md

@@ -16,7 +16,7 @@ npm install --save @vates/json-hash
 
 ## Usage
 
-The use for this library is to create a composite key from a JSON value, ignoring objects identity and properties order.
+The use for this librariy is to create a composite key from a JSON value, ignoring objects identity and properties order.
 
 The hash algorithm used is intended to be fast and with low collisions and is not guaranteed to be be secure.
 

@vates/nbd-client/NbdDisk.mjs

@@ -1,122 +0,0 @@
-import { RandomAccessDisk } from '@xen-orchestra/disk-transform'
-import NbdClient from '@vates/nbd-client'
-
-/**
- * @typedef {{ offset: number, length: number }} DataRange
- */
-
-/**
- * @extends {RandomAccessDisk}
- */
-export class NbdDisk extends RandomAccessDisk {
-  #nbdInfos
-  /** @type {NbdClient|undefined} */
-  #nbdClient
-
-  /** @type {Array<DataRange> | undefined} */
-  #dataMap
-
-  /** @type {number} */
-  #blockSize
-
-  constructor(nbdInfos, blockSize, { dataMap } = {}) {
-    super()
-    this.#blockSize = blockSize
-    this.#nbdInfos = nbdInfos
-    this.#dataMap = dataMap && this.#processDatamap(dataMap)
-  }
-
-  #processDatamap(rawDataMap) {
-    return rawDataMap
-      .filter(({ type }) => type === 0)
-      .map(({ offset, length }) => ({ offset, length }))
-      .sort(({ offset: offset1 }, { offset: offset2 }) => offset1 - offset2)
-  }
-  /**
-   * @param {number} index
-   * @returns {Promise<DiskBlock>}
-   */
-  async readBlock(index) {
-    if (!this.#nbdClient) {
-      throw new Error("can't readBlock before init")
-    }
-    const data = await this.#nbdClient.readBlock(index, this.getBlockSize())
-    return {
-      index,
-      data,
-    }
-  }
-
-  /**
-   * @returns {number}
-   */
-  getVirtualSize() {
-    if (!this.#nbdClient) {
-      throw new Error("can't get size before init")
-    }
-    return Number(this.#nbdClient.exportSize)
-  }
-
-  /**
-   * @returns {number}
-   */
-  getBlockSize() {
-    return this.#blockSize
-  }
-
-  /**
-   * @returns {Promise<void>}
-   */
-  async init() {
-    this.#nbdClient = new NbdClient(this.#nbdInfos)
-    await this.#nbdClient.connect()
-    if (this.#dataMap === undefined) {
-      this.#dataMap = this.#processDatamap(await this.#nbdClient.getMap())
-    }
-  }
-
-  /**
-   * @returns {Promise<void>}
-   */
-  async close() {
-    await this.#nbdClient?.disconnect()
-  }
-
-  /**
-   * @returns {boolean}
-   */
-  isDifferencing() {
-    return false
-  }
-
-  /**
-   * @returns {Array<number>}
-   */
-  getBlockIndexes() {
-    const indexes = new Set()
-    this.#dataMap?.forEach(({ offset, length }) => {
-      if (length === 0) return
-
-      const firstBlockIndex = Math.floor(offset / this.getBlockSize())
-      const lastBlockIndex = Math.floor((offset + length - 1) / this.getBlockSize())
-
-      for (let blockIndex = firstBlockIndex; blockIndex <= lastBlockIndex; blockIndex++) {
-        indexes.add(blockIndex)
-      }
-    })
-    return [...indexes].sort((a, b) => a - b)
-  }
-
-  /**
-   * @param {number} index
-   * @returns {boolean}
-   */
-  hasBlock(index) {
-    if (!this.#dataMap) {
-      throw new Error("can't hasBlock before init")
-    }
-    const blockStart = index * this.getBlockSize()
-    const blockEnd = (index + 1) * this.getBlockSize()
-    return this.#dataMap.some(({ offset, length }) => offset + length > blockStart && blockEnd >= offset)
-  }
-}

@vates/nbd-client/constants.mjs

@@ -1,6 +1,6 @@
 export const INIT_PASSWD = Buffer.from('NBDMAGIC') // "NBDMAGIC" ensure we're connected to a nbd server
 export const OPTS_MAGIC = Buffer.from('IHAVEOPT') // "IHAVEOPT" start an option block
-export const NBD_OPT_REPLY_MAGIC = 1100100111001001n // magic received during negotiation
+export const NBD_OPT_REPLY_MAGIC = 1100100111001001n // magic received during negociation
 export const NBD_OPT_EXPORT_NAME = 1
 export const NBD_OPT_ABORT = 2
 export const NBD_OPT_LIST = 3

@vates/nbd-client/index.mjs

@@ -1,7 +1,7 @@
 import assert from 'node:assert'
 import { Socket } from 'node:net'
 import { connect } from 'node:tls'
-import { pRetry, pDelay, pTimeout, pFromCallback } from 'promise-toolbox'
+import { fromCallback, pRetry, pDelay, pTimeout, pFromCallback } from 'promise-toolbox'
 import { readChunkStrict } from '@vates/read-chunk'
 import { createLogger } from '@xen-orchestra/log'
 import {
@@ -20,8 +20,6 @@ import {
   OPTS_MAGIC,
   NBD_CMD_DISC,
 } from './constants.mjs'
-import { spawn } from 'node:child_process'
-
 const { warn } = createLogger('vates:nbd-client')
 
 // documentation is here : https://github.com/NetworkBlockDevice/nbd/blob/master/doc/proto.md
@@ -44,9 +42,9 @@ export default class NbdClient {
   // AFAIK, there is no guaranty the server answers in the same order as the queries
   // so we handle a backlog of command waiting for response and handle concurrency manually
 
-  #waitingForResponse // there is already a listener waiting for a response
+  #waitingForResponse // there is already a listenner waiting for a response
   #nextCommandQueryId = BigInt(0)
-  #commandQueryBacklog // map of command waiting for a response queryId => { size/*in byte*/, resolve, reject}
+  #commandQueryBacklog // map of command waiting for an response queryId => { size/*in byte*/, resolve, reject}
   #connected = false
 
   #reconnectingPromise
@@ -197,7 +195,7 @@ export default class NbdClient {
     }
 
     // send export name we want to access.
-    // it's implicitly closing the negotiation phase.
+    // it's  implictly closing the negociation phase.
     await this.#write(OPTS_MAGIC)
     await this.#writeInt32(NBD_OPT_EXPORT_NAME)
     const exportNameBuffer = Buffer.from(this.#exportName)
@@ -220,22 +218,8 @@ export default class NbdClient {
   }
 
   #write(buffer) {
-    let timeout
-    const messageTimeout = this.#messageTimeout
-    const socket = this.#serverSocket
-    return Promise.race([
-      new Promise((resolve, reject) => {
-        timeout = setTimeout(() => {
-          reject(new Error('timeout'))
-        }, messageTimeout)
-      }),
-      new Promise(resolve =>
-        socket.write(buffer, () => {
-          clearTimeout(timeout)
-          resolve()
-        })
-      ),
-    ])
+    const promise = fromCallback.call(this.#serverSocket, 'write', buffer)
+    return pTimeout.call(promise, this.#messageTimeout)
   }
 
   async #readInt32() {
@@ -359,40 +343,4 @@ export default class NbdClient {
       },
     })
   }
-
-  /**
-   *  returns the map of the file with holes, zeros and data, useful to handle efficiently sparse source
-   * to implement this internally: use structure response if the server supports it, and then ask for BLOCK_STATUS   *
-   *
-   * @returns {Promise<{ offset: number, length: number, type: number }[]>}
-   * A promise that resolves to an array where each object represents a segment:
-   * - `offset` — The byte offset from the start.
-   * - `length` — The size of the segment in bytes.
-   * - `type` — A numeric code indicating the segment type ( 0 means no data).
-   */
-  /* async */ getMap() {
-    return new Promise((resolve, reject) => {
-      const process = spawn('nbdinfo', [
-        '--json',
-        '--map',
-        `nbd://${this.#serverAddress}:${this.#serverPort}/${encodeURIComponent(this.#exportName)}`,
-      ])
-      let text = ''
-      let errText = ''
-      process.stdout.on('data', data => (text += data))
-      process.stderr.on('data', data => (errText += data))
-      process.on('error', reject)
-      process.on('close', code => {
-        if (code !== 0) {
-          return reject(new Error(`Error during getMap (code: ${code}): ${errText}`))
-        }
-        try {
-          const json = JSON.parse(text)
-          resolve(json)
-        } catch (error) {
-          reject(new Error(`${error} ${text}`))
-        }
-      })
-    })
-  }
 }

@vates/nbd-client/multi.mjs

@@ -30,8 +30,6 @@ export default class MultiNbdClient {
    * open nbdConcurrency connections to NBD servers
    * it must obtain at least one connection to succeed
    * it tries to spread connections on multiple host
-   *
-   * @returns {Promise<void>}
    */
   async connect() {
     const candidates = [...this.#settings]
@@ -70,9 +68,7 @@ export default class MultiNbdClient {
       await _connect()
     }
     if (this.#clients.length === 0) {
-      const error = new Error(`Fail to connect to any Nbd client`, { nbdInfos: this.#settings })
-      error.code = 'NO_NBD_AVAILABLE'
-      throw error
+      throw new Error(`Fail to connect to any Nbd client`)
     }
     if (this.#clients.length < this.#nbdConcurrency) {
       warn(
@@ -81,30 +77,17 @@ export default class MultiNbdClient {
     }
   }
 
-  /**
-   * @returns {Promise<void>}
-   */
   async disconnect() {
     await asyncEach(this.#clients, client => client.disconnect(), {
       stopOnError: false,
     })
   }
 
-  /**
-   *
-   * @param {number} index
-   * @param {number} size
-   * @returns {Promise<Buffer>}
-   */
   async readBlock(index, size = NBD_DEFAULT_BLOCK_SIZE) {
     const clientId = index % this.#clients.length
     return this.#clients[clientId].readBlock(index, size)
   }
 
-  /**
-   *
-   * @param {AsyncGenerator<Buffer>} indexGenerator
-   */
   async *readBlocks(indexGenerator) {
     // default : read all blocks
     const readAhead = []
@@ -129,17 +112,4 @@ export default class MultiNbdClient {
       yield readAhead.shift()
     }
   }
-  /**
-   *  returns the map of the file with holes, zeros and data, useful to handle efficiently sparse source   *
-   *
-   * @returns {Promise<{ offset: number, length: number, type: number }[]>}
-   * A promise that resolves to an array where each object represents a segment:
-   * - `offset` — The byte offset from the start.
-   * - `length` — The size of the segment in bytes.
-   * - `type` — A numeric code indicating the segment type (0 means no data).
-   */
-  async getMap() {
-    // ask the map from one of the connected client
-    return this.#clients[Math.floor(this.#clients.length * Math.random())].getMap()
-  }
 }

@vates/nbd-client/package.json

@@ -13,19 +13,18 @@
     "url": "https://vates.fr"
   },
   "license": "ISC",
-  "version": "3.2.1",
+  "version": "3.1.2",
   "engines": {
     "node": ">=14.0"
   },
   "main": "./index.mjs",
   "dependencies": {
     "@vates/async-each": "^1.0.0",
-    "@vates/read-chunk": "^1.2.1",
+    "@vates/read-chunk": "^1.2.0",
     "@xen-orchestra/async-map": "^0.1.2",
-    "@xen-orchestra/disk-transform": "^1.0.1",
     "@xen-orchestra/log": "^0.7.1",
     "promise-toolbox": "^0.21.0",
-    "xen-api": "^4.7.4"
+    "xen-api": "^4.7.0"
   },
   "devDependencies": {
     "tap": "^18.7.0",

@vates/node-vsphere-soap/README.md

@@ -24,7 +24,7 @@ $ npm install node-vsphere-soap --save
 
 ## Sample Code
 
-### To connect to a vCenter server
+### To connect to a vCenter server:
 
     var nvs = require('node-vsphere-soap');
     var vc = new nvs.Client(host, user, password, sslVerify);
@@ -54,7 +54,7 @@ $ npm install node-vsphere-soap --save
 - userName - username of authenticated user
 - fullName - full name of authenticated user
 
-### To run a command
+### To run a command:
 
     var vcCmd = vc.runCommand( commandToRun, arguments );
     vcCmd.once('result', function( result, raw, soapHeader) {

@vates/node-vsphere-soap/lib/client.mjs

@@ -7,7 +7,7 @@
   This file creates the Client class
 
   - when the class is instantiated, a connection will be made to the ESXi/vCenter server to verify that the creds are good
-  - upon a bad login, the connection will be terminated
+  - upon a bad login, the connnection will be terminated
 
 */
 

@vates/node-vsphere-soap/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vates/node-vsphere-soap",
-  "version": "2.1.2",
+  "version": "2.1.1",
   "description": "interface to vSphere SOAP/WSDL from node for interfacing with vCenter or ESXi, forked from node-vsphere-soap",
   "main": "lib/client.mjs",
   "author": "reedog117",
@@ -12,7 +12,7 @@
   "bugs": "https://github.com/vatesfr/xen-orchestra/issues",
   "dependencies": {
     "@xen-orchestra/log": "^0.7.1",
-    "axios": "^1.8.0",
+    "axios": "^1.4.0",
     "soap": "^1.0.0",
     "soap-cookie": "^0.10.1"
   },

@vates/otp/.USAGE.md

@@ -1,6 +1,6 @@
 ### Usual workflow
 
-> This section presents how this library should be used to implement a classic two factor authentication.
+> This section presents how this library should be used to implement a classic two factor authentification.
 
 #### Setup
 

@vates/otp/README.md

@@ -18,7 +18,7 @@ npm install --save @vates/otp
 
 ### Usual workflow
 
-> This section presents how this library should be used to implement a classic two factor authentication.
+> This section presents how this library should be used to implement a classic two factor authentification.
 
 #### Setup
 

@vates/read-chunk/index.js

@@ -71,17 +71,13 @@ exports.readChunk = readChunk
  * If `size` bytes are not available to be read, the returned promise is rejected.
  *
  * @param {Readable} stream - A readable stream to read from.
- * @param {number?} [size] - The number of bytes to read for binary streams (ignored for object streams).
+ * @param {number} [size] - The number of bytes to read for binary streams (ignored for object streams).
  * @returns {Promise<Buffer|string|unknown>} - A Promise that resolves to the read chunk.
  */
 exports.readChunkStrict = async function readChunkStrict(stream, size) {
   const chunk = await readChunk(stream, size)
   if (chunk === null) {
-    if (size === undefined) {
-      throw new Error(`stream has ended without data`)
-    } else {
-      throw new Error(`stream has ended without data, was looking for ${size} bytes`)
-    }
+    throw new Error('stream has ended without data')
   }
 
   if (size !== undefined && chunk.length !== size) {

@vates/read-chunk/package.json

@@ -19,7 +19,7 @@
     "type": "git",
     "url": "https://github.com/vatesfr/xen-orchestra.git"
   },
-  "version": "1.2.1",
+  "version": "1.2.0",
   "engines": {
     "node": ">=8.10"
   },

@vates/task/.USAGE.md

@@ -63,16 +63,16 @@ task.failure(error)
 Inside a task:
 
 ```js
-// the abort signal of the current task if any; otherwise, is `undefined`
+// the abort signal of the current task if any, otherwise is `undefined`
 Task.abortSignal
 
-// sends an info on the current task if any; otherwise, does nothing
+// sends an info on the current task if any, otherwise does nothing
 Task.info(message, data)
 
-// sends an info on the current task if any; otherwise, does nothing
+// sends an info on the current task if any, otherwise does nothing
 Task.warning(message, data)
 
-// attaches a property to the current task if any; otherwise, does nothing
+// attaches a property to the current task if any, otherwise does nothing
 //
 // the latest value takes precedence
 //
@@ -149,7 +149,7 @@ const onProgress = makeOnProgress({
 
   // This function is called each time a root task or a subtask is updated.
   //
-  // `taskLog.$root` can be used to unconditionally access the root task.
+  // `taskLog.$root` can be used to uncondionally access the root task.
   onTaskUpdate(taskLog) {},
 })
 

@vates/task/README.md

@@ -79,16 +79,16 @@ task.failure(error)
 Inside a task:
 
 ```js
-// the abort signal of the current task if any; otherwise, is `undefined`
+// the abort signal of the current task if any, otherwise is `undefined`
 Task.abortSignal
 
-// sends an info on the current task if any; otherwise, does nothing
+// sends an info on the current task if any, otherwise does nothing
 Task.info(message, data)
 
-// sends an info on the current task if any; otherwise, does nothing
+// sends an info on the current task if any, otherwise does nothing
 Task.warning(message, data)
 
-// attaches a property to the current task if any; otherwise, does nothing
+// attaches a property to the current task if any, otherwise does nothing
 //
 // the latest value takes precedence
 //
@@ -165,7 +165,7 @@ const onProgress = makeOnProgress({
 
   // This function is called each time a root task or a subtask is updated.
   //
-  // `taskLog.$root` can be used to unconditionally access the root task.
+  // `taskLog.$root` can be used to uncondionally access the root task.
   onTaskUpdate(taskLog) {},
 })
 

@vates/task/index.js

@@ -29,7 +29,7 @@ class Task {
   static get abortSignal() {
     const task = getTask()
     if (task !== undefined) {
-      return task._abortController.signal
+      return task.#abortController.signal
     }
   }
 
@@ -65,11 +65,8 @@ class Task {
     }
   }
 
-  // These two properties should not be used outside the class
-  //
-  // To ensure compatibility between multiple versions of this lib, they are kept public for now
-  _abortController = new AbortController()
-  _onProgress
+  #abortController = new AbortController()
+  #onProgress
 
   get id() {
     return (this.id = Math.random().toString(36).slice(2))
@@ -89,23 +86,23 @@ class Task {
     this.#startData = { properties }
 
     if (onProgress !== undefined) {
-      this._onProgress = onProgress
+      this.#onProgress = onProgress
     } else {
       const parent = getTask()
       if (parent !== undefined) {
-        const { signal } = parent._abortController
+        const { signal } = parent.#abortController
         signal.addEventListener('abort', () => {
-          this._abortController.abort(signal.reason)
+          this.#abortController.abort(signal.reason)
         })
 
-        this._onProgress = parent._onProgress
+        this.#onProgress = parent.#onProgress
         this.#startData.parentId = parent.id
       } else {
-        this._onProgress = noop
+        this.#onProgress = noop
       }
     }
 
-    const { signal } = this._abortController
+    const { signal } = this.#abortController
     signal.addEventListener('abort', () => {
       if (this.status === PENDING) {
         this.#maybeStart()
@@ -120,7 +117,7 @@ class Task {
   }
 
   abort(reason) {
-    this._abortController.abort(reason)
+    this.#abortController.abort(reason)
   }
 
   #emit(type, data) {
@@ -129,7 +126,7 @@ class Task {
     data.id = this.id
     data.timestamp = Date.now()
     data.type = type
-    this._onProgress(data)
+    this.#onProgress(data)
   }
 
   #end(status, result) {
@@ -137,7 +134,7 @@ class Task {
     assert.equal(this.#running, false)
 
     this.#emit('end', { status, result })
-    this._onProgress = alreadyEnded
+    this.#onProgress = alreadyEnded
     this.#status = status
   }
 

@vates/task/index.test.js

@@ -29,7 +29,7 @@ function createTask(opts) {
 }
 
 describe('Task', function () {
-  describe('constructor', function () {
+  describe('contructor', function () {
     it('data properties are passed to the start event', async function () {
       const properties = { foo: 0, bar: 1 }
       const task = createTask({ properties })
@@ -265,7 +265,7 @@ describe('Task', function () {
       assert.throws(() => task.set(name, value), { message: /^task has not started yet\b/ })
     })
 
-    it(`emits a property message`, async function () {
+    it(`emits an property message`, async function () {
       const task = createTask()
       await task.run(async () => {
         await Task.run(() => {
@@ -339,7 +339,7 @@ describe('Task', function () {
       Task.warning('foo')
     })
 
-    it('emits a warning message when run inside a task', async function () {
+    it('emits an warning message when run inside a task', async function () {
       const task = createTask()
       await task.run(() => {
         Task.warning('foo')

@vates/task/package.json

@@ -13,7 +13,7 @@
     "url": "https://vates.fr"
   },
   "license": "ISC",
-  "version": "0.6.2",
+  "version": "0.6.0",
   "engines": {
     "node": ">=14"
   },

@vates/types/.npmignore

@@ -1 +0,0 @@
-../../scripts/npmignore

@vates/types/eslint.config.mjs

@@ -1,21 +0,0 @@
-// @ts-check
-
-import eslint from '@eslint/js'
-import tseslint from 'typescript-eslint'
-import { fileURLToPath } from 'node:url'
-import path from 'node:path'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-
-export default tseslint.config(eslint.configs.recommended, tseslint.configs.recommended, {
-  languageOptions: {
-    parserOptions: {
-      tsconfigRootDir: __dirname,
-    },
-  },
-  ignores: ['dist'],
-  rules: {
-    // Allow empty interface for recursive generic typing (E.g. interface XenApiNetworkWrapped)
-    '@typescript-eslint/no-empty-object-type': 'off',
-  },
-})

@vates/types/package.json

@@ -1,56 +0,0 @@
-{
-  "name": "@vates/types",
-  "private": false,
-  "version": "1.12.0",
-  "main": "./dist/index.mjs",
-  "exports": {
-    ".": {
-      "default": "./dist/index.mjs",
-      "types": "./dist/index.d.mts"
-    },
-    "./xo": "./dist/xo.d.mts",
-    "./xen-api": "./dist/xen-api.d.mts",
-    "./common": {
-      "default": "./dist/common.mjs",
-      "types": "./dist/common.d.mts"
-    },
-    "./lib/vates/*": {
-      "default": "./dist/lib/vates-*.mjs",
-      "type": "./dist/lib/vates-*.d.mts"
-    },
-    "./lib/xen-orchestra/*": {
-      "default": "./dist/lib/xen-orchestra-*.mjs",
-      "type": "./dist/lib/xen-orchestra-*.d.mts"
-    },
-    "./lib/complex-matcher": {
-      "default": "./dist/lib/complex-matcher.mjs",
-      "type": "./dist/lib/complex-matcher.d.mts"
-    }
-  },
-  "type": "module",
-  "devDependencies": {
-    "@eslint/js": "^9.19.0",
-    "typescript": "~5.6.3",
-    "typescript-eslint": "^8.23.0"
-  },
-  "scripts": {
-    "build": "tsc",
-    "prepublishOnly": "yarn run build",
-    "postversion": "npm publish --access public"
-  },
-  "license": "AGPL-3.0-or-later",
-  "homepage": "https://github.com/vatesfr/xen-orchestra/tree/master/@vates/types",
-  "bugs": "https://github.com/vatesfr/xen-orchestra/issues",
-  "repository": {
-    "directory": "@vates/types",
-    "type": "git",
-    "url": "https://github.com/vatesfr/xen-orchestra.git"
-  },
-  "author": {
-    "name": "Vates SAS",
-    "url": "https://vates.fr"
-  },
-  "engines": {
-    "node": ">=18.18"
-  }
-}

@vates/types/src/common.mts

@@ -1,742 +0,0 @@
-import type { XoHost } from './xo.mjs'
-
-declare const __brand: unique symbol
-
-export type Branded<TBrand extends string, TType = string> = TType & { [__brand]: TBrand }
-
-export const HOST_POWER_STATE = {
-  RUNNING: 'Running',
-  HALTED: 'Halted',
-  UNKNOWN: 'Unknown',
-} as const
-
-export type HOST_POWER_STATE = (typeof HOST_POWER_STATE)[keyof typeof HOST_POWER_STATE]
-
-export const TASK_ALLOWED_OPERATIONS = {
-  CANCEL: 'cancel',
-  DESTROY: 'destroy',
-} as const
-
-export type TASK_ALLOWED_OPERATIONS = (typeof TASK_ALLOWED_OPERATIONS)[keyof typeof TASK_ALLOWED_OPERATIONS]
-
-export const TASK_STATUS_TYPE = {
-  CANCELLED: 'cancelled',
-  CANCELLING: 'cancelling',
-  FAILURE: 'failure',
-  PENDING: 'pending',
-  SUCCESS: 'success',
-} as const
-
-export type TASK_STATUS_TYPE = (typeof TASK_STATUS_TYPE)[keyof typeof TASK_STATUS_TYPE]
-
-export const EVENT_OPERATION = {
-  ADD: 'add',
-  DEL: 'del',
-  MOD: 'mod',
-} as const
-
-export type EVENT_OPERATION = (typeof EVENT_OPERATION)[keyof typeof EVENT_OPERATION]
-
-export const POOL_ALLOWED_OPERATIONS = {
-  APPLY_UPDATES: 'apply_updates',
-  CERT_REFRESH: 'cert_refresh',
-  CLUSTER_CREATE: 'cluster_create',
-  CONFIGURE_REPOSITORIES: 'configure_repositories',
-  COPY_PRIMARY_HOST_CERTS: 'copy_primary_host_certs',
-  DESIGNATE_NEW_MASTER: 'designate_new_master',
-  EJECT: 'eject',
-  EXCHANGE_CA_CERTIFICATES_ON_JOIN: 'exchange_ca_certificates_on_join',
-  EXCHANGE_CERTIFICATES_ON_JOIN: 'exchange_certificates_on_join',
-  GET_UPDATES: 'get_updates',
-  HA_DISABLE: 'ha_disable',
-  HA_ENABLE: 'ha_enable',
-  SYNC_UPDATES: 'sync_updates',
-  TLS_VERIFICATION_ENABLE: 'tls_verification_enable',
-} as const
-
-export type POOL_ALLOWED_OPERATIONS = (typeof POOL_ALLOWED_OPERATIONS)[keyof typeof POOL_ALLOWED_OPERATIONS]
-
-export const TELEMETRY_FREQUENCY = {
-  DAILY: 'daily',
-  MONTHLY: 'monthly',
-  WEEKLY: 'weekly',
-} as const
-
-export type TELEMETRY_FREQUENCY = (typeof TELEMETRY_FREQUENCY)[keyof typeof TELEMETRY_FREQUENCY]
-
-export const UPDATE_SYNC_FREQUENCY = {
-  DAILY: 'daily',
-  WEEKLY: 'weekly',
-} as const
-
-export type UPDATE_SYNC_FREQUENCY = (typeof UPDATE_SYNC_FREQUENCY)[keyof typeof UPDATE_SYNC_FREQUENCY]
-
-export const AFTER_APPLY_GUIDANCE = {
-  RESTART_HOST: 'restartHost',
-  RESTART_HVM: 'restartHVM',
-  RESTART_PV: 'restartPV',
-  RESTART_XAPI: 'restartXAPI',
-} as const
-
-export type AFTER_APPLY_GUIDANCE = (typeof AFTER_APPLY_GUIDANCE)[keyof typeof AFTER_APPLY_GUIDANCE]
-
-export const UPDATE_AFTER_APPLY_GUIDANCE = {
-  RESTART_HOST: 'restartHost',
-  RESTART_HVM: 'restartHVM',
-  RESTART_PV: 'restartPV',
-  RESTART_XAPI: 'restartXAPI',
-} as const
-
-export type UPDATE_AFTER_APPLY_GUIDANCE = (typeof UPDATE_AFTER_APPLY_GUIDANCE)[keyof typeof UPDATE_AFTER_APPLY_GUIDANCE]
-
-export const LIVEPATCH_STATUS = {
-  OK: 'ok',
-  OK_LIVEPATCH_COMPLETE: 'ok_livepatch_complete',
-  OK_LIVEPATCH_INCOMPLETE: 'ok_livepatch_incomplete',
-} as const
-
-export type LIVEPATCH_STATUS = (typeof LIVEPATCH_STATUS)[keyof typeof LIVEPATCH_STATUS]
-
-export const VM_POWER_STATE = {
-  HALTED: 'Halted',
-  PAUSED: 'Paused',
-  RUNNING: 'Running',
-  SUSPENDED: 'Suspended',
-} as const
-
-export type VM_POWER_STATE = (typeof VM_POWER_STATE)[keyof typeof VM_POWER_STATE]
-
-export const UPDATE_GUIDANCES = {
-  REBOOT_HOST: 'reboot_host',
-  REBOOT_HOST_ON_KERNEL_LIVEPATCH_FAILURE: 'reboot_host_on_kernel_livepatch_failure',
-  REBOOT_HOST_ON_LIVEPATCH_FAILURE: 'reboot_host_on_livepatch_failure',
-  REBOOT_HOST_ON_XEN_LIVEPATCH_FAILURE: 'reboot_host_on_xen_livepatch_failure',
-  RESTART_DEVICE_MODEL: 'restart_device_model',
-  RESTART_TOOLSTACK: 'restart_toolstack',
-  RESTART_VM: 'restart_vm',
-} as const
-
-export type UPDATE_GUIDANCES = (typeof UPDATE_GUIDANCES)[keyof typeof UPDATE_GUIDANCES]
-
-export const ON_SOFTREBOOT_BEHAVIOR = {
-  DESTROY: 'destroy',
-  PRESERVE: 'preserve',
-  RESTART: 'restart',
-  SOFT_REBOOT: 'soft_reboot',
-} as const
-
-export type ON_SOFTREBOOT_BEHAVIOR = (typeof ON_SOFTREBOOT_BEHAVIOR)[keyof typeof ON_SOFTREBOOT_BEHAVIOR]
-
-export const ON_NORMAL_EXIT = {
-  DESTROY: 'destroy',
-  RESTART: 'restart',
-} as const
-
-export type ON_NORMAL_EXIT = (typeof ON_NORMAL_EXIT)[keyof typeof ON_NORMAL_EXIT]
-
-export const VM_OPERATIONS = {
-  ASSERT_OPERATION_VALID: 'assert_operation_valid',
-  AWAITING_MEMORY_LIVE: 'awaiting_memory_live',
-  CALL_PLUGIN: 'call_plugin',
-  CHANGING_DYNAMIC_RANGE: 'changing_dynamic_range',
-  CHANGING_MEMORY_LIMITS: 'changing_memory_limits',
-  CHANGING_MEMORY_LIVE: 'changing_memory_live',
-  CHANGING_NVRAM: 'changing_NVRAM',
-  CHANGING_SHADOW_MEMORY: 'changing_shadow_memory',
-  CHANGING_SHADOW_MEMORY_LIVE: 'changing_shadow_memory_live',
-  CHANGING_STATIC_RANGE: 'changing_static_range',
-  CHANGING_VCPUS: 'changing_VCPUs',
-  CHANGING_VCPUS_LIVE: 'changing_VCPUs_live',
-  CHECKPOINT: 'checkpoint',
-  CLEAN_REBOOT: 'clean_reboot',
-  CLEAN_SHUTDOWN: 'clean_shutdown',
-  CLONE: 'clone',
-  COPY: 'copy',
-  CREATE_TEMPLATE: 'create_template',
-  CREATE_VTPM: 'create_vtpm',
-  CSVM: 'csvm',
-  DATA_SOURCE_OP: 'data_source_op',
-  DESTROY: 'destroy',
-  EXPORT: 'export',
-  GET_BOOT_RECORD: 'get_boot_record',
-  HARD_REBOOT: 'hard_reboot',
-  HARD_SHUTDOWN: 'hard_shutdown',
-  IMPORT: 'import',
-  MAKE_INTO_TEMPLATE: 'make_into_template',
-  METADATA_EXPORT: 'metadata_export',
-  MIGRATE_SEND: 'migrate_send',
-  PAUSE: 'pause',
-  POOL_MIGRATE: 'pool_migrate',
-  POWER_STATE_RESET: 'power_state_reset',
-  PROVISION: 'provision',
-  QUERY_SERVICES: 'query_services',
-  RESUME: 'resume',
-  RESUME_ON: 'resume_on',
-  REVERT: 'revert',
-  REVERTING: 'reverting',
-  SEND_SYSRQ: 'send_sysrq',
-  SEND_TRIGGER: 'send_trigger',
-  SHUTDOWN: 'shutdown',
-  SNAPSHOT: 'snapshot',
-  SNAPSHOT_WITH_QUIESCE: 'snapshot_with_quiesce',
-  START: 'start',
-  START_ON: 'start_on',
-  SUSPEND: 'suspend',
-  UNPAUSE: 'unpause',
-  UPDATE_ALLOWED_OPERATIONS: 'update_allowed_operations',
-} as const
-
-export type VM_OPERATIONS = (typeof VM_OPERATIONS)[keyof typeof VM_OPERATIONS]
-
-export const ON_CRASH_BEHAVIOUR = {
-  COREDUMP_AND_DESTROY: 'coredump_and_destroy',
-  COREDUMP_AND_RESTART: 'coredump_and_restart',
-  DESTROY: 'destroy',
-  PRESERVE: 'preserve',
-  RENAME_RESTART: 'rename_restart',
-  RESTART: 'restart',
-} as const
-
-export type ON_CRASH_BEHAVIOUR = (typeof ON_CRASH_BEHAVIOUR)[keyof typeof ON_CRASH_BEHAVIOUR]
-
-export const DOMAIN_TYPE = {
-  HVM: 'hvm',
-  PV: 'pv',
-  PVH: 'pvh',
-  PV_IN_PVH: 'pv_in_pvh',
-  UNSPECIFIED: 'unspecified',
-} as const
-
-export type DOMAIN_TYPE = (typeof DOMAIN_TYPE)[keyof typeof DOMAIN_TYPE]
-
-export const TRISTATE_TYPE = {
-  NO: 'no',
-  UNSPECIFIED: 'unspecified',
-  YES: 'yes',
-} as const
-
-export type TRISTATE_TYPE = (typeof TRISTATE_TYPE)[keyof typeof TRISTATE_TYPE]
-
-export const VMPP_BACKUP_TYPE = {
-  CHECKPOINT: 'checkpoint',
-  SNAPSHOT: 'snapshot',
-} as const
-
-export type VMPP_BACKUP_TYPE = (typeof VMPP_BACKUP_TYPE)[keyof typeof VMPP_BACKUP_TYPE]
-
-export const VMPP_BACKUP_FREQUENCY = {
-  DAILY: 'daily',
-  HOURLY: 'hourly',
-  WEEKLY: 'weekly',
-} as const
-
-export type VMPP_BACKUP_FREQUENCY = (typeof VMPP_BACKUP_FREQUENCY)[keyof typeof VMPP_BACKUP_FREQUENCY]
-
-export const VMPP_ARCHIVE_FREQUENCY = {
-  ALWAYS_AFTER_BACKUP: 'always_after_backup',
-  DAILY: 'daily',
-  NEVER: 'never',
-  WEEKLY: 'weekly',
-} as const
-
-export type VMPP_ARCHIVE_FREQUENCY = (typeof VMPP_ARCHIVE_FREQUENCY)[keyof typeof VMPP_ARCHIVE_FREQUENCY]
-
-export const VMPP_ARCHIVE_TARGET_TYPE = {
-  CIFS: 'cifs',
-  NFS: 'nfs',
-  NONE: 'none',
-} as const
-
-export type VMPP_ARCHIVE_TARGET_TYPE = (typeof VMPP_ARCHIVE_TARGET_TYPE)[keyof typeof VMPP_ARCHIVE_TARGET_TYPE]
-
-export const VMSS_FREQUENCY = {
-  DAILY: 'daily',
-  HOURLY: 'hourly',
-  WEEKLY: 'weekly',
-} as const
-
-export type VMSS_FREQUENCY = (typeof VMSS_FREQUENCY)[keyof typeof VMSS_FREQUENCY]
-
-export const VMSS_TYPE = {
-  CHECKPOINT: 'checkpoint',
-  SNAPSHOT: 'snapshot',
-  SNAPSHOT_WITH_QUIESCE: 'snapshot_with_quiesce',
-} as const
-
-export type VMSS_TYPE = (typeof VMSS_TYPE)[keyof typeof VMSS_TYPE]
-
-export const VM_APPLIANCE_OPERATION = {
-  CLEAN_SHUTDOWN: 'clean_shutdown',
-  HARD_SHUTDOWN: 'hard_shutdown',
-  SHUTDOWN: 'shutdown',
-  START: 'start',
-} as const
-
-export type VM_APPLIANCE_OPERATION = (typeof VM_APPLIANCE_OPERATION)[keyof typeof VM_APPLIANCE_OPERATION]
-
-export const HOST_ALLOWED_OPERATIONS = {
-  APPLY_UPDATES: 'apply_updates',
-  ENABLE: 'enable',
-  EVACUATE: 'evacuate',
-  POWER_ON: 'power_on',
-  PROVISION: 'provision',
-  REBOOT: 'reboot',
-  SHUTDOWN: 'shutdown',
-  VM_MIGRATE: 'vm_migrate',
-  VM_RESUME: 'vm_resume',
-  VM_START: 'vm_start',
-} as const
-
-export type HOST_ALLOWED_OPERATIONS = (typeof HOST_ALLOWED_OPERATIONS)[keyof typeof HOST_ALLOWED_OPERATIONS]
-
-export const LATEST_SYNCED_UPDATES_APPLIED_STATE = {
-  NO: 'no',
-  UNKNOWN: 'unknown',
-  YES: 'yes',
-} as const
-
-export type LATEST_SYNCED_UPDATES_APPLIED_STATE =
-  (typeof LATEST_SYNCED_UPDATES_APPLIED_STATE)[keyof typeof LATEST_SYNCED_UPDATES_APPLIED_STATE]
-
-export const HOST_DISPLAY = {
-  DISABLED: 'disabled',
-  DISABLE_ON_REBOOT: 'disable_on_reboot',
-  ENABLED: 'enabled',
-  ENABLE_ON_REBOOT: 'enable_on_reboot',
-} as const
-
-export type HOST_DISPLAY = (typeof HOST_DISPLAY)[keyof typeof HOST_DISPLAY]
-
-export const HOST_SCHED_GRAN = {
-  CORE: 'core',
-  CPU: 'cpu',
-  SOCKET: 'socket',
-} as const
-
-export type HOST_SCHED_GRAN = (typeof HOST_SCHED_GRAN)[keyof typeof HOST_SCHED_GRAN]
-
-export const HOST_NUMA_AFFINITY_POLICY = {
-  ANY: 'any',
-  BEST_EFFORT: 'best_effort',
-  DEFAULT_POLICY: 'default_policy',
-} as const
-
-export type HOST_NUMA_AFFINITY_POLICY = (typeof HOST_NUMA_AFFINITY_POLICY)[keyof typeof HOST_NUMA_AFFINITY_POLICY]
-
-export const NETWORK_OPERATIONS = {
-  ATTACHING: 'attaching',
-} as const
-
-export type NETWORK_OPERATIONS = (typeof NETWORK_OPERATIONS)[keyof typeof NETWORK_OPERATIONS]
-
-export const NETWORK_DEFAULT_LOCKING_MODE = {
-  DISABLED: 'disabled',
-  UNLOCKED: 'unlocked',
-} as const
-
-export type NETWORK_DEFAULT_LOCKING_MODE =
-  (typeof NETWORK_DEFAULT_LOCKING_MODE)[keyof typeof NETWORK_DEFAULT_LOCKING_MODE]
-
-export const NETWORK_PURPOSE = {
-  INSECURE_NBD: 'insecure_nbd',
-  NBD: 'nbd',
-} as const
-
-export type NETWORK_PURPOSE = (typeof NETWORK_PURPOSE)[keyof typeof NETWORK_PURPOSE]
-
-export const VIF_OPERATIONS = {
-  ATTACH: 'attach',
-  PLUG: 'plug',
-  UNPLUG: 'unplug',
-} as const
-
-export type VIF_OPERATIONS = (typeof VIF_OPERATIONS)[keyof typeof VIF_OPERATIONS]
-
-export const VIF_LOCKING_MODE = {
-  DISABLED: 'disabled',
-  LOCKED: 'locked',
-  NETWORK_DEFAULT: 'network_default',
-  UNLOCKED: 'unlocked',
-} as const
-
-export type VIF_LOCKING_MODE = (typeof VIF_LOCKING_MODE)[keyof typeof VIF_LOCKING_MODE]
-
-export const VIF_IPV4_CONFIGURATION_MODE = {
-  NONE: 'None',
-  STATIC: 'Static',
-} as const
-
-export type VIF_IPV4_CONFIGURATION_MODE = (typeof VIF_IPV4_CONFIGURATION_MODE)[keyof typeof VIF_IPV4_CONFIGURATION_MODE]
-
-export const VIF_IPV6_CONFIGURATION_MODE = {
-  NONE: 'None',
-  STATIC: 'Static',
-} as const
-
-export type VIF_IPV6_CONFIGURATION_MODE = (typeof VIF_IPV6_CONFIGURATION_MODE)[keyof typeof VIF_IPV6_CONFIGURATION_MODE]
-
-export const PIF_IGMP_STATUS = {
-  DISABLED: 'disabled',
-  ENABLED: 'enabled',
-  UNKNOWN: 'unknown',
-} as const
-
-export type PIF_IGMP_STATUS = (typeof PIF_IGMP_STATUS)[keyof typeof PIF_IGMP_STATUS]
-
-export const IP_CONFIGURATION_MODE = {
-  DHCP: 'DHCP',
-  NONE: 'None',
-  STATIC: 'Static',
-} as const
-
-export type IP_CONFIGURATION_MODE = (typeof IP_CONFIGURATION_MODE)[keyof typeof IP_CONFIGURATION_MODE]
-
-export const IPV6_CONFIGURATION_MODE = {
-  AUTOCONF: 'Autoconf',
-  DHCP: 'DHCP',
-  NONE: 'None',
-  STATIC: 'Static',
-} as const
-
-export type IPV6_CONFIGURATION_MODE = (typeof IPV6_CONFIGURATION_MODE)[keyof typeof IPV6_CONFIGURATION_MODE]
-
-export const PRIMARY_ADDRESS_TYPE = {
-  IPV4: 'IPv4',
-  IPV6: 'IPv6',
-} as const
-
-export type PRIMARY_ADDRESS_TYPE = (typeof PRIMARY_ADDRESS_TYPE)[keyof typeof PRIMARY_ADDRESS_TYPE]
-
-export const BOND_MODE = {
-  ACTIVE_BACKUP: 'active-backup',
-  BALANCE_SLB: 'balance-slb',
-  LACP: 'lacp',
-} as const
-
-export type BOND_MODE = (typeof BOND_MODE)[keyof typeof BOND_MODE]
-
-export const STORAGE_OPERATIONS = {
-  DESTROY: 'destroy',
-  FORGET: 'forget',
-  PBD_CREATE: 'pbd_create',
-  PBD_DESTROY: 'pbd_destroy',
-  PLUG: 'plug',
-  SCAN: 'scan',
-  UNPLUG: 'unplug',
-  UPDATE: 'update',
-  VDI_CLONE: 'vdi_clone',
-  VDI_CREATE: 'vdi_create',
-  VDI_DATA_DESTROY: 'vdi_data_destroy',
-  VDI_DESTROY: 'vdi_destroy',
-  VDI_DISABLE_CBT: 'vdi_disable_cbt',
-  VDI_ENABLE_CBT: 'vdi_enable_cbt',
-  VDI_INTRODUCE: 'vdi_introduce',
-  VDI_LIST_CHANGED_BLOCKS: 'vdi_list_changed_blocks',
-  VDI_MIRROR: 'vdi_mirror',
-  VDI_RESIZE: 'vdi_resize',
-  VDI_SET_ON_BOOT: 'vdi_set_on_boot',
-  VDI_SNAPSHOT: 'vdi_snapshot',
-} as const
-
-export type STORAGE_OPERATIONS = (typeof STORAGE_OPERATIONS)[keyof typeof STORAGE_OPERATIONS]
-
-export const SR_HEALTH = {
-  HEALTHY: 'healthy',
-  RECOVERING: 'recovering',
-} as const
-
-export type SR_HEALTH = (typeof SR_HEALTH)[keyof typeof SR_HEALTH]
-
-export const VDI_OPERATIONS = {
-  BLOCKED: 'blocked',
-  CLONE: 'clone',
-  COPY: 'copy',
-  DATA_DESTROY: 'data_destroy',
-  DESTROY: 'destroy',
-  DISABLE_CBT: 'disable_cbt',
-  ENABLE_CBT: 'enable_cbt',
-  FORCE_UNLOCK: 'force_unlock',
-  FORGET: 'forget',
-  GENERATE_CONFIG: 'generate_config',
-  LIST_CHANGED_BLOCKS: 'list_changed_blocks',
-  MIRROR: 'mirror',
-  RESIZE: 'resize',
-  RESIZE_ONLINE: 'resize_online',
-  SET_ON_BOOT: 'set_on_boot',
-  SNAPSHOT: 'snapshot',
-  UPDATE: 'update',
-} as const
-
-export type VDI_OPERATIONS = (typeof VDI_OPERATIONS)[keyof typeof VDI_OPERATIONS]
-
-export const VDI_TYPE = {
-  CBT_METADATA: 'cbt_metadata',
-  CRASHDUMP: 'crashdump',
-  EPHEMERAL: 'ephemeral',
-  HA_STATEFILE: 'ha_statefile',
-  METADATA: 'metadata',
-  PVS_CACHE: 'pvs_cache',
-  REDO_LOG: 'redo_log',
-  RRD: 'rrd',
-  SUSPEND: 'suspend',
-  SYSTEM: 'system',
-  USER: 'user',
-} as const
-
-export type VDI_TYPE = (typeof VDI_TYPE)[keyof typeof VDI_TYPE]
-
-export const ON_BOOT = {
-  PERSIST: 'persist',
-  RESET: 'reset',
-} as const
-
-export type ON_BOOT = (typeof ON_BOOT)[keyof typeof ON_BOOT]
-
-export const VBD_OPERATIONS = {
-  ATTACH: 'attach',
-  EJECT: 'eject',
-  INSERT: 'insert',
-  PAUSE: 'pause',
-  PLUG: 'plug',
-  UNPAUSE: 'unpause',
-  UNPLUG: 'unplug',
-  UNPLUG_FORCE: 'unplug_force',
-} as const
-
-export type VBD_OPERATIONS = (typeof VBD_OPERATIONS)[keyof typeof VBD_OPERATIONS]
-
-export const VBD_TYPE = {
-  CD: 'CD',
-  DISK: 'Disk',
-  FLOPPY: 'Floppy',
-} as const
-
-export type VBD_TYPE = (typeof VBD_TYPE)[keyof typeof VBD_TYPE]
-
-export const VBD_MODE = {
-  RO: 'RO',
-  RW: 'RW',
-} as const
-
-export type VBD_MODE = (typeof VBD_MODE)[keyof typeof VBD_MODE]
-
-export const VTPM_OPERATIONS = {
-  DESTROY: 'destroy',
-} as const
-
-export type VTPM_OPERATIONS = (typeof VTPM_OPERATIONS)[keyof typeof VTPM_OPERATIONS]
-
-export const PERSISTENCE_BACKEND = {
-  XAPI: 'xapi',
-} as const
-
-export type PERSISTENCE_BACKEND = (typeof PERSISTENCE_BACKEND)[keyof typeof PERSISTENCE_BACKEND]
-
-export const CONSOLE_PROTOCOL = {
-  RDP: 'rdp',
-  RFB: 'rfb',
-  VT100: 'vt100',
-} as const
-
-export type CONSOLE_PROTOCOL = (typeof CONSOLE_PROTOCOL)[keyof typeof CONSOLE_PROTOCOL]
-
-export const CLS = {
-  CERTIFICATE: 'Certificate',
-  HOST: 'Host',
-  POOL: 'Pool',
-  PVS_PROXY: 'PVS_proxy',
-  SR: 'SR',
-  VDI: 'VDI',
-  VM: 'VM',
-  VMPP: 'VMPP',
-  VMSS: 'VMSS',
-} as const
-
-export type CLS = (typeof CLS)[keyof typeof CLS]
-
-export const TUNNEL_PROTOCOL = {
-  GRE: 'gre',
-  VXLAN: 'vxlan',
-} as const
-
-export type TUNNEL_PROTOCOL = (typeof TUNNEL_PROTOCOL)[keyof typeof TUNNEL_PROTOCOL]
-
-export const SRIOV_CONFIGURATION_MODE = {
-  MANUAL: 'manual',
-  MODPROBE: 'modprobe',
-  SYSFS: 'sysfs',
-  UNKNOWN: 'unknown',
-} as const
-
-export type SRIOV_CONFIGURATION_MODE = (typeof SRIOV_CONFIGURATION_MODE)[keyof typeof SRIOV_CONFIGURATION_MODE]
-
-export const PGPU_DOM0_ACCESS = {
-  DISABLED: 'disabled',
-  DISABLE_ON_REBOOT: 'disable_on_reboot',
-  ENABLED: 'enabled',
-  ENABLE_ON_REBOOT: 'enable_on_reboot',
-} as const
-
-export type PGPU_DOM0_ACCESS = (typeof PGPU_DOM0_ACCESS)[keyof typeof PGPU_DOM0_ACCESS]
-
-export const ALLOCATION_ALGORITHM = {
-  BREADTH_FIRST: 'breadth_first',
-  DEPTH_FIRST: 'depth_first',
-} as const
-
-export type ALLOCATION_ALGORITHM = (typeof ALLOCATION_ALGORITHM)[keyof typeof ALLOCATION_ALGORITHM]
-
-export const VGPU_TYPE_IMPLEMENTATION = {
-  GVT_G: 'gvt_g',
-  MXGPU: 'mxgpu',
-  NVIDIA: 'nvidia',
-  NVIDIA_SRIOV: 'nvidia_sriov',
-  PASSTHROUGH: 'passthrough',
-} as const
-
-export type VGPU_TYPE_IMPLEMENTATION = (typeof VGPU_TYPE_IMPLEMENTATION)[keyof typeof VGPU_TYPE_IMPLEMENTATION]
-
-export const PVS_PROXY_STATUS = {
-  CACHING: 'caching',
-  INCOMPATIBLE_PROTOCOL_VERSION: 'incompatible_protocol_version',
-  INCOMPATIBLE_WRITE_CACHE_MODE: 'incompatible_write_cache_mode',
-  INITIALISED: 'initialised',
-  STOPPED: 'stopped',
-} as const
-
-export type PVS_PROXY_STATUS = (typeof PVS_PROXY_STATUS)[keyof typeof PVS_PROXY_STATUS]
-
-export const SDN_CONTROLLER_PROTOCOL = {
-  PSSL: 'pssl',
-  SSL: 'ssl',
-} as const
-
-export type SDN_CONTROLLER_PROTOCOL = (typeof SDN_CONTROLLER_PROTOCOL)[keyof typeof SDN_CONTROLLER_PROTOCOL]
-
-export const VUSB_OPERATIONS = {
-  ATTACH: 'attach',
-  PLUG: 'plug',
-  UNPLUG: 'unplug',
-} as const
-
-export type VUSB_OPERATIONS = (typeof VUSB_OPERATIONS)[keyof typeof VUSB_OPERATIONS]
-
-export const CLUSTER_OPERATION = {
-  ADD: 'add',
-  DESTROY: 'destroy',
-  DISABLE: 'disable',
-  ENABLE: 'enable',
-  REMOVE: 'remove',
-} as const
-
-export type CLUSTER_OPERATION = (typeof CLUSTER_OPERATION)[keyof typeof CLUSTER_OPERATION]
-
-export const CLUSTER_HOST_OPERATION = {
-  DESTROY: 'destroy',
-  DISABLE: 'disable',
-  ENABLE: 'enable',
-} as const
-
-export type CLUSTER_HOST_OPERATION = (typeof CLUSTER_HOST_OPERATION)[keyof typeof CLUSTER_HOST_OPERATION]
-
-export const CERTIFICATE_TYPE = {
-  CA: 'ca',
-  HOST: 'host',
-  HOST_INTERNAL: 'host_internal',
-} as const
-
-export type CERTIFICATE_TYPE = (typeof CERTIFICATE_TYPE)[keyof typeof CERTIFICATE_TYPE]
-
-export const OPAQUE_REF = { EMPTY: 'OpaqueRef:NULL' } as const
-
-export type OPAQUE_REF_NULL = (typeof OPAQUE_REF)['EMPTY']
-
-export const BACKUP_TYPE = { backup: 'backup', metadata: 'metadataBackup', mirror: 'mirrorBackup' } as const
-
-export type BACKUP_TYPE = (typeof BACKUP_TYPE)[keyof typeof BACKUP_TYPE]
-
-export const SUPPORTED_VDI_FORMAT = {
-  raw: 'raw',
-  vhd: 'vhd',
-  qcow2: 'qcow2',
-} as const
-
-export type SUPPORTED_VDI_FORMAT = (typeof SUPPORTED_VDI_FORMAT)[keyof typeof SUPPORTED_VDI_FORMAT]
-
-// ----- XAPI Stats
-
-type XapiStatsResponse<T> = {
-  endTimestamp: number
-  interval: number
-  stats: T
-}
-type StatValues = (number | null)[]
-type RecordStatValues = Record<string, StatValues>
-
-export type XapiStatsGranularity = 'seconds' | 'minutes' | 'hours' | 'days'
-
-export type XapiHostStatsRaw = {
-  cpus?: RecordStatValues
-  ioThroughput?: {
-    r: RecordStatValues
-    w: RecordStatValues
-  }
-  iops?: {
-    r: RecordStatValues
-    w: RecordStatValues
-  }
-  iowait?: RecordStatValues
-  latency?: {
-    r: RecordStatValues
-    w: RecordStatValues
-  }
-  load?: StatValues
-  memory?: StatValues
-  memoryFree?: StatValues
-  pifs?: {
-    rx: RecordStatValues
-    tx: RecordStatValues
-  }
-}
-export type XapiHostStats = XapiStatsResponse<XapiHostStatsRaw>
-
-export type XapiVmStatsRaw = {
-  cpus?: RecordStatValues
-  cpuUsage?: StatValues
-  runstateFullrun?: StatValues
-  runstateFullContention?: StatValues
-  runstatePartialRun?: StatValues
-  runstatePartialContention?: StatValues
-  runstateConcurrencyHazard?: StatValues
-  runstateBlocked?: StatValues
-  iops?: {
-    r: RecordStatValues
-    w: RecordStatValues
-  }
-  memory?: StatValues
-  memoryFree?: StatValues
-  memoryTarget?: StatValues
-  vifs?: {
-    rx: RecordStatValues
-    tx: RecordStatValues
-  }
-  vifErrors?: {
-    rx: RecordStatValues
-    tx: RecordStatValues
-  }
-  xvds?: {
-    w?: RecordStatValues
-    r?: RecordStatValues
-    total?: RecordStatValues
-  }
-  vbdLatency?: {
-    w: RecordStatValues
-    r: RecordStatValues
-  }
-  vbdIowait?: RecordStatValues
-  vbdInflight?: RecordStatValues
-  vbdAvgquSz?: RecordStatValues
-}
-export type XapiVmStats = XapiStatsResponse<XapiVmStatsRaw>
-
-export type XapiPoolStats = Record<XoHost['id'], XapiHostStats | { error: Record<string, unknown> }>

@vates/types/src/index.mts

@@ -1,5 +0,0 @@
-export * from './common.mjs'
-export * from './xen-api.mjs'
-export * from './xo.mjs'
-export * from './lib/xen-orchestra-xapi.mjs'
-export * from './lib/complex-matcher.mjs'

@vates/types/src/lib/complex-matcher.mts

@@ -1,5 +0,0 @@
-export type Or<T> = Id<{ __or: T[] }>
-
-export type Id<T> = { id: T }
-
-export type IdOr<T> = Id<T> | Or<T>

@vates/types/src/lib/vates-task.mts

@@ -1,21 +0,0 @@
-import { Branded } from '../common.mjs'
-
-export interface VatesTask {
-  _abortController: AbortController
-  _onProgress(data?: unknown): void
-
-  id: Branded<'vates-task'>
-  status: 'failure' | 'pending' | 'success'
-
-  abort(reason?: unknown): void
-  failure(error?: unknown): void
-  info(message: string, data: unknown): void
-  run<T>(fn: () => T | (() => Promise<T>)): Promise<T>
-  runInside<T>(fn: () => T | (() => Promise<T>)): Promise<T>
-  set(name: string, value: unknown): void
-  start(): void
-  success(result?: unknown): void
-  warning(message: string, data: unknown): void
-  wrap<T>(fn: () => T | (() => Promise<T>)): Promise<T>
-  wrapInside<T>(fn: () => T | (() => Promise<T>)): Promise<T>
-}

@vates/types/src/lib/xen-orchestra-xapi.mts

@@ -1,204 +0,0 @@
-import {
-  WrappedXenApiRecord,
-  XenApiHost,
-  XenApiNetworkWrapped,
-  XenApiRecord,
-  XenApiSr,
-  XenApiTask,
-  XenApiVdi,
-  XenApiVm,
-  XenApiVmWrapped,
-} from '../xen-api.mjs'
-import type { PassThrough, Readable } from 'node:stream'
-import type {
-  XoGpuGroup,
-  XoVgpuType,
-  XoHost,
-  XoNetwork,
-  XoPif,
-  XoSr,
-  XoUser,
-  XoVdi,
-  XoVm,
-  XoVmTemplate,
-} from '../xo.mjs'
-import type { SUPPORTED_VDI_FORMAT } from '../common.mjs'
-
-export type XcpPatches = {
-  changelog?: {
-    author: string
-    date: number
-    description: string
-  }
-  description: string
-  license: string
-  name: string
-  release: string
-  size: number
-  url: string
-  version: string
-}
-export type XsPatches = {
-  conflicts?: string[]
-  date: string
-  description: string
-  documentationUrl?: string
-  guidances: string
-  name: string
-  id?: string
-  paid?: boolean
-  requirements?: string[]
-  upgrade?: boolean
-  url?: string
-  uuid?: string
-}
-
-export interface Xapi {
-  call: <ReturnType>(...args: unknown[]) => Promise<ReturnType>
-  callAsync: <ReturnType>(...args: unknown[]) => Promise<ReturnType>
-
-  getField<T extends XenApiRecord, K extends keyof T>(
-    type: Extract<WrappedXenApiRecord, T>['$type'],
-    ref: T['$ref'],
-    field: K
-  ): Promise<T[K]>
-  createNetwork(
-    params:
-      | {
-          name: string
-          description?: string
-          mtu?: number
-        }
-      | {
-          name: string
-          description?: string
-          pifId: XoPif['id']
-          mtu?: number
-          /* between 0 and 4094 */
-          vlan: number
-        }
-  ): Promise<XenApiNetworkWrapped>
-  deleteNetwork(id: XoNetwork['id']): Promise<void>
-  exportVmOva(vmRef: XenApiVm['$ref']): Promise<PassThrough>
-  listMissingPatches(host: XoHost['id']): Promise<XcpPatches[] | XsPatches[]>
-  pool_emergencyShutdown(): Promise<void>
-  resumeVm(id: XoVm['id']): Promise<void>
-  unpauseVm(id: XoVm['id']): Promise<void>
-  SR_importVdi(
-    ref: XenApiSr['$ref'],
-    stream: Readable,
-    opts?: {
-      format?: SUPPORTED_VDI_FORMAT
-      name_description?: XoVdi['name_description']
-      name_label?: XoVdi['name_label']
-      other_config?: XoVdi['other_config']
-      read_only?: boolean
-      sharable?: boolean
-      SR?: XenApiSr['$ref']
-      tags?: XoVdi['tags']
-      type?: XoVdi['type']
-      virtual_size?: XoVdi['size']
-      xenstore_data?: Record<string, string>
-    }
-  ): Promise<XenApiVdi['$ref']>
-  startVm(
-    id: XoVm['id'],
-    opts?: {
-      bypassMacAddressesCheck?: boolean
-      force?: boolean
-      hostId?: XoHost['id']
-      /**
-       * if startOnly is true and the VM is not halted, throw VM_BAD_POWER_STATE
-       * otherwise, unpause/resume the VM
-       *
-       * @default false
-       */
-      startOnly?: boolean
-    }
-  ): Promise<void>
-  VM_export(
-    vmRef: XenApiVm['$ref'],
-    opts?: { cancelToken?: unknown; compress?: boolean; useSnapshot?: boolean }
-  ): ReturnType<Xapi['getResource']>
-  VM_import(
-    stream: Readable,
-    srRef?: XenApiSr['$ref'],
-    onVmCreation?: null | ((vm: XenApiVm) => unknown)
-  ): Promise<XenApiVm['$ref']>
-  createVm(
-    templateUuid: XoVmTemplate['uuid'],
-    metadataVm: {
-      affinityHost?: XoHost['id']
-      autoPoweron?: boolean
-      memory?: number
-      name_description?: string
-      name_label: string
-      nameLabel?: string
-      clone?: boolean
-      installRepository?: XoVdi['id'] | '' | null
-      vdis?: (
-        | /** Create VDI */ {
-            name_label: string
-            size: number
-            sr?: XoSr['id']
-            name_description?: string
-          }
-        | /** Update VDI*/ {
-            userdevice: string
-            name_label?: string
-            size?: number
-            sr?: XoSr['id']
-            name_description?: string
-          }
-        | /** Destroy VDI*/ { destroy: true; userdevice: string }
-      )[]
-      vifs?: (
-        | /** Create/update VIF */ {
-            device?: string
-            ipv4_allowed?: string[]
-            ipv6_allowed?: string[]
-            mac?: string
-            mtu?: number
-            network: string
-          }
-        | /** Destroy VIF */ { destroy: true; device: string }
-      )[]
-      existingVdis?: {
-        $SR: XoSr['id']
-        size: number
-        userdevice: string
-      }[]
-      vgpuType?: XoVgpuType['id']
-      gpuGroup?: XoGpuGroup['id']
-      copyHostBiosStrings?: boolean
-    },
-    checkLimits?: boolean,
-    creatorId?: XoUser['id'],
-    opts?: { destroyAllVifs: boolean }
-  ): Promise<XenApiVmWrapped>
-  VDI_destroy(vdiRef: XenApiVdi['$ref']): Promise<void>
-  VDI_destroyCloudInitConfig(vdiRef: XenApiVdi['$ref'], opts?: { timeLimit?: number }): Promise<void>
-  VDI_exportContent(
-    vdiRef: XenApiVdi['$ref'],
-    opts: { baseRef?: string; cancelToken?: unknown; format: SUPPORTED_VDI_FORMAT }
-  ): Promise<Readable & { length?: number }>
-  VM_createCloudInitConfig(
-    vmRef: XenApiVm['$ref'],
-    cloudConfig: string,
-    opts?: { networkConfig?: string }
-  ): Promise<XoVdi['uuid']>
-  VM_destroy(
-    vmRef: XenApiVm['$ref'],
-    opts?: {
-      deleteDisks?: boolean
-      force?: boolean
-      bypassBlockedOperation?: boolean
-      forceDeleteDefaultTemplate?: boolean
-    }
-  ): Promise<void>
-  getResource(
-    pathname: string,
-    params?: { host?: XenApiHost; query?: Record<string, unknown>; task?: boolean | XenApiTask['$ref'] }
-  ): Promise<{ body: Readable }>
-  isHyperThreadingEnabled(hostId: XoHost['id']): Promise<boolean | null>
-}

@vates/types/src/xo.mts

@@ -1,784 +0,0 @@
-// Types based on xapi-object-to-xo
-
-import type {
-  Branded,
-  DOMAIN_TYPE,
-  HOST_ALLOWED_OPERATIONS,
-  HOST_POWER_STATE,
-  IP_CONFIGURATION_MODE,
-  IPV6_CONFIGURATION_MODE,
-  NETWORK_OPERATIONS,
-  PGPU_DOM0_ACCESS,
-  POOL_ALLOWED_OPERATIONS,
-  PRIMARY_ADDRESS_TYPE,
-  STORAGE_OPERATIONS,
-  VDI_OPERATIONS,
-  VDI_TYPE,
-  VIF_LOCKING_MODE,
-  VM_OPERATIONS,
-  VM_POWER_STATE,
-} from './common.mjs'
-import type * as CMType from './lib/complex-matcher.mjs'
-
-type BaseXapiXo = {
-  $pool: XoPool['id']
-  /**
-   * @deprecated use $pool instead
-   */
-  $poolId: XoPool['id']
-
-  _xapiRef: string
-
-  uuid: string
-}
-
-type BaseXoVm = BaseXapiXo & {
-  $VBDs: XoVbd['id'][]
-  $VGPUs: XoVgpu['id'][]
-
-  $container: XoPool['id'] | XoHost['id']
-
-  CPUs: {
-    max: number
-    number: number
-  }
-  PV_args?: string
-  VGPUs: XoVgpu['id'][]
-  VIFs: XoVif['id'][]
-  VTPMs: XoVtpm['id'][]
-
-  addresses: Record<string, string>
-  affinityHost?: XoHost['id']
-  attachedPcis?: string[]
-  auto_poweron: boolean
-  bios_strings: Record<string, string>
-  blockedOperations: Record<VM_OPERATIONS, string>
-  boot: Record<string, string>
-  coresPerSocket?: number
-  cpuCap?: number
-  cpuMask?: number[]
-  cpuWeight?: number
-  creation: Record<string, string>
-  current_operations: Record<string, VM_OPERATIONS>
-  docker?: {
-    containers?: string[]
-    enabled: boolean
-    info?: string
-    /** @deprecated */
-    process?: string
-    version?: string
-  }
-  /**
-   * @deprecated use isNestedVirtEnabled instead
-   */
-  expNestedHvm: boolean
-  isNestedVirtEnabled: boolean
-  hasVendorDevice: boolean
-  high_availability: string
-  installTime?: number | null
-  isFirmwareSupported: boolean
-  memory: {
-    dynamic: number[]
-    size: number
-    static: number[]
-    usage?: number
-  }
-  mainIpAddress?: string
-  managementAgentDetected?: boolean
-  name_description: string
-  name_label: string
-  needsVtpm: boolean
-  nicType?: string
-  notes?: string
-  os_version: Record<string, string> | null
-  other: Record<string, string>
-  parent?: XoVm['id']
-  power_state: VM_POWER_STATE
-  pvDriversDetected?: boolean
-  pvDriversUpToDate?: boolean
-  pvDriversVersion?: string
-  resourceSet?: string
-  secureBoot: boolean
-  snapshots: XoVmSnapshot['id'][]
-  startDelay: number
-  startTime?: number | null
-  suspendSr?: XoSr['id']
-  tags: string[]
-  vga?: string
-  videoram?: number
-  viridian: boolean
-  virtualizationMode: DOMAIN_TYPE
-  xenStoreData: Record<string, string>
-  /**
-   * @deprecated use pvDriversVersion instead
-   */
-  xentools?:
-    | false
-    | {
-        major: null | number
-        minor: null | number
-        version: null | string
-      }
-}
-
-export type XoAlarm = Omit<XoMessage, '$object' | 'body'> & {
-  body: {
-    value?: string
-    name: string
-  }
-  object: {
-    type: XapiXoRecord['type'] | 'unknown'
-    uuid: XapiXoRecord['uuid']
-    href?: string
-  }
-}
-
-// TODO: to be typed when Bastien.N has finished working on the XO task
-type BaseXoLog = {
-  id: Branded<'xo-log'>
-  [key: string]: unknown
-}
-export type XoBackupLog = BaseXoLog & {
-  message: 'backup' | 'metadata'
-}
-export type XoRestoreLog = BaseXoLog & {
-  message: 'restore'
-}
-
-export type XoBackupRepository = {
-  benchmarks?: { readRate: number; timestamp: number; writeRate: number }[]
-  enabled: boolean
-  error?: Record<string, unknown>
-  id: Branded<'backup-repository'>
-  name: string
-  options?: string
-  proxy?: XoProxy['id']
-  url: string
-}
-
-export type XoGpuGroup = BaseXapiXo & {
-  id: Branded<'gpu-group'>
-  type: 'gpuGroup'
-}
-
-export type XoGroup = {
-  id: Branded<'group'>
-  name: string
-  provider?: string
-  providerGroupId?: string
-  users: XoUser['id'][]
-}
-
-export type XoHost = BaseXapiXo & {
-  $PBDs: XoPbd['id'][]
-  $PCIs: XoPci['id'][]
-  $PGPUs: XoPgpu['id'][]
-  $PIFs: XoPif['id'][]
-
-  PCIs: XoPci['id'][]
-  PGPUs: XoPgpu['id'][]
-  PIFs: XoPif['id'][]
-  /**
-   * @deprecated use cpus instead
-   */
-  CPUs: Record<string, string>
-
-  address: string
-  agentStartTime: null | number
-  bios_string: Record<string, string>
-  build: string
-  certificates?: {
-    fingerprint: string
-    notAfter: number
-  }[]
-  chipset_info: {
-    iommu?: boolean
-  }
-  controlDomain?: XoVm['id']
-  cpus: {
-    cores?: number
-    sockets?: number
-  }
-  current_operations: Record<string, HOST_ALLOWED_OPERATIONS>
-  enabled: boolean
-  hostname: string
-  hvmCapable: boolean
-  id: Branded<'host'>
-  iscsiIqn: string
-  license_expiry: null | number
-  license_params: Record<string, string>
-  license_server: Record<string, string>
-  logging: Record<string, string>
-  memory: {
-    size: number
-    /**
-     * @deprecated
-     */
-    total?: number
-    usage: number
-  }
-  multipathing: boolean
-  name_description: string
-  name_label: string
-  otherConfig: Record<string, string>
-  /**
-   * @deprecated
-   */
-  patches: XoHostPatch['id'][]
-  power_state: HOST_POWER_STATE
-  powerOnMode: string
-  productBrand: string
-  rebootRequired: boolean
-  residentVms: XoVm['id'][]
-  startTime: null | number
-  supplementalPacks:
-    | {
-        author: string
-        description: string
-        guidance: string
-        hosts: XoHost['id'][]
-        name: string
-        size: number
-        version: string
-        vdi: XoVdi['id']
-      }[]
-    | {
-        author: string
-        description: string
-        name: string
-        version: string
-      }[]
-  tags: string[]
-  type: 'host'
-  version: string
-  zstdSupported: boolean
-}
-
-export type XoHostPatch = BaseXapiXo & {
-  id: Branded<'host_patch'>
-  type: 'host_patch'
-}
-
-export type XoMessage = BaseXapiXo & {
-  $object: XapiXoRecord['id']
-
-  body: string
-  id: Branded<'message'>
-  name: string
-  time: number
-  type: 'message'
-}
-
-export type XoNetwork = BaseXapiXo & {
-  MTU: number
-  PIFs: XoPif['id'][]
-  VIFs: XoVif['id'][]
-
-  automatic: boolean
-  bridge: string
-  current_operations: Record<string, NETWORK_OPERATIONS>
-  defaultIsLocked: boolean
-  id: Branded<'network'>
-  insecureNbd?: boolean
-  name_description: string
-  name_label: string
-  nbd?: boolean
-  other_config: Record<string, string>
-  tags: string[]
-  type: 'network'
-}
-
-export type XoPbd = BaseXapiXo & {
-  id: Branded<'PBD'>
-  type: 'PBD'
-}
-
-export type XoPci = BaseXapiXo & {
-  $host?: XoHost['id']
-
-  class_name: string
-  device_name: string
-  id: Branded<'PCI'>
-  pci_id: string
-  type: 'PCI'
-}
-
-export type XoPgpu = BaseXapiXo & {
-  $host: XoHost['id']
-  $vgpus: XoVgpu['id'][]
-
-  dom0Access: PGPU_DOM0_ACCESS
-  enabledVgpuTypes: XoVgpuType['id'][]
-  gpuGroup?: XoGpuGroup['id']
-  host: XoHost['id']
-  id: Branded<'PGPU'>
-  isSystemDisplayDevice: boolean
-  pci?: XoPci['id']
-  // it seems that supportedVgpuMaxCapcities always return undefined (See: xapi-objet-to-xo)
-  /**
-   * @deprecated
-   */
-  supportedVgpuMaxCapcities?: never
-  supportedVgpuTypes: XoVgpuType['id'][]
-  type: 'PGPU'
-  vgpus: XoVgpu['id'][]
-}
-
-export type XoPif = BaseXapiXo & {
-  $host: XoHost['id']
-  $network: XoNetwork['id']
-
-  attached: boolean
-  bondMaster?: XoPif['id']
-  bondSalves?: XoPif['id'][]
-  carrier: boolean
-  device: string
-  deviceName?: string
-  disallowUnplug: boolean
-  dns: string
-  gateway: string
-  id: Branded<'PIF'>
-  ip: string
-  ipv6: string[]
-  ipv6Mode: IPV6_CONFIGURATION_MODE
-  isBondMaster: boolean
-  isBondSlave: boolean
-  mac: string
-  management: boolean
-  mode: IP_CONFIGURATION_MODE
-  mtu: number
-  netmask: string
-  physical: boolean
-  primaryAddressType: PRIMARY_ADDRESS_TYPE
-  speed?: number
-  type: 'PIF'
-  vlan: number
-}
-
-export type XoPool = BaseXapiXo & {
-  auto_poweron: boolean
-  cpus: {
-    cores?: number
-    sockets?: number
-  }
-  crashDumpSr?: XoSr['id']
-  current_operations: Record<string, POOL_ALLOWED_OPERATIONS>
-  defaultSr?: XoSr['id']
-  HA_enabled: boolean
-  haSrs: XoSr['id'][]
-  id: Branded<'pool'>
-  master: XoHost['id']
-  migrationCompression?: boolean
-  name_description: string
-  name_label: string
-  otherConfig: Record<string, string>
-  platform_version: string
-  suspendSr?: XoSr['id']
-  tags: string[]
-  type: 'pool'
-  vtpmSupported: boolean
-  xosanPackInstallationTime: number | null
-  zstdSupported: boolean
-}
-
-export type XoProxy = {
-  id: Branded<'proxy'>
-  url: string
-  version?: string
-  name: string
-} & (
-  | { address?: undefined; vmUuid: XoVm['id'] }
-  | { address: string; vmUuid?: undefined }
-  | { address: string; vmUuid: XoVm['id'] }
-)
-
-type BaseXoJob = {
-  id: Branded<'job'>
-  name?: string
-}
-
-type XoBackupJobGeneralSettings = {
-  backupReportTpl?: 'compactMjml' | 'mjml'
-  reportWhen?: 'always' | 'error' | 'failure' | 'never'
-  hideSuccessfulItems?: boolean
-  [key: string]: unknown
-}
-
-export type XoVmBackupJobGeneralSettings = XoBackupJobGeneralSettings & {
-  cbtDestroySnapshotData?: boolean
-  concurrency?: number
-  longTermRetention?: {
-    daily?: { retention: number; settings: Record<string, unknown> }
-    weekly?: { retention: number; settings: Record<string, unknown> }
-    monthly?: { retention: number; settings: Record<string, unknown> }
-    yearly?: { retention: number; settings: Record<string, unknown> }
-  }
-  maxExportRate?: number
-  nbdConcurrency?: number
-  nRetriesVmBackupFailures?: number
-  preferNbd?: boolean
-  timezone?: string
-  mergeBackupsSynchronously?: boolean
-  offlineBackup?: boolean
-  timeout?: number
-  [key: string]: unknown
-}
-export type XoVmBackupJobScheduleSettings = {
-  exportRetention?: number
-  healthCheckVmsWithTags?: XoVm['tags']
-  fullInterval?: number
-  copyRetention?: number
-  snapshotRetention?: number
-  cbtDestroySnapshotData?: boolean
-  healthCheckSr?: XoSr['id']
-  [key: string]: unknown
-}
-export type XoVmBackupJob = BaseXoJob & {
-  compression?: 'native' | 'zstd' | ''
-  proxy?: XoProxy['id']
-  mode: 'full' | 'delta'
-  remotes?: CMType.IdOr<XoBackupRepository['id']>
-  vms: CMType.IdOr<XoVm['id']> | Record<string, unknown>
-  srs?: CMType.IdOr<XoSr['id']>
-  type: 'backup'
-  settings: {
-    '': XoVmBackupJobGeneralSettings
-    [key: XoSchedule['id']]: XoVmBackupJobScheduleSettings | undefined
-  }
-}
-
-export type XoMetadataBackupJobGeneralSettings = XoBackupJobGeneralSettings
-export type XoMetadataBackupJobScheduleSettings = {
-  retentionPoolMetadata?: number
-  retentionXoMetadata?: number
-  [key: string]: unknown
-}
-export type XoMetadataBackupJob = BaseXoJob & {
-  type: 'metadataBackup'
-  pools?: CMType.IdOr<XoPool['id']>
-  remotes: CMType.IdOr<XoBackupRepository['id']>
-  settings: {
-    ''?: XoMetadataBackupJobGeneralSettings
-    [scheduleId: XoSchedule['id']]: XoMetadataBackupJobScheduleSettings | undefined
-  }
-  xoMetadata?: boolean
-  userId: XoUser['id']
-  proxy?: XoProxy['id']
-}
-
-export type XoMirrorBackupGeneralSettings = XoBackupJobGeneralSettings & {
-  concurrency?: number
-  nRetriesVmBackupFailures?: number
-  timeout?: number
-  maxExportRate?: number
-  backupReportTpl?: 'compactMjml'
-  reportWhen: 'failure'
-  [key: string]: unknown
-}
-export type XoMirrorBackupScheduleSettings = {
-  exportRetention?: number
-  healthCheckVmsWithTags?: XoVm['tags']
-  healthCheckSr?: XoSr['id']
-  [key: string]: unknown
-}
-export type XoMirrorBackupJob = BaseXoJob & {
-  type: 'mirrorBackup'
-  mode: 'full' | 'delta'
-  sourceRemote: XoBackupRepository['id']
-  remotes: CMType.IdOr<XoBackupRepository['id']>
-  settings: {
-    '': XoMirrorBackupGeneralSettings
-    [scheduleId: XoSchedule['id']]: XoMirrorBackupScheduleSettings | undefined
-  }
-}
-
-export type XoJob = BaseXoJob & {}
-
-export type XoSchedule = {
-  cron: string
-  enabled: boolean
-  id: Branded<'schedule'>
-  jobId: AnyXoJob['id']
-  name?: string
-  timezone?: string
-}
-
-export type XoServer = {
-  allowUnauthorized: boolean
-  enabled: boolean
-  error?: Record<string, unknown>
-  host: string
-  httpProxy?: string
-  id: Branded<'server'>
-  label?: string
-  master?: XoHost['id']
-  poolId?: XoPool['id']
-  poolNameDescription?: string
-  poolNameLabel?: string
-  readOnly: boolean
-  status: 'connected' | 'disconnected' | 'connecting'
-  username: string
-}
-
-export type XoSr = BaseXapiXo & {
-  $PBDs: XoPbd['id'][]
-
-  $container: XoPool['id'] | XoHost['id']
-
-  VDIs: AnyXoVdi['id'][]
-
-  allocationStrategy: 'thin' | 'thick' | 'unknown'
-  content_type: string
-  current_operations: Record<string, STORAGE_OPERATIONS>
-  id: Branded<'SR'>
-  inMaintenanceMode: boolean
-  name_description: string
-  name_label: string
-  other_config: Record<string, string>
-  physical_usage: number
-  shared: boolean
-  size: number
-  sm_config: Record<string, string>
-  SR_type: string
-  tags: string[]
-  type: 'SR'
-  usage: number
-}
-
-export type XoSm = BaseXapiXo & {
-  id: Branded<'SM'>
-
-  SM_type: string
-  vendor: string
-  name_label: string
-  name_description: string
-  configuration: Record<string, string>
-  features: Record<string, number>
-  driver_filename: string
-  required_cluster_stack: string[]
-  supported_image_formats: string[]
-  type: 'SM'
-}
-
-export type XoTask = {
-  abortionRequestedAt?: number
-  end?: number
-  id: Branded<'task'>
-  infos?: { data: unknown; message: string }[]
-  properties: {
-    method?: string
-    name?: string
-    objectId?: string
-    params?: Record<string, unknown>
-    type?: string
-    userId?: string
-    [key: string]: unknown | undefined
-  }
-  result: Record<string, unknown>
-  start: number
-  status: 'failure' | 'interrupted' | 'pending' | 'success'
-  tasks?: XoTask[]
-  updatedAt?: number
-  warnings?: { data: unknown; message: string }[]
-}
-
-export type XoUser = {
-  authProviders?: Record<string, string>
-  email: string
-  groups: XoGroup['id'][]
-  id: Branded<'user'>
-  name?: string
-  permission: string
-  pw_hash?: string
-  preferences: Record<string, string>
-}
-
-export type XoAuthenticationToken = {
-  client?: {
-    id: string
-    [key: string]: unknown
-  }
-  created_at?: number
-  description?: string
-  user_id: XoUser['id']
-  expiration: number
-  last_uses?: Record<string, { timestamp: number }>
-  id: Branded<'authentication-token'>
-}
-
-export type XoVbd = BaseXapiXo & {
-  attached: boolean
-  bootable: boolean
-  device: string | null
-  id: Branded<'VBD'>
-  is_cd_drive: boolean
-  position: string
-  read_only: boolean
-  type: 'VBD'
-  VDI?: XoVdi['id'] | XoVdiSnapshot['id']
-  VM: AnyXoVm['id']
-}
-
-type BaseXoVdi = BaseXapiXo & {
-  $SR: XoSr['id']
-  $VBDs: XoVbd['id'][]
-
-  VDI_type: VDI_TYPE
-
-  cbt_enabled?: boolean
-  current_operations: Record<string, VDI_OPERATIONS>
-  missing: boolean
-  name_description: string
-  name_label: string
-  other_config: Record<string, string>
-  parent?: XoVdiUnmanaged['id']
-  image_format?: string
-  size: number
-  snapshots: XoVdiSnapshot['id'][]
-  tags: string[]
-  usage: number
-}
-
-export type XoVdi = BaseXoVdi & {
-  id: Branded<'VDI'>
-  type: 'VDI'
-}
-
-export type XoVdiSnapshot = BaseXoVdi & {
-  id: Branded<'VDI-snapshot'>
-  snapshot_time: number
-  $snapshot_of?: XoVdi['id']
-  type: 'VDI-snapshot'
-}
-
-export type XoVdiUnmanaged = Omit<BaseXoVdi, '$VBDs'> & {
-  id: Branded<'VDI-unmanaged'>
-  type: 'VDI-unmanaged'
-
-  /**
-   * Unmanaged VDI have no VBDs
-   */
-  $VBDs: never[]
-}
-
-export type XoVgpu = BaseXapiXo & {
-  id: Branded<'VGPU'>
-  type: 'vgpu'
-}
-
-export type XoVgpuType = BaseXapiXo & {
-  id: Branded<'vgpu-type'>
-  type: 'vgpuType'
-}
-
-export type XoVif = BaseXapiXo & {
-  $VM: XoVm['id']
-
-  $network: XoNetwork['id']
-
-  allowedIpv4Addresses: string[]
-  allowedIpv6Addresses: string[]
-  attached: boolean
-  device: string
-  id: Branded<'VIF'>
-  lockingMode: VIF_LOCKING_MODE
-  MAC: string
-  MTU: number
-  other_config: Record<string, string>
-  /**
-   * In kB/s
-   */
-  rateLimit?: number
-  txChecksumming: boolean
-  type: 'VIF'
-}
-
-export type XoVm = BaseXoVm & {
-  id: Branded<'VM'>
-  type: 'VM'
-  vulnerabilities: { xsa468: boolean | { reason: string; driver?: string; version?: string } }
-}
-
-export type XoVmController = BaseXoVm & {
-  id: Branded<'VM-controller'>
-  type: 'VM-controller'
-}
-
-export type XoVmSnapshot = BaseXoVm & {
-  $snapshot_of: XoVm['id']
-
-  id: Branded<'VM-snapshot'>
-  snapshot_time: number
-  suspendVdi?: XoVdi['id']
-  type: 'VM-snapshot'
-}
-
-export type XoVmTemplate = BaseXoVm & {
-  id: Branded<'VM-template'>
-  isDefaultTemplate: boolean
-  template_info: {
-    arch?: string
-    disks: {
-      bootable: true
-      size: number
-      SR: string
-      type: string
-    }[]
-    install_methods: string[]
-    install_repository?: string
-  }
-  type: 'VM-template'
-}
-
-export type XoVtpm = BaseXapiXo & {
-  id: Branded<'VTPM'>
-  type: 'VTPM'
-}
-
-export type XapiXoRecord =
-  | XoAlarm
-  | XoGpuGroup
-  | XoHost
-  | XoMessage
-  | XoNetwork
-  | XoPci
-  | XoPgpu
-  | XoPif
-  | XoPool
-  | XoSr
-  | XoVbd
-  | XoVdi
-  | XoVdiSnapshot
-  | XoVdiUnmanaged
-  | XoVgpu
-  | XoVgpuType
-  | XoVif
-  | XoVm
-  | XoVmController
-  | XoVmSnapshot
-  | XoVmTemplate
-  | XoVtpm
-  | XoSm
-
-export type NonXapiXoRecord =
-  | AnyXoJob
-  | AnyXoLog
-  | XoGroup
-  | XoProxy
-  | XoBackupRepository
-  | XoSchedule
-  | XoServer
-  | XoTask
-  | XoUser
-
-export type XoRecord = XapiXoRecord | NonXapiXoRecord
-
-export type AnyXoVm = XoVm | XoVmSnapshot | XoVmTemplate | XoVmController
-
-export type AnyXoVdi = XoVdi | XoVdiSnapshot | XoVdiUnmanaged
-
-export type AnyXoJob = XoJob | AnyXoBackupJob
-
-export type AnyXoBackupJob = XoVmBackupJob | XoMetadataBackupJob | XoMirrorBackupJob
-
-export type AnyXoLog = XoBackupLog | XoRestoreLog

@vates/types/tsconfig.json

@@ -1,12 +0,0 @@
-{
-  "compilerOptions": {
-    "outDir": "./dist",
-    "declaration": true,
-    "strict": true,
-    "module": "ESNext",
-    "target": "ES2022",
-    "skipLibCheck": true
-  },
-  "include": ["src/**/*"],
-  "exclude": ["node_modules", "dist"]
-}

@vates/xml/cli.js

@@ -20,7 +20,7 @@ function main([inputPath = 0]) {
   try {
     data = JSON.parse(input)
   } catch (error) {
-    // fall back to XML
+    // fallback to XML
     log(parseXml(input))
     return
   }

@xen-orchestra/audit-core/index.js

@@ -98,17 +98,6 @@ class AuditCore {
     return record
   }
 
-  async _importRecord(record) {
-    if (!record.id || !record.previousId) {
-      throw new Error('Invalid record')
-    }
-    await this._storage.put(record)
-    return {
-      id: record.id,
-      isValid: record.id === createHash(record),
-    }
-  }
-
   async checkIntegrity(oldest, newest) {
     const storage = this._storage
 

@xen-orchestra/audit-core/package.json

@@ -7,7 +7,7 @@
     "type": "git",
     "url": "https://github.com/vatesfr/xen-orchestra.git"
   },
-  "version": "0.4.0",
+  "version": "0.3.1",
   "engines": {
     "node": ">=14"
   },

@xen-orchestra/backups-cli/.USAGE.md

@@ -4,10 +4,6 @@ Usage:
 
     xo-backups clean-vms [--merge] [--remove] xo-vm-backups/*
 
-    alternatively you can target the full remote, thus handling S3/Azure or encrypted remotes
-
-    xo-backups clean-vms [--merge] [--remove] --remote=REMOTEURL
-
       Detects and repair issues with VM backups.
 
       Options:

@xen-orchestra/backups-cli/README.md

@@ -20,10 +20,6 @@ Usage:
 
     xo-backups clean-vms [--merge] [--remove] xo-vm-backups/*
 
-    alternatively you can target the full remote, thus handling S3/Azure or encrypted remotes
-
-    xo-backups clean-vms [--merge] [--remove] --remote=REMOTEURL
-
       Detects and repair issues with VM backups.
 
       Options:

@xen-orchestra/backups-cli/commands/clean-vms.mjs

@@ -1,81 +1,13 @@
-import { asyncEach } from '@vates/async-each'
+import { asyncMap } from '@xen-orchestra/async-map'
 import { RemoteAdapter } from '@xen-orchestra/backups/RemoteAdapter.mjs'
 import { getSyncedHandler } from '@xen-orchestra/fs'
 import getopts from 'getopts'
-import { basename, dirname, join } from 'path'
+import { basename, dirname } from 'path'
 import Disposable from 'promise-toolbox/Disposable'
 import { pathToFileURL } from 'url'
 
-async function cleanFiles(vmDirs, { fix, merge, remove }) {
-  let nbSucceed = 0
-  let nbFailed = 0
-  let nbStarted = 0
-  const total = vmDirs.length
-  // don't assume all the files uses the same remote
-  await asyncEach(
-    vmDirs,
-    vmDir =>
-      Disposable.use(getSyncedHandler({ url: pathToFileURL(dirname(vmDir)).href }), async handler => {
-        try {
-          nbStarted++
-          await new RemoteAdapter(handler).cleanVm(basename(vmDir), {
-            fixMetadata: fix,
-            remove,
-            merge,
-            logInfo: (...args) => console.log(...args),
-            logWarn: (...args) => console.warn(...args),
-          })
-          nbSucceed++
-        } catch (error) {
-          nbFailed++
-          console.error('adapter.cleanVm', vmDir, error, { nbStarted, nbSucceed, nbFailed, total })
-        }
-      }),
-    { concurrency: 2 }
-  )
-  console.log('done', { nbSucceed, nbFailed })
-}
-
-async function cleanRemote(url, { fix, merge, remove }) {
-  let nbSucceed = 0
-  let nbFailed = 0
-  let nbStarted = 0
-  await Disposable.use(getSyncedHandler({ url }), async handler => {
-    const remoteAdapter = new RemoteAdapter(handler)
-    const vmUuids = await remoteAdapter.listAllVms()
-    const total = vmUuids.length
-    await asyncEach(
-      vmUuids,
-      async vmUuid => {
-        try {
-          nbStarted++
-          await remoteAdapter.cleanVm(join('xo-vm-backups', vmUuid), {
-            fixMetadata: fix,
-            remove,
-            merge,
-            logInfo: (...args) => console.log(...args),
-            logWarn: (...args) => console.warn(...args),
-          })
-          nbSucceed++
-        } catch (error) {
-          nbFailed++
-          console.error('adapter.cleanVm', vmUuid, error, { nbStarted, nbSucceed, nbFailed, total })
-        }
-      },
-      { concurrency: 2 }
-    )
-  })
-  console.log('done', { nbSucceed, nbFailed })
-}
-
 export default async function cleanVms(args) {
-  const {
-    _: vmDirs,
-    fix,
-    remote,
-    remove,
-    merge,
-  } = getopts(args, {
+  const { _, fix, remove, merge } = getopts(args, {
     alias: {
       fix: 'f',
       remove: 'r',
@@ -88,9 +20,19 @@ export default async function cleanVms(args) {
     },
   })
 
-  if (remote) {
-    return cleanRemote(remote, { fix, merge, remove })
-  } else {
-    return cleanFiles(vmDirs, { fix, merge, remove })
-  }
+  await asyncMap(_, vmDir =>
+    Disposable.use(getSyncedHandler({ url: pathToFileURL(dirname(vmDir)).href }), async handler => {
+      try {
+        await new RemoteAdapter(handler).cleanVm(basename(vmDir), {
+          fixMetadata: fix,
+          remove,
+          merge,
+          logInfo: (...args) => console.log(...args),
+          logWarn: (...args) => console.warn(...args),
+        })
+      } catch (error) {
+        console.error('adapter.cleanVm', vmDir, error)
+      }
+    })
+  )
 }

@xen-orchestra/backups-cli/package.json

@@ -6,10 +6,9 @@
   "preferGlobal": true,
   "bugs": "https://github.com/vatesfr/xen-orchestra/issues",
   "dependencies": {
-    "@vates/async-each": "^1.0.0",
     "@xen-orchestra/async-map": "^0.1.2",
-    "@xen-orchestra/backups": "^0.64.3",
-    "@xen-orchestra/fs": "^4.6.3",
+    "@xen-orchestra/backups": "^0.57.0",
+    "@xen-orchestra/fs": "^4.3.0",
     "filenamify": "^6.0.0",
     "getopts": "^2.2.5",
     "lodash": "^4.17.15",
@@ -28,7 +27,7 @@
   "scripts": {
     "postversion": "npm publish --access public"
   },
-  "version": "1.1.3",
+  "version": "1.0.28",
   "license": "AGPL-3.0-or-later",
   "author": {
     "name": "Vates SAS",

@xen-orchestra/backups/ImportVmBackup.mjs

@@ -4,13 +4,12 @@ import { formatFilenameDate } from './_filenameDate.mjs'
 import { importIncrementalVm } from './_incrementalVm.mjs'
 import { Task } from './Task.mjs'
 import { watchStreamSize } from './_watchStreamSize.mjs'
+import { VhdNegative, VhdSynthetic } from 'vhd-lib'
 import { decorateClass } from '@vates/decorate-with'
 import { createLogger } from '@xen-orchestra/log'
 import { dirname, join } from 'node:path'
 import pickBy from 'lodash/pickBy.js'
 import { defer } from 'golike-defer'
-import { NegativeDisk } from '@xen-orchestra/disk-transform'
-import { openDiskChain } from './disks/openDiskChain.mjs'
 
 const { debug, info, warn } = createLogger('xo:backups:importVmBackup')
 async function resolveUuid(xapi, cache, uuid, type) {
@@ -60,7 +59,7 @@ export class ImportVmBackup {
     const metadata = this._metadata
     const { mapVdisSrs } = this._importIncrementalVmSettings
     const { vbds, vhds, vifs, vm, vmSnapshot, vtpms } = metadata
-    const disks = {}
+    const streams = {}
     const metdataDir = dirname(metadata._filename)
     const vdis = ignoredVdis === undefined ? metadata.vdis : pickBy(metadata.vdis, vdi => !ignoredVdis.has(vdi.uuid))
 
@@ -111,21 +110,20 @@ export class ImportVmBackup {
         }
       }
 
-      let disk
+      let stream
       const backupWithSnapshotPath = join(metdataDir, backupCandidate ?? '')
       if (vhdPath === backupWithSnapshotPath) {
         // all the data are already on the host
         debug('direct reuse of a snapshot')
-        disk = null
+        stream = null
         vdis[vdiRef].baseVdi = snapshotCandidate
         // go next disk , we won't use this stream
         continue
       }
 
-      const parent = await openDiskChain({
-        handler: this._adapter._handler,
-        path: vhdPath,
-      })
+      let disposableDescendants
+
+      const disposableSynthetic = await VhdSynthetic.fromVhdChain(this._adapter._handler, vhdPath)
 
       // this will also clean if another disk of this VM backup fails
       // if user really only need to restore non failing disks he can retry with ignoredVdis
@@ -134,7 +132,8 @@ export class ImportVmBackup {
         if (!disposed) {
           disposed = true
           try {
-            await parent?.close()
+            await disposableDescendants?.dispose()
+            await disposableSynthetic?.dispose()
           } catch (error) {
             warn('openVhd: failed to dispose VHDs', { error })
           }
@@ -142,10 +141,11 @@ export class ImportVmBackup {
       }
       $defer.onFailure(() => disposeOnce())
 
-      debug('got vhd synthetic of parents', parent)
+      const parentVhd = disposableSynthetic.value
+      await parentVhd.readBlockAllocationTable()
+      debug('got vhd synthetic of parents', parentVhd.length)
 
       if (snapshotCandidate !== undefined) {
-        let descendant, negativeDisk
         try {
           debug('will try to use differential restore', {
             backupWithSnapshotPath,
@@ -153,37 +153,39 @@ export class ImportVmBackup {
             vdiRef,
           })
 
-          descendant = await openDiskChain({
-            handler: this._adapter._handler,
-            path: backupWithSnapshotPath,
+          disposableDescendants = await VhdSynthetic.fromVhdChain(this._adapter._handler, backupWithSnapshotPath, {
             until: vhdPath,
           })
-
+          const descendantsVhd = disposableDescendants.value
+          await descendantsVhd.readBlockAllocationTable()
           debug('got vhd synthetic of descendants')
-          negativeDisk = new NegativeDisk(parent, descendant)
+          const negativeVhd = new VhdNegative(parentVhd, descendantsVhd)
           debug('got vhd negative')
 
           // update the stream with the negative vhd stream
-          disk = negativeDisk
+          stream = await negativeVhd.stream()
           vdis[vdiRef].baseVdi = snapshotCandidate
         } catch (error) {
           // can be a broken VHD chain, a vhd chain with a key backup, ....
-          // not an irrecuperable error, don't dispose parentVhd, and fall back to full restore
+          // not an irrecuperable error, don't dispose parentVhd, and fallback to full restore
           warn(`can't use differential restore`, { error })
-          descendant?.close()
-          negativeDisk?.close()
+          disposableDescendants?.dispose()
         }
       }
-      // didn't make a negative stream : fall back to classic stream
-      if (disk === undefined) {
+      // didn't make a negative stream : fallback to classic stream
+      if (stream === undefined) {
         debug('use legacy restore')
-        disk = parent
+        stream = await parentVhd.stream()
       }
-      info('everything is ready, will transfer', disk)
-      disks[vdiRef] = disk
+
+      stream.on('end', disposeOnce)
+      stream.on('close', disposeOnce)
+      stream.on('error', disposeOnce)
+      info('everything is ready, will transfer', stream.length)
+      streams[`${vdiRef}.vhd`] = stream
     }
     return {
-      disks,
+      streams,
       vbds,
       vdis,
       version: '1.0.0',
@@ -239,6 +241,7 @@ export class ImportVmBackup {
       assert.strictEqual(metadata.mode, 'delta')
 
       backup = await this.#decorateIncrementalVmMetadata()
+      Object.values(backup.streams).forEach(stream => watchStreamSize(stream, sizeContainer))
     }
 
     return Task.run(
@@ -254,14 +257,7 @@ export class ImportVmBackup {
           : await importIncrementalVm(backup, await xapi.getRecord('SR', srRef), {
               newMacAddresses,
             })
-        const remoteName = adapter._handler._remote.name
-        let desc = `Restored on ${formatFilenameDate(+new Date())}`
-        if (remoteName !== undefined) {
-          desc += ` from ${remoteName}`
-        }
-        if (metadata.vm.name_description) {
-          desc += ` - ${metadata.vm.name_description}`
-        }
+
         await Promise.all([
           xapi.call('VM.add_tags', vmRef, 'restored from backup'),
           xapi.call(
@@ -269,7 +265,13 @@ export class ImportVmBackup {
             vmRef,
             `${metadata.vm.name_label} (${formatFilenameDate(metadata.timestamp)})`
           ),
-          xapi.call('VM.set_name_description', vmRef, desc),
+          xapi.call(
+            'VM.set_name_description',
+            vmRef,
+            `Restored on ${formatFilenameDate(+new Date())} from ${adapter._handler._remote.name} -
+             ${metadata.vm.name_description}
+            `
+          ),
         ])
 
         return {

@xen-orchestra/backups/RemoteAdapter.mjs

@@ -2,7 +2,7 @@ import { asyncEach } from '@vates/async-each'
 import { asyncMap, asyncMapSettled } from '@xen-orchestra/async-map'
 import { compose } from '@vates/compose'
 import { createLogger } from '@xen-orchestra/log'
-import { VhdDirectory, VhdSynthetic } from 'vhd-lib'
+import { createVhdDirectoryFromStream, openVhd, VhdAbstract, VhdDirectory, VhdSynthetic } from 'vhd-lib'
 import { decorateMethodsWith } from '@vates/decorate-with'
 import { deduped } from '@vates/disposable/deduped.js'
 import { dirname, join, resolve } from 'node:path'
@@ -10,6 +10,7 @@ import { execFile } from 'child_process'
 import { mount } from '@vates/fuse-vhd'
 import { readdir, lstat } from 'node:fs/promises'
 import { synchronized } from 'decorator-synchronized'
+import { v4 as uuidv4 } from 'uuid'
 import { ZipFile } from 'yazl'
 import Disposable from 'promise-toolbox/Disposable'
 import fromCallback from 'promise-toolbox/fromCallback'
@@ -31,11 +32,6 @@ import { listPartitions, LVM_PARTITION_TYPE } from './_listPartitions.mjs'
 import { lvs, pvs } from './_lvm.mjs'
 import { watchStreamSize } from './_watchStreamSize.mjs'
 
-import { RemoteVhd } from './disks/RemoteVhd.mjs'
-import { openDiskChain } from './disks/openDiskChain.mjs'
-import { toVhdStream, writeToVhdDirectory } from 'vhd-lib/disk-consumer/index.mjs'
-import { ReadAhead } from '@xen-orchestra/disk-transform'
-
 export const DIR_XO_CONFIG_BACKUPS = 'xo-config-backups'
 
 export const DIR_XO_POOL_METADATA_BACKUPS = 'xo-pool-metadata-backups'
@@ -205,7 +201,7 @@ export class RemoteAdapter {
     })
   }
 
-  // check if we will be allowed to merge a vhd created in this adapter
+  // check if we will be allowed to merge a a vhd created in this adapter
   // with the vhd at path `path`
   async isMergeableParent(packedParentUid, path) {
     return await Disposable.use(VhdSynthetic.fromVhdChain(this.handler, path), vhd => {
@@ -427,20 +423,12 @@ export class RemoteAdapter {
   async listAllVms() {
     const handler = this._handler
     const vmsUuids = []
-    try {
-      await asyncEach(await handler.list(BACKUP_DIR), async entry => {
-        // ignore hidden and lock files
-        if (entry[0] !== '.' && !entry.endsWith('.lock')) {
-          vmsUuids.push(entry)
-        }
-      })
-    } catch (error) {
-      // remote without any VM backup are ok
-      if (error.code !== 'ENOENT') {
-        throw error
+    await asyncEach(await handler.list(BACKUP_DIR), async entry => {
+      // ignore hidden and lock files
+      if (entry[0] !== '.' && !entry.endsWith('.lock')) {
+        vmsUuids.push(entry)
       }
-    }
-
+    })
     return vmsUuids
   }
 
@@ -693,32 +681,26 @@ export class RemoteAdapter {
     return path
   }
 
-  async writeVhd(path, disk, { validator = noop, writeBlockConcurrency } = {}) {
+  async writeVhd(path, input, { checksum = true, validator = noop, writeBlockConcurrency } = {}) {
     const handler = this._handler
-
     if (this.useVhdDirectory()) {
-      await writeToVhdDirectory({
-        disk,
-        target: {
-          handler,
-          path,
-          concurrency: writeBlockConcurrency,
-          validator,
-          compression: 'brotli',
+      const dataPath = `${dirname(path)}/data/${uuidv4()}.vhd`
+      const size = await createVhdDirectoryFromStream(handler, dataPath, input, {
+        concurrency: writeBlockConcurrency,
+        compression: this.#getCompressionType(),
+        async validator() {
+          await input.task
+          return validator.apply(this, arguments)
         },
       })
+      await VhdAbstract.createAlias(handler, path, dataPath)
+      return size
     } else {
-      const stream = await toVhdStream(disk)
-      await this.outputStream(path, stream, { validator })
-      await validator(path)
+      return this.outputStream(path, input, { checksum, validator })
     }
   }
 
-  async outputStream(
-    path,
-    input,
-    { checksum = !this._handler.isEncrypted, maxStreamLength, streamLength, validator = noop } = {}
-  ) {
+  async outputStream(path, input, { checksum = true, maxStreamLength, streamLength, validator = noop } = {}) {
     const container = watchStreamSize(input)
     await this._handler.outputStream(path, input, {
       checksum,
@@ -734,16 +716,30 @@ export class RemoteAdapter {
   }
 
   // open the  hierarchy of ancestors until we find a full one
-  async _createVhdDisk(handler, path, { useChain }) {
-    let disk
-    if (useChain) {
-      disk = await openDiskChain({ handler, path })
-    } else {
-      disk = new RemoteVhd({ handler, path })
-      await disk.init()
+  async _createVhdStream(handler, path, { useChain }) {
+    const disposableSynthetic = useChain ? await VhdSynthetic.fromVhdChain(handler, path) : await openVhd(handler, path)
+    // I don't want the vhds to be disposed on return
+    // but only when the stream is done ( or failed )
+
+    let disposed = false
+    const disposeOnce = async () => {
+      if (!disposed) {
+        disposed = true
+        try {
+          await disposableSynthetic.dispose()
+        } catch (error) {
+          warn('openVhd: failed to dispose VHDs', { error })
+        }
+      }
     }
-    disk = new ReadAhead(disk)
-    return disk
+    const synthetic = disposableSynthetic.value
+    await synthetic.readBlockAllocationTable()
+    const stream = await synthetic.stream()
+
+    stream.on('end', disposeOnce)
+    stream.on('close', disposeOnce)
+    stream.on('error', disposeOnce)
+    return stream
   }
 
   async readIncrementalVmBackup(metadata, ignoredVdis, { useChain = true } = {}) {
@@ -751,14 +747,14 @@ export class RemoteAdapter {
     const { vbds, vhds, vifs, vm, vmSnapshot, vtpms } = metadata
     const dir = dirname(metadata._filename)
     const vdis = ignoredVdis === undefined ? metadata.vdis : pickBy(metadata.vdis, vdi => !ignoredVdis.has(vdi.uuid))
-    const disks = {}
+
+    const streams = {}
     await asyncMapSettled(Object.keys(vdis), async ref => {
-      delete vdis[ref].baseVdi
-      disks[ref] = await this._createVhdDisk(handler, join(dir, vhds[ref]), { useChain })
+      streams[`${ref}.vhd`] = await this._createVhdStream(handler, join(dir, vhds[ref]), { useChain })
     })
 
     return {
-      disks,
+      streams,
       vbds,
       vdis,
       version: '1.0.0',
@@ -776,7 +772,7 @@ export class RemoteAdapter {
     let json
     let isImmutable = false
     let remoteIsImmutable = false
-    // if the remote is immutable, check if this metadata is also immutable
+    // if the remote is immutable, check if this metadatas are also immutables
     try {
       // this file is not encrypted
       await this._handler._readFile(IMMUTABILTY_METADATA_FILENAME)
@@ -792,7 +788,7 @@ export class RemoteAdapter {
       json = await this.handler.readFile(path, { flag: 'r+' })
       // s3 handler don't respect flags
     } catch (err) {
-      // retry without triggering immutability check ,only on immutable remote
+      // retry without triggerring immutbaility check ,only on immutable remote
       if (err.code === 'EPERM' && remoteIsImmutable) {
         isImmutable = true
         json = await this._handler.readFile(path, { flag: 'r' })

@xen-orchestra/backups/RestoreMetadataBackup.mjs

@@ -24,7 +24,7 @@ export class RestoreMetadataBackup {
       const dataFileName = resolve('/', backupId, metadata.data ?? 'data.json').slice(1)
       const data = await handler.readFile(dataFileName)
 
-      // if data is JSON, sent it as a plain string; otherwise, consider the data as binary and encode it
+      // if data is JSON, sent it as a plain string, otherwise, consider the data as binary and encode it
       const isJson = dataFileName.endsWith('.json')
       return isJson ? data.toString() : { encoding: 'base64', data: data.toString('base64') }
     }

@xen-orchestra/backups/_cleanVm.integ.mjs

@@ -132,7 +132,7 @@ test('it remove vhd with missing or multiple ancestors', async () => {
   const deletedOrphanVhd = loggued.match(/deleting orphan VHD/g) || []
   assert.equal(deletedOrphanVhd.length, 1) // only one vhd should have been deleted
 
-  // we don't test the file on disk, since they will all be marker as unused and deleted without a metadata.json file
+  // we don't test the filew on disk, since they will all be marker as unused and deleted without a metadata.json file
 })
 
 test('it remove backup meta data referencing a missing vhd in delta backup', async () => {
@@ -287,8 +287,9 @@ test('it finish unterminated merge ', async () => {
 })
 
 // each of the vhd can be a file, a directory, an alias to a file or an alias to a directory
-// the message and resulting files should be identical to the output with vhd files which is tested independently
-describe('tests multiple combination ', { concurrency: 1 }, () => {
+// the message an resulting files should be identical to the output with vhd files which is tested independantly
+
+describe('tests multiple combination ', () => {
   for (const useAlias of [true, false]) {
     for (const vhdMode of ['file', 'directory']) {
       test(`alias : ${useAlias}, mode: ${vhdMode}`, async () => {
@@ -308,9 +309,9 @@ describe('tests multiple combination ', { concurrency: 1 }, () => {
           await VhdAbstract.createAlias(handler, 'broken.alias.vhd', brokenVhdDataPath)
         }
 
-        // a vhd non referenced in metadata
+        // a vhd non referenced in metada
         await generateVhd(`${basePath}/nonreference.vhd`, { useAlias, mode: vhdMode })
-        // an abandoned delta vhd without its parent
+        // an abandonded delta vhd without its parent
         await generateVhd(`${basePath}/abandonned.vhd`, {
           useAlias,
           mode: vhdMode,
@@ -404,7 +405,7 @@ describe('tests multiple combination ', { concurrency: 1 }, () => {
         // size should be the size of children + grand children + clean after the merge
         assert.deepEqual(metadata.size, vhdMode === 'file' ? 6502400 : 6501888)
 
-        // broken vhd, non referenced, abandoned should be deleted ( alias and data)
+        // broken vhd, non referenced, abandonned should be deleted ( alias and data)
         // ancestor and child should be merged
         // grand child and clean vhd should not have changed
         const survivors = await handler.list(basePath)
@@ -430,6 +431,7 @@ describe('tests multiple combination ', { concurrency: 1 }, () => {
     }
   }
 })
+
 test('it cleans orphan merge states ', async () => {
   await handler.writeFile(`${basePath}/.orphan.vhd.merge.json`, '')
 
@@ -454,7 +456,7 @@ test('check Aliases should work alone', async () => {
     logWarn: () => {},
   })
 
-  // only ok have survived
+  // only ok have suvived
   const alias = (await handler.list('vhds')).filter(f => f.endsWith('.vhd'))
   assert.equal(alias.length, 1)
 

@xen-orchestra/backups/_cleanVm.mjs

@@ -129,29 +129,10 @@ export async function checkAliases(
         logWarn('missing target of alias', { alias })
         if (remove) {
           logInfo('removing alias and non VHD target', { alias, target })
+          await handler.unlink(target)
           await handler.unlink(alias)
         }
-        continue
       }
-      if (err.code === 'EISDIR') {
-        logWarn('alias is a vhd directory', { alias })
-        if (remove) {
-          logInfo('removing vhd directory named as alias', { alias, target })
-          await VhdAbstract.unlink(handler, alias)
-        }
-        continue
-      }
-      logWarn('unhandled error while checking alias', { alias, err })
-      continue
-    }
-
-    if (target === '') {
-      logWarn('empty target for alias ', { alias })
-      if (remove) {
-        logInfo('removing alias and non VHD target', { alias, target })
-        await handler.unlink(alias)
-      }
-      continue
     }
 
     if (!isVhdFile(target)) {
@@ -512,7 +493,7 @@ export async function cleanVm(
         remove,
         mergeBlockConcurrency,
       })
-      const metadataPath = vhdsToJSons[chain[chain.length - 1]] // all the chain should have the same metadata file
+      const metadataPath = vhdsToJSons[chain[chain.length - 1]] // all the chain should have the same metada file
       metadataWithMergedVhd[metadataPath] = (metadataWithMergedVhd[metadataPath] ?? 0) + finalVhdSize
     })
   }
@@ -594,8 +575,8 @@ export async function cleanVm(
       if (mergedSize) {
         // all disks are now key disk
         metadata.isVhdDifferencing = {}
-        for (const id of Object.keys(metadata.vdis ?? {})) {
-          metadata.isVhdDifferencing[id] = false
+        for (const id of Object.values(metadata.vdis ?? {})) {
+          metadata.isVhdDifferencing[`${id}.vhd`] = false
         }
       }
       mustRegenerateCache = true

@xen-orchestra/backups/_getOldEntries.mjs

@@ -26,8 +26,8 @@ const LTR_DEFINITIONS = {
 
         copy.date(date.date() - firstDayOfWeek)
         // warning, the year in term of week may different from YYYY
-        // since the computation of the first week of a year is timezone dependent
-        return copy.format('gggg-ww')
+        // since the computation of the first week of a year is timezone dependant
+        return copy.format('gggg-WW')
       }
     },
     ancestor: 'daily',
@@ -59,7 +59,7 @@ const LTR_DEFINITIONS = {
  * return the entries too old to be kept
  *  if multiple entries are i the same time bucket : keep only the most recent one
  *  if an entry is valid in any of the bucket OR the  minRetentionCount : keep it
- *  if a bucket is completely empty : it does not count as one, thus it may extend the retention
+ *  if a bucket is completly empty : it does not count as one, thus it may extend the retention
  * @returns Array<Backup>
  */
 export function getOldEntries(minRetentionCount, entries, { longTermRetention = {}, timezone } = {}) {

@xen-orchestra/backups/_incrementalVm.mjs

@@ -10,11 +10,6 @@ import { Task } from './Task.mjs'
 import pick from 'lodash/pick.js'
 import { BASE_DELTA_VDI, COPY_OF, VM_UUID } from './_otherConfig.mjs'
 
-import { XapiDiskSource } from '@xen-orchestra/xapi'
-import { toVhdStream } from 'vhd-lib/disk-consumer/index.mjs'
-import { toQcow2Stream } from '@xen-orchestra/qcow2'
-import { VHD_MAX_SIZE } from '@xen-orchestra/xapi/disks/Xapi.mjs'
-
 const ensureArray = value => (value === undefined ? [] : Array.isArray(value) ? value : [value])
 
 export async function exportIncrementalVm(
@@ -24,7 +19,7 @@ export async function exportIncrementalVm(
 ) {
   // refs of VM's VDIs → base's VDIs.
 
-  const disks = {}
+  const streams = {}
   const vdis = {}
   const vbds = {}
   await cancelableMap(cancelToken, vm.$VBDs, async (cancelToken, vbd) => {
@@ -57,14 +52,32 @@ export async function exportIncrementalVm(
       $snapshot_of$uuid: vdi.$snapshot_of?.uuid,
       $SR$uuid: vdi.$SR.uuid,
     }
-    disks[vdiRef] = new XapiDiskSource({
-      vdiRef,
-      xapi: vm.$xapi,
-      baseRef: baseVdi?.$ref,
-      nbdConcurrency,
-      preferNbd,
-    })
-    await disks[vdiRef].init()
+    try {
+      streams[`${vdiRef}.vhd`] = await vdi.$exportContent({
+        baseRef: baseVdi?.$ref,
+        cancelToken,
+        format: 'vhd',
+        nbdConcurrency,
+        preferNbd,
+      })
+    } catch (err) {
+      if (err.code === 'VDI_CANT_DO_DELTA') {
+        // fall back to a base
+        Task.info(`Can't do delta, will try to get a full stream`, { vdi })
+        streams[`${vdiRef}.vhd`] = await vdi.$exportContent({
+          cancelToken,
+          format: 'vhd',
+          nbdConcurrency,
+          preferNbd,
+        })
+        // only warn if the fall back succeed
+        Task.warning(`Can't do delta with this vdi, transfer will be a full`, {
+          vdi,
+        })
+      } else {
+        throw err
+      }
+    }
   })
 
   const suspendVdi = vm.$suspend_VDI
@@ -74,13 +87,10 @@ export async function exportIncrementalVm(
       ...suspendVdi,
       $SR$uuid: suspendVdi.$SR.uuid,
     }
-    disks[vdiRef] = new XapiDiskSource({
-      vdiRef: suspendVdi.$ref,
-      xapi: vm.$xapi,
-      nbdConcurrency,
-      preferNbd,
+    streams[`${vdiRef}.vhd`] = await suspendVdi.$exportContent({
+      cancelToken,
+      format: 'vhd',
     })
-    await disks[vdiRef].init()
   }
 
   const vifs = {}
@@ -94,13 +104,11 @@ export async function exportIncrementalVm(
     }
   })
 
-  // Get a fresh list of VM's VTPM to avoid `vm.VTPMs: [undefined]`
-  const vmVtpms = await vm.$xapi.getField('VM', vm.$ref, 'VTPMs')
   const vtpms = await Promise.all(
-    vmVtpms.map(async vtpmRef => {
+    vm.$VTPMs.map(async vtpm => {
       let content
       try {
-        content = await vm.$xapi.call('VTPM.get_contents', vtpmRef)
+        content = await vm.$xapi.call('VTPM.get_contents', vtpm.$ref)
       } catch (err) {
         console.error(err)
       }
@@ -108,17 +116,24 @@ export async function exportIncrementalVm(
     })
   )
 
-  return {
-    version: '1.1.0',
-    vbds,
-    vdis,
-    vifs,
-    vm: {
-      ...vm,
+  return Object.defineProperty(
+    {
+      version: '1.1.0',
+      vbds,
+      vdis,
+      vifs,
+      vm: {
+        ...vm,
+      },
+      vtpms,
     },
-    vtpms,
-    disks,
-  }
+    'streams',
+    {
+      configurable: true,
+      value: streams,
+      writable: true,
+    }
+  )
 }
 
 export const importIncrementalVm = defer(async function importIncrementalVm(
@@ -184,8 +199,8 @@ export const importIncrementalVm = defer(async function importIncrementalVm(
     const vdi = vdiRecords[vdiRef]
     let newVdi
 
-    if (vdi.baseVdi?.$ref !== undefined) {
-      newVdi = await xapi.getRecord('VDI', await xapi.VDI_clone(vdi.baseVdi.$ref))
+    if (vdi.baseVdi !== undefined) {
+      newVdi = await xapi.getRecord('VDI', await vdi.baseVdi.$clone())
       $defer.onFailure(() => newVdi.$destroy())
 
       await newVdi.update_other_config(COPY_OF, vdi.uuid)
@@ -229,28 +244,21 @@ export const importIncrementalVm = defer(async function importIncrementalVm(
     }
   })
 
-  const { disks } = incrementalVm
+  const { streams } = incrementalVm
+
   await Promise.all([
     // Import VDI contents.
     cancelableMap(cancelToken, Object.entries(newVdis), async (cancelToken, [id, vdi]) => {
-      for (const disk of ensureArray(disks[id])) {
-        if (disk === null) {
-          // we restore a backup and reuse completely a local snapshot
+      for (let stream of ensureArray(streams[`${id}.vhd`])) {
+        if (stream === null) {
+          // we restore a backup and reuse completly a local snapshot
           continue
         }
-        await xapi.setField('VDI', vdi.$ref, 'name_label', `[Importing] ${vdiRecords[id].name_label}`)
-
-        let stream, format
-        if (vdi.virtual_size > VHD_MAX_SIZE) {
-          stream = await toQcow2Stream(disk)
-          format = 'qcow2'
-        } else {
-          stream = await toVhdStream(disk)
-          format = 'vhd'
+        if (typeof stream === 'function') {
+          stream = await stream()
         }
-
-        await vdi.$importContent(stream, { cancelToken, format })
-
+        await xapi.setField('VDI', vdi.$ref, 'name_label', `[Importing] ${vdiRecords[id].name_label}`)
+        await vdi.$importContent(stream, { cancelToken, format: 'vhd' })
         await xapi.setField('VDI', vdi.$ref, 'name_label', vdiRecords[id].name_label)
       }
     }),

@xen-orchestra/backups/_isValidXva.mjs

@@ -44,7 +44,7 @@ const isValidTar = async (handler, size, fd) => {
   return buf.every(_ => _ === 0)
 }
 
-// TODO: find a heuristic for compressed files
+// TODO: find an heuristic for compressed files
 export async function isValidXva(path) {
   const handler = this._handler
 

@xen-orchestra/backups/_otherConfig.mjs

@@ -43,7 +43,7 @@ async function getDeltaChainLength(xapi, type, ref) {
 }
 
 /**
- * set the delta chain length ( number of delta since last base backup) to a VM and its associated VDIs
+ * set the delta chain lenght ( number of delta since last base backup) to a VM and its associated VDIs
  *
  * @param {Xapi} xapi
  * @param {String} vmRef
@@ -58,7 +58,7 @@ export async function setVmDeltaChainLength(xapi, vmRef, length) {
 
 /**
  * Compute the delta chain length of a VM and its associated VDIs
- * if there is a discrepancy, use, the highest value
+ * if there is a discrependcy, use, the highest value
  * @param {Xapi} xapi
  * @param {String} vmRef
  * @returns {Promise}
@@ -92,7 +92,7 @@ export function resetVmOtherConfig(xapi, vmRef) {
 
 /**
  *
- * used to ensure compatibility with the previous snapshots that were having the config stored only into VM
+ * used to ensure compatibiliy with the previous snapshots that were having the config stored only into VM
  *
  * @param {Xapi} xapi
  * @param {String} vmRef
@@ -152,7 +152,7 @@ export async function setVmOtherConfig(xapi, vmRef, { timestamp, jobId, schedule
 }
 /**
  *
- * mark the export of he VM and its VDIs as successful
+ * mark the export of he VM and its VDIs as successfull
  *
  * @param {Xapi} xapi
  * @param {String} vmRef

@xen-orchestra/backups/_runners/VmsRemote.mjs

@@ -4,11 +4,11 @@ import { limitConcurrency } from 'limit-concurrency-decorator'
 
 import { extractIdsFromSimplePattern } from '../extractIdsFromSimplePattern.mjs'
 import { Task } from '../Task.mjs'
+import createStreamThrottle from './_createStreamThrottle.mjs'
 import { DEFAULT_SETTINGS, Abstract } from './_Abstract.mjs'
 import { getAdaptersByRemote } from './_getAdaptersByRemote.mjs'
 import { FullRemote } from './_vmRunners/FullRemote.mjs'
 import { IncrementalRemote } from './_vmRunners/IncrementalRemote.mjs'
-import { Throttle } from '@vates/generator-toolbox'
 
 const noop = Function.prototype
 
@@ -41,7 +41,7 @@ export const VmsRemote = class RemoteVmsBackupRunner extends Abstract {
     const schedule = this._schedule
     const settings = this._settings
 
-    const throttleGenerator = new Throttle()
+    const throttleStream = createStreamThrottle(settings.maxExportRate)
 
     const config = this._config
 
@@ -89,7 +89,7 @@ export const VmsRemote = class RemoteVmsBackupRunner extends Abstract {
             schedule,
             settings: vmSettings,
             sourceRemoteAdapter,
-            throttleGenerator,
+            throttleStream,
             vmUuid,
           }
           let vmBackup

@xen-orchestra/backups/_runners/VmsXapi.mjs

@@ -4,12 +4,12 @@ import { limitConcurrency } from 'limit-concurrency-decorator'
 
 import { extractIdsFromSimplePattern } from '../extractIdsFromSimplePattern.mjs'
 import { Task } from '../Task.mjs'
+import createStreamThrottle from './_createStreamThrottle.mjs'
 import { DEFAULT_SETTINGS, Abstract } from './_Abstract.mjs'
 import { runTask } from './_runTask.mjs'
 import { getAdaptersByRemote } from './_getAdaptersByRemote.mjs'
 import { IncrementalXapi } from './_vmRunners/IncrementalXapi.mjs'
 import { FullXapi } from './_vmRunners/FullXapi.mjs'
-import { Throttle } from '@vates/generator-toolbox'
 
 const noop = Function.prototype
 
@@ -55,7 +55,7 @@ export const VmsXapi = class VmsXapiBackupRunner extends Abstract {
     const schedule = this._schedule
     const settings = this._settings
 
-    const throttleGenerator = new Throttle()
+    const throttleStream = createStreamThrottle(settings.maxExportRate)
 
     const config = this._config
 
@@ -147,7 +147,7 @@ export const VmsXapi = class VmsXapiBackupRunner extends Abstract {
                       schedule,
                       settings: vmSettings,
                       srs,
-                      throttleGenerator,
+                      throttleStream,
                       vm,
                     }
 

@xen-orchestra/backups/_runners/_vmRunners/FullRemote.mjs

@@ -24,9 +24,8 @@ export const FullRemote = class FullRemoteVmBackupRunner extends AbstractRemote
         writer =>
           writer.run({
             stream: forkStreamUnpipe(stream),
-            // stream will be forked and transformed, it's not safe to attach additional properties to it
+            // stream will be forked and transformed, it's not safe to attach additionnal properties to it
             streamLength: stream.length,
-            maxStreamLength: stream.maxStreamLength, // on encrypted source
             timestamp: metadata.timestamp,
             vm: metadata.vm,
             vmSnapshot: metadata.vmSnapshot,
@@ -37,6 +36,5 @@ export const FullRemote = class FullRemoteVmBackupRunner extends AbstractRemote
       // for healthcheck
       this._tags = metadata.vm.tags
     }
-    this._hasTransferredData = transferList.length > 0
   }
 }

@xen-orchestra/backups/_runners/_vmRunners/FullXapi.mjs

@@ -13,35 +13,15 @@ export const FullXapi = class FullXapiVmBackupRunner extends AbstractXapi {
     return [FullRemoteWriter, FullXapiWriter]
   }
 
-  async _mustDoSnapshot() {
+  _mustDoSnapshot() {
     const vm = this._vm
+
     const settings = this._settings
-
-    // General setting forcing snapshot at all times
-    if (settings.unconditionalSnapshot) {
-      return true
-    }
-
-    // If offline backup option is not checked
-    // and vm is running, we make a snapshot
-    if (!settings.offlineBackup && vm.power_state === 'Running') {
-      return true
-    }
-
-    // General setting used for snapshot retention
-    // see schedule in backupJob edition
-    if (settings.snapshotRetention !== 0) {
-      return true
-    }
-
-    const vdis = await this._xapi.getRecords('VDI', await vm.$getDisks())
-    const hasNOBAKtag = vdis.some(idx => idx.name_label.includes('[NOBAK]'))
-
-    // if there is a disk with NOBAK tag
-    // we need to make a snapshot
-    if (hasNOBAKtag) {
-      return true
-    }
+    return (
+      settings.unconditionalSnapshot ||
+      (!settings.offlineBackup && vm.power_state === 'Running') ||
+      settings.snapshotRetention !== 0
+    )
   }
   _selectBaseVm() {}
 
@@ -49,24 +29,22 @@ export const FullXapi = class FullXapiVmBackupRunner extends AbstractXapi {
     const { compression } = this.job
     const vm = this._vm
     const exportedVm = this._exportedVm
-    // @todo put back throttle for full backup/Replication
-    const stream =
-      /* this._throttleStream( */
+    const stream = this._throttleStream(
       (
         await this._xapi.VM_export(exportedVm.$ref, {
           compress: Boolean(compression) && (compression === 'native' ? 'gzip' : 'zstd'),
           useSnapshot: false,
         })
       ).body
-    /* ) */
+    )
 
     const vdis = await exportedVm.$getDisks()
     let maxStreamLength = 1024 * 1024 // Ovf file and tar headers are a few KB, let's stay safe
     for (const vdiRef of vdis) {
       const vdi = await this._xapi.getRecord('VDI', vdiRef)
 
-      // the size a of fully allocated vdi will be virtual_size exactly, it's a gross over evaluation
-      // of the real stream size in general, since a disk is never completely full
+      // the size a of fully allocated vdi will be virtual_size  exaclty, it's a gross over evaluation
+      // of the real stream size in general, since a disk is never completly full
       // vdi.physical_size seems to underevaluate a lot the real disk usage of a VDI, as of 2023-10-30
       maxStreamLength += vdi.virtual_size
     }

@xen-orchestra/backups/_runners/_vmRunners/IncrementalRemote.mjs

@@ -3,6 +3,8 @@ import { createLogger } from '@xen-orchestra/log'
 import { asyncEach } from '@vates/async-each'
 import assert from 'node:assert'
 import * as UUID from 'uuid'
+import isVhdDifferencingDisk from 'vhd-lib/isVhdDifferencingDisk.js'
+import mapValues from 'lodash/mapValues.js'
 
 import { AbstractRemote } from './_AbstractRemote.mjs'
 import { forkDeltaExport } from './_forkDeltaExport.mjs'
@@ -10,7 +12,6 @@ import { IncrementalRemoteWriter } from '../_writers/IncrementalRemoteWriter.mjs
 import { Disposable } from 'promise-toolbox'
 import { openVhd } from 'vhd-lib'
 import { getVmBackupDir } from '../../_getVmBackupDir.mjs'
-import { SynchronizedDisk } from '@xen-orchestra/disk-transform'
 
 const { warn } = createLogger('xo:backups:Incrementalremote')
 class IncrementalRemoteVmBackupRunner extends AbstractRemote {
@@ -18,7 +19,7 @@ class IncrementalRemoteVmBackupRunner extends AbstractRemote {
     return IncrementalRemoteWriter
   }
 
-  // we'll transfer the full list if at least one backup should be transferred
+  // we'll transfer the full list if at least one backup should be transfered
   // to ensure we don't cut the delta chain
   _filterTransferList(transferList) {
     if (transferList.some(vmBackupMetadata => this._filterPredicate(vmBackupMetadata))) {
@@ -36,7 +37,7 @@ class IncrementalRemoteVmBackupRunner extends AbstractRemote {
       return
     }
     await asyncEach(Object.entries(metadata.vdis), async ([id, vdi]) => {
-      const isDifferencing = metadata.isVhdDifferencing[id]
+      const isDifferencing = metadata.isVhdDifferencing[`${id}.vhd`]
       if (isDifferencing) {
         const vmDir = getVmBackupDir(metadata.vm.uuid)
         const path = `${vmDir}/${metadata.vhds[id]}`
@@ -70,12 +71,9 @@ class IncrementalRemoteVmBackupRunner extends AbstractRemote {
       // recompute if disks are differencing or not
       const isVhdDifferencing = {}
 
-      for (const key in incrementalExport.disks) {
-        const disk = incrementalExport.disks[key]
-        isVhdDifferencing[key] = disk.isDifferencing()
-        incrementalExport.disks[key] = new SynchronizedDisk(disk)
-      }
-
+      await asyncEach(Object.entries(incrementalExport.streams), async ([key, stream]) => {
+        isVhdDifferencing[key] = await isVhdDifferencingDisk(stream)
+      })
       const hasDifferencingDisk = Object.values(isVhdDifferencing).includes(true)
       if (metadata.isBase === hasDifferencingDisk) {
         warn(`Metadata isBase and real disk value are different`, {
@@ -89,10 +87,11 @@ class IncrementalRemoteVmBackupRunner extends AbstractRemote {
       await this._selectBaseVm(metadata)
       await this._callWriters(writer => writer.prepare({ isBase: metadata.isBase }), 'writer.prepare()')
 
+      incrementalExport.streams = mapValues(incrementalExport.streams, this._throttleStream)
       await this._callWriters(
         writer =>
           writer.transfer({
-            deltaExport: forkDeltaExport(incrementalExport, writer.constructor.name),
+            deltaExport: forkDeltaExport(incrementalExport),
             isVhdDifferencing,
             timestamp: metadata.timestamp,
             vm: metadata.vm,
@@ -115,7 +114,6 @@ class IncrementalRemoteVmBackupRunner extends AbstractRemote {
       // for healthcheck
       this._tags = metadata.vm.tags
     }
-    this._hasTransferredData = transferList.length > 0
   }
 }
 

@xen-orchestra/backups/_runners/_vmRunners/IncrementalXapi.mjs

@@ -1,12 +1,18 @@
+import { asyncEach } from '@vates/async-each'
 import { createLogger } from '@xen-orchestra/log'
+import { pipeline } from 'node:stream'
+import isVhdDifferencingDisk from 'vhd-lib/isVhdDifferencingDisk.js'
 import keyBy from 'lodash/keyBy.js'
+import mapValues from 'lodash/mapValues.js'
+import vhdStreamValidator from 'vhd-lib/vhdStreamValidator.js'
 
 import { AbstractXapi } from './_AbstractXapi.mjs'
-import { forkDeltaExport } from './_forkDeltaExport.mjs'
 import { exportIncrementalVm } from '../../_incrementalVm.mjs'
+import { forkDeltaExport } from './_forkDeltaExport.mjs'
 import { IncrementalRemoteWriter } from '../_writers/IncrementalRemoteWriter.mjs'
 import { IncrementalXapiWriter } from '../_writers/IncrementalXapiWriter.mjs'
 import { Task } from '../../Task.mjs'
+import { watchStreamSize } from '../../_watchStreamSize.mjs'
 import {
   DATETIME,
   DELTA_CHAIN_LENGTH,
@@ -14,16 +20,16 @@ import {
   setVmDeltaChainLength,
   markExportSuccessfull,
 } from '../../_otherConfig.mjs'
-import { SynchronizedDisk } from '@xen-orchestra/disk-transform'
 
 const { debug } = createLogger('xo:backups:IncrementalXapiVmBackup')
+const noop = Function.prototype
 
 export const IncrementalXapi = class IncrementalXapiVmBackupRunner extends AbstractXapi {
   _getWriters() {
     return [IncrementalRemoteWriter, IncrementalXapiWriter]
   }
 
-  async _mustDoSnapshot() {
+  _mustDoSnapshot() {
     return true
   }
 
@@ -39,36 +45,42 @@ export const IncrementalXapi = class IncrementalXapiVmBackupRunner extends Abstr
       nbdConcurrency: this._settings.nbdConcurrency,
       preferNbd: this._settings.preferNbd,
     })
-
-    const isVhdDifferencing = {}
-    let useNbd = false
-    for (const key in deltaExport.disks) {
-      const disk = deltaExport.disks[key]
-      isVhdDifferencing[key] = disk.isDifferencing()
-      if (!isFull && !isVhdDifferencing[key] && key !== exportedVm.$suspend_VDI?.$ref) {
-        Task.warning('Backup fell back to a full')
-      }
-      deltaExport.disks[key] = new SynchronizedDisk(disk)
-      useNbd = useNbd || disk.useNbd()
-    }
-    if (useNbd) {
+    // since NBD is network based, if one disk use nbd , all the disk use them
+    // except the suspended VDI
+    if (Object.values(deltaExport.streams).some(({ _nbd }) => _nbd)) {
       Task.info('Transfer data using NBD')
     }
 
-    // @todo : reimplement throttle
+    const isVhdDifferencing = {}
+    // since isVhdDifferencingDisk is reading and unshifting data in stream
+    // it should be done BEFORE any other stream transform
+    await asyncEach(Object.entries(deltaExport.streams), async ([key, stream]) => {
+      isVhdDifferencing[key] = await isVhdDifferencingDisk(stream)
+    })
+    const sizeContainers = mapValues(deltaExport.streams, stream => watchStreamSize(stream))
+
+    if (this._settings.validateVhdStreams) {
+      deltaExport.streams = mapValues(deltaExport.streams, stream => pipeline(stream, vhdStreamValidator, noop))
+    }
+    deltaExport.streams = mapValues(deltaExport.streams, this._throttleStream)
 
     const timestamp = Date.now()
+
     await this._callWriters(
       writer =>
         writer.transfer({
-          deltaExport: forkDeltaExport(deltaExport, writer.constructor.name),
+          deltaExport: forkDeltaExport(deltaExport),
           isVhdDifferencing,
+          sizeContainers,
           timestamp,
           vm,
           vmSnapshot: exportedVm,
         }),
       'writer.transfer()'
     )
+
+    // we want to control the uuid of the vhd in the chain
+    // and ensure they are correctly chained
     await this._callWriters(
       writer =>
         writer.updateUuidAndChain({
@@ -90,6 +102,15 @@ export const IncrementalXapi = class IncrementalXapiVmBackupRunner extends Abstr
       await markExportSuccessfull(this._xapi, exportedVm.$ref)
     }
 
+    const size = Object.values(sizeContainers).reduce((sum, { size }) => sum + size, 0)
+    const end = Date.now()
+    const duration = end - timestamp
+    debug('transfer complete', {
+      duration,
+      speed: duration !== 0 ? (size * 1e3) / 1024 / 1024 / duration : 0,
+      size,
+    })
+
     await this._callWriters(writer => writer.cleanup(), 'writer.cleanup()')
   }
 

@xen-orchestra/backups/_runners/_vmRunners/_AbstractRemote.mjs

@@ -13,8 +13,6 @@ import { Task } from '../../Task.mjs'
 
 export const AbstractRemote = class AbstractRemoteVmBackupRunner extends Abstract {
   _filterPredicate
-  _hasTransferredData
-
   constructor({
     config,
     job,
@@ -23,7 +21,7 @@ export const AbstractRemote = class AbstractRemoteVmBackupRunner extends Abstrac
     schedule,
     settings,
     sourceRemoteAdapter,
-    throttleGenerator,
+    throttleStream,
     vmUuid,
   }) {
     super()
@@ -36,7 +34,7 @@ export const AbstractRemote = class AbstractRemoteVmBackupRunner extends Abstrac
 
     this._healthCheckSr = healthCheckSr
     this._sourceRemoteAdapter = sourceRemoteAdapter
-    this._throttleGenerator = throttleGenerator
+    this._throttleStream = throttleStream
     this._vmUuid = vmUuid
 
     const allSettings = job.settings
@@ -92,7 +90,7 @@ export const AbstractRemote = class AbstractRemoteVmBackupRunner extends Abstrac
     for (const timestamp of timestamps) {
       if (remoteMetadatas[timestamp] !== nbRemotes) {
         // this backup is not present in all the remote
-        // should be retransferred if not found later
+        // should be retransfered if not found later
         transferList.push(localMetada.get(timestamp))
       } else {
         // backup is present in local and remote : the chain has already been transferred
@@ -143,16 +141,7 @@ export const AbstractRemote = class AbstractRemoteVmBackupRunner extends Abstrac
         })
       }, 'writer.beforeBackup()')
       await this._run()
-
-      if (this._hasTransferredData === undefined) {
-        throw new Error('Missing tag to check there are some transferred data')
-      }
-
-      if (this._hasTransferredData) {
-        await this._healthCheck()
-      } else {
-        Task.info(`No healthCheck needed because no data was transferred.`)
-      }
+      await this._healthCheck()
     })
   }
 }

@xen-orchestra/backups/_runners/_vmRunners/_AbstractXapi.mjs

@@ -28,7 +28,7 @@ export const AbstractXapi = class AbstractXapiVmBackupRunner extends Abstract {
     schedule,
     settings,
     srs,
-    throttleGenerator,
+    throttleStream,
     vm,
   }) {
     super()
@@ -62,7 +62,7 @@ export const AbstractXapi = class AbstractXapiVmBackupRunner extends Abstract {
     this._healthCheckSr = healthCheckSr
     this._jobId = job.id
     this._jobSnapshotVdis = undefined
-    this._throttleGenerator = throttleGenerator
+    this._throttleStream = throttleStream
     this._xapi = vm.$xapi
 
     // Base VM for the export
@@ -141,14 +141,14 @@ export const AbstractXapi = class AbstractXapiVmBackupRunner extends Abstract {
 
     const settings = this._settings
 
-    if (await this._mustDoSnapshot()) {
+    if (this._mustDoSnapshot()) {
       await Task.run({ name: 'snapshot' }, async () => {
         if (!settings.bypassVdiChainsCheck) {
           await vm.$assertHealthyVdiChains()
         }
-        if (settings.cbtDestroySnapshotData) {
+        if (settings.preferNbd) {
           try {
-            // enable CBT on all disks if we want to be able to purge snapshot data
+            // enable CBT on all disks if possible
             const diskRefs = await xapi.VM_getDisks(vm.$ref)
             await Promise.all(diskRefs.map(diskRef => xapi.callAsync('VDI.enable_cbt', diskRef)))
           } catch (error) {
@@ -195,7 +195,7 @@ export const AbstractXapi = class AbstractXapiVmBackupRunner extends Abstract {
       }
     })
     await Promise.all(vmSnapshots.map(snapshot => populateVdisOtherConfig(xapi, snapshot.$ref)))
-    // end of compatibility handling
+    // end of compatibiliy handling
 
     // handle snapshot by VDI
     this._jobSnapshotVdis = []
@@ -298,7 +298,7 @@ export const AbstractXapi = class AbstractXapiVmBackupRunner extends Abstract {
       // preferNbd is not a guarantee that the backup used NBD, depending on the network configuration,
       // in that case next runs will be full, but there is not an easy way to prevent that
       this._settings.preferNbd &&
-      // only delete snapshot data if the config allows it
+      // only delete snapshost data if the config allows it
       this._settings.cbtDestroySnapshotData
     ) {
       Task.info('will delete snapshot data')
@@ -324,7 +324,7 @@ export const AbstractXapi = class AbstractXapiVmBackupRunner extends Abstract {
     throw new Error('Not implemented')
   }
 
-  async _mustDoSnapshot() {
+  _mustDoSnapshot() {
     throw new Error('Not implemented')
   }
 

@xen-orchestra/backups/_runners/_vmRunners/_forkDeltaExport.mjs

@@ -1,16 +1,11 @@
-import { SynchronizedDisk } from '@xen-orchestra/disk-transform'
 import cloneDeep from 'lodash/cloneDeep.js'
+import mapValues from 'lodash/mapValues.js'
 
-export function forkDeltaExport(deltaExport, label) {
-  label += ' ' + Math.random()
-  const { disks, ...rest } = deltaExport
-  const fork = cloneDeep(rest)
-  fork.disks = {}
-  for (const [key, disk] of Object.entries(disks)) {
-    if (!(disk instanceof SynchronizedDisk)) {
-      throw new Error('Can only fork synchronized disks ')
-    }
-    fork.disks[key] = disk.fork(label)
-  }
-  return fork
+import { forkStreamUnpipe } from '../_forkStreamUnpipe.mjs'
+
+export function forkDeltaExport(deltaExport) {
+  const { streams, ...rest } = deltaExport
+  const newMetadata = cloneDeep(rest)
+  newMetadata.streams = mapValues(streams, forkStreamUnpipe)
+  return newMetadata
 }

@xen-orchestra/backups/_runners/_writers/IncrementalRemoteWriter.mjs

@@ -136,7 +136,7 @@ export class IncrementalRemoteWriter extends MixinRemoteWriter(AbstractIncrement
     const { handler } = this._adapter
     const vhds = this.#vhds
     await asyncEach(Object.entries(vdis), async ([id, vdi]) => {
-      const isDifferencing = isVhdDifferencing[id]
+      const isDifferencing = isVhdDifferencing[`${id}.vhd`]
       const path = `${this._vmBackupDir}/${vhds[id]}`
       if (isDifferencing) {
         assert.notStrictEqual(
@@ -202,8 +202,9 @@ export class IncrementalRemoteWriter extends MixinRemoteWriter(AbstractIncrement
 
     let metadataContent = await this._isAlreadyTransferred(timestamp)
     if (metadataContent !== undefined) {
-      // skip backup while being vigilant to not stuck the forked disk
-      await Promise.all(Object.values(deltaExport.disks).map(async disk => disk.close()))
+      // skip backup while being vigilant to not stuck the forked stream
+      Task.info('This backup has already been transfered')
+      Object.values(deltaExport.streams).forEach(stream => stream.destroy())
       return { size: 0 }
     }
 
@@ -222,33 +223,36 @@ export class IncrementalRemoteWriter extends MixinRemoteWriter(AbstractIncrement
       vmSnapshot,
       vtpms: deltaExport.vtpms,
     }
-    let size = 0
-    await Task.run({ name: 'transfer' }, async () => {
+
+    const { size } = await Task.run({ name: 'transfer' }, async () => {
+      let transferSize = 0
       await asyncEach(
-        Object.entries(deltaExport.disks),
-        async ([diskRef, disk]) => {
-          const path = `${this._vmBackupDir}/${vhds[diskRef]}`
-          await adapter.writeVhd(path, disk, {
+        Object.keys(deltaExport.vdis),
+        async id => {
+          const path = `${this._vmBackupDir}/${vhds[id]}`
+          // don't write it as transferSize += await async function
+          // since i += await asyncFun lead to race condition
+          // as explained : https://eslint.org/docs/latest/rules/require-atomic-updates
+          const transferSizeOneDisk = await adapter.writeVhd(path, deltaExport.streams[`${id}.vhd`], {
             // no checksum for VHDs, because they will be invalidated by
-            // merges and chains
+            // merges and chainings
             checksum: false,
             validator: tmpPath => checkVhd(handler, tmpPath),
             writeBlockConcurrency: this._config.writeBlockConcurrency,
           })
-          size = size + disk.getNbGeneratedBlock() * disk.getBlockSize()
+          transferSize += transferSizeOneDisk
         },
         {
           concurrency: settings.diskPerVmConcurrency,
         }
       )
 
-      return { size }
+      return { size: transferSize }
     })
-    metadataContent.size = size // @todo return exactly the size written by this writer
+    metadataContent.size = size
     this._metadataFileName = await adapter.writeVmBackupMetadata(vm.uuid, metadataContent)
 
     // TODO: run cleanup?
-    return { size }
   }
 }
 decorateClass(IncrementalRemoteWriter, {

@xen-orchestra/backups/_runners/_writers/IncrementalXapiWriter.mjs

@@ -11,6 +11,7 @@ import { MixinXapiWriter } from './_MixinXapiWriter.mjs'
 import { listReplicatedVms } from './_listReplicatedVms.mjs'
 import { COPY_OF, setVmOtherConfig, BASE_DELTA_VDI } from '../../_otherConfig.mjs'
 
+import assert from 'node:assert'
 export class IncrementalXapiWriter extends MixinXapiWriter(AbstractIncrementalWriter) {
   async checkBaseVdis(baseUuidToSrcVdi) {
     const sr = this._sr
@@ -21,15 +22,11 @@ export class IncrementalXapiWriter extends MixinXapiWriter(AbstractIncrementalWr
     }
 
     // @todo use an index if possible
-    // @todo : this seems similar to decorateVmMetadata
+    // @todo : this seems similare to decorateVmMetadata
     const replicatedVdis = sr.$VDIs
       .filter(vdi => {
         // REPLICATED_TO_SR_UUID is not used here since we are already filtering from sr.$VDIs
-        return (
-          vdi?.managed &&
-          !vdi?.is_a_snapshot /* only look for real vdi */ &&
-          baseUuidToSrcVdi.has(vdi?.other_config[COPY_OF])
-        )
+        return baseUuidToSrcVdi.has(vdi?.other_config[COPY_OF])
       })
       .map(({ other_config }) => other_config?.[COPY_OF])
       .filter(_ => !!_)
@@ -121,10 +118,9 @@ export class IncrementalXapiWriter extends MixinXapiWriter(AbstractIncrementalWr
         const baseReplicatedTo = replicatedVdis.find(
           replicatedVdi => replicatedVdi.other_config[COPY_OF] === vdi.other_config[BASE_DELTA_VDI]
         )
-        // baseReplicatedTo can be undefined if a new disk is added and other are already replicated
+        assert.notStrictEqual(baseReplicatedTo, undefined)
         vdi.baseVdi = baseReplicatedTo
       } else {
-        // first replication of this disk
         vdi.baseVdi = undefined
       }
       // ensure the VDI are created on the target SR
@@ -134,7 +130,7 @@ export class IncrementalXapiWriter extends MixinXapiWriter(AbstractIncrementalWr
     return backup
   }
 
-  async _transfer({ timestamp, deltaExport, vm }) {
+  async _transfer({ timestamp, deltaExport, sizeContainers, vm }) {
     const { _warmMigration } = this._settings
     const sr = this._sr
     const job = this._job
@@ -145,12 +141,8 @@ export class IncrementalXapiWriter extends MixinXapiWriter(AbstractIncrementalWr
     let targetVmRef
     await Task.run({ name: 'transfer' }, async () => {
       targetVmRef = await importIncrementalVm(this.#decorateVmMetadata(deltaExport), sr)
-      // size is mandatory to ensure the task have the right data
       return {
-        size: Object.values(deltaExport.disks).reduce(
-          (sum, disk) => sum + disk.getNbGeneratedBlock() * disk.getBlockSize(),
-          0
-        ),
+        size: Object.values(sizeContainers).reduce((sum, { size }) => sum + size, 0),
       }
     })
     this._targetVmRef = targetVmRef

@xen-orchestra/backups/_runners/_writers/_AbstractIncrementalWriter.mjs

@@ -20,14 +20,10 @@ export class AbstractIncrementalWriter extends AbstractWriter {
   async transfer({ deltaExport, ...other }) {
     try {
       return await this._transfer({ deltaExport, ...other })
-    } catch (err) {
-      console.error({ err })
     } finally {
-      // ensure all sources are properly closed
-      for (const disk of Object.values(deltaExport.disks)) {
-        try {
-          await disk.close()
-        } catch (err) {}
+      // ensure all streams are properly closed
+      for (const stream of Object.values(deltaExport.streams)) {
+        stream.destroy()
       }
     }
   }

@xen-orchestra/backups/disks/RemoteVhd.mjs

@@ -1,171 +0,0 @@
-// @ts-check
-
-/**
- * @typedef {import('@xen-orchestra/disk-transform').FileAccessor} FileAccessor
- * @typedef {import('@xen-orchestra/disk-transform').DiskBlock} DiskBlock
- * @typedef {import('@xen-orchestra/disk-transform').Disk} Disk
- * @typedef {import('vhd-lib/Vhd/VhdDirectory.js').VhdDirectory} VhdDirectory
- * @typedef {import('vhd-lib/Vhd/VhdFile.js').VhdFile} VhdFile
- */
-
-import { openVhd } from 'vhd-lib'
-import { DISK_TYPES } from 'vhd-lib/_constants.js'
-import { dirname, join } from 'node:path'
-import { RandomAccessDisk } from '@xen-orchestra/disk-transform'
-/**
- * Represents a remote VHD (Virtual Hard Disk) that extends RandomAccessDisk.
- */
-export class RemoteVhd extends RandomAccessDisk {
-  /**
-   * @type {string}
-   */
-  #path
-
-  /**
-   * @type {FileAccessor}
-   */
-  #handler
-
-  /**
-   * @type {VhdFile | VhdDirectory | undefined}
-   */
-  #vhd
-
-  /**
-   * @type {boolean | undefined}
-   */
-  #isDifferencing
-
-  /**
-   * @type {() => any}
-   */
-  #dispose = () => {}
-
-  /**
-   * @returns {string}
-   */
-  get path() {
-    return this.#path
-  }
-
-  /**
-   * @param {Object} params
-   * @param {FileAccessor} params.handler
-   * @param {string} params.path
-   */
-  constructor({ handler, path }) {
-    super()
-    // @todo : ensure this is the full path from the root of the remote
-    this.#path = path
-    this.#handler = handler
-  }
-
-  /**
-   * @returns {number}
-   */
-  getVirtualSize() {
-    if (this.#vhd === undefined) {
-      throw new Error(`can't call getvirtualsize of a RemoteVhd before init`)
-    }
-    return this.#vhd.footer.currentSize
-  }
-
-  /**
-   * @returns {number}
-   */
-  getBlockSize() {
-    return 2 * 1024 * 1024
-  }
-
-  /**
-   * Initializes the VHD.
-   * @returns {Promise<void>}
-   */
-  async init() {
-    const { value, dispose } = await openVhd(this.#handler, this.#path)
-    this.#vhd = value
-    this.#dispose = dispose
-    await this.#vhd.readBlockAllocationTable()
-    this.#isDifferencing = value.footer.diskType === DISK_TYPES.DIFFERENCING
-  }
-
-  /**
-   * Closes the VHD.
-   * @returns {Promise<void>}
-   */
-  async close() {
-    await this.#dispose()
-  }
-
-  /**
-   * Checks if the VHD contains a specific block.
-   * @param {number} index
-   * @returns {boolean}
-   */
-  hasBlock(index) {
-    if (this.#vhd === undefined) {
-      throw new Error(`can't call hasblock of a RemoteVhd before init`)
-    }
-    return this.#vhd.containsBlock(index)
-  }
-
-  /**
-   * Gets the indexes of all blocks in the VHD.
-   * @returns {Array<number>}
-   */
-  getBlockIndexes() {
-    if (this.#vhd === undefined) {
-      throw new Error(`can't call getBlockIndexes of a RemoteVhd before init`)
-    }
-    const index = []
-    for (let blockIndex = 0; blockIndex < this.#vhd.header.maxTableEntries; blockIndex++) {
-      if (this.hasBlock(blockIndex)) {
-        index.push(blockIndex)
-      }
-    }
-    return index
-  }
-
-  /**
-   * Reads a specific block from the VHD.
-   * @param {number} index
-   * @returns {Promise<DiskBlock>}
-   */
-  async readBlock(index) {
-    if (this.#vhd === undefined) {
-      throw new Error(`can't call readBlock of a RemoteVhd before init`)
-    }
-    const { data } = await this.#vhd.readBlock(index)
-    return {
-      index,
-      data,
-    }
-  }
-  /**
-   *
-   * @returns {RandomAccessDisk}
-   */
-  instantiateParent() {
-    if (this.#vhd === undefined) {
-      throw new Error(`can't call openParent of a RemoteVhd before init`)
-    }
-    const parentPath = this.#vhd.header.parentUnicodeName
-    const fullParentPath = join(dirname(this.#path), parentPath)
-    if (!parentPath) {
-      throw new Error(`Disk ${this.#path} doesn't have parents`)
-    }
-    const parent = new RemoteVhd({ handler: this.#handler, path: fullParentPath })
-    return parent
-  }
-
-  /**
-   * Checks if the VHD is a differencing disk.
-   * @returns {boolean}
-   */
-  isDifferencing() {
-    if (this.#isDifferencing === undefined) {
-      throw new Error(`can't call isDifferencing of a RemoteVhd before init`)
-    }
-    return this.#isDifferencing
-  }
-}

@xen-orchestra/backups/disks/openDiskChain.mjs

@@ -1,35 +0,0 @@
-// @ts-check
-/**
- *
- * @typedef {import('../../disk-transform/src/FileAccessor.mjs').FileAccessor} FileAccessor
- */
-import { DiskChain } from '@xen-orchestra/disk-transform'
-import { RemoteVhd } from './RemoteVhd.mjs'
-
-import { defer } from 'golike-defer'
-/**
- * @param {Object} params
- * @param {FileAccessor} params.handler
- * @param {string} params.path
- * @param {string | undefined} params.until
- */
-async function _openDiskChain($defer, { handler, path, until }) {
-  let disk
-  const disks = []
-  $defer.onFailure(() => Promise.all(disks.map(disk => disk.close())))
-  disk = new RemoteVhd({ handler, path })
-
-  await disk.init()
-  disks.push(disk)
-  while (disk.isDifferencing()) {
-    disk = await disk.openParent()
-    if (disk.path === until) {
-      break
-    }
-    disks.unshift(disk)
-  }
-  // the root disk
-  return new DiskChain({ disks })
-}
-
-export const openDiskChain = defer(_openDiskChain)

@xen-orchestra/backups/docs/VM backups/README.md

@@ -29,7 +29,7 @@
      │        └─ <YYYYMMDD>T<HHmmss>.vhd
      ├─ <YYYYMMDD>T<HHmmss>.json // backup metadata
      ├─ <YYYYMMDD>T<HHmmss>.xva
-     └─ <YYYYMMDD>T<HHmmss>.xva.checksum // only on backup before 01/02/2025 or non encrypted
+     └─ <YYYYMMDD>T<HHmmss>.xva.checksum
 ```
 
 ### with vhd directories
@@ -53,7 +53,7 @@ Add the following file: `xo-vm-backups/<VM UUID>/cache.json.gz`.
 
 This cache is compressed in Gzip and contains an JSON object with the metadata for all the backups of this VM indexed by their absolute path (i.e. `/xo-vm-backups/<VM UUID>/<timestamp>.json`).
 
-This file is generated on demand when listing the backups, and directly updated on backup creation/deletion.
+This file is generated on demande when listing the backups, and directly updated on backup creation/deletion.
 
 In case any incoherence is detected, the file is deleted so it will be fully generated when required.
 
@@ -180,7 +180,7 @@ job:
   sourceRemote: IdPattern
 
 # Indicates which XAPI to use to connect to a specific VM or SR
-# for remote to remote backup job,this is only needed if there is healthcheck
+# for remote to remote backup job,this is only needed if there is healtcheck
 recordToXapi:
   [ObjectId]: XapiId
 

@xen-orchestra/backups/package.json

@@ -8,7 +8,7 @@
     "type": "git",
     "url": "https://github.com/vatesfr/xen-orchestra.git"
   },
-  "version": "0.64.3",
+  "version": "0.57.0",
   "engines": {
     "node": ">=14.18"
   },
@@ -25,14 +25,11 @@
     "@vates/decorate-with": "^2.1.0",
     "@vates/disposable": "^0.1.6",
     "@vates/fuse-vhd": "^2.1.2",
-    "@vates/generator-toolbox": "^1.0.4",
-    "@vates/nbd-client": "^3.2.1",
+    "@vates/nbd-client": "^3.1.2",
     "@vates/parse-duration": "^0.1.1",
     "@xen-orchestra/async-map": "^0.1.2",
-    "@xen-orchestra/disk-transform": "^1.1.0",
-    "@xen-orchestra/fs": "^4.6.3",
+    "@xen-orchestra/fs": "^4.3.0",
     "@xen-orchestra/log": "^0.7.1",
-    "@xen-orchestra/qcow2": "^1.0.0",
     "@xen-orchestra/template": "^0.1.0",
     "app-conf": "^3.0.0",
     "compare-versions": "^6.0.0",
@@ -51,8 +48,8 @@
     "tar": "^6.1.15",
     "uuid": "^9.0.0",
     "value-matcher": "^0.2.0",
-    "vhd-lib": "^4.14.1",
-    "xen-api": "^4.7.4",
+    "vhd-lib": "^4.11.2",
+    "xen-api": "^4.7.0",
     "yazl": "^2.5.1"
   },
   "devDependencies": {
@@ -62,7 +59,7 @@
     "tmp": "^0.2.1"
   },
   "peerDependencies": {
-    "@xen-orchestra/xapi": "^8.4.1"
+    "@xen-orchestra/xapi": "^7.10.0"
   },
   "license": "AGPL-3.0-or-later",
   "author": {

@xen-orchestra/backups/runBackupWorker.mjs

@@ -4,22 +4,10 @@ import { fork } from 'child_process'
 const { warn } = createLogger('xo:backups:backupWorker')
 
 const PATH = new URL('_backupWorker.mjs', import.meta.url).pathname
-const DEFAULT_INSPECTOR_PORT = 9229
 
 export function runBackupWorker(params, onLog) {
   return new Promise((resolve, reject) => {
-    // run Node inspector on port+1 if --inspect or --inspect-brk
-    const inspectArg = process.execArgv.find(arg => arg.startsWith('--inspect') || arg.startsWith('--inspect-brk'))
-    const execArgv = inspectArg
-      ? [
-          inspectArg.replace(/^(--inspect(-brk)?)(=([^:]+:)?(\d+))?$/, (_, prefix, brk, _fullMatch, host, port) => {
-            const basePort = port ? parseInt(port) : DEFAULT_INSPECTOR_PORT
-            return `${prefix}=${host || ''}${basePort + 1}`
-          }),
-        ]
-      : []
-
-    const worker = fork(PATH, [], { execArgv })
+    const worker = fork(PATH)
 
     worker.on('exit', (code, signal) => reject(new Error(`worker exited with code ${code} and signal ${signal}`)))
     worker.on('error', reject)

@xen-orchestra/cr-seed-cli/package.json

@@ -18,7 +18,7 @@
   "preferGlobal": true,
   "dependencies": {
     "golike-defer": "^0.5.1",
-    "xen-api": "^4.7.4"
+    "xen-api": "^4.7.0"
   },
   "scripts": {
     "postversion": "npm publish"

@xen-orchestra/disk-transform/.npmignore

@@ -1 +0,0 @@
-../../scripts/npmignore

@xen-orchestra/disk-transform/eslint.config.mjs

@@ -1,6 +0,0 @@
-// @ts-check
-
-import eslint from '@eslint/js'
-import tseslint from 'typescript-eslint'
-
-export default tseslint.config(eslint.configs.recommended, tseslint.configs.recommended)

@xen-orchestra/disk-transform/package.json

@@ -1,40 +0,0 @@
-{
-  "name": "@xen-orchestra/disk-transform",
-  "version": "1.1.0",
-  "main": "./dist/index.mjs",
-  "license": "MIT",
-  "private": false,
-  "type": "module",
-  "devDependencies": {
-    "@eslint/js": "^9.19.0",
-    "rimraf": "^5.0.1",
-    "@types/node": "^22.3.0",
-    "typescript": "~5.6",
-    "typescript-eslint": "^8.23.0"
-  },
-  "scripts": {
-    "build": "tsc",
-    "dev": "tsc --watch",
-    "postversion": "npm publish --access public",
-    "test": "tsc && node --test **/*.test.mjs",
-    "clean": "rimraf dist/"
-  },
-  "homepage": "https://github.com/vatesfr/xen-orchestra/tree/master/@xen-orchestra/disk-transform",
-  "bugs": "https://github.com/vatesfr/xen-orchestra/issues",
-  "repository": {
-    "directory": "@xen-orchestra/disk-transform",
-    "type": "git",
-    "url": "https://github.com/vatesfr/xen-orchestra.git"
-  },
-  "author": {
-    "name": "Vates SAS",
-    "url": "https://vates.fr"
-  },
-  "engines": {
-    "node": ">=20.17"
-  },
-  "dependencies": {
-    "vhd-lib": "^4.14.1",
-    "@vates/generator-toolbox": "^1.0.4"
-  }
-}

@xen-orchestra/disk-transform/src/Disk.mts

@@ -1,112 +0,0 @@
-import { ProgressHandler } from './ProgressHandler.mjs'
-
-export type DiskBlockData = Buffer
-export type DiskBlock = {
-  index: number // the index of the block. Offset in raw disk is index * blockSize
-  data: DiskBlockData // thue Buffer like data of this block. Must be blockSize length
-}
-
-export type BytesLength = number
-
-export abstract class Disk {
-  #generatedDiskBlocks = 0
-  #parent?: Disk
-  progressHandler?: ProgressHandler
-  get parent(): Disk | undefined {
-    return this.#parent
-  }
-
-  abstract getVirtualSize(): number
-  abstract getBlockSize(): number
-  abstract init(): Promise<void>
-  abstract close(): Promise<void>
-
-  abstract isDifferencing(): boolean
-  // optional method
-  instantiateParent(): Disk {
-    throw new Error('Method not implemented.')
-  }
-  async openParent(): Promise<Disk> {
-    if (this.#parent !== undefined) {
-      return this.#parent
-    }
-    if (!this.isDifferencing()) {
-      throw new Error("Can't open the parent of a non differencing disk")
-    }
-    this.#parent = this.instantiateParent()
-    await this.#parent.init()
-    return this.#parent
-  }
-
-  /**
-   * return the block without any order nor stability guarantee
-   */
-  abstract getBlockIndexes(): Array<number>
-  // must return true if the block is present in this disk
-  // without looking at the parent
-  abstract hasBlock(index: number): boolean
-  abstract buildDiskBlockGenerator(): Promise<AsyncGenerator<DiskBlock>> | AsyncGenerator<DiskBlock>
-  async *diskBlocks(uid?: string): AsyncGenerator<DiskBlock> {
-    try {
-      // compute next block while the destination is consuming the current block
-      const blockGenerator = await this.buildDiskBlockGenerator()
-      let next = blockGenerator.next()
-      while (true) {
-        const res = await next
-        next = blockGenerator.next()
-        if (res.done) {
-          break
-        }
-        this.#generatedDiskBlocks++
-        yield res.value
-      }
-    } finally {
-      await this.close()
-    }
-  }
-
-  getNbGeneratedBlock(): number {
-    return this.#generatedDiskBlocks
-  }
-  check() {
-    // @todo move here the checks done in cleanVm if possible
-    return Promise.resolve(true)
-  }
-
-  // @todo readRawData will be needed for file level restore
-  // @todo rename will be needed for merge
-  // @todo : should the alias part moved from remote/vhd ?
-}
-
-/**
- * Disks implementing this class can be accessed in any order
- * For example xva will need to have data in the offset order
- */
-
-export abstract class RandomAccessDisk extends Disk {
-  get parent(): RandomAccessDisk | undefined {
-    if (super.parent !== undefined && !(super.parent instanceof RandomAccessDisk)) {
-      throw new Error('The parent of a random access disk must be a random access disk')
-    }
-    return super.parent
-  }
-  // optional method
-  instantiateParent(): RandomAccessDisk {
-    throw new Error('Method not implemented.')
-  }
-  // can read data parent if block size are not aligned
-  // but only if this disk has data on this block
-  abstract readBlock(index: number): Promise<DiskBlock>
-  async *buildDiskBlockGenerator(): AsyncGenerator<DiskBlock> {
-    try {
-      const indexes = this.getBlockIndexes()
-      for (let i = 0; i < indexes.length; i++) {
-        yield this.readBlock(indexes[i])
-        await this.progressHandler?.setProgress(i / indexes.length)
-      }
-      await this.progressHandler?.setProgress(1)
-    } finally {
-      await this.progressHandler?.done()
-    }
-  }
-}

@xen-orchestra/disk-transform/src/Disk.test.mts

@@ -1,42 +0,0 @@
-import { afterEach, beforeEach, test } from 'node:test'
-import assert from 'node:assert'
-import { DebugDisk } from './utils/DebugDisk.mjs'
-
-test('Disk class', async t => {
-  const blockSize = 1024
-  const nbBlocks = 20
-  let disk: DebugDisk
-  beforeEach(async () => {
-    disk = new DebugDisk({ nbBlocks, blockSize, fillRate: 0 })
-
-    await disk.init()
-  })
-  afterEach(async () => {
-    await disk.close()
-  })
-  await t.test('getVirtualSize', () => {
-    assert.strictEqual(disk.getVirtualSize(), blockSize * nbBlocks)
-  })
-
-  await t.test('getBlockSize', () => {
-    assert.strictEqual(disk.getBlockSize(), blockSize)
-  })
-
-  await t.test('diskBlocks generator', async () => {
-    const keys = [4, 15]
-    for (const key of keys) {
-      disk.blocks[key] = true
-    }
-    const foundKeys = []
-    const generator =  disk.diskBlocks()
-    for await (const block of generator) {
-      foundKeys.push(block.index)
-      assert.strictEqual(block.data.length, blockSize)
-    }
-    assert.strictEqual(disk.getNbGeneratedBlock(), 2)
-
-    keys.sort()
-    foundKeys.sort()
-    assert.deepStrictEqual(keys, foundKeys)
-  })
-})

@xen-orchestra/disk-transform/src/DiskChain.mts

@@ -1,69 +0,0 @@
-import { Disk, RandomAccessDisk, type DiskBlock } from './Disk.mjs'
-
-export class DiskChain extends RandomAccessDisk {
-  #disks: Array<RandomAccessDisk> = []
-
-  constructor({ disks }: { disks: Array<RandomAccessDisk> }) {
-    super()
-    this.#disks = disks
-  }
-  getVirtualSize(): number {
-    return this.#disks[this.#disks.length - 1].getVirtualSize()
-  }
-  getBlockSize(): number {
-    return this.#disks[0].getBlockSize()
-  }
-
-  /**
-   * the main difference with the base disk block method 
-   * is that if any of the disk has this block it return true
-   * 
-   */
-  hasBlock(index: number): boolean {
-    for (let i = this.#disks.length - 1; i >= 0; i--) {
-      if (this.#disks[i].hasBlock(index)) {
-        return true
-      }
-    }
-    return false
-  }
-
-  /**
-   * Read in the last disk first an only go back if the block is not present
-   * @param index 
-   * @returns 
-   */
-  readBlock(index: number): Promise<DiskBlock> {
-    for (let i = this.#disks.length - 1; i >= 0; i--) {
-      if (this.#disks[i].hasBlock(index)) {
-        return this.#disks[i].readBlock(index)
-      }
-    }
-    throw new Error(`Block ${index} not found in chain `)
-  }
-
-  async init(): Promise<void> {
-    await Promise.all(this.#disks.map(disk => disk.init()))
-  }
-  async close(): Promise<void> {
-    await Promise.all(this.#disks.map(disk => disk.close()))
-  }
-  getBlockIndexes(): Array<number> {
-    return [...new Set(this.#disks.map(disk => disk.getBlockIndexes()).flat())]
-  }
-
-  isDifferencing(): boolean {
-    return this.#disks[0].isDifferencing()
-  }
-  static async openFromChild(child:RandomAccessDisk):Promise<RandomAccessDisk>{
-    let disk = child
-    const disks=[disk]
-    while (disk.isDifferencing()) {
-      disk = await disk.openParent() as RandomAccessDisk
-      disks.unshift(disk)
-      // @todo handle until
-    }
-    return new DiskChain({ disks })
-
-  }
-}

@xen-orchestra/disk-transform/src/DiskChain.test.mts

@@ -1,62 +0,0 @@
-import { afterEach, beforeEach, test } from 'node:test'
-import assert from 'node:assert'
-import { DebugDisk } from './utils/DebugDisk.mjs'
-import { DiskChain } from './DiskChain.mjs'
-
-test('Disk Chain class', async t => {
-  const blockSize = 1024
-  const nbBlocks = 20
-  const nbDisks = 5
-  const disks: Array<DebugDisk> = []
-  let chain: DiskChain
-  beforeEach(async () => {
-    for (let i = 0; i < nbDisks; i++) {
-      const disk = new DebugDisk({ nbBlocks: nbBlocks + 1, blockSize, fillRate: 0, filledBy: i })
-      disks.push(disk)
-    }
-    chain = new DiskChain({ disks })
-    await chain.init()
-  })
-  afterEach(async () => {
-    await chain.close()
-  })
-  await t.test('getVirtualSize', () => {
-    // the size of the last one
-    assert.strictEqual(chain.getVirtualSize(), disks[disks.length - 1].getVirtualSize())
-  })
-
-  await t.test('getBlockSize', () => {
-    assert.strictEqual(chain.getBlockSize(), blockSize)
-  })
-
-  await t.test('diskBlocks generator read from the right disk', async () => {
-    const keys = []
-
-    for (let blockIndex = 0; blockIndex < nbBlocks; blockIndex++) {
-      disks[0].blocks[blockIndex] = true //each disk only have one block, its disk index
-      keys.push(blockIndex)
-    }
-    for (let diskIndex = 0; diskIndex < nbDisks; diskIndex++) {
-      disks[diskIndex].blocks[diskIndex] = true //each disk only have one block, its disk index
-    }
-
-    const foundKeys = []
-    const generator =  chain.diskBlocks()
-    for await (const block of generator) {
-      foundKeys.push(block.index)
-      assert.strictEqual(block.data.length, blockSize)
-      if (block.index < nbDisks) {
-        assert.strictEqual(block.data.readUInt8(0), block.index)
-      } else {
-        // read from first
-        assert.strictEqual(block.data.readUInt8(0), 0)
-      }
-    }
-
-    assert.strictEqual(chain.getNbGeneratedBlock(), nbBlocks)
-
-    keys.sort()
-    foundKeys.sort()
-    assert.deepStrictEqual(keys, foundKeys)
-  })
-})