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

Revert "Support Gradle 7 (#1609) (#1622)" #1657

Merged
merged 1 commit into from
Dec 6, 2021

Conversation

reta
Copy link
Collaborator

@reta reta commented Dec 6, 2021

Signed-off-by: Andriy Redko [email protected]

Description

As per discussion opensearch-project/opensearch-build#1247, the decision was taken to keep 1.x on Gradle 6.x in order to not disrupt the plugin developers.

Issues Resolved

opensearch-project/opensearch-build#1247

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • 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.

@opensearch-ci-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 17ae3616e53136046f91853eeb666598cdeb2dd2

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success 17ae3616e53136046f91853eeb666598cdeb2dd2

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 17ae3616e53136046f91853eeb666598cdeb2dd2
Log 1339

Reports 1339

@reta reta force-pushed the issue-1246.1.x.revert branch from 17ae361 to 3781516 Compare December 6, 2021 13:59
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Wrapper Validation success 3781516

@reta
Copy link
Collaborator Author

reta commented Dec 6, 2021

@dlvenable fyi, the 1.x+ would stay on Gradle 6.

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Precommit success 3781516

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 3781516
Log 1340

Reports 1340

@reta
Copy link
Collaborator Author

reta commented Dec 6, 2021

@dblock good to go, the WhiteSource seems to be known issue, thank you!

@dblock dblock merged commit a80a254 into opensearch-project:1.x Dec 6, 2021
@dblock
Copy link
Member

dblock commented Dec 6, 2021

Thank you!

peternied added a commit to peternied/security that referenced this pull request Mar 2, 2022
BWC tooling is built with gradle 6, which has breaking changes that are
not compatiable with gradle 7.  In order to support BWC tests we need to
align with the OpenSearch's gradle version for the 1.3 release.

See Also:
* Gradle 7 PR in OpenSearch opensearch-project/OpenSearch#1622
* Distribution build bugs encountered by plugins opensearch-project/opensearch-build#1247
* Revert of Gradle 7 PR in OpenSearch  opensearch-project/OpenSearch#1657

Signed-off-by: Peter Nied <[email protected]>
peternied added a commit to opensearch-project/security that referenced this pull request Mar 15, 2022
* Downgrade gradle version

BWC tooling is built with gradle 6, which has breaking changes that are
not compatiable with gradle 7.  In order to support BWC tests we need to
align with the OpenSearch's gradle version for the 1.3 release.

See Also:
* Gradle 7 PR in OpenSearch opensearch-project/OpenSearch#1622
* Distribution build bugs encountered by plugins opensearch-project/opensearch-build#1247
* Revert of Gradle 7 PR in OpenSearch  opensearch-project/OpenSearch#1657

Signed-off-by: Peter Nied <[email protected]>
wuychn pushed a commit to ochprince/security that referenced this pull request Mar 16, 2023
* Downgrade gradle version

BWC tooling is built with gradle 6, which has breaking changes that are
not compatiable with gradle 7.  In order to support BWC tests we need to
align with the OpenSearch's gradle version for the 1.3 release.

See Also:
* Gradle 7 PR in OpenSearch opensearch-project/OpenSearch#1622
* Distribution build bugs encountered by plugins opensearch-project/opensearch-build#1247
* Revert of Gradle 7 PR in OpenSearch  opensearch-project/OpenSearch#1657

Signed-off-by: Peter Nied <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants