java.util.concurrent.ExecutionException: java.util.concurrent.RejectedExecutionException
      	at com.google.common.util.concurrent.AbstractFuture$Sync.getValue(AbstractFuture.java:289)
      	at com.google.common.util.concurrent.AbstractFuture$Sync.get(AbstractFuture.java:276)
      	at com.google.common.util.concurrent.AbstractFuture.get(AbstractFuture.java:111)
      	at org.jenkinsci.plugins.workflow.cps.CpsFlowExecution.waitForSuspension(CpsFlowExecution.java:780)
      	at org.jenkinsci.plugins.workflow.cps.CpsScriptTest.methodNameClash(CpsScriptTest.java:84)
      	at ...
      Caused by: java.util.concurrent.RejectedExecutionException
      	at hudson.remoting.SingleLaneExecutorService.execute(SingleLaneExecutorService.java:100)
      	at java.util.concurrent.AbstractExecutorService.submit(AbstractExecutorService.java:112)
      	at com.google.common.util.concurrent.ForwardingExecutorService.submit(ForwardingExecutorService.java:110)
      	at jenkins.util.InterceptingExecutorService.submit(InterceptingExecutorService.java:49)
      	at org.jenkinsci.plugins.workflow.cps.CpsThreadGroup$2.call(CpsThreadGroup.java:252)
      	at org.jenkinsci.plugins.workflow.cps.CpsThreadGroup$2.call(CpsThreadGroup.java:230)
      	at org.jenkinsci.plugins.workflow.cps.CpsVmExecutorService$2.call(CpsVmExecutorService.java:64)
      	at ...
      

      from this.

      I am inclined to just deprecate waitForSuspension in favor of SemaphoreStep. It is mostly only a few really old tests which still use it. There are some tricky places like CpsFlowExecutionTest.quietDown (for which see JENKINS-46074).

          [JENKINS-46073] Test flake in CpsScriptTest.methodNameClash

          Jesse Glick created issue -
          Jesse Glick made changes -
          Description Original: {code:none}
          java.util.concurrent.ExecutionException: java.util.concurrent.RejectedExecutionException
          at com.google.common.util.concurrent.AbstractFuture$Sync.getValue(AbstractFuture.java:289)
          at com.google.common.util.concurrent.AbstractFuture$Sync.get(AbstractFuture.java:276)
          at com.google.common.util.concurrent.AbstractFuture.get(AbstractFuture.java:111)
          at org.jenkinsci.plugins.workflow.cps.CpsFlowExecution.waitForSuspension(CpsFlowExecution.java:780)
          at org.jenkinsci.plugins.workflow.cps.CpsScriptTest.methodNameClash(CpsScriptTest.java:84)
          at ...
          Caused by: java.util.concurrent.RejectedExecutionException
          at hudson.remoting.SingleLaneExecutorService.execute(SingleLaneExecutorService.java:100)
          at java.util.concurrent.AbstractExecutorService.submit(AbstractExecutorService.java:112)
          at com.google.common.util.concurrent.ForwardingExecutorService.submit(ForwardingExecutorService.java:110)
          at jenkins.util.InterceptingExecutorService.submit(InterceptingExecutorService.java:49)
          at org.jenkinsci.plugins.workflow.cps.CpsThreadGroup$2.call(CpsThreadGroup.java:252)
          at org.jenkinsci.plugins.workflow.cps.CpsThreadGroup$2.call(CpsThreadGroup.java:230)
          at org.jenkinsci.plugins.workflow.cps.CpsVmExecutorService$2.call(CpsVmExecutorService.java:64)
          at ...
          {code}

          from [this|https://github.com/jenkinsci/workflow-cps-plugin/blob/9b52631a7c4391d4698811f03f1445968e5a748f/src/test/java/org/jenkinsci/plugins/workflow/cps/CpsScriptTest.java#L84].

          I am inclined to just deprecate {{waitForSuspension}} in favor of {{SemaphoreStep}}. It is mostly only a few really old tests which still use it. There are some tricky places like {{CpsFlowExecutionTest.quietDown}}.
          New: {code:none}
          java.util.concurrent.ExecutionException: java.util.concurrent.RejectedExecutionException
          at com.google.common.util.concurrent.AbstractFuture$Sync.getValue(AbstractFuture.java:289)
          at com.google.common.util.concurrent.AbstractFuture$Sync.get(AbstractFuture.java:276)
          at com.google.common.util.concurrent.AbstractFuture.get(AbstractFuture.java:111)
          at org.jenkinsci.plugins.workflow.cps.CpsFlowExecution.waitForSuspension(CpsFlowExecution.java:780)
          at org.jenkinsci.plugins.workflow.cps.CpsScriptTest.methodNameClash(CpsScriptTest.java:84)
          at ...
          Caused by: java.util.concurrent.RejectedExecutionException
          at hudson.remoting.SingleLaneExecutorService.execute(SingleLaneExecutorService.java:100)
          at java.util.concurrent.AbstractExecutorService.submit(AbstractExecutorService.java:112)
          at com.google.common.util.concurrent.ForwardingExecutorService.submit(ForwardingExecutorService.java:110)
          at jenkins.util.InterceptingExecutorService.submit(InterceptingExecutorService.java:49)
          at org.jenkinsci.plugins.workflow.cps.CpsThreadGroup$2.call(CpsThreadGroup.java:252)
          at org.jenkinsci.plugins.workflow.cps.CpsThreadGroup$2.call(CpsThreadGroup.java:230)
          at org.jenkinsci.plugins.workflow.cps.CpsVmExecutorService$2.call(CpsVmExecutorService.java:64)
          at ...
          {code}
          from [this|https://github.com/jenkinsci/workflow-cps-plugin/blob/9b52631a7c4391d4698811f03f1445968e5a748f/src/test/java/org/jenkinsci/plugins/workflow/cps/CpsScriptTest.java#L84].

          I am inclined to just deprecate {{waitForSuspension}} in favor of {{SemaphoreStep}}. It is mostly only a few really old tests which still use it. There are some tricky places like {{CpsFlowExecutionTest.quietDown}} (for which see JENKINS-46074).
          Vivek Pandey made changes -
          Labels Original: flake New: flake triaged-2018-11

            Unassigned Unassigned
            jglick Jesse Glick
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: