Connect Notifications APIs to Reporting #131

davidcui1225 · 2021-07-21T01:25:48Z

Description

Connect Notifications APIs to Reporting

Channel display in Notification Settings will display available channels with reporting listed
Send test message connects to send a test message to the channels listed
Details pages updated to display correct channels after getting list of available channels
Add needed Notifications APIs implemented in dashboards-reports server to eliminate dependency on dashboards-notifications and only have dependency on opensearch-notifications
Check _cat/plugins to see if opensearch-notifications is installed to toggle whether the Notification settings section is displayed
Jest tests updated
Changed Report Settings to Report settings

Issues Resolved

[List any issues this PR will resolve]

Check List

New functionality includes testing.
- All tests pass, including unit test, integration test and doctest
New functionality has been documented.
- New functionality has javadoc added
- New functionality has user manual doc 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.

Signed-off-by: David Cui <[email protected]>

…e tests Signed-off-by: David Cui <[email protected]>

Signed-off-by: David Cui <[email protected]>

codecov-commenter · 2021-07-21T01:35:01Z

Codecov Report

Merging #131 (65c3871) into main (a031386) will decrease coverage by 0.52%.
The diff coverage is 40.98%.

@@             Coverage Diff              @@
##               main     #131      +/-   ##
============================================
- Coverage     62.94%   62.42%   -0.53%     
  Complexity      309      309              
============================================
  Files           100      101       +1     
  Lines          4391     4484      +93     
  Branches        673      681       +8     
============================================
+ Hits           2764     2799      +35     
- Misses         1459     1517      +58     
  Partials        168      168

Flag	Coverage Δ
dashboards-reports	`73.30% <40.98%> (-1.72%)`	⬇️
reports-scheduler	`53.42% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...rt_definitions/report_settings/report_settings.tsx	`81.97% <ø> (ø)`
dashboards-reports/server/routes/utils/helpers.ts	`44.44% <20.00%> (-5.56%)`	⬇️
...omponents/report_definitions/delivery/delivery.tsx	`31.19% <21.05%> (-20.60%)`	⬇️
...ards-reports/public/components/main/main_utils.tsx	`70.79% <41.66%> (-1.59%)`	⬇️
...report_definitions/delivery/delivery_constants.tsx	`80.00% <75.00%> (-20.00%)`	⬇️
dashboards-reports/common/index.ts	`100.00% <100.00%> (ø)`
...t_definition_details/report_definition_details.tsx	`72.47% <100.00%> (+1.04%)`	⬆️
.../components/main/report_details/report_details.tsx	`76.14% <100.00%> (+0.41%)`	⬆️
dashboards-reports/test/httpMockClient.js	`81.81% <100.00%> (+1.81%)`	⬆️
... and 1 more

Continue to review full report at Codecov.

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

joshuali925

a few questions

dashboards-reports/public/components/report_definitions/delivery/delivery_constants.tsx

dashboards-reports/public/components/main/main_utils.tsx

dashboards-reports/public/components/report_definitions/delivery/delivery.tsx

dashboards-reports/public/components/report_definitions/delivery/delivery_constants.tsx

…to reduce dependency Signed-off-by: David Cui <[email protected]>

…throw error if not Signed-off-by: David Cui <[email protected]>

zhongnansu

two general question

Is there basic input validation logic added for delivery settings in report definition?
Is there logic to check and handle if notification plugin is absent? What's the UX if notifiation is not installed in the cluster? Do we show the notification settings still, or grey them out with pop-up messages

zhongnansu · 2021-07-27T17:50:16Z

...oards-reports/public/components/main/report_definition_details/report_definition_details.tsx

@@ -62,7 +63,7 @@ import {
  permissionsMissingActions,
 } from '../../utils/utils';
 import { GenerateReportLoadingModal } from '../loading_modal';
-import { placeholderChannels } from '../../report_definitions/delivery/delivery_constants';
+import { getChannelsQueryObject } from '../../report_definitions/delivery/delivery_constants';

 const ON_DEMAND = 'On demand';


Np: move constancts like this to a common place for easy reuse. e.g. constants.ts, or /common/index.ts

dashboards-reports/public/components/report_definitions/delivery/delivery.tsx

dashboards-reports/server/routes/notifications.ts

davidcui1225 · 2021-07-27T18:57:30Z

two general question

1. Is there basic input validation logic added for delivery settings in report definition?

2. Is there logic to check and handle if notification plugin is absent? What's the UX if notifiation is not installed in the cluster? Do we show the notification settings still, or grey them out with pop-up messages

Yes, we have input validation logic for delivery.
Good catch, I guess we should only show Notifications settings when opensearch-notifications is installed. Will add a check, should be a simple addition.

…ions section if opensearch notifications is not installed Signed-off-by: David Cui <[email protected]>

Signed-off-by: David Cui <[email protected]>

dashboards-reports/public/components/report_definitions/delivery/delivery.tsx

dashboards-reports/common/index.ts

Signed-off-by: David Cui <[email protected]>

@akbhatta

* Fix release notes version (#28) Signed-off-by: Zhongnan Su <[email protected]> * Migrate Reporting Notebooks Integration (#21) Signed-off-by: David Cui <[email protected]> * Add Logic to Auto-populate Notebooks from Context Menu (#30) Signed-off-by: David Cui <[email protected]> * Change opendistro to opensearch in email template (#31) Signed-off-by: Joshua Li <[email protected]> * Fix case sensitive directory name for chromium zip (#35) Signed-off-by: David Cui <[email protected]> Co-authored-by: Carl Meadows <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: Joshua Li <[email protected]> * Use output_only Parameter for notebook reports (#32) Signed-off-by: David Cui <[email protected]> * replace osd-version header to remove need for version number matching (#37) Signed-off-by: David Cui <[email protected]> * Remove visualization editor in visualization reports (#50) Signed-off-by: Joshua Li <[email protected]> * add condition to fix negative value display (#51) Signed-off-by: David Cui <[email protected]> * [Query Builder] Correctly handle match phrases when a single value is specified or when the match phrases is negated (#33) * [Build query] Correctly handle phrases filter that specify only one single value Contrary to the phrase filter type which always specify a meta.params.query, the phrases filter type never specify a meta.params.query even when a single value is selected. This anomaly trigger an error generated by the MonoFieldQueryBase included in the elastic-builder dependency and prevent the generation of the report. Signed-off-by: Kévin Masseix <[email protected]> * [Build query] Correctly handle negated phrases filter that specify only one single value Signed-off-by: Kévin Masseix <[email protected]> * [Build query] Correctlty handle negated phrases to exclude them from the report instead of including them Signed-off-by: Kévin Masseix <[email protected]> * Pass processing if doc[field] == null (#361) Pass processing if doc[field] == null Based on opendistro-for-elasticsearch/kibana-reports#327 (comment) * add test case for commit 368eb03e200b755870f1ad373e67561201ab6f31 (#370) Add missing test case for the fix "pass doc[field] == null" * Unhash page url in case store URLs in session storage is enabled from Kibana advanced setting (#371) * Update Namespaces/APIs/Documentation for OpenSearch (#55) Signed-off-by: David Cui <[email protected]> * bump path-parse version to 1.0.7 to address CVE (#59) Signed-off-by: David Cui <[email protected]> * increase chromium timeout to 100s (#58) Signed-off-by: Zhongnan Su <[email protected]> * Fix csv parsing function (#53) * Fix csv parsing function Signed-off-by: Joshua Li <[email protected]> * Flatten nested results for csv Signed-off-by: Joshua Li <[email protected]> * Add test case for nested fields Signed-off-by: Joshua Li <[email protected]> * Add release notes for rc1 (#60) Signed-off-by: David Cui <[email protected]> * configure index settings to have default 1 replica and auto-expand upperbound 2 (#62) Signed-off-by: Zhongnan Su <[email protected]> * Rc1 release note patch (#63) Signed-off-by: David Cui <[email protected]> * Bump OpenSearch Dashboards version to 1.0 (#64) Signed-off-by: David Cui <[email protected]> * Revert .opensearch_dashboards index references to .kibana (#67) Signed-off-by: David Cui <[email protected]> * Add diagrams for integration with Notifications plugin (#75) * add puml diagrams for integration with Notifications plugin Signed-off-by: David Cui <[email protected]> * amend Notifications plugin naming in diagram Signed-off-by: David Cui <[email protected]> * Add i18n translation support (#362) * PDF report is no more a screenshot, increasing the overall quality Also a issue with chrome font rendering was fixed * Add Notifications to docs (#87) * moved docs to root folder, update readme to include Notifications roadmap Signed-off-by: David Cui <[email protected]> * update readme to clarify current status of supported features Signed-off-by: David Cui <[email protected]> * Better support sorting for csv report based on saved search (#86) * Change Delivery Request Body for Notifications (#85) * adding opensearch changed api for report definition to save branch Signed-off-by: David Cui <[email protected]> * change delivery request body, update test cases Signed-off-by: David Cui <[email protected]> * remove console.log statement, fix DeliveryType type members to use snake case Signed-off-by: David Cui <[email protected]> * convert all snake case to camel case Signed-off-by: David Cui <[email protected]> * make configId required in schema Signed-off-by: David Cui <[email protected]> * Remove legacy notifications/delivery related code (#94) * remove legacy notifications/delivery related code Signed-off-by: David Cui <[email protected]> * remove commented out reference Signed-off-by: David Cui <[email protected]> * Remove dependency on demo.elastic and use local mock html for testing (#100) * Add code cov back (#98) Signed-off-by: Zhongnan Su <[email protected]> * update workflow to rename artifact in kebab case (#102) Signed-off-by: Zhongnan Su <[email protected]> * bump dependency version (#101) Signed-off-by: Zhongnan Su <[email protected]> * level up markdowns and readme (#97) Signed-off-by: Zhongnan Su <[email protected]> * Bump to version 1.0.0.0 (#103) Signed-off-by: Zhongnan Su <[email protected]> * Bump test resource(job-scheduler) to 1.0.0.0 (#105) Signed-off-by: Zhongnan Su <[email protected]> * fix failed cypress integ-testing (#106) Signed-off-by: Zhongnan Su <[email protected]> * Bump node version, fix workflow and gradle build (#108) * Bump nodejs version to keep up with upstream Signed-off-by: Zhongnan Su <[email protected]> * fix github workflow Signed-off-by: Zhongnan Su <[email protected]> * Add release notes for 1.0.0.0 (#104) Signed-off-by: Zhongnan Su <[email protected]> * Fix notebooks context menu (#109) Signed-off-by: Joshua Li <[email protected]> * Fix context menu download request body after schema change to add notifiaction (#115) Signed-off-by: Zhongnan Su <[email protected]> * Exclude time range from report details for Notebooks (#117) * exclude time range check for notebooks reports Signed-off-by: David Cui <[email protected]> * use report source identifier over url partial match Signed-off-by: David Cui <[email protected]> * add report details fix to release notes Signed-off-by: David Cui <[email protected]> * add more bug fixes to release notes Signed-off-by: David Cui <[email protected]> * Update regex for notebooks reports (#118) Signed-off-by: Joshua Li <[email protected]> * Add integtest script to the repo Signed-off-by: Peter Zhu <[email protected]> * Improve osd bootstrap on dub directory Signed-off-by: Peter Zhu <[email protected]> * Improve osd bootstrap on dub directory Signed-off-by: Peter Zhu <[email protected]> * Exclude Faulty Integration Test (#122) Exclude broken ITs from upstream OpenSearch * Fix cypress reporting (#123) * fix create test Signed-off-by: David Cui <[email protected]> * toggle video on for cypress testing (#124) Signed-off-by: David Cui <[email protected]> * Update Doc Links (#127) * update doc links in dashboards-reports Signed-off-by: David Cui <[email protected]> * Update template to reflect new branding (#128) Signed-off-by: kgcreative <[email protected]> * Update UI for Notifications Integration (#126) Update reporting front-end for Notifications integration Signed-off-by: David Cui <[email protected]> * Add themed logo to README (#130) Signed-off-by: Miki <[email protected]> * Fix url validation (#132) Signed-off-by: Joshua Li <[email protected]> * Fix url validation for context menu (#134) * Fix url validation for context menu Signed-off-by: Joshua Li <[email protected]> * Fix url validation Signed-off-by: Joshua Li <[email protected]> * Connect Notifications APIs to Reporting (#131) * call notifications APIs from reporting, saving progress Signed-off-by: David Cui <[email protected]> * enable notifications integration, update view in details pages, update tests Signed-off-by: David Cui <[email protected]> * address comments, add server APIs to call backend notifications APIs to reduce dependency Signed-off-by: David Cui <[email protected]> * add get_event API to confirm test messages are sent successfully and throw error if not Signed-off-by: David Cui <[email protected]> * add input validation for delivery subject and message, hide notifications section if opensearch notifications is not installed Signed-off-by: David Cui <[email protected]> * renamed constant for notifications to follow API_PREFIX convention Signed-off-by: David Cui <[email protected]> * rename notifications constant to reporting_notifications Signed-off-by: David Cui <[email protected]> * add list of failed channels on test message failure Signed-off-by: David Cui <[email protected]> * Fix snapshot build and upgrade to OpenSearch 1.1. (#140) Signed-off-by: dblock <[email protected]> * Add release notes for 1.0.1 release (#143) Signed-off-by: Joshua Li <[email protected]> * Bump to 1.1 in main (#150) * bump version to 1.1, hide delivery section Signed-off-by: David Cui <[email protected]> * Dont hide delivery by default Signed-off-by: David Cui <[email protected]> * bump version to 1.1 in dashboards reports workflow Signed-off-by: David Cui <[email protected]> * fix OSD branch to use main Signed-off-by: David Cui <[email protected]> * change from main to 1.x for dashboards version Signed-off-by: David Cui <[email protected]> * Bump opensearch ref to 1.1 in CI (#155) * Removed integtest.sh. (#157) Signed-off-by: dblock <[email protected]> * Taking RBAC settings from Alerting plugin default to false Other changes: 1. Legacy settings removed because all relavent settings are removed or taken from Alerting plugin 2. Poll API removed as it is not used [Tests] All existing tests passed Signed-off-by: @akbhatta * Bump tmpl from 1.0.4 to 1.0.5 in /dashboards-reports (#164) Bumps [tmpl](https://github.com/daaku/nodejs-tmpl) from 1.0.4 to 1.0.5. - [Release notes](https://github.com/daaku/nodejs-tmpl/releases) - [Commits](https://github.com/daaku/nodejs-tmpl/commits/v1.0.5) --- updated-dependencies: - dependency-name: tmpl dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix quoting and url-encoding (#153) * Replace '%2F' with '/' in relative dates Signed-off-by: Konstantin Roussou <[email protected]> * Ensure proper quoting of converted dates in URL Signed-off-by: Konstantin Roussou <[email protected]> * Use generic URL decoding Co-authored-by: Zhongnan Su <[email protected]> * Use roundUp for toDate Co-authored-by: Zhongnan Su <[email protected]> * Make date transformation same as in context_menu_helpers.js Co-authored-by: Zhongnan Su <[email protected]> * Integrate notifications backend (#129) * Remove hard coded localhost when calling API (#172) * rename plugin helper config file name to consistent with OSD (#180) Signed-off-by: Tengda He <[email protected]> * Add metrics for sending notifications (#173) * Add logic to build report detail page link and send as part of message for non-email channels (#182) * forward origin plus basePath to backend * Add build report link function and forward link as part of message * Update validation for observability notebooks integration (#174) * Build email message from template with reports links (#184) * remove the email tempalte and helpers from dashboards plugin * build email message from template for reporting notifcation Signed-off-by: Zhongnan Su <[email protected]> * Support range filters for csv reports (#185) * Forward extra headers while using headless chromium (#194) * Use advanced settings for date format in csv reports (#186) * add check in ReportInstanceActions to fix on demand report creation after report definition creation (#196) Signed-off by: David Cui <[email protected]> * Catch Notifications Errors on Details Pages (#197) * add catch statements to notifications get_configs to avoid crashing details pages Signed-off by: David Cui <[email protected]> * remove console log statment Signed-off by: David Cui <[email protected]> * remove notifications integration from details pages and update snapshots (#210) Signed-off-by: David Cui <[email protected]> * fix csv missing metadata fields issue and empty report on all(_source) fields (#206) * remove notifications references from create and edit report definition pages (#212) Signed-off by: David Cui <[email protected]> * refactor logic for creating DSL from saved object using buildOpensearchQuery() (#213) * Remove calling notifications in reports scheduler (#211) * Use advanced settings for csv separator and visual report timezone (#209) * Bump to version 1.2 (#203) * Bump to version 1.2 Bump OpenSearch and OpenSearch Dashboards plugins to 1.2 Related issue: #167 Signed-off-by: Kawika Avilla <[email protected]> * Use maven for building reports scheduler Signed-off-by: Joshua Li <[email protected]> Co-authored-by: Joshua Li <[email protected]> * Add dco and release drafter workflows (#217) Signed-off-by: Joshua Li <[email protected]> * update readme notification section (#216) Signed-off-by: Zhongnan Su <[email protected]> * revert paths to opendistro to keep cluster permissions same with ODFE (#218) Signed-off-by: David Cui <[email protected]> * Create opensearch-dashboards-reports.release-notes-1.2.0.0.md Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Update the copyright notices to reflect OpenSearch Contributors (#220) * update the copyright notices to reflect OpenSearch Contributors Signed-off-by: David Cui <[email protected]> * add .txt to NOTICE to resolve linkchecker error Signed-off-by: David Cui <[email protected]> * changes Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * changes Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Remove notifications actions in reports scheduler (#221) Signed-off-by: Joshua Li <[email protected]> * add reporting osd server configurations (#222) Signed-off-by: Zhongnan Su <[email protected]> * Add suggested font packages in troubleshooting tips (#189) * add suggested font packages in troubleshooting tips Signed-off-by: David Cui <[email protected]> * add links to package documentation in README for font packages Signed-off-by: David Cui <[email protected]> * add more descriptive title under Troubleshooting Signed-off-by: David Cui <[email protected]> * add link to headless-chrome installation and add ipa-gothic-fonts link to centos 7 Signed-off-by: David Cui <[email protected]> * Update Copyright Headers (#226) * update copyright header for all files Signed-off-by: David Cui <[email protected]> * fix ktlint errors with extra whitespace Signed-off-by: David Cui <[email protected]> * Add support for codeowners to repo (#227) Signed-off-by: Ryan Bogan <[email protected]> * Fix reporting validation for observability (#231) * Update reporting validation for observability Signed-off-by: Joshua Li <[email protected]> * update tests Signed-off-by: Joshua Li <[email protected]> * remove poller code from frontend plugin (#235) Signed-off-by: Zhongnan Su <[email protected]> * bump reports-scheduler to 1.3 (#241) Signed-off-by: Zhongnan Su <[email protected]> * Fix apt source and link checker for CI (#245) * Update apt source in CI Signed-off-by: Joshua Li <[email protected]> * Try fix link checker Signed-off-by: Joshua Li <[email protected]> * Add Case Coverage tests for Create Report Definition (#239) * add & refactor cypress tests, add css selectors in report settings & report trigger Signed-off-by: David Cui <[email protected]> * remove comment and update jest snapshot Signed-off-by: David Cui <[email protected]> * add bwc test for reports-scheduler (#244) Signed-off-by: Zhongnan Su <[email protected]> * added code summary table (#243) * added code summary table Signed-off-by: Shenoy Pratik <[email protected]> * modularized tables, fixed links Signed-off-by: Shenoy Pratik <[email protected]> * updated header, changed bwc link Signed-off-by: Shenoy Pratik <[email protected]> * Add Cypress Case Coverage on Details Pages (#250) * add and refactor cypress tests for details pages Signed-off-by: David Cui <[email protected]> * update jest snapshots Signed-off-by: David Cui <[email protected]> * add utils.js file to store functions used in multiple cypress tests Signed-off-by: David Cui <[email protected]> * add gradle task to run integtest against remote cluster (#247) Signed-off-by: Zhongnan Su <[email protected]> * added untriaged and nolabel badges Signed-off-by: Shenoy Pratik <[email protected]> * bump dashboards-reports version to 1.3 (#265) Signed-off-by: Zhongnan Su <[email protected]> * add UT for request/reseponse model (#264) Signed-off-by: Zhongnan Su <[email protected]> * Add .whitesource configuration file * Address Reporting CVEs (#267) * update vulnerable package versions Signed-off-by: David Cui <[email protected]> * change ansi-regex to 5.0.1 to pass bootstrap Signed-off-by: David Cui <[email protected]> * update yarn lock for cve fixes (#271) Signed-off-by: David Cui <[email protected]> * add release notes for 1.2.4 Signed-off-by: David Cui <[email protected]> * remove jcenter repo from gradle build Signed-off-by: Shenoy Pratik <[email protected]> * update guava to 31.0.1-jre Signed-off-by: Shenoy Pratik <[email protected]> * WS package update (#283) * updated ws version Signed-off-by: Shenoy Pratik <[email protected]> * moved ws package to resolutions Signed-off-by: Shenoy Pratik <[email protected]> * updated yarn lock Signed-off-by: Shenoy Pratik <[email protected]> * add auto-backport functionality for reporting (#286) Add auto-backport and auto-delete workflows for reporting Signed-off-by: David Cui <[email protected]> * Change Default CI version to JDK 11 (#290) * change default CI version to JDK 11, add JDK versions to CI matrix and document changes in developer guide Signed-off-by: David Cui <[email protected]> * remove incorrect tag form issue template (#294) Signed-off-by: Zhongnan Su <[email protected]> * Fix empty or multiple date values in csv (#293) Signed-off-by: Zhongnan Su <[email protected]> * Update the saved object id matcher regex to match custom id (#295) Signed-off-by: Zhongnan Su <[email protected]> * Replace Centos links to fix link checker CI (#297) exclude centos links from linkchecker Signed-off-by: David Cui <[email protected]> * update regex to match custom saved object id (#298) Signed-off-by: Zhongnan Su <[email protected]> * Fix reporting uuid parsing (#300) Signed-off-by: Joshua Li <[email protected]> * Add java 8 support in compile and test (#304) (#305) * add java 8 support in compile and test Signed-off-by: Zhongnan Su <[email protected]> (cherry picked from commit e753b07) Co-authored-by: Zhongnan Su <[email protected]> * Add 1.3 release notes (#307) (#308) * Add 1.3 release notes Signed-off-by: Zhongnan Su <[email protected]> (cherry picked from commit 2e904c5) Co-authored-by: Zhongnan Su <[email protected]> * Bump version to 1.3.1 (#324) Signed-off-by: Sayali Gaikawad <[email protected]> * Bump reports dashboards to 1.3.1 (#331) Signed-off-by: Joshua Li <[email protected]> * Incremented version to 1.3.2. (#352) Signed-off-by: Zelin Hao <[email protected]> * Version bump 1.3.3 Signed-off-by: pgodithi <[email protected]> * Version bump 1.3.3 Signed-off-by: pgodithi <[email protected]> * Version bump 1.3.3 Signed-off-by: pgodithi <[email protected]> * [1.3] Bump minimist to 1.2.6 (#367) Signed-off-by: Joshua Li <[email protected]> * Move HTML sanitization from client to server side (#382) (#386) Move HTML sanitization to server, store markdown over HTML, remove unused imports Signed-off-by: David Cui <[email protected]> Co-authored-by: David Cui <[email protected]> * Incremented version to 1.3.4. (#389) * Incremented version to 1.3.4. Signed-off-by: Zelin Hao <[email protected]> * Add JS snapshot Signed-off-by: Zelin Hao <[email protected]> * Add release notes for 1.3.4.0 (#393) (#394) Signed-off-by: Joshua Li <[email protected]> (cherry picked from commit 8e01eaa) Co-authored-by: Joshua Li <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: David Cui <[email protected]> Co-authored-by: Joshua <[email protected]> Co-authored-by: Carl Meadows <[email protected]> Co-authored-by: Kevin Masseix <[email protected]> Co-authored-by: Uzhinskiy Boris <[email protected]> Co-authored-by: David Cui <[email protected]> Co-authored-by: starcatter <[email protected]> Co-authored-by: Elias Soares <[email protected]> Co-authored-by: Peter Zhu <[email protected]> Co-authored-by: Kevin Garcia <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: David Cui <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Anantha Krishna Bhatta <[email protected]> Co-authored-by: Anantha Krishna Bhatta <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: kroussou <[email protected]> Co-authored-by: Tengda-He <[email protected]> Co-authored-by: Kawika Avilla <[email protected]> Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> Co-authored-by: Ryan Bogan <[email protected]> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Sayali Gaikawad <[email protected]> Co-authored-by: Zelin Hao <[email protected]> Co-authored-by: pgodithi <[email protected]> Co-authored-by: Anirudha (Ani) Jadhav <[email protected]>

@akbhatta

* Fix release notes version (#28) Signed-off-by: Zhongnan Su <[email protected]> * Migrate Reporting Notebooks Integration (#21) Signed-off-by: David Cui <[email protected]> * Add Logic to Auto-populate Notebooks from Context Menu (#30) Signed-off-by: David Cui <[email protected]> * Change opendistro to opensearch in email template (#31) Signed-off-by: Joshua Li <[email protected]> * Fix case sensitive directory name for chromium zip (#35) Signed-off-by: David Cui <[email protected]> Co-authored-by: Carl Meadows <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: Joshua Li <[email protected]> * Use output_only Parameter for notebook reports (#32) Signed-off-by: David Cui <[email protected]> * replace osd-version header to remove need for version number matching (#37) Signed-off-by: David Cui <[email protected]> * Remove visualization editor in visualization reports (#50) Signed-off-by: Joshua Li <[email protected]> * add condition to fix negative value display (#51) Signed-off-by: David Cui <[email protected]> * [Query Builder] Correctly handle match phrases when a single value is specified or when the match phrases is negated (#33) * [Build query] Correctly handle phrases filter that specify only one single value Contrary to the phrase filter type which always specify a meta.params.query, the phrases filter type never specify a meta.params.query even when a single value is selected. This anomaly trigger an error generated by the MonoFieldQueryBase included in the elastic-builder dependency and prevent the generation of the report. Signed-off-by: Kévin Masseix <[email protected]> * [Build query] Correctly handle negated phrases filter that specify only one single value Signed-off-by: Kévin Masseix <[email protected]> * [Build query] Correctlty handle negated phrases to exclude them from the report instead of including them Signed-off-by: Kévin Masseix <[email protected]> * Pass processing if doc[field] == null (#361) Pass processing if doc[field] == null Based on opendistro-for-elasticsearch/kibana-reports#327 (comment) * add test case for commit 368eb03e200b755870f1ad373e67561201ab6f31 (#370) Add missing test case for the fix "pass doc[field] == null" * Unhash page url in case store URLs in session storage is enabled from Kibana advanced setting (#371) * Update Namespaces/APIs/Documentation for OpenSearch (#55) Signed-off-by: David Cui <[email protected]> * bump path-parse version to 1.0.7 to address CVE (#59) Signed-off-by: David Cui <[email protected]> * increase chromium timeout to 100s (#58) Signed-off-by: Zhongnan Su <[email protected]> * Fix csv parsing function (#53) * Fix csv parsing function Signed-off-by: Joshua Li <[email protected]> * Flatten nested results for csv Signed-off-by: Joshua Li <[email protected]> * Add test case for nested fields Signed-off-by: Joshua Li <[email protected]> * Add release notes for rc1 (#60) Signed-off-by: David Cui <[email protected]> * configure index settings to have default 1 replica and auto-expand upperbound 2 (#62) Signed-off-by: Zhongnan Su <[email protected]> * Rc1 release note patch (#63) Signed-off-by: David Cui <[email protected]> * Bump OpenSearch Dashboards version to 1.0 (#64) Signed-off-by: David Cui <[email protected]> * Revert .opensearch_dashboards index references to .kibana (#67) Signed-off-by: David Cui <[email protected]> * Add diagrams for integration with Notifications plugin (#75) * add puml diagrams for integration with Notifications plugin Signed-off-by: David Cui <[email protected]> * amend Notifications plugin naming in diagram Signed-off-by: David Cui <[email protected]> * Add i18n translation support (#362) * PDF report is no more a screenshot, increasing the overall quality Also a issue with chrome font rendering was fixed * Add Notifications to docs (#87) * moved docs to root folder, update readme to include Notifications roadmap Signed-off-by: David Cui <[email protected]> * update readme to clarify current status of supported features Signed-off-by: David Cui <[email protected]> * Better support sorting for csv report based on saved search (#86) * Change Delivery Request Body for Notifications (#85) * adding opensearch changed api for report definition to save branch Signed-off-by: David Cui <[email protected]> * change delivery request body, update test cases Signed-off-by: David Cui <[email protected]> * remove console.log statement, fix DeliveryType type members to use snake case Signed-off-by: David Cui <[email protected]> * convert all snake case to camel case Signed-off-by: David Cui <[email protected]> * make configId required in schema Signed-off-by: David Cui <[email protected]> * Remove legacy notifications/delivery related code (#94) * remove legacy notifications/delivery related code Signed-off-by: David Cui <[email protected]> * remove commented out reference Signed-off-by: David Cui <[email protected]> * Remove dependency on demo.elastic and use local mock html for testing (#100) * Add code cov back (#98) Signed-off-by: Zhongnan Su <[email protected]> * update workflow to rename artifact in kebab case (#102) Signed-off-by: Zhongnan Su <[email protected]> * bump dependency version (#101) Signed-off-by: Zhongnan Su <[email protected]> * level up markdowns and readme (#97) Signed-off-by: Zhongnan Su <[email protected]> * Bump to version 1.0.0.0 (#103) Signed-off-by: Zhongnan Su <[email protected]> * Bump test resource(job-scheduler) to 1.0.0.0 (#105) Signed-off-by: Zhongnan Su <[email protected]> * fix failed cypress integ-testing (#106) Signed-off-by: Zhongnan Su <[email protected]> * Bump node version, fix workflow and gradle build (#108) * Bump nodejs version to keep up with upstream Signed-off-by: Zhongnan Su <[email protected]> * fix github workflow Signed-off-by: Zhongnan Su <[email protected]> * Add release notes for 1.0.0.0 (#104) Signed-off-by: Zhongnan Su <[email protected]> * Fix notebooks context menu (#109) Signed-off-by: Joshua Li <[email protected]> * Fix context menu download request body after schema change to add notifiaction (#115) Signed-off-by: Zhongnan Su <[email protected]> * Exclude time range from report details for Notebooks (#117) * exclude time range check for notebooks reports Signed-off-by: David Cui <[email protected]> * use report source identifier over url partial match Signed-off-by: David Cui <[email protected]> * add report details fix to release notes Signed-off-by: David Cui <[email protected]> * add more bug fixes to release notes Signed-off-by: David Cui <[email protected]> * Update regex for notebooks reports (#118) Signed-off-by: Joshua Li <[email protected]> * Add integtest script to the repo Signed-off-by: Peter Zhu <[email protected]> * Improve osd bootstrap on dub directory Signed-off-by: Peter Zhu <[email protected]> * Improve osd bootstrap on dub directory Signed-off-by: Peter Zhu <[email protected]> * Exclude Faulty Integration Test (#122) Exclude broken ITs from upstream OpenSearch * Fix cypress reporting (#123) * fix create test Signed-off-by: David Cui <[email protected]> * toggle video on for cypress testing (#124) Signed-off-by: David Cui <[email protected]> * Update Doc Links (#127) * update doc links in dashboards-reports Signed-off-by: David Cui <[email protected]> * Update template to reflect new branding (#128) Signed-off-by: kgcreative <[email protected]> * Update UI for Notifications Integration (#126) Update reporting front-end for Notifications integration Signed-off-by: David Cui <[email protected]> * Add themed logo to README (#130) Signed-off-by: Miki <[email protected]> * Fix url validation (#132) Signed-off-by: Joshua Li <[email protected]> * Fix url validation for context menu (#134) * Fix url validation for context menu Signed-off-by: Joshua Li <[email protected]> * Fix url validation Signed-off-by: Joshua Li <[email protected]> * Connect Notifications APIs to Reporting (#131) * call notifications APIs from reporting, saving progress Signed-off-by: David Cui <[email protected]> * enable notifications integration, update view in details pages, update tests Signed-off-by: David Cui <[email protected]> * address comments, add server APIs to call backend notifications APIs to reduce dependency Signed-off-by: David Cui <[email protected]> * add get_event API to confirm test messages are sent successfully and throw error if not Signed-off-by: David Cui <[email protected]> * add input validation for delivery subject and message, hide notifications section if opensearch notifications is not installed Signed-off-by: David Cui <[email protected]> * renamed constant for notifications to follow API_PREFIX convention Signed-off-by: David Cui <[email protected]> * rename notifications constant to reporting_notifications Signed-off-by: David Cui <[email protected]> * add list of failed channels on test message failure Signed-off-by: David Cui <[email protected]> * Fix snapshot build and upgrade to OpenSearch 1.1. (#140) Signed-off-by: dblock <[email protected]> * Add release notes for 1.0.1 release (#143) Signed-off-by: Joshua Li <[email protected]> * Bump to 1.1 in main (#150) * bump version to 1.1, hide delivery section Signed-off-by: David Cui <[email protected]> * Dont hide delivery by default Signed-off-by: David Cui <[email protected]> * bump version to 1.1 in dashboards reports workflow Signed-off-by: David Cui <[email protected]> * fix OSD branch to use main Signed-off-by: David Cui <[email protected]> * change from main to 1.x for dashboards version Signed-off-by: David Cui <[email protected]> * Bump opensearch ref to 1.1 in CI (#155) * Removed integtest.sh. (#157) Signed-off-by: dblock <[email protected]> * Taking RBAC settings from Alerting plugin default to false Other changes: 1. Legacy settings removed because all relavent settings are removed or taken from Alerting plugin 2. Poll API removed as it is not used [Tests] All existing tests passed Signed-off-by: @akbhatta * Bump tmpl from 1.0.4 to 1.0.5 in /dashboards-reports (#164) Bumps [tmpl](https://github.com/daaku/nodejs-tmpl) from 1.0.4 to 1.0.5. - [Release notes](https://github.com/daaku/nodejs-tmpl/releases) - [Commits](https://github.com/daaku/nodejs-tmpl/commits/v1.0.5) --- updated-dependencies: - dependency-name: tmpl dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix quoting and url-encoding (#153) * Replace '%2F' with '/' in relative dates Signed-off-by: Konstantin Roussou <[email protected]> * Ensure proper quoting of converted dates in URL Signed-off-by: Konstantin Roussou <[email protected]> * Use generic URL decoding Co-authored-by: Zhongnan Su <[email protected]> * Use roundUp for toDate Co-authored-by: Zhongnan Su <[email protected]> * Make date transformation same as in context_menu_helpers.js Co-authored-by: Zhongnan Su <[email protected]> * Integrate notifications backend (#129) * Remove hard coded localhost when calling API (#172) * rename plugin helper config file name to consistent with OSD (#180) Signed-off-by: Tengda He <[email protected]> * Add metrics for sending notifications (#173) * Add logic to build report detail page link and send as part of message for non-email channels (#182) * forward origin plus basePath to backend * Add build report link function and forward link as part of message * Update validation for observability notebooks integration (#174) * Build email message from template with reports links (#184) * remove the email tempalte and helpers from dashboards plugin * build email message from template for reporting notifcation Signed-off-by: Zhongnan Su <[email protected]> * Support range filters for csv reports (#185) * Forward extra headers while using headless chromium (#194) * Use advanced settings for date format in csv reports (#186) * add check in ReportInstanceActions to fix on demand report creation after report definition creation (#196) Signed-off by: David Cui <[email protected]> * Catch Notifications Errors on Details Pages (#197) * add catch statements to notifications get_configs to avoid crashing details pages Signed-off by: David Cui <[email protected]> * remove console log statment Signed-off by: David Cui <[email protected]> * remove notifications integration from details pages and update snapshots (#210) Signed-off-by: David Cui <[email protected]> * fix csv missing metadata fields issue and empty report on all(_source) fields (#206) * remove notifications references from create and edit report definition pages (#212) Signed-off by: David Cui <[email protected]> * refactor logic for creating DSL from saved object using buildOpensearchQuery() (#213) * Remove calling notifications in reports scheduler (#211) * Use advanced settings for csv separator and visual report timezone (#209) * Bump to version 1.2 (#203) * Bump to version 1.2 Bump OpenSearch and OpenSearch Dashboards plugins to 1.2 Related issue: #167 Signed-off-by: Kawika Avilla <[email protected]> * Use maven for building reports scheduler Signed-off-by: Joshua Li <[email protected]> Co-authored-by: Joshua Li <[email protected]> * Add dco and release drafter workflows (#217) Signed-off-by: Joshua Li <[email protected]> * update readme notification section (#216) Signed-off-by: Zhongnan Su <[email protected]> * revert paths to opendistro to keep cluster permissions same with ODFE (#218) Signed-off-by: David Cui <[email protected]> * Create opensearch-dashboards-reports.release-notes-1.2.0.0.md Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Update the copyright notices to reflect OpenSearch Contributors (#220) * update the copyright notices to reflect OpenSearch Contributors Signed-off-by: David Cui <[email protected]> * add .txt to NOTICE to resolve linkchecker error Signed-off-by: David Cui <[email protected]> * changes Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * changes Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> * Remove notifications actions in reports scheduler (#221) Signed-off-by: Joshua Li <[email protected]> * add reporting osd server configurations (#222) Signed-off-by: Zhongnan Su <[email protected]> * Add suggested font packages in troubleshooting tips (#189) * add suggested font packages in troubleshooting tips Signed-off-by: David Cui <[email protected]> * add links to package documentation in README for font packages Signed-off-by: David Cui <[email protected]> * add more descriptive title under Troubleshooting Signed-off-by: David Cui <[email protected]> * add link to headless-chrome installation and add ipa-gothic-fonts link to centos 7 Signed-off-by: David Cui <[email protected]> * Update Copyright Headers (#226) * update copyright header for all files Signed-off-by: David Cui <[email protected]> * fix ktlint errors with extra whitespace Signed-off-by: David Cui <[email protected]> * Add support for codeowners to repo (#227) Signed-off-by: Ryan Bogan <[email protected]> * Fix reporting validation for observability (#231) * Update reporting validation for observability Signed-off-by: Joshua Li <[email protected]> * update tests Signed-off-by: Joshua Li <[email protected]> * remove poller code from frontend plugin (#235) Signed-off-by: Zhongnan Su <[email protected]> * bump reports-scheduler to 1.3 (#241) Signed-off-by: Zhongnan Su <[email protected]> * Fix apt source and link checker for CI (#245) * Update apt source in CI Signed-off-by: Joshua Li <[email protected]> * Try fix link checker Signed-off-by: Joshua Li <[email protected]> * Add Case Coverage tests for Create Report Definition (#239) * add & refactor cypress tests, add css selectors in report settings & report trigger Signed-off-by: David Cui <[email protected]> * remove comment and update jest snapshot Signed-off-by: David Cui <[email protected]> * add bwc test for reports-scheduler (#244) Signed-off-by: Zhongnan Su <[email protected]> * added code summary table (#243) * added code summary table Signed-off-by: Shenoy Pratik <[email protected]> * modularized tables, fixed links Signed-off-by: Shenoy Pratik <[email protected]> * updated header, changed bwc link Signed-off-by: Shenoy Pratik <[email protected]> * Add Cypress Case Coverage on Details Pages (#250) * add and refactor cypress tests for details pages Signed-off-by: David Cui <[email protected]> * update jest snapshots Signed-off-by: David Cui <[email protected]> * add utils.js file to store functions used in multiple cypress tests Signed-off-by: David Cui <[email protected]> * add gradle task to run integtest against remote cluster (#247) Signed-off-by: Zhongnan Su <[email protected]> * added untriaged and nolabel badges Signed-off-by: Shenoy Pratik <[email protected]> * bump dashboards-reports version to 1.3 (#265) Signed-off-by: Zhongnan Su <[email protected]> * add UT for request/reseponse model (#264) Signed-off-by: Zhongnan Su <[email protected]> * Add .whitesource configuration file * Address Reporting CVEs (#267) * update vulnerable package versions Signed-off-by: David Cui <[email protected]> * change ansi-regex to 5.0.1 to pass bootstrap Signed-off-by: David Cui <[email protected]> * update yarn lock for cve fixes (#271) Signed-off-by: David Cui <[email protected]> * add release notes for 1.2.4 Signed-off-by: David Cui <[email protected]> * remove jcenter repo from gradle build Signed-off-by: Shenoy Pratik <[email protected]> * update guava to 31.0.1-jre Signed-off-by: Shenoy Pratik <[email protected]> * WS package update (#283) * updated ws version Signed-off-by: Shenoy Pratik <[email protected]> * moved ws package to resolutions Signed-off-by: Shenoy Pratik <[email protected]> * updated yarn lock Signed-off-by: Shenoy Pratik <[email protected]> * add auto-backport functionality for reporting (#286) Add auto-backport and auto-delete workflows for reporting Signed-off-by: David Cui <[email protected]> * Change Default CI version to JDK 11 (#290) * change default CI version to JDK 11, add JDK versions to CI matrix and document changes in developer guide Signed-off-by: David Cui <[email protected]> * remove incorrect tag form issue template (#294) Signed-off-by: Zhongnan Su <[email protected]> * Fix empty or multiple date values in csv (#293) Signed-off-by: Zhongnan Su <[email protected]> * Update the saved object id matcher regex to match custom id (#295) Signed-off-by: Zhongnan Su <[email protected]> * Replace Centos links to fix link checker CI (#297) exclude centos links from linkchecker Signed-off-by: David Cui <[email protected]> * update regex to match custom saved object id (#298) Signed-off-by: Zhongnan Su <[email protected]> * Fix reporting uuid parsing (#300) Signed-off-by: Joshua Li <[email protected]> * Add java 8 support in compile and test (#304) (#305) * add java 8 support in compile and test Signed-off-by: Zhongnan Su <[email protected]> (cherry picked from commit e753b07) Co-authored-by: Zhongnan Su <[email protected]> * Add 1.3 release notes (#307) (#308) * Add 1.3 release notes Signed-off-by: Zhongnan Su <[email protected]> (cherry picked from commit 2e904c5) Co-authored-by: Zhongnan Su <[email protected]> * Bump version to 1.3.1 (#324) Signed-off-by: Sayali Gaikawad <[email protected]> * Bump reports dashboards to 1.3.1 (#331) Signed-off-by: Joshua Li <[email protected]> * Incremented version to 1.3.2. (#352) Signed-off-by: Zelin Hao <[email protected]> * Version bump 1.3.3 Signed-off-by: pgodithi <[email protected]> * Version bump 1.3.3 Signed-off-by: pgodithi <[email protected]> * Version bump 1.3.3 Signed-off-by: pgodithi <[email protected]> * [1.3] Bump minimist to 1.2.6 (#367) Signed-off-by: Joshua Li <[email protected]> * Move HTML sanitization from client to server side (#382) (#386) Move HTML sanitization to server, store markdown over HTML, remove unused imports Signed-off-by: David Cui <[email protected]> Co-authored-by: David Cui <[email protected]> * Incremented version to 1.3.4. (#389) * Incremented version to 1.3.4. Signed-off-by: Zelin Hao <[email protected]> * Add JS snapshot Signed-off-by: Zelin Hao <[email protected]> * Add release notes for 1.3.4.0 (#393) (#394) Signed-off-by: Joshua Li <[email protected]> (cherry picked from commit 8e01eaa) Co-authored-by: Joshua Li <[email protected]> * Staging for version increment automation (#391) (#418) * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation Signed-off-by: pgodithi <[email protected]> * Version increment automation: task rename updateVersion Signed-off-by: pgodithi <[email protected]> (cherry picked from commit 6667bfb) Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: pgodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> Co-authored-by: Prudhvi Godithi <[email protected]> * [Release 1.3.5] Version increment (#420) * release 1.3.5 Signed-off-by: prudhvigodithi <[email protected]> * release 1.3.5 Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> * [1.3] Remove zips and download from remote at build time (#337) (#422) * Remove zips and download from remote at build time (#337) Signed-off-by: Joshua Li <[email protected]> * Remove odfe bwc test (#408) Signed-off-by: Shenoy Pratik <[email protected]> Signed-off-by: Joshua Li <[email protected]> Signed-off-by: Shenoy Pratik <[email protected]> Co-authored-by: Shenoy Pratik <[email protected]> * [1.3] restrict chromium requests (#424) Signed-off-by: Joshua Li <[email protected]> * [Release 1.3.5] Add `build.sh` (#426) * release 1.3.5 Signed-off-by: prudhvigodithi <[email protected]> * release 1.3.5 Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> * [1.3] Add Bwc Test for OS 1.1 (#417) (#423) * Add Bwc Test for OS 1.1 (#417) Signed-off-by: Shenoy Pratik <[email protected]> * Fix job-scheduler url Signed-off-by: Joshua Li <[email protected]> * Remove artifact zips Signed-off-by: Joshua Li <[email protected]> Signed-off-by: Shenoy Pratik <[email protected]> Signed-off-by: Joshua Li <[email protected]> Co-authored-by: Shenoy Pratik <[email protected]> * Increment version to 1.3.6-SNAPSHOT (#456) Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> * Merge pull request #472 from sejli/CVE-fix-1.3.6 (#473) Updating Kotlin and Jsoup (cherry picked from commit 50eac17) * Merge pull request #474 from sejli/release-notes-1.3.6 Add release notes for 1.3.6 (cherry picked from commit fd8f9de) * Merge pull request #476 from sejli/1.3.6-release-notes Changing 1.3.6 release notes location (cherry picked from commit 2159036) * update jackson to 2.13.4 (#479) (#480) Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> (cherry picked from commit d0c86aa) Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> * Update 1.3.6 release notes (#481) (#482) Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> (cherry picked from commit dea93af) Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> * [1.3] Upgrade puppeteer (#488) * upgrade puppeteer & change report timeout Signed-off-by: Rupal Mahajan <[email protected]> * Change timeout Signed-off-by: Rupal Mahajan <[email protected]> Signed-off-by: Rupal Mahajan <[email protected]> * Added release notes for 1.3.5 (#451) Signed-off-by: vamsi-amazon <[email protected]> (cherry picked from commit e2e4366) Co-authored-by: vamsi-amazon <[email protected]> * Increment version to 1.3.7-SNAPSHOT (#502) Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: opensearch-ci-bot <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> * Resolve conflicting dependencies Signed-off-by: Rupal Mahajan <[email protected]> * nit Signed-off-by: Rupal Mahajan <[email protected]> * Fix build failure Signed-off-by: Rupal Mahajan <[email protected]> Signed-off-by: Zhongnan Su <[email protected]> Signed-off-by: David Cui <[email protected]> Signed-off-by: Joshua Li <[email protected]> Signed-off-by: Kévin Masseix <[email protected]> Signed-off-by: Peter Zhu <[email protected]> Signed-off-by: kgcreative <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: dblock <[email protected]> Signed-off-by: dblock <[email protected]> Signed-off-by: @akbhatta Signed-off-by: Tengda He <[email protected]> Signed-off-by: Kavitha Conjeevaram Mohan <[email protected]> Signed-off-by: Ryan Bogan <[email protected]> Signed-off-by: Shenoy Pratik <[email protected]> Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: Zelin Hao <[email protected]> Signed-off-by: pgodithi <[email protected]> Signed-off-by: prudhvigodithi <[email protected]> Signed-off-by: opensearch-ci-bot <[email protected]> Signed-off-by: Rupal Mahajan <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: David Cui <[email protected]> Co-authored-by: Joshua <[email protected]> Co-authored-by: Carl Meadows <[email protected]> Co-authored-by: Kevin Masseix <[email protected]> Co-authored-by: Uzhinskiy Boris <[email protected]> Co-authored-by: David Cui <[email protected]> Co-authored-by: starcatter <[email protected]> Co-authored-by: Elias Soares <[email protected]> Co-authored-by: Peter Zhu <[email protected]> Co-authored-by: Kevin Garcia <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: David Cui <[email protected]> Co-authored-by: Daniel Doubrovkine (dB.) <[email protected]> Co-authored-by: Anantha Krishna Bhatta <[email protected]> Co-authored-by: Anantha Krishna Bhatta <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: kroussou <[email protected]> Co-authored-by: Tengda-He <[email protected]> Co-authored-by: Kawika Avilla <[email protected]> Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> Co-authored-by: Kavitha Conjeevaram Mohan <[email protected]> Co-authored-by: Ryan Bogan <[email protected]> Co-authored-by: Shenoy Pratik <[email protected]> Co-authored-by: whitesource-for-github-com[bot] <50673670+whitesource-for-github-com[bot]@users.noreply.github.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Sayali Gaikawad <[email protected]> Co-authored-by: Zelin Hao <[email protected]> Co-authored-by: pgodithi <[email protected]> Co-authored-by: Anirudha (Ani) Jadhav <[email protected]> Co-authored-by: opensearch-ci-bot <[email protected]> Co-authored-by: Sean Li <[email protected]> Co-authored-by: vamsi-amazon <[email protected]>

* call notifications APIs from reporting, saving progress Signed-off-by: David Cui <[email protected]> * enable notifications integration, update view in details pages, update tests Signed-off-by: David Cui <[email protected]> * address comments, add server APIs to call backend notifications APIs to reduce dependency Signed-off-by: David Cui <[email protected]> * add get_event API to confirm test messages are sent successfully and throw error if not Signed-off-by: David Cui <[email protected]> * add input validation for delivery subject and message, hide notifications section if opensearch notifications is not installed Signed-off-by: David Cui <[email protected]> * renamed constant for notifications to follow API_PREFIX convention Signed-off-by: David Cui <[email protected]> * rename notifications constant to reporting_notifications Signed-off-by: David Cui <[email protected]> * add list of failed channels on test message failure Signed-off-by: David Cui <[email protected]>

davidcui1225 added 3 commits July 20, 2021 17:33

call notifications APIs from reporting, saving progress

faedf54

Signed-off-by: David Cui <[email protected]>

enable notifications integration, update view in details pages, updat…

d214d94

…e tests Signed-off-by: David Cui <[email protected]>

remove commented code and console log statement

e02d54d

Signed-off-by: David Cui <[email protected]>

davidcui1225 requested review from joshuali925 and zhongnansu July 21, 2021 01:25

joshuali925 reviewed Jul 26, 2021

View reviewed changes

davidcui1225 added 2 commits July 26, 2021 16:57

address comments, add server APIs to call backend notifications APIs …

9d685a6

…to reduce dependency Signed-off-by: David Cui <[email protected]>

add get_event API to confirm test messages are sent successfully and …

304e2ac

…throw error if not Signed-off-by: David Cui <[email protected]>

davidcui1225 requested a review from joshuali925 July 27, 2021 17:03

zhongnansu reviewed Jul 27, 2021

View reviewed changes

davidcui1225 added 2 commits July 27, 2021 15:51

add input validation for delivery subject and message, hide notificat…

1960bec

…ions section if opensearch notifications is not installed Signed-off-by: David Cui <[email protected]>

renamed constant for notifications to follow API_PREFIX convention

a02678f

Signed-off-by: David Cui <[email protected]>

joshuali925 reviewed Jul 30, 2021

View reviewed changes

dashboards-reports/public/components/report_definitions/delivery/delivery.tsx Outdated Show resolved Hide resolved

dashboards-reports/common/index.ts Outdated Show resolved Hide resolved

davidcui1225 added 2 commits July 30, 2021 14:08

rename notifications constant to reporting_notifications

719f9a7

Signed-off-by: David Cui <[email protected]>

add list of failed channels on test message failure

65c3871

Signed-off-by: David Cui <[email protected]>

davidcui1225 requested review from joshuali925 and zhongnansu July 30, 2021 21:51

joshuali925 approved these changes Jul 30, 2021

View reviewed changes

davidcui1225 merged commit b3cf1d4 into opensearch-project:main Jul 30, 2021

davidcui1225 deleted the connect-notifications-apis branch July 30, 2021 22:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Connect Notifications APIs to Reporting #131

Connect Notifications APIs to Reporting #131

davidcui1225 commented Jul 21, 2021 •

edited

Loading

codecov-commenter commented Jul 21, 2021 •

edited

Loading

joshuali925 left a comment

zhongnansu left a comment •

edited

Loading

zhongnansu Jul 27, 2021

davidcui1225 commented Jul 27, 2021

Connect Notifications APIs to Reporting #131

Connect Notifications APIs to Reporting #131

Conversation

davidcui1225 commented Jul 21, 2021 • edited Loading

Description

Issues Resolved

Check List

codecov-commenter commented Jul 21, 2021 • edited Loading

Codecov Report

joshuali925 left a comment

Choose a reason for hiding this comment

zhongnansu left a comment • edited Loading

Choose a reason for hiding this comment

zhongnansu Jul 27, 2021

Choose a reason for hiding this comment

davidcui1225 commented Jul 27, 2021

davidcui1225 commented Jul 21, 2021 •

edited

Loading

codecov-commenter commented Jul 21, 2021 •

edited

Loading

zhongnansu left a comment •

edited

Loading