Merge lp:~jtv/maas/bug-1305118 into lp:~maas-committers/maas/trunk

Proposed by Jeroen T. Vermeulen
Status: Merged
Approved by: Jeroen T. Vermeulen
Approved revision: no longer in the source branch.
Merged at revision: 2248
Proposed branch: lp:~jtv/maas/bug-1305118
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 26 lines (+3/-2)
2 files modified
src/provisioningserver/config.py (+1/-1)
src/provisioningserver/tests/test_config.py (+2/-1)
To merge this branch: bzr merge lp:~jtv/maas/bug-1305118
Reviewer Review Type Date Requested Status
Raphaël Badin (community) Approve
Review via email: mp+214966@code.launchpad.net

Commit message

For the default simplestreams path, use the -v2 data, not the old data. Otherwise, the import script breaks with a KeyError on items['subarches'].

Description of the change

Discovered this because a new maas-test change tries to make use of the default. We didn't normally notice because both the maas source tree and the packages provide a bootresources.yaml with explicit path values.

Jeroen

To post a comment you must log in.
Revision history for this message
Raphaël Badin (rvb) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/provisioningserver/config.py'
2--- src/provisioningserver/config.py 2014-04-07 09:12:10 +0000
3+++ src/provisioningserver/config.py 2014-04-09 15:02:56 +0000
4@@ -173,7 +173,7 @@
5 if_key_missing = None
6
7 path = String(
8- if_missing="http://maas.ubuntu.com/images/ephemeral/releases/")
9+ if_missing="http://maas.ubuntu.com/images/ephemeral-v2/releases/")
10 keyring = String(
11 if_missing="/usr/share/keyrings/ubuntu-cloudimage-keyring.gpg")
12 selections = ForEach(
13
14=== modified file 'src/provisioningserver/tests/test_config.py'
15--- src/provisioningserver/tests/test_config.py 2014-04-07 09:12:10 +0000
16+++ src/provisioningserver/tests/test_config.py 2014-04-09 15:02:56 +0000
17@@ -432,7 +432,8 @@
18 'sources': [
19 {
20 'path': (
21- 'http://maas.ubuntu.com/images/ephemeral/releases/'),
22+ 'http://maas.ubuntu.com/images/ephemeral-v2/releases/'
23+ ),
24 'keyring': (
25 '/usr/share/keyrings/ubuntu-cloudimage-keyring.gpg'),
26 'selections': [