Skip to content
This repository has been archived by the owner on Jan 4, 2023. It is now read-only.

Commit

Permalink
Add a small wait in the Jenkinsfile used by multibranch/testResults.js (
Browse files Browse the repository at this point in the history
#87)

* Add a small wait in the Jenkinsfile used by multibranch/testResults.js

This is to ensure we don't hit random flaky fails due to JENKINS-36408

* A bit more detail in the comment
  • Loading branch information
tfennelly committed Dec 15, 2016
1 parent 6a8bf7b commit b3b447e
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions src/test/resources/multibranch/test_results/Jenkinsfile
@@ -1,6 +1,18 @@
node {
stage 'Stage 1 - scm'
echo 'Stage 1 - scm'

//
// Need a small wait here so as not to hit JENKINS-36408.
// Without it, the build run SSE end events *can* arrive at the client
// before the it has finished processing the earlier events, which means
// the end events are meaningless to it (no proper context until earlier
// events are processed), causing the UI state for that run to end up in
// limbo state - constant spinning orb etc requiring a page reload to fix.
//
sleep 2


checkout scm
stage 'Stage test'
echo 'stuff'
Expand Down

1 comment on commit b3b447e

@michaelneale
Copy link
Member

@michaelneale michaelneale commented on b3b447e Dec 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, good find!

And this was an unrealistic pipeline (have been bitten by those before) - doens't mean shouldn't fix, but does mean not likely to see it outside of "hello world"

Please sign in to comment.