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

[Tests] remove _type from OpenSearch Archiver #1289

Merged

Conversation

kavilla
Copy link
Member

@kavilla kavilla commented Feb 26, 2022

Description

_type was removed in opensearch-project/OpenSearch#2239

OpenSearch Archiver loads data to OpenSearch from data.json into OpenSearch for
E2E tests and integration tests but will need to verify if this causes breakage in
migration from older versions of the application.

Signed-off-by: Kawika Avilla kavilla414@gmail.com

Issues Resolved

n/a

Check List

  • New functionality includes testing.
    • All tests pass
      • yarn test:jest
      • yarn test:jest_integration
      • yarn test:ftr
  • New functionality has been documented.
  • Commits are signed per the DCO using --signoff

@kavilla kavilla requested a review from a team as a code owner February 26, 2022 00:57
@kavilla
Copy link
Member Author

kavilla commented Feb 26, 2022

Added an issue to verify but this is to unblocked CI. #1290

_type was removed in opensearch-project/OpenSearch#2239

OpenSearch Archiver loads data to OpenSearch from data.json into OpenSearch for
E2E tests and integration tests but will need to verify if this causes breakage in
migration from older versions of the application.

Issue resolved:
n/a

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
@kavilla kavilla force-pushed the avillk/remove_types_from_test branch from f598154 to d5420d1 Compare February 26, 2022 01:06
@kavilla kavilla merged commit 749c858 into opensearch-project:main Feb 26, 2022
@kavilla kavilla deleted the avillk/remove_types_from_test branch February 26, 2022 03:09
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