ec2: setting placement group name does not work

[NdemeraS]

Describe the bug

Unable to manually set/change the name of a placement group using with the L1 or L2 construct

Expected Behavior

As the the feature is documented, the expectation is to able to manually set a placement group name

Current Behavior

The CDK will auto-generate a name for the placement group regardless if you is manually set

Reproduction Steps

import { Duration, Stack, StackProps } from 'aws-cdk-lib';
import * as ec2 from 'aws-cdk-lib/aws-ec2';
import { Construct } from 'constructs';

export class PlacementStack extends Stack {
  constructor(scope: Construct, id: string, props?: StackProps) {
    super(scope, id, props);

    const placem = new ec2.PlacementGroup(this, 'MyPlacementGroup', /* all optional props */ {
    partitions: 1,
    placementGroupName: 'Testing2341413',
//    spreadLevel: ec2.PlacementGroupSpreadLevel.HOST,
    strategy: ec2.PlacementGroupStrategy.PARTITION,
    });
  }
}

Possible Solution

No response

Additional Information/Context

The underlying CFN placement group resource has no support for the placement group name property. Since the CDK is a wrapper for the CFN, there may be some back end integration that is not working as expected

CDK CLI Version

2.108.1 (build 2320255)

Framework Version

No response

Node.js Version

v18.17.1

OS

n/a

Language

TypeScript

Language Version

No response

Other information

No response

[pahud]

Yes. As you mentioned, the placegroup name is not a property in its CFN resource and if we look at the constructor from CDK source code, it does not really pass to the L1 resource.

aws-cdk/packages/aws-cdk-lib/aws-ec2/lib/placement-group.ts

Lines 194 to 204 in b8b6baf

	const resource = new CfnPlacementGroup(this, 'Resource', {
	partitionCount: this.partitions,
	spreadLevel: this.spreadLevel,
	strategy: this.strategy,
	});
	this.placementGroupName = this.getResourceArnAttribute(resource.attrGroupName, {
	service: 'batch',
	resource: 'compute-environment',
	resourceName: this.physicalName,
	});

I guess maybe the L2 resource can create a Name tag with the placegroup name as its value but it's very opinionated and requires some discussion.

Are you able to specify the name from console? What will happen when you specify a name for that? Will the console create a Name tag for you? If yes then it makes perfect sense to create a Name tag in the L2 constructor.

[pahud]

OK I can confirm this should be a gap from cloudformation and it's not possible to specify the Name through CFN and CDK cannot do anything with that.

Please create a feature request in cloudformation-coverage-roadmap github repo and allow the community to vote for more visibility. After CFN supports that, CDK would follow up and support it very soon.