[2.4, UI, b1] Adding pods via UI is broken because of zone
Bug #1755829 reported by
Andres Rodriguez
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
High
|
Newell Jensen |
Bug Description
Adding a pod via the UI is broken because of the zone. While testing adding zones, I came across [1]. I fixed [1] by doing [2], which fixes two issues:
1. allows maas to add a pod via the API without specifying the zone
2. allows the user to add a pod by specifying the zone *name* and not just the database *id*
3. returns zone on the API output.
Note that the above brings correctness and the same user experience across the API.
This however, breaks the UI because I believe the UI hardcodes using ids.
[1]: https:/
[2]: https:/
Related branches
~newell-jensen/maas:lp1755807-1755829
Merged
into
maas:master
- Andres Rodriguez (community): Approve
- MAAS Lander: Needs Fixing
-
Diff: 286 lines (+57/-24)5 files modifiedsrc/maasserver/api/pods.py (+3/-0)
src/maasserver/api/tests/test_pods.py (+2/-1)
src/maasserver/forms/pods.py (+14/-7)
src/maasserver/forms/tests/test_pods.py (+13/-13)
src/maasserver/websockets/handlers/pod.py (+25/-3)
Changed in maas: | |
milestone: | none → 2.4.0beta1 |
assignee: | nobody → Newell Jensen (newell-jensen) |
importance: | Undecided → High |
status: | New → Triaged |
description: | updated |
Changed in maas: | |
status: | Triaged → In Progress |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in maas: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.