[pve-devel] applied: [PATCH storage] fix #862: do not resolve portal adress on storage add
Wolfgang Bumiller
w.bumiller at proxmox.com
Thu Nov 15 10:12:59 CET 2018
applied
On Wed, Nov 14, 2018 at 06:11:02PM +0100, Thomas Lamprecht wrote:
> 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