• Blue Ocean 1.1, Blue Ocean 1.2-beta1, Blue Ocean 1.2-beta2, Blue Ocean 1.2-beta3, Blue Ocean 1.2-beta4, Blue Ocean 1.2, Blue Ocean 1.3

      In Scope

      • We need to ensure that Parallel Stages (JENKINS-41334) can be edited in the same way that parallel can be.
      • When creating parallels we should be supporting this format.
      • Automatically convert old parallels to new format.

      Notes

          [JENKINS-44390] Support for editing parallel stages

          James Dumay added a comment -

          abayer what is the situation with old parallel syntax? When we convert from groovy to JSON will we get a parallel stage block back for the old parallel syntax?

          My worry is that when we introduce config at the parallel level in the editor that we will run into serialization issues.

          James Dumay added a comment - abayer what is the situation with old parallel syntax? When we convert from groovy to JSON will we get a parallel stage block back for the old parallel syntax? My worry is that when we introduce config at the parallel level in the editor that we will run into serialization issues.

          Andrew Bayer added a comment -

          You will not get a parallel stage block back - you'll get the old parallel syntax from the old parallel syntax. No magic.

          Andrew Bayer added a comment - You will not get a parallel stage block back - you'll get the old parallel syntax from the old parallel syntax. No magic.

          James Dumay added a comment -

          kzantow abayer hmm how should we handle the case where a user wants to set an agent for a parallel block but their backing Jenkinsfile uses the old style parallel?

          James Dumay added a comment - kzantow abayer hmm how should we handle the case where a user wants to set an agent for a parallel block but their backing Jenkinsfile uses the old style parallel?

          Andrew Bayer added a comment -

          Well, I suppose the editor could read old-style parallel but output new-style parallel regardless of whether the config changed? I'm wary of making Declarative itself translate from old-style to new-style when going to JSON, but it might be more reasonable to do that in the editor...

          Andrew Bayer added a comment - Well, I suppose the editor could read old-style parallel but output new-style parallel regardless of whether the config changed? I'm wary of making Declarative itself translate from old-style to new-style when going to JSON, but it might be more reasonable to do that in the editor...

          James Dumay added a comment -

          cliffmeyers we'd like you to pick up some more editor work. You will need to organise a handover from kzantow

          James Dumay added a comment - cliffmeyers we'd like you to pick up some more editor work. You will need to organise a handover from kzantow

          Cliff Meyers added a comment -

          Cliff Meyers added a comment - abayer gave me this link as a ref to the most up to date step for parallel stages: https://github.com/abayer/pipeline-model-definition-plugin/blob/jenkins-41334-mk2/rfc/JENKINS-41334-parallel-stage-execution.md

          Cliff Meyers added a comment -

          Cliff Meyers added a comment - And some examples: https://github.com/jenkinsci/pipeline-model-definition-plugin/pull/152/files#diff-13c773a7c988e725755ddaa5e6d910a2

          James Dumay added a comment -

          Ive added a link to the backend PR>

          James Dumay added a comment - Ive added a link to the backend PR>

          James Dumay added a comment -

          cliffmeyers can this be your next task?

          James Dumay added a comment - cliffmeyers can this be your next task?

          Keith Zantow added a comment -

          jamesdumay cliffmeyers I spoke with Andrew about this briefly today as he's eager to get the new parallel syntax in. The changes for this ticket should be fairly trivial and I could probably get it knocked out in a few hours, it only has to do with modifying the declarative JSON conversion functions to a) support the new parallel model and b) always output the new parallel model instead of the old one.

          Keith Zantow added a comment - jamesdumay cliffmeyers I spoke with Andrew about this briefly today as he's eager to get the new parallel syntax in. The changes for this ticket should be fairly trivial and I could probably get it knocked out in a few hours, it only has to do with modifying the declarative JSON conversion functions to a) support the new parallel model and b) always output the new parallel model instead of the old one.

          Cliff Meyers added a comment -

          kzantow I've got easily another day left on JENKINS-42791 so if you want to grab this one, by all means go for it. I can still lend a hand w/ the Git public key credential stuff once that one is done - which hopefully helps you out a bit.

          Cliff Meyers added a comment - kzantow I've got easily another day left on JENKINS-42791 so if you want to grab this one, by all means go for it. I can still lend a hand w/ the Git public key credential stuff once that one is done - which hopefully helps you out a bit.

          Keith Zantow added a comment -

          cliffmeyers yes, I have just a bit of testing on the git backend and I'll pick this up while waiting on reviews and such; a good switch regarding the public key display, it probably is blocked a bit on what you're working on, anyway, since it's directly related to the same area

          Keith Zantow added a comment - cliffmeyers yes, I have just a bit of testing on the git backend and I'll pick this up while waiting on reviews and such; a good switch regarding the public key display, it probably is blocked a bit on what you're working on, anyway, since it's directly related to the same area

          Michael Neale added a comment -

          Is this one really different to https://issues.jenkins-ci.org/browse/JENKINS-41960 - sounds more and more like a duplicate. 

          Michael Neale added a comment - Is this one really different to https://issues.jenkins-ci.org/browse/JENKINS-41960  - sounds more and more like a duplicate. 

          Cliff Meyers added a comment -

          Seems that way to me, yes.

          Cliff Meyers added a comment - Seems that way to me, yes.

          Keith Zantow added a comment -

          michaelneale this is effectively the same as JENKINS-41960 I'll enable the same stage editing as part of this ticket.

          Keith Zantow added a comment - michaelneale this is effectively the same as JENKINS-41960 I'll enable the same stage editing as part of this ticket.

            kzantow Keith Zantow
            jamesdumay James Dumay
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: