Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove node version declaration in package.json #166

Merged
merged 2 commits into from
Mar 1, 2022

Conversation

annie3431
Copy link
Contributor

@annie3431 annie3431 commented Jan 20, 2022

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

Description

Remove explicit declaration in package.json file. This will inherit the node version from Opensearch-Dashboards core.

Issues Resolved

#161

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.

@codecov-commenter
Copy link

codecov-commenter commented Jan 20, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@d01afa1). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #166   +/-   ##
=======================================
  Coverage        ?   50.81%           
=======================================
  Files           ?      200           
  Lines           ?     4668           
  Branches        ?      594           
=======================================
  Hits            ?     2372           
  Misses          ?     2294           
  Partials        ?        2           

Continue to review full report at Codecov.

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

@annie3431 annie3431 marked this pull request as ready for review January 20, 2022 23:16
@annie3431 annie3431 requested a review from a team January 20, 2022 23:16
@annie3431 annie3431 added the dependencies Pull requests that update a dependency file label Jan 20, 2022
@annie3431 annie3431 merged commit 9435bb2 into opensearch-project:main Mar 1, 2022
@annie3431 annie3431 deleted the node-update branch March 1, 2022 23:18
annie3431 added a commit to annie3431/alerting-dashboards-plugin that referenced this pull request Mar 2, 2022
* 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]>
AWSHurneyt pushed a commit to AWSHurneyt/alerting-dashboards-plugin that referenced this pull request Mar 11, 2022
* 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]>
AWSHurneyt pushed a commit to AWSHurneyt/alerting-dashboards-plugin that referenced this pull request Mar 11, 2022
* 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]>
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
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants