Commit Graph

97 Commits

Author SHA1 Message Date
Mihaela Stoica
d7fa39bb75 Merge pull request #655 from agimofcarmen/CA-181442
CA 181442: Cancelling the wizard while upload is in progress does not cause an unhandled exception anymore.
2015-09-08 12:05:30 +01:00
Carmen Agimof
68750627cc CA-181443: The list in the Select Patch Page in the Updates Wizard is not dancing anymore. Changed the AutoSizeMode from Displayed Cells to None. 2015-09-07 12:24:31 +01:00
Carmen Agimof
22b3450dcd CA-181442: Make the list that contained all NewUploadedPatches a Dictionary that matches the patch with its path. This is helpful when trying to upload to a server where the update it is already uploaded (and therefore we do not do the upload again and we cannot get the patch).
Adds a flag in the UploadPatchAction that is used when checking if the file should be deleted on cancel.
2015-09-07 12:18:01 +01:00
Carmen Agimof
05884ea9be CA-181442: Cancelling the wizard while upload is in progress does not cause an unhandled exception anymore. 2015-09-04 17:18:26 +01:00
Carmen Agimof
9b0f478952 CA-181390: Solved issue related to taking a file from disk that is not in the list of updates taken from the Citrix website. Created a dictionary that contains all the uploaded updates(the patch path) and on which hosts they were applied.
- Not applied updates do not appear anymore in the General Tab of a pool. (Related to CA-180577)
- If we apply a patch that was already applied (happens when we take a file from disk that is not in the updates list taken from the Citrix website -> no metadata), on cancel we are now removing the uploaded patch from the host.
2015-09-04 17:09:22 +01:00
Carmen Agimof
6b27d7efdb CA-180831: The issues with the refresh button in the Updates wizard are now fixed. When the refresh button is clicked, the SelectFromDisk radio button will not be selected. 2015-09-02 18:24:29 +01:00
Carmen Agimof
31924f9888 CA-181316: Sort by description is now working correctly. 2015-09-02 18:21:10 +01:00
Carmen Agimof
ef5757705e CA-180581: Modified the table columns size in order for the year in the Date column to be visible. 2015-08-27 12:42:53 +01:00
Carmen Agimof
5f40353d07 Changes Tab Indexing and disables refresh and restore button while check for updates is in progress. 2015-08-10 11:25:11 +01:00
Carmen Agimof
7e3be8f707 CP-13036: Add button to restore dismissed updates in the Updates Wizard. 2015-07-23 14:04:43 +01:00
Mihaela Stoica
f58bfbb310 Merge pull request #568 from agimofcarmen/CA-176932
Makes "Remove hotfix installation files" checkbox invisible, instead …
2015-07-23 13:47:22 +01:00
Carmen Agimof
484806f7b7 Makes "Remove hotfix installation files" checkbox invisible, instead of disabled for supp packs. 2015-07-23 11:30:18 +01:00
Carmen Agimof
037acfc889 Added null reference check in Upload Page. 2015-07-21 11:40:45 +01:00
Carmen Agimof
2720d1bc23 Following code review, adds a few adjustments to the code. 2015-07-21 11:03:34 +01:00
Carmen Agimof
b919565a5a CP-13040: Assume hotfix metadata based on hotfix filename. 2015-07-20 15:27:41 +01:00
Carmen Agimof
027de45dee CA-176932: "Remove hotfix installation files" checkbox available even for supp packs. 2015-07-20 14:34:58 +01:00
Carmen Agimof
4c93dbe92e CP-13245: Check if the update is applicable to the servers on the Select Servers Page.
CA-157785: Hotfix wizard forgets which servers should be greyed out.
When clicking Download and Install update in the Updates Tab, instead of opening a new dialog, it opens the Updates Wizard on the Select Servers Page and download the file in the Upload Page.
Fixes issue regarding the upload of the update every time that the user gets to Upload Page. Now it only uploads it if it has not been uploaded before.
Fixes issue regarding the download of the update every time the user goes back to the Select Patch Page. If the user goes back, but does not change the choice of the update, the download will not be done again.
2015-07-20 14:28:44 +01:00
Carmen Agimof
d44261452e Fixes warning related to unused field. 2015-07-16 11:35:43 +01:00
Carmen Agimof
3f7fd89f87 CP-13038: Takes care of the case when the user downloads multiple files(by going back in the wizard and selecting a different update). When clicking cancel or when the install is finished, all the downloaded files are removed. 2015-07-15 16:36:37 +01:00
Carmen Agimof
149e6f8c1e CP-13038: Shows the progress of downloading a hotfix in the existing Progress Bar in the Upload Page of the Patching Wizard. 2015-07-15 13:03:11 +01:00
Carmen Agimof
82b2354a1b Merge branch 'master' of https://github.com/xenserver/xenadmin into CP-13034,_CP-13035,_CP-13037 2015-07-14 10:35:35 +01:00
Carmen Agimof
dd10561764 CP-13034 : On the Updates Wizard, list hotfixes from updates.xensource.com instead of from other servers.
CP-13035 : Add Refresh button to list of updates on the Updates wizard.
CP-13037 : Provide option from the Updates wizard to navigate to the web page of any hotfix.
2015-07-14 10:05:13 +01:00
Mihaela Stoica
26ee98c5c9 Merge branch 'CP-13039' of https://github.com/agimofcarmen/xenadmin into agimofcarmen-CP-13039
Conflicts:
	XenModel/Messages.Designer.cs
2015-07-14 10:00:11 +01:00
Carmen Agimof
ed05a16346 CP-13037 Adds the web page column.
No functionality is added to the link yet.
2015-07-09 16:59:14 +01:00
Carmen Agimof
365c4d7d6e CP-13039: Provide more obvious functionality to upload a hotfix from a file. 2015-07-09 16:32:08 +01:00
Carmen Agimof
fa61ff27c7 CP-13041 : Remove the hotfix after installing it.
Signed-off-by: Carmen Agimof <Carmen.Agimof@citrix.com>
2015-07-08 17:35: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
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
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
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
Stephen Turner
5990c53617 CA-161460: Correct help text in the case that the patch has been deleted without being installed 2015-02-24 13:42:48 +00:00
Stephen Turner
3d7db537e7 Fix line endings 2015-02-24 13:29:41 +00:00
Stephen Turner
94ec31426f CA-161460: Clear the error message when going Back and Forward 2015-02-20 13:29:00 +00:00
Stephen Turner
d4dc9e2b06 CA-161460: Rename diskSpaceErrorLinkLabel -> errorLinkLabel, as it is now used more widely 2015-02-20 13:15:40 +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
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
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
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
ed8d91a041 CP-10824: Supp Pack installation through XenCenter: Fix error in the wizard
- update the Patch property even when the selected update type is not "Existing patch". This will be set it to null if the selected type is supplemental pack.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-26 15:48:52 +00:00
Mihaela Stoica
35ab8aa825 CP-10824: Supp Pack installation through XenCenter: Delete the temporary VDI after installation
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-26 15:45:55 +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
Gabor Apati-Nagy
d50f8a7d94 Merge branch 'master' into master_github
Conflicts:
	XenModel/Messages.resx
	XenModel/XenAPI-Extensions/Host.cs
	mk/xenadmin-build.sh
2015-01-19 15:22:59 +00:00
Mihaela Stoica
ffd907cd3e CA-147163: Added missing localized resource files
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-15 13:53:03 +00:00
Mihaela Stoica
7fe521faa5 CA-147163: Make new form localizable and fix tests
Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-14 17:01:50 +00:00
Gabor Apati-Nagy
65a46c095a Merge pull request #281 from MihaelaStoica/CA-147163
CA-147163: Non-modal hotfix download and upload progress dialog
2015-01-14 16:22:51 +00:00
Mihaela Stoica
8f942c9b91 CA-147163: Non-modal hotfix upload progress dialog
Show pool (or standalone server) name on the Upload page.
Also corrected some resource strings.

Signed-off-by: Mihaela Stoica <mihaela.stoica@citrix.com>
2015-01-14 15:23:46 +00:00
FrankPetrilli
34628d9943 Fixed typo in patchingwizard
"form the list" > "from the list"
2015-01-13 15:25:20 -08:00