Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

❗ NOTICE(aws-events-targets): Hardcoded AWS Partition in ECS task resource ARN #29610

Closed
1somorphism opened this issue Mar 25, 2024 · 4 comments · Fixed by #29633 · May be fixed by NOUIY/aws-solutions-constructs#98, NOUIY/aws-solutions-constructs#99 or NOUIY/aws-solutions-constructs#101
Closed

❗ NOTICE(aws-events-targets): Hardcoded AWS Partition in ECS task resource ARN #29610

1somorphism opened this issue Mar 25, 2024 · 4 comments · Fixed by #29633 · May be fixed by NOUIY/aws-solutions-constructs#98, NOUIY/aws-solutions-constructs#99 or NOUIY/aws-solutions-constructs#101
Assignees
@paulhcsun
Labels
@aws-cdk/aws-events-targets bug This issue is a bug. effort/small Small work item – less than a day of effort management/tracking Issues that track a subject or multiple issues p0

Comments

@1somorphism
Copy link

1somorphism commented Mar 25, 2024 •
edited by paulhcsun
Loading

Please add your +1 👍 to let us know you have encountered this

Status: RESOLVED

Overview:

Partition in https://github.com/aws/aws-cdk/blob/main/packages/aws-cdk-lib/aws-events-targets/lib/ecs-task.ts#L291 is hardcoded to aws so it doesn't work for other special partitions.

Workaround:

Solution:

Update policy to use this.cluster.stack.partition instead of hardcoded aws.

Fixed in v2.135.0.

Expected Behavior

Partition should be added dynamically.

Current Behavior

Fixed to aws

Reproduction Steps

Create an ScheduledEc2Task or a ScheduledFargateTask following the example in the CDK documentation and attempt to deploy outside the AWS partition.

Possible Solution

Partition should be coming from this.cluster.env.partition?

Related Issues:

#29628

Additional Information/Context

No response

CDK CLI Version

2.132.1

Framework Version

No response

Node.js Version

18.18.2

OS

MacOS 14.3.1

Language

TypeScript

Language Version

5.4.2

Other information

No response
@1somorphism 1somorphism added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Mar 25, 2024
@1somorphism 1somorphism changed the title (aws-events-targets): (Hardcoded AWS Partition in ECS task resource ARN) aws-events-targets: Hardcoded AWS Partition in ECS task resource ARN Mar 25, 2024
@tim-finnigan tim-finnigan self-assigned this Mar 25, 2024
@tim-finnigan tim-finnigan added investigating This issue is being investigated and/or work is in progress to resolve the issue. and removed needs-triage This issue or PR still needs to be triaged. labels Mar 25, 2024
@tim-finnigan
Copy link

Thanks for reporting this issue. It looks like that was recently added in #28898, and the partition should be dynamically populated.

@tim-finnigan tim-finnigan added p1 effort/small Small work item – less than a day of effort and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Mar 25, 2024
@tim-finnigan tim-finnigan removed their assignment Mar 25, 2024
@tim-finnigan tim-finnigan added p0 and removed p1 labels Mar 27, 2024
@tim-finnigan tim-finnigan mentioned this issue Mar 27, 2024
Closed
@paulhcsun paulhcsun mentioned this issue Mar 27, 2024
Merged
1 task
@mergify mergify bot closed this as completed in #29633 Mar 27, 2024
mergify bot pushed a commit that referenced this issue Mar 27, 2024
@paulhcsun
fix(events-targets): hardcoded AWS partition in ECS task resource ARN (…
69cff2e 
…#29633)

### Issue # (if applicable)

Closes #29610 

Related to #28898

### Reason for this change

Previously the AWS partition was hardcoded so it would not work for other special partitions.

### Description of changes

Replace hardcoded `aws` partition to be dynamically added with `this.cluster.stack.partition`. 

### Description of how you validated changes

Ran integ tests suite, no changes. 

### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@github-actions GitHub Actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

@tim-finnigan
Copy link

#29633 has been merged to fix this and should be available soon in an upcoming release.

This was referenced Apr 1, 2024
Closed
Closed
Closed
@paulhcsun paulhcsun added the management/tracking Issues that track a subject or multiple issues label Apr 3, 2024
@paulhcsun paulhcsun pinned this issue Apr 3, 2024
@paulhcsun
Copy link
Contributor

Fix has been released in v2.135.0.

@paulhcsun paulhcsun mentioned this issue Apr 3, 2024
Merged
mergify bot pushed a commit to cdklabs/aws-cdk-notices that referenced this issue Apr 3, 2024
@paulhcsun
chore: notice for aws/aws-cdk#29610 (#461)
e12c58e 
CDK CLI notice for aws/aws-cdk#29610
paulhcsun added a commit to cdklabs/aws-cdk-notices that referenced this issue Apr 3, 2024
@paulhcsun
chore: fix typo for notice for aws/aws-cdk#29610
996d6be
paulhcsun added a commit to cdklabs/aws-cdk-notices that referenced this issue Apr 3, 2024
@paulhcsun
chore: fix typo for notice for aws/aws-cdk#29610
e693b97
@paulhcsun paulhcsun mentioned this issue Apr 3, 2024
Merged
paulhcsun added a commit to cdklabs/aws-cdk-notices that referenced this issue Apr 3, 2024
@paulhcsun
Merge pull request #462 from cdklabs/notice
f418b85 
chore: fix typo for notice for aws/aws-cdk#29610
@paulhcsun paulhcsun changed the title aws-events-targets: Hardcoded AWS Partition in ECS task resource ARN ❗ NOTICE(aws-events-targets): Hardcoded AWS Partition in ECS task resource ARN Apr 9, 2024
@TheRealAmazonKendra TheRealAmazonKendra unpinned this issue Apr 22, 2024
This was referenced May 23, 2024
Open
Open
@NOUIY NOUIY mentioned this issue May 30, 2024
Open
@aws aws locked as resolved and limited conversation to collaborators Jul 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@paulhcsun paulhcsun

Labels
@aws-cdk/aws-events-targets bug This issue is a bug. effort/small Small work item – less than a day of effort management/tracking Issues that track a subject or multiple issues p0
Projects
None yet
Milestone
No milestone
3 participants
@1somorphism @paulhcsun @tim-finnigan