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

RPM post-install should not do a "chown -R ${JENKINS_USER:-jenkins} ${JENKINS_HOME:-/var/lib/jenkins}"

    XMLWordPrintable

Details

    Description

      The jenkins-1.565-1.1 RPM contains a post-install script with performs the following:

      # Ensure the right ownership on files
      . /etc/sysconfig/jenkins
      chown -R ${JENKINS_USER:-jenkins} /var/log/jenkins
      chown -R ${JENKINS_USER:-jenkins} ${JENKINS_HOME:-/var/lib/jenkins}
      

      These chowns can take a very long time on systems with lots of files. A couple of our Jenkins boxes have thousands of files in a the /var/lib/jenkins/workspace directory (which itself is a network mount), which means this chown takes many minutes to complete (like ~15 minutes) and actually changes the ownership of no files. This holds up package deployment unnecessarily.

      I propose that either these chowns be removed entirely (the RPM spec sets the ownership of things the RPM puts down), or better yet that they're restricted to specific files.

      Attachments

        Issue Links

          Activity

            benh57 Ben Hines added a comment -

            This change was never in the release notes. Was it released? 

            benh57 Ben Hines added a comment - This change was never in the release notes. Was it released? 
            oleg_nenashev Oleg Nenashev added a comment -

            benh57 It was (around 2.53). I will make sure changelog is updated. Packaging is asynchronous, so it is hard to track changes in the core. I also want to start a developer ML thread for it

            oleg_nenashev Oleg Nenashev added a comment - benh57 It was (around 2.53). I will make sure changelog is updated. Packaging is asynchronous, so it is hard to track changes in the core. I also want to start a developer ML thread for it

            Code changed in jenkins
            User: Oleg Nenashev
            Path:
            content/_data/changelogs/weekly.yml
            http://jenkins-ci.org/commit/jenkins.io/bd7e8598e4b85d84286ded5f2e6d71d08d8531ee
            Log:
            Add missing changelog entries for JENKINS-23273 and JENKINS-40693

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: content/_data/changelogs/weekly.yml http://jenkins-ci.org/commit/jenkins.io/bd7e8598e4b85d84286ded5f2e6d71d08d8531ee Log: Add missing changelog entries for JENKINS-23273 and JENKINS-40693

            Code changed in jenkins
            User: Daniel Beck
            Path:
            content/_data/changelogs/weekly.yml
            http://jenkins-ci.org/commit/jenkins.io/d6c1369d8ebcdff4a9f126097ecc6a7c30ea1d86
            Log:
            Merge pull request #921 from oleg-nenashev/changelog/missing-entries

            Add missing changelog entries for JENKINS-23273 and JENKINS-40693

            Compare: https://github.com/jenkins-infra/jenkins.io/compare/14212ab18dfd...d6c1369d8ebc

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: content/_data/changelogs/weekly.yml http://jenkins-ci.org/commit/jenkins.io/d6c1369d8ebcdff4a9f126097ecc6a7c30ea1d86 Log: Merge pull request #921 from oleg-nenashev/changelog/missing-entries Add missing changelog entries for JENKINS-23273 and JENKINS-40693 Compare: https://github.com/jenkins-infra/jenkins.io/compare/14212ab18dfd...d6c1369d8ebc

            Code changed in jenkins
            User: Daniel Beck
            Path:
            content/_data/changelogs/lts.yml
            content/_data/changelogs/weekly.yml
            content/doc/upgrade-guide/2.60.adoc
            http://jenkins-ci.org/commit/jenkins.io/800435b6892e68f259f9c6ecbc54e79b1798c656
            Log:
            Mention JENKINS-23273, minor upgrade guide changes

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: content/_data/changelogs/lts.yml content/_data/changelogs/weekly.yml content/doc/upgrade-guide/2.60.adoc http://jenkins-ci.org/commit/jenkins.io/800435b6892e68f259f9c6ecbc54e79b1798c656 Log: Mention JENKINS-23273 , minor upgrade guide changes

            People

              rarabaolaza Raul Arabaolaza
              coofercat Ralph Bolton
              Votes:
              13 Vote for this issue
              Watchers:
              24 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: