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

Log rotator: empty values decorated with 'Not a positive integer'

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • Windows 2012R2 x64
      JRE 1.8_0161 x64
      Jenkins 2.105

      Since (I think) the last update from 2.104 to 2.105 something has changed in validation of integer fields resulting in the 'discard builds' section of a freestyle job showing validation warning 'Not a positive integer'.

      Empty values are (and always have been) stored as -1.

      <logRotator class="hudson.tasks.LogRotator">
        <daysToKeep>-1</daysToKeep>
        <numToKeep>20</numToKeep>
        <artifactDaysToKeep>-1</artifactDaysToKeep>
        <artifactNumToKeep>-1</artifactNumToKeep>
      </logRotator>
      

       

      Changing the value to a positive integer clears the validation warning, but emptying it returns the warning.

          [JENKINS-49520] Log rotator: empty values decorated with 'Not a positive integer'

          Stefan Drissen added a comment - - edited

          I am fairly sure that JENKINS-47793 introduced the issue, the pipe (|) was removed rendering blank an invalid value:

          https://github.com/ksenia-nenasheva/jenkins/commit/f8e3526244af53178439b9a73e1297cf3f7f5ce4#diff-da911448bbb85d374f604da9c1e00136

           

          Stefan Drissen added a comment - - edited I am fairly sure that JENKINS-47793 introduced the issue, the pipe (|) was removed rendering blank an invalid value: https://github.com/ksenia-nenasheva/jenkins/commit/f8e3526244af53178439b9a73e1297cf3f7f5ce4#diff-da911448bbb85d374f604da9c1e00136  

          smd Thanks!
          Fixed in PR #3292

          Kseniia Nenasheva added a comment - smd Thanks! Fixed in PR #3292

          Oleg Nenashev added a comment -

          May need backporting to 2.107 which is the current LTS candidate baseline

          Oleg Nenashev added a comment - May need backporting to 2.107 which is the current LTS candidate baseline

          Code changed in jenkins
          User: Kseniia Nenasheva
          Path:
          core/src/main/java/hudson/model/Computer.java
          core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly
          core/src/main/resources/jenkins/model/Jenkins/MasterComputer/configure.jelly
          core/src/main/resources/jenkins/model/MasterBuildConfiguration/config.groovy
          war/src/main/webapp/scripts/hudson-behavior.js
          http://jenkins-ci.org/commit/jenkins/5c8cc45900bf8d78058cecd471840bdbc88215d3
          Log:
          JENKINS-49387JENKINS-49520 - Validation errors (#3292)

          • Fixes for validation
          • Fix for an empty agent.nExecutors

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kseniia Nenasheva Path: core/src/main/java/hudson/model/Computer.java core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly core/src/main/resources/jenkins/model/Jenkins/MasterComputer/configure.jelly core/src/main/resources/jenkins/model/MasterBuildConfiguration/config.groovy war/src/main/webapp/scripts/hudson-behavior.js http://jenkins-ci.org/commit/jenkins/5c8cc45900bf8d78058cecd471840bdbc88215d3 Log: JENKINS-49387 JENKINS-49520 - Validation errors (#3292) Fixes for validation Fix for an empty agent.nExecutors

          Daniel Beck added a comment -

          Released in 2.108.

          Daniel Beck added a comment - Released in 2.108.

          Code changed in jenkins
          User: Kseniia Nenasheva
          Path:
          core/src/main/java/hudson/model/Computer.java
          core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly
          core/src/main/resources/jenkins/model/Jenkins/MasterComputer/configure.jelly
          core/src/main/resources/jenkins/model/MasterBuildConfiguration/config.groovy
          war/src/main/webapp/scripts/hudson-behavior.js
          http://jenkins-ci.org/commit/jenkins/2ad96a3d5b5a10f11601d427690e765d654941a7
          Log:
          JENKINS-49387JENKINS-49520 - Validation errors (#3292)

          • Fixes for validation
          • Fix for an empty agent.nExecutors

          (cherry picked from commit 5c8cc45900bf8d78058cecd471840bdbc88215d3)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Kseniia Nenasheva Path: core/src/main/java/hudson/model/Computer.java core/src/main/resources/hudson/slaves/DumbSlave/configure-entries.jelly core/src/main/resources/jenkins/model/Jenkins/MasterComputer/configure.jelly core/src/main/resources/jenkins/model/MasterBuildConfiguration/config.groovy war/src/main/webapp/scripts/hudson-behavior.js http://jenkins-ci.org/commit/jenkins/2ad96a3d5b5a10f11601d427690e765d654941a7 Log: JENKINS-49387 JENKINS-49520 - Validation errors (#3292) Fixes for validation Fix for an empty agent.nExecutors (cherry picked from commit 5c8cc45900bf8d78058cecd471840bdbc88215d3)

            ks_nenasheva Kseniia Nenasheva
            smd Stefan Drissen
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: