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

Dashboard Lists Integration #3000

Conversation

pjfitzgibbons
Copy link
Contributor

@pjfitzgibbons pjfitzgibbons commented Dec 1, 2022

Description

  • Configure types for registration of Dashboard List "Providers" and "Creators"

  • Add Left-Nav "top-level" category for Observability

Signed-off-by: Peter Fitzgibbons pjfitz@amazon.com

These changes enable the interactions demonstrated in this animation:

Dashboard List Integration

Issues WIP

[FEATURE] Dashboards List Extensibility & Integrate Observability Apps/Panels/EventAnalytics

  • includes alternate solutions, example animation, and further explanation of this solution.

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

Tommy Markley and others added 30 commits April 19, 2022 16:48
* Renames build and test workflow to eliminate ambiguity.
* Adds coverage generation for jest and mocha tests.
* Expands coverage collection for jest tests.
* Utilizes `lcov` reporter as well as text summary reporter.

Resolves opensearch-project#1339
Prerequisite for opensearch-project#1035

Signed-off-by: Tommy Markley <markleyt@amazon.com>
* feat(Expressions): Initial demo plugin

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* feat(Expressions): Adds handlers tab

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* feat(Expressions): Adds playground tab

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* chore: Better expression playground messaging

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* feat(Expressions): Adds expression explorer

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* feat(Expressions): A better explorer

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* fix(Expressions): Fix basic demo abort error

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* Updates readme and fixes rendering visualizations

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>
* [Bug] fix copy as curl

Copy as curl doesn't work in Dev Tools console due to usage of a
deprecated method:

https://developer.mozilla.org/en-US/docs/Web/API/Document/queryCommandSupported#browser_compatibility

Issue Resolved:
opensearch-project#1471

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

* Adds browser compatibility comment

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>

Co-authored-by: Bishoy Boktor <boktorbb@amazon.com>
Resolves opensearch-project#1035

Signed-off-by: Tommy Markley <markleyt@amazon.com>
* fix(actions): Better type checks for icons

- fix typo in iconType for `replace_panel` action
- change iconType types from `string` to `EuiIconType`
- use `icon` instead of `iconType` for svg pathseslin

Signed-off-by: Josh Romero <rmerqg@amazon.com>

* fix(actions): Remove unnecessary icon assertions

