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

Switch to more reliable OpenSearch Lucene snapshot location #978

Merged
merged 3 commits into from
Jan 12, 2024

Conversation

peternied
Copy link
Member

Description

Switch to more reliable OpenSearch Lucene snapshot location

Issues Resolved

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.

@peternied
Copy link
Member Author

@dbwiddis Could you take a look?

@peternied
Copy link
Member Author

@owaiskazi19 @saratvemulapalli Looks like there is an unrelated build break can you let me know how this should move forward?

/home/runner/work/opensearch-sdk-java/opensearch-sdk-java/src/test/java/org/opensearch/sdk/TestExtensionInterfaces.java:175: error: constructor Parameters in class Parameters cannot be applied to given types;
        Processor.Parameters parameters = new Processor.Parameters(null, null, null, null, null, null, null, null, null);
                                          ^

@owaiskazi19
Copy link
Member

owaiskazi19 commented Jan 11, 2024

@owaiskazi19 @saratvemulapalli Looks like there is an unrelated build break can you let me know how this should move forward?

Should be fixed by #979. Merged. Needs a rebase.

@peternied
Copy link
Member Author

@owaiskazi19 Looks like there are still issues with the PR checks, I've created a issue:

@saratvemulapalli
Copy link
Member

@peternied here is the fix #982
Please rebase after the changes are merged.

@saratvemulapalli saratvemulapalli merged commit 95f0eaf into opensearch-project:main Jan 12, 2024
7 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 12, 2024
- Related opensearch-project/opensearch-build#3874
- Related opensearch-project/OpenSearch#11728

Signed-off-by: Peter Nied <[email protected]>
Signed-off-by: Peter Nied <[email protected]>
Co-authored-by: Owais Kazi <[email protected]>
(cherry picked from commit 95f0eaf)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
saratvemulapalli added a commit that referenced this pull request Jan 12, 2024
)

- Related opensearch-project/opensearch-build#3874
- Related opensearch-project/OpenSearch#11728




(cherry picked from commit 95f0eaf)

Signed-off-by: Peter Nied <[email protected]>
Signed-off-by: Peter Nied <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Owais Kazi <[email protected]>
Co-authored-by: Sarat Vemulapalli <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants