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

Customize where AsyncPeriodicWork are logged

    Details

    • Sprint:
      Evergreen - Milestone 1
    • Similar Issues:

      Description

      Currently, the path where AsyncPeriodWork logs are written is hardcoded to $JENKINS_HOME/logs.

      As per the feedback on JENKINS-49406 on the associated JEP, we'd need to ideally move those logs elsewhere to keep things more cleanly separated.

      So I think we could/should introduce a sysprop to enable putting those in a different location.

        Attachments

          Issue Links

            Activity

            batmat Baptiste Mathus created issue -
            batmat Baptiste Mathus made changes -
            Field Original Value New Value
            Sprint Essentials - Milestone 1 [ 511 ]
            batmat Baptiste Mathus made changes -
            Link This issue blocks JENKINS-49406 [ JENKINS-49406 ]
            batmat Baptiste Mathus made changes -
            Assignee Baptiste Mathus [ batmat ]
            batmat Baptiste Mathus made changes -
            Rank Ranked higher
            batmat Baptiste Mathus made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            batmat Baptiste Mathus made changes -
            Rank Ranked higher
            batmat Baptiste Mathus made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            batmat Baptiste Mathus made changes -
            Remote Link This issue links to "PR (Web Link)" [ 20312 ]
            Hide
            jglick Jesse Glick added a comment -

            Maybe better done via JENKINS-38313.

            Show
            jglick Jesse Glick added a comment - Maybe better done via JENKINS-38313 .
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-38313 [ JENKINS-38313 ]
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            There is no ETA for that in JENKINS-38313. Currently I am working on the redesign of this proposal in order to reduce the scope. Aperiodic Work logging will be likely excluded from the first iteration. No ETA for that

            Show
            oleg_nenashev Oleg Nenashev added a comment - There is no ETA for that in JENKINS-38313 . Currently I am working on the redesign of this proposal in order to reduce the scope. Aperiodic Work logging will be likely excluded from the first iteration. No ETA for that
            batmat Baptiste Mathus made changes -
            Link This issue blocks JENKINS-50428 [ JENKINS-50428 ]
            batmat Baptiste Mathus made changes -
            Status In Review [ 10005 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            batmat Baptiste Mathus made changes -
            Rank Ranked higher
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            It has been released in 2.114

            Show
            oleg_nenashev Oleg Nenashev added a comment - It has been released in 2.114
            batmat Baptiste Mathus made changes -
            Labels essentials essentials essentials-triggered
            batmat Baptiste Mathus made changes -
            Remote Link This issue links to "small followup improvement (Web Link)" [ 20383 ]
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Baptiste Mathus
            Path:
            src/main/java/jenkins/metrics/api/Metrics.java
            src/test/java/jenkins/metrics/api/MetricsTest.java
            http://jenkins-ci.org/commit/metrics-plugin/c0dc0077401211952b216e91a6b16741273652b9
            Log:
            JENKINS-50291 Test showing the issue

            Expected to fail on the second assertion

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Baptiste Mathus Path: src/main/java/jenkins/metrics/api/Metrics.java src/test/java/jenkins/metrics/api/MetricsTest.java http://jenkins-ci.org/commit/metrics-plugin/c0dc0077401211952b216e91a6b16741273652b9 Log: JENKINS-50291 Test showing the issue Expected to fail on the second assertion
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Baptiste Mathus
            Path:
            src/main/java/jenkins/metrics/api/Metrics.java
            http://jenkins-ci.org/commit/metrics-plugin/1c838cc7b78558d79e02f74eec8cfd30b8a7f3c7
            Log:
            JENKINS-50291 Adjust to new logs root location

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Baptiste Mathus Path: src/main/java/jenkins/metrics/api/Metrics.java http://jenkins-ci.org/commit/metrics-plugin/1c838cc7b78558d79e02f74eec8cfd30b8a7f3c7 Log: JENKINS-50291 Adjust to new logs root location
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Stephen Connolly
            Path:
            pom.xml
            src/main/java/jenkins/metrics/api/Metrics.java
            src/test/java/jenkins/metrics/api/MetricsTest.java
            http://jenkins-ci.org/commit/metrics-plugin/39718de3379b73abb17bf80261999e5bca9b378a
            Log:
            Merge pull request #30 from batmat/JENKINS-50291

            JENKINS-50291 Move health-checker.log under the now configurable logs root directory

            Compare: https://github.com/jenkinsci/metrics-plugin/compare/87988e92d3c0...39718de3379b

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: pom.xml src/main/java/jenkins/metrics/api/Metrics.java src/test/java/jenkins/metrics/api/MetricsTest.java http://jenkins-ci.org/commit/metrics-plugin/39718de3379b73abb17bf80261999e5bca9b378a Log: Merge pull request #30 from batmat/ JENKINS-50291 JENKINS-50291 Move health-checker.log under the now configurable logs root directory Compare: https://github.com/jenkinsci/metrics-plugin/compare/87988e92d3c0...39718de3379b
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Baptiste Mathus
            Path:
            core/src/main/java/hudson/triggers/SafeTimerTask.java
            http://jenkins-ci.org/commit/jenkins/d366abc1d7b2a96a4ebd3acf44d5c812a4246d48
            Log:
            Merge pull request #3382 from batmat/JENKINS-50291-followup

            Log the message only once in INFO level

            Compare: https://github.com/jenkinsci/jenkins/compare/70c408cedf42...d366abc1d7b2

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Baptiste Mathus Path: core/src/main/java/hudson/triggers/SafeTimerTask.java http://jenkins-ci.org/commit/jenkins/d366abc1d7b2a96a4ebd3acf44d5c812a4246d48 Log: Merge pull request #3382 from batmat/ JENKINS-50291 -followup Log the message only once in INFO level Compare: https://github.com/jenkinsci/jenkins/compare/70c408cedf42...d366abc1d7b2
            batmat Baptiste Mathus made changes -
            Link This issue blocks JENKINS-52123 [ JENKINS-52123 ]
            batmat Baptiste Mathus made changes -
            Labels essentials essentials-triggered essentials essentials-triggered evergreen
            batmat Baptiste Mathus made changes -
            Labels essentials essentials-triggered evergreen essentials essentials-triggered evergreen evergreen-triggered
            batmat Baptiste Mathus made changes -
            Labels essentials essentials-triggered evergreen evergreen-triggered evergreen evergreen-triggered

              People

              • Assignee:
                batmat Baptiste Mathus
                Reporter:
                batmat Baptiste Mathus
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: