Re: [Cloud-init-dev] [Merge] lp:~rjschwei/cloud-init/openSUSEsupport into lp:cloud-init

2014-03-27 Thread Scott Moser
please avoid hard coded paths and instead use 'util.which'. ie: - if os.path.exists('/usr/bin/hostnamectl'): + if util.which('hostnamectl'): -- https://code.launchpad.net/~rjschwei/cloud-init/openSUSEsupport/+merge/210899 Your team cloud init development team is requested to review the

[Cloud-init-dev] [Merge] lp:~enolfc/cloud-init/opennebula-b64-userdata into lp:cloud-init

2014-03-27 Thread noreply
The proposal to merge lp:~enolfc/cloud-init/opennebula-b64-userdata into lp:cloud-init has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~enolfc/cloud-init/opennebula-b64-userdata/+merge/212646 --

Re: [Cloud-init-dev] [Merge] lp:~rjschwei/cloud-init/openSUSEsupport into lp:cloud-init

2014-03-27 Thread Scott Moser
Hi, I fixed up a few things and proposed a branch for merging into this one. https://code.launchpad.net/~smoser/cloud-init/openSUSEsupport/+merge/213075 merge that in, please. Then, the remaining issue that I see is (_exists, contents) = self._read_conf(filename) there is no '_read_conf'