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

Fix: CI errors due to ml commons publish with JDK-21 and k-NN rename #835

Merged
merged 6 commits into from
Jul 19, 2024

Conversation

yuye-aws
Copy link
Member

@yuye-aws yuye-aws commented Jul 19, 2024

Description

This PR tests whether the current CI failure is due to JDK mismatch between ml-commons and neural-search: #834 (comment)

Issues Resolved

Fix CI test workflow with ml-commons errors.

Check List

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

Signed-off-by: yuye-aws <[email protected]>
@chishui
Copy link
Contributor

chishui commented Jul 19, 2024

Seems that all checks could pass, could you add a changelog entry and publish this PR, I can close mine

Signed-off-by: yuye-aws <[email protected]>
@yuye-aws yuye-aws changed the title test CI error Fix: CI errors due to ml commons publish with JDK-21 Jul 19, 2024
@yuye-aws yuye-aws marked this pull request as ready for review July 19, 2024 07:17
@yuye-aws yuye-aws changed the title Fix: CI errors due to ml commons publish with JDK-21 Fix: CI errors due to ml commons publish with JDK-21 and k-NN rename Jul 19, 2024
@zhichao-aws
Copy link
Member

Please move the change log entry to infratructure section

Signed-off-by: yuye-aws <[email protected]>
@yuye-aws
Copy link
Member Author

Please move the change log entry to infratructure section

Ack. Already changed.

@zhichao-aws zhichao-aws merged commit 2b234f6 into opensearch-project:main Jul 19, 2024
8 of 33 checks passed
@yuye-aws yuye-aws deleted the Test/JDKMismatch branch September 10, 2024 09:29
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.

4 participants