Merge ~newell-jensen/maas:display-pod-host-in-api into maas:master

Proposed by Newell Jensen
Status: Merged
Approved by: Mike Pontillo
Approved revision: 5b359daf5a7089166256cc75964d6c043403c48c
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~newell-jensen/maas:display-pod-host-in-api
Merge into: maas:master
Diff against target: 24 lines (+2/-0)
2 files modified
src/maasserver/api/pods.py (+1/-0)
src/maasserver/api/tests/test_pods.py (+1/-0)
Reviewer Review Type Date Requested Status
MAAS Lander Needs Fixing
Mike Pontillo (community) Approve
Review via email: mp+353608@code.launchpad.net

Commit message

Re-add 'host' to DISPLAYED_POD_FIELDS so that the pod's host can be seen in the API output.

To post a comment you must log in.
Revision history for this message
Mike Pontillo (mpontillo) wrote :

Looks good!

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b display-pod-host-in-api lp:~newell-jensen/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci-jenkins.internal:8080/job/maas/job/branch-tester/3773/console
COMMIT: 5b359daf5a7089166256cc75964d6c043403c48c

review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/maasserver/api/pods.py b/src/maasserver/api/pods.py
2index 599d3cc..4a2106e 100644
3--- a/src/maasserver/api/pods.py
4+++ b/src/maasserver/api/pods.py
5@@ -44,6 +44,7 @@ DISPLAYED_POD_FIELDS = (
6 'memory_over_commit_ratio',
7 'storage_pools',
8 'pool',
9+ 'host',
10 'default_macvlan_mode',
11 )
12
13diff --git a/src/maasserver/api/tests/test_pods.py b/src/maasserver/api/tests/test_pods.py
14index 216b451..5cc1da7 100644
15--- a/src/maasserver/api/tests/test_pods.py
16+++ b/src/maasserver/api/tests/test_pods.py
17@@ -126,6 +126,7 @@ class TestPodsAPI(APITestCase.ForUser, PodMixin):
18 'memory_over_commit_ratio',
19 'storage_pools',
20 'pool',
21+ 'host',
22 'default_macvlan_mode',
23 ],
24 list(parsed_result[0]))

Subscribers

People subscribed via source and target branches