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

Inputs with multiple approvers do not work

    XMLWordPrintable

Details

    • Blue Ocean 1.0-rc3

    Description

      Notes
      This will work when there is a single approver specified but not when there are multiple. Blue Ocean will respond with a 400 error. Due to JENKINS-42920 the user will not receive feedback on why they are not allowed to proceed or any other error, which made this problem more confusing.

      To run these scenarios you will need two users - 'alice' and 'bob'.

      Scenario 1 - current user is 'alice', single approver
      This works fine in Blue Ocean

      node {
          echo 'hello world'
          input message: "Do you want to approve this job to deploy to production?", ok: "OK", submitter: "alice"
      }
      

      Scenario 2 - current user is 'alice', multiple approvers

      node {
          echo 'hello world'
          input message: "Do you want to approve this job to deploy to production?", ok: "OK", submitter: "alice,bob"
      }
      

      Original request
      I use input step like below:

      timeout(time: 5, unit: "DAYS") \{
      
          input message: "Do you want to approve this job to deploy to production?", ok: "OK", submitter: "A,B,C"
      
      }
      

      can not use multi submitter in Blue Ocean UI, but it works fine in the old UI.

      Attachments

        Issue Links

          Activity

            jamesdumay James Dumay added a comment -

            vivek would you mind triaging this one when you have a minute?

            jamesdumay James Dumay added a comment - vivek would you mind triaging this one when you have a minute?
            michaelneale Michael Neale added a comment -

            Actually this may be API only - perhaps imeredith could take a look to diagnose. If it is a big one - lets move it out of this sprint. 

            michaelneale Michael Neale added a comment - Actually this may be API only - perhaps imeredith could take a look to diagnose. If it is a big one - lets move it out of this sprint. 
            michaelneale Michael Neale added a comment -

            hanks there is a fix in review for this, but it uses a deprecated API so trying to get the bottom of that... 

            michaelneale Michael Neale added a comment - hanks there is a fix in review for this, but it uses a deprecated API so trying to get the bottom of that... 
            michaelneale Michael Neale added a comment -

            I am not able to reproduce the setup with the instructions, I can't get classic to accept submitting in any case given, and not sure what matrix settings to use. 

            michaelneale Michael Neale added a comment - I am not able to reproduce the setup with the instructions, I can't get classic to accept submitting in any case given, and not sure what matrix settings to use. 
            hanks hanks justin added a comment -

            michaelneale Thank you for your quick job, Seems there should be no space in submitter settings like below
            submitter: "A,B,C"
            the classic UI

            hanks hanks justin added a comment - michaelneale Thank you for your quick job, Seems there should be no space in submitter settings like below submitter: "A,B,C" the classic UI
            michaelneale Michael Neale added a comment -

            hanks that is the opposite to what imeredith saw though - so I am not sure what is going on. In my case - I can't even get classic to work with this, so not sure what is going on. 

            michaelneale Michael Neale added a comment - hanks that is the opposite to what imeredith saw though - so I am not sure what is going on. In my case - I can't even get classic to work with this, so not sure what is going on. 
            michaelneale Michael Neale added a comment -

            should be in next release shortly...

            michaelneale Michael Neale added a comment - should be in next release shortly...
            hanks hanks justin added a comment -

            Thanks!

            hanks hanks justin added a comment - Thanks!

            People

              imeredith Ivan Meredith
              hanks hanks justin
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: