Project

General

Custom queries

Profile

Actions

User story #4708

closed

Display time of reports in the web interface in the server timezone

Added by Jonathan CLARKE about 11 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
2
Assignee:
-
Category:
Web - Nodes & inventories
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Checked
Regression:

Description

I have a server in CET and a node in CDT. These two timezones have 6 hours difference.

The logs generated on my node look like this:

Mar 30 09:26:12 Message forwarded from sovma136: rudder[512100]: R: @@Common@@log_info@@hasPolicyServer-root@@common-root@@11@@common@@EndRun@@2014-03-30 14:26:00+00:00##f5b7b567-de02-0000-02c0-a8fd88000000@#End execution

As you can see, the time inside the Rudder report is 5 hours ahead of the time in the syslog timestamp. That timestamp comes from the node (it is in CDT).

Our rsyslog config uses that timestamp (from the node!) to insert a line into our PSQL DB:

$template RudderReportsFormat,"insert into RudderSysEvents (executionDate, ...) values ('%timereported:::date-rfc3339%',... )",stdsql

The rsyslog property timereported is defined as follows:

timestamp from the message. Resolution depends on what was provided in the message (in most cases, only seconds)

However, I don't think we should be trusting the time reported by the node (it could be completely out of sync, or worse, lying!), and I don't think we should be using it because we can't know what timezone it's in.

This does not seem to affect reporting, because I think reporting uses the middle field (the time inside the report).

So, I see two solutions:

  1. We change rsyslog to insert the timegenerated (defined as "timestamp when the message was RECEIVED"), and then the display will always be in our local timezone. I don't like this solution, because it means that if logs were delayed (due to unavailable netork or whatever), they will all suddenly appear with the same timestamp, which can be very misleading.
  2. We change the webapp to display the time from the reports field, rather than the syslog time.

I prefer solution 2, but it begs the question: why would be storing the other timestamp too? Maybe we should rename that a "time report received" information that could be useful in advanced debugging or future use?

Thoughts and opinions?


Files

clipboard-202004272303-ibmyd.png (22.6 KB) clipboard-202004272303-ibmyd.png François ARMAND, 2020-04-27 23:03

Related issues 5 (0 open5 closed)

Related to Rudder - Question #5272: Time synchronization is required for reporting ?ResolvedActions
Related to Rudder - Bug #5768: Document how to configure your timezone in rudder webappResolvedActions
Related to Rudder - Bug #7092: Include Timezone in Node InfoReleasedVincent MEMBRÉActions
Related to Rudder - User story #7093: Include Timezone in InventoryReleasedBenoît PECCATTE2016-10-18Actions
Related to Rudder - Bug #8989: Epoch is used for inventory date when inventory is brokenResolvedActions
#1

Updated by Nicolas CHARLES about 11 years ago

  • Assignee changed from Nicolas CHARLES to Jonathan CLARKE
#3

Updated by Vincent MEMBRÉ about 11 years ago

  • Target version changed from 2.6.13 to 2.6.14
#4

Updated by Jonathan CLARKE almost 11 years ago

  • Target version changed from 2.6.14 to 2.6.16
#5

Updated by Jonathan CLARKE almost 11 years ago

  • Target version changed from 2.6.16 to 2.6.17
#6

Updated by Nicolas PERRON almost 11 years ago

  • Target version changed from 2.6.17 to 2.6.18
#7

Updated by Matthieu CERDA over 10 years ago

  • Target version changed from 2.6.18 to 2.6.19
#8

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.6.19 to 2.6.20
#9

Updated by François ARMAND over 10 years ago

  • Target version changed from 2.6.20 to 2.10.10
#10

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.10.10 to 2.10.11
#11

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.10.11 to 2.10.12
#12

Updated by Vincent MEMBRÉ about 10 years ago

  • Target version changed from 2.10.12 to 2.10.13
#13

Updated by Vincent MEMBRÉ about 10 years ago

  • Target version changed from 2.10.13 to 2.10.14
#14

Updated by Benoît PECCATTE about 10 years ago

  • Assignee changed from Jonathan CLARKE to Nicolas CHARLES
#15

Updated by Vincent MEMBRÉ about 10 years ago

  • Target version changed from 2.10.14 to 2.10.15
#16

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.15 to 2.10.16
#17

Updated by Vincent MEMBRÉ almost 10 years ago

  • Target version changed from 2.10.16 to 2.10.17
#18

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.17 to 2.10.18
#19

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.18 to 2.10.19
#20

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.19 to 2.10.20
#21

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.10.20 to 2.11.18
#22

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.11.18 to 2.11.19
#23

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.11.19 to 2.11.20
#24

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.20 to 2.11.21
#25

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.21 to 2.11.22
#26

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.22 to 2.11.23
#27

Updated by François ARMAND almost 9 years ago

  • Related to Bug #7092: Include Timezone in Node Info added
#28

Updated by François ARMAND almost 9 years ago

#29

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.11.23 to 2.11.24
#30

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.11.24 to 308
#31

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 308 to 3.1.14
#32

Updated by François ARMAND over 8 years ago

  • Related to Bug #8989: Epoch is used for inventory date when inventory is broken added
#33

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 3.1.14 to 3.1.15
#34

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 3.1.15 to 3.1.16
#35

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 3.1.16 to 3.1.17
#36

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 3.1.17 to 3.1.18
#37

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 3.1.18 to 3.1.19
#38

Updated by Benoît PECCATTE over 8 years ago

  • Tracker changed from Bug to User story
  • Subject changed from Time of reports appears in the web interface in the timezone local to the node instead of the server to Display time of reports in the web interface in the server timezone
#39

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 3.1.19 to 3.1.20
#40

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 3.1.20 to 3.1.21
#41

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 3.1.21 to 3.1.22
#42

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 3.1.22 to 3.1.23
#43

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 3.1.23 to 3.1.24
#44

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.24 to 3.1.25
#45

Updated by Benoît PECCATTE over 7 years ago

  • Target version changed from 3.1.25 to 4.1.9
#46

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 4.1.9 to 4.1.10
#47

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 4.1.10 to 4.1.11
#48

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 4.1.11 to 4.1.12
#49

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 4.1.12 to 4.1.13
#50

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 4.1.13 to 4.1.14
#51

Updated by Benoît PECCATTE almost 7 years ago

  • Target version changed from 4.1.14 to 4.1.15
#52

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.15 to 4.1.16
#53

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.16 to 4.1.17
#54

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.17 to 4.1.18
#55

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 4.1.18 to 4.1.19
#56

Updated by Alexis Mousset over 6 years ago

  • Target version changed from 4.1.19 to 4.1.20
#57

Updated by Alexis Mousset almost 6 years ago

  • Target version changed from 4.1.20 to 588
#58

Updated by Alexis Mousset almost 6 years ago

  • Target version changed from 588 to 5.0.13
#59

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 5.0.13 to 5.0.14
#60

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.14 to 5.0.15
#61

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.15 to 5.0.16
#62

Updated by Alexis Mousset over 5 years ago

  • Target version changed from 5.0.16 to 5.0.17
#63

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.17 to 5.0.18
#64

Updated by Benoît PECCATTE about 5 years ago

  • Target version changed from 5.0.18 to 6.2.0~beta1
#65

Updated by François ARMAND about 5 years ago

#66

Updated by François ARMAND over 4 years ago

  • Fix check set to Checked
Actions

Also available in: Atom PDF