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

Add maximum size limit to ${FAILED_TESTS}

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      A systematic problem in test setup can at worst cause hundreds or thousands of tests to fail in a CI build. Such a situation currently ends up generating huge emails when a custom notification template with the ${FAILED_TESTS} token is used.

      It would be better if the ${FAILED_TESTS} token had configurable maximum size limit with a reasonable default value like the maxLines option of the ${BUILD_LOG} token.

        Attachments

          Activity

          Hide
          jukka Jukka Zitting added a comment -
          Show
          jukka Jukka Zitting added a comment - See https://github.com/jenkinsci/email-ext-plugin/pull/24 for a proposed solution.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Jukka Zitting
          Path:
          src/main/java/hudson/plugins/emailext/plugins/content/FailedTestsContent.java
          src/test/java/hudson/plugins/emailext/plugins/content/FailedTestsContentTest.java
          http://jenkins-ci.org/commit/email-ext-plugin/580607952350daf008360071fc21c3235483a8ad
          Log:
          JENKINS-11413: Add maximum size limit to ${FAILED_TESTS}

          Add a maxTests argument to the ${FAILED_TESTS} token.

          The setting is undefined by default for backwards compatibility with existing configurations.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jukka Zitting Path: src/main/java/hudson/plugins/emailext/plugins/content/FailedTestsContent.java src/test/java/hudson/plugins/emailext/plugins/content/FailedTestsContentTest.java http://jenkins-ci.org/commit/email-ext-plugin/580607952350daf008360071fc21c3235483a8ad Log: JENKINS-11413 : Add maximum size limit to ${FAILED_TESTS} Add a maxTests argument to the ${FAILED_TESTS} token. The setting is undefined by default for backwards compatibility with existing configurations.
          Hide
          kutzi kutzi added a comment -

          Closing as pull request has been merged

          Show
          kutzi kutzi added a comment - Closing as pull request has been merged

            People

            • Assignee:
              Unassigned
              Reporter:
              jukka Jukka Zitting
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: