You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ru...@apache.org on 2023/04/12 15:30:37 UTC

[superset] branch master updated: fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN (#23604)

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

rusackas pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 350723bff8 fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN (#23604)
350723bff8 is described below

commit 350723bff899fe1c560826a3c7cbb3bad687911f
Author: ʈᵃᵢ <td...@gmail.com>
AuthorDate: Wed Apr 12 08:30:28 2023 -0700

    fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN (#23604)
---
 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..78ac8aa3a8 100644
--- a/superset/reports/commands/execute.py
+++ b/superset/reports/commands/execute.py
@@ -408,7 +408,9 @@ 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. "
+                        "ALERT_REPORTS_NOTIFICATION_DRY_RUN is enabled, "
+                        "set it to False to send notifications.",
                         self._report_schedule.name,
                         recipient.recipient_config_json,
                     )