Commit 7c4c22f5 authored by Manuel Franceschini's avatar Manuel Franceschini
Browse files

gnt-node add: add error msg when using IPv6

Signed-off-by: default avatarManuel Franceschini <>
Reviewed-by: default avatarMichael Hanselmann <>
parent dc092ece
......@@ -269,20 +269,20 @@ def InitCluster(cluster_name, mac_prefix,
if not ipcls.Own(hostname.ip):
raise errors.OpPrereqError("Inconsistency: this host's name resolves"
" to %s,\nbut this ip address does not"
" belong to this host. Aborting." %
" belong to this host" %
hostname.ip, errors.ECODE_ENVIRON)
clustername = netutils.GetHostname(name=cluster_name,
if netutils.TcpPing(clustername.ip, constants.DEFAULT_NODED_PORT, timeout=5):
raise errors.OpPrereqError("Cluster IP already active. Aborting.",
raise errors.OpPrereqError("Cluster IP already active",
if not secondary_ip:
if primary_ip_version == constants.IP6_VERSION:
raise errors.OpPrereqError("When using a IPv6 primary address, a valid"
" IPv4 address must be given as secondary."
" Aborting.", errors.ECODE_INVAL)
" IPv4 address must be given as secondary",
secondary_ip = hostname.ip
if not netutils.IP4Address.IsValid(secondary_ip):
......@@ -2535,7 +2535,7 @@ class LURenameCluster(LogicalUnit):
if new_ip != old_ip:
if netutils.TcpPing(new_ip, constants.DEFAULT_NODED_PORT):
raise errors.OpPrereqError("The given cluster IP address (%s) is"
" reachable on the network. Aborting." %
" reachable on the network" %
new_ip, errors.ECODE_NOTUNIQUE) = new_name
......@@ -3668,9 +3668,10 @@ class LUAddNode(LogicalUnit):
def CheckArguments(self):
self.primary_ip_family = self.cfg.GetPrimaryIPFamily()
# validate/normalize the node name
self.hostname = netutils.GetHostname(name=self.op.node_name,
self.op.node_name =
def BuildHooksEnv(self):
......@@ -3705,6 +3706,10 @@ class LUAddNode(LogicalUnit):
node =
primary_ip = self.op.primary_ip = hostname.ip
if self.op.secondary_ip is None:
if self.primary_ip_family ==
raise errors.OpPrereqError("When using a IPv6 primary address, a valid"
" IPv4 address must be given as secondary",
self.op.secondary_ip = primary_ip
secondary_ip = self.op.secondary_ip
......@@ -6994,7 +6999,7 @@ class LUCreateInstance(LogicalUnit):
elif ip.lower() == constants.VALUE_AUTO:
if not self.op.name_check:
raise errors.OpPrereqError("IP address set to auto but name checks"
" have been skipped. Aborting.",
" have been skipped",
nic_ip = self.hostname1.ip
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment