Incremented version to 2.0.1 #269
Merged
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
Signed-off-by: Zelin Hao <[email protected]>
Codecov Report
@@ Coverage Diff @@
## 2.0 #269 +/- ##
=======================================
Coverage 52.78% 52.78%
=======================================
Files 209 209
Lines 5445 5445
Branches 762 762
=======================================
Hits 2874 2874
Misses 2569 2569
Partials 2 2 Continue to review full report at Codecov.
|
AWSHurneyt
approved these changes
Jun 10, 2022
lezzago
approved these changes
Jun 10, 2022
AWSHurneyt
pushed a commit
to AWSHurneyt/alerting-dashboards-plugin
that referenced
this pull request
Jun 10, 2022
Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: AWSHurneyt <[email protected]>
AWSHurneyt
pushed a commit
to AWSHurneyt/alerting-dashboards-plugin
that referenced
this pull request
Jun 10, 2022
Signed-off-by: Zelin Hao <[email protected]>
AWSHurneyt
pushed a commit
to AWSHurneyt/alerting-dashboards-plugin
that referenced
this pull request
Jun 11, 2022
Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: AWSHurneyt <[email protected]>
AWSHurneyt
added a commit
that referenced
this pull request
Jun 15, 2022
* Incremented version to 2.0.1 (#269) Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: AWSHurneyt <[email protected]> * Drafted 2.0.1 patch notes. (#265) Signed-off-by: AWSHurneyt <[email protected]> Co-authored-by: Zelin Hao <[email protected]>
bowenlan-amzn
pushed a commit
to bowenlan-amzn/alerting-dashboards-plugin
that referenced
this pull request
Jan 23, 2023
…ch-project#73) * Update .opensearch_dashboards-plugin-helpers.json * Add MonitorDetails and DataSource components Signed-off-by: Annie Lee <[email protected]> * Update MonitorExpressions to support new visual query builder Signed-off-by: Annie Lee <[email protected]> * Alert dashboard table column update (opensearch-project#36) * Add new panel monitor details * Add schedule to monitor details panel * Update Frequency.js * Update .opensearch_dashboards-plugin-helpers.json * Update alert dashboard columns * Parsing alerts by triggers data fields to table * Update tableUtils.js * Update Dashboard.js * Update Dashboard.js * Clean up code * Update MultipleExpressions.js * Update MultipleExpressions.js * Update MultipleExpressions.js * Add license header Signed-off-by: Annie Lee <[email protected]> * Update helpers.js * Alert dashboard update on monitor (opensearch-project#38) Alert dashboard update on monitor Signed-off-by: Annie Lee <[email protected]> * Create monitor page, bucket level monitor showing bar graph (opensearch-project#39) * Add new panel monitor details * Add schedule to monitor details panel * Update Frequency.js * Updates to MonitorDetails and DataSource component * Update CreateMonitor.js * Update CreateMonitor.js * Add Query component * Update Query.js * Update Query.js * Add DefineVisualGraph * Added onChange function to Monitor definition checkCard group * Able to select source index and query time field * Update padding * Implemented support for rendering multiple trigger conditions for aggregation monitors. (opensearch-project#257) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. Co-authored-by: Annie <[email protected]> * Support aggregation monitor creation * Add formik to Aggregation Trigger conversion * Minor improvements to the full flow of creating of an Aggregation Monitor * Implemented triggerToFormik logic for aggregation triggers (opensearch-project#258) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. Co-authored-by: Annie <[email protected]> * Fix Triggers selection on Monitor Details page and some minor issues with editing Aggregation Trigger * Add group by validation * Check group by when monitor type is aggregation * Fix triggerToFormik for Aggregation Trigger * Show aggregation alert content on alert dashboards * Refactored support for extraction queries for aggregation triggers (opensearch-project#259) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. * Refactored AggregationTriggerQuery to support 1 definition box. Refactored AggregationTriggerQuery, formikToTrigger, and triggerToFormik to query input in the bucketSelector field of the trigger. * Relabeled code editor component. Co-authored-by: Annie <[email protected]> * Added support for LocalUriInput monitors (opensearch-project#255) * Implemented support for dropdown options * Implemented LocalUriInput monitor creation via path entry field * Implemented frontend tests * Refactored default LocalUriInput host value to localhost * Updated copyright year in modified files, removed some duplicated comments, and refactored help text based on PR feedback. * Implemented support for dropdown options * Implemented LocalUriInput monitor creation via path entry field * Implemented frontend tests * Refactored default LocalUriInput host value to localhost * Updated copyright year in modified files, removed some duplicated comments, and refactored help text based on PR feedback. * Merged LocalUriInput frontend with current state of Doc-level alerting frontend. * Added limit to the number of trigger conditions that can be added to a visual graph aggregation trigger, and fixed a bug that prevented editing triggers. (opensearch-project#260) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. * Refactored AggregationTriggerQuery to support 1 definition box. Refactored AggregationTriggerQuery, formikToTrigger, and triggerToFormik to query input in the bucketSelector field of the trigger. * Relabeled code editor component. * Added limit to the number of trigger conditions that can be added to a visual graph aggregation trigger. Fix a bug that prevented editing triggers. * Fixed a bug that prevented the default trigger condition from populating for traditional extraction query triggers. Co-authored-by: Annie <[email protected]> * Fix getting inner contents of Trigger objects when creating and deleting Triggers * Modify create monitor page UX (opensearch-project#263) * Fixed an issue preventing emails from being added to an email group using Kibana interface. (opensearch-project#252) * Added a lastUpdatedBy column to the Destinations tab (opensearch-project#251) * Added a lastUpdatedBy column to the Destinations tab * Refactored Monitors and Destinations tabs to display - instead of N/A based on UX reviewer's feedback * Query component appearance * Update arrangement of data source and query component to fit inside DefineMonitor Co-authored-by: AWSHurneyt <[email protected]> * Update DefineMonitor to show query graph * Fixed bug that was preventing popover values from saving. (opensearch-project#261) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. * Refactored AggregationTriggerQuery to support 1 definition box. Refactored AggregationTriggerQuery, formikToTrigger, and triggerToFormik to query input in the bucketSelector field of the trigger. * Relabeled code editor component. * Added limit to the number of trigger conditions that can be added to a visual graph aggregation trigger. Fix a bug that prevented editing triggers. * Fixed a bug that prevented the default trigger condition from populating for traditional extraction query triggers. * Fixed an error in switch statement syntax. * Implemented default values logic for visual graph aggregation triggers to prevent popover fields from saving as undefiend. * Implemented support for one WHERE filter when defining a trigger. Co-authored-by: Annie <[email protected]> * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. (opensearch-project#265) * Squashed commit of the following: commit 25fa4454b874de5c6f2d30325b7395fb38c679ff Author: Thomas Hurney <[email protected]> Date: Mon Jun 21 18:57:23 2021 -0700 Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. commit c1b6bc386a491a01da0b456ec24fff3ff62e0c47 Author: Thomas Hurney <[email protected]> Date: Wed Jun 16 21:32:55 2021 -0700 Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. (opensearch-project#266) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and Clean up code * Remove unused files * Update DefineMonitor.js * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Use composite aggregation to build execute API request * Aggregation monitor sending correct request format * Add todo items for visual graphs * Update MetricExpression.js * Update GroupByExpression.js * Refactored traditional trigger assets to support rendering multiple traditional triggers in one panel. Refactored actions panel to align with mocks and support action execution policies for aggregation triggers.. (opensearch-project#269) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Refactored traditional trigger assets to support rendering multiple traditional triggers using ConfigureTriggers. Refactored AggregationTriggerQuery to support rendering multiple extraction query monitor triggers using ConfigureTriggers. Refactored monitor_type and trigger_type checks to check against constants. * Refactored DefineTrigger to be a class that extends Component. Fixed the issue that was causing all trigger condition response windows to display the same response for all of the triggers defined for a traditional extraction query monitor. * Fixed a bug in AggregationTriggerGraph that was preventing unique values from being stored for separate trigger conditions. Fixed a bug in CreateMultipleTriggers that was preventing the dataTypes object from populating. * Fixed a bug that was preventing ConfigureActions::sendTestMessage from sending test notifications. * Refactored actions panel to align with mocks. Implemented support for action execution policies for aggregation triggers. * Refactored EuiRadioGroup to EuiSelect. Co-authored-by: Annie Lee <[email protected]> * Update helpers.js * Showing individual graph title and data * Update VisualGraph.js * Add grouped data * Update y-axis title * Clean up code * Bar graph showing up * Showing graph with x-axis buffer * Refactored CreateMonitor to support defining a monitor, its triggers, and each trigger's actions on one page. (opensearch-project#272) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Refactored traditional trigger assets to support rendering multiple traditional triggers using ConfigureTriggers. Refactored AggregationTriggerQuery to support rendering multiple extraction query monitor triggers using ConfigureTriggers. Refactored monitor_type and trigger_type checks to check against constants. * Refactored DefineTrigger to be a class that extends Component. Fixed the issue that was causing all trigger condition response windows to display the same response for all of the triggers defined for a traditional extraction query monitor. * Fixed a bug in AggregationTriggerGraph that was preventing unique values from being stored for separate trigger conditions. Fixed a bug in CreateMultipleTriggers that was preventing the dataTypes object from populating. * Fixed a bug that was preventing ConfigureActions::sendTestMessage from sending test notifications. * Refactored actions panel to align with mocks. Implemented support for action execution policies for aggregation triggers. * Refactored EuiRadioGroup to EuiSelect. * Fixed a bug that was causing the button text to always reference adding ANOTHER action, even when there were 0 actions. * Implemented separate default message sources for traditional and aggregation monitors. * Refactored Triggers table to no longer display check boxes, edit, delete, or create buttons to align with UX mocks. * Refactored CreateMonitor page to support defining a monitor, its triggers, and each trigger's actions all on 1 page. * Refactored an unnecessary LET to CONST. Co-authored-by: Annie Lee <[email protected]> * Showing multiple data series * Separating bar graphs for better visualization * Show graph hint * Update graph hint * Add legend to graph * Clean up code * Update VisualGraph.js * Clean up code * Refactored extraction query trigger definition components and actions panel based on mocks (opensearch-project#274) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Refactored traditional trigger assets to support rendering multiple traditional triggers using ConfigureTriggers. Refactored AggregationTriggerQuery to support rendering multiple extraction query monitor triggers using ConfigureTriggers. Refactored monitor_type and trigger_type checks to check against constants. * Refactored DefineTrigger to be a class that extends Component. Fixed the issue that was causing all trigger condition response windows to display the same response for all of the triggers defined for a traditional extraction query monitor. * Fixed a bug in AggregationTriggerGraph that was preventing unique values from being stored for separate trigger conditions. Fixed a bug in CreateMultipleTriggers that was preventing the dataTypes object from populating. * Fixed a bug that was preventing ConfigureActions::sendTestMessage from sending test notifications. * Refactored actions panel to align with mocks. Implemented support for action execution policies for aggregation triggers. * Refactored EuiRadioGroup to EuiSelect. * Fixed a bug that was causing the button text to always reference adding ANOTHER action, even when there were 0 actions. * Implemented separate default message sources for traditional and aggregation monitors. * Refactored Triggers table to no longer display check boxes, edit, delete, or create buttons to align with UX mocks. * Refactored CreateMonitor page to support defining a monitor, its triggers, and each trigger's actions all on 1 page. * Refactored an unnecessary LET to CONST. * Refactored EuiSelect for Notify options to FormikFieldRadio to align with mocks. Implemented default selections for actionable alerts. Renamed constants. * Refactored WhereExpression header based on UX reviewer feedback. * Fixed a bug that was preventing the WHERE expressions from being cleared from trigger definitions. * Refactored TriggerQuery and AggregationTriggerQuery based on mocks. Made minor edits to various classes to align with mocks. * Refactored actions panel to align with mocks. Made minor edits to various classes to align with mocks. Co-authored-by: Annie Lee <[email protected]> * Renaming for Bucket-Level Alerting * Remove support for LocalURI input * Clean up code * Clean up code * Update MonitorOverview.test.js.snap * Update AnomalyDetector.test.js.snap * Update DestinationsList.test.js.snap * Update formikToMonitor.js * Update monitor type naming Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Signed-off-by: Annie Lee <[email protected]> * Update helpers.js Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: AWSHurneyt <[email protected]>
bowenlan-amzn
pushed a commit
to bowenlan-amzn/alerting-dashboards-plugin
that referenced
this pull request
Jan 23, 2023
* Update .opensearch_dashboards-plugin-helpers.json * Add MonitorDetails and DataSource components Signed-off-by: Annie Lee <[email protected]> * Update MonitorExpressions to support new visual query builder Signed-off-by: Annie Lee <[email protected]> * Alert dashboard table column update (opensearch-project#36) * Add new panel monitor details * Add schedule to monitor details panel * Update Frequency.js * Update .opensearch_dashboards-plugin-helpers.json * Update alert dashboard columns * Parsing alerts by triggers data fields to table * Update tableUtils.js * Update Dashboard.js * Update Dashboard.js * Clean up code * Update MultipleExpressions.js * Update MultipleExpressions.js * Update MultipleExpressions.js * Add license header Signed-off-by: Annie Lee <[email protected]> * Update helpers.js * Alert dashboard update on monitor (opensearch-project#38) Alert dashboard update on monitor Signed-off-by: Annie Lee <[email protected]> * Create monitor page, bucket level monitor showing bar graph (opensearch-project#39) * Add new panel monitor details * Add schedule to monitor details panel * Update Frequency.js * Updates to MonitorDetails and DataSource component * Update CreateMonitor.js * Update CreateMonitor.js * Add Query component * Update Query.js * Update Query.js * Add DefineVisualGraph * Added onChange function to Monitor definition checkCard group * Able to select source index and query time field * Update padding * Implemented support for rendering multiple trigger conditions for aggregation monitors. (opensearch-project#257) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. Co-authored-by: Annie <[email protected]> * Support aggregation monitor creation * Add formik to Aggregation Trigger conversion * Minor improvements to the full flow of creating of an Aggregation Monitor * Implemented triggerToFormik logic for aggregation triggers (opensearch-project#258) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. Co-authored-by: Annie <[email protected]> * Fix Triggers selection on Monitor Details page and some minor issues with editing Aggregation Trigger * Add group by validation * Check group by when monitor type is aggregation * Fix triggerToFormik for Aggregation Trigger * Show aggregation alert content on alert dashboards * Refactored support for extraction queries for aggregation triggers (opensearch-project#259) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. * Refactored AggregationTriggerQuery to support 1 definition box. Refactored AggregationTriggerQuery, formikToTrigger, and triggerToFormik to query input in the bucketSelector field of the trigger. * Relabeled code editor component. Co-authored-by: Annie <[email protected]> * Added support for LocalUriInput monitors (opensearch-project#255) * Implemented support for dropdown options * Implemented LocalUriInput monitor creation via path entry field * Implemented frontend tests * Refactored default LocalUriInput host value to localhost * Updated copyright year in modified files, removed some duplicated comments, and refactored help text based on PR feedback. * Implemented support for dropdown options * Implemented LocalUriInput monitor creation via path entry field * Implemented frontend tests * Refactored default LocalUriInput host value to localhost * Updated copyright year in modified files, removed some duplicated comments, and refactored help text based on PR feedback. * Merged LocalUriInput frontend with current state of Doc-level alerting frontend. * Added limit to the number of trigger conditions that can be added to a visual graph aggregation trigger, and fixed a bug that prevented editing triggers. (opensearch-project#260) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. * Refactored AggregationTriggerQuery to support 1 definition box. Refactored AggregationTriggerQuery, formikToTrigger, and triggerToFormik to query input in the bucketSelector field of the trigger. * Relabeled code editor component. * Added limit to the number of trigger conditions that can be added to a visual graph aggregation trigger. Fix a bug that prevented editing triggers. * Fixed a bug that prevented the default trigger condition from populating for traditional extraction query triggers. Co-authored-by: Annie <[email protected]> * Fix getting inner contents of Trigger objects when creating and deleting Triggers * Modify create monitor page UX (opensearch-project#263) * Fixed an issue preventing emails from being added to an email group using Kibana interface. (opensearch-project#252) * Added a lastUpdatedBy column to the Destinations tab (opensearch-project#251) * Added a lastUpdatedBy column to the Destinations tab * Refactored Monitors and Destinations tabs to display - instead of N/A based on UX reviewer's feedback * Query component appearance * Update arrangement of data source and query component to fit inside DefineMonitor Co-authored-by: AWSHurneyt <[email protected]> * Update DefineMonitor to show query graph * Fixed bug that was preventing popover values from saving. (opensearch-project#261) * Add addMetric button * Update MonitorExpressions.js * Add metrics and for the last dummy components * Dummy components * Implemented support for rendering multiple trigger conditions in a single trigger. * Showing new metric popover * Implemented support for rendering multiple trigger conditions in a single trigger. * Badge showing up * Implementing support for rendering multiple extraction query trigger conditions. * Removed logic to support rendering multiple trigger conditions for anomaly detector monitors. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented support for rendering multiple trigger conditions in a single trigger. * Implemented support for rendering multiple trigger conditions in a single trigger. * Separated trigger definition assets for traditional and aggregation monitors. Implemented logic to render AND/OR options for aggregation trigger conditions. Implemented logic to render aggregation monitor details on the MonitorDetails page. * Implemented triggerToFormik logic for aggregation triggers. * Refactored AggregationTriggerQuery to support 1 definition box. Refactored AggregationTriggerQuery, formikToTrigger, and triggerToFormik to query input in the bucketSelector field of the trigger. * Relabeled code editor component. * Added limit to the number of trigger conditions that can be added to a visual graph aggregation trigger. Fix a bug that prevented editing triggers. * Fixed a bug that prevented the default trigger condition from populating for traditional extraction query triggers. * Fixed an error in switch statement syntax. * Implemented default values logic for visual graph aggregation triggers to prevent popover fields from saving as undefiend. * Implemented support for one WHERE filter when defining a trigger. Co-authored-by: Annie <[email protected]> * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. (opensearch-project#265) * Squashed commit of the following: commit 25fa4454b874de5c6f2d30325b7395fb38c679ff Author: Thomas Hurney <[email protected]> Date: Mon Jun 21 18:57:23 2021 -0700 Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. commit c1b6bc386a491a01da0b456ec24fff3ff62e0c47 Author: Thomas Hurney <[email protected]> Date: Wed Jun 16 21:32:55 2021 -0700 Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. (opensearch-project#266) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and Clean up code * Remove unused files * Update DefineMonitor.js * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Use composite aggregation to build execute API request * Aggregation monitor sending correct request format * Add todo items for visual graphs * Update MetricExpression.js * Update GroupByExpression.js * Refactored traditional trigger assets to support rendering multiple traditional triggers in one panel. Refactored actions panel to align with mocks and support action execution policies for aggregation triggers.. (opensearch-project#269) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Refactored traditional trigger assets to support rendering multiple traditional triggers using ConfigureTriggers. Refactored AggregationTriggerQuery to support rendering multiple extraction query monitor triggers using ConfigureTriggers. Refactored monitor_type and trigger_type checks to check against constants. * Refactored DefineTrigger to be a class that extends Component. Fixed the issue that was causing all trigger condition response windows to display the same response for all of the triggers defined for a traditional extraction query monitor. * Fixed a bug in AggregationTriggerGraph that was preventing unique values from being stored for separate trigger conditions. Fixed a bug in CreateMultipleTriggers that was preventing the dataTypes object from populating. * Fixed a bug that was preventing ConfigureActions::sendTestMessage from sending test notifications. * Refactored actions panel to align with mocks. Implemented support for action execution policies for aggregation triggers. * Refactored EuiRadioGroup to EuiSelect. Co-authored-by: Annie Lee <[email protected]> * Update helpers.js * Showing individual graph title and data * Update VisualGraph.js * Add grouped data * Update y-axis title * Clean up code * Bar graph showing up * Showing graph with x-axis buffer * Refactored CreateMonitor to support defining a monitor, its triggers, and each trigger's actions on one page. (opensearch-project#272) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Refactored traditional trigger assets to support rendering multiple traditional triggers using ConfigureTriggers. Refactored AggregationTriggerQuery to support rendering multiple extraction query monitor triggers using ConfigureTriggers. Refactored monitor_type and trigger_type checks to check against constants. * Refactored DefineTrigger to be a class that extends Component. Fixed the issue that was causing all trigger condition response windows to display the same response for all of the triggers defined for a traditional extraction query monitor. * Fixed a bug in AggregationTriggerGraph that was preventing unique values from being stored for separate trigger conditions. Fixed a bug in CreateMultipleTriggers that was preventing the dataTypes object from populating. * Fixed a bug that was preventing ConfigureActions::sendTestMessage from sending test notifications. * Refactored actions panel to align with mocks. Implemented support for action execution policies for aggregation triggers. * Refactored EuiRadioGroup to EuiSelect. * Fixed a bug that was causing the button text to always reference adding ANOTHER action, even when there were 0 actions. * Implemented separate default message sources for traditional and aggregation monitors. * Refactored Triggers table to no longer display check boxes, edit, delete, or create buttons to align with UX mocks. * Refactored CreateMonitor page to support defining a monitor, its triggers, and each trigger's actions all on 1 page. * Refactored an unnecessary LET to CONST. Co-authored-by: Annie Lee <[email protected]> * Showing multiple data series * Separating bar graphs for better visualization * Show graph hint * Update graph hint * Add legend to graph * Clean up code * Update VisualGraph.js * Clean up code * Refactored extraction query trigger definition components and actions panel based on mocks (opensearch-project#274) * Commented out a line of code that was preventing trigger WHERE expression popovers from closing, and refactored formikToTrigger to leave the composite_agg_filter undefined if the user doesn't make any selections instead of returning an empty object. * Implemented CreateMultipleTriggers, and refactored several files to support creating multiple triggers on one page. * Modify create monitor page (opensearch-project#267) Modify the components on create monitor page. Co-authored-by: AWSHurneyt <[email protected]> * Update data source component and clean up code (opensearch-project#268) * Update data source component and clean up code * Refactored traditional trigger assets to support rendering multiple traditional triggers using ConfigureTriggers. Refactored AggregationTriggerQuery to support rendering multiple extraction query monitor triggers using ConfigureTriggers. Refactored monitor_type and trigger_type checks to check against constants. * Refactored DefineTrigger to be a class that extends Component. Fixed the issue that was causing all trigger condition response windows to display the same response for all of the triggers defined for a traditional extraction query monitor. * Fixed a bug in AggregationTriggerGraph that was preventing unique values from being stored for separate trigger conditions. Fixed a bug in CreateMultipleTriggers that was preventing the dataTypes object from populating. * Fixed a bug that was preventing ConfigureActions::sendTestMessage from sending test notifications. * Refactored actions panel to align with mocks. Implemented support for action execution policies for aggregation triggers. * Refactored EuiRadioGroup to EuiSelect. * Fixed a bug that was causing the button text to always reference adding ANOTHER action, even when there were 0 actions. * Implemented separate default message sources for traditional and aggregation monitors. * Refactored Triggers table to no longer display check boxes, edit, delete, or create buttons to align with UX mocks. * Refactored CreateMonitor page to support defining a monitor, its triggers, and each trigger's actions all on 1 page. * Refactored an unnecessary LET to CONST. * Refactored EuiSelect for Notify options to FormikFieldRadio to align with mocks. Implemented default selections for actionable alerts. Renamed constants. * Refactored WhereExpression header based on UX reviewer feedback. * Fixed a bug that was preventing the WHERE expressions from being cleared from trigger definitions. * Refactored TriggerQuery and AggregationTriggerQuery based on mocks. Made minor edits to various classes to align with mocks. * Refactored actions panel to align with mocks. Made minor edits to various classes to align with mocks. Co-authored-by: Annie Lee <[email protected]> * Renaming for Bucket-Level Alerting * Remove support for LocalURI input * Clean up code * Clean up code * Update MonitorOverview.test.js.snap * Update AnomalyDetector.test.js.snap * Update DestinationsList.test.js.snap * Update formikToMonitor.js * Update monitor type naming Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Signed-off-by: Annie Lee <[email protected]> * Update helpers.js * Bucket level alerting dev trigger components (opensearch-project#40) * Refactored MonitorType such that the AD definition option cannot be selected for aggregation monitors. Also implemented logic to clear the form when changing from an unsupported definition option to aggregation monitor type. Reimplemented the query response display in the trigger panel when defining an AD trigger condition using the query response option. Also made minor edits to various other classes to align UX with mocks. * Refactored actions panel to align with new mocks. Disabled throttling when per execution is selected in the action configuration. * Refactored the visual editor for bucket-level triggers to align with new mocks. * Refactored the visual and response editor options for query-level AD triggers to align with new mocks. Refactored form reset logic to remove trigger definitions from the form when changing the monitor type. * Implemented logic to set bucket-level monitor search type to a default value when changing from a query-level monitor with an unsupported search type. * Refactored monitor details panel on the monitor creation/edit page to align with mocks. Signed-off-by: Annie Lee <[email protected]> Co-authored-by: AWSHurneyt <[email protected]> Co-authored-by: Mohammad Qureshi <[email protected]> Co-authored-by: AWSHurneyt <[email protected]>
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.
Signed-off-by: Zelin Hao [email protected]
Description
Incremented version to 2.0.1
Issues Resolved
Part of opensearch-project/opensearch-build#2165
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.