You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ta...@apache.org on 2023/04/06 05:18:51 UTC

[superset] branch tai/fix-report-dry-run created (now 532a5f87bb)

This is an automated email from the ASF dual-hosted git repository.

tai pushed a change to branch tai/fix-report-dry-run
in repository https://gitbox.apache.org/repos/asf/superset.git


      at 532a5f87bb fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN

This branch includes the following new commits:

     new 532a5f87bb fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



[superset] 01/01: fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN

Posted by ta...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

tai pushed a commit to branch tai/fix-report-dry-run
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 532a5f87bb2daa7d248458d227f648ee6fec172a
Author: Tai Dupree <td...@gmail.com>
AuthorDate: Wed Apr 5 22:18:05 2023 -0700

    fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN
---
 superset/reports/commands/execute.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/superset/reports/commands/execute.py b/superset/reports/commands/execute.py
index d13d4178e9..f2d9f7b31c 100644
--- a/superset/reports/commands/execute.py
+++ b/superset/reports/commands/execute.py
@@ -408,9 +408,11 @@ class BaseReportState:
             try:
                 if app.config["ALERT_REPORTS_NOTIFICATION_DRY_RUN"]:
                     logger.info(
-                        "Would send notification for alert %s, to %s",
+                        "Would send notification for alert %s, to %s. "
+                        "%s is enabled, set it to False to send notifications.",
                         self._report_schedule.name,
                         recipient.recipient_config_json,
+                        app.config["ALERT_REPORTS_NOTIFICATION_DRY_RUN"],
                     )
                 else:
                     notification.send()