Bug 1000092

Summary: [RFE] Hide Removed systems by default
Product: [Retired] Beaker Reporter: Jeff Burke <jburke>
Component: web UIAssignee: Amit Saha <asaha>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.14CC: aigao, asaha, bpeck, dcallagh, ebaak, jburke, jstancek, pbunyan, qwan, rmancy, tools-bugs, xjia, xma
Target Milestone: 0.17Keywords: FutureFeature
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-06-10 23:27:50 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Comment 2 Nick Coghlan 2013-08-23 05:57:35 UTC
Currently, the Systems page displays *all* systems by default, even those that have been removed. This can be misleading, since it means the "Items count" displayed includes systems that don't actually exist.

We could improve the situation by filtering out removed systems by default, and having a checked-by-default "Exclude removed systems" checkbox in the search options to indicate that this is the case. Unchecking the checkbox would restore the current behaviour.

This approach would also generalise to other features that are likely to get a "Removed" equivalent at some point to handle foreign key constraints (like user groups, access policies and system pools).

Comment 3 Nick Coghlan 2013-08-23 05:59:21 UTC
Note: removed systems should be ignored by default on other pages as well (such as the system lists on group pages)

Comment 5 Nick Coghlan 2014-02-12 03:25:03 UTC
Adding the check box to the existing search patches may be difficult, so another potentially simpler way to do this could be to always include a "not Removed" filter on most pages that display system lists, and then have a separate page dedicated to resurrecting removed systems.

Comment 6 Nick Coghlan 2014-02-21 04:25:02 UTC
We've come up with a simpler proposal for this: filter Removed systems by default on all pages, and have a dedicated "Systems->Removed" page that can be used to resurrect a previously removed system.

Comment 7 Amit Saha 2014-03-11 02:23:37 UTC
(In reply to Nick Coghlan from comment #6)
> We've come up with a simpler proposal for this: filter Removed systems by
> default on all pages, and have a dedicated "Systems->Removed" page that can
> be used to resurrect a previously removed system.

If I understand this correctly, the proposal is to not display systems which have been removed from the following pages:

1. Group systems page
2. User systems ("My Systems")

If I have understood correctly, this hides the removed systems if I do not actively search for them. I am not sure if this is desired. Perhaps there is a better solution from the UI point of view to hide the removed systems by default but also make it clear that there *are* removed systems?

A new top level page Systems->Removed will have all the removed systems. However, on the group/user systems pages, the search can be used to find the broken systems.  

The global Systems->Available and Systems->Free pages doest not show the removed systems at present. The Systems->All page will continue to display *all* systems (broken, automated, removed, manual). So, the global existing pages will have no change.

Comment 8 Nick Coghlan 2014-03-11 07:25:58 UTC
Pages other than Systems->Removed should all have an implied "status != Removed" *even when searching*.

That even includes the Systems->All page should also change (perhaps with a name change to "Installed")

Systems->Removed would then be the only way to resurrect a previously removed system and move it back to one of the Installed states.

The question to ask is "What is the use case for caring about the existence of removed systems?". There's definitely one for admins, which is "we removed it, but now we're reinstalling it with the same details" (if the removal is only temporary for repairs, "Broken" is a better fit).

For everyone else, though, Removed systems are irrelevant - they just clutter up results and mess with system counts.

Perhaps worth starting a beaker-devel thread about it.

Comment 9 Amit Saha 2014-03-13 11:14:50 UTC
Here is a proposed patch: http://gerrit.beaker-project.org/#/c/2912/

Changes:

- A new "Systems->Removed" page and a --removed option for bkr list-systems
- All system search defaults to systems which have not been removed

Comment 10 Amit Saha 2014-03-17 02:09:51 UTC
This changes existing behavior and is best served in a non-maintenance release with a proper release note.

Comment 11 Amit Saha 2014-03-17 02:11:04 UTC
Retargetd to develop: http://gerrit.beaker-project.org/#/c/2926/

Comment 12 Amit Saha 2014-03-18 06:58:28 UTC
(In reply to Amit Saha from comment #9)
> Here is a proposed patch: http://gerrit.beaker-project.org/#/c/2912/
> 
> Changes:
> 
> - A new "Systems->Removed" page and a --removed option for bkr list-systems
> - All system search defaults to systems which have not been removed

The main use case that this RFE addresses is that Beaker users shouldn't see removed systems as part of the default search results (for eg. systems in a certain group, in all systems, etc.).  

Records of these systems will however appear on pages such as the "System Activity"  and search results of previous task executions. None of the existing records of the system are modified from the database.

Comment 15 Amit Saha 2014-04-29 05:09:25 UTC
(In reply to xuezhi ma from comment #14)

> 
> part4: Removed not listed under my systems, result: FAIL
> 1. Loaned test1 machine to myself and change its status to broken.

You probably mean "Removed". 

patch: http://gerrit.beaker-project.org/#/c/3044/1

Comment 18 Dan Callaghan 2014-06-10 23:27:50 UTC
Beaker 0.17.0 has been released.