Commit Graph

241 Commits

Author SHA1 Message Date
Gabor Apati-Nagy
fc0542066e CP-12641 and CP-12475: XenCenter work for thin provisioning
-Changed code, because API needs 0-100 percent values as 0..1
-Code for HBA SRs
-Changed free space check for new VDIs to condider initial disk size only
when the SR is thin provisioned

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-06-23 19:26:27 +01:00
Gabor Apati-Nagy
306a8b0acb CP-12641 and CP-12475: XenCenter work for thin provisioning
-New SR wizard: new page
-Add Virtual Disk dialog: new fields

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-06-17 09:59:21 +01:00
Gabor Apati-Nagy
fa10c0cf54 CP-11985: Implement RPU pre-check phase for safe repartitioning
testSafe2Upgrade pugin function in prepare_host_upgrade.py is called as part of the RPU pre-checks

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-06-04 15:36:39 +01:00
Gabor Apati-Nagy
76dfab006f Merge pull request #440 from MihaelaStoica/CP-11292
CP-11292: Remove storage motion constraints: Wording changes in the cros...
2015-04-22 14:44:26 +01:00
Mihaela Stoica
b7b71c786c CP-11292: Remove storage motion constraints: Fixing tests and some more wording changes
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-04-22 13:21:31 +01:00
Gabor Apati-Nagy
d082d6873b Merge pull request #437 from MihaelaStoica/CA-156200
CA-156200: HANDLE_INVALID shown on Rolling Pool Upgrade completed page
2015-04-21 18:18:17 +01:00
Mihaela Stoica
22ba1544e2 CP-11292: Remove storage motion constraints: Wording changes in the cross pool migrate wizard
- changes to the text to be suitable for migrate and copy operations, where possible
- added different text for copy on some pages
- also different text for single and multiple selection
- changes to text alignment, tab order, hotkeys on the copy pages
- removed Finish page from the intra-pool copy wizard

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-04-21 14:30:41 +01:00
Gabor Apati-Nagy
66415dc172 Merge pull request #426 from MihaelaStoica/CP-11292
CP-11292: Remove storage motion constraints: Allow cross-pool copy for stopped VMs and templates
2015-04-20 12:53:50 +01:00
Mihaela Stoica
8d0c6e64ff CA-156200: HANDLE_INVALID shown on Rolling Pool Upgrade completed page
- while upgrading a host, throw a HOST_OFFLINE ("Server could not be contacted") exception if the host can no longer be resolved (instead of HANDLE_INVALID)
- also corrected the parameters of the HANDLE_INVALID exception, as an extra parameter is needed for the friendly error name.
- in the UpdateManualHostPlanAction class I replaced Host with the private property _host in several places, so we are no longer trying to resolve the host each time we want to write something into logs
- added some null checks, to avoid reporting null reference exception

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-04-17 14:56:40 +01:00
Gabor Apati-Nagy
aefed11165 Revert "CA-149867: Invoke on MainWindow instead of various controls"
This reverts commit 92f0499911.
2015-04-16 14:17:02 +01:00
Gabor Apati-Nagy
74c8880424 Update WizardProgress.cs
Fixed a spell-check problem
2015-04-15 09:39:51 +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
1ea6bd2aa0 Merge pull request #433 from GaborApatiNagy/master_CA-45514_2
CA-45514: XenCenter looks pretty terrible under different DPI settings
2015-04-15 08:55:29 +01:00
Gabor Apati-Nagy
c333c18f24 CA-45514: XenCenter looks pretty terrible under different DPI settings
-Fixing code following the code review

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-04-14 19:26:49 +01:00
Gabor Apati-Nagy
54764570b1 CA-45514: XenCenter looks pretty terrible under different DPI settings
-Fixed the sizing of the background image in the Wizard Progress area of
wizards.
2015-04-13 11:17:57 +01:00
Gabor Apati-Nagy
80e41f19bd CA-45514: XenCenter looks pretty terrible under different DPI settings
-New Network Wizard: Changing the layout on the network type selection page to make this page "dpi aware" using FlowLayoutPanel. Additional info is shown on the bottom of the panel instead of tooltips, some code has been removed that used fix positions and moved controls around.
2015-04-13 11:17:06 +01:00
Gabor Apati-Nagy
92f0499911 CA-149867: Invoke on MainWindow instead of various controls
We observed that some threads can reach deadlock-ish state after they have Invoked into a control's UI thread. When it happens they are all in a waiting for join or in sleep state for very long time, although there should not be any deadlock situations.
It seems this has something to do with multiple parent controls and with which control we invoked on. This should not make a difference, because we have got one UI thread (for MainWindow) they should wait for, but we have seen it does.

