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.

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

aws-scheduler-alpha: Schedule Group is ignored #27885

Closed
tvdijk opened this issue Nov 8, 2023 · 3 comments · Fixed by #27927
Closed

aws-scheduler-alpha: Schedule Group is ignored #27885

tvdijk opened this issue Nov 8, 2023 · 3 comments · Fixed by #27927
Labels
@aws-cdk/aws-scheduler Related to the AWS Scheduler service bug This issue is a bug. effort/medium Medium work item – several days of effort p2

Comments

@tvdijk
Copy link

tvdijk commented Nov 8, 2023

Describe the bug

When I set a Schedule Group on a Schedule, the setting is ignored, and my Schedule ends up in the default group.

Expected Behavior

I expect my Schedule to be associated with the Schedule Group I created and specified.

Current Behavior

The setting is ignored, and my Schedule ends up in the default group. The template does not have a GroupName attribute set on the AWS::Scheduler::Schedule resource.

Reproduction Steps

        final IGroup scheduleGroup = new Group(this, "ScheduleGroup", GroupProps.builder()
                .build());
        final ISchedule schedule = new Schedule(this, "Schedule", ScheduleProps.builder()
                .group(scheduleGroup)
                .key(appKey)
                .schedule(ScheduleExpression.rate(Duration.minutes(10)))
                .target(LambdaInvoke.Builder.create(function)
                                .retryAttempts(0)
                                .input(ScheduleTargetInput.fromText("{}"))
                                .build())
                .build());

Possible Solution

No response

Additional Information/Context

No response

CDK CLI Version

2.104.0 (build 3b99abe)

Framework Version

No response

Node.js Version

npm@9.5.1

OS

Windows 10

Language

Java

Language Version

Java 11

Other information

No response

@tvdijk tvdijk added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Nov 8, 2023
@github-actions github-actions bot added the @aws-cdk/aws-scheduler Related to the AWS Scheduler service label Nov 8, 2023
@msambol
Copy link
Contributor

msambol commented Nov 9, 2023

I'll take this.

@msambol
Copy link
Contributor

msambol commented Nov 9, 2023

@tvdijk I was able to reproduce this. As a workaround for now, when you're creating the group, you can specify groupName (instead of defaulting to the autogenerated name) and the schedule will be added to the group. I'll discuss with the team on how to fix this issue.

@pahud pahud added p2 effort/medium Medium work item – several days of effort and removed needs-triage This issue or PR still needs to be triaged. labels Nov 10, 2023
mergify bot added a commit to msambol/aws-cdk that referenced this issue Nov 20, 2023
@mergify mergify bot closed this as completed in #27927 Nov 20, 2023
mergify bot pushed a commit that referenced this issue Nov 20, 2023
…7927)

Closes #27885.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-scheduler Related to the AWS Scheduler service bug This issue is a bug. effort/medium Medium work item – several days of effort p2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants