fix: update log message for ALERT_REPORTS_NOTIFICATION_DRY_RUN

[nytai]

SUMMARY

Let's make it clearer that dry run is enabled. Many people interpret the current message to mean that the notification was sent.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #23604 (f0eff23) into master (c5eecc7) will decrease coverage by 0.03%.
The diff coverage is 60.89%.

❗ Current head f0eff23 differs from pull request most recent head f41e26d. Consider uploading reports for the commit f41e26d to get more accurate results

@@            Coverage Diff             @@
##           master   #23604      +/-   ##
==========================================
- Coverage   67.72%   67.70%   -0.03%     
==========================================
  Files        1916     1918       +2     
  Lines       74051    74133      +82     
  Branches     8040     8052      +12     
==========================================
+ Hits        50153    50193      +40     
- Misses      21850    21887      +37     
- Partials     2048     2053       +5     

Flag	Coverage Δ
hive	52.73% <38.27%> (-0.02%)	⬇️
mysql	78.51% <83.95%> (+0.02%)	⬆️
postgres	78.59% <83.95%> (+0.02%)	⬆️
presto	52.66% <38.27%> (-0.02%)	⬇️
python	82.43% <83.95%> (+0.01%)	⬆️
sqlite	77.10% <83.95%> (+0.02%)	⬆️
unit	52.63% <43.20%> (-0.02%)	⬇️

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

Impacted Files	Coverage Δ
.../plugin-chart-echarts/src/Funnel/transformProps.ts	72.34% <ø> (ø)
...s/plugin-chart-echarts/src/Graph/transformProps.ts	72.36% <ø> (ø)
...hart-echarts/src/MixedTimeseries/transformProps.ts	0.00% <ø> (ø)
...ins/plugin-chart-echarts/src/Pie/transformProps.ts	55.71% <ø> (ø)
...s/plugin-chart-echarts/src/Radar/transformProps.ts	0.00% <ø> (ø)
...gin-chart-echarts/src/Timeseries/transformProps.ts	58.33% <ø> (ø)
...d/plugins/plugin-chart-echarts/src/utils/series.ts	88.48% <ø> (ø)
...rc/components/Datasource/ChangeDatasourceModal.tsx	78.68% <ø> (ø)
...rset-frontend/src/components/ListView/ListView.tsx	93.75% <ø> (ø)
...src/dashboard/components/PropertiesModal/index.tsx	64.03% <ø> (ø)
... and 109 more

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[villebro]

Would this not produce True is enabled, set it to False to send notifications., as that's the value of the config param, not it's name?

[nytai]

Yes, I thought I pushed a fix for that 😞

[nytai]

... I pushed to the wrong remote. Fixed