Xapi#setPoolProperties() supports autoPowerOn.

src/api/host.coffee

@@ -14,10 +14,17 @@ startsWith = require 'lodash.startswith'
 
 #=====================================================================
 
-set = (params) ->
+set = ({
-  host = extractProperty(params, 'host')
+  host,
 
-  return @getXAPI(host).setHostProperties(host._xapiId, params)
+  # TODO: use camel case.
+  name_label: nameLabel,
+  name_description: nameDescription
+}) ->
+  return @getXAPI(host).setHostProperties(host._xapiId, {
+    nameLabel,
+    nameDescription
+  })
 
 set.description = 'changes the properties of an host'
 

src/api/pool.js

@@ -1,12 +1,18 @@
 import {JsonRpcError} from '../api-errors'
-import {extractProperty} from '../utils'
 
 // ===================================================================
 
-export async function set (params) {
+export async function set ({
-  const pool = extractProperty(params, 'pool')
+  pool,
 
-  await this.getXAPI(pool).setPoolProperties(params)
+  // TODO: use camel case.
+  name_description: nameDescription,
+  name_label: nameLabel
+}) {
+  await this.getXAPI(pool).setPoolProperties({
+    nameDescription,
+    nameLabel
+  })
 }
 
 set.params = {

src/api/sr.js

@@ -1,16 +1,22 @@
 import {
   ensureArray,
-  extractProperty,
   forEach,
   parseXml
 } from '../utils'
 
 // ===================================================================
 
-export async function set (params) {
+export async function set ({
-  const sr = extractProperty(params, 'sr')
+  sr,
 
-  await this.getXAPI(sr).setSrProperties(sr._xapiId, params)
+  // TODO: use camel case.
+  name_description: nameDescription,
+  name_label: nameLabel
+}) {
+  await this.getXAPI(sr).setSrProperties(sr._xapiId, {
+    nameDescription,
+    nameLabel
+  })
 }
 
 set.params = {

src/xapi.js

@@ -234,32 +234,52 @@ export default class Xapi extends XapiBase {
   }
 
   async setHostProperties (id, {
-    name_label,
+    nameLabel,
-    name_description
+    nameDescription
   }) {
     await this._setObjectProperties(this.getObject(id), {
-      name_label,
+      nameLabel,
-      name_description
+      nameDescription
     })
   }
 
   async setPoolProperties ({
-    name_label,
+    autoPowerOn,
-    name_description
+    nameLabel,
+    nameDescription
   }) {
-    await this._setObjectProperties(this.pool, {
+    const { pool } = this
-      name_label,
+
-      name_description
+    const promises = [
-    })
+      this._setObjectProperties(pool, {
+        nameLabel,
+        nameDescription
+      })
+    ]
+
+    // TODO: mutualize this code.
+    if (autoPowerOn != null) {
+      let p = this.call('pool.remove_from_other_config', pool.$ref, 'auto_poweron')
+
+      if (autoPowerOn) {
+        p = p
+          .catch(noop)
+          .then(() => this.call('pool.add_to_other_config', pool.$ref, 'auto_poweron', 'true'))
+      }
+
+      promises.push(p)
+    }
+
+    await Promise.all(promises)
   }
 
   async setSrProperties (id, {
-    name_label,
+    nameLabel,
-    name_description
+    nameDescription
   }) {
     await this._setObjectProperties(this.getObject(id), {
-      name_label,
+      nameLabel,
-      name_description
+      nameDescription
     })
   }