UI doesn't warn about missing power management tools
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
Critical
|
Jeffrey C Jones |
Bug Description
With a fresh install of MAAS powering my NUCs, I can auto-enlist all the machines, but then get red dots next to their names… why? Because amtterm isn't installed on the cluster, so amttool doesn't exist and the cluster can't query the nodes (or power them up, for that matter).
Of course, I'm familiar with the problem, but I'd forgotten about it. Nothing in the UI tells me that this is the issue; I had to look in maas.log to find out what was going on. The errors are clear there, but they should be displayed in the UI, tool, especially since we already warn about missing power parameters. (So you can enter power parameters, the warning will go away, and the nodes will *still not work*).
Bug #1382075 also demonstrates this: ipmitool is missing on the cluster, but nothing tells the user that except the logs.
Related branches
- Andres Rodriguez (community): Approve
- Blake Rouse (community): Approve
-
Diff: 1296 lines (+668/-32)43 files modifiedsrc/maasserver/clusterrpc/power_parameters.py (+7/-3)
src/maasserver/clusterrpc/tests/test_power_parameters.py (+7/-2)
src/maasserver/rpc/tests/test_nodes.py (+2/-2)
src/maasserver/static/js/angular/controllers/node_details.js (+18/-0)
src/maasserver/static/partials/node-details.html (+9/-0)
src/maasserver/tests/test_middleware.py (+1/-1)
src/maastesting/testcase.py (+1/-1)
src/provisioningserver/drivers/__init__.py (+9/-9)
src/provisioningserver/drivers/hardware/apc.py (+4/-0)
src/provisioningserver/drivers/hardware/hmc.py (+4/-0)
src/provisioningserver/drivers/power/__init__.py (+23/-2)
src/provisioningserver/drivers/power/amt.py (+40/-0)
src/provisioningserver/drivers/power/apc.py (+8/-0)
src/provisioningserver/drivers/power/dli.py (+32/-0)
src/provisioningserver/drivers/power/ether_wake.py (+34/-0)
src/provisioningserver/drivers/power/fence_cdu.py (+32/-0)
src/provisioningserver/drivers/power/hmc.py (+8/-0)
src/provisioningserver/drivers/power/ipmi.py (+6/-0)
src/provisioningserver/drivers/power/moonshot.py (+6/-0)
src/provisioningserver/drivers/power/mscm.py (+4/-0)
src/provisioningserver/drivers/power/msftocs.py (+4/-0)
src/provisioningserver/drivers/power/seamicro.py (+6/-0)
src/provisioningserver/drivers/power/tests/test_amt.py (+51/-0)
src/provisioningserver/drivers/power/tests/test_apc.py (+15/-0)
src/provisioningserver/drivers/power/tests/test_base.py (+3/-0)
src/provisioningserver/drivers/power/tests/test_dli.py (+51/-0)
src/provisioningserver/drivers/power/tests/test_ether_wake.py (+51/-0)
src/provisioningserver/drivers/power/tests/test_fence_cdu.py (+51/-0)
src/provisioningserver/drivers/power/tests/test_hmc.py (+15/-0)
src/provisioningserver/drivers/power/tests/test_ipmi.py (+18/-1)
src/provisioningserver/drivers/power/tests/test_moonshot.py (+18/-1)
src/provisioningserver/drivers/power/tests/test_mscm.py (+6/-0)
src/provisioningserver/drivers/power/tests/test_msftocs.py (+6/-0)
src/provisioningserver/drivers/power/tests/test_seamicro.py (+18/-1)
src/provisioningserver/drivers/power/tests/test_ucsm.py (+6/-0)
src/provisioningserver/drivers/power/tests/test_virsh.py (+15/-0)
src/provisioningserver/drivers/power/tests/test_vmware.py (+15/-0)
src/provisioningserver/drivers/power/ucsm.py (+4/-0)
src/provisioningserver/drivers/power/virsh.py (+12/-0)
src/provisioningserver/drivers/power/vmware.py (+6/-0)
src/provisioningserver/drivers/tests/test_base.py (+34/-7)
src/provisioningserver/power/schema.py (+6/-0)
src/provisioningserver/rpc/clusterservice.py (+2/-2)
- Gavin Panella (community): Approve
-
Diff: 804 lines (+319/-23)15 files modifiedsrc/maasserver/api/tests/test_enlistment.py (+6/-0)
src/maasserver/api/tests/test_node.py (+3/-0)
src/maasserver/clusterrpc/power.py (+45/-3)
src/maasserver/clusterrpc/power_parameters.py (+2/-0)
src/maasserver/models/node.py (+25/-2)
src/maasserver/models/tests/test_node.py (+30/-1)
src/maasserver/static/js/angular/controllers/node_details.js (+36/-5)
src/maasserver/static/partials/node-details.html (+15/-4)
src/provisioningserver/power/change.py (+11/-0)
src/provisioningserver/power/query.py (+11/-0)
src/provisioningserver/power/tests/test_change.py (+48/-3)
src/provisioningserver/power/tests/test_query.py (+45/-4)
src/provisioningserver/rpc/cluster.py (+21/-0)
src/provisioningserver/rpc/clusterservice.py (+11/-0)
src/provisioningserver/rpc/tests/test_clusterservice.py (+10/-1)
tags: | added: confusing-ui ui |
Changed in maas: | |
milestone: | none → next |
summary: |
- UI doesn't warn about missing amtterm package for AMT nodes + UI doesn't warn about missing power management tools |
description: | updated |
Changed in maas: | |
assignee: | nobody → Graham Binns (gmb) |
status: | Triaged → In Progress |
Changed in maas: | |
assignee: | nobody → Blake Rouse (blake-rouse) |
Changed in maas: | |
importance: | Low → High |
Changed in maas: | |
importance: | High → Critical |
milestone: | next → 1.8.3 |
milestone: | 1.8.3 → 1.9.0 |
Changed in maas: | |
assignee: | Blake Rouse (blake-rouse) → nobody |
Changed in maas: | |
assignee: | nobody → Jeffrey C Jones (trapnine) |
status: | Triaged → Confirmed |
status: | Confirmed → In Progress |
Changed in maas: | |
status: | In Progress → Fix Committed |
no longer affects: | maas/1.8 |
Changed in maas: | |
status: | Fix Committed → Fix Released |
Although MAAS could warn in advance, if you try to power up your nodes the node event log will contain an error telling you about the missing package.