-
Notifications
You must be signed in to change notification settings - Fork 577
Correct list of dashboards plugins #10264
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
Conversation
Signed-off-by: Fanit Kolchina <[email protected]>
Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged. Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer. When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review. |
@peterzhuamazon Could you please review this PR? |
|
||
Then remove all associated entries from `opensearch_dashboards.yml`. | ||
|
||
For certain plugins, you must also remove the "optimize" bundle. This is a sample command for the Anomaly Detection plugin: | ||
For certain plugins, you must also remove the optimized bundle. This is a sample command for the Anomaly Detection plugin: |
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.
For certain plugins, you must also remove the optimized bundle. This is a sample command for the Anomaly Detection plugin: |
|
||
```bash | ||
sudo rm /usr/share/opensearch-dashboards/optimize/bundles/opensearch-anomaly-detection-opensearch-dashboards.* | ||
``` | ||
{% include copy.html %} | ||
|
||
Then restart OpenSearch Dashboards. After you remove any plugin, OpenSearch Dashboards performs an optimize operation the next time you start it. This operation takes several minutes even on fast machines, so be patient. | ||
|
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.
```bash | |
sudo rm /usr/share/opensearch-dashboards/optimize/bundles/opensearch-anomaly-detection-opensearch-dashboards.* | |
``` | |
{% include copy.html %} | |
Then restart OpenSearch Dashboards. After you remove any plugin, OpenSearch Dashboards performs an optimize operation the next time you start it. This operation takes several minutes even on fast machines, so be patient. |
{% include copy.html %} | ||
|
||
1. Remove the optimized bundle: | ||
|
||
```bash | ||
sudo rm /usr/share/opensearch-dashboards/optimize/bundles/<bundle-name> | ||
``` |
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.
{% include copy.html %} | |
1. Remove the optimized bundle: | |
```bash | |
sudo rm /usr/share/opensearch-dashboards/optimize/bundles/<bundle-name> | |
``` |
|
||
```bash | ||
sudo bin/opensearch-dashboards-plugin remove <plugin-name> | ||
sudo bin/opensearch-dashboards-plugin remove alertingPlugin |
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.
sudo bin/opensearch-dashboards-plugin remove alertingPlugin | |
sudo bin/opensearch-dashboards-plugin remove alertingDashboards |
Signed-off-by: Fanit Kolchina <[email protected]>
Signed-off-by: Fanit Kolchina <[email protected]>
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.
Only 1 issue if you can change alertingPlugin
to alertingDashboards
this PR is good.
Thanks.
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.
Editorial review
Signed-off-by: Nathan Bower <[email protected]>
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.
LGTM
@kolchfa-aws Can you fix Jekyll build failure? |
Signed-off-by: Fanit Kolchina <[email protected]>
Signed-off-by: Fanit Kolchina <[email protected]>
* Correct list of dashboards plugins Signed-off-by: Fanit Kolchina <[email protected]> * Review comments Signed-off-by: Fanit Kolchina <[email protected]> * Change plugin name Signed-off-by: Fanit Kolchina <[email protected]> * Apply suggestions from code review Signed-off-by: Nathan Bower <[email protected]> * Fix links Signed-off-by: Fanit Kolchina <[email protected]> * Fix link Signed-off-by: Fanit Kolchina <[email protected]> --------- Signed-off-by: Fanit Kolchina <[email protected]> Signed-off-by: Nathan Bower <[email protected]> Co-authored-by: Nathan Bower <[email protected]> (cherry picked from commit 54d1f76) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Closes #8582
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.