CE-3036: Corrected impossible host status

To fix impossible disabled state for Hosts we needed to define possible states for hosts.

After discussions, see CE-3036 and checking with @domonkosgulyas this is what we landed in:

• Active (status = AVAILABLE)
• Active with alert (status = AVAILABLE && has warnings or errors)
• Inactive (status = UNREACHABLE && only has info alert like "no IOCs have been deployed")
• Inactive with alert (UNREACHABLE &&  has warnings or errors) which is impossible

To comply with already implemented design for statuses and alerts these are the different scenarios:

• active (available)
• alert (available with severity error)
• warning (available with severity warning)
• inactive (unreachable)
• inactiveAlert (unreachable with severity error)
• inactiveWarning (unreachable with severity warning)

Follow up:

• write tests for different status scenarios, would be good for documental reasons

requested review from @domonkosgulyas, @skybrewer, and @anderslindh1

assigned to @johannaszepanski

Maybe this should be "Inactive Alert" and the other "Inactive Warning". If only for accessibility reasons.

That is a good point. I think that we could be even more specific. When there is an alert or warning on a host the user never gets info whether a host is inactive or active. Maybe better to do "Active with alert" "Active with warning" "Inactive with alert" "Inactive with warning"

What do you think?

changed this line in version 2 of the diff

Looks good to me.

added 1 commit

• f2a41399 - removed all occurances of assigned and updated tooltip titles

Compare with previous version

added 1 commit

• 4cb9cd69 - removed all occurances of assigned and updated tooltip titles

Compare with previous version

added 1 commit

• cb0e20ac - added plural s to popover texts

Compare with previous version

Visual code

approved this merge request

mentioned in commit 6635c061

merged