Signed-off-by: Josh Romero <rmerqg@amazon.com>
…typing (opensearch-project#1491)

* fix(Expressions): Fixes rendering viz when expression changes

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* chore: Improves visualization types

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>

* chore: Updates Render type for other viz expressions

Signed-off-by: Ashwin Pc <ashwinpc@amazon.com>
Increment from 2.0.0 to 3.0.0, also some version checks.
Also, add a release build verification workflow. To verify x64
and ARM64 artifacts are built successfully with dependencies.

Issue:
n/a

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
Bumps [ejs](https://github.com/mde/ejs) from 3.1.6 to 3.1.7.
- [Release notes](https://github.com/mde/ejs/releases)
- [Changelog](https://github.com/mde/ejs/blob/main/CHANGELOG.md)
- [Commits](mde/ejs@v3.1.6...v3.1.7)

---
updated-dependencies:
- dependency-name: ejs
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Anan Zhuang <ananzh@amazon.com>

fix PR comments

Signed-off-by: Anan Zhuang <ananzh@amazon.com>

add CVE details and emoji

Signed-off-by: Anan Zhuang <ananzh@amazon.com>

make node-14 to CVE

Signed-off-by: Anan Zhuang <ananzh@amazon.com>

remove UI lib from breaking changes. correct deprecations and maintanence

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
* [CI] Create doc issue workflow

Add workflow to create documentation issue in
https://github.com/opensearch-project/documentation-website

Update CONTRIBUTING doc on how to use.

Issue:
opensearch-project#1493

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

* follow up

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
Signed-off-by: Tommy Markley <markleyt@amazon.com>
* This workflow is unnecessary now that the org uses the DCO App.
* Removes the DCO badge from the README.

Signed-off-by: Tommy Markley <markleyt@amazon.com>
…-project#1552)

* Addresses CVE-2022-1214
* Bumps and resolves `axios` to ^0.27.2 to address CVE
  * [CHANGELOG](https://github.com/axios/axios/blob/master/CHANGELOG.md)
* Bumps `chromedriver` to v100 to match GitHub actions

Signed-off-by: Bishoy Boktor <boktorbb@amazon.com>
add back icon directive, which injects SVG

Signed-off-by: Josh Romero <rmerqg@amazon.com>
* Exact copy of 2.0.0-rc1 release notes.
* New changes between RC1 and GA must be added as PRs are merged.

Signed-off-by: Tommy Markley <markleyt@amazon.com>
Signed-off-by: Tommy Markley <markleyt@amazon.com>
Addresses CVE-2022-1537

Issue:
opensearch-project#1579

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
…arch-project#1559)

* Utilizes Codecov yaml configuration: https://docs.codecov.com/docs/codecov-yaml
* The purpose of enforcing increasing code coverage over time is to attempt to improve the quality of 
  changes via testing, reduce the need to request tests in PRs, and stay away from enforcing an 
  ambiguous coverage percentage.

Signed-off-by: Tommy Markley <markleyt@amazon.com>
…t#1598)

There is currently to a PR to remove the v7 and v8 (beta) theme
versions within the application but it is getting stale:
opensearch-project#978

Since the v8 (beta) theme isn't actually planned and has the incorrect
version, we do not want end users to be able to change this setting.
In the essence of time, this will prevent this setting from showing in
the Advanced Settings page but will also not break users who set this
version already (however it will lock them into theme until they switched
by manually updating or deleting the config doc). Removing this setting
will force the default to be v8 (beta).

Temporary fix for:
opensearch-project#494

But it should be removed completely.

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
* [Build][Tests] handle painless scripts update

Sample data and functional tests were utilizing deprecated
methods for painless scripts.

However these deprecated methods were removed in:
opensearch-project/OpenSearch#3346

This is to ensure sample data and ftr tests work but end users
saved objects might contain painless scripts that we might not
be able to address. So we should research if this it is
possible to help mitigate this.

Related issue:
opensearch-project#1600

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
…#1626)

package.json and yarn.lock already updated to 100.0.0

Signed-off-by: Josh Romero <rmerqg@amazon.com>
… upload feature (opensearch-project#1632)

* adds logic to accommodate import custom map tab from dashboards-maps plugin

Signed-off-by: Shivam Dhar <dhshivam@amazon.com>

* adds type to http and notifications

Signed-off-by: Shivam Dhar <dhshivam@amazon.com>

Co-authored-by: Shivam Dhar <dhshivam@amazon.com>
* [CI][BWC] add BWC tests to github actions

Add backwards compatibility tests to github workflow for PRs.
Also, fix for checking status of OpenSearch Dashboards since
api/status/ is not protected.

Also, enable ability to create test data since migration from
1.0.0 to 3.0.0 is not supported for OpenSearch currently.

Issue:
opensearch-project#1465
opensearch-project#1651

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

* Address comments

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
…project#1570)

Signed-off-by: Rafael Vargas <rafaelflvargas@gmail.com>
raintygao and others added 16 commits November 11, 2022 12:12
…efox (opensearch-project#2853)

* fix: fix VisBuilder empty animation does not work in firefox

Signed-off-by: raintygao <tygao@amazon.com>

* doc: add changelog for pull request

Signed-off-by: raintygao <tygao@amazon.com>

Signed-off-by: raintygao <tygao@amazon.com>
…pensearch-project#2867)

Signed-off-by: AbhishekReddy1127 <nallamsa@amazon.com>

Signed-off-by: AbhishekReddy1127 <nallamsa@amazon.com>
- Update README with full capabilities
- Add `current_usage` to track current usage

Signed-off-by: Josh Romero <rmerqg@amazon.com>
…project#2734)

* Adding yarn opensearch args to setup opensearch plugin dependencies on snapshot
* Updating the example opensearch snapshot help command with --P arg
* Adding tests to the plugin installation function
* Adding documentation for the new flag in developer guide

Signed-off-by: Manasvini B Suryanarayana <manasvis@amazon.com>
* [WS-2021-0638][Security] bump mocha to 10.1.0

In our current repo, most mocha tests have been changed
to jest in this issue:
opensearch-project#215

The fix for the above issue
(mochajs/mocha@61b4b92)
is only released in the latest version v10.1.0:
https://github.com/mochajs/mocha/blob/5f96d511dbf913f135b92198aab721a27f6b44fe/lib/utils.js#L79

Since we are considering this change might break others, we will not backport to 2.x and release it in 3.0.0

Issue Resolved:
opensearch-project#2269

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
)

* OSD-2847 bump del version to fix race condition

Signed-off-by: Daniel Belmes <3631206+DanielBelmes@users.noreply.github.com>
* register ru locale

Signed-off-by: Dmitry Dudin <d_dudin@step.ru>
…project#2810)

* [Doc] Correct "help" command to reflect actual behaviour

Signed-off-by: Patrick Clausen <clausen.patrick@gmail.com>

* Update CHANGELOG.md

Signed-off-by: Patrick Clausen <clausen.patrick@gmail.com>

Signed-off-by: Patrick Clausen <clausen.patrick@gmail.com>
Co-authored-by: Josh Romero <rmerqg@amazon.com>
The release notes are in the 2.4 branch and 2.4.0 tag but not in main or
2.x which is our usual standard.

Instead of cherry-picking commits I have just copied over the file.

Issue:
n/a

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
* Reenable CI tests for feature branches

Signed-off-by: Ashwin P Chandran <ashwinpc@amazon.com>
…pensearch-project#2964)

* When updating the timestamps were requested, scanCopy first set the times and then copied content into the destination folder. On certain platforms, copying files into a folder updates its "Last Access Time" and that overwrites the just set timestamps. This PR, makes sure the timestamps are set only after copying the content.

Signed-off-by: Miki <amoo_miki@yahoo.com>

Signed-off-by: Miki <amoo_miki@yahoo.com>
…ch-project#2965)

Signed-off-by: Miki <amoo_miki@yahoo.com>

Signed-off-by: Miki <amoo_miki@yahoo.com>
…roject#2963)

* Makes `yarn pkg-version` work across platforms

Signed-off-by: Miki <amoo_miki@yahoo.com>

* Makes `yarn pkg-version` work across platforms

Signed-off-by: Miki <amoo_miki@yahoo.com>

Signed-off-by: Miki <amoo_miki@yahoo.com>
Signed-off-by: Miki <amoo_miki@yahoo.com>

Signed-off-by: Miki <amoo_miki@yahoo.com>
…ross platforms (opensearch-project#2994)

Signed-off-by: Miki <amoo_miki@yahoo.com>

Signed-off-by: Miki <amoo_miki@yahoo.com>
@pjfitzgibbons pjfitzgibbons requested a review from a team as a code owner December 1, 2022 21:42
@pjfitzgibbons pjfitzgibbons mentioned this pull request Dec 1, 2022
8 tasks
Bumps [loader-utils](https://github.com/webpack/loader-utils) from 2.0.3 to 2.0.4.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v2.0.4/CHANGELOG.md)
- [Commits](webpack/loader-utils@v2.0.3...v2.0.4)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@AMoo-Miki
Copy link
Collaborator

Thanks for raising this PR. Why would this not go to main?

@pjfitzgibbons
Copy link
Contributor Author

Why would this not go to main?
@AMoo-Miki This PR is also related to Observability opensearch-project/observability#1314 . We are targeting 2.x on both sides.
I'll be happy to backport this once we get it correctly merged.

AMoo-Miki and others added 2 commits December 2, 2022 12:17
Fixes opensearch-project#2996, opensearch-project#2353

- Bump `leaflet-vega` to `^0.9.0` - they merged in our upstream PR and made a release
- Fix the parameter name for passing `vega.parse` options.

Signed-off-by: Miki <miki@amazon.com>
Basic data points
[OSD] 16 submitted PRs (https://github.com/opensearch-project/OpenSearch-Dashboards/pulls/manasvinibs)
[OSD] 74 reviewed PRs (https://github.com/opensearch-project/OpenSearch-Dashboards/issues?q=reviewed-by%3Amanasvinibs)
[OSD] 28 issues involved (https://github.com/opensearch-project/OpenSearch-Dashboards/issues?page=1&q=involves%3Amanasvinibs+is%3Aissue)

Highlight
Mana is assisting with extensions project which will be the next evolution of extending core functionality from OpenSearch Dashboards
Mana implemented opensearch-project#2734 which allows for a huge quality of life for local development for external plugin developers to utilize snapshots with a single CLI command compared to before when they would had to pull down OpenSearch build, install their plugin on OpenSearch, and ensure the proper configurations. This has caused historically problems when plugin teams do development and miss some steps per their onboard documentation/PR suggestion and get different results than expected.
Mana has assisted reviewing PRs providing great insight on BWC tests, BWC in general, and the release process.
Mana has added documentation from insight she has gained within the informal dev doc repo https://cptnb.github.io/opensearch-dashboards-dev-docs/ ensuring the spread of knowledge.

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>

Signed-off-by: Kawika Avilla <kavilla414@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Merging #3000 (7aaa7bb) into 2.x (96952e7) will decrease coverage by 0.08%.
The diff coverage is 6.45%.

@@            Coverage Diff             @@
##              2.x    #3000      +/-   ##
==========================================
- Coverage   66.73%   66.64%   -0.09%     
==========================================
  Files        3219     3220       +1     
  Lines       61450    61474      +24     
  Branches     9417     9418       +1     
==========================================
- Hits        41007    40969      -38     
- Misses      18203    18258      +55     
- Partials     2240     2247       +7     
Flag Coverage Δ
Linux 66.64% <6.45%> (-0.04%) ⬇️
Windows ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/plugins/dashboard/public/plugin.tsx 0.00% <0.00%> (ø)
...s_react/public/table_list_view/table_list_view.tsx 0.00% <0.00%> (ø)
.../table_list_view/components/create_button_menu.tsx 5.00% <5.00%> (ø)
...rd/public/application/listing/dashboard_listing.js 76.92% <100.00%> (-1.65%) ⬇️
src/dev/build/lib/get_build_number.ts 57.14% <0.00%> (-42.86%) ⬇️
src/setup_node_env/harden/child_process.js 38.46% <0.00%> (-38.47%) ⬇️
packages/osd-cross-platform/src/path.ts 51.21% <0.00%> (-34.15%) ⬇️
...ges/osd-apm-config-loader/src/config.test.mocks.ts 91.30% <0.00%> (-8.70%) ⬇️
src/dev/build/lib/config.ts 79.41% <0.00%> (-5.89%) ⬇️
...s/osd-optimizer/src/node/node_auto_tranpilation.ts 83.67% <0.00%> (-4.09%) ⬇️
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ashwin-pc
Copy link
Member

@pjfitzgibbons can we close this since we have #3090?

@pjfitzgibbons
Copy link
Contributor Author

Duplicate of #3090

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.