-
Notifications
You must be signed in to change notification settings - Fork 867
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
Modify the import for timeline visualization to includes data source name in MDS scenario #6954
Conversation
2964780
to
03ccc53
Compare
❌ Invalid Changelog HeadingThe '## Changelog' heading in your PR description is either missing or malformed. Please make sure that your PR description includes a '## Changelog' heading with proper spelling, capitalization, spacing, and Markdown syntax. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6954 +/- ##
==========================================
+ Coverage 67.44% 67.46% +0.02%
==========================================
Files 3442 3442
Lines 67804 67816 +12
Branches 11025 11027 +2
==========================================
+ Hits 45730 45754 +24
+ Misses 19408 19396 -12
Partials 2666 2666
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Behavior looks fine in the recording 👍🏽. Just make sure the edge cases like mulitple .opensearch()
functions are covered
src/core/server/saved_objects/import/check_conflict_for_data_source.test.ts
Show resolved
Hide resolved
728b135
to
3f04f37
Compare
…name in MDS scenario Signed-off-by: Yuanqi(Ella) Zhu <zhyuanqi@amazon.com>
…name in MDS scenario (#6954) (#6975) (cherry picked from commit 7eaab64) Signed-off-by: Yuanqi(Ella) Zhu <zhyuanqi@amazon.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>
Description
Modify the import for timeline visualization to includes data source name in MDS scenario
Issues Resolved
#6919
Testing the changes
Add data source name when import non-mds timeline to mds timeline
Screen.Recording.2024-06-06.at.1.25.20.PM.mov
When import from MDS to MDS, the data source name stays the same in timeline
Screen.Recording.2024-06-06.at.1.30.01.PM.mov
Import for overwriting scenerio
Screen.Recording.2024-06-06.at.1.31.16.PM.mov
Changelog
Check List
yarn test:jest
yarn test:jest_integration