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

[MultipleSCM integration] - Perforce SCM does not work in MultipleSCMs

XMLWordPrintable

      Initial description:

      Usage of Perforce plugin in the set-up of multiple SCM section leads to Java exception with no explanations of failure. Seems that Perforce plugin somehow ignores "View Map" parameters in "newInstance" handler.

      The real issues:

      • MultipleSCMs use DataBoundConstructor instead of newInstance
      • radioBlock names are not unique
      • ...

          [JENKINS-18583] [MultipleSCM integration] - Perforce SCM does not work in MultipleSCMs

          Oleg Nenashev created issue -

          Rob Petti added a comment -

          Are you setting up more than one PerforceSCM? Does it happen when you only set up one under MultipleSCM?

          Rob Petti added a comment - Are you setting up more than one PerforceSCM? Does it happen when you only set up one under MultipleSCM?

          Oleg Nenashev added a comment -

          Hello, issue is being reproduced for one Perforce SCM as well

          Oleg Nenashev added a comment - Hello, issue is being reproduced for one Perforce SCM as well
          Oleg Nenashev made changes -
          Assignee Original: Rob Petti [ rpetti ] New: Oleg Nenashev [ oleg_nenashev ]
          Oleg Nenashev made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          pom.xml
          src/main/java/com/synopsys/arc/jenkinsci/plugins/perforce/DepotType.java
          src/main/java/hudson/plugins/perforce/PerforceSCM.java
          src/main/resources/hudson/plugins/perforce/PerforceSCM/config.jelly
          http://jenkins-ci.org/commit/perforce-plugin/21f91d6ae68ea120985876e8a6116a0c1dd5ce62
          Log:
          [PerforceSCM.DataBoundConstructor] - Added "proof-of-concept" implementation of the bugfix.
          Related issue: https://issues.jenkins-ci.org/browse/JENKINS-18583

          Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: pom.xml src/main/java/com/synopsys/arc/jenkinsci/plugins/perforce/DepotType.java src/main/java/hudson/plugins/perforce/PerforceSCM.java src/main/resources/hudson/plugins/perforce/PerforceSCM/config.jelly http://jenkins-ci.org/commit/perforce-plugin/21f91d6ae68ea120985876e8a6116a0c1dd5ce62 Log: [PerforceSCM.DataBoundConstructor] - Added "proof-of-concept" implementation of the bugfix. Related issue: https://issues.jenkins-ci.org/browse/JENKINS-18583 Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>

          Oleg Nenashev added a comment -

          Hello Robert,

          Just for review… Pull request: https://github.com/jenkinsci/perforce-plugin/pull/33

          Proposed fix resolves issue with instantiation of Perforce plugin in the MultipleSCM’s hetero list. Issue was in the incomplete handling of data in the DataBoundConstructor.

          If fix is applicable, I’ll modify unit tests and remove temporary stuff from the *.pom. Then, we will be able to merge pull request.

          Oleg Nenashev added a comment - Hello Robert, Just for review… Pull request: https://github.com/jenkinsci/perforce-plugin/pull/33 Proposed fix resolves issue with instantiation of Perforce plugin in the MultipleSCM’s hetero list. Issue was in the incomplete handling of data in the DataBoundConstructor. If fix is applicable, I’ll modify unit tests and remove temporary stuff from the *.pom. Then, we will be able to merge pull request.

          Oleg Nenashev added a comment - - edited

          Proposed fix works for single Perforce SCM only.
          So, you're able to use P4 with other SCMs...

          If we want to have multiple SCMs, we need to fix PerforceSCM's config.jelly:

          • RadioBlock has single-select between multiple P4 instantiations (so we can't set view source for two SCMs)
          • Issue affects all radio-blocks => Probably, we need another approach

          I'll continue work on this issue. Probably, it should be left for future release.

          Oleg Nenashev added a comment - - edited Proposed fix works for single Perforce SCM only. So, you're able to use P4 with other SCMs... If we want to have multiple SCMs, we need to fix PerforceSCM's config.jelly: RadioBlock has single-select between multiple P4 instantiations (so we can't set view source for two SCMs) Issue affects all radio-blocks => Probably, we need another approach I'll continue work on this issue. Probably, it should be left for future release.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          src/main/java/hudson/plugins/perforce/PerforceSCM.java
          src/main/resources/hudson/plugins/perforce/PerforceSCM/config.jelly
          http://jenkins-ci.org/commit/perforce-plugin/96bd5734090a329b168b9763468d34a5ee341ae2
          Log:
          JENKINS-18583 - Added propagation of Instance ID to the newInstance() method

          Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: src/main/java/hudson/plugins/perforce/PerforceSCM.java src/main/resources/hudson/plugins/perforce/PerforceSCM/config.jelly http://jenkins-ci.org/commit/perforce-plugin/96bd5734090a329b168b9763468d34a5ee341ae2 Log: JENKINS-18583 - Added propagation of Instance ID to the newInstance() method Signed-off-by: Oleg Nenashev <nenashev@synopsys.com>
          Oleg Nenashev made changes -
          Description Original: Usage of Perforce plugin in the set-up of multiple SCM section leads to Java exception with no explanations of failure.

          Seems that Perforce plugin somehow ignores "View Map" parameters in "newInstance" handler.           		New: Initial description:
          {quote}
          Usage of Perforce plugin in the set-up of multiple SCM section leads to Java exception with no explanations of failure. Seems that Perforce plugin somehow ignores "View Map" parameters in "newInstance" handler.
          {quote}

          The real issues:
          - MultipleSCMs use DataBoundConstructor instead of newInstance
          - radioBlock names are not unique
          - ...
          Summary Original: [MultipleSCM integration] - "View Map" is empty in case of multiple Perforce SCM instances New: [MultipleSCM integration] - Perforce SCM does not work in MultipleSCMs

            oleg_nenashev Oleg Nenashev
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: