Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Re-add pipeline for flaky test runner job #48781

Merged
merged 5 commits into from
Oct 25, 2019
Merged

Re-add pipeline for flaky test runner job #48781

merged 5 commits into from
Oct 25, 2019

Conversation

brianseeders
Copy link
Contributor

Revert previously reverted commit (due to failing master job) and reconcile changes made to Jenkinsfile since then

This was originally merged here: #46740

@brianseeders brianseeders added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:CI Continuous integration v7.5.0 v7.6.0 v6.8.4 v7.4.1 labels Oct 21, 2019
@brianseeders brianseeders requested review from spalger and a team October 21, 2019 15:19
@brianseeders brianseeders self-assigned this Oct 21, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@brianseeders
Revert "Revert "Add pipeline for flaky test runner job (elastic#46740)""
33623ad 
This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened
@brianseeders
Fix param parsing and add missed change
b9388ea
@brianseeders brianseeders force-pushed the flaky-test-runner-remix branch from 53aa2c1 to b9388ea Compare October 22, 2019 16:23
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

wayneseymour
wayneseymour approved these changes Oct 24, 2019
View reviewed changes
Copy link
Member

@wayneseymour wayneseymour left a comment

Choose a reason for hiding this comment

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

I like it, but it'd be nice to have a lot of these exported.
Is that already "a thing"? For example, if qa ends up having n number of
Jenkinsfiles, could we re-use these defs?

.ci/Jenkinsfile_flaky Show resolved Hide resolved
.ci/Jenkinsfile_flaky Show resolved Hide resolved
src/dev/ci_setup/setup_env.sh Show resolved Hide resolved
test/scripts/jenkins_ci_group.sh Show resolved Hide resolved
tylersmalley
tylersmalley approved these changes Oct 25, 2019
View reviewed changes
Copy link
Contributor

@tylersmalley tylersmalley left a comment

Choose a reason for hiding this comment

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

Read through it and didn't see any issues other than a few questions unrelated to this PR

@brianseeders brianseeders merged commit 9da6c07 into elastic:master Oct 25, 2019
@brianseeders brianseeders deleted the flaky-test-runner-remix branch October 25, 2019 16:57
@spalger
Copy link
Contributor

spalger commented Oct 25, 2019

🎉 💃

spalger pushed a commit to spalger/kibana that referenced this pull request Oct 31, 2019
@brianseeders
Re-add pipeline for flaky test runner job (elastic#48781)
1eaa45e 
* Revert "Revert "Add pipeline for flaky test runner job (elastic#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
@brianseeders brianseeders mentioned this pull request Oct 31, 2019
Merged
brianseeders added a commit to brianseeders/kibana that referenced this pull request Oct 31, 2019
@brianseeders
Re-add pipeline for flaky test runner job (elastic#48781)
cabc514 
* Revert "Revert "Add pipeline for flaky test runner job (elastic#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
@brianseeders brianseeders mentioned this pull request Oct 31, 2019
Merged
brianseeders added a commit to brianseeders/kibana that referenced this pull request Oct 31, 2019
@brianseeders
Re-add pipeline for flaky test runner job (elastic#48781)
aa8e7dc 
* Revert "Revert "Add pipeline for flaky test runner job (elastic#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
@brianseeders brianseeders mentioned this pull request Oct 31, 2019
Merged
brianseeders added a commit to brianseeders/kibana that referenced this pull request Oct 31, 2019
@brianseeders
Re-add pipeline for flaky test runner job (elastic#48781)
f261cc9 
* Revert "Revert "Add pipeline for flaky test runner job (elastic#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
brianseeders added a commit to brianseeders/kibana that referenced this pull request Oct 31, 2019
@brianseeders
Re-add pipeline for flaky test runner job (elastic#48781)
07c6d9b 
* Revert "Revert "Add pipeline for flaky test runner job (elastic#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
@brianseeders brianseeders mentioned this pull request Oct 31, 2019
Closed
brianseeders added a commit that referenced this pull request Nov 15, 2019
@brianseeders
Re-add pipeline for flaky test runner job (#48781) (#49907)
99ff78c 
* Revert "Revert "Add pipeline for flaky test runner job (#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
brianseeders added a commit that referenced this pull request Nov 15, 2019
@brianseeders
[7.5] Re-add pipeline for flaky test runner job (#48781) (#49906)
f68a072 
* Revert "Revert "Add pipeline for flaky test runner job (#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
brianseeders added a commit that referenced this pull request Nov 15, 2019
@brianseeders
[7.x] Re-add pipeline for flaky test runner job (#48781) (#49905)
ec70565 
* Revert "Revert "Add pipeline for flaky test runner job (#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
brianseeders added a commit to brianseeders/kibana that referenced this pull request Nov 26, 2019
@brianseeders
Re-add pipeline for flaky test runner job (elastic#48781)
fc5816b 
* Revert "Revert "Add pipeline for flaky test runner job (elastic#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable
brianseeders added a commit that referenced this pull request Dec 2, 2019
@brianseeders
Backport/6.8 - lots of Pipeline updates (#51754)
c2d5390 
* Re-add pipeline for flaky test runner job (#48781)

* Revert "Revert "Add pipeline for flaky test runner job (#46740)""

This reverts commit 7d96a13.

Also reconcile changes to Jenkinsfile since original revert happened

* Fix param parsing and add missed change

* Add missing variable

* Retry git clone up to 8 times before failing a build (#50734)

* Flaky test runner improvements/fixes (#49367)

* Allocate multiple flaky test runner agents as executions grow

* WIP support for deleting kibana install dir during CI

* Add setup script for testing scripts

* Add REMOVE_KIBANA_INSTALL_DIR=1 to flaky test runner

* Change flaky test runner worker processes from 8 to 12

* Add labels to shell scripts in Jenkins (#49657)

* Change pipeline timeout from 3 hours to 2 hours (#51098)

* Remove flaky pipeline step not relevant for 6.8

* Re-add failed_tests cli

* Fix PARALLEL_PIPELINE_WORKER_INDEX var
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wayneseymour wayneseymour wayneseymour approved these changes

@tylersmalley tylersmalley tylersmalley approved these changes

@spalger spalger Awaiting requested review from spalger

Assignees

@brianseeders brianseeders

Labels
Feature:CI Continuous integration release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v6.8.4 v7.4.1 v7.5.0 v7.6.0 v8.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@brianseeders @elasticmachine @spalger @tylersmalley @wayneseymour