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

AWSHurneyt · 2021-10-04T19:58:56Z

Signed off by Thomas Hurney [email protected]

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.

…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 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]>

* 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]>

…he 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 .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 .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]>

Signed-off-by: Annie Lee <[email protected]>

…gger 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.

Signed-off-by: Annie Lee <[email protected]>

…hboards-plugin into bug-fix

Signed-off-by: Annie Lee <[email protected]>

…hboards-plugin into bug-fix

…hboards-plugin into bug-fix Signed-off-by: Annie Lee <[email protected]>

…hboards-plugin into bug-fix

…plugin into bugFixes

…1. Removed experimental code and comments.

…us commit.

…rror messages that prevented preview graphs from displaying data.

qreshi · 2021-10-04T20:53:19Z

public/pages/Dashboard/containers/Dashboard.js

@@ -459,7 +475,7 @@ export default class Dashboard extends Component {
    };

    const getItemId = (item) => {
-      if (perAlertView && isBucketMonitor) return item.id;
+      if (perAlertView) return isBucketMonitor ? item.id : `${item.id}-${item.version}`;


Just to confirm, since we're using ${item.id}-${item.version} so that selected items are updated upon acknowledgement and we don't do that when isBucketMonitor, does that mean the selected items won't be updated if the user acknowledged from the Monitor details page?

I know the flyout doesn't update without refresh for now, just wondering if that applies to the Monitor details table as well.

It does update on the monitor details page. I think the difference is whether we add another call of getAlerts after acknowledging. Right here adding the version to item id is to support selecting single alert items, since they may share the same id but different version.

Got it, since I believe the Monitor details page shows alerts as "per alert view" for Bucket-level Monitors as well, it could mean that acknowledging Alerts one by one there won't update them since we only use item.id in that scenario.

I think it's fine for now but if that is the case, let's capture it in the same issue we'll be capturing the flyout not updating on acknowledgement.

No problem, thanks!

public/pages/CreateMonitor/components/VisualGraph/utils/helpers.js

…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]>

…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. (#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]>

…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 and others added 30 commits September 7, 2021 17:45

Added badges to the package README, and the Uploads coverage job to t…

588114c

…he 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.

Fixed a few bugs

0ba0ece

Merge remote-tracking branch 'upstream/main' into bug-fix

65eed16

Signed-off-by: Annie Lee <[email protected]>

Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md

3473d51

Signed-off-by: Annie Lee <[email protected]>

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

ebda05c

…gger 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.

77a8cd1

Fixing number of alerts displayed on Monitors tab.

6c00fb5

Merge remote-tracking branch 'annie/bug-fix' into alertFlyoutBugFix

4a2a6d8

Update opensearch-alerting-dashboards-plugin.release-notes-1.1.0.0.md

0add9a2

Signed-off-by: Annie Lee <[email protected]>

Merge branch 'bug-fix' of https://github.com/leeyun-amzn/alerting-das…

b790ddc

…hboards-plugin into bug-fix

More bug fix

9ac8b2d

Signed-off-by: Annie Lee <[email protected]>

Skip test based on modification

c865807

Signed-off-by: Annie Lee <[email protected]>

Skip test based on modification

1ae2215

Signed-off-by: Annie Lee <[email protected]>

Update popover windows to remove item when filed is not defined

d2058b1

Merge branch 'bug-fix' of https://github.com/leeyun-amzn/alerting-das…

6eb7379

…hboards-plugin into bug-fix

Merge branch 'bug-fix' of https://github.com/leeyun-amzn/alerting-das…

0e84b73

…hboards-plugin into bug-fix Signed-off-by: Annie Lee <[email protected]>

Update field validation

756793c

Merge branch 'bug-fix' of https://github.com/leeyun-amzn/alerting-das…

c69a674

…hboards-plugin into bug-fix

Merge branch 'bug-fix' of github.com:leeyun-amzn/alerting-dashboards-…

017cf70

…plugin into bugFixes

Fixed a bug that prevented selecting query-level monitor alerts 1 by …

559fb68

…1. Removed experimental code and comments.

Merge remote-tracking branch 'thomas/alertFlyoutBugFix' into bug-fix

c0d3417

Update Dashboard.js

c4171e1

Support data filter when using null operator

794be0b

Update WhereExpression.js

78916c2

Removing experimental dev code that was accidentally left in a previo…

93cebca

…us commit.

AWSHurneyt and others added 6 commits October 4, 2021 10:36

Removed redundant validation from filter values that was generating e…

de67be8

…rror messages that prevented preview graphs from displaying data.

Removed redundant validation from filter values that was generating e…

c6fb682

…rror messages that prevented preview graphs from displaying data.

Merge remote-tracking branch 'origin/bugFixes' into bugFixes

5919589

Update metric error for query monitors

49348e7

Update MetricExpression.js

ddfc601

Removed experimental dev code.

dc37a6f

AWSHurneyt requested review from lezzago, qreshi, bowenlan-amzn and annie3431 October 4, 2021 19:58

Updated release notes.

4fadf64

AWSHurneyt marked this pull request as ready for review October 4, 2021 20:46

qreshi reviewed Oct 4, 2021

View reviewed changes

lezzago reviewed Oct 4, 2021

View reviewed changes

public/pages/CreateMonitor/components/VisualGraph/utils/helpers.js Show resolved Hide resolved

qreshi approved these changes Oct 4, 2021

View reviewed changes

annie3431 approved these changes Oct 4, 2021

View reviewed changes

lezzago approved these changes Oct 4, 2021

View reviewed changes

AWSHurneyt merged commit ff9edaa into opensearch-project:1.1 Oct 4, 2021

annie3431 mentioned this pull request Oct 12, 2021

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

Merged

5 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

AWSHurneyt commented Oct 4, 2021 •

edited

Loading

qreshi Oct 4, 2021

annie3431 Oct 4, 2021

qreshi Oct 4, 2021

annie3431 Oct 4, 2021

Conversation

AWSHurneyt commented Oct 4, 2021 • edited Loading

Description

Issues Resolved

Check List

qreshi Oct 4, 2021

Choose a reason for hiding this comment

annie3431 Oct 4, 2021

Choose a reason for hiding this comment

qreshi Oct 4, 2021

Choose a reason for hiding this comment

annie3431 Oct 4, 2021

Choose a reason for hiding this comment

AWSHurneyt commented Oct 4, 2021 •

edited

Loading