Feature #6254
closed
Detector list in Event search result table is not populated by all Nodes from original IDEA
Added by Radko Krkoš over 4 years ago.
Updated over 4 years ago.
Category:
Development - GUI
- Category changed from Development - Database to Development - GUI
This is an issue in GUI only, the DB seems correct, the searching also looks good.
- Tracker changed from Bug to Feature
- Status changed from New to Feedback
- Assignee changed from Jan Mach to Radko Krkoš
- To be discussed changed from No to Yes
This is not a bug, but a design choice. The general idea is that when event is passing through relay nodes these nodes may append information about themselves to indicate that particular event passed through them. This is similar mechanism to adding Received headers in email protocol. The event search result table deliberately displays only the first item from Node[] list, because it is the originator of the message and usually what the user is interested in. Displaying information about all relay nodes would bloat the table and in most cases this information is useless. Whether this should be changed is up to the discussion, but it is a design choice, not a bug.
- Assignee changed from Radko Krkoš to Jan Mach
Jan Mach wrote:
The event search result table deliberately displays only the first item from Node[] list, because it is the originator of the message and usually what the user is interested in.
Except that in this case, that is not true. Not the first, but the second Node is displayed (sorry I have not stressed this more in the original message).
mentat_events=# SELECT node_name FROM events WHERE id = 'a52b5673-21c5-4d96-9a6d-219fe97fa7fa';
node_name
--------------------------------------------------------
{cz.cesnet.sabu_testbed.warden_filer,cz.cesnet.tarpit}
OK, no more speculations and retrieving information from memory:
{%- set tmpval = item.get_detectors() %}
{%- if tmpval %}
{{
macros_site.render_widget_csag_detector(
[tmpval[-1]],
form_data['detectors'],
align_right = True,
separate_dropdown = False
)
}}
So the idea is reverse from what I vaguely recovered from memory and stated before, last item is displayed, relays prepend. But it still is a deliberate choice (tmpval[-1]).
- Assignee changed from Jan Mach to Radko Krkoš
- Status changed from Feedback to Rejected
- Target version changed from Backlog to Rejected
Whoa, code quotations... Let's stay civil...
OK, I understand. This is still an error, but it should be fixed in the detector. Closing rejected.
Also available in: Atom
PDF