(applicationautoscaling): Support timezone on ScalableTarget.scaleOnSchedule

[tl24]

Describe the feature

Cloudformation ScalableTarget ScheduledAction supports a timezone property. The timezone argument is not available when calling ScalableTarget.scaleOnSchedule:
https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_applicationautoscaling.ScalableTarget.html#scalewbronwbrscheduleid-action
https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_applicationautoscaling.ScalingSchedule.html

Use Case

Cron time expressions have to be entered in UTC. For timezones observing Daylight Savings Time you have to adjust the schedule when DST starts and again when it ends to get proper scheduling.

Proposed Solution

Add the timezone property to the ScalingSchedule type that is passed to scaleOnSchedule

Other Information

I am using the python library, so I would have no way to add this to the json object for ScalingSchedule like you could in JS/TS since the autogenerated code turns these into individual arguments to the scaleOnSchedule call.

Acknowledgements

• I may be able to implement this feature request
• This feature might incur a breaking change

CDK version used

2.103.1

Environment details (OS name and version, etc.)

Ubuntu 20.04

[msambol]

I'll take this.

[msambol]

On second thought, I'm going to wait for the CDK team to comment. There are definitely some assumptions made regarding UTC:

aws-cdk/packages/aws-cdk-lib/aws-applicationautoscaling/lib/schedule.ts

Lines 151 to 167 in 9babd7a

	function formatISO(date?: Date) {
	if (!date) { return undefined; }
	return date.getUTCFullYear() +
	'-' + pad(date.getUTCMonth() + 1) +
	'-' + pad(date.getUTCDate()) +
	'T' + pad(date.getUTCHours()) +
	':' + pad(date.getUTCMinutes()) +
	':' + pad(date.getUTCSeconds());
	function pad(num: number) {
	if (num < 10) {
	return '0' + num;
	}
	return num;
	}
	}

[sothawo]

This is an old topic, there was some attempt mentioned in #22645 but up to date still no support for this. Sometimes it seems cdk is the ugly sibling of cfn that noone wants do deal with. Eventbridge seems to have the same issue #21181.

[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.

[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.