Commit Graph

109 Commits

Author SHA1 Message Date
Mihaela Stoica
f0e7f5328f CA-12164: Server Status Report wizard: Add option to upload the server status report to CIS
- add the option on the destination page of the wizard
- for authentication purposes, MyCitrix credentials are required.
- these credentials are only used to obtain an upload token that will be used for the upload.
- the upload token expires in 24 hours and is not saved in the xapi database.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-06-25 23:51:11 +01:00
Mihaela Stoica
2dd4c52350 CP-12153: Call Home Authentication
- added getter and setter to the token property to prevent "Field never assigned" warning

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-06-03 15:14:39 +01:00
Mihaela Stoica
f8e47e82b5 CP-12153: Call Home Authentication
- changes following code review
- added some more logging of exceptions, also some null checks

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-06-02 16:47:19 +01:00
Mihaela Stoica
9c3982542e CP-12153: Call Home Authentication
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-06-02 14:48:59 +01:00
Mihaela Stoica
9783c6dc7e CP-12152: Call Home enrollment 2015-05-18 15:23:32 +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
Gabor Apati-Nagy
eb61f8fe0c CA-165367: Reclaim freed space feature fails on Local storage on Slave host 2015-03-27 15:06:17 +00:00
Gabor Apati-Nagy
23319daeba Merge pull request #393 from MihaelaStoica/CP-11032
CP-11032: Remove storage motion constraints: Allow cross-pool move of stopped and suspended VMs
2015-03-24 18:11:48 +00:00
Stephen Turner
1eef9002c8 Hotfix download progress bar makes GUI unresponsive 2015-03-19 14:23:54 +00: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
Gabor Apati-Nagy
35b3edd590 CA-163778: [CAR-1711] Cleanup not working
-When calling cleanup_disk_space, excluded patch is passed only when it is
not null

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-13 13:47:45 +00:00
Gabor Apati-Nagy
f6dad3c2a2 CA-163777: [CAR-1711] Incorrect message for cleanup
-Added missing parameter to get_required_space plugin call.
-The required space is not shown in the notification when we are unable to tell how much space is required .

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-11 14:49:07 +00:00
Gabor Apati-Nagy
be8cd99839 CP-11364: XC to use xapi.secret when creating CIFS SRs 2015-03-10 14:59:05 +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
c7f3ea5b30 Merge pull request #358 from GaborApatiNagy/CP-10144
CP-10144: Support CIFS as an SR type in XenCenter
2015-03-03 17:07:37 +00:00
Gabor Apati-Nagy
2d98dd9d85 CP-10144: Support CIFS as an SR type in XenCenter
-Added support for CIFS (SR type, SR Wizard)
-Added some Debug.Asserts to the Create SR action to watch for clear text paswords

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-03 15:51:56 +00:00
Mihaela Stoica
6d4e68b308 CA-161620: Changes following code review
- log the error description as Error, not Warning
- slightly changed the action title

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-03 14:51:44 +00:00
Mihaela Stoica
de4ff847c5 CA-161620: If the supplemental pack installation fails, then XenCenter will display a message suggesting the user refers to the users guide for installing via the CLI.
- When the install fails, log the full error description
- Added a specific exception for the supp pack installation failure
- Better action title (depending on the number of servers it applies to)
- Added a "More Info" button on the "Update" page, which displays a message with more information on the supp pack installation failure

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-27 17:00:03 +00:00
Mihaela Stoica
9c0bdcf08c Merge branch 'master' of https://github.com/xenserver/xenadmin into CP-10625 2015-02-25 15:50:59 +00:00
Gabor Apati-Nagy
29b7342597 Merge pull request #339 from MihaelaStoica/CA-161097
CA-161097: Pause the automatic refresh on Container processes page when the page is not visible
2015-02-25 15:15:54 +00:00
Mihaela Stoica
bf2e9bcfb1 CP-10625: Provide Xencenter user interface to enable/disable integrated GPU passthrough
- Added a "Integrated GPU passthrough" section to the PoolGpuEdit page.
- This section is only visible for a host that has GPU capability and the enabling/disabling of integrated GPU passthrough is not restricted.
- With this addition, the GPU page can now be displayed for a pooled host as well (previously only pool or standalone host)

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-25 13:01:03 +00:00
Mihaela Stoica
08d4b6e306 CA-161097: Changes following code review
- Disable the Refresh button while a refresh is in progress.
- Introduced ExecuteContainerPluginAction - an action that stores the container it was called for, so that on completion we can decide whether to update the view or not (if the container displayed has changed, then we shouldn't update the view with the results from a previously selected container).

These changes apply to container Processes and Details tabs.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-25 12:44:03 +00:00
Mihaela Stoica
8a9b2d8c70 Merge branch 'master' of https://github.com/xenserver/xenadmin into CP-10926
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>

Conflicts:
	XenModel/XenAPI-Extensions/VDI.cs
	XenModel/XenAPI-Extensions/VM.cs
2015-02-23 17:57:18 +00:00
Stephen Turner
8aae324f0a Merge 2015-02-23 14:34:35 +00:00
Stephen Turner
5adab0cfa6 CAR-1711: Download update from a pool name, not an IP address 2015-02-20 13:39:13 +00:00
Stephen Turner
a2883d8252 CA-161460: Present a helpful error message when a patch we are trying to download from another server has been deleted 2015-02-20 13:07:20 +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
1a484b99ff CP-11135: Expand patch precheck to account for OUT_OF_SPACE exception
- Pool_patch.precheck can fail with the xapi error OUT_OF_SPACE when is trying to copy the patch from master to the slaves. This happens before the patch-precheck checks if enough space is available for installation. Therefore is different from the error we are currently processing, PATCH_PRECHECK_FAILED_OUT_OF_SPACE
- If the precheck fails with OUT_OF_SPACE then we try and retrieve the disk space requirements and report a HostOutOfSpaceProblem.
- Changed the PatchPrecheckCheck to try and find problem from a xapi failure, as well as from the result of the precheck  call.
- Also removed ampersand from the the link labels (More Info and Clean up)

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-16 11:11:30 +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
Gabor Apati-Nagy
878229e9a8 CP-10798: XenCenter work for CAR-1711: Pre-Checks for hotfix installation space
-Fixed code following the code review

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-02-12 14:00:51 +00:00
Gabor Apati-Nagy
91789256e8 CP-10798: XenCenter work for CAR-1711: Pre-Checks for hotfix installation space
-Added HostOutOfSpaceProblem to let the user to clean up disk space when getting PATCH_PRECHECK_FAILED_OUT_OF_SPACE error at precheck stage in the patch install wizard.
-A bit of refactoring of DiskSpaceRequirements class in order to be reused here

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-02-12 13:13:45 +00:00
Mihaela Stoica
fe8c876a83 CP-10927: Add container restart operation
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-11 11:08:37 +00:00
Gabor Apati-Nagy
69b957cc63 Merge pull request #319 from MihaelaStoica/CA-160904
CA-160904: Fix NullReferenceException on the New VM Wizard
2015-02-10 11:45:30 +00:00
Gabor Apati-Nagy
746db783ec Merge pull request #317 from MihaelaStoica/CAR-1736
CP-10900: Add functionality for enabling/disabling/querying the containe...
2015-02-10 11:44:49 +00:00
Mihaela Stoica
640aef8c1e CP-10900: Add functionality for enabling/disabling/querying the container monitoring on a per-VM-basis to XenCenter
- Changed 'Enlightenment' to 'Container Enlightenment'

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-10 11:42:29 +00:00
Mihaela Stoica
dbebab2563 CP-10927: Internationalize existing container commands and actions
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-09 16:53:29 +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
Mihaela Stoica
c3b667ec7f CP-10900: Add functionality for enabling/disabling/querying the container monitoring on a per-VM-basis to XenCenter
New Page: Enlightenment on VM Properties Dialog, visible only for:
- VMs on Cream Or Greater hosts
- that can be enlightened, regardless of their power state

A VM can be enlightened if the key "xscontainer-monitor" exists in other_config.
A VM is enlightened if other_config:xscontainer-monitor if true.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-06 17:46:58 +00:00
Gabor Apati-Nagy
d1b6996809 Merge pull request #316 from MihaelaStoica/CP-10799
CP-10799: Ensuring sufficient disk space for service pack installation: ...
2015-02-06 15:04:55 +00:00
Gabor Apati-Nagy
2d89776d49 Merge pull request #315 from MihaelaStoica/CAR-1711
CP-10797: Ensuring sufficient disk space for service pack installation: ...
2015-02-06 13:51:36 +00:00
Mihaela Stoica
e6784d1837 CP-10799: Ensuring sufficient disk space for service pack installation: Display hotfix installation size on the Updates page
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-06 12:32:29 +00:00
Mihaela Stoica
19f66414a7 CP-10797: Ensuring sufficient disk space for service pack installation: Changes following code review
- don't update the progress bar when checking disk space (only the description)
- the color used for "in progress" action description (above the progress bar) should be the system default font color, not black
- actions should throw ArgumentNullException, not NullReferenceException

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-06 10:34:18 +00:00
Mihaela Stoica
48636e12d6 CP-10797: Ensuring sufficient disk space for service pack installation: In the Install Update Wizard check if there is enough space to upload a hotfix
- Before starting the upload to the master hosts, check if there is enough disk space (check only performed for Cream or greater hosts)
- If enough space available the upload starts automatically; otherwise an error is displayed
- If we can free up enough disk space then we offer the option to Clean up. Otherwise we provide the user with the information on required and available space and the user will have to manually free up required space.

- Also Disable the Upload page for oem updates

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-05 17:01:14 +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
f617b4ca26 CP-10621, CP-10824: Supp Pack installation through XenCenter: Changes following code review
UploadSupplementalPackAction:
- use http-put with the default timeout
- throw current exception in RemoveVDI (instead of encapsulating it in a new one)
- add other_config key before uploading to make sure the vdi has this flag set from the beginning of its lifetime
- upload progress bar now shows the total progress per row

Also included a cosmetic rewrite of PatchingWizard_PatchingPage,GetUpdateName function

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-28 09:51:25 +00:00
Mihaela Stoica
a6b58acbd2 CP-10824: Supp Pack installation through XenCenter: Add other_config entry for the temporary VDI
- add an entry in the VDI's other_config to identify it as supp pack iso, so that we can easily find these vdis and remove if needed

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-27 11:00:35 +00:00
Mihaela Stoica
5b33590df0 CP-10824: Supp Pack installation through XenCenter: Install supp pack from vdi
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-26 15:43:36 +00:00
Mihaela Stoica
11d86f1c90 CP-10621: Supp Pack installation through XenCenter: Select and upload a supp pack in the hotfix wizard
- Select a supp pack in the hotfix wizard
- Recognise that it's a supp pack not a hotfix
- Check if there's enough space on a SR to contain it: (1) default SR if shared or (2) any shared SR or (3) local SR on each host in pool
- Create a VDI to contain it
- Upload the supp pack to the VDI

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-23 12:31:44 +00:00