Values missing from controller websocket model

Bug #1967704 reported by Caleb Ellis
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
High
Christian Grabowski

Bug Description

The underlying controller model has been updated and no longer includes a few values we use in the legacy controller details UI. As far as I can tell, the removed values are:
- permissions
- status_message
- cpu_test_status
- memory_test_status
- interface_test_status
- network_test_status
- other_test_status
- storage_test_status

The ones we need back because they're used by the UI are:
- permissions

I think this might be the relevant commit: https://git.launchpad.net/maas/commit/?id=22a3211ac812a0f9414bb06eb2bf725a6f276b96

Related branches

Revision history for this message
Alberto Donato (ack) wrote :

Why are the *_test_status needed given that we don't run tests for controllers?

Changed in maas:
status: New → Incomplete
Revision history for this message
Caleb Ellis (caleb-ellis) wrote :

Hmm, I was just going by what the legacy code expects - it surfaces the number of failed, pending and successful tests for each category in the controller details cards. Was there a point in the past when controllers were tested?

Either way, I suppose that should all be removed, and in that case it seems like we only need "permissions".

Changed in maas:
status: Incomplete → Opinion
status: Opinion → New
Alberto Donato (ack)
Changed in maas:
status: New → Triaged
importance: Undecided → High
milestone: none → 3.2
description: updated
Changed in maas:
assignee: nobody → Christian Grabowski (cgrabowski)
status: Triaged → In Progress
Changed in maas:
status: In Progress → Fix Committed
Changed in maas:
milestone: 3.2.0 → 3.2.0-beta4
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.