Commit Graph

30 Commits

Author SHA1 Message Date
Stephen Turner
b587ab832e CP-14647: Intermediate check-in #1 2015-11-08 11:27:56 +00:00
Gabor Apati-Nagy
7b7bcb0aa5 CP-14379, CP-14461, CP-14462, CP-14463, CP-14464, CP-14465, CP-14467:
Implementing Upgrading VMs to Windows Update

New screen features:
-Create skeleton of the Upgrade VM dialog
-Implement a Go To Console link (just a link that opens a vm console)
-Implement logic to show the right status for a VM (includes rules to
get the states). Includes updating the dialog whenever the status changes.
-Add status texts to Messages, proper text on the UI
-Add stub methods to VM class in XenAPI-Extensions to r/w states from
Guest Agent
-UI: Implement rules what vms to display, checkbox rules, read only vms,
and when to enable the Upgrade button
-Implement Upgrade button (writes state for GA, mounts ISO) + maximum
parallel executions/actions

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-10-16 12:47:00 +01:00
Mihaela Stoica
f507b65126 CA-183321: Fixed the bug where the disk initial and incremental allocations values were ignored while creating new VM's disk
When creating a new VM, we no longer use vm.provision to create VM's disks, but instead we create the disks ourselves using vdi.create; this way we can specify the initial and incremental allocations for the new disks.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-09-18 15:51:17 +01:00
Gabor Apati-Nagy
191820abf4 CA-177058: [SCTX-2146] VM Power Admin could not create VMs - RBAC issues with XS 6.5 2015-08-04 11:25:59 +01:00
Gabor Apati-Nagy
ffb620461f CA-147254: Adding DVD drive fails with obscure error message 2015-05-15 16:32:37 +01:00
Mihaela Stoica
830d8f6f60 CP-11292: Remove storage motion constraints: changes following code review
- added some null checks
- in the Cross pool migrate wizard, add Transfer network page for all cases except intra-pool move (which is performed via VMMoveAction)
- added comments to VMCrossPoolMigrateAction constructor to make it cleared what the copy parameter means

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-04-15 09:37:12 +01:00
Mihaela Stoica
e314a654f7 CP-11292: Remove storage motion constraints: Allow cross-pool copy for stopped VMs
- modified the VMCopyCommand to launch a cross pool copy if permitted
- added the CrossPoolCopyVMCommand which opens the Cross pool migration wizard in "copy" mode
- modified the cross pool migrate action with an extra "copy" parameter, which will add a copy option to the vm.migrate_send function call

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-31 14:27:04 +01:00
Mihaela Stoica
d616009824 CP-11032: Remove storage motion constraints: Allow cross-pool move of stopped and suspended VMs
- With these changes, a stopped or suspended VM can be moved across pools; this is performed as a vm migrate operation.
- The intra pool move will continue to be a vdi.copy + destroy operation for stopped VMs, but for suspended VMs we need to do it a as vm migrate operation.
- We use the Cross Pool migrate wizard for both intra and cross pool move operations, but introduced a wizard mode parameter in order to adapt the wizard to the specific operation we want to perform (migrate, move or copy)

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-17 16:35:12 +00:00
Mihaela Stoica
18ee60104b CA-162966: Fix the bug where the GPU value could not be updated via VM Properties for Linux HVM
- The problem was that we didn't create a vgpu if the vgpu_type was null. But for Linux HVM, this is gpu passthrough case.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-04 16:50:49 +00:00
Mihaela Stoica
eab66b9bf0 CA-161002: Internationalize strings on the Cloud Config Parameters page and CreateVMAction
- also added Help Id got the Cloud Config Parameters page

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-19 14:48:52 +00:00
Mihaela Stoica
9c652cd687 CP-10926: Add dialog for customizing the config drive in XenCenter after VM creation
- The cloud config parameters page is only visible on a VM that already has a config drive.
- The cloud config parameters can only be changed when the VM is halted.
- In the New VM wizard, if a template is selected that already has a config drive, them the wizard will display the existing configuration, not the default one.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-13 14:43:13 +00:00
Mihaela Stoica
8b9d089687 CA-160904: Fix NullReferenceException on the New VM Wizard
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-09 11:16:01 +00:00
Gabor Apati-Nagy
fddb5a032b Merge remote-tracking branch 'creedence-docker/master' into docker
Conflicts:
	XenAdmin/Wizards/NewVMWizard/NewVMWizard.cs
	XenModel/Messages.resx
2015-02-04 16:23:20 +00:00
Mihaela Stoica
dbdd3cc02c CA-157501: Fixing inconsistent GPU dialogs in XS "Standard" when compared with "Enterprise"
New properties:
- Pool.HasGpu = Pool has at least one PGPU
- Pool.HasVGpu = Pool has at least one PGPU that HasVGpu
- PGPU.HasVGpu = PGPU has at least one supported_VGPU_type that is not pass-through

New or modified helper functions:
- Helpers.GpuCapability = GPU feature not restricted (by licensing) and Pool.HasGpu
- Helpers.VGpuCapability = vGPU feature not restricted (by licensing) and Pool.HasVGpu
- Helpers.ClearwaterSp1OrGreater = API version is 2.1 or greater

