Merge ~chad.smith/cloud-init:ubuntu/devel into cloud-init:ubuntu/devel

[Chad Smith]

Merge upstream snapshot into ubuntu/devel for Artful.

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

PASSED: Continuous integration, rev:27b9782b37d330a991539f843d459b5ca062c4f6
https://jenkins.ubuntu.com/server/job/cloud-init-ci/401/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    SUCCESS: MAAS Compatability Testing
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/401/rebuild

review: Approve (continuous-integration)

diff --git a/cloudinit/netinfo.py b/cloudinit/netinfo.py
index 8f99d99..993b26c 100644
--- a/cloudinit/netinfo.py
+++ b/cloudinit/netinfo.py
@@ -171,7 +171,7 @@ def netdev_pformat():
     else:
         fields = ['Device', 'Up', 'Address', 'Mask', 'Scope', 'Hw-Address']
         tbl = SimpleTable(fields)
-        for (dev, d) in netdev.items():
+        for (dev, d) in sorted(netdev.items()):
             tbl.add_row([dev, d["up"], d["addr"], d["mask"], ".", d["hwaddr"]])
             if d.get('addr6'):
                 tbl.add_row([dev, d["up"],
diff --git a/cloudinit/simpletable.py b/cloudinit/simpletable.py
index 9060322..ca663cc 100644
--- a/cloudinit/simpletable.py
+++ b/cloudinit/simpletable.py
@@ -59,4 +59,4 @@ class SimpleTable(object):
         return '\n'.join(lines)
 
     def get_string(self):
-        return repr(self)
+        return self.__str__()
diff --git a/cloudinit/tests/test_netinfo.py b/cloudinit/tests/test_netinfo.py
new file mode 100644
index 0000000..7dea2e4
--- /dev/null
+++ b/cloudinit/tests/test_netinfo.py
@@ -0,0 +1,106 @@
+# This file is part of cloud-init. See LICENSE file for license information.
+
+"""Tests netinfo module functions and classes."""
+
+from cloudinit.netinfo import netdev_pformat, route_pformat
+from cloudinit.tests.helpers import CiTestCase, mock
+
+
+# Example ifconfig and route output
+SAMPLE_IFCONFIG_OUT = """\
+enp0s25   Link encap:Ethernet  HWaddr 50:7b:9d:2c:af:91
+          inet addr:192.168.2.18  Bcast:192.168.2.255  Mask:255.255.255.0
+          inet6 addr: fe80::8107:2b92:867e:f8a6/64 Scope:Link
+          UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
+          RX packets:8106427 errors:55 dropped:0 overruns:0 frame:37
+          TX packets:9339739 errors:0 dropped:0 overruns:0 carrier:0
+          collisions:0 txqueuelen:1000
+          RX bytes:4953721719 (4.9 GB)  TX bytes:7731890194 (7.7 GB)
+          Interrupt:20 Memory:e1200000-e1220000
+
+lo        Link encap:Local Loopback
+          inet addr:127.0.0.1  Mask:255.0.0.0
+          inet6 addr: ::1/128 Scope:Host
+          UP LOOPBACK RUNNING  MTU:65536  Metric:1
+          RX packets:579230851 errors:0 dropped:0 overruns:0 frame:0
+          TX packets:579230851 errors:0 dropped:0 overruns:0 carrier:0
+          collisions:0 txqueuelen:1
+"""
+
+SAMPLE_ROUTE_OUT = '\n'.join([
+    '0.0.0.0         192.168.2.1     0.0.0.0         UG        0 0          0'
+    ' enp0s25',
+    '0.0.0.0         192.168.2.1     0.0.0.0         UG        0 0          0'
+    ' wlp3s0',
+    '192.168.2.0     0.0.0.0         255.255.255.0   U         0 0          0'
+    ' enp0s25'])
+
+
+NETDEV_FORMATTED_OUT = '\n'.join([
+    '+++++++++++++++++++++++++++++++++++++++Net device info+++++++++++++++++++'
+    '++++++++++++++++++++',
+    '+---------+------+------------------------------+---------------+-------+'
+    '-------------------+',
+    '|  Device |  Up  |           Address            |      Mask     | Scope |'
+    '     Hw-Address    |',
+    '+---------+------+------------------------------+---------------+-------+'
+    '-------------------+',
+    '| enp0s25 | True |         192.168.2.18         | 255.255.255.0 |   .   |'
+    ' 50:7b:9d:2c:af:91 |',
+    '| enp0s25 | True | fe80::8107:2b92:867e:f8a6/64 |       .       |  link |'
+    ' 50:7b:9d:2c:af:91 |',
+    '|    lo   | True |          127.0.0.1           |   255.0.0.0   |   .   |'
+    '         .         |',
+    '|    lo   | True |           ::1/128            |       .       |  host |'
+    '         .         |',
+    '+---------+------+------------------------------+---------------+-------+'
+    '-------------------+'])
+
+ROUTE_FORMATTED_OUT = '\n'.join([
+    '+++++++++++++++++++++++++++++Route IPv4 info++++++++++++++++++++++++++'
+    '+++',
+    '+-------+-------------+-------------+---------------+-----------+-----'
+    '--+',
+    '| Route | Destination |   Gateway   |    Genmask    | Interface | Flags'
+    ' |',
+    '+-------+-------------+-------------+---------------+-----------+'
+    '-------+',
+    '|   0   |   0.0.0.0   | 192.168.2.1 |    0.0.0.0    |   wlp3s0  |'
+    '   UG  |',
+    '|   1   | 192.168.2.0 |   0.0.0.0   | 255.255.255.0 |  enp0s25  |'
+    '   U   |',
+    '+-------+-------------+-------------+---------------+-----------+'
+    '-------+',
+    '++++++++++++++++++++++++++++++++++++++++Route IPv6 info++++++++++'
+    '++++++++++++++++++++++++++++++',
+    '+-------+-------------+-------------+---------------+---------------+'
+    '-----------------+-------+',
+    '| Route |    Proto    |    Recv-Q   |     Send-Q    | Local Address |'
+    ' Foreign Address | State |',
+    '+-------+-------------+-------------+---------------+---------------+'
+    '-----------------+-------+',
+    '|   0   |   0.0.0.0   | 192.168.2.1 |    0.0.0.0    |       UG      |'
+    '        0        |   0   |',
+    '|   1   | 192.168.2.0 |   0.0.0.0   | 255.255.255.0 |       U       |'
+    '        0        |   0   |',
+    '+-------+-------------+-------------+---------------+---------------+'
+    '-----------------+-------+'])
+
+
+class TestNetInfo(CiTestCase):
+
+    maxDiff = None
+
+    @mock.patch('cloudinit.netinfo.util.subp')
+    def test_netdev_pformat(self, m_subp):
+        """netdev_pformat properly rendering network device information."""
+        m_subp.return_value = (SAMPLE_IFCONFIG_OUT, '')
+        content = netdev_pformat()
+        self.assertEqual(NETDEV_FORMATTED_OUT, content)
+
+    @mock.patch('cloudinit.netinfo.util.subp')
+    def test_route_pformat(self, m_subp):
+        """netdev_pformat properly rendering network device information."""
+        m_subp.return_value = (SAMPLE_ROUTE_OUT, '')
+        content = route_pformat()
+        self.assertEqual(ROUTE_FORMATTED_OUT, content)
diff --git a/cloudinit/tests/test_simpletable.py b/cloudinit/tests/test_simpletable.py
index 96bc24c..a12a62a 100644
--- a/cloudinit/tests/test_simpletable.py
+++ b/cloudinit/tests/test_simpletable.py
@@ -97,4 +97,10 @@ class TestSimpleTable(CiTestCase):
         table = SimpleTable(AUTHORIZED_KEYS_FIELDS)
         for row in AUTHORIZED_KEYS_ROWS:
             table.add_row(row)
-        self.assertEqual(str(table), AUTHORIZED_KEYS_TABLE)
+
+    def test_get_string(self):
+        """get_string() method returns the same content as str()."""
+        table = SimpleTable(AUTHORIZED_KEYS_FIELDS)
+        for row in AUTHORIZED_KEYS_ROWS:
+            table.add_row(row)
+        self.assertEqual(table.get_string(), str(table))
diff --git a/debian/changelog b/debian/changelog
index 5e4d730..26d1d45 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,11 @@
+cloud-init (17.1-18-gd4f70470-0ubuntu1) artful; urgency=medium
+
+  * New upstream snapshot.
+    - simpletable: Fix get_string method to return table-formatted string
+      (LP: #1722566)
+
+ -- Chad Smith <[e-mail]>  Tue, 10 Oct 2017 13:40:13 -0600
+
 cloud-init (17.1-17-g45d361cb-0ubuntu1) artful; urgency=medium
 
   * New upstream snapshot.