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

Proposed by Chad Smith
Status: Merged
Merged at revision: 52aff7cfd5fa7df62d25e5ad853b112c49c1f600
Proposed branch: ~chad.smith/cloud-init:ubuntu/devel
Merge into: cloud-init:ubuntu/devel
Diff against target: 43 lines (+15/-2)
2 files modified
debian/changelog (+10/-0)
tests/cloud_tests/testcases/base.py (+5/-2)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
Scott Moser Approve
Review via email: mp+356591@code.launchpad.net

Commit message

Sync integration test fixes for metadata changes persisted to instance-data.json. new-upstream-release

To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

doing a build-and-push here.

review: Approve
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:52aff7cfd5fa7df62d25e5ad853b112c49c1f600
https://jenkins.ubuntu.com/server/job/cloud-init-ci/393/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    IN_PROGRESS: Declarative: Post Actions

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

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 1ebc3c4..117fd16 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,13 @@
6+cloud-init (18.4-7-g4652b196-0ubuntu1) cosmic; urgency=medium
7+
8+ * New upstream snapshot.
9+ - tests: meta_data key changed to meta-data in ec2 instance-data.json
10+ (LP: #1797231)
11+ - tests: fix kvm integration test to assert flexible config-disk path
12+ (LP: #1797199)
13+
14+ -- Chad Smith <chad.smith@canonical.com> Thu, 11 Oct 2018 09:24:46 -0600
15+
16 cloud-init (18.4-5-g6ee8a2c5-0ubuntu1) cosmic; urgency=medium
17
18 * New upstream snapshot.
19diff --git a/tests/cloud_tests/testcases/base.py b/tests/cloud_tests/testcases/base.py
20index 16b268e..fd12d87 100644
21--- a/tests/cloud_tests/testcases/base.py
22+++ b/tests/cloud_tests/testcases/base.py
23@@ -177,7 +177,7 @@ class CloudTestCase(unittest2.TestCase):
24 instance_data['base64_encoded_keys'])
25 ds = instance_data.get('ds', {})
26 v1_data = instance_data.get('v1', {})
27- metadata = ds.get('meta_data', {})
28+ metadata = ds.get('meta-data', {})
29 macs = metadata.get(
30 'network', {}).get('interfaces', {}).get('macs', {})
31 if not macs:
32@@ -262,7 +262,10 @@ class CloudTestCase(unittest2.TestCase):
33 self.assertItemsEqual([], instance_data['base64_encoded_keys'])
34 self.assertEqual('unknown', v1_data['cloud_name'])
35 self.assertEqual('nocloud', v1_data['platform'])
36- self.assertEqual('config-disk (/dev/vda)', v1_data['subplatform'])
37+ subplatform = v1_data['subplatform']
38+ self.assertIsNotNone(
39+ re.match(r'config-disk \(\/dev\/[a-z]{3}\)', subplatform),
40+ 'kvm subplatform "%s" != "config-disk (/dev/...)"' % subplatform)
41 self.assertIsNone(
42 v1_data['availability_zone'],
43 'found unexpected kvm availability_zone %s' %

Subscribers

People subscribed via source and target branches