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

added changes for moving notebooks to .kibana #1937

Merged
merged 16 commits into from
Jul 22, 2024

Conversation

sumukhswamy
Copy link
Collaborator

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@sumukhswamy sumukhswamy added enhancement New feature or request backport 2.x labels Jul 11, 2024
@sumukhswamy sumukhswamy force-pushed the move-notebooks branch 3 times, most recently from 2f781d9 to e95d3a9 Compare July 17, 2024 21:14
@BionIT
Copy link

BionIT commented Jul 17, 2024

@sumukhswamy can you share testing screenshot or videos?

@sumukhswamy
Copy link
Collaborator Author

Screen.Recording.2024-07-19.at.5.30.36.PM.mov

@ps48
Copy link
Member

ps48 commented Jul 22, 2024

@sumukhswamy I see the snapshots still failing are you using the latest main branch from core dashboards?

@sumukhswamy sumukhswamy merged commit 2ab8369 into opensearch-project:main Jul 22, 2024
10 of 19 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 22, 2024
* added changes for moving notebooks to .kibana

Signed-off-by: sumukhswamy <[email protected]>

* updated tests

Signed-off-by: sumukhswamy <[email protected]>

* added changes for older notebooks viewing

Signed-off-by: sumukhswamy <[email protected]>

* mds changes for notebooks -s

Signed-off-by: sumukhswamy <[email protected]>

* resolved conflicts

Signed-off-by: sumukhswamy <[email protected]>

* linter fixes

Signed-off-by: sumukhswamy <[email protected]>

* added changes for mds, persistance and sample notebooks

Signed-off-by: sumukhswamy <[email protected]>

* changed route to /savednotebook

Signed-off-by: sumukhswamy <[email protected]>

* addressed comments

Signed-off-by: sumukhswamy <[email protected]>

* edge cases for migration

Signed-off-by: sumukhswamy <[email protected]>

* adeed unit tests

Signed-off-by: sumukhswamy <[email protected]>

* updated snapshots

Signed-off-by: sumukhswamy <[email protected]>

* updated snapshots

Signed-off-by: sumukhswamy <[email protected]>

* added changes for cypress tests

Signed-off-by: sumukhswamy <[email protected]>

* changed cypress test as rename and duplicate has been removed

Signed-off-by: sumukhswamy <[email protected]>

---------

Signed-off-by: sumukhswamy <[email protected]>
(cherry picked from commit 2ab8369)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
sumukhswamy pushed a commit that referenced this pull request Jul 22, 2024
* added changes for moving notebooks to .kibana



* updated tests



* added changes for older notebooks viewing



* mds changes for notebooks -s



* resolved conflicts



* linter fixes



* added changes for mds, persistance and sample notebooks



* changed route to /savednotebook



* addressed comments



* edge cases for migration



* adeed unit tests



* updated snapshots



* updated snapshots



* added changes for cypress tests



* changed cypress test as rename and duplicate has been removed



---------


(cherry picked from commit 2ab8369)

Signed-off-by: sumukhswamy <[email protected]>
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>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 22, 2024
* added changes for moving notebooks to .kibana

Signed-off-by: sumukhswamy <[email protected]>

* updated tests

Signed-off-by: sumukhswamy <[email protected]>

* added changes for older notebooks viewing

Signed-off-by: sumukhswamy <[email protected]>

* mds changes for notebooks -s

Signed-off-by: sumukhswamy <[email protected]>

* resolved conflicts

Signed-off-by: sumukhswamy <[email protected]>

* linter fixes

Signed-off-by: sumukhswamy <[email protected]>

* added changes for mds, persistance and sample notebooks

Signed-off-by: sumukhswamy <[email protected]>

* changed route to /savednotebook

Signed-off-by: sumukhswamy <[email protected]>

* addressed comments

Signed-off-by: sumukhswamy <[email protected]>

* edge cases for migration

Signed-off-by: sumukhswamy <[email protected]>

* adeed unit tests

Signed-off-by: sumukhswamy <[email protected]>

* updated snapshots

Signed-off-by: sumukhswamy <[email protected]>

* updated snapshots

Signed-off-by: sumukhswamy <[email protected]>

* added changes for cypress tests

Signed-off-by: sumukhswamy <[email protected]>

* changed cypress test as rename and duplicate has been removed

Signed-off-by: sumukhswamy <[email protected]>

---------

Signed-off-by: sumukhswamy <[email protected]>
(cherry picked from commit 2ab8369)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ps48 pushed a commit that referenced this pull request Jul 22, 2024
* added changes for moving notebooks to .kibana



* updated tests



* added changes for older notebooks viewing



* mds changes for notebooks -s



* resolved conflicts



* linter fixes



* added changes for mds, persistance and sample notebooks



* changed route to /savednotebook



* addressed comments



* edge cases for migration



* adeed unit tests



* updated snapshots



* updated snapshots



* added changes for cypress tests



* changed cypress test as rename and duplicate has been removed



---------


(cherry picked from commit 2ab8369)

Signed-off-by: sumukhswamy <[email protected]>
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants