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

Apply get indice error handling in step index pattern #2652

Merged
merged 1 commit into from
Oct 25, 2022

Conversation

kristenTian
Copy link
Contributor

Error handling only happens before loading the indices to step index pattern but not within. Therefore pass the error handling inside the step as well to render error state.

Signed-off-by: Kristen Tian tyarong@amazon.com

Issues Resolved

#2598

Check List

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

@kristenTian kristenTian requested a review from a team as a code owner October 22, 2022 00:36
@kristenTian kristenTian added multiple datasource multiple datasource project backport 2.x v2.4.0 'Issues and PRs related to version v2.4.0' labels Oct 22, 2022
@kristenTian kristenTian force-pushed the OSD-main branch 3 times, most recently from 581020f to 64f3de4 Compare October 24, 2022 20:41
Error handling only happens before loading the indices to step index pattern but not within. Therefore pass the error handling inside the step as well to render error state.

Signed-off-by: Kristen Tian <tyarong@amazon.com>
@zhongnansu zhongnansu merged commit 76d30ec into opensearch-project:main Oct 25, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 25, 2022
Error handling only happens before loading the indices to step index pattern but not within. Therefore pass the error handling inside the step as well to render error state.

Signed-off-by: Kristen Tian <tyarong@amazon.com>
(cherry picked from commit 76d30ec)
zhongnansu pushed a commit that referenced this pull request Oct 25, 2022
Error handling only happens before loading the indices to step index pattern but not within. Therefore pass the error handling inside the step as well to render error state.

Signed-off-by: Kristen Tian <tyarong@amazon.com>
(cherry picked from commit 76d30ec)

Co-authored-by: Kristen Tian <105667444+kristenTian@users.noreply.github.com>
@AMoo-Miki AMoo-Miki added the enhancement New feature or request label Nov 5, 2022
sipopo pushed a commit to sipopo/OpenSearch-Dashboards that referenced this pull request Dec 16, 2022
…ject#2652)

Error handling only happens before loading the indices to step index pattern but not within. Therefore pass the error handling inside the step as well to render error state.

Signed-off-by: Kristen Tian <tyarong@amazon.com>
Signed-off-by: Sergey V. Osipov <sipopo@yandex.ru>
Arpit-Bandejiya pushed a commit to Arpit-Bandejiya/OpenSearch-Dashboards that referenced this pull request Jan 13, 2023
…ject#2652)

Error handling only happens before loading the indices to step index pattern but not within. Therefore pass the error handling inside the step as well to render error state.

Signed-off-by: Kristen Tian <tyarong@amazon.com>
Signed-off-by: Arpit Bandejiya <abandeji@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x enhancement New feature or request multiple datasource multiple datasource project v2.4.0 'Issues and PRs related to version v2.4.0'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants