Commit Graph

4089 Commits

Author SHA1 Message Date
Mihaela Stoica
fe6144ba70 CP-17920: Add dialog for PVS cache configuration
- this commit contains a temporary image for PVS_site

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2016-09-26 17:44:17 +01:00
Mihaela Stoica
0b2f63461a Merge pull request #1181 from xenserver/GaborApatiNagy-CA-223082
CA-223082: Replace PATCH_PRECHECK_LIVEPATCH_MISSING
2016-09-26 13:33:47 +01:00
Mihaela Stoica
25a3770af0 Merge pull request #1183 from MihaelaStoica/CAR-2184-merge
Merge master into CAR-2184
2016-09-26 13:31:40 +01:00
Mihaela Stoica
f2b1443064 Merge branch 'master' into CAR-2184
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>

Conflicts:
	XenModel/Messages.Designer.cs
	XenModel/Messages.resx
2016-09-26 13:26:31 +01:00
Konstantina Chremmou
3d0705bbbc Merge pull request #1182 from MihaelaStoica/CAR-2184
CP-17920: Show the PVS tab only when the supplemental pack is installed & Hide Memory SR
2016-09-26 11:49:07 +01:00
Mihaela Stoica
fe9302a40e CP-17920: Hide memory SR in XenCenter
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2016-09-25 21:16:45 +01:00
Mihaela Stoica
151e6f42fb CP-17920: Show the PVS tab only when the supplemental pack is installed
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2016-09-23 14:51:19 +01:00
Gabor Apati-Nagy
ccaeed67cb CA-223082: Replace PATCH_PRECHECK_LIVEPATCH_MISSING
Replace PATCH_PRECHECK_LIVEPATCH_MISSING to PATCH_PRECHECK_LIVEPATCH_NOT_APPLICABLE due to changes in XAPI
Note: if XAPI change gets active before this change we will see UNKNOWN instead of PATCH_PRECHECK_LIVEPATCH_NOT_APPLICABLE in XenCenter what will make no difference
2016-09-23 13:35:02 +01:00
Mihaela Stoica
c6eab65fdf Merge pull request #1154 from kc284/master
Some corrections on VMStorage page and disk move/migrate/detach
2016-09-22 15:21:48 +01:00
mcintyre94
dd42394a25 CP-16922: Implement GUI for enabling PVS read caching (#1179)
* CP-16922: UI changes to PvsPage to replace the single configure button with a separate enable and disable one, per the design doc

* CP-16922: Intermediate checkin - command + dialog implemented, but PVS page has some layout problems.

* CP-16922:  Implement GUI for enabling PVS read caching and marking/unmarking VMs for prepopulation

Fix the broken layout by reverting MainWindow.resx (which shouldn't have changed) back to the previous commit. Also removed duplicate properties button code.

*  CP-16922: PVS Page changes, to properly display all VMs (not just ones with proxy), only populating proxy columns if there is a proxy set.

* CP-16922: Use a CommandButton for the enable button, connected to the same command as the VM main menu button, and using the table selection as its selection manager.

* CP-16922: Delete the old enable button click behaviour (because it now has a proper command), was causing the dialog to show up twice

* CP-16922: Properly list VMs by hiding those that should be hidden (in addition to non-VMs, also hide the __gui__ prefix and others covered by Show function). Also fixes a bug in NewVMGroupVMsPage discovered when fixing this - it would show VMs that should be hidden if you used the search bar.


* CP-16922: Use ButtonText on the Command to set the text on the PVS enable button

* CP-16922: Add the disable PVS read-caching command, and its associated proxy destroy action. Both based off enable equivalents (main difference is no dialog for now). Not used yet but will be in main window menu + disable button

* CP-16922: Add disable PVS read-caching to the main window menu (under enable), with the disable command attached to it


* CP-16922: Add support for disabling PVS read-caching for selected hosts via the PVS tab page by converting the disable button into a CommandButton in the same way as the enable button, using its own command + selection manager (because Selection Managers only bind to one control)

* CP-16922: Design - adding the "read caching enabled" column from the design, so that we display both whether caching is enabled and whether the VM is currently cached. If it's disabled all other values are missing.


* CP-16922: Add the PVS site column per the design to show the site a VM's PVS Proxy is on if it has a proxy


* CP-16922: Add an event handler for VMs changing so that newly provisioned VMs immediately show up in the PVS tab page


* CP-16922: Support for properly remembering the selection when the VM list is refreshed. A new method returns the list of VMs for the selected rows, and we can then check if the VM for the row being added is in this list. If there was no previous selection then we still select the first row by default.


* CP-16922: Remove prepopulate, which is being removed from the design. The next XAPI version will remove it, so just pass False as prepopulate for now, and remove the option from the dialog and PVS tab page.


* CP-16922: Improve UI appearance for a missing Storage Repository. Per the design, if there is no SR show the message None (memory only) instead of an empty cell

* CP-16922: Improved sorting behaviour. All columns (except VM name, which always uses it) use the VM tag as a sorting fallback (since they all have duplicates). New default sort which sorts by caching enabled first (yes before no), and then VM tag as a tiebreaker. Support for remembering the current sort on refresh, ie when a value changes.

* CP-16922: Refactoring to use a VM extension method to get the PVSProxy (or null if none exists) for a VM. This was used in the PVS Page + commands which are now much simpler.


* CP-16922: Move where the SelectionChanged event handler is set when refreshing the PVS table, so that the command buttons are updated correctly.

* CP-16922: Improvements to PVS read-caching dialog. Set accept/cancel buttons properly + adjust tab values, so focus acts as expected


* CP-16922: Remove the images from the menu for Enable/Disable PVS read-caching, most commands in the menu don't have one

* CP-16922: Sorting out the strings for the actions + ParallelAction calls, and moving them into the resx. Also reverting wrong MainWindow.resx changes.

* CAR-2184: XAPI changes. I've made the minimal possible changes to PvsPage to ensure it compiles, but it's had a lot of feature work (CP-16922) that supersedes this, these small changes should keep that merge painless.


* CAR-2184 XAPI: Reverse the unrelated changes, only keep the PVS ones.

* CP-16922: Merge latest XAPI changes, notably changes to Pvs_site.create

* CP-16922: Change tab show/hide logic for PVS tab, show it even if there are no PVS sites.

* CP-16922: Implement GUI for enabling PVS read caching

Design changes - remove the storage repo column and replace it with status, which is populated with a friendly version of the pvs_proxy_status enum. Also populate site list using site.name_label

Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>

*  CP-16922: Implement GUI for enabling PVS read caching

Adjusting the CanExecute of Enable PVS read-caching command to require at least one PVS site to be created. Since the PVS tab can show up without any sites, we need this because we can't create a proxy without a site.

* CP-16922: Code review changes.

Tidied up messages, and ensured actions run async properly, and without exiting before their task.

* CP-16922: Code review changes.

Update dialog resx, wider combobox on dialog, update action descriptions before setting RelatedTask, add friendly message body for PVS error notifications.
2016-09-22 15:08:15 +01:00
Mihaela Stoica
8ea8c71f74 Merge pull request #1164 from kc284/master2
Did some work on the SrStoragePage:
2016-09-22 14:12:59 +01:00
Mihaela Stoica
502f436b03 Merge pull request #1180 from mcintyre94/CA-218260
CA-218260: Reverting to Snapshots causes VM to go back to parent host
2016-09-22 14:12:05 +01:00
Callum McIntyre
336e4d8149 CA-218260: Reverting to Snapshots causes VM to go back to parent host
Move the test for whether we can boot the VM on the previous host into the VmSnapshotRevertAction, and run it using the Session of the action.
2016-09-22 13:36:43 +01:00
Callum McIntyre
d30ff533fd CA-218260: Correcting nesting
The else if (vm.power_state == vm_power_state.Suspended) block had been incorrectly moved out an indent level to the outer if.

Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>
2016-09-21 17:45:42 +01:00
Callum McIntyre
502e386e53 CA-218260: Reverting to Snapshots causes VM to go back to parent host
Where we have a host the VM is running on (ie vm.resident_on is not null) at the time it is reverted, attempt to start/resume on that host if possible, instead of the home server of the VM.

Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>
2016-09-21 17:39:03 +01:00
Konstantina Chremmou
03639d35b5 Added missing hotkey to the delete toolstripmenuitem; changed the hotkey on the move
toolstripmenuitem so it's the same as the button.

Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2016-09-20 23:12:54 +01:00
Mihaela Stoica
1b8566e149 Merge pull request #1178 from xenserver/revert-1169-master_batch
Revert "CP-18925: Hide batch hotfixing option in XenCenter"
2016-09-20 15:03:58 +01:00
Gabor Apati-Nagy
b09de72f83 Revert "CP-18925: Hide batch hotfixing option in XenCenter" 2016-09-20 15:00:06 +01:00
Mihaela Stoica
a04e8dbccf Merge pull request #1123 from GaborApatiNagy/CA-150816
CA-150816: fix: G11n:I18n: Hardcode issue "Set VM other_config" on Authentication dialog.
2016-09-20 14:02:18 +01:00
Mihaela Stoica
e5eeb29e55 Merge pull request #1170 from kc284/master3
CA-100083 (part of): Set minimum width of the main window's tab contr…
2016-09-20 13:57:39 +01:00
Konstantina Chremmou
6bb71ffd23 Merge pull request #1124 from GaborApatiNagy/CA-174979
CA-174979: Confusing right-click add to pool menu (if you accept the defaults)
2016-09-20 13:15:36 +01:00
Konstantina Chremmou
2aa8d956c0 Fixed tab order.
Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2016-09-20 13:10:02 +01:00
mcintyre94
6707139e6e CAR-2184: XAPI changes. (#1177)
* CAR-2184: XAPI changes. I've made the minimal possible changes to PvsPage to ensure it compiles, but it's had a lot of feature work (CP-16922) that supersedes this, these small changes should keep that merge painless.

Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>

* CAR-2184 XAPI: Reverse the unrelated changes, only keep the PVS ones.

Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>
2016-09-20 12:33:00 +01:00
Mihaela Stoica
963d5211d6 Merge pull request #1174 from mcintyre94/CA-217821
CA-217821: Renaming home server to target server in VM migration wizard
2016-09-20 11:43:24 +01:00
Konstantina Chremmou
a7afe39b90 Merge pull request #1175 from MihaelaStoica/CA-218242
CA-218242: Don't assign values outside the range in the memory spinne…
2016-09-20 11:26:56 +01:00
Konstantina Chremmou
6eed113ef3 Merge pull request #1176 from MihaelaStoica/CAR-2184-CA-218242
CA-218242: Fix crash in Dom0 memory dialog
2016-09-20 11:24:41 +01:00
Gabor Apati-Nagy
6742d72461 CA-174979: Confusing right-click add to pool menu (if you accept the defaults)
Removed a no longer used string from Messages.resx

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2016-09-20 10:43:05 +01:00
Mihaela Stoica
37da772c2d Merge pull request #1172 from GaborApatiNagy/master_CA-222118
CA-222118: CentOS 7 VM's  XenServer Tools installation message is mis…
2016-09-19 23:34:30 +01:00
Mihaela Stoica
a4ffaf239f CA-218242: Don't assign values outside the range in the memory spinner control
- The value of the spinner control is rounded to the nearest, while the minimum is rounded up and the maximum down (or, in some cases, min and max are not rounded at all).
In most cases this is fine, but if the current value (in bytes) is equal to one of these ranges, by rounding this way we might end up with a value that is outside of the control's ranges (causing ArgumentOutOfRange exception).

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2016-09-19 23:14:26 +01:00
Gabor Apati-Nagy
d12481fa0f CA-174979: Confusing right-click add to pool menu (if you accept the defaults)
Leaving the new pool's name field empty, forcing user to enter a name before the Next button is enabled.

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2016-09-19 18:15:44 +01:00
Gabor Apati-Nagy
93a7c3c7c7 CA-150816: fix: G11n:I18n: Hardcode issue "Set VM other_config" on Authentication dialog.
Changed the text to be more meaningful as agreed

Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2016-09-19 17:50:10 +01:00
Callum McIntyre
0c5eaef042 CA-217821: Renaming home server to target server in VM migration wizard
We already renamed the column to target server, this changes the message in the case where the user doesn't choose a server on this page to also use target server.

Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>
2016-09-19 17:00:02 +01:00
Mihaela Stoica
c86a69e3b9 CA-218242: Fix crash in Dom0 memory dialog
- reset the spinner limits before changing the value

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2016-09-19 16:11:13 +01:00
Gabor Apati-Nagy
9b383054af CA-222118: CentOS 7 VM's XenServer Tools installation message is misleading in Notifications
Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2016-09-19 14:10:21 +01:00
Konstantina Chremmou
2fdf033b25 Merge pull request #1150 from mcintyre94/CA-149965
CA-149965: Snapshot name is truncated on “Revert To Snapback” dialog.
2016-09-19 12:22:48 +01:00
Mihaela Stoica
65db5e3320 Merge pull request #1171 from mcintyre94/CA-219777
CA-219777: Bad progress dialog message
2016-09-19 12:03:57 +01:00
Callum McIntyre
6136f70d13 CA-219777: Bad progress dialog message
The bad message was in the dialog created in the BugToolWizard after selecting servers. Since this method is the only one using the ActionProgressDialog(String) constructor, I've modified that ctor to hide the subaction label - which makes sense because there is no action in that ctor (to set a subaction) and no other appropriate label.
2016-09-19 11:31:48 +01:00
Callum McIntyre
c9dc5af32d CA-149965: Snapshot name is truncated on “Revert To Snapback” dialog
Shorten the ellipsise to the first 40 characters instead of 45, which allows the ellipsis, quotation and question marks to fit correctly onto the line.
2016-09-19 10:37:48 +01:00
Mihaela Stoica
db9eea0cf6 Merge pull request #1168 from sriharshak/master
L10N: Ely:Translation check-ins
2016-09-18 07:12:27 +01:00
Konstantina Chremmou
565f95291d CA-100083 (part of): Set minimum width of the main window's tab control to 40% to
avoid buttons on the console tab overlapping. Removed some panels from the console
tab to reduce control nesting.

Signed-off-by: Konstantina Chremmou <konstantina.chremmou@citrix.com>
2016-09-17 01:23:55 +01:00
Mihaela Stoica
90cdd85ad7 Merge pull request #1169 from GaborApatiNagy/master_batch
CP-18925: Hide batch hotfixing option in XenCenter
2016-09-16 14:19:50 +01:00
Gabor Apati-Nagy
bd7d6bcc9c CP-18925: Hide batch hotfixing option in XenCenter
Signed-off-by: Gabor Apati-Nagy <gabor.apati-nagy@citrix.com>
2016-09-16 14:04:10 +01:00
Konstantina Chremmou
fb00d878b9 Merge pull request #1165 from mcintyre94/CP-18417
Cp 18417: Modify the hotfix wizard to make it work for hosts that cannot autorestart
2016-09-16 14:03:47 +01:00
Callum McIntyre
e28bcc76c9 CP-18417 review changes: Fix the broken test that uses the guidance builder method, and update the messages to be more consistent. 2016-09-16 13:36:03 +01:00
Callum McIntyre
d83a9cefb5 CP-18417 review changes: Improved PatchRequiresReboot logic. Introduced a new bool someHostMayRequireRestart, passed as an out parameter into both the guidance string builder methods, and the Build method they both use. In that Build method the parameter is set to true if any restartHost guidance is seen - this check is done before live checks exit for consistency.
Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>
2016-09-15 17:19:47 +01:00
Callum McIntyre
5d53ab3db3 CP-18417 review changes: Simpler other_config key, added a pool extension method to check for it, moved the check on the Select Servers page to be the first check, simpler logic toggling the automatic/manual buttons. Still need to rework the PatchRequiresReboot part
Signed-off-by: Callum McIntyre <callumiandavid.mcintyre@citrix.com>
2016-09-15 16:37:37 +01:00
sri harsha tagore k
49ef8a3aad L10N: Ely:Translation check-ins 2016-09-15 16:47:31 +05:30
Mihaela Stoica
6c61c0d026 Merge pull request #1158 from kc284/CA-219461
CA-219461: A new disk requires VM shutdown-restart in order to appear…
2016-09-15 09:52:29 +01:00
Gabor Apati-Nagy
00a4927f6b Merge pull request #1167 from stephen-turner/CA-220440
CA-220440: Remove CLM link from XenCenter home page
2016-09-14 15:54:41 +01:00
Stephen Turner
3154a0bd92 CA-220440: Remove CLM link from XenCenter home page 2016-09-14 15:31:21 +01:00