Project

General

Profile

Actions

Bug #3064

closed

Display of referenced objects in Event Logs is very poor

Added by Jonathan CLARKE about 10 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
2
Category:
Web - Maintenance
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Regression:

Description

#1686 introduced an improvement to displaying referenced objects in Event Logs by replacing their UUIDs by strings like "directive(Directive's Name)" or "group(All SLES)" or "directive(Configure Time Settings)" or node(49ee7b99-00f5-466a-880a-d2cf27010055).

This display is clearly not satisfactory - one is not trying to practice reading code when looking at event logs.

These labels should be changed as follows:
  • directive(Directive's Name) --> Directive "Directive's Name" (UUID: <uuid>)
  • rule(Rule name) --> Rule "Rule name (UUID: <uuid>)
  • group(All SLES) --> Group "All SLES (UUID: <uuid>)
  • node(49ee7b99-00f5-466a-880a-d2cf27010055) --> Node "<hostname>" (UUID: 49ee7b99-00f5-466a-880a-d2cf27010055)

In each case, the object's name should be a clickable link to display that exact object in the web browser.

Actions #1

Updated by Nicolas PERRON about 10 years ago

  • Target version changed from 2.4.0 to 61
Actions #2

Updated by François ARMAND about 10 years ago

  • Target version changed from 61 to 2.4.2
Actions #3

Updated by Nicolas PERRON about 10 years ago

  • Target version changed from 2.4.2 to 2.4.3
Actions #4

Updated by François ARMAND about 10 years ago

  • Assignee set to Vincent MEMBRÉ
Actions #5

Updated by Vincent MEMBRÉ about 10 years ago

  • Status changed from 8 to Discussion
  • Assignee changed from Vincent MEMBRÉ to Jonathan CLARKE

As we use "Rudder ID" everywhere else in the webapp, what do you think of using "Rudder ID" instead of "UUID"?

for the same reason, UUIDs should be displayed as uppercase.

There is no Rule referenced in the event logs so the changes would be :

  • directive(Directive's Name) --> Directive "Directive's Name" (Rudder ID: <UUID>)
  • group(All SLES) --> Group "All SLES" (Rudder ID: <UUID>)
  • node(49ee7b99-00f5-466a-880a-d2cf27010055) --> Node "<hostname>" (Rudder ID: 49EE7B99-00F5-466A-880A-D2CF27010055)

With links on the name, not on the ID

Actions #6

Updated by Vincent MEMBRÉ about 10 years ago

I made a pull request with those modification : https://github.com/Normation/rudder/pull/70

Actions #7

Updated by Jonathan CLARKE about 10 years ago

  • Target version changed from 2.4.3 to 2.5.0~rc1

Agreed with the proposed changes, this is much more coherant.

Actions #8

Updated by Vincent MEMBRÉ about 10 years ago

  • Status changed from Discussion to Pending technical review
  • Assignee changed from Jonathan CLARKE to François ARMAND

PR retargeted to 2.5 : https://github.com/Normation/rudder/pull/73

now in tech review

Actions #9

Updated by François ARMAND about 10 years ago

  • Status changed from Pending technical review to In progress
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
Actions #10

Updated by Vincent MEMBRÉ about 10 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to François ARMAND
Actions #11

Updated by Vincent MEMBRÉ about 10 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
Actions #12

Updated by Anonymous about 10 years ago

Actions #14

Updated by François ARMAND almost 10 years ago

  • Status changed from Pending release to Released
Actions

Also available in: Atom PDF