[JENKINS-50307] While running in parallel for a loop, the index is wrong for each iteration - Jenkins Jira

XML

Word

Printable

Details

Type: Bug
Status: Resolved (View Workflow)
Priority: Minor
Resolution: Not A Defect
Component/s: workflow-cps-plugin
Labels:

Similar Issues:

Show

Description

pipeline
 {
     agent any
     stages
     {
         stage('Explore bug')
         {
              steps
             {
                  script
                  {
                      int maxIndex1=3
                      def loopers1 = [:]
                      for (int loopIndex1 = 0; loopIndex1 < maxIndex1; loopIndex1++)
                     {
                         echo "loopIndex1=${loopIndex1}"
                         loopers1["loop-$\{loopIndex1}"] =
                         {
                             echo "loopIndex1=${loopIndex1}"
                         }
                    }
                    parallel loopers1
               }
           }
     }
     stage('Bug workaround')
     {
         steps
         {
             script
             {
                 int maxIndex2=3
                 def loopers2 = [:]
                 for (int loopIndex2 = 0; loopIndex2 < maxIndex2; loopIndex2++)
                 {
                     int index2=loopIndex2
                     echo "index2=${index2}"
                     loopers2["loop-$\{index2}"] =
                     {
                         echo "index2=${index2}"
                     }
                }
               parallel loopers2
           } 
         }
       }
     }
 }

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

Jenkinsfile
1 kB
2018-03-21 04:19

Activity

Ascending order - Click to sort in descending order

Yoram Michaeli created issue - 2018-03-21 04:20

Yoram Michaeli made changes - 2018-03-21 04:28

Field	Original Value	New Value
Description	When using a parallel step for an indexed loop, using the loop-iterator-index for internal usage results with wrong value (showing the max-index-value for all iterations). A workaround for it is to set a local variable for saving the current iteration-index and use the local variable. The following pipeline demonstrates the problem (the issue is demonstrated with the internal 'echo' steps): {code:java} // code placeholder {code} pipeline \{ agent any stages \{ stage('Explore bug') \{ steps \{ script \{ int maxIndex1=3 def loopers1 = [:] for (int loopIndex1 = 0; loopIndex1 < maxIndex1; loopIndex1++) \{ echo "loopIndex1=$\{loopIndex1}" loopers1["loop-$\{loopIndex1}"] = \{ echo "loopIndex1=$\{loopIndex1}" } } parallel loopers1 } } } stage('Bug workaround') \{ steps \{ script \{ int maxIndex2=3 def loopers2 = [:] for (int loopIndex2 = 0; loopIndex2 < maxIndex2; loopIndex2++) \{ int index2=loopIndex2 echo "index2=$\{index2}" loopers2["loop-$\{index2}"] = \{ echo "index2=$\{index2}" } } parallel loopers2 } } } } }	When using a parallel step for an indexed loop, using the loop-iterator-index for internal usage results with wrong value (showing the max-index-value for all iterations). A workaround for it is to set a local variable for saving the current iteration-index and use the local variable. The following pipeline demonstrates the problem (the issue is demonstrated with the internal 'echo' steps): {code:java} // code placeholder {code} pipeline \{ agent any stages \{ stage('Explore bug') \{ steps \{ script \{ int maxIndex1=3 def loopers1 = [:] for (int loopIndex1 = 0; loopIndex1 < maxIndex1; loopIndex1++) \{ echo "loopIndex1=$\{loopIndex1}" loopers1["loop-$\\{loopIndex1}"] = \{ echo "loopIndex1=$\{loopIndex1}" } } parallel loopers1 } } } stage('Bug workaround') \{ steps \{ script \{ int maxIndex2=3 def loopers2 = [:] for (int loopIndex2 = 0; loopIndex2 < maxIndex2; loopIndex2++) \{ int index2=loopIndex2 echo "index2=$\{index2}" loopers2["loop-$\\{index2}"] = \{ echo "index2=$\{index2}" } } parallel loopers2 } } } } }

Yoram Michaeli made changes - 2018-03-21 04:29

Description

When using a parallel step for an indexed loop, using the loop-iterator-index for internal usage results with wrong value (showing the max-index-value for all iterations).
A workaround for it is to set a local variable for saving the current iteration-index and use the local variable.
The following pipeline demonstrates the problem (the issue is demonstrated with the internal 'echo' steps):
{code:java}
// code placeholder
{code}
pipeline
\{
agent any
stages
\{
stage('Explore bug')
\{
steps
\{
script
\{
int maxIndex1=3
def loopers1 = [:]
for (int loopIndex1 = 0; loopIndex1 < maxIndex1; loopIndex1++)
\{
echo "loopIndex1=$\{loopIndex1}"
loopers1["loop-$\\{loopIndex1}"] =
\{
echo "loopIndex1=$\{loopIndex1}"
}
}
parallel loopers1
}
}
}
stage('Bug workaround')
\{
steps
\{
script
\{
int maxIndex2=3
def loopers2 = [:]
for (int loopIndex2 = 0; loopIndex2 < maxIndex2; loopIndex2++)
\{
int index2=loopIndex2
echo "index2=$\{index2}"
loopers2["loop-$\\{index2}"] =
\{
echo "index2=$\{index2}"
}
}
parallel loopers2
}
}
}
}
}

