Skip to content

Fix serialization issue with RunTask using Configuration Cache #128596

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 1 commit into from
May 29, 2025
Merged

Fix serialization issue with RunTask using Configuration Cache #128596

merged 1 commit into from
May 29, 2025
+6 −1

Conversation

breskeby
Copy link
Contributor

@breskeby breskeby commented May 28, 2025 •
edited
Loading

This fixes an serialization issue causing the gradle build to fail with

Could not load the value of field `plugins` of task `:run` of type `org.elasticsearch.gradle.testclusters.RunTask`.
> null array

when running the RunTask if configuration cache is used.

@breskeby breskeby requested a review from a team as a code owner May 28, 2025 20:46
@breskeby breskeby added >non-issue :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team auto-backport Automatically create backport pull requests when merged v8.19.0 v9.1.0 v8.16.7 v9.0.3 v8.17.8 v8.18.3 labels May 28, 2025
@breskeby breskeby self-assigned this May 28, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@breskeby breskeby requested a review from lukewhiting May 28, 2025 20:48
@breskeby breskeby merged commit df43f83 into elastic:main May 29, 2025
19 checks passed
breskeby added a commit to breskeby/elasticsearch that referenced this pull request May 29, 2025
@breskeby
Fix serialization issue with RunTask using Configuration Cache (elast…
7e82e88 
…ic#128596)
@breskeby breskeby mentioned this pull request May 29, 2025
Merged
breskeby added a commit to breskeby/elasticsearch that referenced this pull request May 29, 2025
@breskeby
Fix serialization issue with RunTask using Configuration Cache (elast…
5824bdc 
…ic#128596)
@breskeby breskeby mentioned this pull request May 29, 2025
Merged
breskeby added a commit to breskeby/elasticsearch that referenced this pull request May 29, 2025
@breskeby
Fix serialization issue with RunTask using Configuration Cache (elast…
1b8363f 
…ic#128596)
@breskeby breskeby mentioned this pull request May 29, 2025
Open
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
✅ 8.19
❌ 8.16 Commit could not be cherrypicked due to conflicts
✅ 9.0
❌ 8.17 Commit could not be cherrypicked due to conflicts
✅ 8.18

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

elasticsearchmachine pushed a commit that referenced this pull request May 29, 2025
@breskeby
Fix serialization issue with RunTask using Configuration Cache (#128596…
4176113 
…) (#128605)
elasticsearchmachine pushed a commit that referenced this pull request May 29, 2025
@breskeby
Fix serialization issue with RunTask using Configuration Cache (#128596…
5b5982e 
…) (#128606)
Samiul-TheSoccerFan pushed a commit to Samiul-TheSoccerFan/elasticsearch that referenced this pull request Jun 5, 2025
@breskeby @Samiul-TheSoccerFan
Fix serialization issue with RunTask using Configuration Cache (elast…
b229add 
…ic#128596)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brianseeders brianseeders brianseeders approved these changes

@lukewhiting lukewhiting lukewhiting approved these changes

Assignees

@breskeby breskeby

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 v8.16.7 v8.17.8 v8.18.3 v8.19.0 v9.0.3 v9.1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@breskeby @elasticsearchmachine @brianseeders @lukewhiting