The patch was skipped, and the "Launch XenCenter" button did not show anymore. This was because the custom property `XS_WixUIRMPressedOk` is defined within the patch.
Signed-off-by: Danilo Del Busso <danilo.delbusso@cloud.com>
* Simplified the class name by renaming CrossPoolMigrateCanMigrateFilter to CrossPoolMigrateFilter.
* CA-383483: Rewrote (again) the migration logic because the fix to CA-294370 regressed migration from local to local storage.
Also, minor refactoring to some methods for more efficient calculations.
Signed-off-by: Konstantina Chremmou <Konstantina.Chremmou@cloud.com>
* CP-40844 adds download source action that gets the latest source code in the production stage.
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 refactors download file actions
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
Adds source url to branding.sh
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP40844 refactors code and implements reviewers comments, aswell as better message handeling
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 Sorts messages
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 removes stray console logging
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 Changes to use FirstOrDefault to avoid null exceptions/errors
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 removes unessessary usings
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 code tidy up
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 Removes erroneous root tag from merge conflict resolution
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
CP-40844 sorts messages
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 fixes misnamed method and adds sourceurl parameter to brandmanager
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 Source url now uses XCUpdates url and just substitues the name of the file
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 Adds source url to xenadmin-build.ps1
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 defines log4net in each class not just inherited
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 removes unused class variable
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 removes unneeded class variable and uses parent property instead. Simplifies messages for source download and client update.
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 reduces public exposure of member variables/properties
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 merges download x file actions into single file.
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 overide ReleaseDownloadedContent in DownloadAndUpdateClientAction to handle disposal of FileStream
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 minor fixes/tidy up
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 defaults the download latest source button to be invisible.
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 if there is no client update detected then we shouldnt show a where to save dialog
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 Directs user to xenserver website to download source if automatic update checks are turned off. Renames message OUT_OF_DATE_WEBSITE to WEBSITE_DOWNLOADS
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 updates source param to sourceUrl in XCUpdates.xml
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 moves strings to Messages and applys source name string to relevant places
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 removes sourceurl from brand manager
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* CP-40844 uses string literal rather than string join
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
* Some more corrections.
Signed-off-by: Konstantina Chremmou <Konstantina.Chremmou@cloud.com>
---------
Signed-off-by: Chris Lancaster <Christopher.Lancaste1@citrix.com>
Signed-off-by: Konstantina Chremmou <Konstantina.Chremmou@cloud.com>
Co-authored-by: Konstantina Chremmou <Konstantina.Chremmou@cloud.com>
There is a chance that all actions have completed before we hit the `Wait` call, we need to make sure we don't hit a deadlock.
This can happen if for instance there is only one action, and it is a "dummy" action, such as the one used in the EUA check.
Also I have removed the compound assignment for the `volatile _completedActionsCount` since Visual Studio was flagging it as a "suspicious usage of a volatile variable". I personally don't think it's a problem but better safe than sorry.
Contains minor whitespace fixes, too
Signed-off-by: Danilo Del Busso <danilo.delbusso@cloud.com>
- Min and max should also include the defaults when looking across all templates
- Ensure exception isn't hit unnecessarily when parsing restriction value for VMs
- Allow use of `GetRestrictionValueFromMatchingTemplate` with template objects
- Parallelize `GetRestrictionValueAcrossTemplates` call
Signed-off-by: Danilo Del Busso <danilo.delbusso@cloud.com>
