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

Updated repository location on jenkins #900

Merged
merged 2 commits into from
Jul 25, 2023

Conversation

owaiskazi19
Copy link
Member

Description

Describe what this change achieves.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

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.

@saratvemulapalli
Copy link
Member

Could you explain why these changes are required?

@gaiksaya
Copy link
Member

gaiksaya commented Jul 25, 2023

Could you explain why these changes are required?

The path mentioned was incorrect. See the failed workflow here https://build.ci.opensearch.org/view/Release/job/opensearch-sdk-java-release/2/console

Invalid directory /var/jenkins/workspace/opensearch-sdk-java-release/repository/ does not exist

@joshpalis
Copy link
Member

Failing due to :

/home/runner/work/opensearch-sdk-java/opensearch-sdk-java/src/main/java/org/opensearch/sdk/sample/helloworld/rest/RestHelloAction.java:103: error: cannot find symbol
            XContentType contentType = request.getXContentType();

@owaiskazi19
Copy link
Member Author

The build failure are for the imports, which is not related to this PR.

@owaiskazi19 owaiskazi19 merged commit f460d48 into opensearch-project:main Jul 25, 2023
6 of 7 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 25, 2023
Signed-off-by: Owais Kazi <[email protected]>
(cherry picked from commit f460d48)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 25, 2023
Signed-off-by: Owais Kazi <[email protected]>
(cherry picked from commit f460d48)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
saratvemulapalli pushed a commit that referenced this pull request Jul 25, 2023
(cherry picked from commit f460d48)

Signed-off-by: Owais Kazi <[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>
saratvemulapalli pushed a commit that referenced this pull request Jul 25, 2023
(cherry picked from commit f460d48)

Signed-off-by: Owais Kazi <[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>
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.

4 participants