[Improvement-17843][Master] Add IT case for task timeout alert

[shrihari7396]

Purpose

Closes #17843

Adds integration test cases to verify task timeout alert behavior during workflow execution in the master module.

---

Changes

• Added integration test cases:

  • testStartWorkflow_withTimeoutWarnTask
  • testStartWorkflow_withTimeoutWarnFailedTask

• Added YAML workflow definitions for timeout scenarios

• Added repository method to query alerts from the database

• Updated master module to support timeout alert verification in tests

---

Behavior

Before:

• No integration tests to verify timeout alert behavior
• Timeout alerts were not automatically validated

After:

• Integration tests validate timeout alert generation
• Alerts are verified to be correctly generated and persisted in the database

---

Testing

• Ran integration tests:

  • WorkflowStartTestCase#testStartWorkflow_withTimeoutWarnTask
  • WorkflowStartTestCase#testStartWorkflow_withTimeoutWarnFailedTask

• Verified:

  • Workflow execution with timeout configurations
  • Task and workflow states
  • Alert generation and persistence in the database

[boring-cyborg]

Thanks for opening this pull request! Please check out our contributing guidelines. (https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md)

[shrihari7396]

Hi @SbloodyS,

Thank you for the review. This is my first contribution to this project, so I really appreciate your guidance.
I will fix the requested changes and update the PR shortly.

Thanks!

[shrihari7396]

Hi @SbloodyS,

The milestone-label-check is currently failing because no milestone and valid label are assigned to this PR.

Could you please help assign the appropriate milestone and label so the check can pass?

Thank you!

[SbloodyS]

Hi @SbloodyS,

The milestone-label-check is currently failing because no milestone and valid label are assigned to this PR.

Could you please help assign the appropriate milestone and label so the check can pass?

Thank you!

Milestones are required before the merger, and contributors do not need to pay attention to them. You should check failed IT. It did not passed.

[shrihari7396]

Hi @SbloodyS,

I would like to work on adding the integration test case for task timeout alert in the master module.

While preparing the environment, my IDE formatter modified some files unintentionally. I will revert any unrelated formatting changes and ensure only the relevant test case changes are included in the PR.

I will first analyze the current timeout handling and alert triggering flow to ensure the IT covers the correct execution path (timeout detection → task state update → alert creation).

Please let me know if there are any specific scenarios or timeout strategies (e.g., WARN vs FAILED) that you would like the test to cover.

Thanks!

[shrihari7396]

Hi @SbloodyS,

I have applied spotless formatting and updated the PR accordingly.

Thanks.

[shrihari7396]

Thanks for the review. I have reverted the unnecessary changes and updated the PR accordingly.

I also ensured the formatting issues are fixed.

Please let me know if any further modifications are needed.

[shrihari7396]

Hi @SbloodyS,

Just a gentle follow-up on this PR. I have addressed the requested changes.

Could you please take another look when you get time?

Thank you!

[ruanwenjun]

You should do assertion of the alert event in DB.

[shrihari7396]

Hi @ruanwenjun @SbloodyS,

Just a gentle follow-up regarding the milestone label check. Since all CI tests are passing on my side, could you please confirm if a milestone needs to be added or if anything else is required from me?

Thank you!

[shrihari7396]

Hi @ruanwenjun @SbloodyS,

Gentle ping on this PR. Could you please review when you have time?

Please let me know if any further changes are needed from my side.

Thanks!

[shrihari7396]

Hi @ruanwenjun @SbloodyS,

I’ve addressed the requested changes and updated the PR.
Could you please take another look when you have time?

Thanks!

[shrihari7396]

Hi @ruanwenjun, @SbloodyS,
I have updated the branch. Please let me know if any changes are needed.

[SbloodyS]

Please resolve conflicts. @shrihari7396

[shrihari7396]

Hi @ruanwenjun, @SbloodyS, I have resolved the conflicts during the branch merge. Please review and let me know if any changes are needed.

[shrihari7396]

Hi @ruanwenjun, @SbloodyS,
I have updated the branch. Please let me know if any changes are needed.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[shrihari7396]

@ruanwenjun @SbloodyS
All PR checks have completed successfully. The only remaining one is the milestone-labeled check. Could you please take a look or let me know if anything is required from my side?

[shrihari7396]

Hi @ruanwenjun, @SbloodyS,
I have updated the branch. Please let me know if any changes are needed.