Skip to content

Conversation

@neilbhavsar
Copy link
Contributor

@neilbhavsar neilbhavsar commented Oct 21, 2025

Updating perioidic.yml file to add --continue on run-gradle.sh invocations that invoke testing to allow for gradle tasks to continue when tests fail. This will allow for more insight into failures and potentially help triage.

@neilbhavsar neilbhavsar requested a review from a team as a code owner October 21, 2025 19:41
@elasticsearchmachine elasticsearchmachine added v9.3.0 needs:triage Requires assignment of a team area label labels Oct 21, 2025
@neilbhavsar neilbhavsar marked this pull request as draft October 21, 2025 19:42
@neilbhavsar neilbhavsar marked this pull request as ready for review October 23, 2025 19:27
@neilbhavsar neilbhavsar changed the title [ES-13020] Invoking gradle continue flag on periodic runs to allow for more information on test failures [ES-12998] Invoking gradle continue flag on periodic runs to allow for more information on test failures Oct 23, 2025
@jozala jozala self-requested a review October 27, 2025 16:04
Copy link
Contributor

@jozala jozala left a comment

Choose a reason for hiding this comment

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

We just need to move this change to the .buildkite/pipelines/periodic.template.yml to avoid overriding it. See my other comment for details.

@neilbhavsar neilbhavsar requested a review from jozala October 29, 2025 16:23
Copy link
Contributor

@jozala jozala left a comment

Choose a reason for hiding this comment

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

LGTM.
This should be backported to the all active branches.

steps:
- label: "{{matrix.FWC_VERSION}} / fwc"
command: .ci/scripts/run-gradle.sh -Dbwc.checkout.align=true v$$FWC_VERSION#fwcTest -Dtests.bwc.snapshot=false
command: .ci/scripts/run-gradle.sh --continue -Dbwc.checkout.align=true v$$FWC_VERSION#fwcTest -Dtests.bwc.snapshot=false
Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch with adding this to FwC periodic builds as well.

@neilbhavsar neilbhavsar added >non-issue auto-backport Automatically create backport pull requests when merged :Delivery/Build Build or test infrastructure labels Oct 29, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Delivery Meta label for Delivery team label Oct 29, 2025
@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Oct 29, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@neilbhavsar neilbhavsar merged commit 78b4f85 into elastic:main Oct 29, 2025
34 checks passed
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

The backport operation could not be completed due to the following error:

There are no branches to backport to. Aborting.

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 136900

shmuelhanoch pushed a commit to shmuelhanoch/elasticsearch that referenced this pull request Oct 29, 2025
…r more information on test failures (elastic#136900)

* Add --continue flag to invoke maven when a task failure has been hit so that we can see the outcome of more testing over time.
neilbhavsar added a commit to neilbhavsar/elasticsearch that referenced this pull request Oct 30, 2025
…r more information on test failures (elastic#136900)

* Add --continue flag to invoke maven when a task failure has been hit so that we can see the outcome of more testing over time.

(cherry picked from commit 78b4f85)

# Conflicts:
#	.buildkite/pipelines/periodic.yml
neilbhavsar added a commit to neilbhavsar/elasticsearch that referenced this pull request Oct 30, 2025
…r more information on test failures (elastic#136900)

* Add --continue flag to invoke maven when a task failure has been hit so that we can see the outcome of more testing over time.

(cherry picked from commit 78b4f85)

# Conflicts:
#	.buildkite/pipelines/periodic.yml
@neilbhavsar
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
9.2
9.1
8.19

Questions ?

Please refer to the Backport tool documentation

elasticsearchmachine pushed a commit that referenced this pull request Oct 30, 2025
…r more information on test failures (#136900) (#137352)

* Add --continue flag to invoke maven when a task failure has been hit so that we can see the outcome of more testing over time.

(cherry picked from commit 78b4f85)

# Conflicts:
#	.buildkite/pipelines/periodic.yml
neilbhavsar added a commit that referenced this pull request Oct 30, 2025
…r more information on test failures (#136900) (#137354)

* Add --continue flag to invoke maven when a task failure has been hit so that we can see the outcome of more testing over time.

(cherry picked from commit 78b4f85)

# Conflicts:
#	.buildkite/pipelines/periodic.template.yml
#	.buildkite/pipelines/periodic.yml
neilbhavsar added a commit that referenced this pull request Oct 30, 2025
…r more information on test failures (#136900) (#137353)

* Add --continue flag to invoke maven when a task failure has been hit so that we can see the outcome of more testing over time.

(cherry picked from commit 78b4f85)

# Conflicts:
#	.buildkite/pipelines/periodic.yml
chrisparrinello pushed a commit to chrisparrinello/elasticsearch that referenced this pull request Nov 3, 2025
…r more information on test failures (elastic#136900)

* Add --continue flag to invoke maven when a task failure has been hit so that we can see the outcome of more testing over time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants