[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [PATCH] Copy /etc/dhclient-DEV.conf file to target system (#476364)



> @@ -614,6 +612,12 @@ class Network:
>                  destkey = "%s/keys-%s" % (netscripts, device,)
>                  shutil.move(newkey, destkey)
>  
> +            # /etc/dhclient-DEVICE.conf
> +            dhclientconf = '/etc/dhclient-' + device + '.conf'
> +            if os.path.isfile(dhclientconf):
> +                destdhclientconf = '%s%s' % (instPath, dhclientconf,)
> +                shutil.copy(dhclientconf, destdhclientconf)
> +
>          # /etc/sysconfig/network
>          if not os.path.isfile(destnetwork):
>              newnetwork = "%s.new" % (destnetwork,)

I know how much we hate catching exceptions, but perhaps you want to
try...except around shutil.copy?

Otherwise, looks fine.

- Chris


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]