Refactored acknowledge alerts button on Alerts by trigger dashboard page to be a modal experience. #167
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AWSHurneyt
force-pushed
the
main-alerts-modal
branch
from
8f874e8
to
b83d5e6
Compare
…ay a modal. Signed-off-by: AWSHurneyt <[email protected]>
AWSHurneyt
force-pushed
the
main-alerts-modal
branch
from
b83d5e6
to
c99b181
Compare
Signed-off-by: AWSHurneyt <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #167 +/- ##
==========================================
- Coverage 57.08% 56.01% -1.08%
==========================================
Files 194 195 +1
Lines 4395 4490 +95
Branches 591 599 +8
==========================================
+ Hits 2509 2515 +6
- Misses 1884 1973 +89
Partials 2 2
Continue to review full report at Codecov.
|
qreshi
approved these changes
Feb 15, 2022
annie3431
pushed a commit
to annie3431/alerting-dashboards-plugin
that referenced
this pull request
Feb 17, 2022
…age to be a modal experience. (opensearch-project#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]>
annie3431
pushed a commit
to annie3431/alerting-dashboards-plugin
that referenced
this pull request
Feb 17, 2022
…age to be a modal experience. (opensearch-project#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]>
annie3431
pushed a commit
to annie3431/alerting-dashboards-plugin
that referenced
this pull request
Feb 17, 2022
…age to be a modal experience. (opensearch-project#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]>
annie3431
pushed a commit
to annie3431/alerting-dashboards-plugin
that referenced
this pull request
Feb 17, 2022
…age to be a modal experience. (opensearch-project#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]>
AWSHurneyt
added a commit
to AWSHurneyt/alerting-dashboards-plugin
that referenced
this pull request
Mar 11, 2022
…age to be a modal experience. (opensearch-project#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]>
AWSHurneyt
added a commit
to AWSHurneyt/alerting-dashboards-plugin
that referenced
this pull request
Mar 11, 2022
…age to be a modal experience. (opensearch-project#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]>
AWSHurneyt
added a commit
that referenced
this pull request
Mar 11, 2022
* 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>
AWSHurneyt
added a commit
that referenced
this pull request
Mar 11, 2022
* 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>
AWSHurneyt
added a commit
that referenced
this pull request
Mar 11, 2022
* 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>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Refactored acknowledge alerts button on Alerts by trigger dashboard page to be a modal experience, and implemented unit and integration tests.
Issues Resolved
[List any issues this PR will resolve]
Screenshots
Check List
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.