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 build-tools integration tests failures. #465

Merged
merged 1 commit into from
Mar 26, 2021
Merged

Fix build-tools integration tests failures. #465

merged 1 commit into from
Mar 26, 2021

Conversation

adnapibar
Copy link
Contributor

Description

[Describe what this change achieves]

A few build-tools integTests are failing as a result of the renaming to OpenSearch. This was due to some un-renamed files in the fake archives used for testing. This commit renames those.

Issues Resolved

[List any issues this PR will resolve]

It fixes the :build-tools:integTest failures mentioned in issue #441

Check List

- [ ] New functionality includes testing.
- [ ] All tests pass
- [ ] New functionality has been documented.
- [ ] New functionality has javadoc added
- [ ] Commits are signed per the DCO using --signoff

Signed-off-by: Rabi Panda [email protected]

@adnapibar adnapibar added >FORK Related to the fork process >test-failure Test failure from CI, local build, etc. Rename Renaming to OpenSearch labels Mar 26, 2021
@adnapibar adnapibar mentioned this pull request Mar 26, 2021
77 tasks
@odfe-release-bot
Copy link

✅   DCO Check Passed 913581b91f57c8e80ae635b6a277db88e3105c6c

@odfe-release-bot
Copy link

✅   Gradle Wrapper Validation success 913581b91f57c8e80ae635b6a277db88e3105c6c

Copy link
Collaborator

@tlfeng tlfeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I admire you for finding the deeply hidden words to be renamed! 👍

@odfe-release-bot
Copy link

✅   Gradle Precommit success 913581b91f57c8e80ae635b6a277db88e3105c6c

A few build-tools integTests were failing as a result of the renaming to OpenSearch. This was due to some un-renamed files in the fake archives used for testing. This commit renames those.

Signed-off-by: Rabi Panda <[email protected]>
@odfe-release-bot
Copy link

✅   DCO Check Passed 79c77b6

@odfe-release-bot
Copy link

✅   Gradle Wrapper Validation success 79c77b6

@odfe-release-bot
Copy link

✅   Gradle Precommit success 79c77b6

@adnapibar adnapibar merged commit f7a7b80 into opensearch-project:main Mar 26, 2021
@adnapibar adnapibar deleted the fix-build-integtest branch March 26, 2021 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>FORK Related to the fork process Rename Renaming to OpenSearch >test-failure Test failure from CI, local build, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants