mirror of
https://github.com/xcp-ng/xenadmin.git
synced 2024-11-25 14:27:26 +01:00
4b0b83245f
* CA-34231: Replace negative language with `block list` in code and references Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `stop` in code and references Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `freezing` in code Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `run/ran` in code comments Some paramater mentions have been renamed, as they will be renamed in future commits. Excluded mentions in XenAPI/Host.cs and XenAPI/VM.cs since code is autogenerated. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # CFUValidator/CommandLineOptions/CFUCommandLineOptionManager.cs # XenAdmin/Controls/Wlb/WlbOptModeScheduler.cs # XenAdmin/Dialogs/Wlb/WorkloadReports.cs * CA-34231: Replace negative language with `run` in Command.cs Run method Used Resharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/TabPages/GeneralTabPage.cs * CA-34231: Replace negative language with `run` in Command.cs CanRun method Used Resharper rename utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `run` in Command.cs CanRunCore method Used ReSharper rename utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Commands/DRConfigureCommand.cs # XenAdmin/Commands/DRDryrunCommand.cs # XenAdmin/Commands/DRFailbackCommand.cs # XenAdmin/Commands/DRFailoverCommand.cs # XenAdmin/Commands/DisasterRecoveryCommand.cs # XenAdmin/Commands/VMGroupCommand.cs * CA-34231: Replace negative language with `run` in Command.cs RunCore method Used ReSharper rename utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `run` in Command.cs GetCantExecuteReasonCore method Used ReSharper rename utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `run` in Command.cs GetCantRunReasons method Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace remaining negative language with `run` in Command.cs Also applied to related symbols. Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `run` in methods named CanRun Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `run` in methods in CrossPoolMigrateCommand.cs Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Commands/CrossPoolMigrateCommand.cs * CA-34231: Replace negative language with `run` in remaining Run methods Also updated negative language in `DeleteVMCommand` Also updated in comment in `CrossPoolMigrateCommand` Also renamed missed instances of `CanRun` Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `run` in `CanRunCore` and `RunCore` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace remaining negative language with `run` in Commands folder Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace remaining negative language with `run` in XenAdmin/Controls folder Used ReSharper renaming utility. Had to rename `WlbReportView.xs:RunReport()` to `WlbReportView.xs:StartRunReport()` to resolve conflict with change from `WlbReportView:ExecuteReport()` to `WlbReportView:RunReport()` Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Controls/Wlb/WlbOptModeScheduler.cs # XenAdmin/Dialogs/Wlb/WorkloadReports.cs # XenAdminTests/UnitTests/WlbTests/WlbScheduledTaskTests.cs * CA-34231: Replace all remaining negative language with `run` in code Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Commands/Controls/WlbRecommendations.cs # XenAdminTests/UnitTests/WlbTests/WlbScheduledTaskTests.cs # XenModel/WLB/WlbScheduledTask.cs * CA-34231: Replace negative language with `supporter` or `bond member` in code strings Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `supporter` or `bond member` in Messages string Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/MainWindow.cs # XenAdmin/Wizards/PatchingWizard/PatchingWizard_SelectServers.cs # XenModel/Messages.Designer.cs # XenModel/Messages.resx * CA-34231: Replace remaining negative language with `supporter` in .resx files Used ReSharper renaming utility. FriendlyErrorNames.resx has been excluded. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `supporter` or `bond member` in XenAdmin code Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Dialogs/NewPoolDialog.cs * CA-34231: Replace negative language with `supporter` or `bond member` in XenModel code Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `coordinator` in CFUValidator code Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # CFUValidator/CFUValidator.cs * CA-34231: Replace negative language with `coordinator` in solution's comments and hardcoded strings Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `coordinator` in `Helpers:GetCoordinator` method Used ReSharper renaming utility. also renamed similarly named method in `ObjectChange.cs` Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Alerts/Types/HotfixEligibilityAlert.cs # XenAdmin/Controls/CustomDataGraph/ArchiveMaintainer.cs * CA-34231: Replace negative language with `coordinator` in `EvacuateHostDialog.resx` Used ReSharper renaming utility. Excluded jp and zh resources Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Dialogs/EvacuateHostDialog.resx * CA-34231: Replace negative language with `coordinator` in `EvacuateHostDialog.resx` Used ReSharper renaming utility. Excluded jp and zh resource files. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Dialogs/NewPoolDialog.resx * CA-34231: Rename missing language references for change to `coordinator` Misc files were not saved before last commits. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `coordinator` in `Messages.resx`'s code Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Commands/HACommand.cs # XenAdmin/TabPages/AdPage.cs # XenModel/Actions/Network/CreateBondAction.cs # XenModel/Actions/Network/NetworkAction.cs # XenModel/Messages.Designer.cs # XenModel/Messages.resx * CA-34231: Replace negative language with `coordinator` in `ExternalPluginAction.cs` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `coordinator` in `XenAdmin/Commands` Used ReSharper renaming utility. Also replace negative language in missed comment in `ExternalPluginAction.cs` Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Commands/AddHostToPoolCommand.cs # XenAdmin/Commands/HACommand.cs * CA-34231: Replace negative language with `coordinator` in `Helpers.cs` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `coordinator` in `NetworkingHelper.cs` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `coordinator` in `PoolJoinRules.cs` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Dialogs/NewPoolDialog.cs * CA-34231: Replace negative language with `coordinator` in `XenAdmin/Commands`, `Controls`, and `Core` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Commands/AddHostToPoolCommand.cs * CA-34231: Replace negative language with `coordinator` in `XenAdmin/Diagnostics` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `coordinator` in `XenAdmin/Dialogs` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Dialogs/NewPoolDialog.cs * CA-34231: Replace remaining negative language with `coordinator` in `XenAdmin` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/MainWindow.cs # XenAdmin/TabPages/GeneralTabPage.cs # XenAdmin/Wizards/PatchingWizard/PatchingWizard_SelectServers.cs # XenAdmin/Wizards/RollingUpgradeWizard/RollingUpgradeExtrasPage.cs # XenAdmin/Wizards/RollingUpgradeWizard/RollingUpgradeWizard.cs # XenAdmin/Wizards/RollingUpgradeWizard/RollingUpgradeWizardPrecheckPage.cs # XenAdmin/XenSearch/Columns.cs * CA-34231: Replace remaining negative language with `coordinator` in `XenModel\Actions` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenModel/Actions/Network/CreateBondAction.cs * CA-34231: Replace remaining negative language with `coordinator` in `XenModel` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace remaining negative strings with `main` in `Messages.resx` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenModel/Messages.Designer.cs # XenModel/Messages.resx * CA-34231: Replace remaining negative string names with `main` in `Messages.resx` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Dialogs/OptionsPages/SaveAndRestoreOptionsPage.cs # XenModel/Messages.resx * CA-34231: Rename dialogs to replace negative connotations with `main` Used ReSharper renaming utility Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `main` in `SaveAndResoreOptionsPage.cs` Also updated its resx file. Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> # Conflicts: # XenAdmin/Dialogs/OptionsPages/SaveAndRestoreOptionsPage.cs * CA-34231: Fix invalid reference in `SaveAndRestoreOptionsPage.Designer.cs` after renaming Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `main` in `EnterMainPasswordDialog.cs` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `main` in `ChangeMainPasswordDialog.cs` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Fix invalid reference not changed after renaming `EnterMainPasswordDialog.cs` Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace remaining negative language with `main` in `XenAdmin/Dialogs` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Update class names in HelpManager.resx Following renaming of main password dialog. Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace negative language with `main` in `Metadata.cs` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Remove remaning negative language with `coordinator` from `Messages.resx` Used ReSharper renaming Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Remove remaning negative language with `other pool member` from `Messages.resx` Used ReSharper Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CA-34231: Replace remaining negative language in solution Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Remove commented out code in VNCGraphicsClient.cs Code has only been improved partially as it's not strictly relevant to the PR Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Amend negative word replacement in XSVNCScreen.cs Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Simplify debug call formatting in NewPoolDialog.cs Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Add code mistakenly removed in MainWindow.cs Removed as part of `f155f9c8` Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Revert renaming of constant related to Windows' API Considered external to CH Center Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Amend hotkey for Coordinator field in `NewPoolDialog` Previous ALT+<key> hotkey was conflicting with CreatePool Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Remove unecessary whitespace in EvacuateHostDialog.cs Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Update if block to exclude unecessary else in DRFailoverWizard.cs Not relevant to the PR, but the change is small enought to no need extra testing Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Amend typo in PoolJoinRules.cs supporer -> supporter Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Replace missing negative language with `coordinator` in `XenAPI-Extensions/Pool.cs` Variables and methods XenAPI-Extensions were mistakenly skipped Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Replace negative language in `XenApi-Extensions` with `coordinator/interface` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Replace negative language in `XenApi-Extensions` with `supporter/member` Used ReSharper renaming utility. Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Replace remaining negative language in `XenServerHealthCheckBugTool.cs` This is only used to format the string Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Change line-endings to CRLF in `Page_CloudConfigParamters.cs` Done in order to fix merge conflict into master Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com> * CP-34231: Amend misc negative language renaming typos `XenServerHealthCheckBugTool` and `XenServerHealthCheckService` Signed-off-by: Danilo Del Busso <Danilo.Del.Busso@citrix.com>
404 lines
14 KiB
C#
404 lines
14 KiB
C#
/* Copyright (c) Citrix Systems, Inc.
|
|
* All rights reserved.
|
|
*
|
|
* Redistribution and use in source and binary forms,
|
|
* with or without modification, are permitted provided
|
|
* that the following conditions are met:
|
|
*
|
|
* * Redistributions of source code must retain the above
|
|
* copyright notice, this list of conditions and the
|
|
* following disclaimer.
|
|
* * Redistributions in binary form must reproduce the above
|
|
* copyright notice, this list of conditions and the
|
|
* following disclaimer in the documentation and/or other
|
|
* materials provided with the distribution.
|
|
*
|
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
|
|
* CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
|
|
* INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
|
|
* MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
|
|
* DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR
|
|
* CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
|
|
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
|
|
* BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
|
|
* SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
|
|
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
|
|
* WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
|
|
* NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
|
|
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
|
|
* SUCH DAMAGE.
|
|
*/
|
|
|
|
using System;
|
|
using System.Collections.Generic;
|
|
using System.Windows.Forms;
|
|
using XenAdmin.Network;
|
|
using XenAPI;
|
|
using XenAdmin.Core;
|
|
using System.Linq;
|
|
|
|
|
|
namespace XenAdmin.Dialogs
|
|
{
|
|
public partial class VIFDialog : XenDialogBase
|
|
{
|
|
private readonly VIF ExistingVif;
|
|
private readonly int Device;
|
|
private readonly bool allowSriov;
|
|
|
|
public VIFDialog(IXenConnection connection, VIF existingVif, int device, bool allowSriov = false)
|
|
: base(connection)
|
|
{
|
|
InitializeComponent();
|
|
CueBannersManager.SetWatermark(promptTextBoxMac, "aa:bb:cc:dd:ee:ff");
|
|
|
|
ExistingVif = existingVif;
|
|
Device = device;
|
|
this.allowSriov = allowSriov;
|
|
|
|
if (ExistingVif != null)
|
|
{
|
|
Text = Messages.VIRTUAL_INTERFACE_PROPERTIES;
|
|
buttonOk.Text = Messages.OK;
|
|
}
|
|
}
|
|
|
|
protected override void OnLoad(EventArgs e)
|
|
{
|
|
base.OnLoad(e);
|
|
LoadNetworks();
|
|
LoadDetails();
|
|
ValidateInput();
|
|
}
|
|
|
|
private void LoadDetails()
|
|
{
|
|
// Check if vSwitch Controller is configured for the pool (CA-46299)
|
|
Pool pool = Helpers.GetPoolOfOne(connection);
|
|
var vSwitchController = !Helpers.StockholmOrGreater(connection) && pool != null && pool.vSwitchController();
|
|
|
|
if (vSwitchController)
|
|
{
|
|
tableLayoutPanel3.Enabled = checkboxQoS.Enabled = checkboxQoS.Checked = false;
|
|
tableLayoutPanelInfo.Visible = true;
|
|
}
|
|
else
|
|
{
|
|
if (ExistingVif == null)
|
|
{
|
|
promptTextBoxQoS.Text = "";
|
|
checkboxQoS.Checked = false;
|
|
}
|
|
else
|
|
{
|
|
promptTextBoxQoS.Text = ExistingVif.LimitString();
|
|
checkboxQoS.Checked = ExistingVif.qos_algorithm_type == VIF.RATE_LIMIT_QOS_VALUE;
|
|
}
|
|
|
|
tableLayoutPanel3.Enabled = checkboxQoS.Enabled = true;
|
|
tableLayoutPanelInfo.Visible = false;
|
|
}
|
|
|
|
if (ExistingVif == null)
|
|
{
|
|
radioButtonAutogenerate.Checked = true;
|
|
return;
|
|
}
|
|
|
|
foreach (NetworkComboBoxItem item in comboBoxNetwork.Items)
|
|
{
|
|
if (item.Network != null && item.Network.opaque_ref == ExistingVif.network.opaque_ref)
|
|
comboBoxNetwork.SelectedItem = item;
|
|
}
|
|
|
|
promptTextBoxMac.Text = ExistingVif.MAC;
|
|
|
|
if (!string.IsNullOrEmpty(ExistingVif.MAC))
|
|
radioButtonMac.Checked = true;
|
|
else
|
|
radioButtonAutogenerate.Checked = true;
|
|
}
|
|
|
|
private void ValidateInput()
|
|
{
|
|
string error;
|
|
|
|
if (!IsValidNetwork(out error) || !IsValidMAc(out error) || !IsValidQoSLimit(out error))
|
|
{
|
|
buttonOk.Enabled = false;
|
|
|
|
if (string.IsNullOrEmpty(error))
|
|
{
|
|
tableLayoutPanelError.Visible = false;
|
|
}
|
|
else
|
|
{
|
|
tableLayoutPanelError.Visible = true;
|
|
pictureBoxError.Image = Images.StaticImages._000_error_h32bit_16;
|
|
labelError.Text = error;
|
|
}
|
|
}
|
|
else
|
|
{
|
|
buttonOk.Enabled = true;
|
|
tableLayoutPanelError.Visible = false;
|
|
}
|
|
}
|
|
|
|
private bool IsValidNetwork(out string error)
|
|
{
|
|
error = Messages.SELECT_NETWORK_TOOLTIP;
|
|
return comboBoxNetwork.SelectedItem is NetworkComboBoxItem item && item.Network != null;
|
|
}
|
|
|
|
private bool IsValidMAc(out string error)
|
|
{
|
|
error = null;
|
|
|
|
if (!radioButtonMac.Checked)
|
|
return true;
|
|
|
|
if (string.IsNullOrWhiteSpace(promptTextBoxMac.Text))
|
|
return false;
|
|
|
|
error = Messages.MAC_INVALID;
|
|
return Helpers.IsValidMAC(promptTextBoxMac.Text);
|
|
}
|
|
|
|
private bool IsValidQoSLimit(out string error)
|
|
{
|
|
error = null;
|
|
|
|
if (!checkboxQoS.Checked)
|
|
return true;
|
|
|
|
if (string.IsNullOrWhiteSpace(promptTextBoxQoS.Text))
|
|
return false;
|
|
|
|
error = Messages.ENTER_VALID_QOS;
|
|
|
|
if (int.TryParse(promptTextBoxQoS.Text, out var result))
|
|
return result > 0;
|
|
|
|
return false;
|
|
}
|
|
|
|
private void LoadNetworks()
|
|
{
|
|
List<XenAPI.Network> networks = new List<XenAPI.Network>(connection.Cache.Networks);
|
|
networks.Sort();
|
|
foreach (XenAPI.Network network in networks)
|
|
{
|
|
if (!network.Show(Properties.Settings.Default.ShowHiddenVMs) || network.IsMember() || (network.IsSriov() && !allowSriov))
|
|
continue;
|
|
|
|
comboBoxNetwork.Items.Add(new NetworkComboBoxItem(network));
|
|
}
|
|
|
|
if (comboBoxNetwork.Items.Count == 0)
|
|
{
|
|
comboBoxNetwork.Items.Add(new NetworkComboBoxItem(null));
|
|
}
|
|
comboBoxNetwork.SelectedIndex = 0;
|
|
}
|
|
|
|
private XenAPI.Network SelectedNetwork => (comboBoxNetwork.SelectedItem as NetworkComboBoxItem)?.Network;
|
|
|
|
private string SelectedMac => radioButtonAutogenerate.Checked ? "" : promptTextBoxMac.Text;
|
|
|
|
public VIF NewVif()
|
|
{
|
|
var vif = new VIF();
|
|
vif.Connection = connection;
|
|
vif.network = new XenRef<XenAPI.Network>(SelectedNetwork.opaque_ref);
|
|
vif.MAC = SelectedMac;
|
|
vif.device = Device.ToString();
|
|
|
|
if (checkboxQoS.Checked)
|
|
vif.qos_algorithm_type = VIF.RATE_LIMIT_QOS_VALUE;
|
|
|
|
// preserve this param even if we have decided not to turn on qos
|
|
if (!string.IsNullOrWhiteSpace(promptTextBoxQoS.Text))
|
|
vif.qos_algorithm_params = new Dictionary<string, string> {{VIF.KBPS_QOS_PARAMS_KEY, promptTextBoxQoS.Text}};
|
|
|
|
return vif;
|
|
}
|
|
|
|
/// <summary>
|
|
/// Retrieves the new settings as a vif object. You will need to set the VM field to use these settings in a vif action
|
|
/// </summary>
|
|
/// <returns></returns>
|
|
public VIF GetNewSettings()
|
|
{
|
|
var newVif = new VIF();
|
|
if (ExistingVif != null)
|
|
newVif.UpdateFrom(ExistingVif);
|
|
|
|
newVif.network = new XenRef<XenAPI.Network>(SelectedNetwork.opaque_ref);
|
|
newVif.MAC = SelectedMac;
|
|
newVif.device = Device.ToString();
|
|
|
|
if (checkboxQoS.Checked)
|
|
{
|
|
newVif.qos_algorithm_type = VIF.RATE_LIMIT_QOS_VALUE;
|
|
}
|
|
else if (ExistingVif != null && ExistingVif.qos_algorithm_type == VIF.RATE_LIMIT_QOS_VALUE)
|
|
{
|
|
newVif.qos_algorithm_type = "";
|
|
}
|
|
// else ... we leave it alone. Currently we only deal with "ratelimit" and "", don't overwrite the field if it's something else
|
|
|
|
// preserve this param even if we turn off qos
|
|
if (!string.IsNullOrEmpty(promptTextBoxQoS.Text))
|
|
newVif.qos_algorithm_params = new Dictionary<string, string> { { VIF.KBPS_QOS_PARAMS_KEY, promptTextBoxQoS.Text } };
|
|
|
|
return newVif;
|
|
}
|
|
|
|
private bool MACAddressHasChanged()
|
|
{
|
|
if (ExistingVif == null)
|
|
return true;
|
|
return promptTextBoxMac.Text != ExistingVif.MAC;
|
|
}
|
|
|
|
private bool ChangesHaveBeenMade
|
|
{
|
|
get
|
|
{
|
|
if (ExistingVif == null)
|
|
return true;
|
|
|
|
if (ExistingVif.network.opaque_ref != SelectedNetwork.opaque_ref)
|
|
return true;
|
|
|
|
if (ExistingVif.MAC != SelectedMac)
|
|
return true;
|
|
|
|
if (ExistingVif.device != Device.ToString())
|
|
return true;
|
|
|
|
if (ExistingVif.qos_algorithm_type == VIF.RATE_LIMIT_QOS_VALUE)
|
|
{
|
|
if (!checkboxQoS.Checked)
|
|
return true;
|
|
if (ExistingVif.qos_algorithm_params[VIF.KBPS_QOS_PARAMS_KEY] != promptTextBoxQoS.Text)
|
|
return true;
|
|
}
|
|
else if (string.IsNullOrEmpty(ExistingVif.qos_algorithm_type))
|
|
{
|
|
if (checkboxQoS.Checked)
|
|
return true;
|
|
}
|
|
|
|
return false;
|
|
}
|
|
}
|
|
|
|
#region Control event handlers
|
|
|
|
private void VIFDialog_FormClosing(object sender, FormClosingEventArgs e)
|
|
{
|
|
if (DialogResult == DialogResult.Cancel)
|
|
return;
|
|
|
|
if (MACAddressHasChanged())
|
|
foreach (var xenConnection in ConnectionsManager.XenConnectionsCopy.Where(c => c.IsConnected))
|
|
{
|
|
foreach (VIF vif in xenConnection.Cache.VIFs)
|
|
{
|
|
var vm = xenConnection.Resolve(vif.VM);
|
|
if (vif != ExistingVif && vif.MAC == SelectedMac && vm != null && vm.is_a_real_vm())
|
|
{
|
|
using (var dlg = new WarningDialog(string.Format(Messages.PROBLEM_MAC_ADDRESS_IS_DUPLICATE, SelectedMac, vm.NameWithLocation()),
|
|
ThreeButtonDialog.ButtonYes,
|
|
new ThreeButtonDialog.TBDButton(Messages.NO_BUTTON_CAPTION, DialogResult.No, selected: true))
|
|
{WindowTitle = Messages.PROBLEM_MAC_ADDRESS_IS_DUPLICATE_TITLE})
|
|
{
|
|
e.Cancel = dlg.ShowDialog(this) == DialogResult.No;
|
|
return;
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
if (!ChangesHaveBeenMade)
|
|
DialogResult = DialogResult.Cancel;
|
|
}
|
|
|
|
private void comboBoxNetwork_SelectedIndexChanged(object sender, EventArgs e)
|
|
{
|
|
ValidateInput();
|
|
}
|
|
|
|
private void radioButtonAutogenerate_CheckedChanged(object sender, EventArgs e)
|
|
{
|
|
if (radioButtonAutogenerate.Checked)
|
|
ValidateInput();
|
|
}
|
|
|
|
private void radioButtonMac_CheckedChanged(object sender, EventArgs e)
|
|
{
|
|
if (radioButtonMac.Checked)
|
|
ValidateInput();
|
|
}
|
|
|
|
private void promptTextBoxMac_Enter(object sender, EventArgs e)
|
|
{
|
|
radioButtonMac.Checked = true;
|
|
ValidateInput();
|
|
}
|
|
|
|
private void promptTextBoxMac_TextChanged(object sender, EventArgs e)
|
|
{
|
|
ValidateInput();
|
|
}
|
|
|
|
private void checkboxQoS_CheckedChanged(object sender, EventArgs e)
|
|
{
|
|
ValidateInput();
|
|
}
|
|
|
|
private void promptTextBoxQoS_TextChanged(object sender, EventArgs e)
|
|
{
|
|
ValidateInput();
|
|
}
|
|
|
|
private void promptTextBoxQoS_Enter(object sender, EventArgs e)
|
|
{
|
|
checkboxQoS.Checked = true;
|
|
ValidateInput();
|
|
}
|
|
|
|
#endregion
|
|
|
|
internal override string HelpName => ExistingVif == null ? "VIFDialog" : "EditVmNetworkSettingsDialog";
|
|
|
|
|
|
private class NetworkComboBoxItem : IEquatable<NetworkComboBoxItem>
|
|
{
|
|
public XenAPI.Network Network;
|
|
|
|
public NetworkComboBoxItem(XenAPI.Network network)
|
|
{
|
|
Network = network;
|
|
}
|
|
|
|
public override string ToString()
|
|
{
|
|
return Network == null ? Messages.NONE : Helpers.GetName(Network);
|
|
}
|
|
|
|
public bool Equals(NetworkComboBoxItem other)
|
|
{
|
|
if (other == null)
|
|
return false;
|
|
|
|
if (Network == null)
|
|
return other.Network == null;
|
|
|
|
return Network.Equals(other.Network);
|
|
}
|
|
}
|
|
}
|
|
} |