Merge lp:~blake-rouse/maas/fix-1401241 into lp:~maas-committers/maas/trunk

Proposed by Blake Rouse
Status: Merged
Approved by: Blake Rouse
Approved revision: no longer in the source branch.
Merged at revision: 3416
Proposed branch: lp:~blake-rouse/maas/fix-1401241
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 26 lines (+3/-2)
2 files modified
src/provisioningserver/drivers/osystem/custom.py (+2/-1)
src/provisioningserver/drivers/osystem/tests/test_custom.py (+1/-1)
To merge this branch: bzr merge lp:~blake-rouse/maas/fix-1401241
Reviewer Review Type Date Requested Status
Blake Rouse (community) Approve
Review via email: mp+244349@code.launchpad.net

Commit message

Fix missing current subdir in boot-resources path in CustomOS.

To post a comment you must log in.
Revision history for this message
Blake Rouse (blake-rouse) wrote :

Self approving as the path was incorrect in the code and the tests. My bad!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/provisioningserver/drivers/osystem/custom.py'
2--- src/provisioningserver/drivers/osystem/custom.py 2014-12-09 15:51:13 +0000
3+++ src/provisioningserver/drivers/osystem/custom.py 2014-12-10 20:25:19 +0000
4@@ -56,7 +56,8 @@
5 def get_xinstall_parameters(self, arch, subarch, release, label):
6 """Returns the xinstall image name and type for given image."""
7 path = os.path.join(
8- BOOT_RESOURCES_STORAGE, 'custom', arch, subarch, release, label)
9+ BOOT_RESOURCES_STORAGE, 'current', 'custom',
10+ arch, subarch, release, label)
11 if os.path.exists(os.path.join(path, 'root-dd')):
12 return "root-dd", "dd-tgz"
13 else:
14
15=== modified file 'src/provisioningserver/drivers/osystem/tests/test_custom.py'
16--- src/provisioningserver/drivers/osystem/tests/test_custom.py 2014-12-09 15:51:13 +0000
17+++ src/provisioningserver/drivers/osystem/tests/test_custom.py 2014-12-10 20:25:19 +0000
18@@ -35,7 +35,7 @@
19 release = factory.make_name('release')
20 label = factory.make_name('label')
21 dirpath = os.path.join(
22- tmpdir, 'custom', arch, subarch, release, label)
23+ tmpdir, 'current', 'custom', arch, subarch, release, label)
24 os.makedirs(dirpath)
25 factory.make_file(dirpath, filename)
26 self.patch(custom, 'BOOT_RESOURCES_STORAGE', tmpdir)