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

[Build] restore kibana index #374

Merged
merged 2 commits into from
May 26, 2021

Conversation

kavilla
Copy link
Member

@kavilla kavilla commented May 25, 2021

Description

Restoring the index from .opensearch_dashboards to .kibana and then
updated the tests (unit, integ, and func test all passing). This is allowable
because this is for functional purposes and for clusters/plugins that will
migrate to Dashboards. Their index will not require re-indexing for 1.0.0
and shouldn't require a full migration after further updates.

Signed-off-by: Kawika Avilla [email protected]

Issues Resolved

#325
Partial: #334

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

Restoring the index from .opensearch_dashboards to .kibana and then
updated the tests. This is allowable because this is for functional
purposes and for clusters/plugins that will migrate to Dashboards.
Their index will not require re-indexing for 1.0.0 and won't require
migration after further updates.

Signed-off-by: Kawika Avilla <[email protected]>
@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed 234a703

dblock
dblock previously approved these changes May 25, 2021
Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

LGTM

ananzh
ananzh previously approved these changes May 25, 2021
This doesn't have impact on the results of tests but to keep
consistency in the index name these updates were missed.

Signed-off-by: Kawika Avilla <[email protected]>
@kavilla kavilla dismissed stale reviews from ananzh and dblock via b249763 May 25, 2021 22:35
@kavilla kavilla requested review from dblock and ananzh May 25, 2021 22:35
@opensearch-ci-bot
Copy link
Collaborator

✅   DCO Check Passed b249763

@kavilla kavilla requested a review from tmarkley May 25, 2021 22:59
Copy link
Member

@ananzh ananzh left a comment

Choose a reason for hiding this comment

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

LGTM

@kavilla kavilla merged commit 963b640 into opensearch-project:main May 26, 2021
@kavilla kavilla deleted the avillk/restore_kibana_index branch May 26, 2021 20:24
kavilla added a commit that referenced this pull request May 26, 2021
* [Build] restore kibana index

Restoring the index from .opensearch_dashboards to .kibana and then
updated the tests. This is allowable because this is for functional
purposes and for clusters/plugins that will migrate to Dashboards.
Their index will not require re-indexing for 1.0.0 and won't require
migration after further updates.
Signed-off-by: Kawika Avilla <[email protected]>

* [Tests] updates tests for consistency

This doesn't have impact on the results of tests but to keep
consistency in the index name these updates were missed.

Signed-off-by: Kawika Avilla <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards-compatibility build Build related additions or modifications
Projects
None yet
5 participants