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

New flag to use only the parameters from the last comment

    XMLWordPrintable

    Details

    • Type: New Feature
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Cannot Reproduce
    • Labels:
      None
    • Environment:
      Jenkins LTS 1.625.3
      Stash Pull Request Build plugin 1.5.3
      Bitbucket 4.5.1
    • Similar Issues:

      Description

      Hi,

      We are using Jenkins and Bitbucket to manage our CI environment.
      Our pull request jobs are triggered with the Stash pull request builder plugin.

      Last week, we have updated our Jenkins PR job to launch some functional tests if the developer (or the QA) pass some parameters using this syntax:

      test this please
      p:testing_scope=smoke
      

      If Jenkins find a value for testing_scope, it will trigger a PR job + a functional test job (it will use the parameter value to determine the testing scope).
      It's really useful to launch some tests on a pull request.

      If this comment (with the testing_scope parameter) remains on the PR, when a developer launch a new PR build with "test this please" (or the build is launched on a new commit), the plugin will find the testing_scope parameter in the old comment and will use it to trigger the Jenkins PR job.
      In this case, the developer only want to trigger a PR job. He doesn't want to launch the functional tests.

      Is it intentional to scan all the old comments to find the parameters?
      If so, what we propose is to add a new parameter (in the advanced section of the plugin) to read the parameters only in the last Bitbucket comment:
      Only use the parameters from the last comment

      If the parameter value is true, then the plugin will ignore the parameters found in the old comment.
      It will solve our issue.

      Can we propose a development to add this new parameter?

      Thanks,
      Bruno, release manager at https://www.forgerock.com/

        Attachments

          Activity

          Hide
          jbochenski Jakub Bochenski added a comment -

          This issue has been automatically closed because of inactivity. Please reopen it if you think it's still valid

          Show
          jbochenski Jakub Bochenski added a comment - This issue has been automatically closed because of inactivity. Please reopen it if you think it's still valid

            People

            Assignee:
            nemccarthy nathan m
            Reporter:
            brunolavit Bruno Lavit
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: