Thomas Sanders [Tue, 12 Oct 2010 10:59:58 +0000 (11:59 +0100)]
CA-37034 Lenny install-from-CD warning dialog
The installation template for Debian Lenny now warns that the standard CD/DVD
does not support XenServer, and gives directions to find ISOs that work.
For Debian Squeeze, there is now a warning that a multi-arch image is needed.
Signed-off-by: Thomas Sanders <thomas.sanders@citrix.com>
While I reviewed Rok's changes to cli_operations.ml, I realized how
aweful that file looks. I have used our hacked up emacs tuareg-mode
to fix the indentation mechanically and checked whether the result was
sane.
Alex Zeffertt [Tue, 12 Oct 2010 10:54:11 +0000 (11:54 +0100)]
CP-1876: Create RHEL 6 HVM template
Notes:
* We choosing to enable the "no execute" CPU flag in this guest
* We are not indicating the presence of viridian to this guest
Signed-off-by: Alex Zeffertt <alex.zeffertt@eu.citrix.com> Signed-off-by: David Scott <dave.scott@eu.citrix.com> Signed-off-by: Rob Hoes <rob.hoes@gmail.com>
Ewan Mellor [Tue, 12 Oct 2010 09:58:34 +0000 (10:58 +0100)]
The VM metadata export handler (/export_metadata) will output the metadata
of a VM, all its snapshots, and all their attached VDIs. However, it doesn't
walk the tree of VDIs defined by the VDI.sm_config["vhd_parent"] field. For
a full-fidelity representation, having the parent VDIs is very useful.
This patch adds an "include_vhd_parents=(true|false)" option to the the
/export_metadata handler. With this option on, the entire tree of VDIs is
exported.
This patch does not change the /export URL handler, does not change any
behaviour when include_vhd_parents is false or missing, and copes gracefully
with a missing or malformed vhd_parent entry.
Dave: modified to be robust to vhd parent cycles
Signed-off-by: Ewan Mellor <ewan.mellor@eu.citrix.com> Signed-off-by: Dave Scott <dave.scott@eu.citrix.com>
Rok Strnisa [Tue, 12 Oct 2010 09:54:04 +0000 (10:54 +0100)]
CA-27648: xe does not show the tags fields --- FIXED.
Added the ability to read and write tags on pool, host, network, VM, SR, and
VDI through the CLI. For each corresponding function, the patch adds four
lines of the general form:
make_field ~name:"tags"
~get:(fun () -> String.concat ", " (x ()).API.xxx_tags)
~get_set:(fun () -> (x ()).API.xxx_tags)
~add_to_set:(fun tag -> Client.Xxx.add_tags rpc session_id xx tag)
~remove_from_set:(fun tag -> Client.Xxx.remove_tags rpc session_id xx tag) ();
The patch also includes code re-indentation for the functions involved.
It appears as if much of this file could be auto-generated.
Signed-off-by: Rok Strnisa <rok.strnisa@citrix.com>
David Scott [Tue, 12 Oct 2010 09:53:03 +0000 (10:53 +0100)]
CP-1914: Record some extra build-time information in Host.software_version ("product_version_text" and "product_version_text_short" -- useful for UIs to display rather than raw version numbers)
Signed-off-by: David Scott <dave.scott@eu.citrix.com>
The build-number that xapi displays for a host now comes from
/etx/xensource-inventory instead of the branded in build-number, when
xapi is build. The former behaviour lead to discrepancies.
Rok Strnisa [Tue, 12 Oct 2010 09:50:42 +0000 (10:50 +0100)]
CA-44714: Semantics of "xe vm-export" with the metadata option is not as documented --- FIXED.
Consistent requirement for boolean parameters to be specified with true or false.
The patch includes refactoring of cli_operations.ml, which makes consistent
use of two simple functions for extracting information from parameters instead
of the previous 10 complicated ways.
Signed-off-by: Rok Strnisa <rok.strnisa@citrix.com>
Review by Matthias: Changes look good. But indentation in
get_bool_param is as horrible as in the rest of cli_operations.ml.
After-review changes: The horrible indentation (2 spaces) of get_bool_param has
has been corrected (into 1 tab).
Jonathan Knowles [Tue, 12 Oct 2010 09:40:01 +0000 (10:40 +0100)]
Rescues the indendation for several functions within the quicktest module. Splits an often-modified list of tests into multiple lines, reducing the chance that further patches will conflict with one another.
Signed-off-by: Jonathan Knowles <jonathan.knowles@eu.citrix.com>
Ben Pfaff [Tue, 12 Oct 2010 09:38:38 +0000 (10:38 +0100)]
[PATCH] Rename "xs-vif-mac" external_ids key to "attached-mac", for generality.
From e61070c32030d6d00e2eeae213d219320a7cbd10 Mon Sep 17 00:00:00 2001
Date: Wed, 1 Sep 2010 16:49:19 -0700
This can be useful on systems other than XenServer so there is no reason
to make it looks XenServer-specific.
CC: Jeremy Stribling <strib@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
vswitchd/vswitch.xml | 15 ++++++++-------
scripts/vif | 2 +-
2 files changed, 9 insertions(+), 8 deletions(-)
Rob Hoes [Tue, 12 Oct 2010 09:36:54 +0000 (10:36 +0100)]
Require datetime values written to the DB to end with a 'Z'
All datetime values in the DB need to be UTC dates, and are required to ISO 8601 formatted with a trailing 'Z' to indicate they are UTC. The XenAPI did not have any functions that allow users to write datetimes directly to the DB... until recently (e.g. VMPP.set_archive_last_run_time)! It is therefore now necessary to enforce that the datetime values in the parameters of such 'set' functions have the correct form.
For backwards compatibility, XenAPI functions such as message.get_since still allow dates without 'Z', and will assume these are UTC dates.
Rob Hoes [Tue, 12 Oct 2010 09:36:09 +0000 (10:36 +0100)]
Hosts in a pool should use the same networking backend
This patch adds the following restrictions in order to prevent (as much as possible) trouble caused by hosts in a pool not having the same network backends (bridge/openvswitch):
1. Block pool join if the network backend of the joining host is not the same as the master's.
2. Block tunnel.create if at least one of the hosts in the pool uses the bridging backend.
Rob Hoes [Tue, 12 Oct 2010 09:34:47 +0000 (10:34 +0100)]
CP-1591: Synchronise VIF.MTU and PIF.MTU on plug
The VIF.MTU and PIF.MTU fields are now read-only, and will reflect the current state of the network. When you change the Network.MTU, VIFs and PIFs will have to be replugged to have their MTU changed by the networking subsystem. So when Network.MTU and PIF.MTU are not equal, this is a reminder to replug the PIF (same for VIFs).
David Scott [Tue, 12 Oct 2010 09:34:27 +0000 (10:34 +0100)]
CA-44731: rewrite RRD marshalling, unmarshalling code to (i) copy the compact binary representation; and then (ii) stream the XML directly to an fd without creating a temporary buffer of some kind.
This should now work properly when the XML data is > 16MiB
Signed-off-by: David Scott <dave.scott@eu.citrix.com>
David Scott [Tue, 12 Oct 2010 09:32:46 +0000 (10:32 +0100)]
CA-44731: record timings for all xapi startup tasks for diagnostic purposes. Data can be retrieved with existing 'xe diagnostic-timing-stats' CLI command.
Signed-off-by: David Scott <dave.scott@eu.citrix.com>
David Scott [Tue, 12 Oct 2010 09:32:18 +0000 (10:32 +0100)]
CA-41832: clip the target of an 'inactive' domain to be within dynamic_min/dynamic_max
This is potentially dangerous: the reason we attempt to cap the memory usage of an unresponsive domain is to prevent it suddenly waking up and allocating at a bad moment, resulting in either a transient OOM or loss of low memory. With this change the cap will be 'loose', where target can be higher than memory_actual.
Signed-off-by: David Scott <dave.scott@eu.citrix.com>
David Scott [Mon, 11 Oct 2010 22:00:16 +0000 (23:00 +0100)]
CA-46050: allow more than 2 PCI devices to be hotplugged into a domain (!)
There is much confusion about the meaning of the X in
(X, (domain, bus, path, func)). Sometimes it means "order the device was
plugged"; other times it means something more device related.
With this change, more than 2 devices can be plugged into a domain and the
#devices and plug order remains constant over reboot.
Signed-off-by: David Scott <dave.scott@eu.citrix.com>
Jon Ludlam [Mon, 11 Oct 2010 21:57:29 +0000 (22:57 +0100)]
The recent addition of the VDI on_boot and allow_caching parameters affected the allowed operations check. The additional check was put in
the middle of the if block rather than at the end, thus short circuiting some subsequent checks that would otherwise have been done. For
example, checkpoint is allowed in a VM with no tools. This patch fixes that behaviour.
Signed-off-by: Jon Ludlam <jonathan.ludlam@eu.citrix.com>
diff -r 95b9e4f1b9dd ocaml/xapi/xapi_vm_lifecycle.ml
Jon Ludlam [Fri, 24 Sep 2010 16:27:30 +0000 (17:27 +0100)]
Local disk caching API work. Added pool-level API calls to enable/disable caching, host-level API calls for same, per VDI on_boot and allow_caching parameters, and CLI calls for all.
Signed-off-by: Jon Ludlam <Jonathan.Ludlam@eu.citrix.com>
Jon Ludlam [Fri, 24 Sep 2010 15:46:04 +0000 (16:46 +0100)]
Pass the writable flag to the SM backends on activate as well as attach. This is a temporary fix to work around
an implementation hurdle in the backends.
Signed-off-by: Jon Ludlam <jonathan.ludlam@eu.citrix.com>
Ted M Lin [Wed, 4 Aug 2010 19:55:23 +0000 (15:55 -0400)]
MAR-112: RRD transfer lock can prevent master transition
If the master loses connectivity during a RRD transfer, the RRD mutex
will never be released. When performing a emergency master transition,
the first thing that occurs is to save the local RRDs to disk, which
requires the RRD mutex. This will never succeed, thus XAPI gets stuck
trying to shut down.
Instead of locking the mutex, we try to acquire it for a few seconds.
If the lock can not be taken, the loss of some RRD data is preferable
to not being able to transition at all.
Signed-off-by: Ted M Lin <tlin@marathontechnologies.com>