Bug #3064
closed
Display of referenced objects in Event Logs is very poor
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.
Updated by Nicolas PERRON about 12 years ago
- Target version changed from 2.4.0 to 61
Updated by François ARMAND about 12 years ago
- Target version changed from 61 to 2.4.2
Updated by Nicolas PERRON almost 12 years ago
- Target version changed from 2.4.2 to 2.4.3
Updated by Vincent MEMBRÉ almost 12 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
Updated by Vincent MEMBRÉ almost 12 years ago
I made a pull request with those modification : https://github.com/Normation/rudder/pull/70
Updated by Jonathan CLARKE almost 12 years ago
- Target version changed from 2.4.3 to 2.5.0~rc1
Agreed with the proposed changes, this is much more coherant.
Updated by Vincent MEMBRÉ almost 12 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
Updated by François ARMAND almost 12 years ago
- Status changed from Pending technical review to In progress
- Assignee changed from François ARMAND to Vincent MEMBRÉ
Updated by Vincent MEMBRÉ almost 12 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Vincent MEMBRÉ to François ARMAND
Updated by Vincent MEMBRÉ almost 12 years ago
- Status changed from Pending technical review to Pending release
- % Done changed from 0 to 100
Applied in changeset 965a8e6db9c7f6484d9b329d45f7b54afe4a0db3.
Updated by Anonymous almost 12 years ago
Applied in changeset b771f224497ff7f8adb827bcab0664b7c6e25ef5.
Updated by Vincent MEMBRÉ almost 12 years ago
Applied in changeset f3d3625f2dd8c3d480e8616274b1f0cfb5000bab.
Updated by François ARMAND over 11 years ago
- Status changed from Pending release to Released