Uploaded image for project: 'Sirona'
  1. Sirona
  2. SIRONA-57

Graphite plugin uses timestamp in ms instead of seconds

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.2-incubating, 0.3-incubating
    • Fix Version/s: 0.4-incubating
    • Labels:
      None

      Description

      Graphite requires the timestamp to be a Unix Timestamp (aka number of seconds since epocs).

      We use instead currentTimeMillis which is wrong.

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in sirona-1.8 #124 (See https://builds.apache.org/job/sirona-1.8/124/)
        SIRONA-57 Bad timestamp unit for Graphite push (jlmonteiro: rev 1720105)

        • agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteCounterDataStore.java
        • agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteGaugeDataStore.java
        • agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTest.java
        • agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTestBase.java
        • agent/store/graphite/src/test/java/org/apache/sirona/graphite/server/GraphiteMockServer.java
        • agent/store/graphite/src/test/resources/sirona.properties
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in sirona-1.8 #124 (See https://builds.apache.org/job/sirona-1.8/124/ ) SIRONA-57 Bad timestamp unit for Graphite push (jlmonteiro: rev 1720105) agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteCounterDataStore.java agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteGaugeDataStore.java agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTest.java agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTestBase.java agent/store/graphite/src/test/java/org/apache/sirona/graphite/server/GraphiteMockServer.java agent/store/graphite/src/test/resources/sirona.properties
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in sirona-1.7 #123 (See https://builds.apache.org/job/sirona-1.7/123/)
        SIRONA-57 Bad timestamp unit for Graphite push (jlmonteiro: rev 1720105)

        • agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteCounterDataStore.java
        • agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteGaugeDataStore.java
        • agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTest.java
        • agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTestBase.java
        • agent/store/graphite/src/test/java/org/apache/sirona/graphite/server/GraphiteMockServer.java
        • agent/store/graphite/src/test/resources/sirona.properties
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in sirona-1.7 #123 (See https://builds.apache.org/job/sirona-1.7/123/ ) SIRONA-57 Bad timestamp unit for Graphite push (jlmonteiro: rev 1720105) agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteCounterDataStore.java agent/store/graphite/src/main/java/org/apache/sirona/graphite/GraphiteGaugeDataStore.java agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTest.java agent/store/graphite/src/test/java/org/apache/sirona/graphite/GraphiteTestBase.java agent/store/graphite/src/test/java/org/apache/sirona/graphite/server/GraphiteMockServer.java agent/store/graphite/src/test/resources/sirona.properties

          People

          • Assignee:
            jlmonteiro Jean-Louis MONTEIRO
            Reporter:
            jlmonteiro Jean-Louis MONTEIRO
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development