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

Display trends graph last 10/20 sucessful/fail builds in CPPcheck plugin.

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • cppcheck-plugin
    • software platform

      I have seen the trends as last sucessful builds say job#50 to the currrent failed builds say job#337.

      Instead of shows last successful and current failed builds. Its better to show only the recent 20/30 depends on configure eventhout all are failed/sucessful.

      Please find the attachement it not looks as the expected.

          [JENKINS-23247] Display trends graph last 10/20 sucessful/fail builds in CPPcheck plugin.

          Senthil Nagarajan created issue -

          Michal Turek added a comment -

          Hi Senthil,

          I'm sorry, I don't understand your description, can you rewrite it in different words, please? Concrete questions:

          • What happened during builds 50 and 337? Were they successful or failed? I guess all of them failed, correct?
          • Does build.xml results contain Cppcheck data for the builds between 50 and 337? If no, is something important in the build logs for that builds?
          • What does "last successful and current failed" mean?
          • What does "20/30" mean?
          • What do you expect on the screenshot?
          • What version of plugin do you use? Which platform, etc.

          Michal Turek added a comment - Hi Senthil, I'm sorry, I don't understand your description, can you rewrite it in different words, please? Concrete questions: What happened during builds 50 and 337? Were they successful or failed? I guess all of them failed, correct? Does build.xml results contain Cppcheck data for the builds between 50 and 337? If no, is something important in the build logs for that builds? What does "last successful and current failed" mean? What does "20/30" mean? What do you expect on the screenshot? What version of plugin do you use? Which platform, etc.

          Hi
          > During From 50 to 337 all builds are failed. yes.
          > Nope we have configured only 30 builds or 15 days builds alone archiving.
          > In the trend graph, I would like to view only the recent builds as per I would like to configure.
          For example,
          The graph trend shows from 50 to 337 which doesnt required instead of that the graph has to show only the recent 5 or 10 builds say (330 to 337) as per the configurable count.
          > I used 1.18 version on the same, platform linux, C++.

          Senthil Nagarajan added a comment - Hi > During From 50 to 337 all builds are failed. yes. > Nope we have configured only 30 builds or 15 days builds alone archiving. > In the trend graph, I would like to view only the recent builds as per I would like to configure. For example, The graph trend shows from 50 to 337 which doesnt required instead of that the graph has to show only the recent 5 or 10 builds say (330 to 337) as per the configurable count. > I used 1.18 version on the same, platform linux, C++.
          Michal Turek made changes -
          Priority Original: Blocker [ 1 ] New: Minor [ 4 ]

          Michal Turek added a comment -

          I have updated the priority from "Blocker" to "Minor". If I understand correctly, everything works as expected and this is only a feature request for showing only last N builds in the trend graph.

          Michal Turek added a comment - I have updated the priority from "Blocker" to "Minor". If I understand correctly, everything works as expected and this is only a feature request for showing only last N builds in the trend graph.

          Hi Mike

          Could you please work on this request as some high priority as we are expecting as earliest on shows last N builds in the trend graph.

          Senthil Nagarajan added a comment - Hi Mike Could you please work on this request as some high priority as we are expecting as earliest on shows last N builds in the trend graph.

          Michal Turek added a comment -

          Hi, I'm not interested in this feature and I have nearly no free time so I don't expect any work on this task from my side, sorry. But anyone can implement it and send a pull request...

          Michal Turek added a comment - Hi, I'm not interested in this feature and I have nearly no free time so I don't expect any work on this task from my side, sorry. But anyone can implement it and send a pull request...

          Daniel Beck added a comment -

          mixalturek Just in case Senthil is really interested in the feature, would you do it if a bounty was offered? If so, how much?

          Daniel Beck added a comment - mixalturek Just in case Senthil is really interested in the feature, would you do it if a bounty was offered? If so, how much?
          Michal Turek made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]

          Code changed in jenkins
          User: Michal Turek
          Path:
          src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckProjectAction.java
          src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckPublisher.java
          src/main/java/org/jenkinsci/plugins/cppcheck/config/CppcheckConfigGraph.java
          src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckPublisher/config.jelly
          src/main/resources/util/thresholds.jelly
          src/main/resources/util/thresholds.properties
          http://jenkins-ci.org/commit/cppcheck-plugin/4c62a2568d6674311730246060f2bb84596b3d2b
          Log:
          JENKINS-23247 Display trends graph last 10/20 sucessful/fail builds in CPPcheck plugin.

          • Number of builds in the trend graph can be limited in configuration, default behavior is unlimited.
          • Label for "Chart height" fixed, it pointed to "Chart width" box.
          • Label for "Display warnings" fixed, there was typo in target name.
          • Targets for "Ignore blank files" and "Do not fail build" defined.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michal Turek Path: src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckProjectAction.java src/main/java/org/jenkinsci/plugins/cppcheck/CppcheckPublisher.java src/main/java/org/jenkinsci/plugins/cppcheck/config/CppcheckConfigGraph.java src/main/resources/org/jenkinsci/plugins/cppcheck/CppcheckPublisher/config.jelly src/main/resources/util/thresholds.jelly src/main/resources/util/thresholds.properties http://jenkins-ci.org/commit/cppcheck-plugin/4c62a2568d6674311730246060f2bb84596b3d2b Log: JENKINS-23247 Display trends graph last 10/20 sucessful/fail builds in CPPcheck plugin. Number of builds in the trend graph can be limited in configuration, default behavior is unlimited. Label for "Chart height" fixed, it pointed to "Chart width" box. Label for "Display warnings" fixed, there was typo in target name. Targets for "Ignore blank files" and "Do not fail build" defined.

            gbois Gregory Boissinot
            nagase Senthil Nagarajan
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: