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

[Build] Fix build finished hooks on ci when using configuration cache #116888

Merged
merged 3 commits into from
Nov 18, 2024

Conversation

breskeby
Copy link
Contributor

@breskeby breskeby commented Nov 16, 2024

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

  • referencing static methods from build script
  • referencing gradle object from closure

@breskeby breskeby requested a review from a team as a code owner November 16, 2024 10:52
@breskeby breskeby added >non-issue :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team v8.17.0 labels Nov 16, 2024
@breskeby breskeby self-assigned this Nov 16, 2024
@elasticsearchmachine
Copy link
Collaborator

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

@breskeby
Copy link
Contributor Author

This was hidden from signaling a failure as it happened in the build finished hook after the build is already marked as successful. But it showed in the logs here https://buildkite.com/elastic/elasticsearch-gradle-cache-validation/builds/68#01932b58-cfe0-4d08-ab8c-1153cbe88f24

@breskeby breskeby enabled auto-merge (squash) November 16, 2024 10:56
@breskeby breskeby changed the title [Gradle] Fix build finished hooks on ci when using configuration cache [Build] Fix build finished hooks on ci when using configuration cache Nov 18, 2024
@breskeby breskeby merged commit 929d398 into elastic:main Nov 18, 2024
18 checks passed
salvatore-campagna pushed a commit to salvatore-campagna/elasticsearch that referenced this pull request Nov 18, 2024
elastic#116888)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure
alexey-ivanov-es pushed a commit to alexey-ivanov-es/elasticsearch that referenced this pull request Nov 28, 2024
elastic#116888)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Feb 17, 2025
elastic#116888)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure

(cherry picked from commit 929d398)
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Feb 17, 2025
elastic#116888)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure

(cherry picked from commit 929d398)
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Feb 17, 2025
elastic#116888)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure

(cherry picked from commit 929d398)

# Conflicts:
#	build-tools-internal/src/main/groovy/elasticsearch.build-scan.gradle
@breskeby
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.17
8.16
7.17

Questions ?

Please refer to the Backport tool documentation

breskeby added a commit that referenced this pull request Feb 18, 2025
…on cache (#116888) (#122744)

* [Gradle] Fix build finished hooks on ci when using configuration cache (#116888)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure

(cherry picked from commit 929d398)

# Conflicts:
#	build-tools-internal/src/main/groovy/elasticsearch.build-scan.gradle

* Update elasticsearch.build-scan.gradle

fix license

* Fix license header
breskeby added a commit that referenced this pull request Feb 26, 2025
#116888) (#122740)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure

(cherry picked from commit 929d398)
breskeby added a commit that referenced this pull request Feb 26, 2025
#116888) (#122739)

Fixes two incompatibilities with Gradle configuration cache in our build scan build finished hook:

referencing static methods from build script
referencing gradle object from closure

(cherry picked from commit 929d398)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants