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

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

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      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.

        Attachments

          Issue Links

            Activity

            smd Stefan Drissen created issue -
            smd Stefan Drissen made changes -
            Field Original Value New Value
            Description 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'.

            !image-2018-02-13-00-14-48-787.png!


            Empty values are stored as -1.

            Changing the value to a positive integer clears the validation warning, but emptying it returns the warning.
            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'.

            !image-2018-02-13-00-14-48-787.png!

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

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

            Changing the value to a positive integer clears the validation warning, but emptying it returns the warning.
            smd Stefan Drissen made changes -
            Description 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'.

            !image-2018-02-13-00-14-48-787.png!

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

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

            Changing the value to a positive integer clears the validation warning, but emptying it returns the warning.
            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'.

            Probably a result of changes made in JENKINS-47793

            !image-2018-02-13-00-14-48-787.png!

            Empty values are (and always have been) stored as -1.
            {code:java}
            <logRotator class="hudson.tasks.LogRotator">
              <daysToKeep>-1</daysToKeep>
              <numToKeep>20</numToKeep>
              <artifactDaysToKeep>-1</artifactDaysToKeep>
              <artifactNumToKeep>-1</artifactNumToKeep>
            </logRotator>
            {code}
             

            Changing the value to a positive integer clears the validation warning, but emptying it returns the warning.
            smd Stefan Drissen made changes -
            Description 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'.

            Probably a result of changes made in JENKINS-47793

            !image-2018-02-13-00-14-48-787.png!

            Empty values are (and always have been) stored as -1.
            {code:java}
            <logRotator class="hudson.tasks.LogRotator">
              <daysToKeep>-1</daysToKeep>
              <numToKeep>20</numToKeep>
              <artifactDaysToKeep>-1</artifactDaysToKeep>
              <artifactNumToKeep>-1</artifactNumToKeep>
            </logRotator>
            {code}
             

            Changing the value to a positive integer clears the validation warning, but emptying it returns the warning.
            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'.

            !image-2018-02-13-00-14-48-787.png!

            Empty values are (and always have been) stored as -1.
            {code:java}
            <logRotator class="hudson.tasks.LogRotator">
              <daysToKeep>-1</daysToKeep>
              <numToKeep>20</numToKeep>
              <artifactDaysToKeep>-1</artifactDaysToKeep>
              <artifactNumToKeep>-1</artifactNumToKeep>
            </logRotator>
            {code}
             

            Changing the value to a positive integer clears the validation warning, but emptying it returns the warning.
            ks_nenasheva Kseniia Nenasheva made changes -
            Assignee Kseniia Nenasheva [ ks_nenasheva ]
            ks_nenasheva Kseniia Nenasheva made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            ks_nenasheva Kseniia Nenasheva made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            oleg_nenashev Oleg Nenashev made changes -
            Labels lts-candidate regression
            danielbeck Daniel Beck made changes -
            Link This issue is related to JENKINS-49387 [ JENKINS-49387 ]
            danielbeck Daniel Beck made changes -
            Status In Review [ 10005 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            danielbeck Daniel Beck made changes -
            Labels lts-candidate regression 2.107.1-fixed regression

              People

              • Assignee:
                ks_nenasheva Kseniia Nenasheva
                Reporter:
                smd Stefan Drissen
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: