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

Warnings plugin doesn't set build status if one input file is empty

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: warnings-plugin
    • Labels:
      None
    • Environment:
      Jenkins ver. 2.107.2
      Warnings ver. 4.65
    • Similar Issues:
    • Released As:
      5.0.0-beta2

      Description

      We have two different PEP8 linter programs that run as part of our build, using

      flake8 | tee build/tests/flake8.txt

      textlint | tee build/tests/textlint.txt

      Both are included as input to the warnings plugin using the includes pattern "build/tests/flake8.txt,build/tests/textlint.txt"

      If one file contains warnings, but the other file is empty, the warnings plugin reports an error: "Module : Skipping file of module <project workspace>/build/tests/textlint.txt because it's empty" and it will not update the build status, even if the threshold for an unstable build is 0 warnings and the other file contains warnings. It will report the warnings from the other file on the build page, though.

      We worked around this by removing any empty files output by the linters, which causes the plugin to update the build status properly.

        Attachments

          Activity

          Show
          drulli Ulli Hafner added a comment - See https://github.com/jenkinsci/warnings-plugin/commit/35b73011c57f47d38e3472fb056318dd99fce713
          Hide
          drulli Ulli Hafner added a comment -

          Released in 5.0.0-beta2.

          Show
          drulli Ulli Hafner added a comment - Released in 5.0.0-beta2.

            People

            Assignee:
            drulli Ulli Hafner
            Reporter:
            sigmaris Hugh Cole-Baker
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: