[Cloud-init-dev] [Merge] lp:~utlemming/cloud-init/smartos_base64 into lp:cloud-init

2013-08-23 Thread noreply
The proposal to merge lp:~utlemming/cloud-init/smartos_base64 into lp:cloud-init has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~utlemming/cloud-init/smartos_base64/+merge/177716 --

Re: [Cloud-init-dev] [Merge] lp:~utlemming/cloud-init/smartos_base64 into lp:cloud-init

2013-08-20 Thread Ben Howard
Scott, have a look at these changesI think this resolves the last of the concerns. -- https://code.launchpad.net/~utlemming/cloud-init/smartos_base64/+merge/177716 Your team cloud init development team is requested to review the proposed merge of lp:~utlemming/cloud-init/smartos_base64

Re: [Cloud-init-dev] [Merge] lp:~utlemming/cloud-init/smartos_base64 into lp:cloud-init

2013-08-15 Thread Ben Howard
the class definition has: class DataSourceSmartOS(sources.DataSource): def __init__(self, sys_cfg, distro, paths): sources.DataSource.__init__(self, sys_cfg, distro, paths) So we are going to get .ds_cfg out of it if I understand the comment. --

Re: [Cloud-init-dev] [Merge] lp:~utlemming/cloud-init/smartos_base64 into lp:cloud-init

2013-08-05 Thread Scott Moser
for getting config out of sys_config, you need to get at sysconfig['datasource']['smartos']. if you've called DataSource's __init__, it will have set .ds_cfg for you. class DataSource(object): __metaclass__ = abc.ABCMeta def __init__(self, sys_cfg, distro, paths, ud_proc=None):

[Cloud-init-dev] [Merge] lp:~utlemming/cloud-init/smartos_base64 into lp:cloud-init

2013-07-30 Thread Scott Moser
You have been requested to review the proposed merge of lp:~utlemming/cloud-init/smartos_base64 into lp:cloud-init. For more details, see: https://code.launchpad.net/~utlemming/cloud-init/smartos_base64/+merge/177716 Added base64 support for parsing meta-data values. SmartOS CLI tools will