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

Initial plugin setup needs to consider implied dependencies

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Implied dependencies are not considered at all on the setup dialog. If we think they're so important that we don't allow disabling the plugins when an implied dependent plugin is enabled, we should consider them important enough to install them when such a plugin is installed, no? Otherwise this could result in a giant mess of broken plugins.

      Assigning to Tom as he wrote the plugin wizard. CC Keith Zantow

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Daniel Beck
            Path:
            core/src/main/java/hudson/ClassicPluginStrategy.java
            core/src/main/java/hudson/PluginManager.java
            core/src/main/java/hudson/model/UpdateSite.java
            core/src/main/java/hudson/security/HudsonPrivateSecurityRealm.java
            core/src/main/resources/hudson/security/HudsonPrivateSecurityRealm/_entryForm.jelly
            war/src/main/js/api/plugins.js
            war/src/main/js/pluginSetupWizardGui.js
            http://jenkins-ci.org/commit/jenkins/449719ca8aaad1a177f81dbb6a35df76ca82d012
            Log:
            Merge pull request #2122 from kzantow/2.0

            JENKINS-33296 - plugin dependency issues during install

            Compare: https://github.com/jenkinsci/jenkins/compare/f4817411c403...449719ca8aaa

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: core/src/main/java/hudson/ClassicPluginStrategy.java core/src/main/java/hudson/PluginManager.java core/src/main/java/hudson/model/UpdateSite.java core/src/main/java/hudson/security/HudsonPrivateSecurityRealm.java core/src/main/resources/hudson/security/HudsonPrivateSecurityRealm/_entryForm.jelly war/src/main/js/api/plugins.js war/src/main/js/pluginSetupWizardGui.js http://jenkins-ci.org/commit/jenkins/449719ca8aaad1a177f81dbb6a35df76ca82d012 Log: Merge pull request #2122 from kzantow/2.0 JENKINS-33296 - plugin dependency issues during install Compare: https://github.com/jenkinsci/jenkins/compare/f4817411c403...449719ca8aaa
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: kzantow
            Path:
            core/src/main/java/hudson/model/UpdateSite.java
            core/src/main/java/hudson/security/HudsonPrivateSecurityRealm.java
            core/src/main/resources/hudson/security/HudsonPrivateSecurityRealm/_entryForm.jelly
            war/src/main/js/api/plugins.js
            war/src/main/js/pluginSetupWizardGui.js
            http://jenkins-ci.org/commit/jenkins/3edb9fdcecedb4514e4b0396306cb0f115a0b321
            Log:
            JENKINS-33296 - plugin dependency issues during install

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: kzantow Path: core/src/main/java/hudson/model/UpdateSite.java core/src/main/java/hudson/security/HudsonPrivateSecurityRealm.java core/src/main/resources/hudson/security/HudsonPrivateSecurityRealm/_entryForm.jelly war/src/main/js/api/plugins.js war/src/main/js/pluginSetupWizardGui.js http://jenkins-ci.org/commit/jenkins/3edb9fdcecedb4514e4b0396306cb0f115a0b321 Log: JENKINS-33296 - plugin dependency issues during install
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Daniel Beck
            Path:
            war/src/main/js/api/plugins.js
            http://jenkins-ci.org/commit/jenkins/7f87486c73b1c63cc595c0fc380d42ada45f6116
            Log:
            Temporarily remove plugins affected by JENKINS-33296

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Beck Path: war/src/main/js/api/plugins.js http://jenkins-ci.org/commit/jenkins/7f87486c73b1c63cc595c0fc380d42ada45f6116 Log: Temporarily remove plugins affected by JENKINS-33296
            Hide
            danielbeck Daniel Beck added a comment -

            Keith Zantow Right, that probably needs to be fed from ClassicPluginStrategy (and may actually require some reworking there if it is currently only available for already installed plugins).

            Show
            danielbeck Daniel Beck added a comment - Keith Zantow Right, that probably needs to be fed from ClassicPluginStrategy (and may actually require some reworking there if it is currently only available for already installed plugins).
            Hide
            kzantow Keith Zantow added a comment -

            Daniel Beck ah yes, this is a larger issue: these plugins also don't have the metadata available to the setup wizard (e.g. junit).

            Show
            kzantow Keith Zantow added a comment - Daniel Beck ah yes, this is a larger issue: these plugins also don't have the metadata available to the setup wizard (e.g. junit).

              People

              • Assignee:
                kzantow Keith Zantow
                Reporter:
                danielbeck Daniel Beck
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: