Skip to content

Jenkinsfile: split e2e tests to separate stages#2433

Merged
silvin-lubecki merged 1 commit intodocker:masterfrom
thaJeztah:jenkins_split
Apr 21, 2020
Merged

Jenkinsfile: split e2e tests to separate stages#2433
silvin-lubecki merged 1 commit intodocker:masterfrom
thaJeztah:jenkins_split

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Apr 10, 2020

The non-experimental, experimental, and ssh-connhelper stages are running the same tests, which makes it difficult to find which variant of the test failed if they all run in the same stage.

With this change:

Screenshot 2020-04-10 at 16 23 07

The non-experimental, experimental, and ssh-connhelper stages
are running the same tests, which makes it difficult to find
_which_ variant of the test failed if they all run in the
same stage.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@tiborvass
Copy link
Collaborator

I'd rather avoid jenkinsifying too much. Can't we add it to the test name instead?

@thaJeztah
Copy link
Member Author

They're the same tests, only with different environment variables set; see it as similar to running tests with CGO_ENABLED=1 and CGO_ENABLED=0. If we migrate to github actions, they will probably end up in a matrix, which is similar to this

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM

@silvin-lubecki silvin-lubecki merged commit 7a006fb into docker:master Apr 21, 2020
@thaJeztah thaJeztah deleted the jenkins_split branch April 21, 2020 15:02
@thaJeztah thaJeztah added this to the 20.03.0 milestone Apr 28, 2020
@thaJeztah
Copy link
Member Author

looks like this was rolled up into #2538 for 19.03, so removing the "cherry-pick" label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants