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

Use separate targets for security enabled tests #113

Conversation

ansjcy
Copy link
Member

@ansjcy ansjcy commented Sep 10, 2024

Description

Use separate targets for security enabled tests.

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.

@deshsidd
Copy link
Collaborator

Thanks, is this the reason why the ITs are failing?

@ansjcy
Copy link
Member Author

ansjcy commented Sep 10, 2024

@deshsidd no, we need to look into the failures later, but it should be safe to disable them since we already have a dedicated target to run integration tests with security.

@ansjcy ansjcy merged commit 839e6f7 into opensearch-project:main Sep 10, 2024
17 of 18 checks passed
@ansjcy ansjcy deleted the separate-targets-for-security-enabled-tests branch September 10, 2024 00:48
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 10, 2024
Signed-off-by: Chenyang Ji <[email protected]>
(cherry picked from commit 839e6f7)
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 Sep 10, 2024
Signed-off-by: Chenyang Ji <[email protected]>
(cherry picked from commit 839e6f7)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ansjcy pushed a commit that referenced this pull request Sep 10, 2024
(cherry picked from commit 839e6f7)

Signed-off-by: Chenyang Ji <[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>
ansjcy pushed a commit that referenced this pull request Sep 10, 2024
(cherry picked from commit 839e6f7)

Signed-off-by: Chenyang Ji <[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.

2 participants