← Back to team overview

cloud-init-dev team mailing list archive

Re: [Merge] lp:~harlowja/cloud-init/cloud-init-net-sysconfig into lp:cloud-init

 


Diff comments:

> === modified file 'cloudinit/distros/rhel.py'
> --- cloudinit/distros/rhel.py	2015-06-02 20:27:57 +0000
> +++ cloudinit/distros/rhel.py	2016-06-14 17:59:09 +0000
> @@ -59,10 +61,16 @@
>          # should only happen say once per instance...)
>          self._runner = helpers.Runners(paths)
>          self.osfamily = 'redhat'
> +        self._net_renderer = sysconfig.Renderer()
>  
>      def install_packages(self, pkglist):
>          self.package_command('install', pkgs=pkglist)
>  
> +    def _write_network_config(self, netconfig):
> +        self._net_renderer.render_network_state(
> +            target="/", parse_net_config_data(netconfig))

Yup, there are differences; I can move those differences to be arguments options passed to the Renderer() __init__ method, or do them here but they will likely always be needed, thoughts?

> +        return []
> +
>      def _write_network(self, settings):
>          # TODO(harlowja) fix this... since this is the ubuntu format
>          entries = net_util.translate_network(settings)


-- 
https://code.launchpad.net/~harlowja/cloud-init/cloud-init-net-sysconfig/+merge/297115
Your team cloud init development team is requested to review the proposed merge of lp:~harlowja/cloud-init/cloud-init-net-sysconfig into lp:cloud-init.


References