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
diff --git a/debian/changelog b/debian/changelog
index 1ebc3c4..117fd16 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,13 @@
1cloud-init (18.4-7-g4652b196-0ubuntu1) cosmic; urgency=medium
2
3 * New upstream snapshot.
4 - tests: meta_data key changed to meta-data in ec2 instance-data.json
5 (LP: #1797231)
6 - tests: fix kvm integration test to assert flexible config-disk path
7 (LP: #1797199)
8
9 -- Chad Smith <chad.smith@canonical.com> Thu, 11 Oct 2018 09:24:46 -0600
10
1cloud-init (18.4-5-g6ee8a2c5-0ubuntu1) cosmic; urgency=medium11cloud-init (18.4-5-g6ee8a2c5-0ubuntu1) cosmic; urgency=medium
212
3 * New upstream snapshot.13 * New upstream snapshot.
diff --git a/tests/cloud_tests/testcases/base.py b/tests/cloud_tests/testcases/base.py
index 16b268e..fd12d87 100644
--- a/tests/cloud_tests/testcases/base.py
+++ b/tests/cloud_tests/testcases/base.py
@@ -177,7 +177,7 @@ class CloudTestCase(unittest2.TestCase):
177 instance_data['base64_encoded_keys'])177 instance_data['base64_encoded_keys'])
178 ds = instance_data.get('ds', {})178 ds = instance_data.get('ds', {})
179 v1_data = instance_data.get('v1', {})179 v1_data = instance_data.get('v1', {})
180 metadata = ds.get('meta_data', {})180 metadata = ds.get('meta-data', {})
181 macs = metadata.get(181 macs = metadata.get(
182 'network', {}).get('interfaces', {}).get('macs', {})182 'network', {}).get('interfaces', {}).get('macs', {})
183 if not macs:183 if not macs:
@@ -262,7 +262,10 @@ class CloudTestCase(unittest2.TestCase):
262 self.assertItemsEqual([], instance_data['base64_encoded_keys'])262 self.assertItemsEqual([], instance_data['base64_encoded_keys'])
263 self.assertEqual('unknown', v1_data['cloud_name'])263 self.assertEqual('unknown', v1_data['cloud_name'])
264 self.assertEqual('nocloud', v1_data['platform'])264 self.assertEqual('nocloud', v1_data['platform'])
265 self.assertEqual('config-disk (/dev/vda)', v1_data['subplatform'])265 subplatform = v1_data['subplatform']
266 self.assertIsNotNone(
267 re.match(r'config-disk \(\/dev\/[a-z]{3}\)', subplatform),
268 'kvm subplatform "%s" != "config-disk (/dev/...)"' % subplatform)
266 self.assertIsNone(269 self.assertIsNone(
267 v1_data['availability_zone'],270 v1_data['availability_zone'],
268 'found unexpected kvm availability_zone %s' %271 'found unexpected kvm availability_zone %s' %

Subscribers

People subscribed via source and target branches