Diff comments:

> diff --git a/cloudinit/util.py b/cloudinit/util.py
> index eb3e589..b9c88c9 100644
> --- a/cloudinit/util.py
> +++ b/cloudinit/util.py
> @@ -286,6 +287,20 @@ class ProcessExecutionError(IOError):
>          if not hasattr(self, 'message'):
>              self.message = message
>  
> +    def _indent_text(self, text, indent_level=8):
> +        """
> +        indent text on all but the first line, allowing for easy to read 
> output
> +        """
> +        # if we are running on python3, then we need to make sure that this 
> is
> +        # a str, not bytes. on python2, this is not so important as bytes is
> +        # basically just str
> +        if six.PY3:

That makes sense, I'll get that switched over and add a test to verify the 
correct type gets passed back later today

> +            if isinstance(text, bytes):
> +                text = text.decode()
> +        # remove any newlines at end of text first to prevent unneeded blank
> +        # line in output
> +        return text.rstrip('\n').replace('\n', '\n' + ' ' * indent_level)
> +
>  
>  class SeLinuxGuard(object):
>      def __init__(self, path, recursive=False):


-- 
https://code.launchpad.net/~wesley-wiedenmeier/cloud-init/+git/cloud-init/+merge/306731
Your team cloud init development team is requested to review the proposed merge 
of ~wesley-wiedenmeier/cloud-init:error-output into cloud-init:master.

_______________________________________________
Mailing list: https://launchpad.net/~cloud-init-dev
Post to     : cloud-init-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~cloud-init-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to