[2.2] Adding a pod should add machines with their given names
Bug #1677933 reported by
Mike Pontillo
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| MAAS |
Fix Released
|
High
|
Newell Jensen | ||
Bug Description
If I use 'Add Hardware > Chassis' to add a virsh system as a chassis, I see the virtual machine (aka "domain") names I have given them via virt-manager.
If I add the same system as a pod, I see random names, and there is no easy way to figure out which is which without examining each machine's interfaces and looking at the MAC address.
Related branches
lp://staging/~newell-jensen/maas/fix-1677933
- Blake Rouse (community): Approve
- Mike Pontillo (community): Needs Information
-
Diff: 644 lines (+101/-29)14 files modifiedsrc/maasserver/api/tests/test_pods.py (+1/-0)
src/maasserver/forms/pods.py (+17/-0)
src/maasserver/forms/tests/test_pods.py (+1/-0)
src/maasserver/models/bmc.py (+13/-1)
src/maasserver/models/tests/test_bmc.py (+4/-0)
src/provisioningserver/drivers/pod/__init__.py (+2/-0)
src/provisioningserver/drivers/pod/rsd.py (+7/-4)
src/provisioningserver/drivers/pod/tests/test_base.py (+21/-8)
src/provisioningserver/drivers/pod/tests/test_rsd.py (+7/-0)
src/provisioningserver/drivers/pod/tests/test_virsh.py (+12/-8)
src/provisioningserver/drivers/pod/virsh.py (+10/-8)
src/provisioningserver/rpc/tests/test_arguments.py (+2/-0)
src/provisioningserver/rpc/tests/test_clusterservice.py (+2/-0)
src/provisioningserver/rpc/tests/test_pods.py (+2/-0)
| tags: | added: rsd |
| tags: |
added: regression removed: rsd |
| Changed in maas: | |
| importance: | Undecided → Critical |
| milestone: | none → 2.2.0rc1 |
| status: | New → Triaged |
Revision history for this message
| Blake Rouse (blake-rouse) wrote | #1 |
| Changed in maas: | |
| importance: | Critical → High |
| tags: |
added: rsd removed: regression |
Revision history for this message
| Blake Rouse (blake-rouse) wrote | #2 |
| summary: |
- [2.2] Adding a virsh pod should add machines with their given names + [2.2] Adding a pod should add machines with their given names |
| Changed in maas: | |
| assignee: | nobody → Newell Jensen (newell-jensen) |
| status: | Triaged → In Progress |
| Changed in maas: | |
| milestone: | 2.2.0rc1 → 2.2.0rc2 |
| Changed in maas: | |
| status: | In Progress → Fix Committed |
| Changed in maas: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is not Critical it does not prevent MAAS from operating correctly. This is also not a regression as it never did this before. The add_chassis operation is still present and will do the correct thing.