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

[8.x][ci] Run linting before tests (#197310) #198096

Merged
merged 1 commit into from
Oct 29, 2024
Merged

[8.x][ci] Run linting before tests (#197310) #198096

merged 1 commit into from
Oct 29, 2024

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Oct 28, 2024

@jbudz
[ci] Run linting before tests (elastic#197310)
262c192 
Linting is a frequent source of build failures. By increasing the cpu
count we can run this check before starting our highly-parallel tests
without impacting total build time.
@jbudz jbudz marked this pull request as ready for review October 29, 2024 18:48
@jbudz jbudz requested a review from a team as a code owner October 29, 2024 18:48
@jbudz
Copy link
Member Author

jbudz commented Oct 29, 2024

buildkite test this

@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 29, 2024 •
edited
Loading

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #59 / task_manager with update by query task claimer task manager metrics task claim should increment task claim success/total counters

Metrics [docs]

✅ unchanged

History

@jbudz jbudz merged commit d3ccb92 into elastic:8.x Oct 29, 2024
48 of 51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ikuni17 Ikuni17 Ikuni17 approved these changes

Assignees
No one assigned
Labels
backport ci:all-cypress-suites
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jbudz @elasticmachine @Ikuni17