Merge lp:~harlowja/cloud-init/fix-auto-bool into lp:~cloud-init-dev/cloud-init/trunk

[Joshua Harlow]

Remove usage of bool as string

The net_util format is that 'auto' if
it exists will be a boolean so it is likely
that the code that exists does not correctly
work in this mode (since it tries to do a
lower() call on a boolean, which will not work).

[Server Team CI bot (server-team-bot)]

FAILED: Continuous integration, rev:963
https://server-team-jenkins.canonical.com/job/cloud-init-ci/54/
Executed test runs:
    None: https://server-team-jenkins.canonical.com/job/lp-vote-on-merge/28/console

Click here to trigger a rebuild:
https://server-team-jenkins.canonical.com/job/cloud-init-ci/54/rebuild

review: Needs Fixing (continuous-integration)

[Scott Moser (smoser)]

Hello,
Thank you for taking the time to contribute to cloud-init. Cloud-init has moved its revision control system to git. As a result, we are marking all bzr merge proposals as 'rejected'. If you would like to re-submit this proposal for review, please do so by following the current HACKING documentation at http://cloudinit.readthedocs.io/en/latest/topics/hacking.html .

Unmerged revisions

963. By Joshua Harlow on 2014-03-02

Remove usage of bool as string

The net_util format is that 'auto' if
it exists will be a boolean so it is likely
that the code that exists does not correctly
work in this mode (since it tries to do a
lower() call on a boolean, which will not work).

=== modified file 'cloudinit/distros/sles.py'
--- cloudinit/distros/sles.py	2014-01-23 19:06:13 +0000
+++ cloudinit/distros/sles.py	2014-03-02 01:30:27 +0000
@@ -65,7 +65,7 @@
         for (dev, info) in entries.iteritems():
             net_fn = self.network_script_tpl % (dev)
             mode = info.get('auto')
-            if mode and mode.lower() == 'true':
+            if mode:
                 mode = 'auto'
             else:
                 mode = 'manual'