Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1707168

Re: [PATCH] hv_set_ifconfig.sh double check before setting ip

Show all headers | View raw

On 08/09/2017 06:11 AM, David Miller wrote:
> From: Eduardo Otubo <otubo@redhat.com>
> Date: Tue,  8 Aug 2017 15:53:45 +0200
> 
>> This patch fixes the behavior of the hv_set_ifconfig script when setting
>> the interface ip. Sometimes the interface has already been configured by
>> network daemon, in this case hv_set_ifconfig causes "RTNETLINK: file
>> exists error"; in order to avoid this error this patch makes sure double
>> checks the interface before trying anything.
>>
>> Signed-off-by: Eduardo Otubo <otubo@redhat.com>
> 
> And if the daemon sets the address after you test it but before
> you try to set it in the script, what happens?
> 
> This is why I hate changes like this.  They don't remove the problem,
> they make it smaller.  And smaller in a bad way.  Smaller makes the
> problem even more harder to diagnose when it happens.
> 
> There is implicitly no synchonization between network configuration
> daemons and things people run by hand like this script.
> 
> So, caveat emptor.
> 
> I'm not applying this, sorry.
> 

This is just part of the resolution, actually. For RHEL I also configure 
hyperv-daemons' systemd config file to be run only after network service 
is up.

So perhaps my solution should be distro-agnostic and only involve this 
script as part of it? In this case I'll elaborate a little more then.

Thanks for the comment.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] hv_set_ifconfig.sh double check before setting ip Eduardo Otubo <otubo@redhat.com> - 2017-08-08 16:00 +0200
  Re: [PATCH] hv_set_ifconfig.sh double check before setting ip David Miller <davem@davemloft.net> - 2017-08-09 06:20 +0200
    Re: [PATCH] hv_set_ifconfig.sh double check before setting ip Eduardo Otubo <otubo@redhat.com> - 2017-08-09 11:10 +0200
      Re: [PATCH] hv_set_ifconfig.sh double check before setting ip Eduardo Otubo <otubo@redhat.com> - 2017-08-10 09:50 +0200
        Re: [PATCH] hv_set_ifconfig.sh double check before setting ip David Miller <davem@davemloft.net> - 2017-08-10 18:20 +0200

csiph-web