Skip to content

Update to latest spec while fixing workflows#1066

Open
harshavamsi wants to merge 9 commits intoopensearch-project:mainfrom
harshavamsi:pr-1065
Open

Update to latest spec while fixing workflows#1066
harshavamsi wants to merge 9 commits intoopensearch-project:mainfrom
harshavamsi:pr-1065

Conversation

@harshavamsi
Copy link
Collaborator

Description

Describe what this change achieves.

Issues Resolved

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

Check List

  • New functionality includes testing.
    • All tests pass
  • Linter check was successfull - yarn run lint doesn't show any errors
  • Commits are signed per the DCO using --signoff
  • Changelog was updated.

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.

nhtruong and others added 2 commits November 9, 2025 00:11
…-11-09)

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
- Add tests for giveAwsCredentialProviderLoader error paths
- Covers SDK not found and provider credential errors
- Improves coverage for lib/aws/shared.js error handling

Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
@harshavamsi
Copy link
Collaborator Author

@nhtruong could use your help here to review this PR. We haven't released a version of the js client in a while

@harshavamsi
Copy link
Collaborator Author

@timursaurus are you available to review these changes

@prudhvigodithi
Copy link
Member

Seeing this error in CI's

OpenSearch requires Java 21; your Java version from [/usr/lib/jvm/temurin-17-jdk-amd64] does not meet this requirement

@harshavamsi
Copy link
Collaborator Author

@prudhvigodithi the problem here is that the runner is using a cached build, let me show up a build where it passes

@harshavamsi
Copy link
Collaborator Author

Actually let me re-run the workflow

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