Skip to content

ci(jenkins): reuse agents (4 less agents to be used) #606

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
Merged
merged 4 commits into from
Oct 10, 2019
Merged

ci(jenkins): reuse agents (4 less agents to be used) #606

merged 4 commits into from
Oct 10, 2019

Conversation

@v1v
Copy link
Member

@v1v v1v commented Oct 9, 2019 •
edited
Loading

Highlights

  • Let's reuse the top-level agent to reduce the number of fo requested agents and speed up just a bit the builds.
  • Reset labels to be linux && immutable to help with what kind of OS is required to be used.
  • Overall time improvements from 4.30m to 3m until the test stage.

Test cases

  • Release
  • Validate the pre-commit docker doesn't leave any leftovers

@v1v v1v self-assigned this Oct 9, 2019
@v1v v1v added the automation label Oct 9, 2019
@v1v v1v force-pushed the feature/optimise_agents branch from 3f0d44c to 30b43cf Compare October 9, 2019 14:25
@v1v v1v added the ci label Oct 9, 2019
@v1v v1v requested a review from beniwohli October 9, 2019 16:58
@v1v v1v marked this pull request as ready for review October 9, 2019 16:58
beniwohli
beniwohli approved these changes Oct 10, 2019
View reviewed changes
Copy link
Contributor

@beniwohli beniwohli left a comment

Choose a reason for hiding this comment

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

LGTM!

@v1v v1v merged commit bddb81d into elastic:master Oct 10, 2019
@v1v v1v deleted the feature/optimise_agents branch October 10, 2019 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kuisathaverat kuisathaverat kuisathaverat approved these changes

+2 more reviewers

@beniwohli beniwohli beniwohli approved these changes

@mdelapenya mdelapenya mdelapenya approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@v1v v1v

Labels

automation ci

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@v1v @beniwohli @mdelapenya @kuisathaverat