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

Bundle naming strategy should be able to specify an instance type

    Details

    • Similar Issues:

      Description

      When one has to handle a lot of support bundles coming from different servers, it can make sense to be able to override the default prefix support (see related code).

      So I think the existing code should be refactored to accept using something like a pluggable strategy.

      Currently, there is already the SupportProvider.getName() class which lets you customize the beginning of the bundle name. Being able to insert an additional information would be helpful for the purpose explained above.

        Attachments

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            Please remember to link from PR to JIRA and back, and set the status.

            Show
            jglick Jesse Glick added a comment - Please remember to link from PR to JIRA and back, and set the status.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Baptiste Mathus
            Path:
            src/main/java/com/cloudbees/jenkins/support/BundleNameInstanceTypeProvider.java
            src/main/java/com/cloudbees/jenkins/support/SupportPlugin.java
            src/test/java/com/cloudbees/jenkins/support/BundleNamePrefixTest.java
            http://jenkins-ci.org/commit/support-core-plugin/b7ee94fd00eaf77fd0526acffdbd5641f5b7abc6
            Log:
            JENKINS-40098 Bundle naming strategy should be able to specify an instance type

            Introduced a change enabling both overriding through using a sysprop,
            or by implementing a new Extension Point.

            The approach with Extension Points should enable more dynamic needs,
            when the sysprop one will be immediately usable by anyone without
            having to develop an extension.

            Note: if found, the extension is preferred over the sysprop.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Baptiste Mathus Path: src/main/java/com/cloudbees/jenkins/support/BundleNameInstanceTypeProvider.java src/main/java/com/cloudbees/jenkins/support/SupportPlugin.java src/test/java/com/cloudbees/jenkins/support/BundleNamePrefixTest.java http://jenkins-ci.org/commit/support-core-plugin/b7ee94fd00eaf77fd0526acffdbd5641f5b7abc6 Log: JENKINS-40098 Bundle naming strategy should be able to specify an instance type Introduced a change enabling both overriding through using a sysprop, or by implementing a new Extension Point. The approach with Extension Points should enable more dynamic needs, when the sysprop one will be immediately usable by anyone without having to develop an extension. Note: if found, the extension is preferred over the sysprop.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Baptiste Mathus
            Path:
            src/main/java/com/cloudbees/jenkins/support/BundleNameInstanceTypeProvider.java
            src/main/java/com/cloudbees/jenkins/support/SupportAction.java
            src/main/java/com/cloudbees/jenkins/support/SupportCommand.java
            src/main/java/com/cloudbees/jenkins/support/SupportLogFormatter.java
            src/main/java/com/cloudbees/jenkins/support/SupportPlugin.java
            src/test/java/com/cloudbees/jenkins/support/BundleNamePrefixTest.java
            http://jenkins-ci.org/commit/support-core-plugin/80b611d0d3a3fff0cb5593c75d6d8b9338d862e1
            Log:
            Merge pull request #92 from batmat/JENKINS-40098-take-2

            JENKINS-40098 Bundle naming strategy should be able to specify an instance type

            Compare: https://github.com/jenkinsci/support-core-plugin/compare/fc3119380c83...80b611d0d3a3

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Baptiste Mathus Path: src/main/java/com/cloudbees/jenkins/support/BundleNameInstanceTypeProvider.java src/main/java/com/cloudbees/jenkins/support/SupportAction.java src/main/java/com/cloudbees/jenkins/support/SupportCommand.java src/main/java/com/cloudbees/jenkins/support/SupportLogFormatter.java src/main/java/com/cloudbees/jenkins/support/SupportPlugin.java src/test/java/com/cloudbees/jenkins/support/BundleNamePrefixTest.java http://jenkins-ci.org/commit/support-core-plugin/80b611d0d3a3fff0cb5593c75d6d8b9338d862e1 Log: Merge pull request #92 from batmat/ JENKINS-40098 -take-2 JENKINS-40098 Bundle naming strategy should be able to specify an instance type Compare: https://github.com/jenkinsci/support-core-plugin/compare/fc3119380c83...80b611d0d3a3
            Hide
            batmat Baptiste Mathus added a comment -

            Released in 2.37

            Show
            batmat Baptiste Mathus added a comment - Released in 2.37

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: