Merge ~bjornt/maas:bug-1772906-fix-pod-update-api into maas:master

Proposed by Björn Tillenius
Status: Merged
Approved by: Björn Tillenius
Approved revision: f1a1cdfde86f33481b60e7ef2dd9b92cf37a4018
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~bjornt/maas:bug-1772906-fix-pod-update-api
Merge into: maas:master
Diff against target: 343 lines (+64/-119)
6 files modified
src/maasserver/api/tests/test_pods.py (+20/-0)
src/maasserver/forms/pods.py (+5/-1)
src/maasserver/forms/tests/test_pods.py (+23/-12)
src/maasserver/models/tests/test_bmc.py (+2/-2)
src/maasserver/testing/factory.py (+4/-1)
src/maasserver/websockets/handlers/tests/test_pod.py (+10/-103)
Reviewer Review Type Date Requested Status
MAAS Lander Needs Fixing
Alberto Donato (community) Approve
Review via email: mp+346735@code.qastaging.launchpad.net

Commit message

LP: #1772906 - Pods can't be updated via the API without passing power_address

To post a comment you must log in.
Revision history for this message
Alberto Donato (ack) wrote :

+1

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

UNIT TESTS
-b bug-1772906-fix-pod-update-api lp:~bjornt/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci-jenkins.internal:8080/job/maas/job/branch-tester/2911/console
COMMIT: aa1df742fc57ac9acb2b9254284ce89fda6ef091

review: Needs Fixing

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches