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

Update traces integration tests #391

Merged
merged 4 commits into from
Apr 19, 2023

Conversation

rupal-bq
Copy link
Contributor

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.

Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
@rupal-bq
Copy link
Contributor Author

rupal-bq commented Apr 19, 2023

maharup@b0f1d879dff7 dashboards-observability % yarn cypress run --spec ".cypress/integration/4_trace_analytics_dashboard.spec.js"
yarn run v1.22.19
$ /Users/maharup/GitRepoWorkspace/OpenSearch-Dashboards/plugins/dashboards-observability/node_modules/.bin/cypress run --spec .cypress/integration/4_trace_analytics_dashboard.spec.js
The `experimentalNetworkStubbing` configuration option was removed in Cypress version `6.0.0`.
It is no longer necessary for using `cy.intercept()` (formerly `cy.route2()`).

You can safely remove this option from your config.
Nothing to watch. Set the "cypress-watch-and-reload" object in the Cypress configuration
see https://github.com/bahmutov/cypress-watch-and-reload#use

====================================================================================================

  (Run Starting)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:    6.9.1                                                                              │
  │ Browser:    Electron 87 (headless)                                                             │
  │ Specs:      1 found (4_trace_analytics_dashboard.spec.js)                                      │
  │ Searched:   .cypress/integration/4_trace_analytics_dashboard.spec.js                           │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


────────────────────────────────────────────────────────────────────────────────────────────────────
                                                                                                    
  Running:  4_trace_analytics_dashboard.spec.js                                             (1 of 1)
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db

Why you should do it regularly:
https://github.com/browserslist/browserslist#browsers-data-updating


  Dump test data
    ✓ Indexes test data (2025ms)

  Testing dashboard table empty state
    ✓ Renders empty state (7465ms)

  Testing dashboard table
    ✓ Renders the dashboard table (11925ms)
    ✓ Has working breadcrumbs (8550ms)
    ✓ Adds the percentile filters (14251ms)
    ✓ Opens latency trend popover (17705ms)
    ✓ Redirects to traces table with filter (11558ms)

  Testing plots
    ✓ Renders service map (7737ms)
    ✓ Renders plots (6814ms)

  Latency by trace group table
    ✓ Verify columns in Latency by trace group table along with pagination functionality (6457ms)
    ✓ Sorts the Latency by trace group table (8156ms)
    ✓ Verify tooltips in Latency by trace group table (10222ms)
    ✓ Verify Search engine on Trace dashboard (16765ms)

  Testing filters on trace analytics page
    ✓ Verify Change all filters (6904ms)
    ✓ Verify Add filter section (8846ms)

  Dump jaeger test data
    ✓ Indexes test data (1567ms)

  Testing switch mode to jaeger
    ✓ Verifies errors mode columns and data (8667ms)
    ✓ Verifies traces links to traces page (10249ms)
    ✓ Switches to throughput mode and verifies columns and data (7803ms)


  19 passing (3m)


  (Results)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Tests:        19                                                                               │
  │ Passing:      19                                                                               │
  │ Failing:      0                                                                                │
  │ Pending:      0                                                                                │
  │ Skipped:      0                                                                                │
  │ Screenshots:  0                                                                                │
  │ Video:        true                                                                             │
  │ Duration:     2 minutes, 54 seconds                                                            │
  │ Spec Ran:     4_trace_analytics_dashboard.spec.js                                              │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


  (Video)

  -  Started processing:  Compressing to 32 CRF                                                     
  -  Finished processing: /Users/maharup/GitRepoWorkspace/OpenSearch-Dashboards/plugi    (9 seconds)
                          ns/dashboards-observability/.cypress/videos/4_trace_analyti               
                          cs_dashboard.spec.js.mp4                                                  


====================================================================================================

  (Run Finished)


       Spec                                              Tests  Passing  Failing  Pending  Skipped  
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  4_trace_analytics_dashboard.spec.js      02:54       19       19        -        -        - │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✔  All specs passed!                        02:54       19       19        -        -        -  

✨  Done in 194.88s.

@rupal-bq
Copy link
Contributor Author

maharup@b0f1d879dff7 dashboards-observability % yarn cypress run --spec ".cypress/integration/5_trace_analytics_services.spec.js"
yarn run v1.22.19
$ /Users/maharup/GitRepoWorkspace/OpenSearch-Dashboards/plugins/dashboards-observability/node_modules/.bin/cypress run --spec .cypress/integration/5_trace_analytics_services.spec.js
The `experimentalNetworkStubbing` configuration option was removed in Cypress version `6.0.0`.
It is no longer necessary for using `cy.intercept()` (formerly `cy.route2()`).

You can safely remove this option from your config.
Nothing to watch. Set the "cypress-watch-and-reload" object in the Cypress configuration
see https://github.com/bahmutov/cypress-watch-and-reload#use

====================================================================================================

  (Run Starting)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:    6.9.1                                                                              │
  │ Browser:    Electron 87 (headless)                                                             │
  │ Specs:      1 found (5_trace_analytics_services.spec.js)                                       │
  │ Searched:   .cypress/integration/5_trace_analytics_services.spec.js                            │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


────────────────────────────────────────────────────────────────────────────────────────────────────
                                                                                                    
  Running:  5_trace_analytics_services.spec.js                                              (1 of 1)
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db

Why you should do it regularly:
https://github.com/browserslist/browserslist#browsers-data-updating


  Testing services table empty state
    ✓ Renders empty state (8014ms)

  Testing services table
    ✓ Renders the services table (10240ms)
    ✓ Searches correctly (7910ms)
    ✓ Verify columns in Services table (6548ms)
    ✓ Navigate from Services to Traces (5778ms)

  Testing service view empty state
    ✓ Renders service view empty state (1903ms)

  Testing service view
    ✓ Renders service view (9127ms)
    ✓ Has working breadcrumbs (16285ms)
    ✓ Renders spans data grid, flyout, filters (12030ms)

  Testing Service map
    ✓ Render Service map (6160ms)

  Testing traces Spans table verify table headers functionality
    ✓ Renders the spans table and verify columns headers (6669ms)
    ✓ Toggle columns and verify the columns hidden text verify rows (7021ms)
    ✓ Show all button Spans table (7086ms)
    ✓ Hide all button Spans table (6654ms)
    ✓ Render Spans table and change data table Density (7225ms)
    ✓ Render Spans table and and click on sort (9907ms)

  Testing traces Spans table and verify columns functionality
    ✓ Renders the spans table and click on first span to verify details (11167ms)
    ✓ Render Spans table and verify Column functionality (7928ms)

  Testing switch mode to jaeger
    ✓ Verifies columns and data (7362ms)
    ✓ Verifies traces links to traces page with filter applied (8904ms)


  20 passing (3m)


  (Results)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Tests:        20                                                                               │
  │ Passing:      20                                                                               │
  │ Failing:      0                                                                                │
  │ Pending:      0                                                                                │
  │ Skipped:      0                                                                                │
  │ Screenshots:  0                                                                                │
  │ Video:        true                                                                             │
  │ Duration:     2 minutes, 44 seconds                                                            │
  │ Spec Ran:     5_trace_analytics_services.spec.js                                               │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


  (Video)

  -  Started processing:  Compressing to 32 CRF                                                     
  -  Finished processing: /Users/maharup/GitRepoWorkspace/OpenSearch-Dashboards/plugi    (9 seconds)
                          ns/dashboards-observability/.cypress/videos/5_trace_analyti               
                          cs_services.spec.js.mp4                                                   


====================================================================================================

  (Run Finished)


       Spec                                              Tests  Passing  Failing  Pending  Skipped  
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  5_trace_analytics_services.spec.js       02:44       20       20        -        -        - │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✔  All specs passed!                        02:44       20       20        -        -        -  

✨  Done in 184.73s.

@rupal-bq
Copy link
Contributor Author

maharup@b0f1d879dff7 dashboards-observability % yarn cypress run --spec ".cypress/integration/6_trace_analytics_traces.spec.js"  
yarn run v1.22.19
$ /Users/maharup/GitRepoWorkspace/OpenSearch-Dashboards/plugins/dashboards-observability/node_modules/.bin/cypress run --spec .cypress/integration/6_trace_analytics_traces.spec.js
The `experimentalNetworkStubbing` configuration option was removed in Cypress version `6.0.0`.
It is no longer necessary for using `cy.intercept()` (formerly `cy.route2()`).

You can safely remove this option from your config.
Nothing to watch. Set the "cypress-watch-and-reload" object in the Cypress configuration
see https://github.com/bahmutov/cypress-watch-and-reload#use

====================================================================================================

  (Run Starting)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:    6.9.1                                                                              │
  │ Browser:    Electron 87 (headless)                                                             │
  │ Specs:      1 found (6_trace_analytics_traces.spec.js)                                         │
  │ Searched:   .cypress/integration/6_trace_analytics_traces.spec.js                              │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


────────────────────────────────────────────────────────────────────────────────────────────────────
                                                                                                    
  Running:  6_trace_analytics_traces.spec.js                                                (1 of 1)
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db

Why you should do it regularly:
https://github.com/browserslist/browserslist#browsers-data-updating


  Testing traces table empty state
    ✓ Renders empty state (7666ms)

  Testing traces table
    ✓ Renders the traces table (7871ms)
    ✓ Sorts the traces table (5791ms)
    ✓ Searches correctly (7246ms)

  Testing trace view
    ✓ Renders the trace view (10289ms)
    ✓ Has working breadcrumbs (17528ms)
    ✓ Renders data grid, flyout and filters (14311ms)

  Testing traces table
    ✓ Renders the traces table and verify Table Column, Pagination and Rows Data  (9768ms)

  Testing switch mode to jaeger
    ✓ Verifies columns and data (6638ms)
    ✓ Verifies Trace View (5995ms)


  10 passing (2m)


  (Results)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Tests:        10                                                                               │
  │ Passing:      10                                                                               │
  │ Failing:      0                                                                                │
  │ Pending:      0                                                                                │
  │ Skipped:      0                                                                                │
  │ Screenshots:  0                                                                                │
  │ Video:        true                                                                             │
  │ Duration:     1 minute, 33 seconds                                                             │
  │ Spec Ran:     6_trace_analytics_traces.spec.js                                                 │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


  (Video)

  -  Started processing:  Compressing to 32 CRF                                                     
  -  Finished processing: /Users/maharup/GitRepoWorkspace/OpenSearch-Dashboards/plugi    (5 seconds)
                          ns/dashboards-observability/.cypress/videos/6_trace_analyti               
                          cs_traces.spec.js.mp4                                                     


====================================================================================================

  (Run Finished)


       Spec                                              Tests  Passing  Failing  Pending  Skipped  
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  6_trace_analytics_traces.spec.js         01:33       10       10        -        -        - │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✔  All specs passed!                        01:33       10       10        -        -        -  

✨  Done in 109.70s.

Signed-off-by: Rupal Mahajan <[email protected]>
Copy link
Collaborator

@derek-ho derek-ho left a comment

Choose a reason for hiding this comment

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

ty

@rupal-bq rupal-bq merged commit b8f9ca3 into opensearch-project:main Apr 19, 2023
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 19, 2023
* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 19, 2023
* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)
rupal-bq added a commit that referenced this pull request Apr 19, 2023
* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)

Co-authored-by: Rupal Mahajan <[email protected]>
rupal-bq added a commit that referenced this pull request Apr 19, 2023
* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)

Co-authored-by: Rupal Mahajan <[email protected]>
ps48 added a commit that referenced this pull request May 24, 2023
* Fix for hover issue on docs of discover/explorer data grid (#329) (#358)

Signed-off-by: Eric Wei <[email protected]>
(cherry picked from commit 19f29e1)

Co-authored-by: Eric Wei <[email protected]>

* [2.7] Refactor Saved objects and add visualization embeddable (#341) (#353)

Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>

* Cohesion main to 2.7 (#382)

Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL (#383) (#384)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 23e09e5)

Co-authored-by: Joshua Li <[email protected]>

* [2.7] Redo pick from main to 2.7 (#385)

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Metrics analytics support for SOpanels  (#386) (#388)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 1ecd7ea)

Co-authored-by: Shenoy Pratik <[email protected]>

* change plugin ordering (#389) (#390)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a6c1ebc)

Co-authored-by: Shenoy Pratik <[email protected]>

* notebooks fix for cypress and minor bugs (#392) (#393)

* notebooks fix for cypress and minor bugs

Signed-off-by: Shenoy Pratik <[email protected]>

* remove comments

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 3219f14)

* Update traces integration tests (#391) (#397)

* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)

Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL from dashboards (#399) (#403)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 05ef180)

Co-authored-by: Joshua Li <[email protected]>

* Adjust metrics top menu layout to avoid overflow (#398) (#402)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 6309e63)

Co-authored-by: Joshua Li <[email protected]>

* Support duplicate visualization in dashboard (#400) (#405)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 3d74bac)

Co-authored-by: Joshua Li <[email protected]>

* fix panel visualization preview, new viz workflow (#401) (#404)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 78347d9)

Co-authored-by: Shenoy Pratik <[email protected]>

* Fix saving multiple metrics to SOpanels (#407) (#408)

* fix saving multiple metrics to SOpanels

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused variable

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a4cb6d1)

Co-authored-by: Shenoy Pratik <[email protected]>

* Panel table fixes (#406)

* Fix Panel View (legacy)
  - Duplicate
  - Rename
  - Delete

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Release Notes 2.7.0.0 (#394) (#412)

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
(cherry picked from commit 969e639)

Co-authored-by: Peter Fitzgibbons <[email protected]>

* [Dashboard listing] update edit url and view url (#416) (#421)

* [Dashboard listing] update edit url and view url

Update to include BASE_URL if basePath is defined. Since create url is just
a direct nav and needs the basePath when passing props to the provider, the source
code no longer appends the basePath to keep these URLs consistent.

A feature should consolidate the basePaths in the dashboard listing service so that
createUrl also adds the base path. At which point we no longer need the base path.

Also removed some rogue URLs.

Core PR:
opensearch-project/OpenSearch-Dashboards#3922

---------

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
(cherry picked from commit f94c21e)

Co-authored-by: Kawika Avilla <[email protected]>

* Fix invalid time range in new panels (#419)

Signed-off-by: Joshua Li <[email protected]>

* open new tab for editing panel visualization (#430)

Signed-off-by: Eric Wei <[email protected]>

* Save new prometheus metrics to OSD saved objects (#428)

* Save new prometheus metrics to OSD saved objects

Signed-off-by: Joshua Li <[email protected]>

* Return if failed at saving metrics

Signed-off-by: Joshua Li <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>

* fixes edit mode callback bug in Saved Object Panels (#420)

* fix for SOpanels editMode

Signed-off-by: Shenoy Pratik <[email protected]>

* rename edit button data-test-subj

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>

* Add Toasts to Observability Dashboards  (#435)

* Fixes
* Panel View (legacy)
  - Duplicate
  - Rename

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Toasts use hook from useOpenSearchDashboards context provider

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Testing for CustomPanel Toast

Signed-off-by: Peter Fitzgibbons <[email protected]>

* update catches from comments, minor code cleaning

Signed-off-by: Shenoy Pratik <[email protected]>

* update tests

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused redux slice

Signed-off-by: Shenoy Pratik <[email protected]>

* revert cypress changes

Signed-off-by: Shenoy Pratik <[email protected]>

* add toasts to SOflyout

Signed-off-by: Shenoy Pratik <[email protected]>

* fix messaging for multiple delete

Signed-off-by: Derek Ho <[email protected]>

* fix up toast and error handling for create and delete flows

Signed-off-by: Derek Ho <[email protected]>

* fix up clone

Signed-off-by: Derek Ho <[email protected]>

* fix rename in table

Signed-off-by: Derek Ho <[email protected]>

* fix rename in custom panel so view

Signed-off-by: Derek Ho <[email protected]>

* fix up panel toasts

Signed-off-by: Derek Ho <[email protected]>

* fix up for flyout

Signed-off-by: Derek Ho <[email protected]>

* code cleanup

Signed-off-by: Derek Ho <[email protected]>

* finish merge

Signed-off-by: Derek Ho <[email protected]>

* fix up PR comments

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>

* change samples to saved object (#427) (#436)

* change samples to saved object



* add sample to so



* fix saved object sample



* fix up



* add back toast



* revert file



* refactoring



* use constant



---------


(cherry picked from commit ce06ccb)

Signed-off-by: Derek Ho <[email protected]>

* Add Error handling on duplicate dashboard names (#441)

* Add check for existing panel name

Signed-off-by: Rupal Mahajan <[email protected]>

* Nit

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove double await

Signed-off-by: Rupal Mahajan <[email protected]>

* Add rename name check test

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove test

Signed-off-by: Rupal Mahajan <[email protected]>

* Add unique id to sample panel name

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>

* fix timestamp overriding issue (#449)

Signed-off-by: Eric Wei <[email protected]>

* Renaming custom panels to Observability Dashboards (#450)

* change custom panels -> observability dashboards

Signed-off-by: Eric Wei <[email protected]>

* rest of the panel renaming

Signed-off-by: Eric Wei <[email protected]>

---------

Signed-off-by: Eric Wei <[email protected]>

* Fix delete failure when no osd item selected (#452)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit a4a310e)

* Open log patterns by default (#444) (#459)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 72970aa)

* diff merging between 2.x and 2.7

Signed-off-by: Shenoy Pratik <[email protected]>

* add service fields length check in app analytics

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
…project#396)

* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)

Co-authored-by: Rupal Mahajan <[email protected]>
(cherry picked from commit 5b7612b)
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
* Fix for hover issue on docs of discover/explorer data grid (opensearch-project#329) (opensearch-project#358)

Signed-off-by: Eric Wei <[email protected]>
(cherry picked from commit 19f29e1)

Co-authored-by: Eric Wei <[email protected]>

* [2.7] Refactor Saved objects and add visualization embeddable (opensearch-project#341) (opensearch-project#353)

Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>

* Cohesion main to 2.7 (opensearch-project#382)

Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL (opensearch-project#383) (opensearch-project#384)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 23e09e5)

Co-authored-by: Joshua Li <[email protected]>

* [2.7] Redo pick from main to 2.7 (opensearch-project#385)

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Metrics analytics support for SOpanels  (opensearch-project#386) (opensearch-project#388)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 1ecd7ea)

Co-authored-by: Shenoy Pratik <[email protected]>

* change plugin ordering (opensearch-project#389) (opensearch-project#390)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a6c1ebc)

Co-authored-by: Shenoy Pratik <[email protected]>

* notebooks fix for cypress and minor bugs (opensearch-project#392) (opensearch-project#393)

* notebooks fix for cypress and minor bugs

Signed-off-by: Shenoy Pratik <[email protected]>

* remove comments

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 3219f14)

* Update traces integration tests (opensearch-project#391) (opensearch-project#397)

* Update traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Update traces cypress test

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix traces cypress tests

Signed-off-by: Rupal Mahajan <[email protected]>

* Fix filters test

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>
(cherry picked from commit b8f9ca3)

Co-authored-by: Rupal Mahajan <[email protected]>

* Fix redirection to legacy event_analytics URL from dashboards (opensearch-project#399) (opensearch-project#403)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 05ef180)

Co-authored-by: Joshua Li <[email protected]>

* Adjust metrics top menu layout to avoid overflow (opensearch-project#398) (opensearch-project#402)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 6309e63)

Co-authored-by: Joshua Li <[email protected]>

* Support duplicate visualization in dashboard (opensearch-project#400) (opensearch-project#405)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 3d74bac)

Co-authored-by: Joshua Li <[email protected]>

* fix panel visualization preview, new viz workflow (opensearch-project#401) (opensearch-project#404)

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit 78347d9)

Co-authored-by: Shenoy Pratik <[email protected]>

* Fix saving multiple metrics to SOpanels (opensearch-project#407) (opensearch-project#408)

* fix saving multiple metrics to SOpanels

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused variable

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>
(cherry picked from commit a4cb6d1)

Co-authored-by: Shenoy Pratik <[email protected]>

* Panel table fixes (opensearch-project#406)

* Fix Panel View (legacy)
  - Duplicate
  - Rename
  - Delete

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Shenoy Pratik <[email protected]>

* Release Notes 2.7.0.0 (opensearch-project#394) (opensearch-project#412)

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Release Notes 2.7.0.0

Signed-off-by: Peter Fitzgibbons <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
(cherry picked from commit 969e639)

Co-authored-by: Peter Fitzgibbons <[email protected]>

* [Dashboard listing] update edit url and view url (opensearch-project#416) (opensearch-project#421)

* [Dashboard listing] update edit url and view url

Update to include BASE_URL if basePath is defined. Since create url is just
a direct nav and needs the basePath when passing props to the provider, the source
code no longer appends the basePath to keep these URLs consistent.

A feature should consolidate the basePaths in the dashboard listing service so that
createUrl also adds the base path. At which point we no longer need the base path.

Also removed some rogue URLs.

Core PR:
opensearch-project/OpenSearch-Dashboards#3922

---------

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
(cherry picked from commit f94c21e)

Co-authored-by: Kawika Avilla <[email protected]>

* Fix invalid time range in new panels (opensearch-project#419)

Signed-off-by: Joshua Li <[email protected]>

* open new tab for editing panel visualization (opensearch-project#430)

Signed-off-by: Eric Wei <[email protected]>

* Save new prometheus metrics to OSD saved objects (opensearch-project#428)

* Save new prometheus metrics to OSD saved objects

Signed-off-by: Joshua Li <[email protected]>

* Return if failed at saving metrics

Signed-off-by: Joshua Li <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>

* fixes edit mode callback bug in Saved Object Panels (opensearch-project#420)

* fix for SOpanels editMode

Signed-off-by: Shenoy Pratik <[email protected]>

* rename edit button data-test-subj

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Shenoy Pratik <[email protected]>

* Add Toasts to Observability Dashboards  (opensearch-project#435)

* Fixes
* Panel View (legacy)
  - Duplicate
  - Rename

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Toasts use hook from useOpenSearchDashboards context provider

Signed-off-by: Peter Fitzgibbons <[email protected]>

* Testing for CustomPanel Toast

Signed-off-by: Peter Fitzgibbons <[email protected]>

* update catches from comments, minor code cleaning

Signed-off-by: Shenoy Pratik <[email protected]>

* update tests

Signed-off-by: Shenoy Pratik <[email protected]>

* remove unused redux slice

Signed-off-by: Shenoy Pratik <[email protected]>

* revert cypress changes

Signed-off-by: Shenoy Pratik <[email protected]>

* add toasts to SOflyout

Signed-off-by: Shenoy Pratik <[email protected]>

* fix messaging for multiple delete

Signed-off-by: Derek Ho <[email protected]>

* fix up toast and error handling for create and delete flows

Signed-off-by: Derek Ho <[email protected]>

* fix up clone

Signed-off-by: Derek Ho <[email protected]>

* fix rename in table

Signed-off-by: Derek Ho <[email protected]>

* fix rename in custom panel so view

Signed-off-by: Derek Ho <[email protected]>

* fix up panel toasts

Signed-off-by: Derek Ho <[email protected]>

* fix up for flyout

Signed-off-by: Derek Ho <[email protected]>

* code cleanup

Signed-off-by: Derek Ho <[email protected]>

* finish merge

Signed-off-by: Derek Ho <[email protected]>

* fix up PR comments

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>

* change samples to saved object (opensearch-project#427) (opensearch-project#436)

* change samples to saved object

* add sample to so

* fix saved object sample

* fix up

* add back toast

* revert file

* refactoring

* use constant

---------

(cherry picked from commit ce06ccb)

Signed-off-by: Derek Ho <[email protected]>

* Add Error handling on duplicate dashboard names (opensearch-project#441)

* Add check for existing panel name

Signed-off-by: Rupal Mahajan <[email protected]>

* Nit

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove double await

Signed-off-by: Rupal Mahajan <[email protected]>

* Add rename name check test

Signed-off-by: Rupal Mahajan <[email protected]>

* Remove test

Signed-off-by: Rupal Mahajan <[email protected]>

* Add unique id to sample panel name

Signed-off-by: Rupal Mahajan <[email protected]>

---------

Signed-off-by: Rupal Mahajan <[email protected]>

* fix timestamp overriding issue (opensearch-project#449)

Signed-off-by: Eric Wei <[email protected]>

* Renaming custom panels to Observability Dashboards (opensearch-project#450)

* change custom panels -> observability dashboards

Signed-off-by: Eric Wei <[email protected]>

* rest of the panel renaming

Signed-off-by: Eric Wei <[email protected]>

---------

Signed-off-by: Eric Wei <[email protected]>

* Fix delete failure when no osd item selected (opensearch-project#452)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit a4a310e)

* Open log patterns by default (opensearch-project#444) (opensearch-project#459)

Signed-off-by: Joshua Li <[email protected]>
(cherry picked from commit 72970aa)

* diff merging between 2.x and 2.7

Signed-off-by: Shenoy Pratik <[email protected]>

* add service fields length check in app analytics

Signed-off-by: Shenoy Pratik <[email protected]>

---------

Signed-off-by: Joshua Li <[email protected]>
Signed-off-by: Eric Wei <[email protected]>
Signed-off-by: Peter Fitzgibbons <[email protected]>
Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Joshua Li <[email protected]>
Co-authored-by: Eric Wei <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Derek Ho <[email protected]>
Co-authored-by: Peter Fitzgibbons <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]>
Co-authored-by: Kawika Avilla <[email protected]>
(cherry picked from commit 776d1ed)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants