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.

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

Migrate job to build test docker images to beats repo #18785

Merged
merged 2 commits into from
May 28, 2020

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented May 27, 2020

What does this PR do?

Migrate job to pre-build docker images used in integration tests to beats repository.

Why is it important?

To have the definition of this job in the beats repository, as is related to the beats project.

@jsoriano jsoriano added in progress Pull request is currently in progress. [zube]: In Progress Team:Automation Label for the Observability productivity team labels May 27, 2020
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels May 27, 2020
@jsoriano jsoriano force-pushed the build-docker-images-jenkins branch from 73a0821 to 9f6ba83 Compare May 27, 2020 17:24
@elasticmachine
Copy link
Collaborator

elasticmachine commented May 27, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #18785 updated]

  • Start Time: 2020-05-27T17:37:23.878+0000

  • Duration: 84 min 48 sec

Test stats 🧪

Test Results
Failed 0
Passed 9242
Skipped 1542
Total 10784

@jsoriano jsoriano self-assigned this May 27, 2020
@mdelapenya
Copy link
Contributor

What do you think about moving the job as-is, and then once working apply the filebeat changes?

@jsoriano
Copy link
Member Author

Changes for filebeat reverted, now the pipeline and the script are [mostly] as the original ones.

@jsoriano jsoriano added [zube]: In Review review and removed [zube]: In Progress in progress Pull request is currently in progress. labels May 27, 2020
@jsoriano jsoriano marked this pull request as ready for review May 27, 2020 17:42
@jsoriano jsoriano merged commit 1012d52 into elastic:master May 28, 2020
@jsoriano jsoriano deleted the build-docker-images-jenkins branch May 28, 2020 09:37
quietPeriod(10)
}
triggers {
cron '@daily'
Copy link
Contributor

Choose a reason for hiding this comment

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

We usually schedule jobs on a range of time (to avoid overload the CI) an from Monday to Friday, on weekend the CI rest :D

Suggested change
cron '@daily'
cron 'H H(0-5) * * 1-5'

Copy link
Contributor

@kuisathaverat kuisathaverat May 28, 2020

Choose a reason for hiding this comment

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

@daily means '0 0 * * *'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Team:Automation Label for the Observability productivity team [zube]: In Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants