[2.2+] pods don't have availability zones
Bug #1706438 reported by
Jason Hobbs
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| MAAS |
Fix Released
|
Wishlist
|
Newell Jensen | ||
Bug Description
At least for virsh pods, where a pod is one physical machine, pods should have a 'zone' field, and all of the machines in a pod should be associated with the zone of the pod automatically.
Right now, MAAS will try to create machines in a pod using whatever zone is requested in the 'allocate' API call. In reality, a pod is in a specific zone and this doesn't make sense. MAAS should only allocate from the pod if no zone was requested or if the pod's zone matches the allocate request's zone. See bug 1706196.
Related branches
~newell-jensen/maas:ui-pod-zones
- Andres Rodriguez (community): Needs Fixing
- MAAS Lander: Needs Fixing
- Lee Trager (community): Approve
-
Diff: 693 lines (+221/-35)10 files modifiedsrc/maasserver/api/tests/test_pods.py (+3/-0)
src/maasserver/forms/pods.py (+10/-3)
src/maasserver/forms/tests/test_pods.py (+49/-7)
src/maasserver/static/js/angular/controllers/pod_details.js (+35/-7)
src/maasserver/static/js/angular/controllers/pods_list.js (+7/-5)
src/maasserver/static/js/angular/controllers/tests/test_pod_details.js (+77/-4)
src/maasserver/static/js/angular/controllers/tests/test_pods_list.js (+9/-5)
src/maasserver/static/partials/pod-details.html (+25/-4)
src/maasserver/static/partials/pods-list.html (+2/-0)
src/maasserver/websockets/handlers/tests/test_pod.py (+4/-0)
~newell-jensen/maas:pod-zones
- MAAS Lander: Needs Fixing
- Mike Pontillo (community): Approve
-
Diff: 236 lines (+107/-7)6 files modifiedsrc/maasserver/api/machines.py (+9/-2)
src/maasserver/api/tests/test_machines.py (+38/-0)
src/maasserver/migrations/builtin/maasserver/0147_pod_zones.py (+25/-0)
src/maasserver/models/bmc.py (+17/-2)
src/maasserver/models/tests/test_bmc.py (+15/-1)
src/maasserver/websockets/handlers/tests/test_pod.py (+3/-2)
| description: | updated |
| Changed in maas: | |
| status: | New → Triaged |
| importance: | Undecided → Wishlist |
| summary: |
- pods don't have availability zones + [2.2+] pods don't have availability zones |
| description: | updated |
| tags: | added: foundations-engine |
| tags: | added: internal |
| Changed in maas: | |
| milestone: | none → 2.4.x |
| assignee: | nobody → Newell Jensen (newell-jensen) |
| tags: | added: pod |
| Changed in maas: | |
| status: | Triaged → In Progress |
| Changed in maas: | |
| status: | In Progress → Confirmed |
| Changed in maas: | |
| status: | Confirmed → In Progress |
| Changed in maas: | |
| status: | In Progress → Fix Committed |
| Changed in maas: | |
| milestone: | 2.4.x → 2.4.0alpha2 |
| Changed in maas: | |
| status: | Fix Committed → 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.
