Fixed a bug that displayed all alerts for a monitor on individual tri… #130

annie3431 · 2021-10-12T18:20:33Z

Cherry-pick the content of #123 from branch 1.1

Description

Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts.
Fixed a bug that displayed incorrect source for the condition field on the alerts flyout.
Fixed a bug that displayed incorrect severity on the alerts flyout.
Updated release notes to reflect bug fix.
Fixed bug relating to validation of popovers when defining monitor queries. Additional validation on query definition combo boxes to ensure no additional values are provided when switching monitor type. Additional validation on query definition combo boxes to ensure empty values are not passed in.
Fix the problem of bucket level monitor preview graph not showing up when minDate equals to maxDate.
A few text update.
Fix the bug when bucket-level monitor metrics fields contains '.', the data won't show correctly in preview graph. Added a parsing function to parse '.' to '_'.
Removing a redundant comma in bucket-level alerting graph.
Refactored Dashboard.js Acknowledge button to support bulk acknowledging alerts.
Remove "Acknowledge" action option on monitors page when selecting multiple monitors.

Issues Resolved

This PR is a consolidation of PRs 121, and 122.
Issue [BUG] Bugs found during 1.1 pre-release sanity testing #127.

Check List

New functionality includes testing.
- All tests pass
New functionality has been documented.
- New functionality has javadoc 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.

…ggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (opensearch-project#123) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (opensearch-project#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (opensearch-project#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (opensearch-project#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (opensearch-project#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * More bug fix Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]>

codecov-commenter · 2021-10-12T18:37:50Z

Codecov Report

Merging #130 (8dbe8cf) into main (725c2d7) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
- Coverage   50.93%   50.79%   -0.15%     
==========================================
  Files         200      200              
  Lines        4639     4666      +27     
  Branches      586      594       +8     
==========================================
+ Hits         2363     2370       +7     
- Misses       2274     2294      +20     
  Partials        2        2

Impacted Files	Coverage Δ
...nitors/components/MonitorActions/MonitorActions.js	`88.46% <0.00%> (-11.54%)`	⬇️
...gin/public/pages/Dashboard/containers/Dashboard.js	`62.50% <0.00%> (-3.88%)`	⬇️
...MonitorExpressions/expressions/MetricExpression.js	`74.07% <0.00%> (-1.93%)`	⬇️
...onitorExpressions/expressions/GroupByExpression.js	`75.00% <0.00%> (-1.93%)`	⬇️
...ateMonitor/components/VisualGraph/utils/helpers.js	`46.23% <0.00%> (-1.55%)`	⬇️
...lerting-dashboards-plugin/public/utils/validate.js	`68.65% <0.00%> (-1.50%)`	⬇️
...ublic/components/Flyout/flyouts/alertsDashboard.js	`13.15% <0.00%> (-0.36%)`	⬇️
...nents/MonitorExpressions/expressions/MetricItem.js	`0.00% <0.00%> (ø)`
...ents/MonitorExpressions/expressions/GroupByItem.js	`0.00% <0.00%> (ø)`
.../MonitorExpressions/expressions/utils/constants.js	`100.00% <0.00%> (ø)`
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 725c2d7...8dbe8cf. Read the comment docs.

…ggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (opensearch-project#123) (opensearch-project#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (opensearch-project#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (opensearch-project#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (opensearch-project#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (opensearch-project#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * More bug fix Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]>

* Bumps version to 1.2 (#128) * Bumps version to 1.2 * Changes test workflows to follow Dashboards 1.x Signed-off-by: Clay Downs <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (#123) (#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * More bug fix Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: AWSHurneyt <[email protected]>

…t#131) * Bumps version to 1.2 (opensearch-project#128) * Bumps version to 1.2 * Changes test workflows to follow Dashboards 1.x Signed-off-by: Clay Downs <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (opensearch-project#123) (opensearch-project#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (opensearch-project#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (opensearch-project#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (opensearch-project#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (opensearch-project#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * More bug fix Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: AWSHurneyt <[email protected]>

…t#131) * Bumps version to 1.2 (opensearch-project#128) * Bumps version to 1.2 * Changes test workflows to follow Dashboards 1.x Signed-off-by: Clay Downs <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (opensearch-project#123) (opensearch-project#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (opensearch-project#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (opensearch-project#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (opensearch-project#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (opensearch-project#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (opensearch-project#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * More bug fix Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]>

* OpenSearch 1.1.0 release (#103) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Updates to workflow, unit tests, and some appearance (#114) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Cherry-pick commits from main branch to 1.x branch (#131) * Bumps version to 1.2 (#128) * Bumps version to 1.2 * Changes test workflows to follow Dashboards 1.x Signed-off-by: Clay Downs <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (#123) (#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * More bug fix Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Cherry-picking commits from main to 1.x branch for 1.2 release (#142) * Fixes flaky test and removes local publishing of plugin dependencies (#135) * Fixes 'bucket level monitor can be created by extraction query' flaky test * Removes local publishing of plugin dependencies for github cypress tests Signed-off-by: Clay Downs <[email protected]> * Update copyright notice (#140) Signed-off-by: Mohammad Qureshi <[email protected]> * Added 1.2 release notes. (#141) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Added 1.2 release notes. Signed-off-by: Thomas Hurney <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Added 1.2 release notes. Signed-off-by: Thomas Hurney <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Updated 1.2 release notes. Signed-off-by: Thomas Hurney <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Added DCO section to CONTRIBUTING.md and GitHub workflows. Updated copyright notice. Signed-off-by: Thomas Hurney <[email protected]> (#145) Signed-off-by: AWSHurneyt <[email protected]> * support creating monitor for anomaly detector with custom result index (#143) (#147) * support creating monitor for anomaly detector with custom result index Signed-off-by: Yaliang Wu <[email protected]> * add release note Signed-off-by: Yaliang Wu <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Bumping version to 1.3. Signed-off-by: AWSHurneyt <[email protected]> (#159) Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * initial commit (#150) Signed-off-by: CEHENKLE <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add .whitesource configuration file (#153) Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Signed-off-by: AWSHurneyt <[email protected]> * Implemented a toast to display successful attempts to acknowledge alerts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. (#160) * Implemented unit and integ tests for the alerts dashboard flyout. Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Removed an unused test variable. Signed-off-by: AWSHurneyt <[email protected]> * Removed debug logs. Signed-off-by: AWSHurneyt <[email protected]> * Implemented unit test. Refactored integration tests to use fewer wait periods. Signed-off-by: AWSHurneyt <[email protected]> * Examining flakiness in cypress test. Signed-off-by: AWSHurneyt <[email protected]> * Added short wait period to flyout cypress tests to alleviate flakiness. Signed-off-by: AWSHurneyt <[email protected]> * Refactored flyout cypress tests to use aliases. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Adding basic unit tests (#151) * Add flyout render Signed-off-by: Annie Lee <[email protected]> * Add some basic tests Signed-off-by: Annie Lee <[email protected]> * Add more tests Signed-off-by: Annie Lee <[email protected]> * Update jest.config.js Signed-off-by: Annie Lee <[email protected]> * Add tests Signed-off-by: Annie Lee <[email protected]> * Update dashboard sample alerttime Signed-off-by: Annie Lee <[email protected]> * Update dashboard test alert time snapshot Signed-off-by: Annie Lee <[email protected]> * Update Dashboard.test.js Signed-off-by: Annie Lee <[email protected]> * Remove test alert start time Signed-off-by: Annie Lee <[email protected]> * Update VisualGraph test and code Signed-off-by: Annie Lee <[email protected]> * Update package.json to run unit tests in UTC timezone Signed-off-by: Annie Lee <[email protected]> * Update dashboard sample alert start time Signed-off-by: Annie Lee <[email protected]> * Update Dashboard test Signed-off-by: Annie Lee <[email protected]> * Update Dashboard.test.js.snap Signed-off-by: Annie Lee <[email protected]> * Update snapshot file Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Fix the error handling when config index is not found (#173) Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Updated copyright notices and headers. (#168) * Updated CONTRIBUTING.md and NOTICE files. Signed-off-by: AWSHurneyt <[email protected]> * Updated license headers. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Refactored acknowledge alerts button on Alerts by trigger dashboard page to be a modal experience. (#167) * Refactored Acknowledge button on Alerts by trigger dashboard to display a modal. Signed-off-by: AWSHurneyt <[email protected]> * Updated license headers. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Adding a few more basic unit tests (#180) * Add alerting dashboards flyout render unit test Signed-off-by: Annie Lee <[email protected]> * Add MonitorTimeFields validation tests Signed-off-by: Annie Lee <[email protected]> * Refactor error messages to add period Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Remove node version declaration in package.json (#166) * Remove node version declaration in package.json Signed-off-by: Annie Lee <[email protected]> * Remove react-router-dom version declaration Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Update DestinationsService.js (#182) Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add backport workflow (#176) * Add backport workflow Signed-off-by: Annie Lee <[email protected]> * Create delete_backport_branch.yml Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Configure test workflows to run on 1.x branches (#183) Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add 1.3 release notes (#175) (#185) * Create opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <[email protected]> (cherry picked from commit 98f7904) Co-authored-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add backport documentation link (#184) (#187) * Create BACKPORT.md Signed-off-by: Annie Lee <[email protected]> * Replace documentation with a link Signed-off-by: Annie Lee <[email protected]> (cherry picked from commit 0a0c8c5) Co-authored-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Implemented support for cluster metrics monitors (#162) (#189) * Implemented LocalUriInput component, and supporting methods. Implemented unit and integration tests for LocalUriInput. Refactored various other classes and components to support LocalUriInput. Signed-off-by: AWSHurneyt <[email protected]> * Refactored learn more links to open new tabs. Signed-off-by: AWSHurneyt <[email protected]> * Implemented default trigger conditions for API supported by the LocalUriInput feature. Signed-off-by: AWSHurneyt <[email protected]> * Refactored button text and size based on UX reviewer feedback. Signed-off-by: AWSHurneyt <[email protected]> * Added documentation link for cluster stats API. Signed-off-by: AWSHurneyt <[email protected]> * Adjusted some wording based on UX reviewer feedback. Implemented modal that displays when changing request type. Signed-off-by: AWSHurneyt <[email protected]> * Refactored feature assets to use ClusterMetrics naming convention instead of LocalUriInput. Signed-off-by: AWSHurneyt <[email protected]> * Refactored behavior of cluster metrics clear triggers modal, and implemented tests. Signed-off-by: AWSHurneyt <[email protected]> * Added periods to error messages. Signed-off-by: AWSHurneyt <[email protected]> * Removed fixed width for clearTriggersModal. Signed-off-by: AWSHurneyt <[email protected]> * Refactored the cluster metrics radio card to be a monitor type instead of monitor definition type. Refactored request type selection combobox so it's not clearable, and reworded the clearTriggersModal text and refactored tests accordingly. Signed-off-by: AWSHurneyt <[email protected]> * Updated copyright headers. Signed-off-by: AWSHurneyt <[email protected]> * Refactoring monitorType checking logic to accommodate cluster metrics monitors. Signed-off-by: AWSHurneyt <[email protected]> * Refactored help text to use lowercase letters. Signed-off-by: AWSHurneyt <[email protected]> * Implemented a helper method as a temporary solution to some incorrect formatting of cluster metrics monitors returned by the getMonitors API. Signed-off-by: AWSHurneyt <[email protected]> * Refactored cluster metrics feature to remove support for CAT repositories. Signed-off-by: AWSHurneyt <[email protected]> * Refactored cluster metrics request types labels to no longer reference CAT. Signed-off-by: AWSHurneyt <[email protected]> * Adjusted the example trigger conditions for two request types. Signed-off-by: AWSHurneyt <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: Yaliang <[email protected]> Co-authored-by: CEHENKLE <[email protected]> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>

* Add backport documentation link (#184) (#187) * Create BACKPORT.md Signed-off-by: Annie Lee <[email protected]> * Replace documentation with a link Signed-off-by: Annie Lee <[email protected]> (cherry picked from commit 0a0c8c5) Co-authored-by: Annie Lee <[email protected]> * 1.3release - cherry-picking from main to 1.x branch (#192) * OpenSearch 1.1.0 release (#103) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Updates to workflow, unit tests, and some appearance (#114) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Cherry-pick commits from main branch to 1.x branch (#131) * Bumps version to 1.2 (#128) * Bumps version to 1.2 * Changes test workflows to follow Dashboards 1.x Signed-off-by: Clay Downs <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual triggers' flyouts. Fixed a bug that displayed incorrect source for the condition field on the alerts flyout. Fixed a bug that displayed incorrect severity on the alerts flyout. Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Fixed bug relating to validation of popovers when defining monitor queries. (#123) (#130) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md (#101) * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Create opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update comments Signed-off-by: Annie Lee <[email protected]> * Update version in package.json (#102) * Update package.json * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Text updates (#105) * Add icon tooltip * Update text and rename MonitorDefinitionCard directory * Update Schedule.js * Update Schedule.js Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md * Remove license Signed-off-by: Annie Lee <[email protected]> * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. (#104) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml * Update jest unit tests (#112) * Update .opensearch_dashboards-plugin-helpers.json * Update snapshots * Update whereExpression.test and some snapshots * Update whereExpression.test and some snapshots Signed-off-by: Annie Lee <[email protected]> * Update snapshots * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Update formikToTrigger.test.js * Add icon tooltip * Add test * Update tests * remove license * Update TriggerExpressions.test.js * Update Triggers.test.js * Update validation test * Update getOverviewStats.test.js * Update validate.test.js * Update helpers.test.js and remove unused import * Update Triggers.test.js * Update helpers.js * Update CreateMonitor test and clean up code * Update CreateMonitor test and clean up code Signed-off-by: Annie Lee <[email protected]> * Update release note and adding more tests * Add test and modify cypress common-utils branch * Update MonitorDefinitionCard.test.js.snap * Update cypress-workflow.yml Signed-off-by: Annie Lee <[email protected]> * Fixed a few bugs * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * Fixed a bug that displayed all alerts for a monitor on individual trigger flyouts. Fixed a bug that diplayed incorrect source for the condition field on the alerts flyout. Fixed a bug that diplaying incorrect severity on the alerts flyout. * Updated release notes to reflect PR 122 bug fix. * Fixing number of alerts displayed on Monitors tab. * Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md Signed-off-by: Annie Lee <[email protected]> * More bug fix Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Skip test based on modification Signed-off-by: Annie Lee <[email protected]> * Update popover windows to remove item when filed is not defined * Update field validation * Fixed a bug that prevented selecting query-level monitor alerts 1 by 1. Removed experimental code and comments. * Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix * Update Dashboard.js * Support data filter when using null operator * Update WhereExpression.js * Fixed a bug that was causing incorrect pagination display on alerts flyout. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Removed redundant validation from filter values that was generating error messages that prevented preview graphs from displaying data. * Update metric error for query monitors * Update MetricExpression.js * Removed experimental dev code. * Updated release notes. Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Cherry-picking commits from main to 1.x branch for 1.2 release (#142) * Fixes flaky test and removes local publishing of plugin dependencies (#135) * Fixes 'bucket level monitor can be created by extraction query' flaky test * Removes local publishing of plugin dependencies for github cypress tests Signed-off-by: Clay Downs <[email protected]> * Update copyright notice (#140) Signed-off-by: Mohammad Qureshi <[email protected]> * Added 1.2 release notes. (#141) * Added badges to the package README, and the Uploads coverage job to the unit tests workflow. * Removing code coverage upload token. * Added 1.2 release notes. Signed-off-by: Thomas Hurney <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Added 1.2 release notes. Signed-off-by: Thomas Hurney <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Updated 1.2 release notes. Signed-off-by: Thomas Hurney <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Added DCO section to CONTRIBUTING.md and GitHub workflows. Updated copyright notice. Signed-off-by: Thomas Hurney <[email protected]> (#145) Signed-off-by: AWSHurneyt <[email protected]> * support creating monitor for anomaly detector with custom result index (#143) (#147) * support creating monitor for anomaly detector with custom result index Signed-off-by: Yaliang Wu <[email protected]> * add release note Signed-off-by: Yaliang Wu <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Bumping version to 1.3. Signed-off-by: AWSHurneyt <[email protected]> (#159) Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * initial commit (#150) Signed-off-by: CEHENKLE <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add .whitesource configuration file (#153) Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Signed-off-by: AWSHurneyt <[email protected]> * Implemented a toast to display successful attempts to acknowledge alerts. Refactored alerts dashboard flyout to refresh its alerts table when alerts are acknowledged. (#160) * Implemented unit and integ tests for the alerts dashboard flyout. Refactored AlertsDashboardFlyoutComponent::getBucketLevelGraphConditions to return a string with line breaks instead of an array of HTML elements. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Removed an unused test variable. Signed-off-by: AWSHurneyt <[email protected]> * Removed debug logs. Signed-off-by: AWSHurneyt <[email protected]> * Implemented unit test. Refactored integration tests to use fewer wait periods. Signed-off-by: AWSHurneyt <[email protected]> * Examining flakiness in cypress test. Signed-off-by: AWSHurneyt <[email protected]> * Added short wait period to flyout cypress tests to alleviate flakiness. Signed-off-by: AWSHurneyt <[email protected]> * Refactored flyout cypress tests to use aliases. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Adding basic unit tests (#151) * Add flyout render Signed-off-by: Annie Lee <[email protected]> * Add some basic tests Signed-off-by: Annie Lee <[email protected]> * Add more tests Signed-off-by: Annie Lee <[email protected]> * Update jest.config.js Signed-off-by: Annie Lee <[email protected]> * Add tests Signed-off-by: Annie Lee <[email protected]> * Update dashboard sample alerttime Signed-off-by: Annie Lee <[email protected]> * Update dashboard test alert time snapshot Signed-off-by: Annie Lee <[email protected]> * Update Dashboard.test.js Signed-off-by: Annie Lee <[email protected]> * Remove test alert start time Signed-off-by: Annie Lee <[email protected]> * Update VisualGraph test and code Signed-off-by: Annie Lee <[email protected]> * Update package.json to run unit tests in UTC timezone Signed-off-by: Annie Lee <[email protected]> * Update dashboard sample alert start time Signed-off-by: Annie Lee <[email protected]> * Update Dashboard test Signed-off-by: Annie Lee <[email protected]> * Update Dashboard.test.js.snap Signed-off-by: Annie Lee <[email protected]> * Update snapshot file Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Fix the error handling when config index is not found (#173) Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Updated copyright notices and headers. (#168) * Updated CONTRIBUTING.md and NOTICE files. Signed-off-by: AWSHurneyt <[email protected]> * Updated license headers. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Refactored acknowledge alerts button on Alerts by trigger dashboard page to be a modal experience. (#167) * Refactored Acknowledge button on Alerts by trigger dashboard to display a modal. Signed-off-by: AWSHurneyt <[email protected]> * Updated license headers. Signed-off-by: AWSHurneyt <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Adding a few more basic unit tests (#180) * Add alerting dashboards flyout render unit test Signed-off-by: Annie Lee <[email protected]> * Add MonitorTimeFields validation tests Signed-off-by: Annie Lee <[email protected]> * Refactor error messages to add period Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Remove node version declaration in package.json (#166) * Remove node version declaration in package.json Signed-off-by: Annie Lee <[email protected]> * Remove react-router-dom version declaration Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Update DestinationsService.js (#182) Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add backport workflow (#176) * Add backport workflow Signed-off-by: Annie Lee <[email protected]> * Create delete_backport_branch.yml Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Configure test workflows to run on 1.x branches (#183) Signed-off-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add 1.3 release notes (#175) (#185) * Create opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <[email protected]> * Update opensearch-alerting-dashboards-plugin.release-notes-1.3.0.0.md Signed-off-by: Annie Lee <[email protected]> (cherry picked from commit 98f7904) Co-authored-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Add backport documentation link (#184) (#187) * Create BACKPORT.md Signed-off-by: Annie Lee <[email protected]> * Replace documentation with a link Signed-off-by: Annie Lee <[email protected]> (cherry picked from commit 0a0c8c5) Co-authored-by: Annie Lee <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Implemented support for cluster metrics monitors (#162) (#189) * Implemented LocalUriInput component, and supporting methods. Implemented unit and integration tests for LocalUriInput. Refactored various other classes and components to support LocalUriInput. Signed-off-by: AWSHurneyt <[email protected]> * Refactored learn more links to open new tabs. Signed-off-by: AWSHurneyt <[email protected]> * Implemented default trigger conditions for API supported by the LocalUriInput feature. Signed-off-by: AWSHurneyt <[email protected]> * Refactored button text and size based on UX reviewer feedback. Signed-off-by: AWSHurneyt <[email protected]> * Added documentation link for cluster stats API. Signed-off-by: AWSHurneyt <[email protected]> * Adjusted some wording based on UX reviewer feedback. Implemented modal that displays when changing request type. Signed-off-by: AWSHurneyt <[email protected]> * Refactored feature assets to use ClusterMetrics naming convention instead of LocalUriInput. Signed-off-by: AWSHurneyt <[email protected]> * Refactored behavior of cluster metrics clear triggers modal, and implemented tests. Signed-off-by: AWSHurneyt <[email protected]> * Added periods to error messages. Signed-off-by: AWSHurneyt <[email protected]> * Removed fixed width for clearTriggersModal. Signed-off-by: AWSHurneyt <[email protected]> * Refactored the cluster metrics radio card to be a monitor type instead of monitor definition type. Refactored request type selection combobox so it's not clearable, and reworded the clearTriggersModal text and refactored tests accordingly. Signed-off-by: AWSHurneyt <[email protected]> * Updated copyright headers. Signed-off-by: AWSHurneyt <[email protected]> * Refactoring monitorType checking logic to accommodate cluster metrics monitors. Signed-off-by: AWSHurneyt <[email protected]> * Refactored help text to use lowercase letters. Signed-off-by: AWSHurneyt <[email protected]> * Implemented a helper method as a temporary solution to some incorrect formatting of cluster metrics monitors returned by the getMonitors API. Signed-off-by: AWSHurneyt <[email protected]> * Refactored cluster metrics feature to remove support for CAT repositories. Signed-off-by: AWSHurneyt <[email protected]> * Refactored cluster metrics request types labels to no longer reference CAT. Signed-off-by: AWSHurneyt <[email protected]> * Adjusted the example trigger conditions for two request types. Signed-off-by: AWSHurneyt <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: Yaliang <[email protected]> Co-authored-by: CEHENKLE <[email protected]> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Annie Lee <[email protected]> Co-authored-by: Clay Downs <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: Yaliang <[email protected]> Co-authored-by: CEHENKLE <[email protected]> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com>

annie3431 requested review from qreshi and AWSHurneyt October 12, 2021 18:20

AWSHurneyt approved these changes Oct 12, 2021

View reviewed changes

annie3431 merged commit 0ba3b22 into opensearch-project:main Oct 12, 2021

annie3431 deleted the merge-to-main branch October 12, 2021 20:00

annie3431 mentioned this pull request Oct 12, 2021

Cherry-pick commits from main branch to 1.x branch #131

Merged

5 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fixed a bug that displayed all alerts for a monitor on individual tri… #130

Fixed a bug that displayed all alerts for a monitor on individual tri… #130

annie3431 commented Oct 12, 2021

codecov-commenter commented Oct 12, 2021

Fixed a bug that displayed all alerts for a monitor on individual tri… #130

Fixed a bug that displayed all alerts for a monitor on individual tri… #130

Conversation

annie3431 commented Oct 12, 2021

Description

Issues Resolved

Check List

codecov-commenter commented Oct 12, 2021

Codecov Report