Merge lp:~blake-rouse/maas/fix-1384383 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: 3292
Proposed branch: lp:~blake-rouse/maas/fix-1384383
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 26 lines (+3/-0)
2 files modified
src/maasserver/api/boot_source_selections.py (+1/-0)
src/maasserver/api/tests/test_boot_source_selections.py (+2/-0)
To merge this branch: bzr merge lp:~blake-rouse/maas/fix-1384383
Reviewer Review Type Date Requested Status
Christian Reis (community) Approve
Graham Binns (community) Approve
Review via email: mp+239393@code.launchpad.net

Commit message

Show the os field in the boot source selections API.

To post a comment you must log in.
Revision history for this message
Graham Binns (gmb) :
review: Approve
Revision history for this message
Christian Reis (kiko) wrote :

+1 for 1.7 and trunk.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/maasserver/api/boot_source_selections.py'
--- src/maasserver/api/boot_source_selections.py 2014-09-04 14:51:02 +0000
+++ src/maasserver/api/boot_source_selections.py 2014-10-23 15:28:24 +0000
@@ -31,6 +31,7 @@
3131
32DISPLAYED_BOOTSOURCESELECTION_FIELDS = (32DISPLAYED_BOOTSOURCESELECTION_FIELDS = (
33 'id',33 'id',
34 'os',
34 'release',35 'release',
35 'arches',36 'arches',
36 'subarches',37 'subarches',
3738
=== modified file 'src/maasserver/api/tests/test_boot_source_selections.py'
--- src/maasserver/api/tests/test_boot_source_selections.py 2014-09-20 15:42:12 +0000
+++ src/maasserver/api/tests/test_boot_source_selections.py 2014-10-23 15:28:24 +0000
@@ -115,8 +115,10 @@
115 def test_PUT_updates_boot_source_selection(self):115 def test_PUT_updates_boot_source_selection(self):
116 self.become_admin()116 self.become_admin()
117 boot_source_selection = factory.make_BootSourceSelection()117 boot_source_selection = factory.make_BootSourceSelection()
118 new_os = factory.make_name('os')
118 new_release = factory.make_name('release')119 new_release = factory.make_name('release')
119 new_values = {120 new_values = {
121 'os': new_os,
120 'release': new_release,122 'release': new_release,
121 'arches': [factory.make_name('arch'), factory.make_name('arch')],123 'arches': [factory.make_name('arch'), factory.make_name('arch')],
122 'subarches': [124 'subarches': [