The solution that fixed this issue was to invoke on the MainWindow instead of various controls (see a4fe507adf ).

This changeset is changing all our Invokes to invoke into MainWindow
instead of a control itself. (MainWindow's UI thread is the only UI thread
all Control is using in XenCenter)
This changeset should be in place until we have found the root cause or the exact reason for the above.
2015-04-08 15:16:10 +01:00
Mihaela Stoica
6d996c03a4 CP-11292: Remove storage motion constraints: Allow cross-pool copy for templates
- modified CopyTemplateCommand to launch a cross pool copy if permitted
- added CrossPoolCopyTemplateCommand which opens the Cross pool migration wizard in "copy" mode

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-31 17:40:05 +01:00
Mihaela Stoica
ef5249671c CP-11292: Remove storage motion constraints: Allow cross-pool copy for stopped VMs
- added a "copy mode" page to the Cross pool migrate wizard (when in copy mode) where the user can select intra-pool or cross-pool copy
- If intra-pool option is selected, then the wizard will display a page similar to what we currently have in the Copy VM dialog
- If cross-pool option is selected, then the wizard will display all the cross pool migration pages

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-31 15:13:46 +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
799d856a70 CA-148967: G11n: I18n: Un-localized issue for string "iSCSI login failed, verify CHAP credentials" in New Storage dialog.
Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-30 16:06:49 +01:00
Gabor Apati-Nagy
fd20f6f4f4 CA-148967: G11n: I18n: Un-localized issue for string "iSCSI login failed, verify CHAP credentials" in New Storage dialog.
Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-30 14:46:30 +01:00
Stephen Turner
65d9b92ec8 Merge 2015-03-27 13:38:17 +00:00
Stephen Turner
c3490265fc CA-165121: Scientific Linux icon (placeholder for the moment). Plus Ubuntu in search results. 2015-03-27 13:34:59 +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
Mihaela Stoica
cbedbd43cd CP-11032: Allow cross-pool move of stopped and suspended VMs
Changes following code review:
- add static methods to the cross pool migrate and move commands to determine if the command can execute
- In the Cross Pool migrate wizard: add Transfer network page for cross-pool move (and copy) operation (as well as all migrate cases)

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-24 17:14:03 +00:00
Gabor Apati-Nagy
b8d3ef2bd4 CA-164800: New Storage wizard: hide CIFS option for old servers and not licensed pools
-Making CIFS radio button invisible (instead of visible but disabled) when CIFS is restricted (license)

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-23 16:01:25 +00:00
Gabor Apati-Nagy
e0f4b60858 CA-164800: New Storage wizard: hide CIFS option for old servers and not licensed pools
-New SR Wizard - SR Type Selection page: radio buttons are now on a TableLayoutPanel with auto sized rows

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-23 15:48:08 +00:00
Mihaela Stoica
4e4f88ec99 Merge pull request #389 from GaborApatiNagy/CA-164186_2
CA-164186: NFSv4: end-to-end test and clean-up
2015-03-17 17:37:41 +00:00
Gabor Apati-Nagy
e900e6a3fd CA-164186: NFSv4: end-to-end test and clean-up
-Fixes following the code review (not sending NFS version when reattaching; tab orders; margins)
2015-03-17 17:33:09 +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
bfdfaaabd8 CA-164186: NFSv4: end-to-end test and clean-up
-Also calling Trim() when using user input from the Share Name field

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-16 18:17:49 +00:00
Gabor Apati-Nagy
a2a624b2a0 CA-164186: NFSv4: end-to-end test and clean-up
-NFS version selector radio buttons now belong to the Create new SR option. This section is disabled on reattaching.

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-16 15:27:49 +00:00
Gabor Apati-Nagy
6cb9ffc83f CA-164186: NFSv4: end-to-end test and clean-up
New SR Wizard:
* Removed "V" Alt-key from "NFS Version" text
* Changed the alignment of the NFS version selector radio buttons to vertical
* Fixed tab orders at all places
2015-03-16 11:26:35 +00:00
Gabor Apati-Nagy
4b2b4e627c CA-164184: Text fixes in CIFS feature
"* New SR wizard -> Type page: Add more info for CIFS (similar to what we display for NFS)
* New SR wizard -> Type page: Rename the other types to:
** Virtual disk storage: NFS, iSCSI, Hardware HBA, CIFS
** ISO library: Windows File Sharing (CIFS), NFS ISO
* Autogenerated description should say "CIFS SR" (not NFS SR)
* SR type (as displayed on the General tab for a CIFS SR) should by CIFS (not Unknown)
* New SR wizard ->  Location page for CIFS: correct tab order"

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-12 17:31:38 +00:00
Mihaela Stoica
51ee306718 Merge pull request #372 from GaborApatiNagy/CP-10889
CP-10889: XenCenter work for CAR-1720: Add support for NFS version 4
2015-03-10 13:22:21 +00:00
Gabor Apati-Nagy
c2173c6e6a CP-10889: XenCenter work for CAR-1720: Add support for NFS version 4
-Fixes following the code review

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-10 11:33:29 +00:00
Gabor Apati-Nagy
92bc32b6aa CP-10889: XenCenter work for CAR-1720: Add support for NFS version 4
-Added NFS version selectors for NFS and NFS ISO SRs in the New SR Wizard

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-09 20:05:31 +00:00
Gabor Apati-Nagy
34df0ca614 CP-11372: XenCenter: Update the model with the latest api bindings, including the new cifs specific errors
-updating Errors property in SrWizardType_Cifs with the Failures that can occur when doing CIFS SR operations

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-05 19:33:07 +00:00
Mihaela Stoica
dcd83c9cec CA-163138: Include the pre-check for VM migration status for supplemental pack installation, as the guidance for these is to restart the host after install.
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-05 10:25:30 +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
0ad4a68875 CP-10144: Support CIFS as an SR type in XenCenter
-Fixes following the code review

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2015-03-03 16:48:27 +00:00
Gabor Apati-Nagy
ea0e043dcf Merge pull request #354 from MihaelaStoica/CP-10926
CP-10926: Handle exception when running the action that gets the config drive parameters
2015-03-03 15:58:42 +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
10152683ba CP-10926: In the New VM Wizard, keep the cloud config parameters textbox editable even if the default config could not be retrieved.
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-03-03 15:22:00 +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
5559fb9049 CP-10926: Handle exception when running the action that gets the config drive parameters
- If failed to retrieve the config parameters, then make the ConfigDriveTemplate textbox read-only and display the reason why it is not editable (similar with what we do for running VMs).
- In the New VM Wizard, if the action on "Reset to Default" button fails, then popup a message box with an error saying that we were unable to retrieve the default parameters. In this case, the ConfigDriveTemplate textbox remains unchanged.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-26 14:12:16 +00:00
Mihaela Stoica
f3e8e690f8 CP-10926: Make the textbox on Cloud-Config Parameters page read-only (instead of disabled) if the config drive parameters cannot be changed
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-02-26 11:07:12 +00:00
Gabor Apati-Nagy
350804fda4 Merge pull request #347 from MihaelaStoica/CP-11124
CP-11124 Better CoreOS icon for the Container Management page
2015-02-24 16:28:32 +00:00
Mihaela Stoica
aa6e701ce2 Merge pull request #349 from stephen-turner/CA-161460
CA-161460: Correct help text in the case that the patch has been deleted without being installed
2015-02-24 13:48:29 +00:00