The GPU dialogs are displayed as follows:
- GPU page on VM properties dialog: Visible only if VM.CanHaveGpu and the GPU feature not restricted (by licensing)
- GPU page on New VM Wizard: Visible only if VM.CanHaveGpu and the pool has GPU capability (Helpers.GpuCapability)
- GPU page on Pool properties dialog: Visible only if the pool has vGPU capability (Helpers.VGpuCapability)
- GPU tab: Visible only if the pool has GPU capability (Helpers.GpuCapability) and is Clearwater SP1 or greater
- On the GPU tab, the "Placement policy" panel: Visible only if the pool has vGPU capability (Helpers.VGpuCapability)
- On the GPU tab, the "Edit" button on the "vGPU types" panel: Visible only if the PGPU.HasVGpu and vGPU feature not restricted (by licensing)

Also:
- VM.CanHaveVGpu function renamed to CanHaveGpu
- On the GPU tab, renamed "Allowed vGPU types" to "vGPU types
2015-01-19 16:49:47 +00:00
Gabor Apati-Nagy
afbbf25325 CA-153179: Add options for customizing the config drive to the XC VM creation wizard
Todo: tidy up code & code review

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2014-12-04 18:04:43 +00:00
Stephen Turner
577363a3a1 CA-141389 Log all GET and POST calls 2014-08-07 16:23:06 +01:00
Mihaela Stoica
3c051654d6 CA-136173: Getting several notifications for one user-level action (e.g. Updating network operation creates 4 entries in the Notifications/Events view)
-  Suppress reporting of success and failure for sub-actions:
The suppress history flag is set when the action is created and if is false (by default) the action is added to the history (the Events list).
In order to suppress history for the subactions, we need change all actions used in Edit pages so their constructor can set the SuppressHistory flag and then use these constructors with suppressHistory = true on all the implementations of IEditPage.SaveSettings() where an action is created
(then we need to remember to do the same everytime we introduce a new page and / or "save" action).

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2014-07-21 11:19:04 +01:00
Mihaela Stoica
d69df44cf1 CA-137305: When creating a VM, ensure that the vGPUs are unset if not needed.
If a VM is created from snapshot which has a vGPU and "No" vGPU is selected  then the newly created VM shouldn't have a vGPU.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2014-06-30 09:44:19 +01:00
Mihaela Stoica
8f1eb99e76 CP-8120 Set cores per socket through XenCenter: Added 'Topology' field to the New VM Wizard.
- Text on CPU page - changed to include topology.
- Summary page on the New VM Wizard – changed to include topology

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2014-06-05 13:03:06 +01:00
Gabor Apati-Nagy
73fccdab61 CA-115429: New fix as per code review: Deletion of snapshot is successful but XenCenter logs show error. 2014-05-15 14:18:28 +01:00
Gabor Apati-Nagy
f511403e87 CA-115429: Removed code, because it does not needed for supported XS versions
Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2014-05-14 16:04:56 +01:00
Gabor Apati-Nagy
4ee0642a54 CA-115429: Fixed: Deletion of snapshot is successful but XenCenter logs show error.
XAPI (from 1.7) destroy destroys suspend VDIs as well (code is here: xapi: ocaml/xapi/cli_operations.ml). Depending on actual race conditions, this can lead to a situation when we try to destroy a Suspend VDI that has just been deleted.

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2014-05-14 14:41:07 +01:00
Konstantina Chremmou
c843f5b867 CP-6511: Ovf import/export for vGPU enabled VMs.
# HG changeset patch
# User Konstantina Chremmou <Konstantina.Chremmou@citrix.com>
# Date 1384181202 0
#      Mon Nov 11 14:46:42 2013 +0000
# Node ID 6810ba763a896bcf7dd4295f30c6d5f42b02e9de
# Parent  37666849a54470e0da04b8d14c17940f8b580636

Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2013-11-14 12:29:30 +00:00
Konstantina Chremmou
a879e50df1 CP-6317: Importing from branch clearwater-ln.
# HG changeset patch
# User Stephen Turner <stephen.turner@citrix.com>
# Date 1378992269 -3600
# Node ID 5b5548029247d300baddfac9f2aac881ed6cc086
# Parent  28eb1719f7f648773bd707c3cb81bb537d8f78d8
CA-114912: Unlocalised strings

Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2013-09-26 15:01:14 +01:00
Konstantina Chremmou
89fa1950a4 CP-6083: Updated test. Prevent potential NullReferenceException.
Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2013-09-20 14:33:23 +01:00
Konstantina Chremmou
8ae4eaa060 CP-6084: Added vGPUs to New VM Wizard.
Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2013-09-20 11:24:46 +01:00
Konstantina Chremmou
cab8dca36b CP-6083: Added vGPUs to the VM's property page.
Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2013-09-18 13:06:47 +01:00
Konstantina Chremmou
077cb17deb CP-5750: Changed the declaration for the ActionBase events Completed, Changed and
NewAction to use the Action<ActionBase> delegate instead of EventHandler. Thus
there is no need to fire them with Empty or null EventArgs and on several occasions
we avoid casting objects in the event handlers.

Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2013-08-09 17:20:33 +01:00
Mihaela Stoica
b7d6b03577 CA-111310: Call VDI.copy asynchronously to avoid timeout [SCTX-1366]
- Changed the synchronous call to VDI.copy to an async one in VMMoveAction and MoveVirtualAction;
- Corrected the calculation of action step in VMMoveAction.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2013-07-16 10:25:02 +01:00
Mihaela Stoica
bd36a85bff CP-4816: Initial commit to git repo
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2013-06-24 12:41:48 +01:00