[Dynamic Config] Fix bug when attempting dynamic config index creation#8184
Merged
huyaboo merged 2 commits intoopensearch-project:mainfrom Sep 13, 2024
Merged
Conversation
Contributor
❌ Empty Changelog SectionThe Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section. |
opensearch-changeset-bot bot
added a commit
to huyaboo/OpenSearch-Dashboards
that referenced
this pull request
Sep 13, 2024
Signed-off-by: Huy Nguyen <73027756+huyaboo@users.noreply.github.com>
6a94643 to
f8964ad
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8184 +/- ##
==========================================
- Coverage 64.04% 64.04% -0.01%
==========================================
Files 3740 3740
Lines 88582 88608 +26
Branches 13791 13799 +8
==========================================
+ Hits 56731 56745 +14
- Misses 31253 31265 +12
Partials 598 598
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
zhyuanqi
approved these changes
Sep 13, 2024
zhongnansu
approved these changes
Sep 13, 2024
opensearch-trigger-bot bot
pushed a commit
that referenced
this pull request
Sep 13, 2024
#8184) * Fix bug when attempting dynamic config index creation Signed-off-by: Huy Nguyen <73027756+huyaboo@users.noreply.github.com> * Changeset file for PR #8184 created/updated --------- Signed-off-by: Huy Nguyen <73027756+huyaboo@users.noreply.github.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit 678c644) 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 13, 2024
#8184) * Fix bug when attempting dynamic config index creation Signed-off-by: Huy Nguyen <73027756+huyaboo@users.noreply.github.com> * Changeset file for PR #8184 created/updated --------- Signed-off-by: Huy Nguyen <73027756+huyaboo@users.noreply.github.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit 678c644) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
BionIT
pushed a commit
that referenced
this pull request
Sep 13, 2024
#8184) (#8192) * Fix bug when attempting dynamic config index creation * Changeset file for PR #8184 created/updated --------- (cherry picked from commit 678c644) Signed-off-by: Huy Nguyen <73027756+huyaboo@users.noreply.github.com> 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> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
BionIT
pushed a commit
that referenced
this pull request
Sep 13, 2024
#8184) (#8193) * Fix bug when attempting dynamic config index creation * Changeset file for PR #8184 created/updated --------- (cherry picked from commit 678c644) Signed-off-by: Huy Nguyen <73027756+huyaboo@users.noreply.github.com> 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> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR fixes a bug that can occur if the alias
.opensearch_dashboards_dynamic_configexists but not the indexopensearch_dashboards_dynamic_config_N. In particular the following test cases are as follows:.opensearch_dashboards_dynamic_config_N.opensearch_dashboards_dynamic_config_1.opensearch_dashboards_dynamic_config_*index.opensearch_dashboards_dynamic_config_*indexIssues Resolved
Testing the changes
Attached are the following cases when calling dev tools or by looking in application logs:
.opensearch_dashboards_dynamic_config_1index.opensearch_dashboards_dynamic_config_Npresent (used below curl command)curl -X PUT "http://localhost:9200/.opensearch_dashboards_dynamic_config_4".opensearch_dashboards_dynamic_config_1present (used below curl command)curl -X PUT "http://localhost:9200/.opensearch_dashboards_dynamic_config_1".random_index_3pointed to alias (used below curl command).opensearch_dashboards_dynamic_config_1pointed to aliasChangelog
Check List
yarn test:jestyarn test:jest_integration