fix(helm): add missing task modules to default CeleryConfig imports

[rusackas]

Follow-up to #37259. That PR removed the unused SQLALCHEMY_TRACK_MODIFICATIONS setting from the Helm chart's default Celery/config block. While in that area, I noticed the chart's built-in default CeleryConfig.imports tuple (used when cache.enabled is true and no custom config.celeryConfig is supplied) is missing superset.tasks.slack, even though superset/config.py's own default CELERY_CONFIG.imports includes it alongside scheduler, thumbnails, and cache (which the chart already had).

SUMMARY

Add superset.tasks.slack to the default CeleryConfig.imports tuple rendered by the Helm chart's _helpers.tpl. Without it, Celery workers deployed via this chart's default config never register the Slack task module, so alerts/reports configured to notify Slack silently fail to send.

Also bumps the chart's patch version (0.22.4 -> 0.22.5) per this chart's versioning convention, and regenerates README.md via helm-docs accordingly.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A (Helm template change)

TESTING INSTRUCTIONS

1. Render the chart with cache.enabled: true and no config.celeryConfig override.
2. Confirm the generated superset_config.py's CeleryConfig.imports tuple includes "superset.tasks.slack" alongside the existing entries.
3. helm unittest helm/superset passes locally (105/105 tests).

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

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

[bito-code-review]

Code Review Agent Run #d4c8c4

Actionable Suggestions - 0

Review Details

• Files reviewed - 2 · Commit Range: 62e6660..62e6660

  • helm/superset/Chart.yaml
  • helm/superset/templates/_helpers.tpl
• Files skipped - 1

  • helm/superset/README.md - Reason: Filter setting
• Tools

  • Whispers (Secret Scanner) - ✔︎ Successful
  • Detect-secrets (Secret Scanner) - ✔︎ Successful

---

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

• /review - Manually triggers a full AI review.

• /pause - Pauses automatic reviews on this pull request.

• /resume - Resumes automatic reviews.

• /resolve - Marks all Bito-posted review comments as resolved.

• /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

• Customize agent settings
• Review rules
• General documentation
• FAQ

AI Code Review powered by

[bito-code-review]

Bito Automatic Review Skipped – PR Already Merged

Bito scheduled an automatic review for this pull request, but the review was skipped because this PR was merged before the review could be run.
No action is needed if you didn't intend to review it. To get a review, you can type /review in a comment and save it