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

[Backport 2.x]Updated plugin code based on OpenSearch-3.0 upstream changes (#766) #844

Merged
merged 1 commit into from
May 24, 2023

Conversation

monusingh-1
Copy link
Collaborator

@monusingh-1 monusingh-1 commented May 22, 2023

Description

Original PR #766 was created due to upstream changes opensearch-project/OpenSearch#7612

The change was also backported to 2.x but the CCR change was not backported

Issues Resolved

Build failures for 2.8 and 2.x

https://build.ci.opensearch.org/blue/organizations/jenkins/distribution-build-opensearch/detail/distribution-build-opensearch/7869/pipeline/219/

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.

@monusingh-1 monusingh-1 changed the title Updated plugin code based on OpenSearch-3.0 upstream changes (#766) [Backport]Updated plugin code based on OpenSearch-3.0 upstream changes (#766) May 22, 2023
@monusingh-1 monusingh-1 marked this pull request as ready for review May 24, 2023 04:19
@monusingh-1
Copy link
Collaborator Author

re-run jobs and merge

@monusingh-1 monusingh-1 changed the title [Backport]Updated plugin code based on OpenSearch-3.0 upstream changes (#766) [Backport 2.x]Updated plugin code based on OpenSearch-3.0 upstream changes (#766) May 24, 2023
@monusingh-1
Copy link
Collaborator Author

The upstream change is merged, local build passes with above change. For some reason CI fails on github actions, will merge for now to unblock everyone and take up this issue with the build team.

@ankitkala ankitkala enabled auto-merge (squash) May 24, 2023 05:32
@ankitkala ankitkala merged commit acc51d5 into opensearch-project:2.x May 24, 2023
saikaranam-amazon added a commit to saikaranam-amazon/cross-cluster-replication-2 that referenced this pull request May 31, 2023
@monusingh-1 monusingh-1 deleted the backport_2 branch November 2, 2023 12:54
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