eschutho opened a new pull request, #35807:
URL: https://github.com/apache/superset/pull/35807
### SUMMARY
This PR adds the celery task execution ID to the email notification log
message to improve traceability when debugging report and alert executions.
Previously, the log message "Report sent to email, notification content is
%s" did not include the task execution ID, making it difficult to correlate
email notifications with specific celery task executions. This change adds the
task_id to the log output.
**Changes:**
- Added `execution_id` field to `HeaderDataType` TypedDict
- Updated `_get_log_data()` method to include execution_id in the log data
- Modified email notification log to display task_id before notification
content
- Updated all unit tests to include execution_id in expected results
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A - This is a logging enhancement
### TESTING INSTRUCTIONS
1. Set up a report or alert with email notifications
2. Trigger the report/alert execution
3. Check the application logs for the email notification message
4. Verify that the log now includes `task_id: <execution_id>` in the format:
```
Report sent to email, task_id: <uuid>, notification content is {...}
```
5. Run unit tests to verify they pass:
```bash
pytest tests/unit_tests/commands/report/execute_test.py
```
### ADDITIONAL INFORMATION
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in
[SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] 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
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
