fix(ecs-patterns): respect taskDefinition prop in QueueProcessingFargateService.

[alhridoy]

Title: Fix for Issue #27360: taskDefinition prop ignored on QueueProcessingFargateService

Comment:

This pull request addresses the bug reported in Issue #27360. The QueueProcessingFargateService was not correctly handling the taskDefinition prop. If a taskDefinition was provided, it was being ignored and a new one was being created regardless.

Changes:

• Modified the constructor of QueueProcessingFargateService to correctly handle the taskDefinition prop. If a taskDefinition is provided in the props, it is now used. If not, a new FargateTaskDefinition is created.

This change fixes the bug and allows the QueueProcessingFargateService to correctly use a provided taskDefinition.

Please review the changes and provide feedback.

Fixes #27360

[aws-cdk-automation]

The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.

A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.

[alhridoy]

Exemption Request

[vinayak-kukreja]

Hey, I see you have requested for an exemption. Could you tell me why integration tests are not required for this change?

[vinayak-kukreja]

The build is also currently failing. We would not be able to prioritize reviewing this PR if the build if failing.

Let me know if you have any questions.

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
• Commit ID: f496b52
• Result: FAILED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[aws-cdk-automation]

This PR has been in the CHANGES REQUESTED state for 3 weeks, and looks abandoned. To keep this PR from being closed, please continue work on it. If not, it will automatically be closed in a week.

[aws-cdk-automation]

This PR has been deemed to be abandoned, and will be automatically closed. Please create a new PR for these changes if you think this decision has been made in error.

[aws-cdk-automation]

The pull request linter fails with the following errors:

❌ Fixes must contain a change to an integration test file and the resulting snapshot.

PRs must pass status checks before we can provide a meaningful review.

If you would like to request an exemption from the status checks or clarification on feedback, please leave a comment on this PR containing Exemption Request and/or Clarification Request.