This all looks fine as far as fixing the immediate problem.

I can't help but feel that there must be a better way to cooperate with 
networkmanager, because there are some situations in which it's really useful 
to have nm managing dns (vpn environments in which you want certain domains to 
resolve via one set of nameservers and everything else via another comes to 
mind), but this patch is certainly an improvement over the existing situation.

-- 
https://code.launchpad.net/~bregeer-ctl/cloud-init/+git/cloud-init/+merge/305058
Your team cloud init development team is requested to review the proposed merge 
of ~bregeer-ctl/cloud-init:bugfix/centos7_resolv into cloud-init:master.

_______________________________________________
Mailing list: https://launchpad.net/~cloud-init-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~cloud-init-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to