Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-67158

Investigate if usage stats are being correctly captured on Java 11

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Minor
    • Resolution: Fixed
    • core
    • None

    Description

      In https://groups.google.com/g/jenkinsci-dev/c/YghQ0YP4m78/m/eDNEkwJIAAAJ

      danielbeck reported a suspicion that we aren't collecting stats properly for Java 11+

      Need to investigate this

      Attachments

        Activity

          Ascending order - Click to sort in descending order
          danielbeck Daniel Beck added a comment -

          https://github.com/jenkins-infra/infra-statistics/blob/85a7c65fce6d368a27fcec2da99d28bf43e680ca/createJson.groovy#L178 is probably too restrictive.

          danielbeck Daniel Beck added a comment - https://github.com/jenkins-infra/infra-statistics/blob/85a7c65fce6d368a27fcec2da99d28bf43e680ca/createJson.groovy#L178 is probably too restrictive.
          danielbeck Daniel Beck added a comment -

          https://github.com/jenkinsci/jenkins/blob/090779fb4e88f8105a1cde0e2eaca725e7b3966d/core/src/main/java/hudson/model/UsageStatistics.java#L147

          Looks like we're collecting the wrong version too, but probably too late? ( java.specification.version should be better?)

          danielbeck Daniel Beck added a comment - https://github.com/jenkinsci/jenkins/blob/090779fb4e88f8105a1cde0e2eaca725e7b3966d/core/src/main/java/hudson/model/UsageStatistics.java#L147 Looks like we're collecting the wrong version too, but probably too late? (  java.specification.version should be better?)
          danielbeck Daniel Beck added a comment -

          Looks like a combination of https://github.com/jenkins-infra/infra-statistics/blob/85a7c65fce6d368a27fcec2da99d28bf43e680ca/createJson.groovy#L178 and https://github.com/jenkins-infra/infra-statistics/blob/85a7c65fce6d368a27fcec2da99d28bf43e680ca/createJson.groovy#L188? The first three chars of 11.0.13 are "11." and not just "11". Unverified.

          danielbeck Daniel Beck added a comment - Looks like a combination of https://github.com/jenkins-infra/infra-statistics/blob/85a7c65fce6d368a27fcec2da99d28bf43e680ca/createJson.groovy#L178 and https://github.com/jenkins-infra/infra-statistics/blob/85a7c65fce6d368a27fcec2da99d28bf43e680ca/createJson.groovy#L188 ? The first three chars of 11.0.13 are " 11. " and not just " 11 ". Unverified.

          People

            Unassigned Unassigned
            timja Tim Jacomb
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: