(@aws-cdk/aws-dlm): Policy type is not synthesized when using CfnLifecyclePolicy construct

[khushail]

Describe the bug

CX is using the construct CfnLifecyclePolicy with Policy_type with the given code snippet. After synth, the policy_type details are msising from the template.

Synthesized template -

Resources:
  lifecyclepolicy:
    Type: AWS::DLM::LifecyclePolicy
    Properties:
      Description: EC2 Backup Policy
      ExecutionRoleArn: arn:aws:iam::404890032397:role/AWSDataLifecycleManagerDefaultRole
      PolicyDetails:
        ResourceTypes:
          - INSTANCE
        Schedules:
          - CopyTags: true
            CreateRule:
              CronExpression: cron(0 2 * * ? *)
              Interval: 2
              IntervalUnit: HOURS
            Name: DailyBackup
            RetainRule:
              Count: 12
            TagsToAdd:
              - Key: Purpose
                Value: Backup
        TargetTags:
          - Key: altci
            Value: ServiceNowDiscovery
      State: ENABLED

Additionally Cx mentions that they observe life cycle manager PolicyType to "EBS snapshot policy", instead of "EBS-backed AMI policy".

Expected Behavior

Synthesized template should include the policy_type property.

Current Behavior

Property is missing from the template

Reproduction Steps

dlm.CfnLifecyclePolicy(
self,
id="lifecyclepolicy",
description="EC2 Backup Policy",
state='ENABLED',
execution_role_arn='',
policy_details={
'policy_type' : 'IMAGE_MANAGEMENT',
'resourceTypes': ['INSTANCE'],
'targetTags': [{
'key': 'altci',
'value': 'ServiceNowDiscovery'
}],
'schedules': [{
'name': 'DailyBackup',
'createRule': {
'interval': 2,
'intervalUnit': 'HOURS',
'cronExpression': 'cron(0 2 * * ? *)'
},
'retainRule': {
'count':12
},
'copyTags': True,
'tagsToAdd': [{'key': 'Purpose', 'value': 'Backup'}]

                }]
            }
        )

Possible Solution

No response

Additional Information/Context

No response

CDK CLI Version

2.103.1

Framework Version

No response

Node.js Version

v18.15.0

OS

Mac

Language

Python

Language Version

No response

Other information

No response

[msambol]

@khushail I'll take this.

[msambol]

@khushail I was able to get this to deploy by changing policy_type to policyType.

[khushail]

Thanks @msambol , appreciate your efforts. Please feel free to submit the PR.

[msambol]

@khushail I don't think there needs to be one? policy_type was user input, correct?

[khushail]

Yes, that works. Thanks so much @msambol , for pointing this out. We would need to update the Python doc which are automatically generated, so I would be creating an issue for the documentation update and add the required label jsii.

[github-actions]

⚠️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.