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 Break] Disable gradlew build cache to ensure most up-to-date dependencies #3186

Merged
merged 1 commit into from
Aug 15, 2023

Conversation

peternied
Copy link
Member

Description

Disable gradlew build cache to ensure most up-to-date dependencies

We suspect that we are seeing the snapshot build of opensearch be cached in some circumstances and not cached in others that is creating a mix of build failures that are hard to interpret.

Follow up issue to revisit and potentially renable this setting #3185

Issues Resolved

Check List

  • New functionality includes testing
  • New functionality has been documented
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…ndency is used

We suspect that we are seeing the snapshot build of opensearch be cached
in some circumstances and not cached in others that is creating a mix of
build failures that are hard to interpret.

Follow up issue to revisit and potentially renable this setting opensearch-project#3185

Signed-off-by: Peter Nied <petern@amazon.com>
@peternied peternied changed the title Disable gradlew build cache to ensure most up-to-date dependencies [Build Break] Disable gradlew build cache to ensure most up-to-date dependencies Aug 15, 2023
@peternied
Copy link
Member Author

@cwperks As soon as I have confirmed that the impacted jobs are not broken I am going to push this change directly to main and then do the same with a 2.x backport

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #3186 (bc6c79d) into main (0b89858) will decrease coverage by 41.51%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main    #3186       +/-   ##
=============================================
- Coverage     62.43%   20.93%   -41.51%     
+ Complexity     3353      914     -2439     
=============================================
  Files           254      254               
  Lines         19749    19749               
  Branches       3334     3334               
=============================================
- Hits          12331     4135     -8196     
- Misses         5789    15022     +9233     
+ Partials       1629      592     -1037     

see 190 files with indirect coverage changes

@peternied peternied merged commit 5e8f12c into opensearch-project:main Aug 15, 2023
@peternied peternied deleted the disable-gradle-cache branch August 15, 2023 19:51
@peternied peternied added the backport 2.x backport to 2.x branch label Aug 15, 2023
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/security/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/security/backport-2.x
# Create a new branch
git switch --create backport/backport-3186-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 5e8f12ce5afe95f2f510cddf2a5b2cf50c076a66
# Push it to GitHub
git push --set-upstream origin backport/backport-3186-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/security/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-3186-to-2.x.

peternied added a commit to peternied/security that referenced this pull request Aug 15, 2023
…t up-to-date dependencies

Backport 5e8f12c from opensearch-project#3186.

Disable gradlew build cache to ensure most up-to-date dependencies

We suspect that we are seeing the snapshot build of opensearch be cached
in some circumstances and not cached in others that is creating a mix of
build failures that are hard to interpret.

Follow up issue to revisit and potentially renable this setting
opensearch-project#3185

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit 5e8f12c)
peternied added a commit that referenced this pull request Aug 15, 2023
…t up-to-date dependencies (#3187)

Backport 5e8f12c from #3186.

Disable gradlew build cache to ensure most up-to-date dependencies

We suspect that we are seeing the snapshot build of opensearch be cached
in some circumstances and not cached in others that is creating a mix of
build failures that are hard to interpret.

Follow up issue to revisit and potentially renable this setting
#3185

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit 5e8f12c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x backport to 2.x branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants