[pve-devel] [PATCH storage] fix #862: do not resolve portal adress on storage add

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Nov 14 18:11:02 CET 2018


as described in #862:

> I experienced a problem with ISCSI portal when using a hostname and
> not IP.
> The GUI resolves the hostname to an IP and writes it to storage.cfg.
> As my setup requires hostnames, i needed to change the config
> manually back to the hostname which is working fine.
>
> Why is this conversion done? If I enter a hostname, i want to have a
> hostname. If i enter an IP address i want to have an IP address.

This makes sense to me, a feature of using domains is that they
are/should be resolved when actually using (i.e., connecting to them)
so resolving it once on add does not seems like a good idea (if I do
not miss something - as this is a classic "imported from SVN" I do
not have any rationale to look at).

So save the work and pass it as is.

Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
 PVE/API2/Storage/Config.pm | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/PVE/API2/Storage/Config.pm b/PVE/API2/Storage/Config.pm
index b16054b..c114ddc 100755
--- a/PVE/API2/Storage/Config.pm
+++ b/PVE/API2/Storage/Config.pm
@@ -143,10 +143,6 @@ __PACKAGE__->register_method ({
 	    }
 	}
 
-	if ($param->{portal}) {
-	    $param->{portal} = PVE::Storage::resolv_portal($param->{portal});
-	}
-
 	my $plugin = PVE::Storage::Plugin->lookup($type);
 	my $opts = $plugin->check_config($storeid, $param, 1, 1);
 
-- 
2.19.1





More information about the pve-devel mailing list