Project

General

Profile

Bug #26113

Smart proxy log times not displayed.

Added by Alex Fisher 5 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomer Brisker
Category:
Web Interface
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/6511
Team Backlog:
Fixed in Releases:
1.21.0
Found in Releases:
1.21.0

Description

Foreman and smart-proxy version: 1.21.0-RC5

Log entries are not displaying timestamps.
Screenshot

missing-times.png View missing-times.png 98.5 KB Screenshot Alex Fisher, 02/22/2019 11:12 AM
Missing times

Related issues

Related to Foreman - Feature #19047: Unify the way how we display datesClosed2017-03-28
Related to Foreman - Feature #21312: Foreman should provide react components for formatting datesClosed
Has duplicate Foreman - Bug #25982: Smart Proxy | Time column in logs view is emptyDuplicate

Associated revisions

Revision 8da0a865 (diff)
Added by Tomer Brisker 5 months ago

Fixes #26113 - Show timestamp in proxy log pane

When unifing the date time columns, the datatables initializer for the
logs table was changes to render the time column as it is. This does not
work properly with the new react date component, causing the column to
be empty. This commit removes the unneeded render function for this
column and corrects the helper to also display seconds for the
timestamps.

History

#1 Updated by Ewoud Kohl van Wijngaarden 5 months ago

  • Category set to Web Interface
  • Found in Releases 1.21.0 added

#2 Updated by Ewoud Kohl van Wijngaarden 5 months ago

  • Triaged changed from No to Yes

#3 Updated by Tomer Brisker 5 months ago

Digging in to this, looks like the datatable is initialized before react renders the date component, causing the component to be rendered off-screen for some reason. I'm continuing to investigate.

#4 Updated by Tomer Brisker 5 months ago

#5 Updated by The Foreman Bot 5 months ago

  • Assignee set to Tomer Brisker
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6511 added

#6 Updated by Ohad Levy 5 months ago

  • Target version set to 1.22.0
  • Fixed in Releases 1.22.0 added

#7 Updated by Tomer Brisker 5 months ago

  • Fixed in Releases 1.21.0 added
  • Fixed in Releases deleted (1.22.0)

#8 Updated by Tomer Brisker 5 months ago

  • Status changed from Ready For Testing to Closed

#9 Updated by Ondřej Ezr 4 months ago

  • Has duplicate Bug #25982: Smart Proxy | Time column in logs view is empty added

#10 Updated by Tomer Brisker 4 months ago

  • Related to Feature #21312: Foreman should provide react components for formatting dates added

#11 Updated by Tomer Brisker about 1 month ago

  • Target version deleted (1.22.0)

Also available in: Atom PDF