When using a parallel step for an indexed loop, using the loop-iterator-index for internal usage results with wrong value (showing the max-index-value for all iterations).
A workaround for it is to set a local variable for saving the current iteration-index and use the local variable.
The following pipeline demonstrates the problem (the issue is demonstrated with the internal 'echo' steps):
{code:java}
// code placeholder
{code}
pipeline
\{
   agent any
   stages
   \{
   stage('Explore bug')
   \{
   steps
   \{
   script
   \{
   int maxIndex1=3
   def loopers1 = [:]
   for (int loopIndex1 = 0; loopIndex1 < maxIndex1; loopIndex1++)
   \{
   echo "loopIndex1=$\{loopIndex1}"
   loopers1["loop-$\{loopIndex1}"] =
   \{
   echo "loopIndex1=$\{loopIndex1}"
   }
   }
   parallel loopers1
   }
   }
   }
   stage('Bug workaround')
   \{
   steps
   \{
   script
   \{
   int maxIndex2=3
   def loopers2 = [:]
   for (int loopIndex2 = 0; loopIndex2 < maxIndex2; loopIndex2++)
   \{
   int index2=loopIndex2
   echo "index2=$\{index2}"
   loopers2["loop-$\{index2}"] =
   \{
   echo "index2=$\{index2}"
   }
   }
   parallel loopers2
   }
   }
   }
   }
}

Yoram Michaeli made changes - 2018-03-21 12:19

Description

When using a parallel step for an indexed loop, using the loop-iterator-index for internal usage results with wrong value (showing the max-index-value for all iterations).
A workaround for it is to set a local variable for saving the current iteration-index and use the local variable.
The following pipeline demonstrates the problem (the issue is demonstrated with the internal 'echo' steps):
{code:java}
// code placeholder
{code}
pipeline
\{
   agent any
   stages
   \{
   stage('Explore bug')
   \{
   steps
   \{
   script
   \{
   int maxIndex1=3
   def loopers1 = [:]
   for (int loopIndex1 = 0; loopIndex1 < maxIndex1; loopIndex1++)
   \{
   echo "loopIndex1=$\{loopIndex1}"
   loopers1["loop-$\{loopIndex1}"] =
   \{
   echo "loopIndex1=$\{loopIndex1}"
   }
   }
   parallel loopers1
   }
   }
   }
   stage('Bug workaround')
   \{
   steps
   \{
   script
   \{
   int maxIndex2=3
   def loopers2 = [:]
   for (int loopIndex2 = 0; loopIndex2 < maxIndex2; loopIndex2++)
   \{
   int index2=loopIndex2
   echo "index2=$\{index2}"
   loopers2["loop-$\{index2}"] =
   \{
   echo "index2=$\{index2}"
   }
   }
   parallel loopers2
   }
   }
   }
   }
}

When using a parallel step for an indexed loop, using the loop-iterator-index for internal usage results with wrong value (showing the max-index-value for all iterations).
A workaround for it is to set a local variable for saving the current iteration-index and use the local variable.
The following pipeline demonstrates the problem (the issue is demonstrated with the internal 'echo' steps):
{code:java}
pipeline
{
   agent any
   stages
   {
   stage('Explore bug')
   {
   steps
   {
   script
   {
   int maxIndex1=3
   def loopers1 = [:]
   for (int loopIndex1 = 0; loopIndex1 < maxIndex1; loopIndex1++)
   {
   echo "loopIndex1=${loopIndex1}"
   loopers1["loop-$\{loopIndex1}"] =
   {
   echo "loopIndex1=${loopIndex1}"
   }
   }
   parallel loopers1
   }
   }
   }
   stage('Bug workaround')
   {
   steps
   {
   script
   {
   int maxIndex2=3
   def loopers2 = [:]
   for (int loopIndex2 = 0; loopIndex2 < maxIndex2; loopIndex2++)
   {
   int index2=loopIndex2
   echo "index2=${index2}"
   loopers2["loop-$\{index2}"] =
   {
   echo "index2=${index2}"
   }
   }
   parallel loopers2
   }
   }
   }
   }
}{code}

Andrew Bayer made changes - 2018-03-27 17:58

Component/s		workflow-cps-plugin [ 21713 ]
Component/s	pipeline [ 21692 ]

Vivek Pandey made changes - 2018-11-15 22:53

Labels

parallel pipeline

parallel pipeline triaged-2018-11

Jesse Glick made changes - 2019-04-04 21:13

Resolution		Not A Defect [ 7 ]
Status	Open [ 1 ]	Resolved [ 5 ]

People

Assignee:: Unassigned

Reporter:: Yoram Michaeli

Votes:: 7 Vote for this issue

Watchers:: 2 Start watching this issue

Dates

Created:: 2018-03-21 04:20

Updated:: 2019-04-04 21:13

Resolved:: 2019-04-04 21:13