@aws-cdk/ecs: Override container name
See original GitHub issueAllow to override container name by constructor property.
Use Case
With the current implementation, these will be the same value implicitly. This is confusing for our team because the naming conventions for the names in our current(exists) configuration and the Construct IDs are different. So we need to manage the Construct ID and the name separately.
Proposed Solution
Add containerName
property to ContainerDefinitionProps
and override containerName
by this.
this.containerName = props.containerName ?? this.node.id;
Other
- 👋 I may be able to implement this feature request
- ⚠️ This feature might incur a breaking change
This is a 🚀 Feature Request
Issue Analytics
- State:
- Created 3 years ago
- Reactions:2
- Comments:9 (8 by maintainers)
Top Results From Across the Web
interface ContainerOverride · AWS CDK
A list of container overrides that specify the name of a container and the overrides it should receive. Example. // The code below...
Read more >@aws-cdk/aws-ecs-patterns - npm
Start using @aws-cdk/aws-ecs-patterns in your project by running `npm ... but you can override by providing a value for the platformVersion ...
Read more >In CDK step functions, how to pass the commands or ...
Other things to point out when trigger fargate/ecs from step function, ... import targets = require('@aws-cdk/aws-events-targets'); const ...
Read more >Amazon Elastic Container Service (ECS) / Fargate
The Jenkins Amazon EC2 Container Service plugin will use this ECS cluster and ... override the Jenkins URL in the Advanced Configuration of...
Read more >Use the Override ECS Docker Command to Run Database ...
Using overrides allows you to change the default command for the container specified in your task definitions. For example, instead of running ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
This error is already fixed. I don’t think this is a feature and thus changed the title of the PR.
@karupanerura looking forward to your feedback. I just copy and paste your code in the description.